Skip to content

Replace raw usage of fmt.Print with pcio.Print#73

Merged
austin-denoble merged 1 commit intomainfrom
adenoble/fix-raw-print-in-prompts
Mar 19, 2026
Merged

Replace raw usage of fmt.Print with pcio.Print#73
austin-denoble merged 1 commit intomainfrom
adenoble/fix-raw-print-in-prompts

Conversation

@austin-denoble
Copy link
Collaborator

@austin-denoble austin-denoble commented Mar 19, 2026

Problem

There's confirmation prompts where we're using fmt.Print directly rather than pcio.Print. We're going to be refactoring the pcio package more broadly, so for now just align the raw Print calls with the shim'd package.

Solution

Replace raw usage of fmt.Print with pcio.Print.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Infrastructure change (CI configs, etc)
  • Non-code change (docs, etc)
  • None of the above: (explain here)

Test Plan

There should be no user-visible change due to this code change.


Note

Low Risk
Low risk: small output-path changes for interactive prompts, with one error-message formatting tweak; no API or deletion logic changes.

Overview
Switches several destructive-action confirmation prompts (API key, project, organization, and local key prune) from fmt.Print to pcio.Print so prompt output respects the CLI’s pcio shim/quiet-mode behavior.

Also standardizes the input read error path in auth local-keys prune to use msg.FailMsg instead of printing directly to stdout.

Written by Cursor Bugbot for commit 4f97284. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@austin-denoble austin-denoble merged commit 1990286 into main Mar 19, 2026
8 checks passed
@austin-denoble austin-denoble deleted the adenoble/fix-raw-print-in-prompts branch March 19, 2026 21:07
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.

1 participant