Skip to content

Update sui-prover.yml#482

Open
cos wants to merge 6 commits intomainfrom
cos-patch-1
Open

Update sui-prover.yml#482
cos wants to merge 6 commits intomainfrom
cos-patch-1

Conversation

@cos
Copy link
Contributor

@cos cos commented Feb 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 3, 2026 22:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the GitHub Actions workflow configuration for the sui-prover by changing the runner environment for the integer-mate-specs job from a GitHub-hosted runner to a self-hosted runner.

Changes:

  • Modified the runner specification for the integer-mate-specs job from macos-latest to runner-base-1

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cursor
Copy link

cursor bot commented Feb 3, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 1.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cos
Copy link
Contributor Author

cos commented Feb 3, 2026

@claude, use the same setup from scallop-bpl-generation in the integer-mate runner

@claude
Copy link

claude bot commented Feb 3, 2026

Claude finished @cos's task in 47s —— View job


Analysis Complete

  • Read the workflow file to understand current configuration
  • Identify setup from scallop-bpl-generation job
  • Identify differences to apply to integer-mate-specs job

Findings

I've compared the scallop-bpl-generation job with integer-mate-specs and identified the key differences that need to be applied.

Current integer-mate-specs setup issues:

  • Uses a symlink for boogie: ln -s ~/prover-bundle/boogie/BoogieDriver ~/.local/bin/boogie
  • Missing .NET 8 setup (needed for running Boogie via dotnet)
  • No boogie version verification

scallop-bpl-generation setup (the correct approach):

  • Includes Setup .NET 8 step
  • Uses a wrapper script for boogie that calls dotnet to execute BoogieDriver.dll
  • Includes boogie version verification

Permission Limitation

I cannot modify workflow files due to GitHub App permissions restrictions. Workflow modifications require elevated permissions for security reasons.

Exact Changes Needed

Here are the specific changes to apply to the integer-mate-specs job in .github/workflows/sui-prover.yml:

1. Add .NET 8 setup step after line 247 (after the Download step):

      - name: Setup .NET 8
        uses: actions/setup-dotnet@v4
        with:
          dotnet-version: '8.0.x'

2. Replace lines 257 in the Setup step (change the boogie symlink to a wrapper script):

Replace this line:

ln -s ~/prover-bundle/boogie/BoogieDriver ~/.local/bin/boogie

With:

# Create a wrapper script for boogie instead of symlink
cat <<'EOF' > ~/.local/bin/boogie
#!/bin/bash
exec dotnet "$HOME/prover-bundle/boogie/BoogieDriver.dll" "$@"
EOF

3. Add verification at the end of the Setup step (after line 267):

# Verify boogie version
~/.local/bin/boogie /version

Why These Changes Matter

The self-hosted runner (runner-base-1) needs to use the dotnet wrapper approach because:

  • Symlinks may not work correctly with Boogie on self-hosted runners
  • The wrapper script ensures Boogie runs through the .NET runtime properly
  • This matches the proven working setup from scallop-bpl-generation

You'll need to apply these changes manually to the workflow file.


cos and others added 3 commits February 3, 2026 15:33
Updates the integer-mate-specs job to use the same proven setup approach
as scallop-bpl-generation and other BPL generation jobs:
- Add .NET 8 setup step
- Replace boogie symlink with wrapper script that calls dotnet
- Add boogie version verification

This ensures Boogie runs correctly through the .NET runtime on self-hosted runner.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

3 participants