Skip to content

prime-agent 0.7.0 (new formula) - #297372

Open
kennethnym wants to merge 1 commit into
Homebrew:mainfrom
kennethnym:prime-agent
Open

prime-agent 0.7.0 (new formula)#297372
kennethnym wants to merge 1 commit into
Homebrew:mainfrom
kennethnym:prime-agent

Conversation

@kennethnym

@kennethnym kennethnym commented Aug 6, 2026

Copy link
Copy Markdown

this pr adds prime-agent to homebrew/core.


  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>?
  • Is your test running fine brew test <formula>?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

  • I did not use AI/LLM to create this PR, or I disclosed the tool/model below and reviewed its output; I did not attribute commits to AI and will answer maintainer questions and review comments myself without AI/LLM.

The Ruby code was generated initially using brew create. The install script was generated via prime-agent using Claude Fable 5. I verified the changes by running the requested audits and running the locally installed prime-agent after the fact. All audit passes. This PR description is written by myself.


@github-actions github-actions Bot added nodejs Node or npm use is a significant feature of the PR or issue new formula PR adds a new formula to Homebrew/homebrew-core labels Aug 6, 2026
Comment thread Formula/p/prime-agent.rb
Comment on lines +20 to +24
zeromq = node_modules/"zeromq/build"
zeromq.children.each { |entry| rm_r(entry) if entry.directory? && entry.basename.to_s != os }
(zeromq/os).children.each { |dir| rm_r(dir) if dir.basename.to_s != arch }
(zeromq/os/arch/"node").children.each { |dir| rm_r(dir) if dir.basename.to_s.start_with?("musl") } if OS.linux?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can this not use the zeromq formula?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

unfortunately no. the zeromq package vendors its own libzmq inside the npm tarball itself. the library currently does not provide a way to point the library to a shared installation of zeromq, so even if zeromq is declared as a dependency here, it won't be consumed at runtime.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you file an issue for it, because this currently violates https://docs.brew.sh/Acceptable-Formulae#vendored-dependencies

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done! it is now linked here. thanks for the initial review!

@fettpl

fettpl commented Aug 7, 2026

Copy link
Copy Markdown

Local preflight found that the current formula passes its existing checks but still installs the packaged, statically linked ZeroMQ addon. cmake-ts reports ninja not found and exits successfully, so npm --build-from-source can appear successful without compiling.

I submitted zeromq.js #737 to restore zmq_shared=true. A local formula prototype now:

  • declares cmake, ninja, and pkgconf as build dependencies plus zeromq as a runtime dependency;
  • sets npm_config_zmq_shared=true and a macOS deployment target before npm install;
  • removes CMake staging files and sanitizes the required build/manifest.json shim paths;
  • passes brew test, brew linkage --test, brew audit --strict --new --online, and brew style;
  • links /opt/homebrew/opt/zeromq/lib/libzmq.5.dylib and successfully creates a zeromq.Pair socket.

There are two Prime Agent release blockers before this should merge:

@kennethnym please update this PR after zeromq.js publishes the shared-link fix and Prime Agent publishes a corrected stable artifact. The full coordination record is PrimeIntellect-ai/prime-agent#843.

@kennethnym

Copy link
Copy Markdown
Author

will do! really appreciate your help here. i will keep track of this. i'll bump the other 2 issues internally.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions Bot added the stale No recent activity label Aug 29, 2026
@fettpl

fettpl commented Aug 29, 2026

Copy link
Copy Markdown

This submission is still active and should remain open. The upstream Bedrock bundle blocker is being coordinated in Prime Agent Discussion #1410; a validated fix is available at 7acfae7b but is awaiting a maintainer-approved contribution path and corrected stable release.

zeromq@6.6.0 and the shared-link build path remain ready. @kennethnym please keep this existing PR open and update it only after the corrected Prime Agent archive passes artifact verification; no duplicate Homebrew submission will be created.

@github-actions github-actions Bot removed the stale No recent activity label Aug 29, 2026
@fettpl

fettpl commented Sep 13, 2026

Copy link
Copy Markdown

Upstream status, 2026-09-13:

  • The Bedrock bundle fix Prime Agent #2134 and compiled install/update stack #2140, #2165, and #2166 are merged.
  • I verified beta 0.9.4-beta.688.1.878410b: the npm artifact checksum matches, contains dist/bundle/amazon-bedrock.js, exports streamBedrock and streamSimpleBedrock, and passes version/help smoke checks with lifecycle scripts disabled.
  • Its darwin-arm64 archive checksum also matches, but the executable still fails strict codesign verification and version/help exit 137. The maintainer acknowledged the report and opened #2265; its current CI, including final macOS archive validation on ARM64 and Intel, passes and it awaits review/merge.
  • Stable remains v0.9.4, predating these changes. This formula should therefore remain on hold rather than be updated to the current beta.

@kennethnym, after #2265 merges and a corrected stable release passes artifact verification, the remaining owner action is to update this existing PR, tick the existing AI-disclosure checkbox, and rerun the complete source-build, functional test, shared Homebrew zeromq linkage, audit, style, and livecheck preflight.

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

Labels

new formula PR adds a new formula to Homebrew/homebrew-core nodejs Node or npm use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants