Skip to content

--cache-right: never replay a hang, keep POST bodies, flag live misses - #23

Merged
tmashara merged 1 commit into
mainfrom
cache-right-fixes
Oct 24, 2026
Merged

tmashara merged 1 commit into
mainfrom
cache-right-fixes

Conversation

@tmashara

Copy link
Copy Markdown
Owner

Fixes from the review of #21.

  • Bug: cancelled requests (status: -1) were kept in the recording, and Playwright replays them as a request that never settles. A recorded stylesheet hung goto until timeout. Only 200–399 is kept now.
  • Bug: POSTs were deduped by method + URL, so a GraphQL/RPC endpoint kept only its first body and the rest went live. The key now includes the request body.
  • Live misses: a request the recording lacks still goes to the network, but the shot now lists N request(s) not in the recording, loaded live under failed on the right. A half-live reference is never clean and --rerun-failed doesn't reuse it. Checked by hand: removing the stylesheet from a recording gives failed right: 1 request(s) not in the recording, loaded live.
  • Service workers are blocked on both sides while --cache-right is on (a HAR can't see what they serve).
  • The recording is written 0600 with a .gitignore (*) beside it, since it holds the right side's cookies and auth headers.
  • With --rerun-failed --cache-right, both right: lines print when a run replays and also records.
  • Docs (how-it-works, skill reference) say what is and isn't replayed.

Not done: --accept + --cache-right is still allowed; deleting expired recordings is left to the next wipe.

Unit 232/232 and visual 306/306 pass locally.

🤖 Generated with Claude Code

Drops cancelled (-1) entries along with errors, keys entries by request body
too, and lists requests the recording lacks as a failed asset on the right so a
half-live reference is never clean or reused. Blocks service workers on both
sides while caching, writes the recording 0600 with a .gitignore, and prints
both right: lines when a rerun replays and records.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tmashara
tmashara merged commit 52ea369 into main Oct 24, 2026
3 checks passed
@tmashara tmashara mentioned this pull request Oct 24, 2026
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