fix(cli): handle audit file write errors gracefully#13
Conversation
Resolve conflicts: - packages/cli/src/commands/run.ts: Keep resolve() for absolute path handling - packages/core/src/service/load-service.ts: Keep cross-field validation
- Wrap writeFile in try/catch to handle filesystem errors - Add cross-field validation for preflight.dependencies (infoCount < warnCount) - Bump versions to 0.7.2
📝 WalkthroughWalkthroughThis release bumps all packages from version 0.6.x to 0.7.2, enhances error handling for CLI audit file writes with try/catch, and adds cross-field preflight configuration validation rules to enforce numeric constraint ordering (warn < fail thresholds across memory, dependencies, image, and timeout settings). Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
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 |
Resolve version conflicts, keeping 0.7.2
Summary
preflight.dependencies(infoCount < warnCount)Changes
packages/cli/src/commands/run.ts: Non-fatal error handling for audit outputpackages/core/src/service/load-service.ts: Dependency config validationTesting
Summary by CodeRabbit
Chores
Bug Fixes
✏️ Tip: You can customize this high-level summary in your review settings.