accept

Accept Step

Finalize the current implementation step with quality gates before committing. Updates CMCM knowledge files (patterns, handoff).

Step to accept: $ARGUMENTS (if empty, infer from TODO.md — the first step with status in-progress)

Instructions

1. Identify the step

Read TODO.md to find the step:

• If $ARGUMENTS is a number, use that step.
• Otherwise find the first in-progress step.
• If no in-progress step exists, inspect git diff HEAD and git status to summarize changes, pick the next step number, and add a new row with status in-progress.

2. Simplify (analyze + fix)

Review changed files and apply simplification fixes:

• Read CC.md first — Node.js / TypeScript clean code rules apply here as the simplify checklist.
• Run git diff HEAD to identify changed files and hunks.
• For each changed area, check for: deep nesting (use guard clauses), long functions (split), generic names (make descriptive), dead code (remove), redundant abstractions (inline), unnecessary comments (delete), plus the rules in CC.md (non-null assertions, floating promises, any types, missing return types on exports, etc.).
• Apply fixes incrementally — one at a time.
• After all simplify fixes, verify: run npm run typecheck, npm run lint, npm test (as separate commands).
• If verification fails, fix the failure before continuing.
• Record what was fixed for the summary in step 5.

3. Codex review (on post-simplify code)

Remove any previous CODEX_REVIEW.md. Then launch Codex CLI in background (Bash with run_in_background: true):

codex exec -s read-only -o CODEX_REVIEW.md "You are a senior code reviewer. Review the recent changes (unstaged and staged diffs) in this codebase.

For each finding report:
- Severity: CRITICAL, HIGH, MEDIUM, or LOW
- File path and line number
- What the issue is
- Suggested fix (code snippet if applicable)

Group by severity (CRITICAL first). Format as markdown checklist with - [ ] items." < /dev/null