Skip to content

[#412] Change turbo ui back to tui#426

Open
alexanderpaolini wants to merge 1 commit intomainfrom
turbo-stream-tui
Open

[#412] Change turbo ui back to tui#426
alexanderpaolini wants to merge 1 commit intomainfrom
turbo-stream-tui

Conversation

@alexanderpaolini
Copy link
Contributor

@alexanderpaolini alexanderpaolini commented Mar 24, 2026

Turbo UI at some point was changed from tui to stream (using --ui steam). I presume this was someone's AI agent doing this.

Closes: #412

Test Plan

LGTM! 😂

Screenshot 2026-03-24 at 3 05 43 AM

Checklist

  • Database: No schema changes, OR I have contacted the Development Lead to run db:push before merging
  • Environment Variables: No environment variables changed, OR I have contacted the Development Lead to modify them on Coolify BEFORE merging.

Summary by CodeRabbit

  • Chores
    • Simplified build configuration scripts to optimize development and production workflows.

Not sure why this was even changed. At some point it was. I think LLMs work better with stream than with TUI.
@alexanderpaolini alexanderpaolini requested a review from a team as a code owner March 24, 2026 07:09
@coderabbitai
Copy link

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 64153d10-2364-4f07-90f2-3af5036c9e83

📥 Commits

Reviewing files that changed from the base of the PR and between a2a5348 and 68c4e96.

📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

Removed the --ui stream flag from three Turbo script commands in the root package.json. The flag was preventing Turbo from displaying a proper task overview in the terminal output, which this change addresses.

Changes

Cohort / File(s) Summary
Turbo Configuration
package.json
Removed --ui stream flag from build, dev, and dev:blade scripts to restore Turbo's task overview display in terminal output.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

Bug, Minor

🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title meets all requirements: starts with issue number [#412], is concise and descriptive, and is well under 72 characters (34 chars).
Linked Issues check ✅ Passed The changes directly address issue #412 by removing --ui stream flags from build and dev scripts, restoring Turbo's tui UI mode for proper overview display.
Out of Scope Changes check ✅ Passed All changes are scoped to the linked issue—only package.json scripts modified to remove --ui stream flags with no extraneous alterations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
No Hardcoded Secrets ✅ Passed Pull request only modifies Turbo command-line arguments in package.json, removing --ui stream option from build/dev scripts with no sensitive data introduced.
Validated Env Access ✅ Passed PR modifies only package.json scripts section, removing --ui stream flag from Turbo commands with no code files, environment variables, or process.env usage.
No Typescript Escape Hatches ✅ Passed The PR only modifies package.json by removing --ui stream flags from turbo scripts. No TypeScript code changes are present, so no escape hatches exist in this PR.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch turbo-stream-tui

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

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

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.

Turbo no longer gives an overview

1 participant