Skip to content

[codex] Restore arm64 macOS release packaging#160

Merged
rz-openai merged 3 commits into
mainfrom
dev/rz/restore-native-electron-helper
Jun 18, 2026
Merged

[codex] Restore arm64 macOS release packaging#160
rz-openai merged 3 commits into
mainfrom
dev/rz/restore-native-electron-helper

Conversation

@rz-openai

@rz-openai rz-openai commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • restore native-architecture packaging for the Electron Network Inspector helper
  • build macOS Release archives explicitly for arm64 so the host, CLI, and nested helper use the same architecture
  • make package:mac:release select arm64 regardless of the Node process architecture
  • revert the unintended universal helper build introduced in [codex] Build Electron helper only when needed #156
  • fix release archives failing when Xcode encounters the unsigned universal nested app

Failure

Testing

  • Built and ran Snap-O locally (Release archive built successfully; app not launched)
  • Added or updated tests (not applicable; existing tests pass)
  • npm run package:mac:release
  • npm run format:check
  • npm run lint
  • npm run typecheck
  • npm test
  • Release archive using the snap-o-release signing settings:
    xcodebuild -project Snap-O.xcodeproj -scheme Snap-O -configuration Release -archivePath /tmp/snap-o-arm64-deterministic.xcarchive -derivedDataPath /tmp/snap-o-arm64-deterministic-derived -sdk macosx COMPILER_INDEX_STORE_ENABLE=NO CODE_SIGN_STYLE=Manual CODE_SIGN_IDENTITY=- "OTHER_CODE_SIGN_FLAGS=--options runtime" archive
  • Verified archived Snap-O, snapo, and Snap-O Network Inspector executables are all arm64

Checklist

  • I have read the Contributing guidelines
  • Documentation has been updated (not needed for this packaging correction)

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

Copy link
Copy Markdown

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: 237f7b505c

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread snapo-desktop-electron/scripts/package-macos.mjs Outdated
@rz-openai rz-openai changed the title [codex] Restore native Electron helper packaging [codex] Restore arm64 macOS release packaging Jun 18, 2026
@rz-openai

Copy link
Copy Markdown
Collaborator Author

@codex review

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

Copy link
Copy Markdown

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: 0c08ee78b3

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread snapo-desktop-electron/scripts/package-macos.mjs Outdated
@rz-openai

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Hooray!

Reviewed commit: d3dbf5e89c

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@rz-openai rz-openai mentioned this pull request Jun 18, 2026
12 tasks
@rz-openai rz-openai merged commit 4eb8976 into main Jun 18, 2026
3 checks passed
@rz-openai rz-openai deleted the dev/rz/restore-native-electron-helper branch June 18, 2026 15:12
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