Skip to content

feat(codex): enrich Bash file-list PostToolUse output#215

Merged
zzet merged 1 commit into
zzet:mainfrom
blankanswer:feat/codex-file-list-posttooluse
Jul 2, 2026
Merged

feat(codex): enrich Bash file-list PostToolUse output#215
zzet merged 1 commit into
zzet:mainfrom
blankanswer:feat/codex-file-list-posttooluse

Conversation

@blankanswer

Copy link
Copy Markdown
Contributor

Summary

  • Extend Codex Bash PostToolUse enrichment to classifier-recognized find ... -name / find ... -iname file-list output.
  • Normalize Codex Bash BashActionFindName events to the existing Glob PostToolUse shape and reuse postGlob indexed file summary enrichment.
  • Preserve existing grep-family and source-read normalization behavior.
  • Keep Codex soft-only: only emit hookSpecificOutput.additionalContext; no deny, suppress, or input rewrite.

Design notes

  • This is intentionally limited to BashActionFindName from the existing Bash classifier.
  • Bash stdout is passed through unchanged as tool_response; postGlob already handles newline-separated file lists.
  • Non-classified file-list commands such as ls, fd, tree, and git ls-files remain silent.
  • Claude, Gemini, and Antigravity hook behavior remains unchanged.

Out of scope

  • ls / fd / tree / git ls-files
  • sed / awk
  • general Bash stdout enrichment
  • apply_patch
  • MCP tool-name matching
  • deny / enforcement behavior
  • input rewrite
  • output suppression
  • resolver / indexer / daemon / graph / MCP tool changes

Tests

  • go test ./internal/hooks
  • go test ./internal/agents/codex
  • go test ./cmd/gortex
  • git diff --check
  • Added regression coverage that sed / awk source reads remain silent.

@zzet zzet merged commit 3befe2d into zzet:main Jul 2, 2026
10 checks passed
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.

2 participants