Skip to content

paperai 2.5.0 (new formula)#3515

Open
chenrui333 wants to merge 1 commit into
mainfrom
paperai
Open

paperai 2.5.0 (new formula)#3515
chenrui333 wants to merge 1 commit into
mainfrom
paperai

Conversation

@chenrui333
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings December 29, 2025 01:11
@chenrui333 chenrui333 added the in progress Stale bot should stay away label Dec 29, 2025
@github-actions github-actions Bot added new formula python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue labels Dec 29, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new Homebrew formula for paperai version 2.5.0, an AI tool for medical and scientific papers. The formula includes extensive Python dependencies managed through virtualenv.

Key Changes

  • New formula for paperai 2.5.0 with Python 3.14 support
  • Declares system dependencies for certifi, cryptography, pydantic, and rpds-py with :no_linkage
  • Includes 91 Python package resources, with some fetched from GitHub sources (bitsandbytes, faiss-cpu, sqlite-vec)

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

Comment thread Formula/p/paperai.rb Outdated
Comment on lines +305 to +309
resource "pydantic" do
url "https://files.pythonhosted.org/packages/69/44/36f1a6e523abc58ae5f928898e4aca2e0ea509b5aa6f6f392a5d882be928/pydantic-2.12.5.tar.gz"
sha256 "4d351024c75c0f085a9febbb665ce8c0c6ec5d30e903bdb6394b7ede26aebb49"
end

Copy link

Copilot AI Dec 29, 2025

Choose a reason for hiding this comment

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

The resource definition for "pydantic" should be removed. This package is already provided as a system dependency with depends_on "pydantic" => :no_linkage on line 14. When a package is declared as a dependency with :no_linkage, it should not also be defined as a resource, as the system-provided version will be used instead.

Suggested change
resource "pydantic" do
url "https://files.pythonhosted.org/packages/69/44/36f1a6e523abc58ae5f928898e4aca2e0ea509b5aa6f6f392a5d882be928/pydantic-2.12.5.tar.gz"
sha256 "4d351024c75c0f085a9febbb665ce8c0c6ec5d30e903bdb6394b7ede26aebb49"
end

Copilot uses AI. Check for mistakes.
Comment thread Formula/p/paperai.rb Outdated
Comment on lines +370 to +374
resource "rpds-py" do
url "https://files.pythonhosted.org/packages/20/af/3f2f423103f1113b36230496629986e0ef7e199d2aa8392452b484b38ced/rpds_py-0.30.0.tar.gz"
sha256 "dd8ff7cf90014af0c0f787eea34794ebf6415242ee1d6fa91eaba725cc441e84"
end

Copy link

Copilot AI Dec 29, 2025

Choose a reason for hiding this comment

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

The resource definition for "rpds-py" should be removed. This package is already provided as a system dependency with depends_on "rpds-py" => :no_linkage on line 16. When a package is declared as a dependency with :no_linkage, it should not also be defined as a resource, as the system-provided version will be used instead.

Suggested change
resource "rpds-py" do
url "https://files.pythonhosted.org/packages/20/af/3f2f423103f1113b36230496629986e0ef7e199d2aa8392452b484b38ced/rpds_py-0.30.0.tar.gz"
sha256 "dd8ff7cf90014af0c0f787eea34794ebf6415242ee1d6fa91eaba725cc441e84"
end

Copilot uses AI. Check for mistakes.
Comment thread Formula/p/paperai.rb Outdated
Comment on lines +65 to +69
resource "certifi" do
url "https://files.pythonhosted.org/packages/a2/8c/58f469717fa48465e4a50c014a0400602d3c437d7c0c468e17ada824da3a/certifi-2025.11.12.tar.gz"
sha256 "d8ab5478f2ecd78af242878415affce761ca6bc54a22a27e026d7c25357c3316"
end

Copy link

Copilot AI Dec 29, 2025

Choose a reason for hiding this comment

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

The resource definition for "certifi" should be removed. This package is already provided as a system dependency with depends_on "certifi" => :no_linkage on line 11. When a package is declared as a dependency with :no_linkage, it should not also be defined as a resource, as the system-provided version will be used instead.

Suggested change
resource "certifi" do
url "https://files.pythonhosted.org/packages/a2/8c/58f469717fa48465e4a50c014a0400602d3c437d7c0c468e17ada824da3a/certifi-2025.11.12.tar.gz"
sha256 "d8ab5478f2ecd78af242878415affce761ca6bc54a22a27e026d7c25357c3316"
end

Copilot uses AI. Check for mistakes.
Comment thread Formula/p/paperai.rb Outdated
Comment on lines +85 to +89
resource "cryptography" do
url "https://files.pythonhosted.org/packages/9f/33/c00162f49c0e2fe8064a62cb92b93e50c74a72bc370ab92f86112b33ff62/cryptography-46.0.3.tar.gz"
sha256 "a8b17438104fed022ce745b362294d9ce35b4c2e45c1d958ad4a4b019285f4a1"
end

Copy link

Copilot AI Dec 29, 2025

Choose a reason for hiding this comment

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

The resource definition for "cryptography" should be removed. This package is already provided as a system dependency with depends_on "cryptography" => :no_linkage on line 12. When a package is declared as a dependency with :no_linkage, it should not also be defined as a resource, as the system-provided version will be used instead.

Suggested change
resource "cryptography" do
url "https://files.pythonhosted.org/packages/9f/33/c00162f49c0e2fe8064a62cb92b93e50c74a72bc370ab92f86112b33ff62/cryptography-46.0.3.tar.gz"
sha256 "a8b17438104fed022ce745b362294d9ce35b4c2e45c1d958ad4a4b019285f4a1"
end

Copilot uses AI. Check for mistakes.
@chenrui333 chenrui333 added CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. and removed in progress Stale bot should stay away labels Feb 19, 2026
@chenrui333 chenrui333 marked this pull request as draft March 14, 2026 21:39
@chenrui333 chenrui333 marked this pull request as ready for review April 27, 2026 17:08
Copy link
Copy Markdown

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

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: 69341ed154

ℹ️ 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 Formula/p/paperai.rb
sha256 "e13e4d6d17bfb115daab1fc8f76e850afba35627c097c93ca8f5bb4ed354afbf"
license "Apache-2.0"

depends_on "rust" => :build
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Add CMake build dependency for faiss-cpu build

faiss-cpu is built from source in resource("faiss-cpu").stage, and this formula even patches cmake/default_config.cmake, which means the install path depends on having a cmake binary available during brew install --build-from-source. With only rust declared as a build dependency, clean environments that do not already have CMake will fail while pip builds that resource. Declare depends_on "cmake" => :build so the formula can build reproducibly.

Useful? React with 👍 / 👎.

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

Labels

CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants