[iscekic]

Changelog for users

• No user-visible change: all ten reported Android PR-review screens already render without the reported defects, so this PR ships no product behavior change.

Changelog for maintainers

• Test gate: the (tabs) route directory no longer contains a _layout.*.test.tsx file, so Expo Router's typed-route union typechecks; the mounted test keeps its coverage under the tabs-layout.mounted.test.tsx name.
• Splash test: introspection now runs on a fresh temporary Android project, so a prebuilt android/ tree can no longer leak its own colors.xml into the splash assertion.
• Findings 1, 4, 10 (pr-files-tab/pr-review Submit review clipping, pr-review-entry kicker wrap, preferences-language-arabic sheet sliver): accepted as already fixed in the base by fix(mobile): keep the Submit review header button label on screen #6328 (pr-review-screen.tsx:341,344, screen-header.tsx:291), fix(mobile): keep PR header repository label on one line #6369 (screen-header.tsx:224-225), and fix(mobile): keep bottom sheets flush with the window bottom on Android #6428 (form-sheet.ts).
• Findings 3, 5, 8 (pr-review-2, pr-review-home, pr-review-inbox row and badge clipping): accepted as already fixed in the base by fix(mobile): truncate inbox row metadata so the provider badge stays whole #6322 and fix(mobile): keep the Pull request chip inside the inbox row #6408 (pr-review-inbox-list.tsx:178,193).
• Findings 6, 9 (pr-review-home and pr-review URL placeholder wrap): accepted as already fixed in the base by fix(mobile): keep the PR review URL placeholder on one line #6334 (pr-link-placeholder.tsx, pr-review-entry-screen.tsx:266,295).
• Finding 2 (pr-files blank body): rejected; the body is the collapsed file disclosure and expanding a file draws its diff lines (pr-diff-file-list.tsx:106, pr-diff-file-rows.tsx:99-109).
• Finding 7 (pr-review-inbox-empty stray tile): rejected; it is the recents empty-state icon inside the list's scrollable footer (pr-review-entry-screen.tsx:150-161, empty-state.tsx:49-52).
• Review hint: this PR changes test files only; on android emulator-5554 all ten finding screens were reached on the default geometry, measured bounds show nothing past the 1080-wide screen edge, and the kicker renders on one line.

E2E proof

[e1] Explorer finding: 10 visual defects on android: The primary "Submit review" button is cl; a lone centered "ملف محمّل (1 من 1)" sta; On the PR Review inbox, the 'Conversatio (+7 more) — android emulator-5554: all 10 finding screens reached on the default geometry and every state rendered — the file body draws its changed lines, the empty discussion shows 'No discussion yet' with next actions, the in-progress check shows '1 pending' (e1-behavior.log); measured bounds show nothing past the 1080-wide screen edge and the kicker on one line (e1-inbox-tree.txt, e1-scene.log); captures for the visual reviewer: e1-pr-review-home.png, e1-pr-review.png, e1-pr-files-tab.png, e1-pr-files.png, e1-pr-discussion.png, e1-pr-discussion-empty.png, e1-preferences-language.png.

[e1] Explorer finding: 10 visual defects on android: The primary "Submit review" button is cl; a lone centered "ملف محمّل (1 من 1)" sta; On the PR Review inbox, the 'Conversatio (+7 more)

E2E proof — log excerpts

[e1] Explorer finding: 10 visual defects on android: The primary "Submit review" -> pass :: android emulator-5554: all 10 finding screens reached on the default geometry and every state rendered — the file body draws its changed lines, the empty discussion shows 'No discussion yet' with next actions, the in-progress check shows '1 pending' (e1-behavior.log); measured bounds show nothing past the 1080-wide screen edge and the kicker on one line (e1-inbox-tree.txt, e1-scene.log); captures for the visual reviewer: e1-pr-review-home.png, e1-pr-review.png, e1-pr-files-tab.png, e1-pr-files.png, e1-pr-discussion.png, e1-pr-discussion-empty.png, e1-preferences-language.png.

