Skip to content

Fix Codex Desktop hook coverage without breaking Claude Code - #1

Merged
pwya merged 2 commits into
mainfrom
agent/codex-hook-tool-shapes
Jul 28, 2026
Merged

pwya merged 2 commits into
mainfrom
agent/codex-hook-tool-shapes

Conversation

@pwya

@pwya pwya commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • recognize Codex Desktop's actual FREEFORM apply_patch payloads and shell_command events
  • keep Claude Code support through an allowlisted shell-neutral Python runner for the existing PowerShell hooks
  • update installers, static verification, cross-harness guidance, and isolate the two live-probe actions

Validation

  • full portable CI command set passes locally
  • verify_hooks.ps1: 6/6 green from both release and local skill trees
  • installed Codex policy self-test passes; deployed/local/release policy hashes match
  • all 9 CC Switch Claude provider templates use the canonical runner

Runtime note

The new Codex md-paper hook definition still requires explicit Trust in /hooks and a new Codex task before the current-session live probe can pass. The task used to install it correctly reports both actions as not denied because its hook snapshot predates the new definition.

@pwya
pwya marked this pull request as ready for review July 28, 2026 14:10
@pwya
pwya merged commit 7bf57a0 into main Jul 28, 2026
1 check passed
@pwya
pwya deleted the agent/codex-hook-tool-shapes branch July 28, 2026 14:11
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.

1 participant