Added dr plugin version#656
Open
carsongee wants to merge 1 commit into
Open
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 96ad6e3. Configure here.
96ad6e3 to
8174985
Compare
8174985 to
7da4d2d
Compare
Code OwnershipCli Maintainers
Review requested from the teams above. Labels will be removed automatically upon approval. |
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.

RATIONALE
Plugins always have a version in their manifest, but we don't have a way to output them. This adds that
CHANGES
dr plugin version <plugin>command that returns the current version.Note
Low Risk
Read-only CLI and isolated plugin lookup; no changes to install/auth or data handling.
Overview
Adds
dr plugin version <plugin-name>so users can read a plugin’s manifest version without listing every plugin.Resolution uses a new
FindPluginhelper that follows the same priority as full discovery (managed dirs → project-local → PATH) but only touches the requested name, avoiding unrelated PATH scans and duplicate-name warnings fromDiscoverPluginsWithContext.Output is the version string by default, a message when the manifest has no version, or JSON via
--output-format json. Telemetry records the plugin name. Unit tests coverFindPluginand the not-found CLI path.Reviewed by Cursor Bugbot for commit 96ad6e3. Configure here.