Fix confusing error message for unsupported output format; remove plain-text as advertised format

[Copilot]

Description

This PR improves the CLI output format handling by:

1. Fixing the confusing error message — previously, passing an unsupported output format produced a vague error. Now it clearly tells the user which format was rejected and lists the supported formats (json, table).
2. Removing plain-text as an advertised output format — plain-text was only accepted by rad recipe-pack show and produced output identical to table. It is no longer shown in help text or flag descriptions.
3. Maintaining backwards compatibility — users who pass --output plain-text (e.g. in scripts) will silently get table-formatted output instead of an error.

Changes

Output format normalization

• Added NormalizeFormat() in pkg/cli/output/formats.go that maps "plain-text" → "table"
• Added FormatPlainText case in NewFormatter() that returns TableFormatter (defense in depth)

Unified validation

• Rewrote RequireOutput() in pkg/cli/clivalidation.go to normalize "plain-text" → "table" before validation
• Deleted RequireOutputAllowPlainText() and requireOutputFromFormats() — no longer needed
• Deleted AddOutputFlagWithPlainText() from pkg/cli/cmd/commonflags/flags.go

Command updates

• rad recipe-pack show: switched from AddOutputFlagWithPlainText + RequireOutputAllowPlainText to standard AddOutputFlag + RequireOutput
• rad version: switched from manual flag reading to cli.RequireOutput() for consistent validation; registered output flag via commonflags.AddOutputFlag(cmd) in NewCommand()

Tests

• Updated Test_RequireOutput: "plain-text" now expects "table" (not an error)
• Removed Test_RequireOutputAllowPlainText (function deleted)
• Added pkg/cli/output/formats_test.go — tests for NormalizeFormat
• Added pkg/cli/output/formatter_test.go — tests for NewFormatter including "plain-text" case
• Added table format test case in pkg/cli/cmd/recipepack/show/show_test.go

Backwards Compatibility

User input	Before	After
rad app list -o json	JSON output	JSON output (unchanged)
rad app list -o table	Table output	Table output (unchanged)
rad app list -o plain-text	Error: unsupported format	Table output (backwards compat)
rad app list -o text	Error: unsupported format	Error: unsupported format (unchanged)
rad app list --help	"json, table"	"json, table" (unchanged)
rad recipe-pack show X -o plain-text	Table output (accepted)	Table output (still accepted, normalized)
rad recipe-pack show X -o table	Error: unsupported format	Table output (now accepted)
rad recipe-pack show --help	"plain-text, json"	"json, table" (consistent)

Key point: "plain-text" was never meaningfully different from "table" — it always produced the same output. This change makes the CLI consistent while ensuring existing scripts continue to work.

Type of change

• This pull request fixes a bug in Radius and has an approved issue (issue link required).

Contributor checklist

Please verify that the PR meets the following requirements, where applicable:

• An overview of proposed schema changes is included in a linked GitHub issue.
  • Yes
  • Not applicable
• A design document PR is created in the design-notes repository, if new APIs are being introduced.
  • Yes
  • Not applicable
• The design document has been reviewed and approved by Radius maintainers/approvers.
  • Yes
  • Not applicable
• A PR for the samples repository is created, if existing samples are affected by the changes in this PR.
  • Yes
  • Not applicable
• A PR for the documentation repository is created, if the changes in this PR affect the documentation or any user facing updates are made.
  • Yes
  • Not applicable
• A PR for the recipes repository is created, if existing recipes are affected by the changes in this PR.
  • Yes
  • Not applicable

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[brooke-hamilton]

@copilot fix this. There should not be help text saying that plain-text is a viable output unless it is supported. Also, it doesn't make sense for one of the responses to be an error and the other response to be not an error. Both inputs should result in the same response because both inputs, text and plain-text, are invalid.

/workspace/temp/testfix $ rad app list -o text
unsupported output format "text", supported formats are: json, table, plain-text

/workspace/temp/testfix $ rad app list -o plain-text
Error: unsupported format "plain-text", supported formats are: json, table

TraceId:  dab174b39e31c7772b44b9354eb66765

[Copilot]

@copilot fix this. There should not be help text saying that plain-text is a viable output unless it is supported. Also, it doesn't make sense for one of the responses to be an error and the other r...

Fixed in a924a54. RequireOutput now validates against SupportedFormats() (json, table) instead of AllFormats(), so both text and plain-text produce the same friendly error:

$ rad app list -o text
unsupported output format "text", supported formats are: json, table

$ rad app list -o plain-text
unsupported output format "plain-text", supported formats are: json, table

Added RequireOutputAllowPlainText for rad recipepack show which legitimately supports plain-text.

[brooke-hamilton]

@copilot does the same fix need to be applied to other commands?

[Copilot]

@copilot does the same fix need to be applied to other commands?

No, the fix already covers all commands. RequireOutput() validates against SupportedFormats() (json, table only), and every command uses RequireOutput() except rad recipepack show, which uses RequireOutputAllowPlainText() because it legitimately supports plain-text. The AddOutputFlag() help text and the root command's persistent flag also use SupportedFormats(), so no command advertises plain-text unless it actually supports it.

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

[brooke-hamilton]

@copilot apply changes based on the comments in this thread

[Copilot]

@copilot apply changes based on the comments in this thread

Applied both changes in 6f65972:

1. Registered the output flag in the version command via commonflags.AddOutputFlag(cmd) in NewCommand().
2. Removed the trailing blank line in clivalidation_test.go.

[radius-functional-tests]

Radius functional test overview

🔍 Go to test action run

Click here to see the test run details

Name	Value
Repository	radius-project/radius
Commit ref	21ffbe5
Unique ID	func11b9a80673
Image tag	pr-func11b9a80673

• gotestsum 1.13.0
• KinD: v0.29.0
• Dapr: 1.14.4
• Azure KeyVault CSI driver: 1.4.2
• Azure Workload identity webhook: 1.3.0
• Bicep recipe location ghcr.io/radius-project/dev/test/testrecipes/test-bicep-recipes/<name>:pr-func11b9a80673
• Terraform recipe location http://tf-module-server.radius-test-tf-module-server.svc.cluster.local/<name>.zip (in cluster)
• applications-rp test image location: ghcr.io/radius-project/dev/applications-rp:pr-func11b9a80673
• dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-func11b9a80673
• controller test image location: ghcr.io/radius-project/dev/controller:pr-func11b9a80673
• ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-func11b9a80673
• deployment-engine test image location: ghcr.io/radius-project/deployment-engine:latest

Test Status

⌛ Building Radius and pushing container images for functional tests...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
❌ Test recipe publishing failed
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting corerp-cloud functional tests...
⌛ Starting ucp-cloud functional tests...
✅ ucp-cloud functional tests succeeded
✅ corerp-cloud functional tests succeeded