feat: rename binary from privateer to pvtr#202
Merged
Conversation
## What Renamed the CLI binary and all build artifacts from privateer to pvtr. The cobra root command, GoReleaser config, Makefile, Dockerfile, install script, and integration tests all reference the new binary name. A backward-compatible symlink (privateer → pvtr) is included in the Homebrew formula via GoReleaser. ## Why The project is rebranding the CLI binary name from privateer to pvtr while keeping the GitHub org, repo, and Go module path unchanged. ## Notes - The plugin filter in cmd/env.go now excludes both "pvtr" and "privateer" named binaries so legacy installs don't appear as plugins - The Homebrew formula includes bin.install_symlink "pvtr" => "privateer" for backward compatibility — remove this in a future release - Config paths (~/.privateer/) and test config service keys are intentionally unchanged; the SDK still owns those paths Signed-off-by: jmeridth <jmeridth@gmail.com>
jmeridth
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
Renames the CLI binary from privateer to pvtr across the codebase while keeping the GitHub org/repo and Go module path unchanged, and adds Homebrew symlink compatibility (privateer → pvtr).
Changes:
- Renames the cobra root command and user-facing CLI references from
privateertopvtr. - Updates build/release artifacts and packaging references (Makefile, Dockerfile, GoReleaser, install script).
- Updates docs and tests to use the new binary name and adjusts plugin filtering behavior.
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| test/integration_test.sh | Updates integration script to execute ./pvtr and adjusts related comments/config wording. |
| main.go | Updates top-level package/main documentation to refer to pvtr. |
| install.sh | Renames installed binary target from privateer to pvtr and updates completion messaging. |
| cmd/version.go | Updates version command help text to refer to pvtr. |
| cmd/generate-plugin.go | Updates generate-plugin command docs to refer to pvtr. |
| cmd/execute.go | Renames cobra root Use value from privateer to pvtr. |
| cmd/env_test.go | Updates tests around plugin discovery filtering to include pvtr naming. |
| cmd/env.go | Updates plugin discovery filtering to exclude both pvtr and privateer. |
| README.md | Updates README branding and install instructions to pvtr. |
| Makefile | Renames build outputs from privateer-* to pvtr-*. |
| Dockerfile | Updates container entrypoint to /app/pvtr. |
| .goreleaser.yaml | Renames the Homebrew formula to pvtr and adds a symlink for backward compatibility. |
| .gitignore | Adds ignores for pvtr artifacts and keeps legacy ignores. |
| .github/ISSUE_TEMPLATE/bug_report.yml | Updates issue template wording to request pvtr version. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
## What Added explicit `binary: pvtr` to GoReleaser builds config so the archive contains the correctly named binary. Changed the plugin discovery filter from substring match to exact match so pvtr-* plugins like pvtr-github-repo-scanner remain discoverable. Reverted the README project structure directory name back to privateer/ since the repo is not being renamed. ## Why Copilot review correctly identified that GoReleaser would default the binary name to the module name (privateer), causing Homebrew install to fail. The substring filter would have hidden all pvtr-prefixed plugins from env output. ## Notes - The plugin filter now uses exact name matching (name == "pvtr" || name == "privateer") instead of strings.Contains - project_name remains privateer (archive naming unchanged), only the binary inside the archive is renamed to pvtr Signed-off-by: jmeridth <jmeridth@gmail.com>
eddie-knight
previously approved these changes
Mar 14, 2026
…generated by goreleaser
eddie-knight
approved these changes
Mar 14, 2026
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.
What
Renamed the CLI binary and all build artifacts from privateer to pvtr. The cobra root command, GoReleaser config, Makefile, Dockerfile, install script, and integration tests all reference the new binary name. A backward-compatible symlink (privateer → pvtr) is included in the Homebrew formula via GoReleaser.
Why
The project is rebranding the CLI binary name from privateer to pvtr while keeping the GitHub org, repo, and Go module path unchanged.
Notes
bin.install_symlink "pvtr" => "privateer"for backward compatibility — remove this in a future release