Skip to content

Remove the parallelism and typos rows from the end screen - #8

Merged
Byntham merged 1 commit into
mainfrom
remove-parallelism-typos-rows
Nov 5, 2026
Merged

Byntham merged 1 commit into
mainfrom
remove-parallelism-typos-rows

Conversation

@Byntham

@Byntham Byntham commented Nov 5, 2026

Copy link
Copy Markdown
Owner

Problem

The end screen showed two stats that weren't useful. "Parallelism" almost never goes above 1 in real play. "Typos taken literally" repeats the count in the session summary sentence directly above it.

Fix

Both rows are gone. Nothing else showed parallelism, so I deleted its tracking (runningAgentMs, peakAgents, and the sampling in advance). The store keeps typosLiteral because the summary sentence and the "What the agents read" list still use it.

Proof

End screen after a full automated session:

Before After
before after
  • tests/play.mjs now expects 5 stat rows instead of 7. I dropped its parallelism > 1 check.
  • npm run lint, npm run typecheck, and npm test (28 unit tests plus the full Playwright run) pass.

Reviewed with sol-loop: 1 round with gpt-6.1-sol. Sol verdict: approved

Created with Claude Opus 5.5 in Claude Code (T3 Code).

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Byntham
Byntham merged commit 49d3ad4 into main Nov 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant