Skip to content

chore(acp): build native packages on latest mac#8075

Merged
alexhancock merged 1 commit intomainfrom
alexhancock/macos-latest
Mar 23, 2026
Merged

chore(acp): build native packages on latest mac#8075
alexhancock merged 1 commit intomainfrom
alexhancock/macos-latest

Conversation

@alexhancock
Copy link
Collaborator

Fixes issues like popped up in the first run of this in main

https://github.com/block/goose/actions/runs/23451231842

[Build native binaries / Build darwin-x64](https://github.com/block/goose/actions/runs/23451231842/job/68228626788)
The configuration 'macos-13-us-default' is not supported

@alexhancock alexhancock requested a review from jamadeo March 23, 2026 18:16
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2ad9546093

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines 34 to +35
- platform: darwin-x64
os: macos-13
os: macos-latest

Choose a reason for hiding this comment

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

P2 Badge Build the darwin-x64 package on an Intel runner

I checked GitHub's hosted-runner reference, and macos-latest now resolves to the arm64 image while Intel builds use macos-15-intel. On this matrix leg that means @aaif/goose-acp-server-darwin-x64 (ui/goose-acp-server/goose-acp-server-darwin-x64/package.json) is no longer built or exercised on Intel at all; this workflow only cross-compiles and uploads the artifact. Any x86_64-on-Intel linker/runtime regression would now bypass CI and ship straight to Intel macOS users, so this leg should stay on an explicit Intel label rather than macos-latest.

Useful? React with 👍 / 👎.

@alexhancock alexhancock force-pushed the alexhancock/macos-latest branch from 2ad9546 to 4c346ec Compare March 23, 2026 18:28
@alexhancock alexhancock added this pull request to the merge queue Mar 23, 2026
Merged via the queue into main with commit 2ff4ab2 Mar 23, 2026
27 checks passed
@alexhancock alexhancock deleted the alexhancock/macos-latest branch March 23, 2026 19:35
wpfleger96 added a commit that referenced this pull request Mar 23, 2026
* origin/main:
  fix: handle reasoning content blocks in OpenAI-compat streaming parser (#8078)
  chore(acp): build native packages on latest mac (#8075)
  Display delegate sub agents logs in UI (#7519)
  Update tar version to avoid CVE-2026-33056 (#8073)
  refactor: consolidate duplicated dependencies into workspace (#8041)
  tui: set up for publishing via github actions (#8020)
  feat: feature-gate local inference dependencies (#7976)
  feat: ability to manage sub recipes in desktop ui (#6360)
lifeizhou-ap added a commit that referenced this pull request Mar 24, 2026
* main: (37 commits)
  fix: handle reasoning content blocks in OpenAI-compat streaming parser (#8078)
  chore(acp): build native packages on latest mac (#8075)
  Display delegate sub agents logs in UI (#7519)
  Update tar version to avoid CVE-2026-33056 (#8073)
  refactor: consolidate duplicated dependencies into workspace (#8041)
  tui: set up for publishing via github actions (#8020)
  feat: feature-gate local inference dependencies (#7976)
  feat: ability to manage sub recipes in desktop ui (#6360)
  Tweak the release process: no more merge to main (#7994)
  fix: gemini models via databricks (#8042)
  feat(apps): Pass toolInfo to MCP Apps via hostContext (#7506)
  fix: remove configured marker when deleting oauth provider configuration (#7887)
  docs: add vmware-aiops MCP extension documentation (#8055)
  Show setup instructions for ACP providers in settings modal (#8065)
  deps: replace sigstore-verification with sigstore-verify to kill vulns (#8064)
  feat(acp): add session/set_config and stabilize list, delete and close (#7984)
  docs: Correct `gosoe` typo to `goose` (#8062)
  fix: use default provider and model when provider in session no longer exists (#8035)
  feat: add GOOSE_SHELL env var to configure preferred shell (#7909)
  fix(desktop): fullscreen header bar + always-visible close controls (#8033)
  ...
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.

2 participants