fix(ci): avoid ARC pnpm cache conflicts and bump sdk#1082
Merged
willgriffin merged 1 commit intomainfrom Mar 27, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates CI setup to avoid pnpm store corruption on ARC/self-hosted runners by skipping actions/cache when a mounted PNPM_STORE_DIR is present, while also bumping the aligned @happyvertical/* SDK dependencies from 0.71.18 to 0.71.19 (and refreshing the lockfile accordingly).
Changes:
- Add ARC-aware pnpm store detection to the
setup-environmentcomposite action and conditionally disableactions/cacherestore. - Bump aligned
@happyvertical/*dependencies to^0.71.19across workspace/package definitions. - Refresh
pnpm-lock.yamlto reflect the version bump and resulting dependency graph changes.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.github/actions/setup-environment/action.yml |
Adds pnpm store strategy detection (mounted store vs local) and gates actions/cache usage to prevent ARC store conflicts. |
package.json |
Bumps aligned @happyvertical/* dependency versions to ^0.71.19. |
pnpm-workspace.yaml |
Updates the workspace catalog entries for aligned @happyvertical/* versions to ^0.71.19. |
pnpm-lock.yaml |
Regenerates lockfile to incorporate the SDK bump and related dependency updates. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
actions/cachefor pnpm when ARC already provides a mountedPNPM_STORE_DIR@happyvertical/*SDK packages from0.71.18to0.71.19and refreshpnpm-lock.yamlWhy
Recent CI failures on March 26, 2026 showed
ERR_PNPM_ENOENTduring install after restoring the pnpm cache into the same store path already mounted on ARC runners. This change avoids restoring a tarball into that mounted store while preserving the normal cache flow for non-ARC environments.Validation
bash scripts/check-sdk-versions.shpnpm install --lockfile-onlypnpm install --frozen-lockfilepnpm run buildpnpm --filter @happyvertical/smrt-core test:integrationNote
pnpm run typecheckstill reports an existing unrelated@happyvertical/smrt-profilesresolution issue inpackages/userson current main.