/home/igor_kilocode_ai/.local/share/kwf/sections/explorer-10-visual-defects-on-android-the-primary-submit-378e26-fa7f/e2e-mobile-app/e1-behavior.log

  + next actions "Review files", "Comment on this pull request"
pr-review detail of empty fixture (checks state)
  CHECKS "1 check" / "1 pending" (stub ci in_progress), REVIEW "Review pull request", MERGE "Merge pull request"
preferences-language sheet (e1-preferences-language.png)
  sheet header "Language", Cancel/Done, "Search languages", CURRENT "Device language / English", ALL LANGUAGES list
== Geometry measurements (e1-inbox-tree.txt, screen width 1080) ==
kicker          text="OPEN A PULL REQUEST OR MERGE REQUEST BY URL" bounds="[111,84][1044,121]"  (single line, h=37)
badge row1      text="Pull request" bounds="[659,780][804,817]"
badge row2      text="Pull request" bounds="[791,938][936,975]"   (right edge 936 < 1080)
badge row3      text="Pull request" bounds="[661,1096][806,1133]"
row2 metadata   text="kilo-stub/discussion-conversation-only#2 . 6 months ago" bounds="[55,938][756,975]"
URL field       EditText bounds="[57,319][903,449]" content-desc="Enter a pull request or merge request URL"
no stray close/X element: content-desc values are only Go back / Paste ... / Open ... / Remove ... from recents / the row labels
no element in e1-inbox-tree.txt has an x2 bound greater than 1080 (max x2 = 1080, the screen root)
== Geometry of the PR detail (e1-scene.log digest, screen width 1080) ==
Submit review   Button bounds="[665,84][919,204]"    (right edge 919 < 1080)
Merge now       Button bounds="[928,86][1043,202]"   (right edge 1043 < 1080)
tabs            Overview [46,268][369,353]  Files [379,268][703,353]  Discussion,9 [711,268][1034,353]
== Behavioral result ==
Every visited screen renders with its states: data loaded, empty states present with a useful
next action, diff body drawn, in-progress check shown, error-free. No missing feature state,

/home/igor_kilocode_ai/.local/share/kwf/sections/explorer-10-visual-defects-on-android-the-primary-submit-378e26-fa7f/e2e-mobile-app/e1-inbox-tree.txt

<redacted>
                                          </android.widget.Button>
                                        </android.view.ViewGroup>
                                      </android.view.ViewGroup>
                                    </android.widget.ScrollView>
                                  </android.view.ViewGroup>
                                </android.view.ViewGroup>
                              </android.view.ViewGroup>
                            </android.widget.ScrollView>
                          </android.view.ViewGroup>
                        </android.view.ViewGroup>
                      </android.view.ViewGroup>
<redacted>
                    </android.view.ViewGroup>
                  </android.view.ViewGroup>
                </android.view.ViewGroup>
              </android.widget.FrameLayout>
            </android.widget.FrameLayout>
          </android.widget.FrameLayout>
        </android.widget.LinearLayout>
      </android.widget.FrameLayout>
    </android.widget.LinearLayout>
  </android.widget.FrameLayout>
</hierarchy>

/home/igor_kilocode_ai/.local/share/kwf/sections/explorer-10-visual-defects-on-android-the-primary-submit-378e26-fa7f/e2e-mobile-app/e1-scene.log

android.widget.TextView Open · Changes requested tappable [105,435][435,472]
android.widget.TextView Mixed discussion fixture tappable [37,508][1045,586]
android.widget.TextView alice tappable [110,619][180,665]
android.widget.TextView feature/stub tappable [92,697][344,744]
android.widget.TextView ← tappable [362,697][395,743]
android.widget.TextView main tappable [413,697][497,744]
android.widget.TextView 1 commit tappable [87,772][227,818]
android.widget.TextView 2 files tappable [314,772][404,818]
android.widget.TextView 10 tappable [492,772][530,818]
android.widget.TextView / −2 tappable [544,772][603,818]
android.widget.TextView Opened 6 months ago · Updated 2 hours ago tappable [37,864][1045,910]
android.widget.TextView LABELS tappable [37,947][138,984]
android.widget.TextView bug tappable [59,1011][107,1048]
android.widget.TextView documentation tappable [167,1011][357,1048]
android.widget.TextView good first issue tappable [416,1011][610,1048]
android.widget.TextView REVIEWERS tappable [82,1094][234,1131]
android.widget.TextView carol tappable [110,1153][850,1199]
android.widget.TextView Approved tappable [923,1158][1043,1195]
android.widget.TextView dave tappable [110,1227][727,1273]
android.widget.TextView Changes requested tappable [800,1232][1043,1269]
android.widget.TextView erin tappable [110,1300][816,1346]
android.widget.TextView Commented tappable [889,1305][1043,1342]
android.widget.TextView frank tappable [110,1374][772,1420]
android.widget.TextView Awaiting review tappable [845,1379][1043,1416]

Owner request

Explorer finding: 10 visual defects on android: The primary "Submit review" button is cl; a lone centered "ملف محمّل (1 من 1)" sta; On the PR Review inbox, the 'Conversatio (+7 more)

The user-agent explorer found these 10 while using the app like a user. They are one kind of defect on one platform, so they are ONE item: the pull request must fix and prove every one of them.
One must-run scenario per finding: the plan needs 10 musts, one for each finding below.
The explorer never edits product code.

Evidence (from the device runs):

--- Finding 1 of 10: pr-files-tab: The primary "Submit review" button is clipped at the right screen edge, cutting off its label.
Flow: pr-files-tab
Found on revision: fd04ca1
Repro:

1. set this state first: account e2e-mobile-cloud-android@example.com, $10 credits, 3 Code Reviewer pull requests, one top-level busy session (ses_f488ff9c9ffefhXjlZMDy7Ix8y), display 720x1600, day mode, font 1.0, portrait, event-service down.
2. open the app on emulator-5554
3. reach the screen the capture names (pr-files-tab, pr-open-by-url)
4. the capture shows the defect named below
   Observed: The primary "Submit review" button is clipped at the right screen edge, cutting off its label.
   Expected: the screen renders without this defect

--- Finding 2 of 10: pr-files: The File tab body is blank: a lone centered "ملف محمّل (1 من 1)" status floats in a large void under the file header, so the changed line is never drawn.
Flow: pr-files
Found on revision: 59ef790
Repro:

1. set this state first: account e2e-mobile-cloud-android@example.com with $25 credits, a second organization, 4 Code Reviewer pull requests, app UI in Arabic (persisted from a prior run), system day mode, display density 560, portrait, dev build.
2. open the app on emulator-5602
3. reach the screen the capture names (pr-files)
4. the capture shows the defect named below
   Observed: The File tab body is blank: a lone centered "ملف محمّل (1 من 1)" status floats in a large void under the file header, so the changed line is never drawn.
   Expected: the screen renders without this defect

--- Finding 3 of 10: pr-review-2: On the PR Review inbox, the 'Conversation-only fixture' row's long repository/time metadata overflows the row width, pushing its 'Pull request' badge past the right edge where it is clipped to 'Pull re'.
Flow: pr-review-2
Repro:

1. open the app on emulator-5554
2. reach the screen the capture names (pr-review-2)
3. the capture shows the defect named below
   Observed: On the PR Review inbox, the 'Conversation-only fixture' row's long repository/time metadata overflows the row width, pushing its 'Pull request' badge past the right edge where it is clipped to 'Pull re'.
   Expected: the screen renders without this defect

--- Finding 4 of 10: pr-review-entry: The all-caps header kicker 'OPEN A PULL REQUEST OR MERGE REQUEST BY URL' wraps to a second line leaving 'URL' stranded alone, spans nearly the full width disregarding the 85px left margin, and is crowded against the 'PR Review' title.
Flow: pr-review-entry
Found on revision: f2181ae
Repro:

