feat: Add Fabric support with new tree node providers and experiences#2921
Open
bk201- wants to merge 32 commits into
Open
feat: Add Fabric support with new tree node providers and experiences#2921bk201- wants to merge 32 commits into
bk201- wants to merge 32 commits into
Conversation
# Conflicts: # package-lock.json # package.json # src/constants.ts # src/extension.ts
# Conflicts: # package-lock.json
# Conflicts: # src/extension.ts
# Conflicts: # l10n/bundle.l10n.json # package-lock.json # package.json # src/extension.ts
…ricTreeNodeProvider
…ent getArtifact method
…ent getArtifact method
mkrueger
previously approved these changes
Apr 14, 2026
# Conflicts: # l10n/bundle.l10n.json # src/constants.ts # src/extension.ts
# Conflicts: # l10n/bundle.l10n.json # package-lock.json # src/commands/createContainer/CosmosDBExecuteStep.ts # src/commands/deleteContainer/deleteContainer.ts # src/commands/deleteItems/deleteItems.ts # src/commands/filterTreeItems/filterTreeItems.ts # src/commands/openNoSqlQueryEditor/openNoSqlQueryEditor.ts # src/commands/sortTreeItems/sortTreeItems.ts # src/constants.ts # src/extension.ts # src/extensionVariables.ts
…-role Workaround for oxc-project/oxc#19395 (fixed-size allocator pool memory panic on Windows): - Bump oxlint ~1.51.0 -> ~1.66.0 and oxlint-tsgolint ~0.22.1 -> ~0.23.0 - Cap oxlint parallelism with --threads=4 in lint and lint-fix scripts to keep total reserved virtual address space within Windows commit limits Fix accessibility errors surfaced by the upgraded ruleset: - MigrationAssistant: replace 5 occurrences of role=status (Text/div) with the native <output> element which has implicit role=status and aria-live=polite - MigrationAssistant: disable prefer-tag-over-role on the composite SVG icon <span role=img> (native <img> cannot wrap a Fluent SVG component) - GenerateQueryInput: disable prefer-tag-over-role on the like/dislike rating <div role=group> (fieldset would impose default border/padding/margin and break the inline flex layout)
Resolves residual diffs after merging main into the supporting-fabric branch. No behavior changes intended; touches imports, types, and minor refactors carried over from main. Validation: npm run l10n, npm run prettier-fix, npm run lint (oxlint + eslint) all pass with 0 errors.
Contributor
🎉 Build Summary🔗 Source
📦 Package Information
🧪 Test Results
✅ Build StatusAll checks completed successfully! |
# Conflicts: # package-lock.json # package.json # src/tree/cosmosdb/AccountInfo.ts # src/webviews/cosmosdb/Migration/state/MigrationChannel.ts # src/webviews/cosmosdb/QueryEditor/QueryPanel/GenerateQueryInput.tsx
languy
previously approved these changes
Jun 10, 2026
| @@ -0,0 +1,11 @@ | |||
| /*--------------------------------------------------------------------------------------------- | |||
Contributor
There was a problem hiding this comment.
nit: is it really necessary to have a single file for each model? It doesn't matter much, since everything will be bundled, but it's a bit easier to read code if everything is in one file.
# Conflicts: # l10n/bundle.l10n.json
Contributor
🔬 NoSQL language-service integrationCommit: 9537ac9 🧪 Result
📥 Artifacts (run)
|
Contributor
🎭 E2E Tests (Playwright + VS Code)Commit: 9537ac9 🧪 Result
📥 Artifacts (run)
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces improvements for Cosmos DB integration and workspace management, including enhanced error handling, expanded localization, and support for new workspace views. It also updates dependencies and refines command activation conditions for better compatibility with the
vscode-fabric.view.workspace. The changes are primarily focused on user experience, workspace integration, and dependency maintenance.Workspace and Cosmos DB Integration:
vscode-fabric.view.workspaceview inpackage.json, enabling Cosmos DB commands and features to be activated in this new workspace context. [1] [2] [3]vscode-fabric.view, ensuring Cosmos DB actions are available in both Azure and Fabric workspace views. [1] [2]Localization and Error Handling Improvements:
l10n/bundle.l10n.json. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]Dependency Updates:
package.json, includingeslint-plugin-jest,jest-mock-vscode,typescript-eslint, and added@microsoft/vscode-fabric-apifor Fabric workspace support. [1] [2] [3] [4]Codebase Cleanup:
activatefunction inmain.tsby removing performance stats tracking and adjusting the return type to allow forvoid, streamlining extension activation. [1] [2]Dependency Overrides:
@prantlf/jsonlintto use a specific version ofajv, ensuring compatibility and stability.