Skip to content

fix(wake): make interrupt injection opt-in#299

Merged
avivsinai merged 1 commit into
mainfrom
codex1/p0-interrupt-default-none-20260726
Jul 26, 2026
Merged

fix(wake): make interrupt injection opt-in#299
avivsinai merged 1 commit into
mainfrom
codex1/p0-interrupt-default-none-20260726

Conversation

@avivsinai

Copy link
Copy Markdown
Owner

Summary

  • default standalone wake interrupt commands to none so urgent labelled messages cannot inject Ctrl+C implicitly
  • preserve interrupt notices and explicit --interrupt-cmd ctrl-c behavior
  • document that Ctrl+C sends real SIGINT and can interrupt or crash the foreground agent

Verification

  • RED on base: go test ./internal/cli -run '^TestRunWakeWithLoopInterruptCommandDefaultsOffAndRemainsOptIn$' -count=1\n- focused regression + help tests pass\n- make check-skills\n- go test ./internal/cli/...\n- make ci\n\nExact approved tree: 36c3c6d04bf4c054c59052227da0301485626772.

@avivsinai
avivsinai merged commit bf11814 into main Jul 26, 2026
5 checks passed
@avivsinai
avivsinai deleted the codex1/p0-interrupt-default-none-20260726 branch July 26, 2026 12:17
avivsinai added a commit that referenced this pull request Jul 26, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.47.3](v0.47.2...v0.47.3)
(2026-07-26)


### Bug Fixes

* **cli:** reject stray leaf arguments
([#293](#293))
([9bd6c1a](9bd6c1a))
* **install:** verify release checksums fail closed
([#291](#291))
([321881c](321881c))
* **wake:** admit wake when an unverified generic lock cannot be proven
([#301](#301))
([d177948](d177948))
* **wake:** close wake terminal descriptors on exec
([#300](#300))
([e4db6ea](e4db6ea))
* **wake:** make interrupt injection opt-in
([#299](#299))
([bf11814](bf11814))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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