Skip to content

Honor explicit no-more-tools Stop boundaries - #9

Open
BananaAccurate wants to merge 1 commit into
avansaber:mainfrom
BananaAccurate:codex/stop-defer-contract
Open

Honor explicit no-more-tools Stop boundaries#9
BananaAccurate wants to merge 1 commit into
avansaber:mainfrom
BananaAccurate:codex/stop-defer-contract

Conversation

@BananaAccurate

Copy link
Copy Markdown

Summary

  • preserve newly queued files while honoring /tailtest defer and explicit no-further-tools boundaries
  • read only the bounded latest user message from Codex-owned session transcripts
  • fail closed for assistant text, fenced examples, and external transcript paths
  • document the one-turn defer contract and add seven regression scenarios

Verification

  • python -m pytest -q tests/test_stop_hook.py — 27 passed on Windows
  • python3 -m pytest -q — 407 passed under Ubuntu/WSL from the exact clean worktree
  • candidate adds no Ruff findings relative to origin/main (both report the same 15 pre-existing findings)
  • live installed-plugin canary observed file change → CANARY_SAFE → turn completion, with no subsequent tool call; adversarial filename remained data and no PWNED artifact was created

@BananaAccurate

Copy link
Copy Markdown
Author

@avansaber This is ready for maintainer review and merge. The PR is scoped to five files and is currently mergeable. Verification: 407 tests passed on Ubuntu/WSL from the exact clean worktree; 27 focused Stop-hook tests passed on Windows; the live installed-plugin canary completed immediately after CANARY_SAFE with no later tool call, and the adversarial filename remained data. Maintainer edits are enabled. I left release versioning/tagging to the maintainer workflow documented in CONTRIBUTING.md.

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