Fixed build by skipping API key tests onPublic

Agent

Model

Claude Opus 4.5 (latest)

Started

Jan 19, 3:42 PM

Messages51

Downloads0

Summary

Closed, Completed

Understand
Identified failing test due to missing API keys.
Understand
Edit
Added helpers and skip logic for missing API keys.
Edit
Verify
Confirmed tests skip correctly without API keys.
Verify
Cleanup
Committed 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 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 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:

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 let me run the check to verify:

Check passes. Let me verify the test will skip correctly without API keys by temporarily testing it:

Tests properly skip when no API keys are available. Let me commit:

Fixed and pushed. The changes:

Added hasApiKey() and hasAnyApiKey() helpers for synchronous API key checking
Changed describe() to describe.skipIf(!hasAnyApiKey()) to skip the entire suite when no API keys are available
Changed beforeAll to skip providers without auth (using continue) instead of throwing errors
Updated individual tests to use it.skipIf() pattern

The tests now skip cleanly on CI without API keys, consistent with other tests in the package.