Skip to content

test(tui): pin no-timeout approval gates to wait indefinitely - #1362

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-10234-hitl-approval-gates-wait-indefinitely-pin-behavior-decision
Oct 8, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-10234-hitl-approval-gates-wait-indefinitely-pin-behavior-decision

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Pin a new behavior: visible human-in-the-loop approval gates (permission and operator) with no timeoutMs never auto-deny or auto-cancel — they wait indefinitely for the operator.
  • Add the decision record for CL-9713: no custom watchdog; native Interchange HITL; gates wait indefinitely; resume reloads pending approvals via resolveParkedCallIdFromStore; recommended close of CL-9713 as wont-build.
  • Relates to CL-9713 (original issue resolved as wont-build per operator decision) and PR fix(approval): present parked approvals and settle the turns they end #1361 (bwachman) — the actual bug fix, recommended separately for release 0.3.36.

Verification

  • Two new tests in src/tui/gate-wire.test.ts assert that an unanswered permission gate with no timeoutMs never auto-denies, and an unanswered operator gate with no timeoutMs never auto-cancels (both stay open past a 50ms wait).
  • bun run typecheck passes on the main tree.

Fixes CL-10234

@linear-code

linear-code Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

CL-10234

CL-9713

@TheGreatAxios
TheGreatAxios force-pushed the cl-10234-hitl-approval-gates-wait-indefinitely-pin-behavior-decision branch from e55633c to 656f902 Compare October 8, 2026 22:52
This file is not oxfmt-clean and the static-analysis CI job runs
bunx oxfmt --check . across the plans/ directory, which .oxfmtrc.json
does not exclude, so oxfmt rejects it. Removing it keeps the PR CI green.
@TheGreatAxios
TheGreatAxios merged commit 76fee53 into main Oct 8, 2026
13 checks passed
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.

1 participant