Skip to content

Conversation

@dev-dami
Copy link
Owner

@dev-dami dev-dami commented Jan 26, 2026

Summary

  • Wrap audit file write in try/catch to handle filesystem errors (permission, disk, invalid path)
  • Add cross-field validation for preflight.dependencies (infoCount < warnCount)
  • Bump versions to 0.7.2

Changes

  • packages/cli/src/commands/run.ts: Non-fatal error handling for audit output
  • packages/core/src/service/load-service.ts: Dependency config validation

Testing

  • All 80 tests passing
  • Typecheck and lint clean

Summary by CodeRabbit

  • Chores

    • Version bump to 0.7.2 across all packages.
  • Bug Fixes

    • Improved error handling for audit output; prevents crashes with descriptive error logging.
    • Added configuration validation to enforce correct relationships between memory, dependencies, image, and timeout settings.

✏️ Tip: You can customize this high-level summary in your review settings.

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
@coderabbitai
Copy link

coderabbitai bot commented Jan 26, 2026

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Version Bumps (Monorepo)
package.json, packages/cli/package.json, packages/core/package.json, packages/http/package.json, packages/shared/package.json
Synchronized version updates from 0.6.0–0.6.1 to 0.7.2 across all package manifests
CLI Version String
packages/cli/src/index.ts
Updated CLI version display string to reflect 0.7.2 release
CLI Audit Output Handling
packages/cli/src/commands/run.ts
Replaced join import with resolve; refactored audit file path to use resolve(process.cwd(), ...); wrapped writeFile in try/catch to gracefully log errors instead of crashing on write failure
Preflight Configuration Validation
packages/core/src/service/load-service.ts
Added cross-field numeric constraint validation: enforces warnRatio < failRatio (memory), infoCount < warnCount (dependencies), warnMb < failMb (image), and minMs < maxMs (timeout); pushes descriptive errors when constraints violated

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

🐰 Version bumps hop across the warren,
Validation rules now guard each parcel,
Audit writes catch errors gracefully—
Our code runs safer, more orderly,
To 0.7.2 we march! 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately describes the main fix: handling audit file write errors gracefully in the CLI, which is the primary functional change.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dev-dami dev-dami merged commit 9fb2f18 into master Jan 26, 2026
3 checks passed
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.

2 participants