Skip to content

ci(estate-audit): repoint at the SHA-pinned cicd-suite reusable (probe → fix) - #42

Merged
hyperpolymath merged 2 commits into
mainfrom
ci/estate-audit-repoint
Sep 30, 2026
Merged

hyperpolymath merged 2 commits into
mainfrom
ci/estate-audit-repoint

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Draft: a positive control for hyperpolymath/cicd-suite#36.

Central Estate CI/CD Audit has startup-failed with zero jobs on every push to main since 2026-09-23. The reusable it calls (cicd-suite@6cb08dd) references @main and @v7.0.1 internally, and this repo's sha_pinning_required: true refuses those refs.

This PR repoints the caller at cicd-suite#36's head (8035446), where all 27 inner refs are full SHAs.

  • Pass: the estate-audit job spawns and its steps run. Whether individual gates pass is a separate question from startup.
  • Fail: another startup_failure.

After cicd-suite#36 squash-merges, this will be repointed at the landed commit and marked ready.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WRvDivYwLSeVCJUrfjic3f

Positive control for hyperpolymath/cicd-suite#36. Under this repo's
sha_pinning_required, the current pin 6cb08dd startup-fails with zero
jobs. If the job spawns and runs at 8035446, #36 is the cure. After #36
squash-merges, this will be repointed at the landed commit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WRvDivYwLSeVCJUrfjic3f
@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 45 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 1a4cbd06-51f2-42eb-9b33-2688ed61c180

📥 Commits

Reviewing files that changed from the base of the PR and between aedfc2f and 3e5ced1.

📒 Files selected for processing (1)
  • .github/workflows/main-estate-audit.yml

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.

Replaces the pre-merge probe SHA 8035446 with the squash commit that landed
it. The reusable and actions/ tree are byte-identical between the two, so the
probe run 36696994940 (estate-audit 30/30) covers this ref. Rewrites the
caller comment: the gates are now fixed by literal SHAs inside the reusable,
not by cicd-suite's actions.lock, which this repo's sha_pinning_required
never honoured.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WRvDivYwLSeVCJUrfjic3f
@hyperpolymath
hyperpolymath marked this pull request as ready for review September 30, 2026 09:42
@hyperpolymath
hyperpolymath merged commit 7ec7e08 into main Sep 30, 2026
13 checks passed
@hyperpolymath
hyperpolymath deleted the ci/estate-audit-repoint branch September 30, 2026 09:46
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.

1 participant