Skip to content

fix(ci): correct Turbo cache inputs and reuse build cache - #2981

Open
antonvishal wants to merge 1 commit into
browserbase:mainfrom
antonvishal:turbo-go-hash
Open

antonvishal wants to merge 1 commit into
browserbase:mainfrom
antonvishal:turbo-go-hash

Conversation

@antonvishal

@antonvishal antonvishal commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Fixes a few issues with the Turbo cache in CI:

  • Go SDK changes now invalidate the docs unit-test cache.
  • Build caches can be reused by downstream CI jobs.
  • Workflow-only changes no longer invalidate every Turbo task.
  • Adds the Eve example to typechecking and updates Turbo to 2.10.13.

Validation

Ran the frozen install, build, check, and unit tests locally.

Also tested this in fork PR #7 with CI run 35319601825. Build, Check, and TypeScript unit tests all passed. The unit job restored the build cache from the same commit, with all 13 required build tasks hitting the cache.

The Browserbase smoke test failed because the fork doesn't have a BROWSERBASE_API_KEY secret. Regression evals weren't run.


Summary by cubic

Fixes CI Turbo caching so build artifacts are reused across jobs and cache invalidation reflects actual source changes.

  • Scopes cache keys by CI job so the build cache is restored by later jobs.
  • Saves the Turbo cache after unit-test and build jobs, even on failure.
  • Adds Go SDK files to the docs unit-test task inputs so Go changes invalidate that cache.
  • Removes .github/workflows/ci.yml from global dependencies so workflow-only edits no longer invalidate every Turbo task.
  • Adds the Eve example to typechecking and updates Turbo to 2.10.13.

Written for commit ad60423. Summary will update on new commits.

Review in cubic

@changeset-bot

changeset-bot Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: ad60423

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

Copy link
Copy Markdown
Contributor

This PR is from an external contributor and must be approved by a stagehand team member with write access before CI can run.
Approving the latest commit mirrors it into an internal PR owned by the approver.
If new commits are pushed later, the internal PR stays open but is marked stale until someone approves the latest external commit and refreshes it.

@github-actions github-actions Bot added external-contributor Tracks PRs mirrored from external contributor forks. external-contributor:awaiting-approval Waiting for a stagehand team member to approve the latest external commit. labels Sep 18, 2026
@socket-security

socket-security Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​turbo@​2.10.7 ⏵ 2.10.1310010085 +197100

View full report

This branch has not been deployed

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

Labels

external-contributor:awaiting-approval Waiting for a stagehand team member to approve the latest external commit. external-contributor Tracks PRs mirrored from external contributor forks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant