Upgrade @objectstack dependencies to 0.9.1#370
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR attempts to upgrade all @objectstack dependencies from version 0.9.0 to 0.9.1. According to the PR description, the upgrade includes enhanced plugin schema with additional capabilities while maintaining backward compatibility.
Changes:
- Updated @objectstack dependencies to 0.9.1 in root package.json and core packages
- Updated pnpm-lock.yaml to reflect new dependency versions
- Upgraded packages: @objectstack/cli, @objectstack/core, @objectstack/driver-memory, @objectstack/objectql, @objectstack/plugin-msw, @objectstack/runtime, @objectstack/plugin-hono-server, @objectstack/spec, @objectstack/client
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated @objectstack devDependencies and dependencies from ^0.9.0 to ^0.9.1 |
| packages/types/package.json | Updated @objectstack/spec dependency to ^0.9.1 |
| packages/core/package.json | Updated @objectstack/spec dependency to ^0.9.1 |
| packages/react/package.json | Updated @objectstack/spec dependency to ^0.9.1 |
| packages/data-objectstack/package.json | Updated @objectstack/client dependency to ^0.9.1 |
| pnpm-lock.yaml | Reflects all dependency resolution changes including new 0.9.1 package definitions and snapshots |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
| "@objectstack/cli": "^0.9.1", | ||
| "@objectstack/core": "^0.9.1", | ||
| "@objectstack/driver-memory": "^0.9.1", | ||
| "@objectstack/objectql": "^0.9.1", | ||
| "@objectstack/plugin-msw": "^0.9.1", | ||
| "@objectstack/runtime": "^0.9.1", |
There was a problem hiding this comment.
The upgrade to @objectstack 0.9.1 is incomplete. While the root package.json and several packages were updated, the following package.json files still reference version 0.9.0 and need to be updated:
- apps/console/package.json (lines 39-44)
- examples/crm/package.json (lines 18-20, 24)
- examples/kitchen-sink/package.json (lines 16, 19)
- examples/msw-todo/package.json (lines 14-19)
- examples/todo/package.json (lines 14-15, 18)
This inconsistency could lead to version conflicts and unexpected behavior when running these applications or examples. All @objectstack dependencies across the entire monorepo should be upgraded to 0.9.1 to ensure consistency.
| "@objectstack/cli": "^0.9.1", | ||
| "@objectstack/core": "^0.9.1", | ||
| "@objectstack/driver-memory": "^0.9.1", | ||
| "@objectstack/objectql": "^0.9.1", | ||
| "@objectstack/plugin-msw": "^0.9.1", | ||
| "@objectstack/runtime": "^0.9.1", |
There was a problem hiding this comment.
According to Coding Guideline Rule #2 (Documentation Driven Development), any feature implemented or refactored must update the corresponding documentation. This dependency upgrade should be documented in CHANGELOG.md under the [Unreleased] section. The CHANGELOG should note the upgrade from @objectstack 0.9.0 to 0.9.1 and any relevant changes or improvements that come with it.
📦 Bundle Size Report
Size Limits
|
|
✅ All checks passed!
|
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
| checked={!!value} | ||
| onCheckedChange={(checked) => onChange(!!checked)} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| checked={!!value} | ||
| onCheckedChange={onChange} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| value={value || ''} | ||
| onChange={(e) => onChange(e.target.value)} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| value={value || ''} | ||
| onChange={(e) => onChange(e.target.value)} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| onChange={(e) => onChange(e.target.value)} | ||
| placeholder={config?.placeholder || 'email@example.com'} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| onChange={(e) => onChange(e.target.value)} | ||
| placeholder={textareaField?.placeholder} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| onChange={(e) => onChange(e.target.value)} | ||
| placeholder={fieldData?.placeholder} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| onChange={(e) => onChange(e.target.value)} | ||
| placeholder={fieldData?.placeholder} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| value={value || ''} | ||
| onChange={(e) => onChange(e.target.value)} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
| onChange={(e) => onChange(e.target.value)} | ||
| placeholder={config?.placeholder || 'https://example.com'} | ||
| disabled={readonly || props.disabled} | ||
| disabled={readonly || domProps.disabled} |
Check warning
Code scanning / CodeQL
Useless conditional Warning
📦 Bundle Size Report
Size Limits
|
|
✅ All checks passed!
|
…pgrade Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
📦 Bundle Size Report
Size Limits
|
|
✅ All checks passed!
|
- Add findOne method to MockDataSource - Increase timeout for form field loading to 15 seconds - Wrap each field label check in waitFor to handle async rendering - Fixes "expected 0 to be greater than 0" assertion error for Date field Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
- Reduce initial timeout from 15s to 10s - Use Promise.all to check all field labels concurrently instead of sequentially - Improves test execution time while maintaining reliability Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
- Add expectLabelToExist helper function to reduce code duplication - Improves readability and maintainability - All tests still passing Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
…rror-another-one
📦 Bundle Size Report
Size Limits
|
|
✅ All checks passed!
|
Upgrades @objectstack kernel and related packages from 0.9.0 to 0.9.1 for latest protocol support.
Changes
Dependencies updated across 5 package.json files:
@objectstack/{cli,core,runtime,objectql}→ 0.9.1@objectstack/{driver-memory,plugin-msw,plugin-hono-server}→ 0.9.1@objectstack/{spec,client}→ 0.9.1Notes
Version 0.9.1 adds enhanced plugin schema with additional
capabilitiesandcontributesconfiguration options. All changes are backward compatible—no code modifications required.Updated lockfile reflects mixed 0.9.0/0.9.1 dependencies due to workspace resolution.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.