From cddc90ae3719329f23567341814fe3bcae358ff1 Mon Sep 17 00:00:00 2001 From: Vance Ingalls Date: Tue, 28 Jul 2026 01:29:58 -0700 Subject: [PATCH] fix(cli): add required gpu prop to power-state test calls trackRenderComplete requires `gpu: boolean`; the two new opt-out test calls omitted it, failing Typecheck in CI. The fix already existed on the stacked branch, so only this base branch was broken. Co-Authored-By: Claude Opus 5 (1M context) --- packages/cli/src/telemetry/events.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/cli/src/telemetry/events.test.ts b/packages/cli/src/telemetry/events.test.ts index d614f6723..029471d71 100644 --- a/packages/cli/src/telemetry/events.test.ts +++ b/packages/cli/src/telemetry/events.test.ts @@ -672,7 +672,7 @@ describe("power-state sampling respects the telemetry opt-out", () => { }); it("samples power state for a tracked render", () => { - trackRenderComplete({ durationMs: 1, fps: 30, quality: "high", docker: false }); + trackRenderComplete({ durationMs: 1, fps: 30, quality: "high", docker: false, gpu: false }); expect(getPowerState).toHaveBeenCalled(); const props = trackEvent.mock.calls.at(-1)?.[1] as Record; expect(props.on_battery).toBe(true); @@ -685,7 +685,7 @@ describe("power-state sampling respects the telemetry opt-out", () => { // guard — an opted-out install would otherwise pay two blocking // subprocess spawns per render for an event that is then discarded. shouldTrack.mockReturnValue(false); - trackRenderComplete({ durationMs: 1, fps: 30, quality: "high", docker: false }); + trackRenderComplete({ durationMs: 1, fps: 30, quality: "high", docker: false, gpu: false }); expect(getPowerState).not.toHaveBeenCalled(); }); });