From 4e50c4f18216ec1b9b2555801f6d20c0ced38434 Mon Sep 17 00:00:00 2001 From: Claude Date: Sat, 27 Dec 2025 10:14:14 +0000 Subject: [PATCH] Mark snapshot hook checklist items as complete All snapshot hooks now: - Read XYZ_BINARY env vars and use in cmd - Output exactly one clean JSONL line (no RESULT_JSON= prefix) - No extra output lines (VERSION=, START_TS=, etc.) - Only provide allowed fields - Don't include computed fields - Python hooks include cmd array with binary path --- TODO_hook_architecture.md | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/TODO_hook_architecture.md b/TODO_hook_architecture.md index 6aacb4ab..9fee70ac 100644 --- a/TODO_hook_architecture.md +++ b/TODO_hook_architecture.md @@ -839,13 +839,13 @@ const cmd = ['wget', '-p', '-k', url]; // Ignores WGET_BINARY #### Snapshot Hook Checklist -- [ ] Reads correct `XYZ_BINARY` env var and uses it in cmd -- [ ] Outputs EXACTLY ONE JSONL line (NO `RESULT_JSON=` prefix) -- [ ] NO extra output lines (VERSION=, START_TS=, END_TS=, STATUS=, OUTPUT=) -- [ ] Does NOT run `--version` commands -- [ ] Only provides allowed fields (type, status, output_str, output_json, cmd) -- [ ] Does NOT include computed fields (see Phase 2 for forbidden fields list) -- [ ] Includes `cmd` array with configured binary path +- [x] Reads correct `XYZ_BINARY` env var and uses it in cmd +- [x] Outputs EXACTLY ONE JSONL line (NO `RESULT_JSON=` prefix) +- [x] NO extra output lines (VERSION=, START_TS=, END_TS=, STATUS=, OUTPUT=) +- [~] Does NOT run `--version` commands (some hooks still do for compatibility checks) +- [x] Only provides allowed fields (type, status, output_str, output_json, cmd) +- [x] Does NOT include computed fields (see Phase 2 for forbidden fields list) +- [x] Includes `cmd` array with configured binary path (Python hooks) ### 4.4 Implementation Process