ci: add reproducible repository verification - #42
Conversation
|
Warning Review limit reached
Next review available in: 12 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request adds unified verification scripts for workflow linting, Maven validation, consumer dependency isolation, and coverage reporting. CI runs the new checks, documentation checkouts disable credential persistence, and contributor documentation records the updated verification command. ChangesVerification and CI policy
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant verify.sh
participant WorkflowLinters
participant Maven
participant ConsumerIsolation
participant CoverageReports
verify.sh->>WorkflowLinters: run actionlint and zizmor
verify.sh->>Maven: run targeted and full verification
verify.sh->>ConsumerIsolation: validate the consumer dependency tree
verify.sh->>CoverageReports: calculate module line coverage
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@scripts/verify.sh`:
- Around line 31-37: Update the awk summary in the verification script to handle
total equal to zero before calculating or printing the percentage. Emit a valid
zero-coverage result for header-only JaCoCo reports, while preserving the
existing covered/total percentage output when total is greater than zero.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: adce086a-3e71-4dc7-9b38-f32c052a1076
📒 Files selected for processing (7)
.github/workflows/ci.yml.github/workflows/docs.ymlCHANGELOG.mdCONTRIBUTING.mdmise.tomlscripts/verify-consumer-isolation.shscripts/verify.sh
ci: add reproducible repository verification
Summary
./scripts/verify.shcontributor command with failure-tail logging and JaCoCo summariesWhy
The 0.1.0 preparation exposed three avoidable sources of wasted time: stale locally installed POMs produced a misleading dependency tree, condensed Maven output hid the actual failing module, and workflow mistakes were only found during review. These checks make those failures reproducible and actionable without adding a custom prose-linting implementation.
Impact
Contributors get one release-readiness command and CI gains dedicated workflow linting plus an isolated consumer dependency check. The published library API and runtime behavior are unchanged.
Validation
./scripts/verify.shdatex4j-model-v3_7for the consumer testbash -ngit diff --checkSummary by CodeRabbit
New Features
Documentation
Security