Skip to content

feat(loo-4729): api-key command#2

Merged
notnmeyer merged 7 commits intomainfrom
feat/api-key
Mar 17, 2026
Merged

feat(loo-4729): api-key command#2
notnmeyer merged 7 commits intomainfrom
feat/api-key

Conversation

@notnmeyer
Copy link
Copy Markdown
Member

@notnmeyer notnmeyer commented Mar 16, 2026

  • adds api-key subcommand for /api-key
➜ ENV=staging task -- api-key
task: [default] go run ./... api-key
Valid API key for team: nate-send4.l3s.email
  • verify a key is legit before saving with login
# boo
➜ task -- login
task: [default] go run ./... login
Enter your API key:
Error: API key verification failed: invalid API key
Usage:
  loops login [flags]

Flags:
  -h, --help   help for login

exit status 1
task: Failed to run task "default": exit status 1

# yay
➜ task -- login
task: [default] go run ./... login
Enter your API key:
API key saved. Authenticated as team: nate-send4.l3s.email
  • dev: the default task adds support for a dotfile file supplied with ENV, as seen above

@notnmeyer notnmeyer changed the title feat: /api-key feat: api-key command Mar 16, 2026
@notnmeyer notnmeyer self-assigned this Mar 17, 2026
@notnmeyer notnmeyer marked this pull request as ready for review March 17, 2026 12:44
@notnmeyer notnmeyer requested a review from nalanj March 17, 2026 12:44
Comment thread cmd/api_key.go
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think in general I prefer [noun] [verb] command patterns for this kind of thing. So like on auth I'd probably do auth login, auth logout, and auth status

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

login was added in #1. can make a linear for it though, wont address it here. i was debating originally whether to have the top level be auth or not. 👍

Comment thread internal/api/client.go Outdated
Comment thread internal/api/api_key.go Outdated
@notnmeyer notnmeyer requested a review from nalanj March 17, 2026 13:51
@notnmeyer notnmeyer merged commit 4f725ca into main Mar 17, 2026
2 checks passed
@notnmeyer notnmeyer changed the title feat: api-key command feat(loo-4729): api-key command Mar 17, 2026
@linear
Copy link
Copy Markdown

linear Bot commented Mar 17, 2026

LOO-4729

@notnmeyer notnmeyer deleted the feat/api-key branch March 30, 2026 18:48
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