docs: log 2025-10-20 debrief

Author: flyingrobots

DEBRIEF.json

@@ -23,3 +23,4 @@
 {"date":"2025-10-20","time":"15:55","summary":"Closed the loop on clang-tidy’s implicit-bool complaint by reinstating the explicit cast and confirmed the audit now reports PIE correctly in CI.","topics":[{"topic":"Digits buffer assert","what":"Restored the (_Bool) cast in the _Static_assert guarding the 64-byte scratch array","why":"GNU-GON-CRY clang-tidy treats int-to-bool conversions as errors","context":"readability-implicit-bool-conversion flagged the newer form","issue":"CI failed after removing the cast","resolution":"Reintroduced the explicit cast to satisfy the lint rule","future_work":"None","time_percent":60},{"topic":"CI audit parity","what":"Verified the updated PIE detection logic against the build artifacts","why":"Ensure Linux release jobs stop flagging false negatives","context":"Security audit now prints the report inline on failure","issue":"Needed a local run to confirm","resolution":"Ran the audit targeting build/ and observed PIE marked enabled","future_work":"Monitor the next Quality Matrix run","time_percent":40}],"key_decisions":[],"action_items":[]}
 {"date":"2025-10-20","time":"11:35","summary":"Unblocked the sanitizer CI leg by disabling conflicting ASAN flags when running the MSAN job.","topics":[{"topic":"Sanitizer matrix","what":"Gated METAGRAPH_*SAN toggles per workflow matrix leg","why":"CI memory sanitizer run failed because both -fsanitize=memory and -fsanitize=address were set","context":"GitHub Actions sanitizers job on feat/minimal-dpoi-qca-loop","issue":"ASAN defaults stayed enabled when requesting MSAN, causing clang to reject the flag combination","resolution":"Updated ci.yml to map each matrix entry to explicit METAGRAPH_{A,U,T,M}SAN settings before invoking CMake","future_work":"Watch the next CI cycle in case MSAN still lacks instrumented runtimes","time_percent":100}],"key_decisions":["Disable ASAN/UBSAN when invoking the MSAN and TSAN legs"],"action_items":[]}
 ```
+{"date":"2025-10-20","time":"12:18","summary":"Split historical debriefs into DEBRIEF.json and hardened CI lint tooling for PR #70.","topics":[{"topic":"Debrief archive","what":"Moved JSONL history from AGENTS.md into a dedicated DEBRIEF.json and updated docs","why":"Review feedback flagged repeated markdown lint issues in AGENTS.md","context":"PR #70 documentation polish","issue":"Embedded code fence kept breaking lint and reviews","resolution":"Created DEBRIEF.json, refreshed AGENTS.md guidance, and synced DEBRIEF_FORMAT.md","future_work":"Monitor future agents to ensure they append to the new file","time_percent":40},{"topic":"Sanitizer workflow","what":"Added unsupported-sanitizer guard and compile_commands.json validation to CI","why":"Reviewer requested early failure when matrix entries drift","context":"GitHub Actions sanitizers and clang-tidy jobs","issue":"Previous case block lacked fallback and tidy job assumed compile DB","resolution":"Extended bash case with explicit error path and added verification step","future_work":"Watch next CI run for regressions","time_percent":30},{"topic":"Commit lint dependencies","what":"Introduced package.json plus npm ci step so commitlint resolves cleanly","why":"PR gatekeeper job failed to locate @commitlint/config-conventional","context":"scripts/ci/lint-commits.sh invoked via pr-guard.yml","issue":"npx attempted to download missing modules on every run and failed","resolution":"Added devDependencies, npm ci step, and switched script to npx --no-install","future_work":"Consider caching npm modules if the workflow becomes a bottleneck","time_percent":30}],"key_decisions":["Store debrief history exclusively in DEBRIEF.json to avoid AGENTS.md churn","Rely on npm-managed commitlint packages instead of ephemeral npx downloads"],"action_items":[]}