Skip to content

fix(tui): guide bug reports toward failure causes#2553

Draft
cyq1017 wants to merge 1 commit into
Hmbown:mainfrom
cyq1017:codex/2022-feedback-bug-hints
Draft

fix(tui): guide bug reports toward failure causes#2553
cyq1017 wants to merge 1 commit into
Hmbown:mainfrom
cyq1017:codex/2022-feedback-bug-hints

Conversation

@cyq1017
Copy link
Copy Markdown
Contributor

@cyq1017 cyq1017 commented Jun 1, 2026

Summary:

  • add privacy-first diagnostics guidance to the /feedback bug message
  • point users toward environment/tool failure categories before filing
  • keep the GitHub template URL unprefilled so prompts, logs, and paths are not copied automatically

Validation:

  • cargo test -p codewhale-tui --all-features --locked commands::feedback -- --nocapture
  • cargo fmt --all -- --check
  • cargo check -p codewhale-tui --all-features --locked
  • git diff --check

Refs #2022

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

cyq1017 has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@Hmbown
Copy link
Copy Markdown
Owner

Hmbown commented Jun 2, 2026

Harvested into #2504 as 7c06cf598 for v0.8.50 triage.

I kept this as a narrow #2022 slice: /feedback bug now adds privacy-first diagnostics guidance, keeps the issue URL template-only, and avoids prefilled prompts/logs/paths. Local verification on the harvest branch:

  • cargo test -p codewhale-tui --all-features --locked commands::feedback -- --nocapture
  • cargo fmt --all -- --check
  • git diff --check
  • ./scripts/release/check-versions.sh
  • cargo clippy -p codewhale-tui --all-targets --all-features --locked -- -D warnings

Thanks @cyq1017.

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.

2 participants