test: add hermetic CLI end-to-end coverage - #143
Conversation
There was a problem hiding this comment.
Automated review risk: 5/5.
The workflow currently fails without TEST_CLI_TOKEN; configuring it would expose the credential and private fixtures to same-repository pull-request code.
Critical or high-priority changes must be addressed.
Automatic approval was not submitted: checks failed: rust-tests.
There was a problem hiding this comment.
Automated review risk: 5/5.
The workflow exposes a private-repository credential to same-repository PR-controlled execution and currently fails trusted builds when that credential is unavailable.
Critical or high-priority changes must be addressed.
Automatic approval was not submitted: automated review found critical or high-priority findings.
There was a problem hiding this comment.
Automated review risk: 5/5.
The workflow exposes a private-repository credential and fixtures to pull-request-controlled code, and currently fails trusted builds because the required secret is unavailable.
Critical or high-priority changes must be addressed.
Automatic approval was not submitted: automated review found critical or high-priority findings.
What changed
Why
Required CLI correctness checks should run against the exact source revision without cross-repository credentials, branch coordination, or private fixtures. The CLI repository already contained stronger fixture coverage for each individual ecosystem; only the nested mixed-ecosystem scenario needed to move.
Validation