Skip to content

feat: add bash-notify-background example for timeout handoff - #119

Merged
378-kaiabot[bot] merged 1 commit into
mainfrom
feat/bash-notify-background-example
Oct 2, 2026
Merged

378-kaiabot[bot] merged 1 commit into
mainfrom
feat/bash-notify-background-example

Conversation

@aliou

@aliou aliou commented Sep 30, 2026

Copy link
Copy Markdown
Owner

Companion to bash-background.ts and bash-to-process.ts: a third bash tool override that changes what a timeout does. The native tool kills the process tree and fails; this one hands the running command to pi-processes via the processes:command:adopt channel and returns the background process id in the tool result, so the agent can keep tracking the long-running job.

  • Same parameters and output streaming as the native tool (command, timeout); only the timeout path differs.
  • Requires a pi-processes version that provides the processes:command:adopt channel.
  • README examples list updated.

No changeset: examples are shipped but this does not change extension behavior.

@aliou
aliou marked this pull request as draft September 30, 2026 09:57
@378-kaiabot
378-kaiabot Bot force-pushed the feat/bash-notify-background-example branch 2 times, most recently from a0f977f to 76f3028 Compare October 2, 2026 20:27
Overrides the bash tool so a command that hits its timeout is adopted by
pi-processes through the processes:command:adopt channel instead of being
killed, returning the background process id so the agent can keep tracking
it. Mirrored parameters (command, timeout) and output streaming keep it a
drop-in replacement for the native tool.
@378-kaiabot
378-kaiabot Bot force-pushed the feat/bash-notify-background-example branch from 76f3028 to c320f2c Compare October 2, 2026 20:29
@aliou
aliou marked this pull request as ready for review October 2, 2026 20:33
@378-kaiabot
378-kaiabot Bot merged commit b781df1 into main Oct 2, 2026
2 checks passed
@378-kaiabot
378-kaiabot Bot deleted the feat/bash-notify-background-example branch October 2, 2026 20:34
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