Skip to content

fix(deps): drop dumpStats "full" arg removed in core 0.8.7#131

Merged
veksen merged 1 commit into
mainfrom
veksen/dumpstats-fix
Apr 28, 2026
Merged

fix(deps): drop dumpStats "full" arg removed in core 0.8.7#131
veksen merged 1 commit into
mainfrom
veksen/dumpstats-fix

Conversation

@veksen
Copy link
Copy Markdown
Member

@veksen veksen commented Apr 28, 2026

Summary

  • @query-doctor/core@0.8.7 dropped the third mode parameter from Statistics.dumpStats, breaking typecheck on main at three call sites.
  • Removes the now-invalid "full" literal in src/remote/query-optimizer.ts, src/remote/remote.ts, and src/sync/syncer.ts.

Test plan

  • npm run typecheck
  • npx vitest run (24 files, 169 tests passing)

🤖 Generated with Claude Code

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

@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

2 queries analyzed

2 pre-existing issues

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

Copy link
Copy Markdown
Collaborator

@Xetera Xetera left a comment

Choose a reason for hiding this comment

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

I thought I'd already made these changes though

@Xetera
Copy link
Copy Markdown
Collaborator

Xetera commented Apr 28, 2026

oh it's in the other PR

@veksen veksen merged commit 93c8b21 into main Apr 28, 2026
6 checks passed
@veksen veksen deleted the veksen/dumpstats-fix branch April 29, 2026 07:19
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.

2 participants