Skip to content

fix(permissions): add typed parent approval - #1440

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-10696-add-typed-parent-approval-for-pending-worker-denied-calls
Oct 15, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-10696-add-typed-parent-approval-for-pending-worker-denied-calls

Conversation

@bwachman

Copy link
Copy Markdown
Contributor

Summary

  • Provides typed parent-side approval for an exact pending worker denied call.
  • Keeps send_input non-authoritative and binds approval strictly to the target, question, request, and denied-call identity.
  • Consumes approved retries once, includes the approval control plane for nested orchestrators, and closes the authorization/approval race.

Verification

  • bun test src/permission/worker-grant.test.ts src/permission/reactor-authorize.test.ts src/subagent/lifecycle-tools.test.ts src/subagent/authority.test.ts src/subagent/run-authority.test.ts passes (96 pass, 498 assertions).
  • git diff --check passes.
  • bun run typecheck and the pre-commit build pass.
  • bun run check remains blocked at oxfmt preflight by pre-existing formatting failures in untracked BUILD_PLAN.md and SOLUTION_SCOPE.md.

Fixes CL-10696

@linear-code

linear-code Bot commented Oct 14, 2026

Copy link
Copy Markdown

CL-10696

@TheGreatAxios
TheGreatAxios marked this pull request as ready for review October 15, 2026 06:11
@corbits-triage

corbits-triage Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Parked denied-call asks name approve_worker_call and the grant request id.
Revoking an authorization records a revoked audit event.
@TheGreatAxios
TheGreatAxios merged commit 5f5b96e into main Oct 15, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants