[codex] Clarify routed memory update scope - #1
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6f76503c7d
ℹ️ 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".
| 6. Identify conflicts instead of smoothing them over. | ||
| 7. Return a compact restart plan. | ||
| 2. Identify which memory files are relevant to the current resume task; do not preemptively read the full memory set. | ||
| 3. Read the latest checkpoint in `RECOVERY_NOTES.md` when relevant. |
There was a problem hiding this comment.
Require reading RECOVERY_NOTES in resume workflow
Making RECOVERY_NOTES.md optional in the resume flow can degrade restart accuracy because this file is the canonical checkpoint for where work paused; if the agent decides it is not "relevant," it may produce a restart plan from stale context or miss the latest blocker/next step. In a chat-loss or migration scenario (the exact use case of this task), skipping that checkpoint undermines the purpose of the workflow.
Useful? React with 👍 / 👎.
Summary
SKILL.mdso agents do not treat the memory file set as a sync checklist.CONTEXT_MANIFEST.md, avoid preemptively reading the full memory set, and briefly state read scope for non-trivial work.tasks/update_memory.mdrequiring a short update plan before editing memory files.templates/DOCS_GUIDE.md,templates/CONTEXT_MANIFEST.md,README.md, andtemplates/README.md.Why
This makes the intended operating model explicit: project-memory should route each piece of information to its canonical home, and most sessions should read and update only the files relevant to the current task.
Validation
python -m unittest discover -s tests -vgit diff --checkNote: I also checked the GitHub base branch before editing; local
masterwas aligned withorigin/master.