Skip to content

chore(deps): adopt @query-doctor/core ^0.23.0 - #225

Merged
veksen merged 1 commit into
mainfrom
chore-adopt-core-0-23-0
Aug 7, 2026
Merged

chore(deps): adopt @query-doctor/core ^0.23.0#225
veksen merged 1 commit into
mainfrom
chore-adopt-core-0-23-0

Conversation

@veksen

@veksen veksen commented Aug 7, 2026

Copy link
Copy Markdown
Member

Goal

Get the analyzer onto the core it needs to dump a schema correctly. Follows Query-Doctor/Site#3925, which published @query-doctor/core@0.23.0. This is the last step; nothing else is waiting on it.

What

Before: on 0.22.0, dumpSchema runs SQL whose CASE con.contype resolves to the char type, so every constraint name is cut to its first character. primary_key is dumped as p and foreign_key as f. Any consumer matching on the full name sees nothing.

After: constraint names arrive whole.

How

The range moves from ^0.22.0 to ^0.23.0, and the lockfile follows. No source changes.

0.23.0 also maps the old one-character codes back to names when it parses a snapshot, so schemas dumped by an analyzer still on 0.22.0 keep reading correctly. That is what makes this bump safe to land on its own schedule rather than in lockstep with anything.

Tests

No behavior in this repo changed, so no test was added. npm test passes locally against 0.23.0: 440 tests across 43 files, including the runs that dump a real Postgres schema in a container. tsc --noEmit is clean.

0.22.0 dumped a schema whose every constraintType was truncated to its first
character, so primary_key arrived as p and foreign_key as f. 0.23.0 fixes the
dump SQL, and normalizes the old codes back to names on the way in, so a
snapshot taken by either version reads the same.

Co-Authored-By: Claude <noreply@anthropic.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Query Doctor — 6 successful checks

 Cost regression — No query went up more than 5%
 Untested data access — No changed data-access file without a test
 New query — No new queries
 New query with index recommendation — No new query ships an index recommendation
 Schema drift — No schema changes
 High-value nudge — No index or rewrite past the threshold


More details via MCP → get_ci_run({ runId: "019fdc56-9f17-7e4b-96d4-3bfa35bff20b" }) · view run · docs
3 queries read against main on assumed statistics of 10,000,000 rows per table. Sync production stats for costs measured against your real data.

@veksen
veksen merged commit b458be5 into main Aug 7, 2026
6 of 8 checks passed
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