Skip to content

fix: select Darwin x64 inode ABI#66

Merged
roodboi merged 1 commit into
mainfrom
codex/HACK-924-darwin-x64
Jul 15, 2026
Merged

fix: select Darwin x64 inode ABI#66
roodboi merged 1 commit into
mainfrom
codex/HACK-924-darwin-x64

Conversation

@roodboi

@roodboi roodboi commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • select Apple SDK readdir$INODE64 on Intel macOS so FFI receives the modern 64-bit dirent layout
  • retain the unsuffixed modern ABI on Apple Silicon and the existing Linux ABI
  • add architecture-specific ABI selection coverage

Release incident

v2.25.1 published to npm/GitHub, but its macOS Intel asset verification failed with Descriptor-bound directory entry is malformed; downstream release assets/Homebrew were therefore skipped. Apple SDK headers confirm that Intel callers are redirected to the $INODE64 symbol while direct FFI lookup of unsuffixed readdir reaches the legacy layout.

Verification

  • focused safe-openat suite: 7/7
  • type-check and Ultracite green
  • arm64 compiled binary build/verify green
  • package dry-run green
  • final macOS Intel proof will be the release asset job after merge

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@linear-code

linear-code Bot commented Jul 15, 2026

Copy link
Copy Markdown

HACK-924

@roodboi

roodboi commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@codex review exact head b3e2a43. Please review the Darwin x64 FFI ABI selection, cross-platform safety, and public-content privacy. This fixes the v2.25.1 macos-15-intel release-asset verification failure.

@roodboi

roodboi commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Exact review target: b3e2a43.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Can't wait for the next one!

Reviewed commit: b3e2a43881

ℹ️ 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".

@roodboi
roodboi merged commit d9617d2 into main Jul 15, 2026
2 checks passed
@roodboi
roodboi deleted the codex/HACK-924-darwin-x64 branch July 15, 2026 19:31
github-actions Bot pushed a commit that referenced this pull request Jul 15, 2026
## [2.25.2](v2.25.1...v2.25.2) (2026-07-15)

### Bug Fixes

* select Darwin x64 inode ABI ([#66](#66)) ([d9617d2](d9617d2))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.25.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant