Fix duplicate success marker in gh aw fix --write output

[Copilot]

gh aw fix --write printed per-file success lines with a duplicated checkmark (✓ ✓ file.md), which reduced output readability during bulk codemod runs. This change normalizes the success line to a single marker while preserving existing fix detail output.

• Output formatting correction

  • Updated processWorkflowFileWithInfo in pkg/cli/fix_command.go to pass only the filename to console.FormatSuccessMessage, removing the redundant inline ✓ prefix from the call site.
  • Resulting output now renders as ✓ <file> instead of ✓ ✓ <file>.

• Regression coverage

  • Added TestProcessWorkflowFileWithInfo_WriteOutputUsesSingleCheckmark in pkg/cli/fix_command_test.go.
  • The test captures stderr for a write-path codemod application and asserts:
    • expected single-checkmark line is present
    • duplicate-checkmark variant is absent

// before
fmt.Fprintf(os.Stderr, "%s\n", console.FormatSuccessMessage("✓ "+fileName))

// after
fmt.Fprintf(os.Stderr, "%s\n", console.FormatSuccessMessage(fileName))

[Copilot]

Pull request overview

This PR improves gh aw fix --write console output readability by eliminating a duplicated success marker in per-file success lines, and adds a regression test to prevent the duplication from reappearing.

Changes:

• Updated processWorkflowFileWithInfo to pass only the filename to console.FormatSuccessMessage, avoiding ✓ ✓ <file> output.
• Added a regression test covering the write-path output formatting.

Show a summary per file

File	Description
pkg/cli/fix_command.go	Removes redundant inline ✓ prefix for per-file success output (but the non-write warning path still has a duplicated marker).
pkg/cli/fix_command_test.go	Adds a regression test for single-checkmark output (assertions need to be robust to ANSI styling/TTY behavior).

Copilot's findings

Tip

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

• Files reviewed: 2/2 changed files
• Comments generated: 2