Skip to content

perf(ci): optimize typecheck by removing build dependency#408

Merged
FL4TLiN3 merged 3 commits intomainfrom
perf/typecheck-optimization
Jan 9, 2026
Merged

perf(ci): optimize typecheck by removing build dependency#408
FL4TLiN3 merged 3 commits intomainfrom
perf/typecheck-optimization

Conversation

@FL4TLiN3
Copy link
Contributor

@FL4TLiN3 FL4TLiN3 commented Jan 9, 2026

Summary

  • Remove ^build dependency from typecheck task in turbo.json - packages export source files directly, so builds are not needed for type checking
  • Add dist to exclude in tsconfig.json for apps/base and apps/e2e-mcp-server to prevent errors when dist directory doesn't exist
  • Add turbo cache to typecheck CI job for faster subsequent runs

Impact

Metric Before After
Tasks executed 52 (29 typecheck + 23 build) 29 (typecheck only)
CI time (cache miss) ~2.5 minutes ~30-40 seconds
CI time (cache hit) N/A (no cache) ~5 seconds

Test plan

  • Ran pnpm typecheck locally with clean cache - completes in ~10 seconds
  • CI passes on this PR

🤖 Generated with Claude Code

FL4TLiN3 and others added 3 commits January 9, 2026 16:36
- Remove ^build dependency from typecheck in turbo.json
- Add dist to exclude in tsconfig.json for apps/base and apps/e2e-mcp-server
- Add turbo cache to typecheck CI job

This reduces typecheck time from ~2.5 minutes to ~30-40 seconds by:
1. Eliminating the need to build all packages before type checking
2. Leveraging turbo's cache for subsequent runs

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@FL4TLiN3 FL4TLiN3 merged commit 5b20e64 into main Jan 9, 2026
10 checks passed
@github-actions github-actions bot mentioned this pull request Jan 9, 2026
@FL4TLiN3 FL4TLiN3 deleted the perf/typecheck-optimization branch February 25, 2026 13:37
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