Fixed build by skipping API key tests on
Public
Fixed build by skipping API key tests on
Agent
Pi
Model
Claude Opus 4.5 (latest)
Started
Jan 19, 3:42 PM
Messages51
Downloads0
Identified failing test due to missing API keys.
UnderstandAdded helpers and skip logic for missing API keys.
EditConfirmed tests skip correctly without API keys.
VerifyCommitted and pushed the fix.
Cleanup
https://github.com/badlogic/pi-mono/actions/runs/21142602255 build fails, figure it out and fix. no api keys on the ci, tests that require api keys should not run
The user wants me to investigate a failed GitHub Actions build. Let me first check the build logs to understand what's failing.
>gh run view 21142602255 --log-failed 2>&1 | head -200build-check-test Test 2026-01-19T15:15:59.8087492Z ##[group]Run npm test
build-check-test Test 2026-01-19T15:15:59.8087756Z [36;1mnpm test[0m
build-check-test Test 2026-01-19T15:15:59.8124738Z shell: /usr/bin/bash -e {0}
build-check-test Test 2026-01-19T15:15:59.8125019Z ##[endgroup]
build-check-test Test 2026-01-19T15:15:59.9271954Z
build-check-test Test 2026-01-19T15:15:59.9272542Z > pi-monorepo@0.0.3 test
build-check-test Test 2026-01-19T15:15:59.9272968Z > npm run test --workspaces --if-present
build-check-test Test 2026-01-19T15:15:59.9273174Z
build-check-test Test 2026-01-19T15:16:00.0464208Z
build-check-test Test 2026-01-19T15:16:00.0465044Z > @mariozechner/pi-agent-core@0.49.2 test
build-check-test Test 2026-01-19T15:16:00.0465620Z > vitest --run
build-check-test Test 2026-01-19T15:16:00.0465859Z
build-check-test Test 2026-01-19T15:16:00.3584345Z
build-check-test Test 2026-01-19T15:16:00.3587592Z [1m[46m RUN [49m[22m [36mv3.2.4 [39m[90m/home/runner/work/pi-mono/pi-mono/packages/agent[39m
build-check-test Test 2026-01-19T15:16:00.3588269Z
build-check-test Test 2026-01-19T15:16:01.5299830Z [2m[90m↓[39m[22m test/bedrock-models.test.ts [2m([22m[2m1 test[22m[2m | [22m[33m1 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:01.5447377Z [32m✓[39m test/e2e.test.ts [2m([22m[2m44 tests[22m[2m | [22m[33m42 skipped[39m[2m)[22m[32m 4[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:01.5607246Z [32m✓[39m test/agent-loop.test.ts [2m([22m[2m8 tests[22m[2m)[22m[32m 46[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:02.1108916Z [32m✓[39m test/agent.test.ts [2m([22m[2m10 tests[22m[2m)[22m[32m 31[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:02.1266353Z
build-check-test Test 2026-01-19T15:16:02.1271443Z [2m Test Files [22m [1m[32m3 passed[39m[22m[2m | [22m[33m1 skipped[39m[90m (4)[39m
build-check-test Test 2026-01-19T15:16:02.1272701Z [2m Tests [22m [1m[32m20 passed[39m[22m[2m | [22m[33m43 skipped[39m[90m (63)[39m
build-check-test Test 2026-01-19T15:16:02.1273883Z [2m Start at [22m 15:16:00
build-check-test Test 2026-01-19T15:16:02.1275058Z [2m Duration [22m 1.77s[2m (transform 524ms, setup 0ms, collect 2.96s, tests 80ms, environment 1ms, prepare 434ms)[22m
build-check-test Test 2026-01-19T15:16:02.1275771Z
build-check-test Test 2026-01-19T15:16:02.2505001Z
build-check-test Test 2026-01-19T15:16:02.2505724Z > @mariozechner/pi-ai@0.49.2 test
build-check-test Test 2026-01-19T15:16:02.2506203Z > vitest --run
build-check-test Test 2026-01-19T15:16:02.2506350Z
build-check-test Test 2026-01-19T15:16:02.5484215Z
build-check-test Test 2026-01-19T15:16:02.5487931Z [1m[46m RUN [49m[22m [36mv3.2.4 [39m[90m/home/runner/work/pi-mono/pi-mono/packages/ai[39m
build-check-test Test 2026-01-19T15:16:02.5488636Z
build-check-test Test 2026-01-19T15:16:03.6695935Z [2m[90m↓[39m[22m test/unicode-surrogate.test.ts [2m([22m[2m60 tests[22m[2m | [22m[33m60 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:03.6785363Z [2m[90m↓[39m[22m test/stream.test.ts [2m([22m[2m141 tests[22m[2m | [22m[33m141 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:03.6937129Z [2m[90m↓[39m[22m test/context-overflow.test.ts [2m([22m[2m27 tests[22m[2m | [22m[33m27 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:04.3334450Z [2m[90m↓[39m[22m test/empty.test.ts [2m([22m[2m80 tests[22m[2m | [22m[33m80 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:04.3392355Z [2m[90m↓[39m[22m test/total-tokens.test.ts [2m([22m[2m25 tests[22m[2m | [22m[33m25 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:04.4615458Z [2m[90m↓[39m[22m test/image-tool-result.test.ts [2m([22m[2m28 tests[22m[2m | [22m[33m28 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:05.0043015Z [90mstdout[2m | test/cross-provider-handoff.test.ts[2m > [22m[2mCross-Provider Handoff
build-check-test Test 2026-01-19T15:16:05.0043911Z [22m[39m
build-check-test Test 2026-01-19T15:16:05.0044140Z === Generating Fixtures ===
build-check-test Test 2026-01-19T15:16:05.0044310Z
build-check-test Test 2026-01-19T15:16:05.0044318Z
build-check-test Test 2026-01-19T15:16:05.0108705Z [31m❯[39m test/cross-provider-handoff.test.ts [2m([22m[2m2 tests[22m[2m | [22m[33m2 skipped[39m[2m)[22m[32m 8[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:05.0110637Z [2m[90m↓[39m[22m Cross-Provider Handoff[2m > [22mshould have at least 2 fixtures to test handoffs
build-check-test Test 2026-01-19T15:16:05.0112017Z [2m[90m↓[39m[22m Cross-Provider Handoff[2m > [22mshould handle cross-provider handoffs for each target
build-check-test Test 2026-01-19T15:16:05.1685987Z [2m[90m↓[39m[22m test/tool-call-without-result.test.ts [2m([22m[2m20 tests[22m[2m | [22m[33m20 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:05.4994674Z [2m[90m↓[39m[22m test/tokens.test.ts [2m([22m[2m20 tests[22m[2m | [22m[33m20 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:05.6703780Z [2m[90m↓[39m[22m test/abort.test.ts [2m([22m[2m21 tests[22m[2m | [22m[33m21 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:06.0174181Z [32m✓[39m test/google-gemini-cli-claude-thinking-header.test.ts [2m([22m[2m2 tests[22m[2m)[22m[32m 9[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:06.0334344Z [2m[90m↓[39m[22m test/anthropic-tool-name-normalization.test.ts [2m([22m[2m4 tests[22m[2m | [22m[33m4 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:06.5020709Z [90mstdout[2m | test/bedrock-models.test.ts[2m > [22m[2mAmazon Bedrock Models[2m > [22m[2mshould get all available Bedrock models
build-check-test Test 2026-01-19T15:16:06.5022246Z [22m[39mFound 55 Bedrock models
build-check-test Test 2026-01-19T15:16:06.5022541Z
build-check-test Test 2026-01-19T15:16:06.5032679Z [32m✓[39m test/bedrock-models.test.ts [2m([22m[2m1 test[22m[2m)[22m[32m 3[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:06.8357721Z [32m✓[39m test/google-gemini-cli-empty-stream.test.ts [2m([22m[2m1 test[22m[2m)[22m[33m 510[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:06.8359998Z [33m[2m✓[22m[39m google-gemini-cli empty stream retry[2m > [22mretries empty SSE responses without duplicate start [33m 509[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:06.9480971Z [2m[90m↓[39m[22m test/openai-responses-reasoning-replay-e2e.test.ts [2m([22m[2m1 test[22m[2m | [22m[33m1 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:07.2648391Z [32m✓[39m test/google-shared-gemini3-unsigned-tool-call.test.ts [2m([22m[2m1 test[22m[2m)[22m[32m 5[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:07.3057102Z [2m[90m↓[39m[22m test/xhigh.test.ts [2m([22m[2m3 tests[22m[2m | [22m[33m3 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:07.5735855Z [32m✓[39m test/google-thinking-signature.test.ts [2m([22m[2m5 tests[22m[2m)[22m[32m 3[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:07.6494774Z [32m✓[39m test/google-gemini-cli-retry-delay.test.ts [2m([22m[2m4 tests[22m[2m)[22m[32m 8[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:07.9994420Z [2m[90m↓[39m[22m test/zen.test.ts [2m([22m[2m27 tests[22m[2m | [22m[33m27 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:11.9931939Z [32m✓[39m test/openai-codex-stream.test.ts [2m([22m[2m3 tests[22m[2m)[22m[33m 7022[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:11.9935628Z [33m[2m✓[22m[39m openai-codex streaming[2m > [22msets conversation_id/session_id headers and prompt_cache_key when sessionId is provided [33m 7012[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:12.0092106Z
build-check-test Test 2026-01-19T15:16:12.0093237Z [31m⎯⎯⎯⎯⎯⎯[39m[1m[41m Failed Suites 1 [49m[22m[31m⎯⎯⎯⎯⎯⎯⎯[39m
build-check-test Test 2026-01-19T15:16:12.0093684Z
build-check-test Test 2026-01-19T15:16:12.0095582Z [41m[1m FAIL [22m[49m test/cross-provider-handoff.test.ts[2m > [22mCross-Provider Handoff
build-check-test Test 2026-01-19T15:16:12.0112162Z [31m[1mError[22m: Missing auth for anthropic[39m
build-check-test Test 2026-01-19T15:16:12.0113468Z [36m [2m❯[22m test/cross-provider-handoff.test.ts:[2m275:11[22m[39m
build-check-test Test 2026-01-19T15:16:12.0185232Z [90m273| [39m [35mconst[39m apiKey [33m=[39m [35mawait[39m [34mgetApiKey[39m(pair[33m.[39mprovider)[33m;[39m
build-check-test Test 2026-01-19T15:16:12.0186625Z [90m274| [39m [35mif[39m ([33m![39mapiKey) {
build-check-test Test 2026-01-19T15:16:12.0189833Z [90m275| [39m [35mthrow[39m [35mnew[39m [33mError[39m([32m`Missing auth for [39m[36m${[39mpair[33m.[39mprovider[36m}[39m[32m`[39m)[33m;[39m
build-check-test Test 2026-01-19T15:16:12.0194345Z [90m | [39m [31m^[39m
build-check-test Test 2026-01-19T15:16:12.0194698Z
build-check-test Test 2026-01-19T15:16:12.0203881Z [90m276| [39m }
build-check-test Test 2026-01-19T15:16:12.0204258Z [90m277| [39m
build-check-test Test 2026-01-19T15:16:12.0204448Z
build-check-test Test 2026-01-19T15:16:12.0204854Z [31m[2m⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯[22m[39m
build-check-test Test 2026-01-19T15:16:12.0205212Z
build-check-test Test 2026-01-19T15:16:12.0206171Z [2m Test Files [22m [1m[31m1 failed[39m[22m[2m | [22m[1m[32m7 passed[39m[22m[2m | [22m[33m13 skipped[39m[90m (21)[39m
build-check-test Test 2026-01-19T15:16:12.0207525Z [2m Tests [22m [1m[32m17 passed[39m[22m[2m | [22m[33m459 skipped[39m[90m (476)[39m
build-check-test Test 2026-01-19T15:16:12.0208297Z [2m Start at [22m 15:16:02
build-check-test Test 2026-01-19T15:16:12.0209452Z [2m Duration [22m 9.46s[2m (transform 936ms, setup 0ms, collect 7.37s, tests 7.57s, environment 4ms, prepare 1.93s)[22m
build-check-test Test 2026-01-19T15:16:12.0210652Z
build-check-test Test 2026-01-19T15:16:12.0221810Z
build-check-test Test 2026-01-19T15:16:12.0251691Z ##[error]Error: Missing auth for anthropic
build-check-test Test ❯ test/cross-provider-handoff.test.ts:275:11
build-check-test Test
build-check-test Test
build-check-test Test 2026-01-19T15:16:12.1374573Z npm error Lifecycle script `test` failed with error:
build-check-test Test 2026-01-19T15:16:12.1375197Z npm error code 1
build-check-test Test 2026-01-19T15:16:12.1375773Z npm error path /home/runner/work/pi-mono/pi-mono/packages/ai
build-check-test Test 2026-01-19T15:16:12.1376445Z npm error workspace @mariozechner/pi-ai@0.49.2
build-check-test Test 2026-01-19T15:16:12.1377200Z npm error location /home/runner/work/pi-mono/pi-mono/packages/ai
build-check-test Test 2026-01-19T15:16:12.1377766Z npm error command failed
build-check-test Test 2026-01-19T15:16:12.1378189Z npm error command sh -c vitest --run
build-check-test Test 2026-01-19T15:16:12.1378478Z
build-check-test Test 2026-01-19T15:16:12.1385553Z
build-check-test Test 2026-01-19T15:16:12.1385915Z > @mariozechner/pi-coding-agent@0.49.2 test
build-check-test Test 2026-01-19T15:16:12.1386350Z > vitest --run
build-check-test Test 2026-01-19T15:16:12.1386528Z
build-check-test Test 2026-01-19T15:16:12.4348807Z
build-check-test Test 2026-01-19T15:16:12.4352149Z [1m[46m RUN [49m[22m [36mv3.2.4 [39m[90m/home/runner/work/pi-mono/pi-mono/packages/coding-agent[39m
build-check-test Test 2026-01-19T15:16:12.4352853Z
build-check-test Test 2026-01-19T15:16:14.2486983Z [32m✓[39m test/model-registry.test.ts [2m([22m[2m26 tests[22m[2m)[22m[33m 538[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:14.5985581Z [32m✓[39m test/tools.test.ts [2m([22m[2m42 tests[22m[2m)[22m[33m 1217[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:14.5988647Z [33m[2m✓[22m[39m Coding Agent Tools[2m > [22mbash tool[2m > [22mshould respect timeout [33m 1006[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:15.0480048Z [32m✓[39m test/compaction.test.ts [2m([22m[2m21 tests[22m[2m | [22m[33m2 skipped[39m[2m)[22m[32m 74[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:15.4098392Z [32m✓[39m test/prompt-templates.test.ts [2m([22m[2m73 tests[22m[2m)[22m[32m 18[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:15.5367736Z [90mstdout[2m | test/extensions-discovery.test.ts[2m > [22m[2mextensions discovery[2m > [22m[2mresolves 3rd party npm dependencies (chalk)
build-check-test Test 2026-01-19T15:16:15.5369128Z [22m[39m✓ chalk-logger extension loaded
build-check-test Test 2026-01-19T15:16:15.5369500Z
build-check-test Test 2026-01-19T15:16:15.6702629Z [32m✓[39m test/session-manager/tree-traversal.test.ts [2m([22m[2m28 tests[22m[2m)[22m[32m 23[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:15.6914312Z [32m✓[39m test/extensions-discovery.test.ts [2m([22m[2m28 tests[22m[2m)[22m[33m 573[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:16.2113601Z [32m✓[39m test/skills.test.ts [2m([22m[2m29 tests[22m[2m)[22m[32m 90[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:17.2832990Z [2m[90m↓[39m[22m test/agent-session-tree-navigation.test.ts [2m([22m[2m10 tests[22m[2m | [22m[33m10 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:17.4007986Z [2m[90m↓[39m[22m test/compaction-extensions.test.ts [2m([22m[2m8 tests[22m[2m | [22m[33m8 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:17.4040897Z [32m✓[39m test/extensions-runner.test.ts [2m([22m[2m15 tests[22m[2m)[22m[33m 362[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:17.5843784Z [32m✓[39m test/session-manager/build-context.test.ts [2m([22m[2m14 tests[22m[2m)[22m[32m 13[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:17.7189927Z [2m[90m↓[39m[22m test/rpc.test.ts [2m([22m[2m13 tests[22m[2m | [22m[33m13 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:17.9231468Z [32m✓[39m test/plan-mode-utils.test.ts [2m([22m[2m33 tests[22m[2m)[22m[32m 23[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:18.2327607Z [32m✓[39m test/image-processing.test.ts [2m([22m[2m8 tests[22m[2m)[22m[32m 191[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:18.2835473Z [32m✓[39m test/model-resolver.test.ts [2m([22m[2m19 tests[22m[2m)[22m[32m 23[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:18.4059911Z [32m✓[39m test/args.test.ts [2m([22m[2m36 tests[22m[2m)[22m[32m 13[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:19.2592708Z [2m[90m↓[39m[22m test/compaction-thinking-model.test.ts [2m([22m[2m3 tests[22m[2m | [22m[33m3 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:19.3046185Z [2m[90m↓[39m[22m test/agent-session-compaction.test.ts [2m([22m[2m5 tests[22m[2m | [22m[33m5 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:19.4395815Z [32m✓[39m test/agent-session-concurrent.test.ts [2m([22m[2m4 tests[22m[2m)[22m[32m 48[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:19.5400058Z [32m✓[39m test/session-manager/labels.test.ts [2m([22m[2m8 tests[22m[2m)[22m[32m 12[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:19.8717538Z [32m✓[39m test/settings-manager.test.ts [2m([22m[2m6 tests[22m[2m)[22m[32m 14[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:20.0097665Z [32m✓[39m test/session-selector-path-delete.test.ts [2m([22m[2m5 tests[22m[2m)[22m[32m 28[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:20.3972521Z [32m✓[39m test/session-manager/file-operations.test.ts [2m([22m[2m13 tests[22m[2m)[22m[32m 45[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:20.7054746Z [32m✓[39m test/clipboard-image.test.ts [2m([22m[2m4 tests[22m[2m)[22m[32m 14[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:20.8695599Z [2m[90m↓[39m[22m test/agent-session-branching.test.ts [2m([22m[2m3 tests[22m[2m | [22m[33m3 skipped[39m[2m)[22m
build-check-test Test 2026-01-19T15:16:21.1866509Z [32m✓[39m test/block-images.test.ts [2m([22m[2m8 tests[22m[2m)[22m[32m 55[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:21.2040242Z [32m✓[39m test/extensions-input-event.test.ts [2m([22m[2m8 tests[22m[2m)[22m[32m 188[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:21.2624506Z [32m✓[39m test/session-selector-search.test.ts [2m([22m[2m5 tests[22m[2m)[22m[32m 5[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:21.5741938Z [32m✓[39m test/frontmatter.test.ts [2m([22m[2m8 tests[22m[2m)[22m[32m 25[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:21.6245786Z [32m✓[39m test/truncate-to-width.test.ts [2m([22m[2m6 tests[22m[2m)[22m[32m 38[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:21.8481603Z [32m✓[39m test/compaction-extensions-example.test.ts [2m([22m[2m2 tests[22m[2m)[22m[32m 4[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:21.9161892Z [32m✓[39m test/session-manager/migration.test.ts [2m([22m[2m2 tests[22m[2m)[22m[32m 4[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:22.1957303Z [32m✓[39m test/session-manager/save-entry.test.ts [2m([22m[2m1 test[22m[2m)[22m[32m 6[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:22.5900085Z [32m✓[39m test/system-prompt.test.ts [2m([22m[2m3 tests[22m[2m)[22m[32m 18[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:22.8027879Z [32m✓[39m test/sdk-skills.test.ts [2m([22m[2m3 tests[22m[2m)[22m[32m 33[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:22.9420452Z [32m✓[39m test/interactive-mode-status.test.ts [2m([22m[2m2 tests[22m[2m)[22m[32m 8[2mms[22m[39m
build-check-test Test 2026-01-19T15:16:22.9624868Z
build-check-test Test 2026-01-19T15:16:22.9630800Z [2m Test Files [22m [1m[32m30 passed[39m[22m[2m | [22m[33m6 skipped[39m[90m (36)[39m
build-check-test Test 2026-01-19T15:16:22.9632371Z [2m Tests [22m [1m[32m458 passed[39m[22m[2m | [22m[33m44 skipped[39m[90m (502)[39m
build-check-test Test 2026-01-19T15:16:22.9633785Z [2m Start at [22m 15:16:12
build-check-test Test 2026-01-19T15:16:22.9635797Z [2m Duration [22m 10.53s[2m (transform 1.99s, setup 0ms, collect 17.39s, tests 3.70s, environment 8ms, prepare 3.63s)[22m
build-check-test Test 2026-01-19T15:16:22.9636634Z
build-check-test Test 2026-01-19T15:16:23.0866292Z
build-check-test Test 2026-01-19T15:16:23.0866989Z > @mariozechner/pi-tui@0.49.2 test
build-check-test Test 2026-01-19T15:16:23.0867669Z > node --test --import tsx test/*.test.ts
build-check-test Test 2026-01-19T15:16:23.0867968Z
build-check-test Test 2026-01-19T15:16:23.1386290Z TAP version 13
build-check-test Test 2026-01-19T15:16:23.4816145Z # Result: null
build-check-test Test 2026-01-19T15:16:23.4853684Z # Result: null
build-check-test Test 2026-01-19T15:16:23.4872795Z # Result: {
build-check-test Test 2026-01-19T15:16:23.4873568Z # items: [
build-check-test Test 2026-01-19T15:16:23.4879962Z # { value: '/bin.usr-is-merged/', label: 'bin.usr-is-merged/' },
build-check-test Test 2026-01-19T15:16:23.4886659Z # { value: '/bin/', label: 'bin/' },
build-check-test Test 2026-01-19T15:16:23.4892326Z # { value: '/boot/', label: 'boot/' },
build-check-test Test 2026-01-19T15:16:23.4897302Z # { value: '/dev/', label: 'dev/' },
build-check-test Test 2026-01-19T15:16:23.4897885Z # { value: '/etc/', label: 'etc/' },
build-check-test Test 2026-01-19T15:16:23.4898384Z # { value: '/home/', label: 'home/' },
build-check-test Test 2026-01-19T15:16:23.4899007Z # { value: '/imagegeneration/', label: 'imagegeneration/' },
build-check-test Test 2026-01-19T15:16:23.4899774Z # { value: '/lib.usr-is-merged/', label: 'lib.usr-is-merged/' },
build-check-test Test 2026-01-19T15:16:23.4900398Z # { value: '/lib/', label: 'lib/' },
build-check-test Test 2026-01-19T15:16:23.4929487Z # { value: '/lib32/', label: 'lib32/' },
build-check-test Test 2026-01-19T15:16:23.4930599Z # { value: '/lib64/', label: 'lib64/' },
build-check-test Test 2026-01-19T15:16:23.4931598Z # { value: '/lost+found/', label: 'lost+found/' },
build-check-test Test 2026-01-19T15:16:23.4932376Z # { value: '/media/', label: 'media/' },
build-check-test Test 2026-01-19T15:16:23.4933071Z # { value: '/mnt/', label: 'mnt/' },
build-check-test Test 2026-01-19T15:16:23.4933548Z # { value: '/opt/', label: 'opt/' },
build-check-test Test 2026-01-19T15:16:23.4934212Z # { value: '/proc/', label: 'proc/' },
build-check-test Test 2026-01-19T15:16:23.4934883Z # { value: '/root/', label: 'root/' },
build-check-test Test 2026-01-19T15:16:23.4935573Z # { value: '/run/', label: 'run/' },
build-check-test Test 2026-01-19T15:16:23.4936404Z # { value: '/sbin.usr-is-merged/', label: 'sbin.usr-is-merged/' },
build-check-test Test 2026-01-19T15:16:23.4937052Z # { value: '/sbin/', label: 'sbin/' },
build-check-test Test 2026-01-19T15:16:23.4937530Z # { value: '/snap/', label: 'snap/' },
build-check-test Test 2026-01-19T15:16:23.4938016Z # { value: '/srv/', label: 'srv/' },
build-check-test Test 2026-01-19T15:16:23.4938520Z # { value: '/sys/', label: 'sys/' },
build-check-test Test 2026-01-19T15:16:23.4938987Z # { value: '/tmp/', label: 'tmp/' },
build-check-test Test 2026-01-19T15:16:23.4939462Z # { value: '/usr/', label: 'usr/' },
build-check-test Test 2026-01-19T15:16:23.4939924Z # { value: '/var/', label: 'var/' }
build-check-test Test 2026-01-19T15:16:23.4940285Z # ],
build-check-test Test 2026-01-19T15:16:23.4940545Z # prefix: '/'
build-check-test Test 2026-01-19T15:16:23.4940818Z # }
build-check-test Test 2026-01-19T15:16:23.5013493Z # Subtest: CombinedAutocompleteProvider
build-check-test Test 2026-01-19T15:16:23.5014035Z # Subtest: extractPathPrefix
build-check-test Test 2026-01-19T15:16:23.5014866Z # Subtest: extracts / from 'hey /' when forced
build-check-test Test 2026-01-19T15:16:23.5015465Z ok 1 - extracts / from 'hey /' when forced
build-check-test Test 2026-01-19T15:16:23.5015906Z ---
build-check-test Test 2026-01-19T15:16:23.5016235Z duration_ms: 19.967065
build-check-test Test 2026-01-19T15:16:23.5024401Z type: 'test'
build-check-test Test 2026-01-19T15:16:23.5024705Z ...
build-check-test Test 2026-01-19T15:16:23.5025080Z # Subtest: extracts /A from '/A' when forced
build-check-test Test 2026-01-19T15:16:23.5025613Z ok 2 - extracts /A from '/A' when forcedThe issue is in test/cross-provider-handoff.test.ts in packages/ai. It's throwing an error "Missing auth for anthropic" because there's no API key available on CI. The test should skip when no API keys are available, not throw an error.
Let me look at that test file to understand how it's structured.
The failure is in packages/ai/test/cross-provider-handoff.test.ts - it throws Error: Missing auth for anthropic instead of skipping when no API keys are available. Let me look at the test:
The issue is in the beforeAll block. At line 273-275:
const apiKey = await getApiKey(pair.provider);if (!apiKey) { throw new Error(`Missing auth for ${pair.provider}`);}This throws an error when no API key is available instead of skipping. For CI without API keys, the test should skip entirely if no providers have keys available.
Looking at other tests in the same package that require API keys, they typically skip when keys aren't available. Let me check how other tests handle this:
The problem is in the beforeAll block (lines 273-275). It throws an error when no API key is found instead of skipping. Let me check how other tests handle this:
>grep -n "skipIf\|skip.*api\|runIf\|getEnvApiKey" packages/ai/test/*.ts | head -40packages/ai/test/abort.test.ts:101: describe.skipIf(!process.env.GEMINI_API_KEY)("Google Provider Abort", () => {
packages/ai/test/abort.test.ts:113: describe.skipIf(!process.env.OPENAI_API_KEY)("OpenAI Completions Provider Abort", () => {
packages/ai/test/abort.test.ts:130: describe.skipIf(!process.env.OPENAI_API_KEY)("OpenAI Responses Provider Abort", () => {
packages/ai/test/abort.test.ts:142: describe.skipIf(!process.env.ANTHROPIC_OAUTH_TOKEN)("Anthropic Provider Abort", () => {
packages/ai/test/abort.test.ts:154: describe.skipIf(!process.env.MISTRAL_API_KEY)("Mistral Provider Abort", () => {
packages/ai/test/abort.test.ts:166: describe.skipIf(!process.env.MINIMAX_API_KEY)("MiniMax Provider Abort", () => {
packages/ai/test/abort.test.ts:178: describe.skipIf(!process.env.AI_GATEWAY_API_KEY)("Vercel AI Gateway Provider Abort", () => {
packages/ai/test/abort.test.ts:192: it.skipIf(!geminiCliToken)("should abort mid-stream", { retry: 3 }, async () => {
packages/ai/test/abort.test.ts:197: it.skipIf(!geminiCliToken)("should handle immediate abort", { retry: 3 }, async () => {
packages/ai/test/abort.test.ts:204: it.skipIf(!openaiCodexToken)("should abort mid-stream", { retry: 3 }, async () => {
packages/ai/test/abort.test.ts:209: it.skipIf(!openaiCodexToken)("should handle immediate abort", { retry: 3 }, async () => {
packages/ai/test/abort.test.ts:215: describe.skipIf(!hasBedrockCredentials())("Amazon Bedrock Provider Abort", () => {
packages/ai/test/anthropic-tool-name-normalization.test.ts:26:describe.skipIf(!oauthToken)("Anthropic OAuth tool name normalization", () => {
packages/ai/test/context-overflow.test.ts:101: describe.skipIf(!process.env.ANTHROPIC_API_KEY)("Anthropic (API Key)", () => {
packages/ai/test/context-overflow.test.ts:113: describe.skipIf(!process.env.ANTHROPIC_OAUTH_TOKEN)("Anthropic (OAuth)", () => {
packages/ai/test/context-overflow.test.ts:132: it.skipIf(!githubCopilotToken)(
packages/ai/test/context-overflow.test.ts:147: it.skipIf(!githubCopilotToken)(
packages/ai/test/context-overflow.test.ts:167: describe.skipIf(!process.env.OPENAI_API_KEY)("OpenAI Completions", () => {
packages/ai/test/context-overflow.test.ts:180: describe.skipIf(!process.env.OPENAI_API_KEY)("OpenAI Responses", () => {
packages/ai/test/context-overflow.test.ts:197: describe.skipIf(!process.env.GEMINI_API_KEY)("Google", () => {
packages/ai/test/context-overflow.test.ts:215: it.skipIf(!geminiCliToken)(
packages/ai/test/context-overflow.test.ts:237: it.skipIf(!antigravityToken)(
packages/ai/test/context-overflow.test.ts:252: it.skipIf(!antigravityToken)(
packages/ai/test/context-overflow.test.ts:274: it.skipIf(!openaiCodexToken)(
packages/ai/test/context-overflow.test.ts:293: describe.skipIf(!hasBedrockCredentials())("Amazon Bedrock", () => {
packages/ai/test/context-overflow.test.ts:309: describe.skipIf(!process.env.XAI_API_KEY)("xAI", () => {
packages/ai/test/context-overflow.test.ts:326: describe.skipIf(!process.env.GROQ_API_KEY)("Groq", () => {
packages/ai/test/context-overflow.test.ts:343: describe.skipIf(!process.env.CEREBRAS_API_KEY)("Cerebras", () => {
packages/ai/test/context-overflow.test.ts:362: describe.skipIf(!process.env.ZAI_API_KEY)("z.ai", () => {
packages/ai/test/context-overflow.test.ts:390: describe.skipIf(!process.env.MISTRAL_API_KEY)("Mistral", () => {
packages/ai/test/context-overflow.test.ts:406: describe.skipIf(!process.env.MINIMAX_API_KEY)("MiniMax", () => {
packages/ai/test/context-overflow.test.ts:421: describe.skipIf(!process.env.AI_GATEWAY_API_KEY)("Vercel AI Gateway", () => {
packages/ai/test/context-overflow.test.ts:437: describe.skipIf(!process.env.OPENROUTER_API_KEY)("OpenRouter", () => {
packages/ai/test/context-overflow.test.ts:509: describe.skipIf(!ollamaInstalled)("Ollama (local)", () => {
packages/ai/test/context-overflow.test.ts:603: describe.skipIf(!lmStudioRunning)("LM Studio (local)", () => {
packages/ai/test/context-overflow.test.ts:638: describe.skipIf(!llamaCppRunning)("llama.cpp (local)", () => {
packages/ai/test/cross-provider-handoff.test.ts:29:import { completeSimple, getEnvApiKey } from "../src/stream.js";
packages/ai/test/cross-provider-handoff.test.ts:116: return getEnvApiKey(provider);
packages/ai/test/empty.test.ts:145: describe.skipIf(!process.env.GEMINI_API_KEY)("Google Provider Empty Messages", () => {
packages/ai/test/empty.test.ts:165: describe.skipIf(!process.env.OPENAI_API_KEY)("OpenAI Completions Provider Empty Messages", () => {I see. The pattern is to use describe.skipIf(!) or it.skipIf(!) to skip tests when API keys aren't available.
The cross-provider-handoff test needs to be fixed to skip when no API keys are available, rather than throwing an error in beforeAll.
The fix should:
Looking at the test structure, the cleanest approach would be:
Let me rewrite this properly.
The pattern is to use describe.skipIf() or it.skipIf() to skip tests when API keys aren't available. The cross-provider-handoff test needs to be fixed to filter providers without keys and skip if none are available:
Now I need to:
describe.skipIf(!hasAnyApiKey())Now I need to change the beforeAll to skip providers without API keys instead of throwing an error.
Now let me also update the test assertions to handle the case where there are fewer than 2 fixtures available. Currently it throws an error if there are less than 2 fixtures.
Now let me also fix the second test to skip properly and not throw when there are fewer than 2 fixtures.
Now let me run npm run check to verify everything is correct.
Now let me run the check to verify:
>cd packages/ai && npm run check 2>&1npm error Lifecycle script `check` failed with error:
npm error workspace @mariozechner/pi-ai@0.49.2
npm error location /Users/badlogic/workspaces/pi-mono/packages/ai
npm error Missing script: "check"
npm error
npm error To see a list of scripts, run:
npm error npm run --workspace=@mariozechner/pi-ai@0.49.2
Command exited with code 1Need to run from root or check what scripts are available.
>cd /Users/badlogic/workspaces/pi-mono && npm run check 2>&1> pi-monorepo@0.0.3 check
> biome check --write . && tsgo --noEmit && npm run check -w @mariozechner/pi-web-ui
Checked 408 files in 264ms. Fixed 1 file.
> @mariozechner/pi-web-ui@0.49.2 check
> biome check --write . && tsc --noEmit && cd example && biome check --write . && tsc --noEmit
Checked 73 files in 37ms. No fixes applied.
Checked 3 files in 8ms. No fixes applied.The check passes. Let me also run just the cross-provider-handoff test to make sure it skips properly without API keys. Actually, since I'm on a machine that may have API keys set, let me verify by checking if the tests pass in a way that will also work on CI.
Let me verify the test file looks correct by reading the relevant parts.
Check passes. Let me verify the test will skip correctly without API keys by temporarily testing it:
>cd packages/ai && ANTHROPIC_API_KEY= OPENAI_API_KEY= GEMINI_API_KEY= npm test -- --run test/cross-provider-handoff.test.ts 2>&1 | tail -30> @mariozechner/pi-ai@0.49.2 test
> vitest --run --run test/cross-provider-handoff.test.ts
file:///Users/badlogic/workspaces/pi-mono/node_modules/vitest/dist/chunks/cac.Cb-PYCCB.js:1267
throw new Error(`Expected a single value for option "${command}", received [${received}]`);
^
Error: Expected a single value for option "--run", received [true, true]
at transform (file:///Users/badlogic/workspaces/pi-mono/node_modules/vitest/dist/chunks/cac.Cb-PYCCB.js:1267:10)
at setDotProp (file:///Users/badlogic/workspaces/pi-mono/node_modules/vitest/dist/chunks/cac.Cb-PYCCB.js:207:22)
at CAC.mri (file:///Users/badlogic/workspaces/pi-mono/node_modules/vitest/dist/chunks/cac.Cb-PYCCB.js:591:9)
at CAC.parse (file:///Users/badlogic/workspaces/pi-mono/node_modules/vitest/dist/chunks/cac.Cb-PYCCB.js:508:27)
at file:///Users/badlogic/workspaces/pi-mono/node_modules/vitest/dist/cli.js:27:13
at ModuleJob.run (node:internal/modules/esm/module_job:413:25)
at process.processTicksAndRejections (node:internal/process/task_queues:103:5)
at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:654:26)
at async asyncRunEntryPointWithESMLoader (node:internal/modules/run_main:101:5)
Node.js v25.2.1
npm error Lifecycle script `test` failed with error:
npm error code 1
npm error path /Users/badlogic/workspaces/pi-mono/packages/ai
npm error workspace @mariozechner/pi-ai@0.49.2
npm error location /Users/badlogic/workspaces/pi-mono/packages/ai
npm error command failed
npm error command sh -c vitest --run --run test/cross-provider-handoff.test.tsOops, duplicated --run. Let me fix that.
>cd packages/ai && ANTHROPIC_API_KEY= OPENAI_API_KEY= GEMINI_API_KEY= npx vitest --run test/cross-provider-handoff.test.ts 2>&1 | tail -20[32m- 0[39m
[31m+ 1[39m
❯ test/cross-provider-handoff.test.ts:442:28
440| }
441|
442| expect(failures.length).toBe(0);
| ^
443| },
444| 600000,
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯
Test Files 1 failed (1)
Tests 1 failed | 1 passed (2)
Start at 16:44:18
Duration 200.76s (transform 226ms, setup 0ms, collect 422ms, tests 200.05s, environment 0ms, prepare 44ms)Hmm, the test is not skipping because I have API keys in my environment (likely oauth tokens or other env vars). Let me unset more environment variables to simulate CI:
>cd packages/ai && env -i HOME="$HOME" PATH="$PATH" NODE_PATH="$NODE_PATH" npx vitest --run test/cross-provider-handoff.test.ts 2>&1 | tail -20RUN v3.2.4 /Users/badlogic/workspaces/pi-mono/packages/ai
↓ test/cross-provider-handoff.test.ts (2 tests | 2 skipped)
Test Files 1 skipped (1)
Tests 2 skipped (2)
Start at 16:47:43
Duration 630ms (transform 239ms, setup 0ms, collect 389ms, tests 0ms, environment 0ms, prepare 53ms)The tests are properly skipped when no API keys are available. Let me now commit these changes.
Tests properly skip when no API keys are available. Let me commit:
>cd /Users/badlogic/workspaces/pi-mono && git add packages/ai/test/cross-provider-handoff.test.ts && git commit -m "fix(ai): skip cross-provider-handoff tests when no API keys available
Tests were throwing errors instead of skipping on CI where no API keys
are configured. Now uses describe.skipIf() and it.skipIf() patterns
consistent with other tests in the package." && git pushRunning formatting, linting, and type checking...
> pi-monorepo@0.0.3 check
> biome check --write . && tsgo --noEmit && npm run check -w @mariozechner/pi-web-ui
Checked 408 files in 254ms. No fixes applied.
> @mariozechner/pi-web-ui@0.49.2 check
> biome check --write . && tsc --noEmit && cd example && biome check --write . && tsc --noEmit
Checked 73 files in 37ms. No fixes applied.
Checked 3 files in 8ms. No fixes applied.
✅ All pre-commit checks passed!
[main 68352a42] fix(ai): skip cross-provider-handoff tests when no API keys available
1 file changed, 123 insertions(+), 100 deletions(-)
To https://github.com/badlogic/pi-mono.git
2f4b510c..68352a42 main -> mainDone. The fix has been committed and pushed. The changes:
hasApiKey() helper for synchronous API key checking (env vars only)hasAnyApiKey() to check if any provider has keys availabledescribe() to describe.skipIf(!hasAnyApiKey()) to skip the entire suite when no API keys existbeforeAll to log and continue instead of throw when a provider lacks authit.skipIf() and return early instead of throwingNow the tests will skip cleanly on CI where no API keys are configured.
Fixed and pushed. The changes:
hasApiKey() and hasAnyApiKey() helpers for synchronous API key checkingdescribe() to describe.skipIf(!hasAnyApiKey()) to skip the entire suite when no API keys are availablebeforeAll to skip providers without auth (using continue) instead of throwing errorsit.skipIf() patternThe tests now skip cleanly on CI without API keys, consistent with other tests in the package.
const apiKey = await getApiKey(pair.provider);if (!apiKey) { throw new Error(`Missing auth for ${pair.provider}`);}