fix(ci): align Lean action preflight with estate allowlist - #744
fix(ci): align Lean action preflight with estate allowlist#744hyperpolymath wants to merge 4 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📜 Recent review details⏰ Context from checks skipped due to timeout. (2)
|
| Layer / File(s) | Summary |
|---|---|
Allow Lean Prover action rhodium-standard-repositories/actions-allowlist/allowed-actions.json, .machine_readable/REGISTRY.a2ml |
Adds leanprover/lean-action@* to patterns_allowed and updates the corresponding source_hash. |
Priority: ⬇️ Low
Estimated code review effort: 1 (Trivial) | ~2 minutes
Merge Risk: ⚪ Minimal · up to 486dd
This permits the Lean Prover GitHub Action and updates its registry metadata. The allowlist gap is resolved and no current merge-blocking risk remains.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description check | ✅ Passed | The description clearly explains that the pull request adds leanprover/lean-action to the CI preflight allowlist and resolves the reported gaps. |
| Title check | ✅ Passed | The title clearly identifies the CI allowlist alignment for the Lean action, which matches the main change. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
- Create stacked PR
- Commit on current branch
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.
A rabbit checked the action gate
Lean Prover may now pass its state
The registry hash aligns
With freshly tracked design
Small changes keep the build on time
Comment @coderabbitai help to get the list of available commands.
|
✅ Coding Agent task started: View task and status The task will inspect the CI failures, validate its fix, and open a stacked fix pull request automatically.
|
Rate Limit Exceeded
|
Update the generated registry artifact with the current source hash for the Rhodium Standard Repositories specification, resolving registry drift behind the CI failure. Validation: `bash scripts/build-registry.sh --check` passed. [View coding task](https://app.coderabbit.ai/code/tasks/a75ce906-3932-4507-adf1-ab9cb07ded4f?source=coding_agent_github_pr_description) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
|



GNPL and Lithoglyph fail
Allowlist Preflightbecause the preflight list omitsleanprover/lean-action, althoughconfig/settings/actions-allowlist.jsonalready permits it. Add the same action to the preflight list so the two policy sources agree.The authoritative
scripts/check-allowed-actions.shreproduced one Lean-action gap in each project before this change and reports zero gaps afterward (18 GNPL references and 26 Lithoglyph references). JSON parsing and patch whitespace checks pass. The projects retain their existing action lockfile validation and pinned Lean version.