Skip to content

CLI cleanups - #3709

Open
noise64 wants to merge 81 commits into
mainfrom
cli-cleanups
Open

CLI cleanups#3709
noise64 wants to merge 81 commits into
mainfrom
cli-cleanups

Conversation

@noise64

@noise64 noise64 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor
  • resolves GOL-330: command / args / ux
    • explicit --auth oauth2|static profile auth mode, masked static-token prompt, clearer help/docs
    • unified structured output for bulk agent ops (update / redeploy / delete): each agent with its component and its revision/version transition
    • misc arg/help/error-context cleanups (parse_cursor errors, bail instead of unimplemented)
  • resolves GOL-331: text / output rendering
    • structural agent-id formatting for terminal display, integrated with the table rendering
  • resolves GOL-332: dead code & structural moves
    • unify the agent identifier as agentId in the output and rename agent_name -> agent_id throughout the cli internals (worker<->agent naming)
    • remove dead code (OverwriteSafeAction), streamline guess_language, make WorkerCreateView agent id non-optional
    • dissolve model/text/: command-output views now live next to their domain entity in model/<domain>; only the generic formatting toolkit remains, as model/text_format.rs
    • rename the minijinja Template trait/module to TemplateRender / model/template_render.rs; split the MCP deployment model into its own model/mcp
  • resolves GOL-333: validation & infra hardening
    • validate component preset references, with proper errors during environment / runtime selection
    • validation guards: template cycles, duplicate initial-file targets, duplicate component matches
    • healthcheck cleanups, configurable agent stream ping interval, staging approval prompt context

Deletion-output unification (output-schema change)

  • every delete / revoke / unregister renders a consistent Deleted <entity> <id> view showing id-like fields only (id + cheap on-hand handles like name / version / domain / environment), and never secret / credential / config-body values (e.g. secret delete no longer emits the secret value)
  • bulk ops (agent delete-all, card revoke) render a table + count summary
  • renamed *Result table views to *View to match the *View = renders / *Result = structured-only convention

TODO: follow-ups tracked separately — GOL-411 (redeploy fail-fast vs best-effort), GOL-412 (first-class on-demand templates), GOL-413 (non-manifest server flags / help->STDERR).

@netlify

netlify Bot commented Jul 21, 2026

Copy link
Copy Markdown

Deploy Preview for golemcloud canceled.

Name Link
🔨 Latest commit d94531a
🔍 Latest deploy log https://app.netlify.com/projects/golemcloud/deploys/6a7a38e377a47d0008436725

@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

📖 Docs preview: https://docs-hm1lbwpnt-golem-cloud.vercel.app

Built from commit d94531a88e68b6ff8607a78ae7bfaf279d24e112 by docs.yaml.

@noise64
noise64 marked this pull request as ready for review August 7, 2026 17:31
@noise64
noise64 requested a review from a team August 7, 2026 17:31
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