Skip to content

fix(output): render next actions for custom views - #116

Merged
jpage-godaddy merged 1 commit into
mainfrom
shopping-human-rendering
Sep 11, 2026
Merged

fix(output): render next actions for custom views#116
jpage-godaddy merged 1 commit into
mainfrom
shopping-human-rendering

Conversation

@sswaminathan-godaddy

Copy link
Copy Markdown
Collaborator

Objective

Make human output consistent for commands that register a custom human renderer: preserve the renderer body and append the standard Next steps: footer from the command result.

Changes

  • Append existing envelope next actions after a custom human renderer returns.
  • Reuse cli-engine's existing parameter substitution and footer behavior.
  • Add focused integration coverage for a custom view with a parameterized next action.
  • Move the focused test to its own integration-test file to retain the 1,000-line module limit.

Verification

cargo fmt --all --check
cargo clippy --all-targets -- -D warnings
RUSTDOCFLAGS='-D warnings' cargo doc --package cli-engine --no-deps
cargo test --all-targets
cargo test --doc
./cli-engine/scripts/check-module-size.sh

All passed.

🤖 Generated with Claude Code

Ensure custom human renderers retain the standard next-action footer, including structured parameter substitution, so consumer CLIs do not need to mirror engine rendering logic.

Co-Authored-By: Claude <noreply@anthropic.com>
@jpage-godaddy
jpage-godaddy merged commit ef1132e into main Sep 11, 2026
2 checks passed
@jpage-godaddy
jpage-godaddy deleted the shopping-human-rendering branch September 11, 2026 22:21
@github-actions github-actions Bot mentioned this pull request Sep 11, 2026
jpage-godaddy pushed a commit that referenced this pull request Sep 11, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>cli-engine: 0.9.5</summary>

##
[0.9.5](cli-engine-v0.9.4...cli-engine-v0.9.5)
(2026-09-11)


### Features

* **structure:** enforce 1000-line module cap and split oversized files
([#113](#113))
([860e15f](860e15f))


### Bug Fixes

* **output:** render next actions for custom views
([#116](#116))
([ef1132e](ef1132e))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.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.

2 participants