Skip to content

fix: update macOS x64 runners from macos-13 to macos-15-large#330

Merged
khaliqgant merged 1 commit intomainfrom
fix/macos-runner
Jan 28, 2026
Merged

fix: update macOS x64 runners from macos-13 to macos-15-large#330
khaliqgant merged 1 commit intomainfrom
fix/macos-runner

Conversation

@khaliqgant
Copy link
Collaborator

@khaliqgant khaliqgant commented Jan 28, 2026

Summary

Updates deprecated macos-13 runners to macos-15-large for Intel x64 binary verification.

Problem

The publish workflow was being cancelled because macos-13 runners are retired:

The macOS-13 based runner images are now retired.

Solution

Switch to macos-15-large which provides Intel x64 environment.

🤖 Generated with Claude Code


Open with Devin

macos-13 runners are now retired per GitHub runner-images/issues/13046.
Switch to macos-15-large for Intel x64 binary verification.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@khaliqgant khaliqgant merged commit dbc8152 into main Jan 28, 2026
1 check passed
@khaliqgant khaliqgant deleted the fix/macos-runner branch January 28, 2026 10:32
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View issue and 3 additional flags in Devin Review.

Open in Devin Review

Comment on lines +423 to 425
- os: macos-15-large # Intel runner (macos-13 is retired)
arch: x64
binary: relay-pty-darwin-x64
Copy link
Contributor

Choose a reason for hiding this comment

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

🔴 macos-15-large is ARM64, not Intel x64 - binary verification will fail

The PR changes the runner from macos-13 to macos-15-large with the comment "Intel runner (macos-13 is retired)", but this is incorrect. According to GitHub's documentation, macos-15-large is an Apple Silicon (ARM64) runner, not an Intel x64 runner.

Click to expand

GitHub macOS Runner Architecture

  • macos-13, macos-13-large, macos-13-xlarge - Intel x64 (all retired)
  • macos-14, macos-14-large, macos-14-xlarge - Apple Silicon ARM64
  • macos-15, macos-15-large, macos-15-xlarge - Apple Silicon ARM64

Impact

The verification job at .github/workflows/publish.yml:423-425 and .github/workflows/verify-publish.yml:317-318 will attempt to run an x64 binary (relay-pty-darwin-x64) on an ARM64 runner. This will either:

  1. Fail immediately with an architecture mismatch error
  2. Potentially work via Rosetta 2 translation (if available), but this defeats the purpose of verifying the x64 binary works natively on Intel hardware

Actual vs Expected

  • Actual: x64 binary runs on ARM64 runner (fails or runs via emulation)
  • Expected: x64 binary runs on native Intel x64 hardware to verify it works correctly

Recommendation

Since GitHub has retired all Intel macOS runners, consider:

  1. Removing the x64 verification step entirely (the build uses cross-compilation which is reliable)
  2. Using a self-hosted Intel Mac runner if native verification is critical
  3. Documenting that x64 binaries are cross-compiled but not natively verified

Recommendation: Since GitHub has retired all Intel macOS runners, either remove the x64 verification job, use a self-hosted Intel runner, or document that x64 binaries are cross-compiled but not natively verified on Intel hardware.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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