-
Notifications
You must be signed in to change notification settings - Fork 0
fix(cli): handle audit file write errors gracefully #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.