Skip to content

chore: drop OPENCODE_API_KEY and REQUIRE_PAID from e2e workflow#20530

Merged
kitlangton merged 2 commits intokit/llm-server-auto-ackfrom
kit/drop-e2e-paid-requirement
Apr 1, 2026
Merged

chore: drop OPENCODE_API_KEY and REQUIRE_PAID from e2e workflow#20530
kitlangton merged 2 commits intokit/llm-server-auto-ackfrom
kit/drop-e2e-paid-requirement

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • Remove OPENCODE_API_KEY, OPENCODE_E2E_MODEL, and OPENCODE_E2E_REQUIRE_PAID from the CI workflow
  • Remove the paid-model guard and Provider import from seed-e2e.ts
  • Fixes the Windows CI failure where OPENCODE_API_KEY was unavailable

Context

All e2e tests now use the mock LLM server (via withBackendProject + withMockOpenAI) or noReply seeding. No test makes real provider calls, so the API key and paid-model enforcement are dead config.

Stacked on #20528.

Test plan

  • bun run typecheck — no new errors
  • Seed script still creates session metadata with default opencode/gpt-5-nano model

All e2e tests now use the mock LLM server or noReply seeding — no test
makes real provider calls. Remove the API key, model override, and
paid-model guard from the CI workflow and seed script.

Fixes the Windows CI failure where OPENCODE_API_KEY was unavailable.
@kitlangton kitlangton force-pushed the kit/drop-e2e-paid-requirement branch from 5650e7d to bf6907d Compare April 1, 2026 19:01
Switch 5 seedSessionQuestion tests and 1 seedSessionTask test from
real LLM calls to the mock LLM server via withMockOpenAI. Queue
deterministic tool call responses before each seed function call.

No e2e test makes real provider calls after this change.
@kitlangton kitlangton requested a review from adamdotdevin as a code owner April 1, 2026 19:17
@kitlangton kitlangton merged commit 169e276 into kit/llm-server-auto-ack Apr 1, 2026
2 checks passed
@kitlangton kitlangton deleted the kit/drop-e2e-paid-requirement branch April 1, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant