report-lane-outcome: record a key-to-subscription fallback as a lane-record token #806

Activity

1. github-actions

   added

   needs-triageNot yet classified. Floor until a type and one priority tier are set.Not yet classified. Floor until a type and one priority tier are set.

   on Oct 20, 2026
2. kyle-sexton commented on Oct 20, 2026

   

   kyle-sexton

   on Oct 20, 2026

   MemberAuthor

   More actions

   Agent-ready brief (triaged 2026-10-20)

   • Decision: Add an optional fallback-reason input to report-lane-outcome with exactly two values, credit-stop and no-key, printed as a fallback=<reason> token in the lane record. Wire it into ci-workflows' own intake-triage subscription job in the same PR.

   • Current state (origin/main 834f4e1):

     • report-lane-outcome inputs (action.yml:30-140) have no fallback input. recordTokens (classify.cjs:310-330) emits resets=, resets-window=, credential= (:320), turns=, max-turns= and models=.
     • In intake-triage the subscription job runs only when needs.claude.outputs.lane-class == 'credit-stop' or needs.claude.outputs.credential-present == 'false' (intake-triage.yml:832-835). The key job skips its report step on a credit stop (:626-628), so the subscription job's "Report triage outcome" step (:1174-1186) writes the run's only lane record, with credential: but no sign of why it ran.
     • PR #744 (about to merge) changes the intake-triage apply steps and header, not the report steps. intake-triage: give the model a read-only token; move label writes to a model-free job #804 restructures the same two jobs; rebase onto whichever lands first.

   • Scope:

     • report-lane-outcome/action.yml: new optional input fallback-reason, default empty, documented with its two values.
     • classify.cjs recordTokens: when the input is set, push fallback=<reason> immediately after credential=.
     • Reject any other non-empty value: the step fails with a message naming the allowed values.
     • intake-triage subscription job report step: pass fallback-reason: credit-stop when needs.claude.outputs.lane-class == 'credit-stop', else no-key when needs.claude.outputs.credential-present == 'false'.
     • README or action description entry for the new input.

   • Out of scope: an auth value for a dead key; claude-code-plugins changes (pr-verify passing the input, the harness-ops morning brief counting the token) follow in that repo after a repin.

   • Acceptance criteria:

     • fallback-reason: credit-stop with credential: X_CLAUDE_CODE_OAUTH_TOKEN yields one lane record containing both credential=X_CLAUDE_CODE_OAUTH_TOKEN and fallback=credit-stop, with fallback= right after credential= (test in classify.test.cjs or outcome-step.test.cjs).
     • fallback-reason: no-key yields fallback=no-key (test).
     • An empty fallback-reason yields no fallback= token, and existing records are byte-identical (existing tests unchanged and passing).
     • A value outside credit-stop / no-key fails the step (test).
     • The intake-triage subscription job passes fallback-reason derived from needs.claude.outputs.lane-class and needs.claude.outputs.credential-present; the leg parity test .github/scripts/intake-triage-legs.test.cjs is updated to allow this one difference, and node --test .github/scripts/*.test.cjs passes.

   • Sequencing / dependencies: independent; touches the same subscription job as #804, so rebase whichever lands second. Downstream consumer: claude-code-plugins #8577 workaround and the harness-ops morning brief.

   • Evidence: the vocabulary matches the two triggers of the subscription job's if at intake-triage.yml:835. Placement next to credential= follows classify.cjs:320. No external research was needed.
3. kyle-sexton

   added a commit that references this issue on Oct 20, 2026

   fix(intake-triage): run the model read-only and apply in a model-free…

   ...

   fd0c3e8
4. kyle-sexton

   closed this as completedin #834on Oct 20, 2026
5. kyle-sexton

   added a commit that references this issue on Oct 20, 2026

   feat(report-lane-outcome): record why a fallback credential ran (#834)

   ...

   This issue will close once commit a4e6c20 is merged into the 'main' branch.

   a4e6c20