Skip to content

fix(runner): Tester says a session has no runners when it holds one - #1665

Merged
Alex Pezzati (alex-pezzati) merged 5 commits into
mainfrom
alxp/runner-list-here-keeps-qawolf-runner-id
Oct 16, 2026
Merged

Alex Pezzati (alex-pezzati) merged 5 commits into
mainfrom
alxp/runner-list-here-keeps-qawolf-runner-id

Conversation

@alex-pezzati

@alex-pezzati Alex Pezzati (alex-pezzati) commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Overview of Problem

As a user, I want Tester to list the runners my session holds, so that I can see and use them.

Context

  • qawolf runner list --here leaves out the runner named by QAWOLF_RUNNER_ID, which is how a Tester session gets its own runner, so Tester answers that there are none.
  • The CLI's help says that runner is included, so Tester reasonably reaches for --here.

Overview of Changes

  • runner list --here keeps the runner QAWOLF_RUNNER_ID names, next to the runners this directory launched.
  • The runner list guide describes both the team-wide list and --here.

Fixes AGE-5721

Before changes in this PR

  • Scenario can list the runners the session holds passes 0/3 on platform main with CLI 2.0.0
Result Session Time Cost
❌ qumzudve2bj77y34l3zlit8gocht 51s $0.12
❌ tikxyhkyfjyupj94vq4vmgakocht 52s $0.12
❌ n46zn5hn5hc1q8kb63g49v64ocht 52s $0.12

After changes in this PR

  • Scenario can list the runners the session holds passes 3/3 with this change patched into CLI 2.0.0
Result Session Time Cost
✅ z0zgnsfjydosl8kyaj82ppqjocht 64s $0.05
✅ lwjga24ilpm5jtp82u1cbta7ocht 64s $0.09
✅ l5pnmmke6ujftlbnaky1n41hocht 60s $0.02

Testing

bun run typecheck
bun run lint
bun run format:check
bun run knip
bun run test

Checklist

  • Changes follow the code style of this project
  • Self-review completed
  • Tests added/updated (or not applicable)
  • No breaking changes (or described below)

@alex-pezzati
Alex Pezzati (alex-pezzati) force-pushed the alxp/runner-list-here-keeps-qawolf-runner-id branch from 7a28e54 to ccfee63 Compare October 14, 2026 05:15
@alex-pezzati Alex Pezzati (alex-pezzati) changed the title fix(runner): Keep the QAWOLF_RUNNER_ID runner in runner list --here fix(runner): Tester says a session has no runners when it holds one Oct 14, 2026
@alex-pezzati
Alex Pezzati (alex-pezzati) marked this pull request as ready for review October 14, 2026 06:06
@alex-pezzati
Alex Pezzati (alex-pezzati) merged commit ab84d52 into main Oct 16, 2026
4 checks passed
@alex-pezzati
Alex Pezzati (alex-pezzati) deleted the alxp/runner-list-here-keeps-qawolf-runner-id branch October 16, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants