Conversation
Revert extra changes
…ft/vscode-documentdb into dev/xingfan/addtelemetry
…om/microsoft/vscode-documentdb into dev/xingfan/useextensiondata-clean
…trpc messaging infra
This reverts commit c698437.
This reverts commit 789b324.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
khelanmodi
approved these changes
Dec 18, 2025
Contributor
There was a problem hiding this comment.
Pull request overview
This PR releases version 0.6.3 from the next branch to main. The release focuses on optimizing Query Insights AI recommendations by reusing execution plans, improving telemetry coverage for AI features, and updating tRPC dependencies to address a security vulnerability.
Key changes:
- Query Insights Stage 3 now reuses the execution plan from Stage 2 instead of running a redundant explain query, improving performance and ensuring consistency between displayed metrics and AI recommendations
- Enhanced telemetry tracking across AI services (CopilotService, PromptTemplateService, QueryInsightsAIService) with detailed error codes and preloaded data tracking
- Updated tRPC dependencies (
@trpc/clientand@trpc/server) from ~11.7.1 to ~11.8.0 to address a prototype pollution security vulnerability
Reviewed changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Bumped version to 0.6.3 and updated tRPC dependencies to ~11.8.0 |
| package-lock.json | Updated lockfile with new tRPC dependency versions |
| CHANGELOG.md | Added 0.6.3 release notes documenting Query Insights optimization, telemetry improvements, and security update |
| docs/release-notes/0.6.md | Added detailed release notes for v0.6.3 with feature descriptions and links to issues/PRs |
| docs/index.md | Updated release version links to include 0.6.3 |
| src/webviews/api/extension-server/trpc.ts | Refactored middleware to inject telemetry into context and added publicProcedureWithTelemetry helper and WithTelemetry type |
| src/webviews/api/configuration/appRouter.ts | Added optional telemetry to BaseRouterContext and created BaseRouterContextWithTelemetry type |
| src/webviews/documentdb/documentView/documentsViewRouter.ts | Migrated to publicProcedureWithTelemetry and WithTelemetry<RouterContext>, fixed spelling of "parameters" |
| src/webviews/documentdb/collectionView/collectionViewRouter.ts | Migrated all procedures to publicProcedureWithTelemetry, added platform telemetry tracking, removed minimum duration delays from Stage 2, added execution plan caching support for Stage 3 |
| src/webviews/documentdb/collectionView/components/queryInsightsTab/QueryInsightsTab.tsx | Moved minimum duration timing logic from backend to frontend for Stage 2 |
| src/services/copilotService.ts | Wrapped sendMessage in telemetry handler to track model usage and LLM errors |
| src/services/promptTemplateService.ts | Added telemetry tracking for template load failures |
| src/services/ai/QueryInsightsAIService.ts | Added executionPlan parameter to getOptimizationRecommendations, enhanced error telemetry with specific error codes for action failures |
| src/documentdb/ClusterSession.ts | Added getRawExplainOutput method for cached execution plan retrieval, updated getExecutionStats to use allPlansExecution verbosity for vCore clusters |
| src/commands/llmEnhancedCommands/indexAdvisorCommands.ts | Enhanced preloaded data handling to support optional collectionStats and indexStats, improved telemetry tracking for preloaded vs fetched data |
| src/documentdb/queryInsights/StagePropertyExtractor.test.ts | Added comprehensive test suite for stage property extraction across multiple scenarios (collscan, query insights, sharded) |
| resources/debug/examples/*.json | Added three new example JSON files for testing with allPlansExecution data |
| l10n/bundle.l10n.json | Added localization strings for preloaded data messages and updated explain completion message |
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
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 version 0.6.3 of the DocumentDB for VS Code extension, focusing on performance improvements for Query Insights, enhanced telemetry for AI features, and important security updates for dependencies. The release also updates documentation and localization files to reflect these changes, and adds new debug data examples for development and troubleshooting.
Query Insights and AI Features:
executionStatsfrom analysis, improving both performance and consistency of AI recommendations. The system now avoids redundant database queries and ensures insights are based on the same metrics shown to users. [1] [2] [3]Dependency and Security Updates:
@trpc/clientand@trpc/serverdependencies to address a high-severity security vulnerability related to prototype pollution.Documentation and Localization:
Debug Data and Internal Improvements:
allPlansExecutionfor Azure Cosmos DB vCore clusters andexecutionStatsfor others, providing richer plan data where available. [1] [2]Version and Changelog Updates: