-
Notifications
You must be signed in to change notification settings - Fork 31
ENG-2726: Add ability to search envs + fetch actions status/logs #333
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
a579c1f to
0e7d0a1
Compare
There was a problem hiding this comment.
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 and found 3 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
5f3cd11 to
91abff6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, couple of edge cases:
Why can't I refer to my locally installed ones without a slug like what we support for prime eval? E.g. prime env status mcp-universe.
Why can't I sort by most starred envs or recently updated (I would favor that over lately created)?
| @@ -264,7 +611,76 @@ def list_cmd( | |||
| raise typer.Exit(1) | |||
|
|
|||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not moving this to time utils?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is in a utils file, i think you may have reviewed an older version
| @@ -264,7 +611,76 @@ def list_cmd( | |||
| raise typer.Exit(1) | |||
|
|
|||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not moving this to time utils?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is in a utils file, i think you may have reviewed an older version
Note
Medium Risk
Adds new CLI commands that call new hub endpoints and introduces a log-following loop with rate-limit handling; risk is mostly around API compatibility and potential noisy polling/edge cases, not data/security.
Overview
Enhances
prime envwith hub discovery improvements and action (CI job) inspection tooling.prime env listnow supports search, tag filtering, sorting, starred/mine filters (auth-gated), and optional inclusion of latest CI status; table/JSON output now includes version/stars/updated date and can show a colored action-status column. Addsprime env statusfor per-environment latest version + action status, and introducesprime env actionsubcommands to list actions, fetch/follow logs (with ANSI stripping and basic rate-limit backoff), and retry an action.Also adds
format_time_agototime_utilsand uses it for action timestamps, updates RL failure messaging to point to the new action list command, and makes minor CLI help formatting/panel organization tweaks (including\bexamples blocks).Written by Cursor Bugbot for commit c32730c. This will update automatically on new commits. Configure here.