Skip to content

Harden Playwright cache restore behavior - #154

Merged
BigSimmo merged 3 commits into
mainfrom
reapply-ci-cache-hardening
Jul 2, 2026
Merged

Harden Playwright cache restore behavior#154
BigSimmo merged 3 commits into
mainfrom
reapply-ci-cache-hardening

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Summary\n- remove broad Playwright cache restore keys in CI verify job\n- remove broad Playwright cache restore keys in release browser matrix job\n\n## Why\nUsing exact cache keys tied to package-lock.json reduces stale cache restores and lowers cache poisoning risk while keeping deterministic cache hits.

BigSimmo and others added 2 commits July 2, 2026 17:39
Remove broad restore-keys from Playwright browser caches so CI only restores exact lockfile-matched cache entries, reducing stale cache flakiness and cache poisoning surface.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@BigSimmo
BigSimmo enabled auto-merge July 2, 2026 10:30
@BigSimmo
BigSimmo merged commit befe75b into main Jul 2, 2026
4 checks passed
@BigSimmo
BigSimmo deleted the reapply-ci-cache-hardening branch July 2, 2026 16:31
BigSimmo added a commit that referenced this pull request Jul 31, 2026
…esson (#1511)

Two gaps: work was done and not recorded.

- docs/branch-review-ledger.md had no row for claude/root-dir-coverage-gate-v2
  (#1458), claude/pre-commit-fail-open (#1494) or claude/ledger-relanding
  (#1508). AGENTS.md requires one per reviewed branch; appended retrospectively
  with the merged heads.
- #154 records why three separate "did it land" checks returned false answers
  this session: title greps (reworded concurrently), id greps (ids reallocated
  in parallel branches), and PR state fields (squash merges, and a MERGED PR
  whose content had not reached the fetched ref). Resolve the blob and grep for
  distinctive prose instead.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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