Skip to content

release(preview): carry lipo verification fix - #5536

Merged
lidge-jun merged 1 commit into
previewfrom
codex/preview-lipo-order
Sep 22, 2026
Merged

lidge-jun merged 1 commit into
previewfrom
codex/preview-lipo-order

Conversation

@lidge-jun

Copy link
Copy Markdown
Owner

Summary

Verification

  • Preview dry-run 35720315462, macOS job106721669999, printed the invalid architecture error and lipo <input_file> <command> usage. The universal binary creation completed before this assertion failed.
  • Linux AppImage/deb packaging and extracted CLI byte equality/version probe passed in this run.
  • Source/security review; hosted verification pending. Local tests/typechecks/builds/probes: NOT RUN by owner instruction. Commit/push use --no-verify.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed: existing architecture contract is unchanged.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

@lidge-jun
lidge-jun requested a review from Ingwannu as a code owner September 22, 2026 11:27
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: lidge-jun/opencodex/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7f5321fc-0ae0-436f-8456-2dd3046aed50

📥 Commits

Reviewing files that changed from the base of the PR and between 5cfe75d and d5771d4.

📒 Files selected for processing (2)
  • .github/workflows/release.yml
  • tests/ci-workflows/release-desktop-scripts.test.ts
 _____________________________
< I wish to make a complaint. >
 -----------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@lidge-jun
lidge-jun merged commit 7906a73 into preview Sep 22, 2026
0 of 10 checks passed
@lidge-jun
lidge-jun deleted the codex/preview-lipo-order branch September 22, 2026 11:27
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