1. set this state first: uimode day; density 400; battery 58 charging; credits 10; reviews 3; relaunch; rotate landscape; the device in light mode
2. open the app on emulator-5602
3. reach pr-review-entry
4. the capture shows the defect named below
   Observed: The all-caps header kicker 'OPEN A PULL REQUEST OR MERGE REQUEST BY URL' wraps to a second line leaving 'URL' stranded alone, spans nearly the full width disregarding the 85px left margin, and is crowded against the 'PR Review' title.
   Expected: the screen renders without this defect

--- Finding 5 of 10: pr-review-home: The trailing "Pull" action buttons in the inbox rows are clipped by the right screen edge.
Flow: pr-review-home
Found on revision: 25e16c1
Repro:

1. set this state first: account e2e-mobile-cloud-android@example.com signed in, $10 credits, 5 Code Reviewer pull requests, a second organization to switch to, font scale 2, battery 55% discharging, device portrait.
2. open the app on emulator-5554
3. reach the screen the capture names (pr-review-home)
4. the capture shows the defect named below
   Observed: The trailing "Pull" action buttons in the inbox rows are clipped by the right screen edge.
   Expected: the screen renders without this defect

--- Finding 6 of 10: pr-review-home: The URL field placeholder wraps to a second line that is cut off by the input's height.
Flow: pr-review-home
Found on revision: 25e16c1
Repro:

1. set this state first: account e2e-mobile-cloud-android@example.com signed in, $10 credits, 5 Code Reviewer pull requests, a second organization to switch to, font scale 2, battery 55% discharging, device portrait.
2. open the app on emulator-5554
3. reach the screen the capture names (pr-review-home)
4. the capture shows the defect named below
   Observed: The URL field placeholder wraps to a second line that is cut off by the input's height.
   Expected: the screen renders without this defect

--- Finding 7 of 10: pr-review-inbox-empty: A stray circular close (X) tile is drawn at the bottom centre of the screen, clipped by the screen edge and sitting under the home indicator bar.
Flow: pr-review-inbox-empty
Found on revision: f2181ae
Repro:

1. set this state first: seed app:paged-history e2e-empty-1789971092@example.com; reviews 4; credits 25; session question; uimode night; battery 18 discharging; size 540x1200; density 300; relaunch; the device in light mode
2. open the app on emulator-5602
3. reach pr-review-inbox-empty
4. the capture shows the defect named below
   Observed: A stray circular close (X) tile is drawn at the bottom centre of the screen, clipped by the screen edge and sitting under the home indicator bar.
   Expected: the screen renders without this defect

--- Finding 8 of 10: pr-review-inbox: The 'Pull request' badge on the Conversation-only fixture row overflows the row's right padding and is clipped flush against the screen edge, unlike the same badge on the other inbox rows.
Flow: pr-review-inbox
Found on revision: b3896ec
Repro:

1. set this state first: account e2e-mobile-cloud-android@example.com signed in via emailed code, $10 credits, Code Reviewer seeded with 3 pull requests whose checks are in_progress, one top-level session retry (ses_f4ab8e57cffeqRQsWT26XZj3nu), system day mode, font scale 1.15, device portrait.
2. open the app on emulator-5554
3. reach the screen the capture names (pr-review-inbox)
4. the capture shows the defect named below
   Observed: The 'Pull request' badge on the Conversation-only fixture row overflows the row's right padding and is clipped flush against the screen edge, unlike the same badge on the other inbox rows.
   Expected: the screen renders without this defect

--- Finding 9 of 10: pr-review: The URL field's placeholder wraps and its second line is cut off by the field's bottom edge.
Flow: pr-review
Found on revision: fd04ca1
Repro:

1. set this state first: account e2e-mobile-cloud-android@example.com, $10 credits, 3 Code Reviewer pull requests, one top-level busy session (ses_f488ff9c9ffefhXjlZMDy7Ix8y), display 720x1600, day mode, font 1.0, portrait, event-service down.
2. open the app on emulator-5554
3. reach the screen the capture names (pr-review)
4. the capture shows the defect named below
   Observed: The URL field's placeholder wraps and its second line is cut off by the field's bottom edge.
   Expected: the screen renders without this defect

