[Yash-Singh1]

The "Earlier output not shown" notice remained visible above the logs because it sat outside the output scroller.

Move it to the beginning of the scrollable content on web, desktop, and mobile so it scrolls away with the retained output. The web panel also drops the now-unneeded outer wrapper.

Validation: vp run check:changed passed, including format, lint, knip, affected typechecks, React compiler checks, and seven selected tests. Visual verification is pending.

Model: GPT 6.1 Sol. Harness: Codex in T3 Code.

[macroscopeapp]

Approvability

Verdict: Approved at 14b6e4f

Macroscope's review found this PR approvable — The truncation notice now scrolls with command output on web and mobile instead of remaining above it. This is a limited presentation change with no changes to command execution or stored output.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The mobile and web command output panels now place the truncation notice inside the scrollable container, before the terminal output.

Changes: Command output notice placement

Layer / File(s)	Summary
Place notice in scrollable output apps/mobile/src/features/threads/thread-command-output.tsx, apps/web/src/components/chat/CommandOutputPanel.tsx	Both panels render the truncation notice inside the scrollable container and before the output. The mobile panel shows the notice when data?.output.truncated is true.

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Bug fix

Suggested labels: bugfix, 📱 Native Change, size:S

Suggested reviewers: maria-rcks

Merge Risk: Minimal · up to 14b6e

The notice now scrolls with retained output. No actionable merge-blocking risk is identified; visual verification remains pending.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly describes the main change: making the truncated command output notice scroll with the logs.
Description check		The description explains the problem, the change across clients, and the reported automated checks. It does not provide scope approval or explain why the focused fix qualifies without it. It also stat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description explains the problem, the change across clients, and the reported automated checks. It does not provide scope approval or explain why the focused fix qualifies without it. It also states that visual verification is pending, but the template requires before-and-after screenshots for UI changes.

Finishing Touches

Generate docstrings

• Commit to this branch
• Create a new PR

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: t3/move-output-notice-top
• File: .coderabbit.config.ts
• Commit: 14b6e4f
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Team
• Run ID: 1905b33e-60b8-4137-a717-9bb23b265503

Commits

Reviewing files that changed from the base of the PR and between 1fed70c and 14b6e4f.

Files selected for processing: 2

• apps/mobile/src/features/threads/thread-command-output.tsx
• apps/web/src/components/chat/CommandOutputPanel.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 2 remain after this review.