fix(operations): restack runner JSON integrity on current main - #338
fix(operations): restack runner JSON integrity on current main#338seonghobae wants to merge 3 commits into
Conversation
|
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 |
|
Superseded by clean current-line successor #341, which retains the exact runner-assignment JSON byte-integrity and production coverage implementation. Closing this predecessor; its historical CI, scanner, and review evidence is not reused. |
Draft successor to #330 on protected main
26de1c3ab69106900410f6ceb18420109bfd9f8f. Exact successor headc62e19eb31b52028b7ad8bd37b04effc3c13df79is constructed directly on that base and changes onlyscripts/actions-runner-assignment-audit.mjs,test/actions-runner-assignment-cli.test.ts, andvitest.config.ts. The three protected target blobs were unchanged from #330's construction base immediately before the restack, so no intervening protected-main bytes are overwritten. The retained behavior rejects malformed UTF-8 and duplicate decoded JSON object names before runner-assignment evidence is parsed. This is read-only evidence hardening and does not prove runner assignment or operational acceptance. Keep Draft until fresh exact-head application CI, reviewer-ci, protected-base central Security Scan, configured 100% owned production statement/branch/function/line coverage with realistic tests, and zero valid unresolved findings are all terminal-passing. No predecessor evidence transfers. Supersedes #330 only after protected integration. Related: #30.