Clean up Memory Recall repository and product identity#20
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c688180f10
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| { | ||
| "schemaVersion": "1.0.0", | ||
| "project": "open-agent-fabric", | ||
| "project": "memory-recall", |
There was a problem hiding this comment.
Keep project status schema-valid
With PROJECT_STATUS.json now declaring memory-recall, any consumer that validates this canonical status file against packages/protocol/schemas/project-status.schema.json rejects it because the schema still has project.const set to open-agent-fabric; I confirmed the repo's validateJsonSchema reports a $.project const failure. Please update the governing schema/fixtures or keep a compatibility value so the status document remains schema-valid.
Useful? React with 👍 / 👎.
What changed
oafcommands, URIs, task IDs, package scopes, and server keys where they are compatibility contracts.scratch/review artifacts and regenerated repository/release provenanceWhy
The rename had reached the primary product shell, but canonical repository metadata, CLI topic banners, generated memory copy, API/schema titles, and release evidence could still revive the retired product name. That made GitHub and installed-package truth inconsistent.
User and developer impact
New output consistently says Memory Recall. Existing
oafcompatibility entrypoints and protocol identifiers continue to work. This does not publishmemory-recall@1.1.0or enable external writes, models, adapters, or network defaults.Root cause
The rebrand was handled as several surface edits rather than a tested canonical identity boundary. Generated release evidence also retained the old 655-test snapshot after four new regressions were added.
Verification
npm run ci: 969 repository files checked, 182/182 protocol fixtures, 659/659 Node tests, 144/144 evaluationsnpm run consumer:smoke: package, first-run Recall Map, temp-HOME install, real MCP client, and Control API flows passednpm run consumer:browser-smoke: desktop/mobile Recall Map, handoff, harness, Token Saver, memory, and graph flows passednpm pack --dry-run --json:memory-recall@1.1.0package dry-run succeedednpm audit --omit=dev: 0 vulnerabilitiesRepository cleanup completed separately
v1.0.5at the exact git commit recorded by npm