Skip to content

feat: harden local CLI worker - #51

Merged
EmersonBraun merged 1 commit into
mainfrom
codex/issue-40-bounded-worker
Aug 26, 2026
Merged

feat: harden local CLI worker#51
EmersonBraun merged 1 commit into
mainfrom
codex/issue-40-bounded-worker

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Summary

  • Extends the shared runLocalCli boundary with abort cancellation, process-group termination, timeout/output ceilings, and parent-signal cleanup.
  • Makes isolated execution use temporary HOME/TMPDIR, a minimal environment, and only explicitly selected credentials; trusted-local remains explicit.
  • Redacts provider credentials and known token formats from bounded diagnostics.
  • Propagates cancellation and worker policy into Codex and Claude adapters without changing the AdapterFactory contract.
  • Adds versioned worker limits to .agentskit-review.json and offline coverage for isolation, inheritance, abort, overflow, redaction, limits, and missing executables.

Validation

  • npm run check
  • npm pack --dry-run
  • 45/45 tests passing

Scope

Reuses the existing process boundary; no provider-specific subprocess implementation was added.

Closes #40

@EmersonBraun
EmersonBraun merged commit c077d2b into main Aug 26, 2026
5 checks passed
@EmersonBraun
EmersonBraun deleted the codex/issue-40-bounded-worker branch August 26, 2026 01: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.

feat: harden the bounded local CLI worker

1 participant