chore(ci): fix closed PR cache cleanup - #813
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
skill-check — worker0 verified, 60 skipped (no docs/).
Four for four. Nicely done. |
Summary
GETRoot cause
Passing
-fand-Ffields togh apiwithout an explicit method switched the request toPOST. The cache-list endpoint rejected that request with404. Because the request ran inside process substitution, the failure did not stopmapfile, and the error response was later treated as a cache ID.Impact
Closed pull request caches can be enumerated and deleted again, reducing stale cache accumulation without attempting malformed deletion URLs.
Validation
python3 -m pytest -q .github/scripts/tests/- 188 passed, 3 subtests passedactionlint 1.7.12bash -ngit diff --checkrefs/pull/811/mergereturned 8 matching caches