Greptile Summary

This PR fixes two visual alignment bugs in the inline AI note rendering: it removes the extraneous blank spacer row that was appended below split-layout note cards, and it corrects column alignment by moving the note guide │ from a separate sibling <box> element into an inline <span> within the row's text content.

Key changes:

• AgentInlineNote: height formula simplified from 3 + lines.length + (anchorSide && layout === \"split\" ? 1 : 0) to 3 + lines.length, and the corresponding JSX connector block removed
• renderRows.tsx: guide rendering refactored — guideOnNewSide appends a │ span after rightRenderWidth (narrowed by 1 to preserve total width), while guideOnOldSide replaces the leftPrefix.text with │ (no width change, since it replaces marker() which is the same 1-char width). Both non-wrapped and wrapped paths for split-line and stack-line rows are updated consistently
• The old sibling-box approach subtracted the guide column from a shared contentWidth, shrinking both left and right cells equally; the new approach only narrows the side that actually hosts the guide, which also improves measureRenderedRowHeight accuracy for the left cell
• Two new/updated tests cover the blank-row removal and verify that the + rail marker is column-aligned between annotated and non-annotated rows in split layout

Confidence Score: 5/5

Safe to merge — the fix is logically consistent across all rendering paths and is well-covered by regression tests.

All three changed files are correct: the height formula and JSX removal in AgentInlineNote are exact inverses of the old spacer logic; the column-budget arithmetic in renderRows is self-consistent (left swap is width-neutral, right reduction matches the appended span); and the tests assert both the blank-row removal and the column-alignment property directly. No P0/P1 findings identified.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant RRP as reviewRenderPlan
    participant PIV as PierreDiffView
    participant AIN as AgentInlineNote
    participant DRV as DiffRowView / renderRow

    RRP->>PIV: PlannedReviewRow[] (inline-note, diff-row w/ noteGuideSide, note-guide-cap)
    PIV->>AIN: kind="inline-note" → renders note card (height = 3 + lines.length)
    Note over AIN: No trailing blank connector row (removed)
    PIV->>DRV: kind="diff-row", noteGuideSide="new"|"old"
    alt guideOnNewSide
        DRV->>DRV: rightRenderWidth = rightWidth - 1
        DRV->>DRV: append │ span at column width-1
    else guideOnOldSide
        DRV->>DRV: leftPrefix.text = "│" (replaces marker, same width)
    end
    PIV->>PIV: kind="note-guide-cap" → AgentInlineNoteGuideCap (╵ at col 0 or width-1)

_{Reviews (1): Last reviewed commit: "fix: align inline note rendering" | Re-trigger Greptile}