Skip to content

test: avoid broken pipes in monitor runtime checks - #225

Open
Inference1 wants to merge 1 commit into
PolyArch:devfrom
Inference1:fix/codex-hooks-feature-rename
Open

test: avoid broken pipes in monitor runtime checks#225
Inference1 wants to merge 1 commit into
PolyArch:devfrom
Inference1:fix/codex-hooks-feature-rename

Conversation

@Inference1

@Inference1 Inference1 commented Jul 13, 2026

Copy link
Copy Markdown

Summary

  • replace echo "$output" | grep -q ... assertions in test-monitor-runtime.sh with here-string input
  • prevent grep -q from closing the pipe early and causing intermittent echo: write error: Broken pipe failures under set -o pipefail

Testing

  • bash tests/test-monitor-runtime.sh — 18 passed, 0 failed (5 consecutive runs)
  • bash -n tests/test-monitor-runtime.sh
  • git diff --check upstream/dev...HEAD

@Inference1

Copy link
Copy Markdown
Author

@tony-nowatzki Would you please review this pr?

@Inference1
Inference1 changed the base branch from main to dev July 15, 2026 16:39
@Inference1 Inference1 changed the title fix: support renamed Codex hooks feature test: avoid broken pipes in monitor runtime checks Jul 15, 2026
@Inference1

Copy link
Copy Markdown
Author

@tony-nowatzki The CI fixes are pushed and the branch is now synced with dev. GitHub marked the new workflow runs as action_required for this fork PR. Could you please approve the workflow runs? The local monitor runtime suite passes 18/18.

@Inference1
Inference1 force-pushed the fix/codex-hooks-feature-rename branch from 37bd729 to 2ce4050 Compare July 21, 2026 09:10
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.

Codex 0.139.0 renamed codex_hooks to hooks, causing Codex install to fail

1 participant