Repository navigation
fix: resolve P0 dependency issues from audit - #4
Conversation
Bundle Size Report
|
|
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: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
e427b99 to
f8c8a8c
Compare
- Use import.meta.url instead of __dirname for ESM compatibility - Consolidate readPkg calls — one per package, multiple assertions - Update CLI test to verify deps are in dependencies (not devDeps) since they are external and needed at runtime Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
f8c8a8c to
cde330b
Compare
Summary
@standard-schema/specto@workkit/astrodevDependencies (used in tests/helpers.ts)zodto@workkit/honodevDependencies (used in tests/middleware.test.ts)@workkit/typesand@workkit/errorsfrom@workkit/cachedependencies@workkit/typesand@workkit/errorsfrom@workkit/cryptodependencies@workkit/types,@workkit/errors,@workkit/env,@workkit/d1) to devDependencies (only used in codegen string templates)@standard-schema/specfrom dependencies to peerDependencies in@workkit/remix(type-only import)Test plan
bun run typecheckpasses (25/25 tasks)bun run buildpasses (21/21 tasks)bun run testpasses (21/21 tasks)e2e/dependency-audit.test.tsvalidates all 12 dependency constraints🤖 Generated with Claude Code