Skip to content

fix hook context output#1

Merged
danielmorozoff merged 1 commit intov5from
fix/codex-55-hook-context
Apr 24, 2026
Merged

fix hook context output#1
danielmorozoff merged 1 commit intov5from
fix/codex-55-hook-context

Conversation

@danielmorozoff
Copy link
Copy Markdown
Owner

@danielmorozoff danielmorozoff commented Apr 24, 2026

Summary

  • Fix PreToolUse hook output to emit top-level systemMessage instead of unsupported hookSpecificOutput.additionalContext.
  • Add a regression test covering unsupported hook fields.
  • Ignore local assistant artifacts in .gitignore.

Why

Newer CLI versions reject hook output containing additionalContext with:

PreToolUse hook returned unsupported additionalContext

The generated hook had regressed to a Claude-specific output shape.

Validation

  • .venv/bin/pip install -e '.[all]'
  • .venv/bin/pip check
  • env HOME=/tmp/graphify-pytest-home .venv/bin/python -m pytest -q
  • .venv/bin/graphify update .
  • Smoke-tested the platform install and verified the generated hook output contains systemMessage and no additionalContext.

@danielmorozoff danielmorozoff marked this pull request as ready for review April 24, 2026 11:22
@danielmorozoff danielmorozoff merged commit 705e77b into v5 Apr 24, 2026
@danielmorozoff danielmorozoff changed the title [codex] fix codex hook context output fix hook context output Apr 24, 2026
danielmorozoff pushed a commit that referenced this pull request Apr 24, 2026
fix hook context output
@danielmorozoff danielmorozoff deleted the fix/codex-55-hook-context branch April 24, 2026 11:27
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