Skip to content

Added dr plugin version#656

Open
carsongee wants to merge 1 commit into
datarobot-oss:mainfrom
carsongee:carsongee/add_plugin_version
Open

Added dr plugin version#656
carsongee wants to merge 1 commit into
datarobot-oss:mainfrom
carsongee:carsongee/add_plugin_version

Conversation

@carsongee

@carsongee carsongee commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

RATIONALE

Plugins always have a version in their manifest, but we don't have a way to output them. This adds that

CHANGES

  • Creates dr plugin version <plugin> command that returns the current version.
  • Improved plugin pathing to deduplicate paths so duplicate plugin warnings are real
  • Refactored GetPlugins to use a cache and return a warnings struct along with the plugin struct for the caller to decide if warnings should be displayed and for which plugins

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 FindPlugin helper 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 from DiscoverPluginsWithContext.

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 cover FindPlugin and the not-found CLI path.

Reviewed by Cursor Bugbot for commit 96ad6e3. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

Fix All in Cursor

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 96ad6e3. Configure here.

Comment thread internal/plugin/lookup.go Outdated
Comment thread cmd/plugin/version/cmd.go
@carsongee carsongee force-pushed the carsongee/add_plugin_version branch from 96ad6e3 to 8174985 Compare July 9, 2026 15:58
@datarobot-pr-review-router

Copy link
Copy Markdown

Code Ownership

Cli Maintainers

  • cmd/plugin/cmd.go
  • cmd/plugin/discovery.go
  • cmd/plugin/list/cmd.go
  • cmd/plugin/version/cmd.go
  • cmd/plugin/version/cmd_test.go
  • internal/plugin/discover.go
  • internal/plugin/discover_test.go
  • internal/plugin/types.go

Review requested from the teams above. Labels will be removed automatically upon approval.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants