Skip to content

feat(agents): providers create/update CLI flags (stacked on lab_week_3)#4

Open
cmarguta-alg wants to merge 7 commits intolab_week_3from
feat/agents-providers-arg-friendly
Open

feat(agents): providers create/update CLI flags (stacked on lab_week_3)#4
cmarguta-alg wants to merge 7 commits intolab_week_3from
feat/agents-providers-arg-friendly

Conversation

@cmarguta-alg
Copy link
Copy Markdown
Owner

Stacked on lab_week_3 (same line as PR #212 on upstream). This PR contains only the provider flag shortcuts + related docs/lint—not the full Agent Studio tree.

Commits (vs lab_week_3)

  • feat(agents): add flag shortcuts for providers create/update
  • docs: docs/agents.md + docs/qa/arg_friendly_providers_SIGNOFF.md (Anya command sheet; vet reports stay local under tmp/ only)
  • gofumpt + .golangci.yml exclude tmp/ for lint noise

Upstream review: merge PR algolia#212 first (or reconcile), then this stack can merge into lab_week_3, or cherry-pick these commits elsewhere.

Supersedes upstream PR #214 (was feat/...main; scope was wrong for review).

Test plan

  • go test ./pkg/cmd/agents/providers/...
  • go test ./e2e -tags=e2e -run TestAgents with app credentials (includes providers.txtar)

cmarguta-alg and others added 5 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>
Co-authored-by: Cursor <cursoragent@cursor.com>
tmp is cleaned by Taskfile and may contain local QA Go packages outside
review; use issues.exclude-dirs instead of deprecated run.skip-dirs.

Co-authored-by: Cursor <cursoragent@cursor.com>
cmarguta-alg and others added 2 commits May 7, 2026 17:03
Reuse build ldflags (including Agent Studio URL from .env); output binary
algolia-beta and default VERSION main+beta. Document in AGENTS and README.

Co-authored-by: Cursor <cursoragent@cursor.com>
Inject version.Distribution via task build-beta ldflags (empty for releases).
PersistentPreRunE + help wrapper on agents group warns on stderr; defaults
stay on stdout pipelines. Docs + DIST template in Taskfile.

Co-authored-by: Cursor <cursoragent@cursor.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.

1 participant