Skip to content

ci: expose built percy on PATH in CLI-branch injection (PER-9772) - #244

Merged
pranavz28 merged 2 commits into
masterfrom
PER-9772_fix-cli-branch-injection
Jun 29, 2026
Merged

pranavz28 merged 2 commits into
masterfrom
PER-9772_fix-cli-branch-injection

Conversation

@pranavz28

Copy link
Copy Markdown
Contributor

Fixes CLI-branch SDK regression for this repo: npx percy couldn't resolve the yarn-linked @percy/cli (link symlinks the package, not its bin) → it downloaded percy@5.0.0 and the inject step failed. Now we add the built CLI's bin to PATH and verify with percy --version; the yarn link is best-effort so non-JS repos don't abort on a missing lockfile. Part of PER-9772. 🤖 Generated with Claude Code

pranavz28 and others added 2 commits June 29, 2026 02:46
npx percy could not resolve the yarn-linked @percy/cli (yarn link symlinks the
package but not its bin), so it downloaded the unrelated public percy@5.0.0 and
the inject step failed. Add the yarn global bin to PATH and verify with
`percy --version`; link is now best-effort so non-JS repos don't abort.

Part of PER-9772.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ction)

git clone interpolated github.event.inputs.branch directly into the shell.
Pass it via env ($BRANCH); it is regex-validated upstream. Part of PER-9772.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@pranavz28
pranavz28 marked this pull request as ready for review June 29, 2026 11:48
@pranavz28
pranavz28 requested a review from a team as a code owner June 29, 2026 11:48
@pranavz28
pranavz28 merged commit 97ebc60 into master Jun 29, 2026
10 checks passed
@pranavz28
pranavz28 deleted the PER-9772_fix-cli-branch-injection branch June 29, 2026 14:12
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.

2 participants