Skip to content

DEVEX-695: Restore agent-first root discovery for bare godaddy#52

Merged
jpage-godaddy merged 2 commits into
rust-portfrom
DEVEX-695
Jun 5, 2026
Merged

DEVEX-695: Restore agent-first root discovery for bare godaddy#52
jpage-godaddy merged 2 commits into
rust-portfrom
DEVEX-695

Conversation

@jpage-godaddy
Copy link
Copy Markdown
Collaborator

Addresses parity gap #2 from the PR #49 Rust-port review (DEVEX-695): the TypeScript CLI returned JSON discovery on bare godaddy, but the Rust port rendered clap long-help text for the empty command path — an agent-first discovery regression.

What this does

  • Bumps cli-engine to the published 0.1.3 (drops the temporary local path patch used during development).
  • Registers root next-actions via the new with_root_next_actions hook so bare godaddy:
    • piped / CI / agent (non-TTY, or --output json) → a JSON discovery envelope ({ data: { description, version }, next_actions }),
    • interactive terminal → long help + a "Suggested next actions" section.

next_actions are pointers to existing commands (auth status, env get, application list, tree) — the agent follows only what it needs, instead of one fat payload. The engine-side capability (and the curated-help / TTY-aware-output changes) shipped in cli-engine#13 → released as cli-engine 0.1.3.

Behavior

Invocation Result
godaddy (piped) JSON discovery envelope
godaddy (TTY) human help + suggested next actions
godaddy --output json / --json JSON envelope
GODADDY_OUTPUT=json godaddy JSON (env override)

Base branch

Targets rust-port (stacks on the open port PR #49), matching the convention from #50 (DEVEX-694). It will land on main with the rest of the port.

Testing

cargo build / cargo test (114 pass) / cargo clippy --all-targets -- -D warnings / cargo fmt --check all green against the published cli-engine 0.1.3. Manually verified the bare-godaddy JSON envelope (piped) and human help + suggested actions (--human).

🤖 Generated with Claude Code

Bump cli-engine to the published 0.1.3 (dropping the temporary local path
patch) and register root next-actions so bare `godaddy` returns JSON
discovery for agents and human help + suggested actions in a terminal.

Addresses parity gap #2 from the PR #49 Rust-port review.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Restores agent-first root discovery behavior for bare godaddy in the Rust CLI by upgrading cli-engine and registering curated root-level next actions so non-interactive invocations can return a discovery envelope instead of only clap help.

Changes:

  • Bump cli-engine dependency from 0.1.20.1.3.
  • Register root “next actions” via with_root_next_actions(...) for improved root discovery behavior.
  • Update Cargo.lock to reflect the dependency graph changes from the cli-engine upgrade.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
rust/src/main.rs Adds root next-actions registration for bare-command discovery flow.
rust/Cargo.toml Updates cli-engine dependency version to 0.1.3.
rust/Cargo.lock Updates resolved crate versions/checksums after dependency bump.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread rust/src/main.rs Outdated
Drop the `godaddy ` prefix so the root discovery next-actions match the
relative-path convention used by every other NextAction in the CLI
(e.g. `application list`, `api search <query>`). Addresses Copilot review on #52.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.

@jpage-godaddy jpage-godaddy merged commit 50222d0 into rust-port Jun 5, 2026
2 checks passed
@jpage-godaddy jpage-godaddy deleted the DEVEX-695 branch June 5, 2026 14:55
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.

3 participants