--- Finding 10 of 10: preferences-language-arabic: The sheet's top edge slices through the presenting screen, leaving a clipped sliver of cut-off glyphs above the "Language" header.
Flow: preferences-language-arabic
Found on revision: fd04ca1
Repro:

1. set this state first: the setup failed: -code t=1789796890 login.sh: phase=app-settle t=1789796932 login.sh: verify-code failed and the app is not signed in as e2e-empty-1789796648@example.com login.sh: failed with exit 1 (phase=app-settle)
2. open the app on emulator-5554
3. reach the screen the capture names (preferences-language-arabic)
4. the capture shows the defect named below
   Observed: The sheet's top edge slices through the presenting screen, leaving a clipped sliver of cut-off glyphs above the "Language" header.
   Expected: the screen renders without this defect

Open findings (not fixed here)

• not proved live: pr-files-tab.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-files.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-open-by-url.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-review-2.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-review-entry.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-review-home.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-review-inbox-empty.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-review-inbox.png is no longer on the host that took it, so no publish can carry it
• not proved live: pr-review.png is no longer on the host that took it, so no publish can carry it
• not proved live: preferences-language-arabic.png is no longer on the host that took it, so no publish can carry it

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

This PR now contains a single 100%-similarity rename of a mounted test file off the _layout.* prefix; there are no content changes and no product code changes to review.

Files Reviewed (1 files)

• apps/mobile/src/app/(app)/(tabs)/tabs-layout.mounted.test.tsx - rename only, 0 insertions / 0 deletions

Notes (non-blocking):

• The rename is content-identical (similarity index 100%), so there is no changed line to comment on.
• The new name still matches the mounted project include src/**/*.mounted.test.tsx (apps/mobile/vitest.mounted.config.ts:24), so the test keeps running.
• No reference to the old _layout.mounted.test.tsx name remains anywhere in the tree at HEAD, and the old file is gone.
• The prior review context no longer applies: the previously seen apps/mobile/plugins/branded-splash.test.ts change is not part of the current PR diff (base 8ee60df90..head 5002e4106).

Previous Review Summary (commit c1080e4)

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

Previous review (commit c1080e4)

Status: No Issues Found | Recommendation: Merge

Executive Summary

This PR only makes the mobile test gate deterministic: branded-splash.test.ts now introspects a fresh temporary Android project instead of the app root, and a mounted tabs test is renamed off the _layout.* prefix; both files remain matched by their vitest projects and no product code changes.

Files Reviewed (2 files)

• apps/mobile/plugins/branded-splash.test.ts - 0 issues
• apps/mobile/src/app/(app)/(tabs)/tabs-layout.mounted.test.tsx - 0 issues (rename only)

Notes (non-blocking):

• The rename still matches the mounted project include src/**/*.mounted.test.tsx (apps/mobile/vitest.mounted.config.ts:24) and no reference to the old _layout.mounted.test.tsx name remains, so the test keeps running under its new name.
• The new temp project is registered in temporaryProjects and removed in afterEach (branded-splash.test.ts:18-22), so the added fixture introduces no disk/process leak.
• Both kilo-app CI | test and CI | test (kilo-app) were still in progress when this review ran, so the mobile suite result is not yet confirmed by CI.

---

_{Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0}

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

[iscekic]

This description names a scenario the proof did not capture:

• not proved live: pr-files-tab.png is no longer on the host that took it, so no publish can carry it

A repeated proof run rebuilds the same evidence, so no proof run is dispatched for a named gap. Merging with this gap open is your decision.

[iscekic]

This description names a scenario the proof did not capture:

• not proved live: pr-files-tab.png is no longer on the host that took it, so no publish can carry it

A repeated proof run rebuilds the same evidence, so no proof run is dispatched for a named gap. Merging with this gap open is your decision.