fix(mobile): stabilize PR link input row

[iscekic]

Summary

What

• Stabilizes the mobile PR-link input row when its conditional clear control appears.
• Adds pr-link-input-row test hooks for direct native layout verification.

Why

• The clear button changed the bordered row height and shifted content below it.

How

• Gives the existing bordered row a min-h-14 floor, accounting for its border-box measurement and the clear control.
• Preserves accessibility scaling, the iOS-safe native clear path, input alignment, and the existing touch target.
• Leaves manual-review-screen.tsx unchanged: it has a fixed h-12 input and no clear button.

Verification

• cd apps/mobile && pnpm format && pnpm typecheck && pnpm lint && pnpm check:unused && pnpm test
  • 296 test files and 2,578 tests passed.
• Repair check: pnpm format apps/mobile/src/components/pr-review/pr-review-entry-screen.tsx && pnpm lint apps/mobile/src/components/pr-review/pr-review-entry-screen.tsx
• Fresh implementation review: no findings.
• GitHub CI and Kilo Code Review are green on fbaeb6ac5.
• Bot E2E passed on the final head:
  • iOS: row height 50pt empty and filled (0pt delta); clear removed visible field text and disabled Open.
  • Android: row height 129px empty and filled (0px delta); clear removed visible field text and disabled Open.

Visual Changes

iOS

Empty	Filled

Android

Empty	Filled

Reviewers

• Requested from file-history ranking: pandemicsyn, jeanduplessis.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The only PR-owned change remains the single min-h-14 floor plus test hooks on the bordered PR-link row in pr-review-entry-screen.tsx; the incremental commits since the last review are a main merge and an empty retrigger commit, so there is nothing new to flag (high confidence).

Files Reviewed (1 file)

• apps/mobile/src/components/pr-review/pr-review-entry-screen.tsx

Notes

• Verified at HEAD 109894448: line 190 is min-h-14 min-w-0 flex-1 flex-row items-center rounded-md border border-border bg-card with testID="pr-link-input-row" and collapsable={false}.
• The earlier min-h-13 border-box height-jump concern is no longer present in current code and is not carried forward; the PR description now reports E2E measurements with a 0pt/0px delta between empty and filled states on iOS and Android.
• No memory-leak, security, or async-correctness concerns in the changed line; the focus-effect cancellation and echo-FIFO logic are untouched.
• Files brought in by the main merge are outside this PR's authored scope and were not reviewed here.

Previous Review Summaries (2 snapshots, latest commit fbaeb6a)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit fbaeb6a)

Status: No Issues Found | Recommendation: Merge

Executive Summary

The incremental change raises the bordered PR-link row from min-h-13 to min-h-14, which per this app's measured scale (h-13 = 45pt, h-12 = 42pt) gives roughly 48pt border-box and now clears the 45pt clear button plus its 1pt top/bottom borders, resolving the previously reported height-jump risk.

Files Reviewed (1 file)

• apps/mobile/src/components/pr-review/pr-review-entry-screen.tsx

Notes

• Previous WARNING on line 190 (min-h-13 measured border-box) is considered resolved by the min-h-14 bump; the row's minimum now exceeds the clear button's outer height in both empty and filled states.
• Under large accessibility text scaling the input itself drives row height in both states, so the row remains stable.
• No memory-leak, security, or async-correctness concerns in the single changed line; the focus-effect cancellation and echo-FIFO logic are untouched.
• Device verification (iOS/Android captures) is still listed as blocked in the PR description, so visual confirmation of the final row height remains unavailable.

Previous review (commit 2b5f1a2)

Status: 1 Issue Found | Recommendation: Address before merge

Executive Summary

The new min-h-13 on the bordered PR-link row is measured border-box, so the fixed-height clear button can still push the row 2pt taller than the empty state, meaning the reported height jump may persist.

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
apps/mobile/src/components/pr-review/pr-review-entry-screen.tsx	190	min-h-13 includes the 1pt top/bottom borders, leaving a ~43pt content box; the clear button's fixed h-13 (45pt) can still grow the row to 47pt, so the height jump may not be eliminated

Files Reviewed (1 file)

• apps/mobile/src/components/pr-review/pr-review-entry-screen.tsx - 1 issue

Notes

• testID plus collapsable={false} are appropriate for native layout measurement; no test files are included in this PR despite the description mentioning test hooks.
• No memory-leak, security, or async-correctness concerns in the changed lines; the focus-effect cancellation and echo-FIFO logic are untouched.
• Device verification (iOS/Android captures) is still listed as pending in the PR description, which is the evidence needed to confirm or dismiss the finding above.

Fix these issues in Kilo Cloud

---

_{Reviewed by claude-opus-5 · Input: 22 · Output: 2.8K · Cached: 397.2K}

_{Review guidance: REVIEW.md from base branch main}