fix: exclude 220MB of unused .node binaries from dbt-tools bundle#320
fix: exclude 220MB of unused .node binaries from dbt-tools bundle#320anandgupta42 merged 1 commit intomainfrom
Conversation
`bun build` copies `@altimateai/altimate-core` native binaries into `dbt-tools/dist/` as transitive build artifacts, but dbt-tools never loads them. Selectively copy only `index.js` and `altimate_python_packages/` instead of the entire dist directory (223MB → 3MB). Closes #319 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What does this PR do?
Fixes dbt-tools bundle copying 220MB of unnecessary
@altimateai/altimate-core.nodenative binaries thatbun buildproduces as transitive artifacts but dbt-tools never loads at runtime.Selectively copies only
index.js(860KB) andaltimate_python_packages/(2.2MB) instead of the entiredist/directory — 223MB → 3MB.Credit: @suryaiyer95 (#316 comment)
Type of change
Issue for this PR
Closes #319
How did you verify your code works?
publish.tsno longer does blanketcp -r ../dbt-tools/distChecklist
🤖 Generated with Claude Code