Skip to content

improvement(ci): add sticky disk caches and bump runner for faster builds#3373

Merged
waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1/ci-blacksmith-optimize
Feb 27, 2026
Merged

improvement(ci): add sticky disk caches and bump runner for faster builds#3373
waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1/ci-blacksmith-optimize

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • Add Next.js build cache sticky disk to persist incremental compilation across runs
  • Add Turbo cache sticky disk so turbo can skip unchanged tasks
  • Bump test-build runner from 4vcpu to 8vcpu for faster CPU-bound steps
  • Enable cancel-in-progress for PR runs to avoid wasting runner minutes on stale pushes

Type of Change

  • Enhancement / improvement

Testing

Will verify on first CI run after merge

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 27, 2026 8:09am

Request Review

@waleedlatif1 waleedlatif1 force-pushed the waleedlatif1/ci-blacksmith-optimize branch from ca9875c to c8bce3f Compare February 27, 2026 07:46
@waleedlatif1
Copy link
Collaborator Author

@cursor review

@waleedlatif1
Copy link
Collaborator Author

@greptile

@waleedlatif1 waleedlatif1 force-pushed the waleedlatif1/ci-blacksmith-optimize branch from c8bce3f to 747d44d Compare February 27, 2026 07:48
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

Added performance optimizations to the CI pipeline by introducing sticky disk caching for Turbo (./.turbo) and GitHub Actions caching for Next.js build artifacts (./apps/sim/.next/cache). The Turbo cache uses a custom directory to avoid conflicts with the existing node_modules sticky disk mount. The runner was upgraded from 4vcpu to 8vcpu for faster CPU-bound operations. Additionally, enabled cancel-in-progress for pull request runs to save runner minutes when new commits are pushed.

  • Turbo cache configured with TURBO_CACHE_DIR=.turbo environment variable in both test and build steps
  • Next.js cache key based on bun.lock with fallback restore keys for partial cache hits
  • Cancel-in-progress only applies to PRs, not main/staging pushes
  • All cache directories properly ignored in .gitignore

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • All changes are well-thought-out CI optimizations with no code changes. The Turbo cache directory is properly separated from node_modules to avoid conflicts, addressing previous review concerns. The Next.js cache key is simplified to only use bun.lock rather than source files, preventing unnecessary cache invalidation. The runner upgrade and cancel-in-progress changes are straightforward improvements with no risk.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/ci.yml Enabled cancel-in-progress for PR runs to save runner time on stale pushes
.github/workflows/test-build.yml Upgraded runner to 8vcpu, added Turbo cache sticky disk at ./.turbo, added Next.js build cache with bun.lock-based key

Last reviewed commit: e9ca3ca

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

@waleedlatif1
Copy link
Collaborator Author

@greptile

@waleedlatif1
Copy link
Collaborator Author

@cursor review

@waleedlatif1 waleedlatif1 force-pushed the waleedlatif1/ci-blacksmith-optimize branch from 3d27b9e to 5dce1f1 Compare February 27, 2026 07:55
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

@waleedlatif1
Copy link
Collaborator Author

@greptile

@waleedlatif1 waleedlatif1 force-pushed the waleedlatif1/ci-blacksmith-optimize branch from cadfab2 to e9ca3ca Compare February 27, 2026 08:06
@waleedlatif1
Copy link
Collaborator Author

@greptile

@waleedlatif1
Copy link
Collaborator Author

@cursor review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

@waleedlatif1 waleedlatif1 merged commit b3424e2 into staging Feb 27, 2026
12 checks passed
@waleedlatif1 waleedlatif1 deleted the waleedlatif1/ci-blacksmith-optimize branch February 27, 2026 08: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.

1 participant