[intel352]

Summary

• refactor status output into a testable helper
• always print Dirty: true/false so wfctl can fail closed on incomplete status output

Verification

• GOWORK=off go test ./pkg/cli -count=1
• GOWORK=off go test ./... -count=1
• git diff --check

[Copilot]

Pull request overview

Refactors the migration status command output into a helper function and ensures the “Dirty” state is always printed explicitly (true/false) so downstream tooling can reliably fail closed when status output is incomplete.

Changes:

• Extracted status rendering into a writeStatus(io.Writer, interfaces.MigrationStatus) helper.
• Updated the status command to write via Cobra’s OutOrStdout() and always print Dirty: %t.
• Added a unit test to assert Dirty: true/false is always present and that dirty state prints a warning.

Reviewed changes

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

File	Description
pkg/cli/root.go	Refactors status output into writeStatus and always prints explicit dirty state via the command’s writer.
pkg/cli/root_test.go	Adds coverage to ensure status output includes Dirty: true/false and emits a warning when dirty.

---

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