Add blueprint release summary#384
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughAdds a blueprint release summary exporter that aggregates dataset metrics from DatasetStore (readiness tiers, validation/judge/attempt counts, materialization IDs, organ/setting breakdowns), produces a non-ready issues list, writes the summary as sorted pretty JSON, and includes tests; DatasetStore listing APIs now support ChangesBlueprint Release Summary Export
Possibly Related PRs
Estimated Code Review Effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/casecrawler/export/blueprint_release.py`:
- Around line 19-22: The calls to store.list_blueprints,
store.list_judge_reports, and store.list_generation_attempts currently pass a
hard cap limit=100_000 which silently truncates large datasets; update these
usages to read unbounded (e.g., remove the numeric limit and use limit=None) or
implement the same pagination/exhaustion loop used by store.list_records so that
blueprints, judge_reports, and attempts are fully iterated (refer to the
functions list_blueprints, list_judge_reports, list_generation_attempts and
mirror the records handling).
- Around line 90-99: The JSON shape changes because _tier_counts builds the
Counter dynamically; ensure the "missing" bucket is always present by seeding
counts["missing"] = 0 before iterating validation_reports. In function
_tier_counts, initialize the Counter with a zeroed "missing" key (and keep the
existing loop that increments counts["missing"] when report is None), and leave
the subsequent loop over ReleaseReadinessTier that sets default tier keys
unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: d065fb86-67e2-4786-9365-be5b577dbb1f
📒 Files selected for processing (2)
src/casecrawler/export/blueprint_release.pytests/test_blueprint_release_summary.py
Summary
Tests
Summary by CodeRabbit
New Features
Tests