Skip to content

ci: fix CLI-from-git setup in test.yml (PER-9772)#589

Merged
pranavz28 merged 1 commit into
masterfrom
PER-9772_fix-cli-from-git-setup
Jul 3, 2026
Merged

ci: fix CLI-from-git setup in test.yml (PER-9772)#589
pranavz28 merged 1 commit into
masterfrom
PER-9772_fix-cli-from-git-setup

Conversation

@pranavz28

Copy link
Copy Markdown
Contributor

Same bug as percy/percy-selenium-ruby#39, same fix.

Problem

The Set up @percy/cli from git step (workflow_dispatch path, used by the percy/cli SDK regression) always fails: after yarn remove @percy/cli there is no local percy bin (yarn link symlinks packages, not bins), so npx percy --version falls back to downloading the unrelated public percy@5.0.0 package, which prints "Heads up! It looks like @percy/cli is not installed!" and exits 1.

Failing job: https://github.com/percy/percy-appium-js/actions/runs/28640658402

Fix

Mirror the working percy-capybara pattern: put $(yarn global bin) on GITHUB_PATH after yarn global:link, call percy --version directly, and make remove/link best-effort.

🤖 Generated with Claude Code

Same bug as percy/percy-selenium-ruby#39: after 'yarn remove
@percy/cli' there is no local percy bin (yarn link symlinks packages,
not bins), so 'npx percy --version' downloads the unrelated public
'percy' package and the step exits 1. Mirror percy-capybara: expose
$(yarn global bin) on PATH after 'yarn global:link' and call 'percy'
directly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@pranavz28 pranavz28 requested a review from a team as a code owner July 3, 2026 05:42
@pranavz28 pranavz28 merged commit a6dd99a into master Jul 3, 2026
8 checks passed
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