fix(ios): add compact header menu clearance

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}

[saphid]

What Changed

• Add 10 points of trailing clearance to the existing thread header only at compact width.
• Keep the regular-width header and the 44-point thread-actions menu unchanged.

Why

On a compact iPhone, a long thread title and live status metadata can run visually into the trailing actions menu. The header already owns both rows, so applying the clearance there keeps their leading/trailing alignment together without changing menu behavior or wider layouts.

This remains a two-line layout fix, separate from the live-duration work in #5611 and stacked on #5178.

UI Changes

Before	After

This is a static spacing change. It does not alter motion, timing, transitions, or interaction, so the navigation-only video was removed.

Verification

• git diff --check upstream/t3code/rebuild-mobile-app-swift...HEAD — exit 0.
• SwiftUI Debug build on iPhone 17e / iOS 26.5 — succeeded.
• Focused HomeThreadMetadataTests Simulator run — 5 passed, 0 failed.
• Integrated compact-width pass on iPhone 17e: long title, branch/environment metadata, and live working status remained clear of the menu; the independently exposed “Thread actions” target opened its full menu.
• Integrated regular-width pass on iPad Pro 11-inch / iOS 26.5: existing split-view header layout and 44-point menu target remained unchanged.

Scope

SwiftUI mobile only. No web, desktop, React Native mobile, provider, contract, connection-mode, or documentation behavior changes.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included clear before/after screenshots
• Video is not applicable because animation and interaction behavior are unchanged
• The branch contains no unrelated, generated, personal, or private files

PR refresh and Simulator verification used GPT-5.6 Sol in T3 Code.

Summary by CodeRabbit

• Bug Fixes
  • Improved compact-width thread layouts by adding clearance between the thread header content and its trailing menu.

---

Note

Low Risk
SwiftUI layout-only tweak on compact iPhone; no logic, API, or interaction changes.

Overview
Adds 10pt trailing padding to the thread navigation header (threadHeaderTitle) when horizontalSizeClass is compact, so long titles and metadata no longer sit under the trailing Thread actions menu.

Regular-width layouts are unchanged (0 padding); the menu’s 44pt tap target and behavior are untouched.

^{Reviewed by Cursor Bugbot for commit fca9ced. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add 10pt trailing padding to compact header menu in ThreadDetailView

Fixes header menu clipping in compact width by adding 10pt of trailing padding when horizontalSizeClass is .compact. No padding is added in regular width. The fix is applied in ThreadDetailView.swift before the existing frame modifier.

^{Macroscope summarized fca9ced.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The thread header now adds 10 points of trailing padding on compact-width layouts. Regular-width layouts remain unchanged.

Changes

Thread Header Spacing

Layer / File(s)	Summary
Compact header padding apps/swift-ios/Features/Chat/ThreadDetailView.swift	The thread header reserves 10 points of trailing space on compact-width layouts and no additional space on regular-width layouts.

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

Suggested reviewers:t3dotgg

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the compact iOS header menu clearance fix.
Description check	✅ Passed	The description covers the change, rationale, UI screenshots, verification, scope, and checklist for this focused UI fix.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[saphid]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[saphid]

Branch-owner status — verified head fca9ced31

The PR patch is intentionally still one commit, one file, and two added lines. I rebased it from the prior stacking-base SHA onto current ab95c07bd7df4afb015f52c8920d40f85ae523a0; git range-diff showed the patch unchanged.

Completed evidence and review:

• Public before/after images render without authentication. The navigation-only video was removed because this change has no motion, timing, transition, or interaction behavior.
• SwiftUI Debug build succeeded on iPhone 17e / iOS 26.5.
• Focused HomeThreadMetadataTests: 5 passed, 0 failed.
• Integrated compact pass covered a long title, branch/environment metadata, live working duration, and opening the independently exposed Thread actions menu. Integrated iPad pass confirmed the regular-width split layout remains unchanged.
• CodeRabbit manually reviewed exactly ab95c07..fca9ced and generated no actionable comments.
• Fresh read-only cross-provider review launcher: claude --model claude-opus-5 --effort high --permission-mode plan --output-format json -p …. Actual model: Claude Opus 5; process exit status: 0; target: ab95c07..fca9ced; findings: no actionable findings.

This PR is currently blocked only by shared stacking-base CI:

• CI / Check fails in scripts/generate-swift-wire-fixtures.ts on two namespace-import lint errors.
• SwiftUI iOS / Contract fixtures and native tests fails in fixture generation because the shared script cannot resolve effect, before native tests start.
• The separate Vercel result requires external-fork authorization and is not a code failure.

Those defects are outside this two-line ThreadDetailView.swift patch. The shared repair in #5674 is currently green for both CI and SwiftUI iOS, but remains a draft and has not landed in t3code/rebuild-mobile-app-swift. I am therefore keeping #5663 draft rather than misrepresenting it as ready. Once #5674 (or its equivalent) reaches the stacking base, the remaining mechanical steps are: refresh this branch once, verify the range-diff, let checks rerun, and mark ready for human maintainer review.

[macroscopeapp]

Approvability

Verdict: Approved fca9ced

Minor UI fix adding 10px trailing padding on compact iOS screens to provide clearance for a menu. The change is purely cosmetic and self-contained with no logic or behavioral impact.

^{You can customize Macroscope's approvability policy. Learn more.}