Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 26, 2025

  • Review vscode-cosmosdb PR #2760 for complete dependency updates
  • Update additional dev dependencies missed in initial review
  • Update runtime dependencies (MongoDB v7, BSON v7, Azure SDKs)
  • Migrate SlickGrid from v5.14.1 to v9.9.0 (breaking changes)
  • Fix SlickGrid v9 prop name changes in DataViewPanelTable and DataViewPanelTree
  • Update glob to v12.0.0 and add dompurify security override
  • Update MIGRATION-SUMMARY.md with SlickGrid v9 migration details
  • Update package-lock.json with npm install
  • All builds, linting, and tests pass (147 tests)
  • Document SlickGrid future deprecations for v10 awareness
Original prompt

This section details on the original issue you should resolve

<issue_title>chore: update dependencies and move to react 19</issue_title>
<issue_description>We've got time to update dependencies and to move to React 19.

Our partner extension has done this job and the related PR can be reviewed here:
microsoft/vscode-cosmosdb#2760

The task is to apply the changes made there in our codebase.
Focus on configuration changes, on package.json changes, and the required changes in the code base.
Not all changes made in sources in the vscode-cosmosdb project are needed as our feature implementations differ, especially around webviews. Focus on concepts, not a verbatim copy.

Leave a summary of concepts applied in our code base and concepts left or future consideration.</issue_description>

<agent_instructions>don't use 'npm run compile', use 'npm run build' instead.
use 'npm run jesttest' to run tests.

finally, make sure 'npm run lint' completes with no errors.
'npm run l10n' will update bundle dictionaries, don't attempt to create them, just regenerate with the l10n tool via 'npm run l10'.

'npm run prettier-fix' will ensure files are formatted correctly to pass 'npm run lint' tests.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update dependencies and migrate to React 19 chore: migrate to React 19 and update dependencies Nov 26, 2025
Copilot AI requested a review from tnaum-ms November 26, 2025 13:13
Copilot finished work on behalf of tnaum-ms November 26, 2025 13:13
@tnaum-ms
Copy link
Collaborator

@copilot you were previously unable to access https://api.github.com/repos/microsoft/vscode-cosmosdb/pulls/2760

The firewall settings have been modified. Review the referenced PR as instructed in the original instructions - the PR might contain insights which are hard to discover.
Read original instructions from the original issue: #406

- Update additional dev dependencies (@eslint/js, @swc/core, jest, mocha, sass, etc.)
- Update runtime dependencies (bson v7, mongodb v7, Azure SDKs, trpc, etc.)
- Migrate SlickGrid from v5 to v9 (ESM-only, new prop names)
- Update glob to v12 and add dompurify override for security
- Fix SlickGrid v9 breaking changes (gridOptions→options, columnDefinitions→columns)
- Update MIGRATION-SUMMARY.md with complete details including SlickGrid v9 migration

Co-authored-by: tnaum-ms <[email protected]>
Copy link
Contributor Author

Copilot AI commented Nov 26, 2025

@copilot you were previously unable to access https://api.github.com/repos/microsoft/vscode-cosmosdb/pulls/2760

The firewall settings have been modified. Review the referenced PR as instructed in t...

I've now reviewed the complete vscode-cosmosdb PR #2760 and applied the additional changes that were missing. The key updates include:

Additional Dependencies Updated:

  • Dev tools: @eslint/js 9.39.1, @swc/core 1.15.1, jest 30.2.0, glob 12.0.0, and many others
  • Runtime: MongoDB v7, BSON v7, Azure SDKs, tRPC 11.7.1, monaco-editor 0.54.0
  • Security: Added dompurify 3.3.0 override

SlickGrid v5→v9 Migration (Major):

  • Updated prop names: gridOptionsoptions, columnDefinitionscolumns
  • Fixed in DataViewPanelTable.tsx and DataViewPanelTree.tsx
  • ESM-only builds (CommonJS dropped)
  • Documented future v10 deprecations for awareness

All builds, tests, and linting pass. See MIGRATION-SUMMARY.md for complete technical details including the SlickGrid migration guide reference.

Commit: 0d00d84

Copilot finished work on behalf of tnaum-ms November 26, 2025 14:31
@tnaum-ms
Copy link
Collaborator

@copilot Please update the package-lock.json file with npm install.

Copy link
Contributor Author

Copilot AI commented Nov 26, 2025

@copilot Please update the package-lock.json file with npm install.

Updated package-lock.json with npm install. The lock file now has the latest dependency resolutions.

Commit: d2d5e94

Copilot finished work on behalf of tnaum-ms November 26, 2025 15:59
@tnaum-ms tnaum-ms force-pushed the copilot/update-dependencies-react-19 branch from 07d7db5 to 2297627 Compare November 27, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: update dependencies and move to react 19

2 participants