← Files stark AI DeveloperARCHIVED FILE
skills/codex-memory-curator/references/example-review-report.md
3.98 KB · Oct 2, 2026 · 00:31 UTC
# Example Review Report Use this as a shape reference, not as a fixed answer. Selected workflow: `review-chat`. This example creates no durable curation report. # Codex Memory Review ## Top Decisions 1. Move the repo-specific test command to `AGENTS.md` if it is still current. 2. Delete stale branch state after backup and approval. 3. Rewrite the service-boundary preference so it is scoped to repo conventions. ## Summary - Memory files inspected: 2 - Entries extracted: 6 - Keep: 1 - Rewrite: 1 - Move to AGENTS.md: 1 - Move to repo docs: 1 - Move to skill: 1 - Move to config: 0 - Delete: 1 - Ask user: 0 ## Highest-Risk Memories | ID | Entry | Risk | Recommendation | | --- | ----------------------------- | ----------------------------------------- | --------------------------------- | | M-4 | `Always use service classes.` | Too broad; may fight current repo design. | Rewrite with scope and caveat. | | M-6 | `old-branch-name migration` | One-off branch state. | Delete after approval and backup. | ## Proposed Cleanup Table | ID | Current memory | Classification | Risk tags | Confidence | Reason | Proposed action | | --- | -------------------------------- | ------------------- | ----------------- | ---------- | -------------------------------------- | --------------------------------------------------------------------------------- | | M-1 | `User prefers pnpm.` | `KEEP` | `useful` | high | Stable user preference. | Leave unchanged. | | M-2 | `Run pnpm turbo test here.` | `MOVE TO AGENTS.md` | `repo-specific` | medium | Repo-specific validation command. | Move to current repo `AGENTS.md` if still current. | | M-3 | `Long refactors need handoffs.` | `MOVE TO SKILL` | `workflow` | medium | Reusable workflow with steps. | Move to a handoff/refactor skill. | | M-4 | `Always use service classes.` | `KEEP BUT REWRITE` | `too-broad` | medium | Useful preference but too absolute. | Rewrite: `Prefer explicit service boundaries when repo conventions support them.` | | M-5 | `Architecture decision details.` | `MOVE TO REPO DOCS` | `too-specific` | high | Too long and repo-specific for memory. | Move to `docs/architecture.md`. | | M-6 | `old-branch-name migration` | `DELETE` | `temporary,stale` | high | Temporary, stale branch detail. | Remove after backup. | ## Conflict Notes | ID | Higher source | Conflict | Recommendation | | --- | -------------- | ----------------------------------------- | ------------------------------------------- | | M-2 | `package.json` | Current validation command may differ. | Verify before moving into `AGENTS.md`. | | M-4 | Current source | Current repo may not use service classes. | Rewrite as a preference, not a global rule. | ## Plan - Not applicable: `review-chat` does not plan or execute cleanup. ## Execution Receipt - Not applicable: read-only route. ## Deferred Work - M-2 through M-6 remain recommendations only. ## Backup - Not applicable: no context mutation. ## Verification - Review classifications reconciled against the cited repository evidence. ## Config Recommendation Use Exact Repo Work Mode for the current refactor because stale memory assumptions are likely to be harmful. ## Recommended Next Action Invoke a cleanup route explicitly if these recommendations should be applied; `plan-run-cleanup-file` matches cleanup with a persisted plan and receipt.
SHA-256: 892bcefe16bf1c9d6045a3b355e738a6a0fb44288b6b931580db60c1d6244d19