Skip to content

fix(agentic-jujutsu): add macOS arm64 optional dependency#160

Open
LaloLalo1999 wants to merge 1 commit into
ruvnet:mainfrom
LaloLalo1999:fix/agentic-jujutsu-darwin-arm64
Open

fix(agentic-jujutsu): add macOS arm64 optional dependency#160
LaloLalo1999 wants to merge 1 commit into
ruvnet:mainfrom
LaloLalo1999:fix/agentic-jujutsu-darwin-arm64

Conversation

@LaloLalo1999

Copy link
Copy Markdown

Summary

Fix agentic-jujutsu package metadata so Apple Silicon installs can resolve the generated N-API native package.

Context

On macOS arm64, jj-agent currently fails after a global install because the generated loader attempts to require agentic-jujutsu-darwin-arm64, but the main package metadata does not list that platform package as an optional dependency. This was reproduced with agentic-jujutsu@2.3.6 on Node v24.15.0 / macOS arm64.

Related issue: #159

Changes

  • Add agentic-jujutsu-darwin-arm64 to packages/agentic-jujutsu/package.json optional dependencies.
  • Align all platform optional dependency versions with the package version 2.3.6.
  • Update packages/agentic-jujutsu/package-lock.json to match package metadata.

Key Implementation Details

The existing N-API loader already contains the correct macOS arm64 resolution path:

require('agentic-jujutsu-darwin-arm64')

This PR makes the package metadata consistent with that loader path so npm can install the platform package once it is published as part of the release process.

Testing

Verified locally on macOS arm64:

cd packages/agentic-jujutsu
node -e "const p=require('./package.json'); const lock=require('./package-lock.json'); const deps=p.optionalDependencies; const locked=lock.packages[''].optionalDependencies; for (const name of ['agentic-jujutsu-darwin-arm64','agentic-jujutsu-darwin-x64','agentic-jujutsu-linux-x64-gnu','agentic-jujutsu-linux-x64-musl','agentic-jujutsu-linux-arm64-musl','agentic-jujutsu-linux-arm-gnueabihf','agentic-jujutsu-win32-x64-msvc']) { if (deps[name] !== '2.3.6') throw new Error('package.json mismatch '+name); if (locked[name] !== '2.3.6') throw new Error('lock mismatch '+name); } console.log('optionalDependencies metadata ok');"
npm pack --dry-run
npm run build
node -e "const m=require('./'); console.log(Object.keys(m).includes('JjWrapper') ? 'native load ok' : 'missing JjWrapper')"
node bin/cli.js version

Also passed repository pre-push checks when pushing the branch:

npm test
tsc --noEmit --project ./agentic-flow/config/tsconfig.json

Notes

The npm registry still needs a published agentic-jujutsu-darwin-arm64@2.3.6 artifact for the existing released version to install cleanly on Apple Silicon.

Copilot AI review requested due to automatic review settings June 1, 2026 08:17

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates platform-specific optional dependency artifacts for agentic-jujutsu to a new release and adds support for Apple Silicon on macOS.

Changes:

  • Bump all existing agentic-jujutsu-* optionalDependencies from 2.3.5 to 2.3.6
  • Add new optionalDependency for agentic-jujutsu-darwin-arm64 at 2.3.6
Files not reviewed (1)
  • packages/agentic-jujutsu/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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