Skip to content

Add query observability instrumentation - #2724

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue1679-1786-1789
May 31, 2026
Merged

Add query observability instrumentation#2724
Widthdom merged 4 commits into
mainfrom
fix-issue1679-1786-1789

Conversation

@Widthdom

@Widthdom Widthdom commented May 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add opt-in ActivitySource tracing for MCP requests and tracked SQLite queries.
  • Accept MCP params._meta.traceparent as parent W3C trace context.
  • Add CDIDX_SLOW_QUERY_MS stderr slow-query diagnostics and document query profiling controls.

Documentation and changelog

  • Updated DEVELOPER_GUIDE.md with the CodeIndex ActivitySource name and profiling controls.
  • Added changelog.d/unreleased/1679.added.md.
  • Added changelog.d/unreleased/1786.fixed.md.
  • Added changelog.d/unreleased/1789.fixed.md.

Fixes #1679
Fixes #1786
Fixes #1789

Validation

  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release -p:UseSharedCompilation=false --filter "FullyQualifiedNameDbDebugTests.ExecuteTrackedReader_EmitsActivityAndSlowQueryLog|FullyQualifiedNameMcpServerTests.ProcessFrame_UsesTraceParentFromMetaAsActivityParent|FullyQualifiedName~McpServerTests.ProcessFrame_IgnoresNonStringTraceParent"
  • dotnet run --project tools/CodeIndex.Changelog -- check

Review

  • Adversarial review found traceparent type handling and duplicate slow-query logging issues; both were fixed.

Follow-up candidates

  • None.

@Widthdom
Widthdom marked this pull request as ready for review May 31, 2026 05:21
@Widthdom
Widthdom merged commit 58879ea into main May 31, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue1679-1786-1789 branch May 31, 2026 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant