Skip to content

fix(ci): call idris2 by full path before it is on PATH (required gate red since #820) - #880

Merged
hyperpolymath merged 2 commits into
mainfrom
fix/idris2-version-before-path
Sep 30, 2026
Merged

hyperpolymath merged 2 commits into
mainfrom
fix/idris2-version-before-path

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

What

The required check abi-codegen-drift has failed on every uncached run since #820. Main was red on 09-27, 09-28 and 09-29, and #876/#877 are both stuck on it. The cause:

/home/runner/work/_temp/….sh: line 10: idris2: command not found
##[error]Process completed with exit code 127.

#820 moved make install to PREFIX="$HOME/.idris2", but the same step still ends with a bare idris2 --version. $HOME/.idris2/bin only joins $GITHUB_PATH in the next step. The fix calls the binary by its full path. verify-proofs.yml has the identical line and gets the same one-line fix.

Why it matters

abi-codegen-drift is one of main's two required contexts, so while it is red no hypatia PR can merge. That includes #876, which restores compilation after #862. Every Hypatia scan in the estate is failing until #876 lands.

Verification

This PR's own abi-codegen-drift run is the control. It either gets past Build Idris 2 from source or it doesn't.

🤖 Generated with Claude Code

https://claude.ai/code/session_0136eszqrQ53Kj7aBH1D4rXK

… red since #820)

#820 moved `make install` from /usr/local (already on PATH) to
$HOME/.idris2, but left `idris2 --version` as the last line of the SAME
step, before the "Put Idris 2 on PATH" step runs. Every uncached run
dies there with exit 127 ("idris2: command not found").

abi-codegen-drift is a REQUIRED status check on main, so this has held
every hypatia PR since (main runs red 09-27, 09-28, 09-29). verify-proofs
carries the identical line and is fixed the same way.

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

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 34 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: 23eb4e78-b6b2-49cb-a6a7-2d5c6e3b6730

📥 Commits

Reviewing files that changed from the base of the PR and between d369a80 and c9cd28b.

📒 Files selected for processing (2)
  • .github/workflows/abi-codegen-drift.yml
  • .github/workflows/verify-proofs.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.

@hyperpolymath
hyperpolymath enabled auto-merge (squash) September 30, 2026 09:59
@hyperpolymath
hyperpolymath merged commit 948f311 into main Sep 30, 2026
45 of 51 checks passed
@hyperpolymath
hyperpolymath deleted the fix/idris2-version-before-path branch September 30, 2026 10:12
hyperpolymath added a commit to hyperpolymath/cicd-squabbler that referenced this pull request Sep 30, 2026
The base gate feeds only the evidence-free listing once a PR is merged or
closed, never an agent item; pinned by a new core test (killed by a mutant
that routes every state through the open-PR path). Skipping the call keeps
the commonest done-claim off the REST secondary limit this PAT shares with
every other session -- live, 'landed hyperpolymath/hypatia#880' failed
closed on exactly that 403.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136eszqrQ53Kj7aBH1D4rXK
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