Skip to content

fix(harness): map native plan surfaces for R25 receipt events - #5203

Merged
MohabMohie merged 3 commits into
mainfrom
ChaosEngine/r25-plan-surfaces
Aug 19, 2026
Merged

fix(harness): map native plan surfaces for R25 receipt events#5203
MohabMohie merged 3 commits into
mainfrom
ChaosEngine/r25-plan-surfaces

Conversation

@MohabMohie

Copy link
Copy Markdown
Contributor

Summary

  • Map generic plan surfaces (enter_plan_mode / exit_plan_mode / todo_write, plus PascalCase forms) into R25 compare-proven-approaches and record-plan in portable guard.py, keeping update_plan behavior.
  • Track origin .grok/hooks/lifecycle.json and widen Claude/Codex/Grok hook matchers so PreToolUse/PostToolUse can observe those tools.
  • Add focused lifecycle coverage for the new mappings and for tracked Grok lifecycle hooks.

Test plan

  • py -3 -m unittest tests.scripts.test_chaos_engine_hook tests.scripts.test_guard_lifecycle tests.scripts.test_chaos_engine_research (298 OK)
  • Focused mapping/matcher/tracking tests green after RED

Closes #5195
Related to #5186
Tracker remains #5186

Teach portable `_research_preflight_events` to record compare-proven-approaches
and record-plan from enter_plan_mode, exit_plan_mode, and todo_write while
keeping update_plan behavior. Track origin .grok/hooks/lifecycle.json and widen
host hook matchers so PreToolUse/PostToolUse observe those tools.

Closes #5195
Related to #5186
Capture snake_case/PascalCase normalization and hook-matcher allowlist coupling
from #5195 / PR #5203 for reuse.
@codacy-production

codacy-production Bot commented Aug 19, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@MohabMohie
MohabMohie merged commit f748d77 into main Aug 19, 2026
27 checks passed
@MohabMohie
MohabMohie deleted the ChaosEngine/r25-plan-surfaces branch August 19, 2026 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Grok cannot emit R25 compare-proven-approaches without update_plan

1 participant