chore(release): v0.19.2

CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.19.2
+
+### Patch Changes
+
+- b4a1eaa: Fix `deploy-render.test.mjs` CPU resource expectations to match the AgentHost/exec
+  resource rebalance from #886 (agent-host 400m/1000m -> 300m/800m, exec
+  600m/1000m -> 700m/1200m). The Node toolchain tests job is path-conditional and
+  didn't run for that k8s-only change, so the stale expectations went undetected
+  until the v0.19.1 release PR touched a triggering path.
+- 96ad613: Fix the `tool.approval_context` SSE event not being handled by the frontend
+  timeline reducer, so approval context is now correctly applied to the
+  coordinator run model instead of being silently dropped.
+
 ## 0.19.1
 
 ### Patch Changes

VERSION

@@ -1 +1 @@
-0.19.1
+0.19.2

apps/web/src/__tests__/timelineReducer.test.ts

@@ -374,6 +374,43 @@ describe('timelineReducer', () => {
     expect(card.childRunId).toBeNull();
   });
 
+  // DurableToolApprovalGate emits "tool.approval_context" (not "tool.approval_required")
+  // with PascalCase keys. Verify the reducer builds a pending ApprovalRequestItem so
+  // the "Needs input:" banner can appear for HITL gates like start_preview.
+  it('tool.approval_context (PascalCase payload) creates pending approval card', () => {
+    const s = fold([
+      makeEvent('agent.turn.start', { turnId: 'T1' }),
+      makeEvent('tool.approval_context', {
+        RequestId: 'ctx-req-001',
+        ToolName: 'start_preview',
+        Url: 'sandbox-preview:3423',
+      }),
+    ]);
+    const turn = s.items[0] as TurnGroupItem;
+    expect(turn.steps).toHaveLength(1);
+    const card = turn.steps[0] as ApprovalRequestItem;
+    expect(card.kind).toBe('approval-request');
+    expect(card.requestId).toBe('ctx-req-001');
+    expect(card.toolName).toBe('start_preview');
+    expect(card.url).toBe('sandbox-preview:3423');
+    expect(card.resolved).toBe(false);
+    expect(card.childRunId).toBeNull();
+    expect(s.pendingApprovals.has('ctx-req-001')).toBe(true);
+  });
+
+  it('tool.approval_context then tool.approval_resolved resolves the card', () => {
+    const s = fold([
+      makeEvent('agent.turn.start', { turnId: 'T1' }),
+      makeEvent('tool.approval_context', { RequestId: 'ctx-req-002', ToolName: 'start_preview', Url: 'sandbox-preview:3423' }),
+      makeEvent('tool.approval_resolved', { requestId: 'ctx-req-002', approved: true, expired: false }),
+    ]);
+    const turn = s.items[0] as TurnGroupItem;
+    const card = turn.steps[0] as ApprovalRequestItem;
+    expect(card.resolved).toBe(true);
+    expect(card.resolvedScope).toBe('once');
+    expect(s.pendingApprovals.has('ctx-req-002')).toBe(false);
+  });
+
   // T-14: tool.approval_resolved with expired=true marks the card expired
   it('tool.approval_resolved (expired) marks pending approval as expired', () => {
     const s = fold([

apps/web/src/api/sse.ts

@@ -87,6 +87,7 @@ export type EventType =
   | 'tool.output'
   | 'tool.exec_result'
   | 'shell.approval_required'
+  | 'tool.approval_context'
   | 'tool.approval_required'
   | 'sandbox.selected'
   | 'sandbox.warning'

apps/web/src/hooks/useCoordinatorRunModel.ts

@@ -113,7 +113,7 @@ export function useCoordinatorRunModel(runId: string, runStatus?: string): Coord
         }
       } else if (item.kind === 'lifecycle') {
         const t = item.event.type;
-        if (t === 'tool.approval_required' || t === 'coordinator.child_approval_required' || t === 'shell.approval_required') {
+        if (t === 'tool.approval_context' || t === 'tool.approval_required' || t === 'coordinator.child_approval_required' || t === 'shell.approval_required') {
           openApprovalCount += 1; // conservative; Timeline pairs resolution for display
         }
       }

apps/web/src/timeline/reducer.ts

@@ -536,13 +536,23 @@ function processEvent(
 
     // A coordinator bubbles a child subtask's tool approval as this type; its payload carries the
     // owning child run id so approve/deny can target it (issue #196). Handled identically here.
+    // tool.approval_context is the DurableToolApprovalGate event type for HITL gates (e.g.
+    // start_preview); it carries the same requestId/toolName/url payload as tool.approval_required
+    // but uses PascalCase keys. Both must produce an ApprovalRequestItem so the "Needs input:"
+    // banner appears and the operator can allow/deny.
+    case 'tool.approval_context':
+    // falls through
     case 'tool.approval_required':
     // falls through
     case 'coordinator.child_approval_required': {
-      // Server emits camelCase (requestId, toolName); accept both for resilience.
-      const requestId = String(event.payload['request_id'] ?? event.payload['requestId'] ?? '');
-      const toolName = String(event.payload['tool_name'] ?? event.payload['toolName'] ?? '');
-      const url = event.payload['url'] != null ? String(event.payload['url']) : null;
+      // Server emits camelCase (requestId, toolName); tool.approval_context from
+      // DurableToolApprovalGate uses PascalCase (RequestId, ToolName, Url) because
+      // the payload is serialized from a C# record. Accept all forms for resilience.
+      const requestId = String(event.payload['request_id'] ?? event.payload['requestId'] ?? event.payload['RequestId'] ?? '');
+      const toolName = String(event.payload['tool_name'] ?? event.payload['toolName'] ?? event.payload['ToolName'] ?? '');
+      const url = (event.payload['url'] ?? event.payload['Url']) != null
+        ? String(event.payload['url'] ?? event.payload['Url'])
+        : null;
       const childRunId = event.payload['childRunId'] ?? event.payload['child_run_id'];
       const approvalItem: ApprovalRequestItem = {
         kind: 'approval-request',

package.json

@@ -41,5 +41,5 @@
     "@types/node": "^26.2.0",
     "playwright": "^1.62.1"
   },
-  "version": "0.19.1"
+  "version": "0.19.2"
 }