Revert provider API-key fallback (#978, #980) (#981)

* Revert "ci: make the provider-key resolver warn-only in the daily (#976) (#980)"

This reverts commit ecb99b6.

* Revert "feat(ci): provider API-key fallback — resolve a live key before the suite (#976) (#978)"

This reverts commit 45a542a.

---------

Co-authored-by: Daniel Licnerski Borges <daniellicnerski@MacBook-Pro-de-Daniel.local>

Author: daniellicnerski1

.github/workflows/daily-stable.yml

@@ -206,36 +206,6 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
-      # Pick a LIVE key per provider before anything consumes one (#976). When a
-      # provider's key dies, collect-models records it "inactive" and the 17
-      # specs reading providers.json skip every target bound to it — silently,
-      # because a skip never trips the daily-failure gate (24 specs lost that way
-      # on 2026-07-27, #967). This probes <PROVIDER>_API_KEY, _2, _3... and
-      # re-exports the first that answers under the canonical name, so the
-      # Collect models step AND the sharded run below both inherit a validated
-      # key. It runs here rather than inside collect-models because those are
-      # separate steps: a fallback confined to collect-models would import the
-      # backup into Langflow while this job's test step still held the dead
-      # primary, and the first spec calling setupAnthropic would overwrite the
-      # working credential with it.
-      # WARN-ONLY, deliberately. An earlier revision failed the step when a
-      # configured provider had no live key; on 2026-07-27 that killed all four
-      # shards at the first step over a drained Anthropic account, blocking the
-      # dozens of specs that never touch Anthropic. Picking a live key is worth
-      # doing on its own — gating the run on it is not. A provider with no
-      # usable candidate falls through to exactly the pre-existing behaviour:
-      # collect-models records it inactive and its specs skip.
-      - name: Resolve provider keys
-        run: npx ts-node scripts/resolve-provider-keys.ts
-        env:
-          PROVIDER_KEYS_STRICT: "0"
-          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
-          OPENAI_API_KEY_2: ${{ secrets.OPENAI_API_KEY_2 }}
-          ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
-          ANTHROPIC_API_KEY_2: ${{ secrets.ANTHROPIC_API_KEY_2 }}
-          GOOGLE_API_KEY: ${{ secrets.GOOGLE_API_KEY }}
-          GOOGLE_API_KEY_2: ${{ secrets.GOOGLE_API_KEY_2 }}
-
       # Guard: the @playwright/test version (from npm) MUST equal the container
       # image tag, or the runner looks for a browser revision the image doesn't
       # ship and every test fails at launch with a cryptic error. Fail fast with

.github/workflows/pr-validation.yml

@@ -191,22 +191,6 @@ jobs:
       # Skipped when no impacted spec needs provider model data (see detect-specs
       # → needs_models). Keeps a provider billing/quota outage from reddening a
       # PR whose specs never use that provider (#915/#910/#911 class).
-      # Same key resolution the daily runs (#976) — pick a live candidate before
-      # Collect models consumes one. WARN-ONLY here: this job must not go red
-      # because an account was drained, which is the whole point of #952/#955.
-      # Gated on needs_models for the same reason Collect models is.
-      - name: Resolve provider keys
-        if: needs.detect-specs.outputs.needs_models == 'true'
-        run: npx ts-node scripts/resolve-provider-keys.ts
-        env:
-          PROVIDER_KEYS_STRICT: "0"
-          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
-          OPENAI_API_KEY_2: ${{ secrets.OPENAI_API_KEY_2 }}
-          ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
-          ANTHROPIC_API_KEY_2: ${{ secrets.ANTHROPIC_API_KEY_2 }}
-          GOOGLE_API_KEY: ${{ secrets.GOOGLE_API_KEY }}
-          GOOGLE_API_KEY_2: ${{ secrets.GOOGLE_API_KEY_2 }}
-
       - name: Collect models
         if: needs.detect-specs.outputs.needs_models == 'true'
         env:

package.json

@@ -19,8 +19,7 @@
     "coverage:summary": "ts-node scripts/coverage-summary.ts && ts-node scripts/stable-tests.ts",
     "impacted": "ts-node scripts/impacted-tests.ts",
     "validate:specs": "ts-node scripts/validate-spec-deps.ts",
-    "check:nightly-delta": "ts-node scripts/check-nightly-delta.ts",
-    "resolve:provider-keys": "ts-node scripts/resolve-provider-keys.ts"
+    "check:nightly-delta": "ts-node scripts/check-nightly-delta.ts"
   },
   "dependencies": {
     "dotenv": "^16.4.5"