fix: adds optimize indexes (33) to migration runner#330
Conversation
🦋 Changeset detectedLatest commit: d4fd34e The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
emdash-playground | d4fd34e | Apr 07 2026, 09:53 AM |
There was a problem hiding this comment.
Pull request overview
Registers the existing 033_optimize_content_indexes database migration in the static migration runner so it actually runs on startup and applies the intended composite/partial index optimizations.
Changes:
- Statically imports
033_optimize_content_indexesand adds it toStaticMigrationProvider.getMigrations(). - Adds a changeset to ship the fix as a patch release of
emdash.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/core/src/database/migrations/runner.ts | Adds migration 033 import and registration so it is executed by the Kysely migrator. |
| .changeset/fix-migration-033-runner-registration.md | Documents the patch release for registering migration 033. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@emdash-cms/admin
@emdash-cms/auth
@emdash-cms/blocks
@emdash-cms/cloudflare
emdash
create-emdash
@emdash-cms/gutenberg-to-portable-text
@emdash-cms/x402
@emdash-cms/plugin-ai-moderation
@emdash-cms/plugin-atproto
@emdash-cms/plugin-audit-log
@emdash-cms/plugin-color
@emdash-cms/plugin-embeds
@emdash-cms/plugin-forms
@emdash-cms/plugin-webhook-notifier
commit: |
MattieTK
force-pushed
the
fix/optimize-indexes-in-runner
branch
from
2b2831a to
4ba5016
Compare
|
You'll need to update migrations.test.ts, which hs an expected number of migrations |
|
Fixed and also added #347 as optional that would make maintaining this number easier in the future |
What does this PR do?
Registers migration
033_optimize_content_indexesin the static migration runner so it actually executes on startup.The migration itself was already written but had been omitted from runner.ts, meaning the composite indexes and partial indexes it defines were never applied. This adds the static import and registers it in getMigrations().
Type of change
Checklist
pnpm typecheckpassespnpm --silent lint:json | jq '.diagnostics | length'returns 0pnpm testpasses (or targeted tests for my change)pnpm formathas been runAI-generated code disclosure