Skip to content

Commit a6e4517

Browse files
author
Tehan
committed
chore(external-memory): pin external-recall migration to v54 (LATEST=54)
Renumber external-recall v51→v54 so it holds the same slot on this standalone branch AND in the magic_packet union (skill-memory owns v51/52/53). Eliminates the union renumber step and matches the live dev DB. The 51/52/53 gap here is intentional — the schema-fence test asserts LATEST == max migration version, not contiguity; the runner filters version>currentVersion so a fresh DB runs upstream 1..50 then 54 cleanly. Renamed migrations-v51.test.ts → migrations-v54.test.ts (dynamic LATEST_MIGRATION_VERSION assertion, no hardcoded number).
1 parent 8808b34 commit a6e4517

3 files changed

Lines changed: 8 additions & 4 deletions

File tree

packages/plugin/src/features/magic-context/migrations-v51.test.ts renamed to packages/plugin/src/features/magic-context/migrations-v54.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ function columnNames(db: Database, table: string): string[] {
1212
);
1313
}
1414

15-
describe("migration v51 — external recall snapshot + m[0] marker", () => {
15+
describe("migration v54 — external recall snapshot + m[0] marker", () => {
1616
test("adds external recall columns to session_meta on a fresh DB, idempotently", () => {
1717
const db = new Database(":memory:");
1818
try {

packages/plugin/src/features/magic-context/migrations.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1891,8 +1891,12 @@ const MIGRATIONS: Migration[] = [
18911891
// Renumbered across many upstream rebases (v31→33→37→38→39→42→50); now
18921892
// v51 after upstream v0.31 added its own v50 (ctx-wrapup marker). The
18931893
// body is ensureColumn-idempotent, so a dev DB that already ran it under
1894-
// an old number re-applies harmlessly.
1895-
version: 51,
1894+
// an old number re-applies harmlessly. Pinned to v54 so external-recall
1895+
// holds the same slot on the standalone branch AND in the magic_packet
1896+
// union (where skill-memory occupies v51/52/53) — no renumber on merge,
1897+
// and it matches the live dev DB. The 51/52/53 gap on this standalone
1898+
// branch is intentional (fence asserts LATEST == max version, not contiguity).
1899+
version: 54,
18961900
description: "External memory v2: session recall snapshot + m[0] recall marker",
18971901
up: (db: Database) => {
18981902
// session_meta existence guard — see v30's comment (partial test fixtures).

packages/plugin/src/features/magic-context/storage-db.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export function getSchemaFenceRejection(): {
4444
return lastSchemaFenceRejection;
4545
}
4646

47-
export const LATEST_SUPPORTED_VERSION = 51;
47+
export const LATEST_SUPPORTED_VERSION = 54;
4848

4949
// chmod is meaningless on Windows (POSIX modes are not honored), so all
5050
// permission tightening is skipped there. mkdir's `mode` is likewise ignored.

0 commit comments

Comments
 (0)