Skip to content

feat(api-keys): roboflow api-key CLI#495

Open
yeldarby wants to merge 6 commits into
mainfrom
api-key-management
Open

feat(api-keys): roboflow api-key CLI#495
yeldarby wants to merge 6 commits into
mainfrom
api-key-management

Conversation

@yeldarby

Copy link
Copy Markdown
Contributor

Adds a roboflow api-key CLI command group (list / get / create / update / revoke / protect / disable / publishable) wrapping the new REST endpoints.

  • Secret shown once on create; non-secret keyId handles; --scope / --metadata on create + update; protected-key 409 and plan-feature 403 hints; consistent exit codes and --json envelopes.
  • Tests in tests/cli/test_api_key.py; version bump.

🤖 Generated with Claude Code

yeldarby and others added 2 commits June 19, 2026 23:06
Checkpoint of the API key management feature work (round 1 + refinement pass).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
… hint

Round-2 refinement fixes (independent re-review).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Comment thread roboflow/cli/handlers/api_key.py Fixed
yeldarby and others added 4 commits June 20, 2026 10:02
Emit the create result (incl. the one-time secret) via the shared output() helper like every other handler, instead of a raw print(). Resolves a CodeQL clear-text-logging alert and matches the repo's documented output idiom; stdout is unchanged in both --json and human modes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…sets

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
… + presets

External review (P2): omitting scopes inherits the caller's scopes (not unconditional full access); note role:<name> presets and [] for no abilities.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
External review (skills PR): the command-groups reference omitted api-key, so agents couldn't discover it. The group ships in this PR.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants