Skip to content

fix: harden driver lifecycle supervision - #84

Merged
WH-2099 merged 14 commits into
mainfrom
fix/harden-driver-lifecycle
Jul 27, 2026
Merged

fix: harden driver lifecycle supervision#84
WH-2099 merged 14 commits into
mainfrom
fix/harden-driver-lifecycle

Conversation

@WH-2099

@WH-2099 WH-2099 commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • Validate driver wire inputs and unsupported provider configuration before execution.
  • Make cancellation, terminal delivery, and shutdown cleanup fail closed.
  • Supervise ACP, Claude, and OpenAI process trees through provider exit and driver crash.
  • Cache verified Linux process identities so unrelated restricted /proc entries cannot disable cleanup.
  • Preserve terminal exit-before-release ordering when cleanup ownership changes after a deadline.
  • Buffer ACP session updates until session/new establishes the authoritative session ID.
  • Expand packed-artifact live coverage for lifecycle, permission, resume, MCP, signal, heartbeat, and disconnect boundaries.
  • Make long-tool startup fail early on terminal, protocol, exit, or dead-process evidence.
  • Improve structured diagnostics while redacting credentials before JSON escaping.
  • Remove duplicated lifecycle harness setup without removing scenarios or assertions.

Validation

  • vp run check: 1231 passed, 40 live or capability-gated tests skipped, 0 failed, 3602 assertions.
  • vp exec bun test --coverage: 90.37% functions, 90.28% lines, 1231 passed, 0 failed.
  • Restricted /proc integration with CAP_SYS_PTRACE dropped: 45 passed, 2 capability-gated tests skipped, 0 failed.
  • Process-group cleanup stress: 100 rounds passed with no leaked process.
  • vp run test:live: 38 passed, 0 failed, 17544 assertions in 577.40 seconds.

Follow-up

Closes #83

@WH-2099
WH-2099 force-pushed the fix/harden-driver-lifecycle branch from 7d51cea to 2dca7cf Compare July 27, 2026 03:34
@WH-2099
WH-2099 marked this pull request as ready for review July 27, 2026 03:36
Copilot AI review requested due to automatic review settings July 27, 2026 03:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

@WH-2099
WH-2099 merged commit 334b895 into main Jul 27, 2026
2 checks passed
@WH-2099
WH-2099 deleted the fix/harden-driver-lifecycle branch July 27, 2026 03:37
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.

fix: harden driver lifecycle supervision

2 participants