Display error with TUI and suppress duplicate error

[carole-lavillonniere]

Output of lstk stop for container not running:

Before

After

• Show errors in the TUI error display
• Suppress duplicate plain-text output
• Cobra's SilenceErrors = true stops from printing RunE errors itself.
• Cobras' SilenceUsage = true stops from dumping the full usage block on every runtime error (e.g. "container not running"). Flag parsing errors are caught by cobra before RunE is reached, so usage suppression does not affect those.

[anisaoshafi]

Nice! 🧹

[coderabbitai]

Warning

Rate limit exceeded

@carole-lavillonniere has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 28 minutes and 19 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: a6c909f8-49e6-4998-a2c7-71b5091531b8

📥 Commits

Reviewing files that changed from the base of the PR and between 08b3364 and 1367bfa.

📒 Files selected for processing (4)

• cmd/root.go
• internal/ui/app.go
• internal/ui/run.go
• internal/ui/run_stop.go

📝 Walkthrough

Walkthrough

The changes modify error handling and display behavior in the CLI application. The root command now silences errors and usage output, error messages are displayed through an ErrorEvent system in the UI app, and application errors are wrapped with a silent error wrapper before being returned to callers.

Changes

Cohort / File(s)	Summary
Root Command Configuration cmd/root.go	Enables silencing of errors and usage output on the root Cobra command.
Error Display and Wrapping internal/ui/app.go, internal/ui/run.go, internal/ui/run_stop.go	Updates error visibility via ErrorEvent display in app error handler and wraps app errors with SilentError wrapper in error return paths.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: displaying errors in the TUI and suppressing duplicate error output through Cobra's silence flags.
Description check	✅ Passed	The description is directly related to the changeset, explaining the motivation (showing errors in TUI, suppressing duplicates), the implementation approach (Cobra silence flags), and including before/after screenshots demonstrating the improvement.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-stop-err

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@cmd/root.go`:
- Around line 37-38: The PR description incorrectly states that "Flag parsing
errors (caught by Cobra before `RunE`) are unaffected by usage suppression";
update the PR description to accurately reflect that Cobra respects
`root.SilenceUsage` for flag parsing errors as well, so when `root.SilenceUsage
= true` usage output is suppressed for flag parsing errors; mention
`root.SilenceUsage` (and `root.SilenceErrors` if relevant) in the description
and clarify whether the current behavior—no usage output on any error—is
intentional.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 93c37bd9-b51c-4fe4-b61b-820ca0fef72e

📥 Commits

Reviewing files that changed from the base of the PR and between 4819f11 and 08b3364.

📒 Files selected for processing (4)

• cmd/root.go
• internal/ui/app.go
• internal/ui/run.go
• internal/ui/run_stop.go