fix: install Pinecone CLI as a Homebrew cask (formula disabled 2026-03-30) - #46
Open
mcdgavin wants to merge 1 commit into
Open
fix: install Pinecone CLI as a Homebrew cask (formula disabled 2026-03-30)#46mcdgavin wants to merge 1 commit into
mcdgavin wants to merge 1 commit into
Conversation
mcdgavin
force-pushed
the
fix/brew-cask-install
branch
2 times, most recently
from
September 9, 2026 16:44
5eda7eb to
b7e25b8
Compare
mcdgavin
force-pushed
the
fix/brew-cask-install
branch
from
September 9, 2026 16:51
b7e25b8 to
a739a2d
Compare
This was referenced Sep 9, 2026
mcdgavin
force-pushed
the
fix/brew-cask-install
branch
from
September 9, 2026 21:08
a739a2d to
002e406
Compare
mcdgavin
added a commit
to mcdgavin/skills
that referenced
this pull request
Sep 9, 2026
…3-30)
The `pinecone-io/tap/pinecone` formula was disabled on 2026-03-30 when the
CLI migrated to a cask, so the documented `brew install pinecone-io/tap/pinecone`
now fails and tells the user to reinstall with `--cask`.
Switch every install instruction to the single fully qualified command:
brew install --cask pinecone-io/tap/pinecone
The separate `brew tap pinecone-io/tap` step is dropped. `brew install`
taps automatically for a fully qualified name, and under Homebrew 6 tap
trust the fully qualified form grants trust to only this cask. This is the
form Homebrew's Tap Trust docs give and the form pinecone-io/cli's README
already uses.
Updated:
- README.md
- skills/pinecone-cli/SKILL.md (plus a note for users upgrading from the
old formula)
- skills/pinecone-help/SKILL.md
- tools/test_build.py (fixture string kept in step with the docs)
Rendered `--target claude-code` output for the cli and help skills is
byte-identical to pinecone-io/pinecone-claude-code-plugin#46.
…3-30)
The `pinecone-io/tap/pinecone` formula was disabled on 2026-03-30 when the
CLI migrated to a cask, so the documented `brew install pinecone-io/tap/pinecone`
now fails with:
Error: pinecone-io/tap/pinecone has been disabled because it has migrated
to a cask. Reinstall with: brew uninstall pinecone-io/tap/pinecone &&
brew install --cask pinecone-io/tap/pinecone!
Switch the plugin-owned install instructions to the single fully qualified
command:
brew install --cask pinecone-io/tap/pinecone
The separate `brew tap pinecone-io/tap` step is dropped. `brew install`
taps automatically for a fully qualified name, and under Homebrew 6 tap
trust the fully qualified form grants trust to only this cask. This is the
form Homebrew's Tap Trust docs give and the form pinecone-io/cli's README
already uses.
Updated:
- README.md (plus a note for users upgrading from the old formula)
- hooks/pinecone-api-key-check.sh (session-start CLI guidance)
The two skills/ files carry the same command but are sync-managed from
pinecone-io/skills, where the maintainers are carrying the fix; they are
left for the sync.
mcdgavin
force-pushed
the
fix/brew-cask-install
branch
from
September 10, 2026 01:55
002e406 to
8815b73
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The "Install the Pinecone CLI (Optional)" instructions no longer work. The
pinecone-io/tap/pineconeformula was disabled on 2026-03-30 when the CLI migrated to a cask, so following the README today produces:Verified against the tap:
Formula/pinecone.rbcarriesdisable! date: "2026-03-30"pointing at the cask, andCasks/pinecone.rbis the live GoReleaser-generated cask (currently 1.0.1). pinecone-io/cli's README already documents the cask form.Fix
Switch every install instruction in the plugin to a single fully qualified command:
README.md--caskcommand + upgrade notehooks/pinecone-api-key-check.sh--caskcommandskills/cli/SKILL.mdandskills/help/SKILL.mdcarry the same stale command but are rendered from pinecone-io/skills, where the maintainers are carrying the fix (pinecone-io/skills#33). Left for the sync.The upgrade note tells anyone who installed before the migration to run
brew uninstall pinecone-io/tap/pineconefirst, since Homebrew refuses to install the cask over the disabled formula.The hook matters most: when
pcis not installed it injects the install command into Claude's context at every session start, so the agent was confidently repeating a command that errors out.Why drop the separate
brew tapstepbrew installtaps automatically for a fully qualifieduser/repo/name(cmd/install.rbcallstap.ensure_installed!on each named argument).brew trust. Homebrew's Tap Trust docs givebrew install --cask user/repository/caskas the canonical form.brew install hashicorp/tap/terraform, Supabasebrew install supabase/tap/supabase).Related
Follow-up, not in this PR
The hook detects the CLI with
command -v pc, so a user who installed the old formula before 2026-03-30 still reads as "installed" and never sees the migration note; their nextbrew upgradefails on the disabled formula with no warning from the plugin. A one-linereadlinkcheck for aCellar/pineconetarget would catch it. Left out here to keep this PR to the install string; it is #47, independent of this one.Verification
brew install --cask pinecone-io/tap/pineconealone tappedpinecone-io/tap, printedTrusted cask pinecone-io/tap/pinecone(item-level trust, not whole-tap), installed cask 1.0.1, andpc versionprints 1.0.1. Exit 0. Note the tap still ships the disabled formula alongside the cask (Tapped 1 cask and 1 formula), which is why the old command produced the confusing "Treating ... as a formula" warning before failing.bash -n hooks/pinecone-api-key-check.shpasses, and running the hook with noPINECONE_API_KEYemits the new--caskcommand.grep -rn "brew tap" README.md hooksreturns nothing;grep -rn "brew install" README.md hooksshows only the--caskform (plus the unrelatedbrew install uv).CHANGELOG.mdis left alone sincerelease.ymlgenerates the entry from this PR's commit headline at merge time.