Release 0.7.3

[tnaum-ms]

This pull request delivers the 0.7.3 release for the DocumentDB for VS Code extension. It introduces a user-facing feature to cancel in-progress cluster connections, upgrades the extension’s runtime and dependencies to Node.js 22 and the latest package versions, and refines internal telemetry for improved diagnostics. Additionally, it updates CI/CD pipeline templates and enhances the ESLint configuration for React and TypeScript codebases.

Key changes:

New Features:

• Users can now cancel ongoing cluster connection attempts via the progress notification, improving usability when connections hang or take too long. [1] [2] [3] [4]

Dependency and Platform Upgrades:

• Upgrades Node.js requirement from v20 to v22 across the project (.nvmrc, package.json, and documentation). [1] [2] [3] [4]
• Updates multiple dependencies and devDependencies to their latest versions, including core libraries, ESLint plugins, React, and FluentUI. [1] [2] [3] [4] [5] [6]

Developer Experience and Tooling:

• Refines ESLint configuration to better support TypeScript, React, and accessibility linting, and aligns rules for both unit (Jest) and integration (Mocha) test files. [1] [2] [3] [4] [5] [6]
• Updates documentation and changelogs to reflect the new release and its features. [1] [2] [3]

CI/CD Pipeline Improvements:

• Adds placeholder Azure DevOps YAML pipelines for build and release, enabling configuration validation and future CI/CD automation. [1] [2]

Localization:

• Adds and updates localized strings to support new connection and cancellation messages. [1] [2]

[Copilot]

Pull request overview

This PR prepares the DocumentDB for VS Code extension v0.7.3 release by adding cancellable cluster connections, upgrading the runtime/tooling baseline to Node.js 22, and refining telemetry + developer tooling (ESLint/webpack/CI docs & templates).

Changes:

• Add cancellable connection UX (progress notification cancel) by plumbing AbortSignal through ClustersClient connection creation and emitting connection correlation telemetry.
• Upgrade Node/VS Code engine requirements and refresh dependencies; update webpack builds with filesystem caching and refine ESLint flat config for TSX/React/a11y.
• Expand telemetry measurements/properties (connection/discovery/query insights) and update docs/changelog + add placeholder ADO pipeline YAMLs.

Reviewed changes

Copilot reviewed 50 out of 51 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
webpack.config.views.js	Enable filesystem build cache for webview bundles.
webpack.config.ext.js	Enable filesystem build cache for extension bundle.
src/webviews/WebviewContext.tsx	Convert React import to type-only.
src/webviews/theme/state/ThemeContext.tsx	Rename theme getter; use functional state updates; add hook lint suppression.
src/webviews/index.tsx	Convert React import to type-only.
src/webviews/documentdb/documentView/documentView.tsx	Refactor initial editor content + editor refs/layout helpers.
src/webviews/documentdb/collectionView/components/toolbar/ToolbarDividerTransparent.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/resultsTab/DataViewPanelTable.tsx	Add eslint suppression for ref usage with Slickgrid.
src/webviews/documentdb/collectionView/components/resultsTab/DataViewPanelJSON.tsx	Reorder resize handler definition (no behavioral change intended).
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/summaryCard/SummaryCard.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/summaryCard/GenericCell.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/summaryCard/custom/PerformanceRatingCell.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/summaryCard/CellBase.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/QuickActions.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/queryPlanSummary/StageDetailCard.tsx	Change React import form (now type-only).
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/optimizationCards/custom/GetPerformanceInsightsCard.tsx	Change React import form (now type-only).
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/metricsRow/TimeMetric.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/metricsRow/RatioMetric.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/metricsRow/MetricsRow.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/metricsRow/MetricBase.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/metricsRow/GenericMetric.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/metricsRow/CountMetric.tsx	Convert React import to type-only.
src/webviews/documentdb/collectionView/components/queryInsightsTab/components/animatedCardList/AnimatedCardList.tsx	Use functional state updates to avoid stale closures when diffing items.
src/webviews/documentdb/collectionView/collectionViewRouter.ts	Add telemetry breakdown counts for recommendation actions.
src/webviews/documentdb/collectionView/CollectionView.tsx	Minor lint suppressions + HTML escaping in UI error fallback.
src/webviews/components/MonacoAutoHeight.tsx	Reorder helper definitions; preserve existing resize/tab handling logic.
src/webviews/api/webview-client/accessibility/Announcer.tsx	Improve re-announcement behavior for identical messages + timer cleanup.
src/tree/documentdb/ClusterItemBase.ts	Add cancellable connection progress wrapper + cancellation telemetry/logging.
src/tree/connections-view/DocumentDBClusterItem.ts	Add connection source/type telemetry + cancellable connect path + correlation ID.
src/tree/azure-resources-view/mongo-ru/RUCoreResourceItem.ts	Use cancellable connect path + connection telemetry fields.
src/tree/azure-resources-view/documentdb/VCoreResourceItem.ts	Use cancellable connect path + connection telemetry fields.
src/services/ai/QueryInsightsAIService.ts	Add telemetry breakdown counts for recommendation actions.
src/plugins/service-azure-vm/discovery-tree/vm/AzureVMResourceItem.ts	Use cancellable connect path + connection correlation telemetry.
src/plugins/service-azure-mongo-vcore/discovery-tree/documentdb/DocumentDBResourceItem.ts	Use cancellable connect path + cancellation telemetry + correlation ID.
src/plugins/service-azure-mongo-ru/discovery-tree/documentdb/MongoRUResourceItem.ts	Use cancellable connect path + cancellation telemetry + correlation ID.
src/documentdb/utils/getClusterMetadata.ts	Add pre-connection domain-only metadata helper.
src/documentdb/ClustersClient.ts	Add abortable connect, correlation ID, and static metadata telemetry emission.
src/commands/updateCredentials/updateCredentials.ts	Add telemetry property for invocation source.
src/commands/removeDiscoveryRegistry/removeDiscoveryRegistry.ts	Add telemetry for discovery provider removal + active count.
src/commands/addDiscoveryRegistry/ExecuteStep.ts	Add telemetry for discovery provider addition + active count.
src/commands/addConnectionFromRegistry/addConnectionFromRegistry.ts	Add telemetry property for source view.
package.json	Bump version to 0.7.3; update engines + dependency refresh.
l10n/bundle.l10n.json	Add new connection/cancellation/localized error strings.
eslint.config.mjs	Split TS/TSX configs; add react/react-hooks/jsx-a11y; refine test file overrides.
docs/release-notes/0.7.md	Add v0.7.3 release notes entry.
docs/index.md	Link v0.7.3 release notes section.
CHANGELOG.md	Add v0.7.3 changelog entry.
.nvmrc	Update Node version to 22.18.
.azure-pipelines/release-npm-packages.yml	Add placeholder ADO release pipeline for config validation.
.azure-pipelines/build-npm-packages.yml	Add placeholder ADO build pipeline for config validation.

Comments suppressed due to low confidence (6)

src/webviews/documentdb/collectionView/components/resultsTab/DataViewPanelTable.tsx:213

• /* eslint-disable react-hooks/refs */ disables the rule for the rest of the file. Prefer a narrower suppression (e.g., eslint-disable-next-line react-hooks/refs on the ref={gridRef} line, or re-enable immediately after) to avoid masking future violations.
  src/tree/connections-view/DocumentDBClusterItem.ts:244
• Avoid (error as Error).message in catch blocks; error can be non-Error and this will produce misleading/undefined messages (and violates the repo’s safe-error-handling pattern). Extract errorMessage via error instanceof Error ? error.message : String(error) and use that in output/UI.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }
                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

src/tree/azure-resources-view/mongo-ru/RUCoreResourceItem.ts:130

• Avoid (error as Error).message in this catch block; use a safe errorMessage extraction (error instanceof Error ? error.message : String(error)) before logging/showing the error.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }

                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

src/tree/azure-resources-view/documentdb/VCoreResourceItem.ts:178

• Avoid (error as Error).message in this catch block; use a safe errorMessage extraction (error instanceof Error ? error.message : String(error)) before logging/showing the error.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }

                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

src/plugins/service-azure-mongo-vcore/discovery-tree/documentdb/DocumentDBResourceItem.ts:195

• Avoid (error as Error).message in this catch block; use a safe errorMessage extraction (error instanceof Error ? error.message : String(error)) before logging/showing the error.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.measurements.connectionEstablishmentTimeMs = Date.now() - connectionStartTime;
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }

                // Add error telemetry
                context.telemetry.measurements.connectionEstablishmentTimeMs = Date.now() - connectionStartTime;
                context.telemetry.properties.connectionResult = 'failed';
                context.telemetry.properties.connectionErrorType = error instanceof Error ? error.name : 'UnknownError';

                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

src/plugins/service-azure-mongo-ru/discovery-tree/documentdb/MongoRUResourceItem.ts:146

• Avoid (error as Error).message in this catch block; use a safe errorMessage extraction (error instanceof Error ? error.message : String(error)) before logging/showing the error.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.measurements.connectionEstablishmentTimeMs = Date.now() - connectionStartTime;
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }

                // Add error telemetry
                context.telemetry.measurements.connectionEstablishmentTimeMs = Date.now() - connectionStartTime;
                context.telemetry.properties.connectionResult = 'failed';
                context.telemetry.properties.connectionErrorType = error instanceof Error ? error.name : 'UnknownError';

                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

[Copilot]

Pull request overview

Copilot reviewed 50 out of 51 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (4)

src/tree/connections-view/DocumentDBClusterItem.ts:245

• This catch block uses (error as Error).message when logging and showing the error. If a non-Error value is thrown, this will itself throw and hide the original failure. Extract a safe errorMessage via error instanceof Error ? error.message : String(error) and use that for both output + UI.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }
                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },
                );

src/tree/azure-resources-view/mongo-ru/RUCoreResourceItem.ts:130

• This catch block uses (error as Error).message when logging and building the modal detail. If the thrown value isn’t an Error, this can crash the handler and obscure the real issue. Use a safe errorMessage (error instanceof Error ? error.message : String(error)) and use that consistently.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }

                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

src/tree/azure-resources-view/documentdb/VCoreResourceItem.ts:178

• This catch block uses (error as Error).message when logging and building the modal detail. If the thrown value isn’t an Error, accessing .message can throw and mask the underlying error. Prefer a safe extraction (error instanceof Error ? error.message : String(error)) and use that for output + UI.

            } catch (error) {
                if (error instanceof UserCancelledError) {
                    context.telemetry.properties.connectionResult = 'cancelled';
                    throw error;
                }

                ext.outputChannel.appendLine(l10n.t('Error: {error}', { error: (error as Error).message }));

                void vscode.window.showErrorMessage(
                    l10n.t('Failed to connect to "{cluster}"', { cluster: this.cluster.name }),
                    {
                        modal: true,
                        detail:
                            l10n.t('Revisit connection details and try again.') +
                            '\n\n' +
                            l10n.t('Error: {error}', { error: (error as Error).message }),
                    },

src/webviews/documentdb/collectionView/components/resultsTab/DataViewPanelTable.tsx:213

• /* eslint-disable react-hooks/refs */ here disables the rule for the rest of the file (there’s no matching eslint-enable). If the intent is only to allow the single ref={gridRef} usage, prefer eslint-disable-next-line react-hooks/refs on the ref line (or re-enable immediately after) to keep the suppression scoped.

[Copilot]

Pull request overview

Copilot reviewed 50 out of 51 changed files in this pull request and generated 3 comments.