Add release readiness dashboard#163
Conversation
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
|
@coderabbitai review Please review this Gate G release-readiness dashboard/checker PR. Scope is documentation, Python checker/tests, local/CI gate wiring, and generated release evidence only; no Solidity behavior changes are intended. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThis PR implements Gate G release-readiness by adding ChangesGate G Release-Readiness Feature
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
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 |
🧩 Analysis chain🏁 Script executed: #!/bin/bash
# Get the list of files changed in PR `#163`
gh pr view 163 --repo 6529-Collections/6529Stream --json files --jq '.files[].path'Length of output: 743 🏁 Script executed: #!/bin/bash
# Get the PR diff
gh pr diff 163 --repo 6529-Collections/6529Stream 2>/dev/null | head -3000Length of output: 100210 🏁 Script executed: #!/bin/bash
# Read the full check_release_readiness.py
cat scripts/check_release_readiness.pyLength of output: 7793 🏁 Script executed: #!/bin/bash
# Read the full test_release_readiness.py
cat scripts/test_release_readiness.pyLength of output: 7826 🏁 Script executed: #!/bin/bash
# Read the test_release_manifest.py to check the seed function
cat scripts/test_release_manifest.py | head -120Length of output: 4549 🏁 Script executed: #!/bin/bash
# Check the full release-readiness.md
cat docs/release-readiness.mdLength of output: 9846 🏁 Script executed: #!/bin/bash
# Verify that release-artifacts/README.md is in the actual repo (linked from release-readiness.md)
fd "README.md" release-artifacts/Length of output: 258
Review: Gate G Release-Readiness Dashboard (
|
| Item | Risk | Suggested follow-up |
|---|---|---|
No test for missing-document error path in validate_release_readiness |
Low | Add test_rejects_missing_document to test_release_readiness.py |
--release-readiness CLI arg untested |
Low | Add test_custom_release_readiness_path that passes a non-default path |
Everything else looks correct. The Gate G dashboard and checker are ready to merge.
✅ Action performed
Review finished.
Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
1 participant
Summary
Closes #162.
This PR adds a Gate G release-readiness dashboard and local/CI checker for the current local release baseline. It keeps the change documentation/tooling-only: no Solidity behavior changes.
What changed
docs/release-readiness.mdto separate local baseline evidence from public-beta blockers and production-release blockers.scripts/check_release_readiness.pyandscripts/test_release_readiness.pyto enforce required headings, maturity language, blocker language, evidence links, release commands, missing linked files, and path-boundary rejection.make check,scripts/check.sh,scripts/check.ps1, and CI.docs/release-readiness.mdto release-manifest governance docs and refreshed the release manifest/checksum bundle.Local validation
python scripts\test_release_readiness.pypython scripts\check_release_readiness.pypython scripts\test_audit_package.pypython scripts\check_audit_package.pypython scripts\test_release_manifest.pypython scripts\generate_release_manifest.py --checkpython scripts\test_release_checksums.pypython scripts\generate_release_checksums.py --checkpython -m py_compile scripts\check_release_readiness.py scripts\test_release_readiness.py scripts\check_audit_package.py scripts\test_audit_package.py scripts\generate_release_manifest.py scripts\test_release_manifest.py scripts\generate_release_checksums.py scripts\test_release_checksums.py scripts\check_changelog.py scripts\test_changelog_check.pypython scripts\test_changelog_check.pypython scripts\check_changelog.pybash -n scripts/check.shscripts\check.ps1git diff --check(passes; Git reports the existingscripts/check.ps1line-ending warning)make checkpowershell -ExecutionPolicy Bypass -File scripts\check.ps1Notes
The full local gates pass with the repository's existing Foundry warning profile. Linux CI and CodeRabbit review are the next gates.
Summary by CodeRabbit
New Features
Documentation
Chores
Tests