Skip to content

fix(memory): safely migrate legacy bookmarks and refresh README - #151

Merged
wefio merged 2 commits into
mainfrom
codex/db-migration
Nov 3, 2026
Merged

wefio merged 2 commits into
mainfrom
codex/db-migration

Conversation

@wefio

@wefio wefio commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Change summary

What: Safely reconcile legacy bookmarks during database initialization and refresh the bilingual onboarding documentation.

Why: Existing databases can contain both legacy and current tables, or partially migrated references. These cases need to preserve all records and remain recoverable. New users also need a concrete first experience and an agent-assisted setup path.

Changes:

  • Merge legacy records under a savepoint, reject conflicting records and unsupported legacy schemas, and repair legacy marker references.
  • Add migration regression tests and a reproducible historical characterization, with architecture baseline and decision documentation.
  • Rewrite the English and Chinese READMEs with a first-recall example, a copyable agent setup prompt, and a host capability matrix.

Not verified

Not verified: No migration was run against a real user memory database. Crash/power-loss and concurrent-process scenarios, installation in actual agent hosts, and cloud integration were not tested. Local verification ran on Windows.


Completion checklist

Local quality checks

  • Static checks passed through agent:verify, including the checks covered by verify:static.
  • test:product passed through agent:verify: 2,091 passed, 2 skipped.
  • Dependency/lockfile changes: not applicable to this PR.
  • Subpackage changes: not applicable to this PR.
  • Documentation and glossary checks passed.
  • ## Not verified is filled in.
  • New code and tests were reviewed; no reproducible build artifacts are included.
  • Full mutation:teeth passed for all 23 targets and restored the target files.
  • Targeted migration, storage and tutorial tests passed (39 tests).

The documentation check reported one existing bilingual-link warning in an unrelated decision; lint reported one existing warning in an unrelated integration test.

RCP (Repository Control Plane)

  • An in-flight goal was registered before substantive work.
  • agent:verify reconciled the changed paths and passed.
  • Confirm all-green CI through RCP observation.
  • Only files owned by this task were committed.

CI completion confirmation

  • All checks passed.
  • CodeFactor passes.
  • Static job passes.

wefio added 2 commits November 2, 2026 22:28
Preserve legacy bookmark data through atomic conflict-aware migration, and document the measured architecture and first-run experience.
@wefio
wefio marked this pull request as ready for review November 3, 2026 08:39
@wefio
wefio merged commit 6326647 into main Nov 3, 2026
8 checks passed
@wefio
wefio deleted the codex/db-migration branch November 3, 2026 08:39
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