Skip to content

hooks: Fail open on intake errors in Rust CLI and agent-trace plugin#106

Merged
davidabram merged 2 commits into
mainfrom
hooks/intake-fail-open
Jul 1, 2026
Merged

hooks: Fail open on intake errors in Rust CLI and agent-trace plugin#106
davidabram merged 2 commits into
mainfrom
hooks/intake-fail-open

Conversation

@stefanskoricdev

@stefanskoricdev stefanskoricdev commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@stefanskoricdev stefanskoricdev force-pushed the hooks/intake-fail-open branch from 1f5abe9 to 9cf41dd Compare June 30, 2026 15:02
@stefanskoricdev stefanskoricdev marked this pull request as ready for review June 30, 2026 17:06
@stefanskoricdev stefanskoricdev force-pushed the hooks/intake-fail-open branch from 0fda92a to c746137 Compare July 1, 2026 09:21
stefanskoricdev and others added 2 commits July 1, 2026 11:39
Wrap diff-trace and conversation-trace Rust subcommand handlers with
Ok(...) so non-zero exits don't propagate when intake fails. Update the
agent-trace plugin's run hooks to ignore child stderr and resolve
unconditionally, preventing sce intake errors from leaking into the
OpenCode TUI.
Add fail-open missing-CLI guidance for generated Claude hook commands, OpenCode plugin wrappers, and the git post-commit hook. Preserve normal hook delegation when sce is installed while surfacing the install URL when hook runtime cannot find the CLI.

Co-authored-by: SCE <sce@crocoder.dev>
@stefanskoricdev stefanskoricdev force-pushed the hooks/intake-fail-open branch from c746137 to 7142524 Compare July 1, 2026 09:53
@davidabram davidabram merged commit 616f4fe into main Jul 1, 2026
6 checks passed
@davidabram davidabram deleted the hooks/intake-fail-open branch July 1, 2026 10:08
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