-
Notifications
You must be signed in to change notification settings - Fork 5
Release 0.6.3 to main
#432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 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: