Skip to content

Conversation

@kb-typeform
Copy link
Contributor

@kb-typeform kb-typeform commented Dec 22, 2025

This PR enhances the setup-node-with-cache action with comprehensive cache
integrity verification and verbose debugging to support the ongoing rollout
of centralized GitHub Actions across frontend projects.

BREAKING: Removes incorrect Turbo optimization that skipped yarn install

The previous logic incorrectly assumed Turbo monorepos don't need yarn install
on cache hits. This is FALSE for Yarn workspaces because:

1. Workspace symlinks are NOT preserved in GitHub Actions cache
2. Yarn creates symlinks between workspace packages during install
3. Without symlinks, workspace dependencies won't be found (jest: not found)
4. Turbo handles BUILD caching, not workspace linking
5. This is fast (~10-20s) because packages are cached

This fix ensures yarn install always runs for Yarn workspaces, preventing
'module not found' errors in CI for bob-the-builder and other Turbo+Yarn repos.
@kb-typeform kb-typeform marked this pull request as ready for review January 15, 2026 14:52
@kb-typeform kb-typeform requested a review from a team as a code owner January 15, 2026 14:52
@gitstream-cm
Copy link

gitstream-cm bot commented Jan 15, 2026

This PR is missing a Jira ticket reference in the title or description.
Please add a Jira ticket reference to the title or description of this PR.

@gitstream-cm
Copy link

gitstream-cm bot commented Jan 15, 2026

🥷 Code experts: no user but you matched threshold 10

kb-typeform has most 👩‍💻 activity in the files.
kb-typeform has most 🧠 knowledge in the files.

See details

shared-actions/setup-node-with-cache/README.md

Activity based on git-commit:

kb-typeform
JAN
DEC 178 additions & 0 deletions
NOV
OCT
SEP
AUG

Knowledge based on git-blame:
kb-typeform: 100%

shared-actions/setup-node-with-cache/action.yml

Activity based on git-commit:

kb-typeform
JAN
DEC 146 additions & 1 deletions
NOV
OCT
SEP
AUG

Knowledge based on git-blame:
kb-typeform: 100%

✨ Comment /gs review for LinearB AI review. Learn how to automate it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants