Skip to content

feat: include sqlcommenter tags in CI query payload#98

Merged
veksen merged 1 commit intomainfrom
veksen/ci-expose-tags
Mar 26, 2026
Merged

feat: include sqlcommenter tags in CI query payload#98
veksen merged 1 commit intomainfrom
veksen/ci-expose-tags

Conversation

@veksen
Copy link
Member

@veksen veksen commented Mar 26, 2026

Summary

  • Extract SQLCommenter tags from Analyzer.analyze() and include in QueryProcessResult
  • Add tags: SQLCommenterTag[] to CiQueryPayload interface
  • Map tags through all mapResultToQuery() cases to the Site API payload
  • Update test fixtures

Context

Companion to Query-Doctor/Site#2615. The Site repo now accepts and stores tags on CiRunQueryResult (optional, defaults to []). This PR makes the analyzer actually send them.

Once both PRs are merged, CI-captured queries will show sqlcommenter metadata (route, method, file, etc.) in the query detail view.

Test plan

  • TypeScript compiles cleanly (tsc --noEmit)
  • All QueryProcessResult return paths include tags
  • All mapResultToQuery() cases include tags: result.tags
  • Test fixtures updated

🤖 Generated with Claude Code

Extract SQLCommenter tags from Analyzer.analyze() and thread them
through QueryProcessResult and CiQueryPayload to the Site API.
This enables the Site frontend to display route, method, file, and
other sqlcommenter metadata for CI-captured queries.

Companion to Query-Doctor/Site#2615

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Query Doctor Analysis

View full run details

28 queries analyzed

2 pre-existing issues

Using assumed statistics (10000 rows/table). For better results, sync production stats.

@veksen veksen merged commit 728434f into main Mar 26, 2026
6 checks passed
@veksen veksen deleted the veksen/ci-expose-tags branch March 26, 2026 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant