Skip to content

feature: Memory review UX before commit#1

Draft
jonparker wants to merge 4 commits intons408:mainfrom
jonparker:main
Draft

feature: Memory review UX before commit#1
jonparker wants to merge 4 commits intons408:mainfrom
jonparker:main

Conversation

@jonparker
Copy link

No description provided.

claude and others added 4 commits March 2, 2026 05:33
Adds scripts/review-memory-write.sh, a Claude Code hook that intercepts
every write to MEMORY.md and shows a unified diff before the change
becomes permanent. The user can approve, open the file in $EDITOR, or
discard (restores from a backup taken by the PreToolUse phase).

Also adds templates/settings.json with the hook registration block,
updates README with setup instructions, marks CONTRIBUTING problem #1
as solved, and extends the lint workflow to syntax-check the new script.

https://claude.ai/code/session_01PT5BQ1jNj5XVFLFMSvLkP9
Drops the python3 dependency. The hook JSON from Claude Code is
structured enough that two grep -o | head -1 | sed calls can extract
hook_event_name and file_path reliably without any external interpreter.

https://claude.ai/code/session_01PT5BQ1jNj5XVFLFMSvLkP9
grep+sed is fragile for nested keys and harder to follow.
python3 is standard on macOS and Linux so the dependency is acceptable.
Also documents python3 and diff as explicit dependencies in the README.

https://claude.ai/code/session_01PT5BQ1jNj5XVFLFMSvLkP9
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.

2 participants