Skip to content

feat(agents): providers create/update flag shortcuts (draft)#3

Closed
cmarguta-alg wants to merge 3 commits intolab_week_3from
feat/agents-providers-arg-friendly
Closed

feat(agents): providers create/update flag shortcuts (draft)#3
cmarguta-alg wants to merge 3 commits intolab_week_3from
feat/agents-providers-arg-friendly

Conversation

@cmarguta-alg
Copy link
Copy Markdown
Owner

Summary

Adds optional CLI flags for common agents providers create / update flows (--name, --provider, --api-key / --api-key-stdin / --api-key-env, optional --base-url). -F JSON path unchanged; flags and -F are mutually exclusive. Azure / openai_compatible still require JSON files.

Stacked on lab_week_3 (same commits as supersede fork PR #2; prefer merging this branch and closing #2).

Documentation

  • docs/agents.md — Providers: -F vs flags
  • docs/qa/arg_friendly_providers_SIGNOFF.md — Lab Week planning-team sign-off matrix (Iris, Marek, Yuki, Diego, Priya) + Anya exit criteria and ordered testing command sheet

Test plan

  • go test ./pkg/cmd/agents/providers/...
  • With creds: go test ./e2e -tags=e2e -run TestAgents (covers providers.txtar)

Made with Cursor

cmarguta-alg and others added 3 commits May 7, 2026 15:15
Allow common provider flows without -F: --name/--provider plus
--api-key, --api-key-stdin, or --api-key-env for create (openai,
anthropic, google_genai, deepseek); patches via --name, API key flags,
and optional --base-url for update. Azure and openai_compatible still
require JSON files.

Keeps file and shortcut paths mutually exclusive. Updates agents e2e
providers.txtar for the relaxed -F requirement.

Co-authored-by: Cursor <cursoragent@cursor.com>
Adds docs/agents.md section on providers -F vs flags; links to
docs/qa/arg_friendly_providers_SIGNOFF.md (Lab Week planning-team
alignment + ordered test commands).

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@cmarguta-alg
Copy link
Copy Markdown
Owner Author

Superseded by upstream draft PR against main: algolia#214 — closing this fork-only PR to avoid duplicate review.

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.

1 participant