-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: gateway.test.ts e2e fails on clean main — test config omits the "dev" agent its sessionKey requires #99513
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.no-staleExclude from stale automationExclude from stale automation
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.no-staleExclude from stale automationExclude from stale automation
Type
Fields
Priority
None yet
Summary
The clean-base e2e test
src/gateway/gateway.test.ts > gateway e2e > accepts a gateway agent request over ws and returns a run idfails on currentmainwith:This blocks local pre-push gates (
pnpm check:changed/ precheck flows) for contributors, because the failure reproduces on an untouched checkout ofmainregardless of their diff.Reproduction
Result on
776b34a4e642(also reproduced earlier ondf350e6720and8d535fb039):Source-level cause
The failing test requests
sessionKey = "agent:dev:mock-openai"(src/gateway/gateway.test.ts:227), but its gateway config (src/gateway/gateway.test.ts:195-217) only definesagents.defaults— there is noagents.listentry declaring an agent with iddev. The gateway therefore rejects the run with "Agent "dev" no longer exists in configuration".The sibling test in the same file ("lists agents over ws",
src/gateway/gateway.test.ts:292-294) declaresagents.list: [{ id: "main", default: true, ... }]and passes, which suggests the fix is to declare thedevagent in the failing test's config (or switch the session key to a declared agent).Expected behavior
pnpm test src/gateway/gateway.test.tspasses on a clean checkout ofmain.Notes
gateway.test.tsclean-base failure above; no claim about other scheduled/live suites on current main.Environment
776b34a4e642(Linux, Node 22.22.2, pnpm 11.2.2)🤖 Filed with AI assistance (Claude Fable 5); failure and source-level cause verified by hand on the SHA above.