Skip to content

feat(kt-db): bump DefaultGraphTypePlugin to v2 with noop migration (Phase 7 #60)#270

Merged
charlie83Gs merged 1 commit intomainfrom
feat/default-plugin-v2
Apr 20, 2026
Merged

feat(kt-db): bump DefaultGraphTypePlugin to v2 with noop migration (Phase 7 #60)#270
charlie83Gs merged 1 commit intomainfrom
feat/default-plugin-v2

Conversation

@charlie83Gs
Copy link
Copy Markdown
Contributor

Summary

First real exercise of the Phase 7 migration framework end-to-end.
Composition is unchanged from v1 — the only effect of bumping is that
every v1 graph auto-dispatches onto `graph_migration_wf` at worker
boot, the banner briefly shows `read_only_reason='migrating'`, one
audit row lands, and the graph ends at v2.

Intentional operator-visible exercise of the migration path before
a real v3 recipe change ships.

Proves

What it's NOT

  • Not a composition change. Provider ids are identical to v1.
  • Not a data migration. Uses `NoopMigration` — the ready-made class
    `plan.md` specifies for version bumps that change composition-as-
    config but require no data work.

Dependencies

Depends on PR #267 (workflow) and PR #269 (auto-dispatch) being merged
for the plugin bump to actually drive a migration round-trip. Merging
this in isolation is safe — the bump just pins `current_version=2` in
the plugin; without the workflow + dispatcher registered, graphs stay
at v1 until those land (then re-register on next boot).

Test plan

  • `uv run --project libs/kt-config pytest libs/kt-config/tests/ -x` — 236/236
  • `uv run --project libs/kt-db pytest libs/kt-db/tests/integration/ -x` — 230/230
  • `uv run --project services/api pytest services/api/tests/integration/ -x` — 131/131
  • CI green

🤖 Generated with Claude Code

…hase 7 #60)

First real exercise of the migration framework end-to-end. Composition
is unchanged from v1 — the only effect of bumping is that every v1 graph
auto-dispatches onto graph_migration_wf at worker boot, the banner
briefly shows read_only_reason='migrating', one audit row lands, and
the graph ends at v2.

Intentional operator-visible exercise of the migration path before a
real v3 recipe change ships. Proves:

* Startup auto-dispatch (#58) picks up out-of-date graphs.
* The Hatchet workflow (#57) commits per-hop.
* The audit table + the read-only banner surface the transition.

Migration is NoopMigration from kt_config.migrations — the same
ready-made class plan.md specifies for version bumps that change
composition-as-config but require no data work.
@charlie83Gs charlie83Gs merged commit 6e998a0 into main Apr 20, 2026
24 checks passed
@charlie83Gs charlie83Gs deleted the feat/default-plugin-v2 branch April 20, 2026 21:40
@github-actions
Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

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