Skip to content

Commit 3df306c

Browse files
committed
Fix stale goal references reintroduced by the CL-5664 merge
Rebasing onto main picked up #407's rewrite of gate-wire.ts (display- gated timers, onOperator safety net, attachApprovalBudget). Two of its new comments and one test fixture still said "goal-mode timeout" / "goal mode" — composed correctly with this branch's removal in substance (the timeout is generic plumbing either way) but stale in wording now that there's no goal subsystem to name. Reworded only; no logic changed.
1 parent 4774d52 commit 3df306c

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/tui-opentui/gate-wire.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -985,7 +985,7 @@ describe("operator.gate auto-cancel", () => {
985985
resolved = result
986986
},
987987
timeoutMs: 5,
988-
timeoutMessage: "goal mode: no answer in time",
988+
timeoutMessage: "auto-cancel: no answer in time",
989989
})
990990
expect(shell.overlayKind).toBe("operator")
991991

src/tui-opentui/gate-wire.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -493,7 +493,7 @@ export function wireGates(
493493
// entry, which is itself the single-resolve guard, so the other side is
494494
// simply a no-op once it runs.
495495
//
496-
// The goal-mode timeout is display-dependent and arms inside `open`
496+
// The auto-deny timeout is display-dependent and arms inside `open`
497497
// (below), not here: a request sitting behind others in `pending` must
498498
// not burn its timeout while the operator has never seen it. Abort is not
499499
// display-dependent — it reflects the tool having already finished or
@@ -540,7 +540,7 @@ export function wireGates(
540540
// whether the host has since moved on to a newer one.
541541
let openedGeneration: number | undefined
542542

543-
// Mirrors the permission gate: watchdog abort and the goal-mode timeout
543+
// Mirrors the permission gate: watchdog abort and the auto-deny timeout
544544
// both race an operator who may never answer, and unlike the permission
545545
// path this gate previously had no safety net at all — a queued question
546546
// behind a stuck overlay hung the run forever. The timeout is

0 commit comments

Comments
 (0)