refactor: Use IOStreams for command output instead of os.Stdout and log

[kaizakin]

Changes

Fixes #90

The original issue was about inconsistent user facing logging.
Turns out 95% of the codebase is already following the best practice as the same as kubectl.

Following @IrvingMg's suggestion,
I've updated some inconsistencies to follow the same structure as the rest of the commands.

/kind feature

Submitter Checklist

• Includes tests if functionality changed/was added
• Includes docs if changes are user-facing
• Set a kind label on this PR
• Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.

Release Notes

Update user facing logs to follow the same consistent pattern

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign qu1queee for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Copilot]

Pull request overview

This PR standardizes user-facing CLI output by routing it through Kubernetes IOStreams (instead of writing directly to os.Stdout / fmt.Printf / log.Printf), aligning subcommands with the CLI’s existing stream-injection pattern.

Changes:

• Update version subcommand to accept IOStreams and write to ioStreams.Out.
• Switch multiple list subcommands’ tabwriter output from os.Stdout to io.Out.
• Replace log.Printf/os.Stdout output in build upload with fmt.Fprintf(ioStreams.Out, ...) and thread streams through helpers.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
pkg/shp/cmd/version/version.go	Accepts IOStreams and writes version output to ioStreams.Out.
pkg/shp/cmd/root.go	Passes ioStreams into the version command constructor.
pkg/shp/cmd/clusterbuildstrategy/list.go	Writes tabular list output to io.Out instead of os.Stdout.
pkg/shp/cmd/buildstrategy/list.go	Writes tabular list output to io.Out instead of os.Stdout.
pkg/shp/cmd/buildrun/list.go	Writes tabular list output to io.Out instead of os.Stdout.
pkg/shp/cmd/build/upload.go	Uses IOStreams for user-facing progress/info output during buildrun creation and streaming.

---

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