Skip to content

ci(ai): Route AI integration tests through OpenRouter - #6979

Merged
HazAT merged 3 commits into
masterfrom
ci/ai-openrouter-api-key
Jul 31, 2026
Merged

ci(ai): Route AI integration tests through OpenRouter#6979
HazAT merged 3 commits into
masterfrom
ci/ai-openrouter-api-key

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Wire the AI integration harness caller to OpenRouter so Anthropic-compatible tests no longer need a direct Anthropic key.

Action input

.github/workflows/ai-integration-test.yml now passes openrouter-api-key: ${{ secrets.OPENROUTER_API_KEY }} instead of anthropic-api-key. The harness remains pinned to the merged OpenRouter migration revision (318143d106d4a847dfac3804bb2663d1b852cf28) for pipeline stability and supply-chain safety.

Companion: getsentry/testing-ai-sdk-integrations#154

Secrets

This workflow uses the AI Integrations Tests environment. Add OPENROUTER_API_KEY there (environment secret), not only as a plain repo secret, then ANTHROPIC_API_KEY can be removed after this ships.

Pass openrouter-api-key instead of anthropic-api-key and pin the
testing-ai-sdk-integrations action to the OpenRouter migration commit.

Companion: getsentry/testing-ai-sdk-integrations#154
Co-Authored-By: Daniel Griesser <dgriesser@sentry.io>
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

94982 passed | ⏭️ 6343 skipped | Total: 101325 | Pass Rate: 93.74% | Execution Time: 347m 38s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2513 uncovered lines.
❌ Project coverage is 89.88%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.91%    89.88%    -0.03%
==========================================
  Files          193       193         —
  Lines        24832     24832         —
  Branches      8912      8912         —
==========================================
+ Hits         22326     22319        -7
- Misses        2506      2513        +7
- Partials      1417      1417         —

Generated by Codecov Action

Drop the commit pin for the internal AI integration harness action.

Co-Authored-By: Daniel Griesser <dgriesser@sentry.io>
Comment thread .github/workflows/ai-integration-test.yml Outdated
@HazAT
HazAT marked this pull request as ready for review July 31, 2026 18:17
@HazAT
HazAT requested a review from a team as a code owner July 31, 2026 18:17

@ericapisani ericapisani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changeover to the openrouter api key looks fine but I think we should undo the change that was introduced to what version of testing-ai-sdk-integrations we test against in the pipeline

Co-Authored-By: Daniel Griesser <dgriesser@sentry.io>
Comment thread .github/workflows/ai-integration-test.yml
@HazAT
HazAT merged commit 10b4f5b into master Jul 31, 2026
145 checks passed
@HazAT
HazAT deleted the ci/ai-openrouter-api-key branch July 31, 2026 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants