testing: portrait positioning, bottom-fill conversation, CLAUDE.md cleanup

[arcaven]

Changes to test

Three commits landed on develop (646438b, 1d6d09c, c141918):

1. Portrait positioning — flush to corners (refs #38, #39)

What changed:

• Removed extra margin between portrait and terminal edges
• TopRight: flush with top of conversation area (was 1-cell gap)
• BottomRight: flush against input border (was 1-cell gap above status bar)
• Right edge keeps 1-cell margin for breathing room
• Default changed: top-right medium → bottom-right large

Test objectives:

• Launch forestage — portrait appears bottom-right, large size
• Portrait bottom edge sits directly against the input area border (no blank line gap)
• Portrait right edge is close to terminal right (1 cell margin max)
• Ctrl+P toggles to top-right — portrait sits flush with conversation top
• Alt+S cycles sizes — positioning stays anchored at all sizes
• Resize terminal — portrait re-anchors correctly in both positions
• With no portrait loaded (theme without portraits) — no rendering artifacts

2. Bottom-fill conversation (refs #41)

What changed:

• Conversation content now renders from bottom of viewport upward when content is shorter than the viewport
• First message appears just above the input, not pinned to top with blank space below

Test objectives:

• Fresh session: first prompt + response appear near the bottom, above input
• Empty space is above the content, not below it
• As conversation grows past one viewport, scrolling works normally
• Manual scroll (PgUp/PgDn/arrows) still works
• Auto-scroll follows new content during streaming
• End key re-enables auto-scroll after manual scroll

3. CLAUDE.md cleanup

• Removed stale hooks.rs reference
• Added all current source files to architecture tree
• No behavioral change — documentation only

Feedback wanted

• Does the bottom-right large portrait feel like the right default? Too large? Right position?
• Does the bottom-fill feel natural on first message, or is the transition jarring when content fills the viewport?
• Any rendering artifacts or edge cases with portrait + bottom-fill interaction?

[arcaven]

Portrait Positioning Test Results

Tester: Michael
Build: develop (646438b, 1d6d09c, c141918)

Default position and size

• Default is bottom-right — pass
• Default is medium (not large per commit notes) — pass (acceptable; Default portrait position should be bottom-right large, not top-right medium #39 asked for bottom-right large but medium may be the better default — need feedback)

Bottom position — vertical gap varies by size

• Original: ~0.5 line gap above top bar — closest to correct
• Small: ~1 full cursor height above top bar — fail
• Medium: ~1.25 cursor height above top bar — fail
• Large: ~1 cursor height above top bar — fail

The gap between the portrait bottom edge and the input area border is size-dependent, suggesting the anchor calculation doesn't account for the image height correctly at each size. Likely the vertical position is computed from a fixed offset rather than (border_row - image_height_in_cells).

Bottom position — horizontal

• ~0.5 cursor width from right edge — pass (acceptable breathing room)

Top position

• Portrait positioning is perfect at all sizes — pass

Summary

Top-right anchoring works. Bottom-right has a size-dependent vertical gap — the larger the portrait, the further it drifts from the input border. The anchor point needs to be computed relative to the bottom edge (input border row) minus the portrait's cell height, not from a fixed position.

Refs #38, #39

[arcaven]

Input & Text Test Results

Bottom-fill conversation

• Text feeds from the bottom — pass

User message display

• "You:" preamble wastes a line — fail → filed Remove "You:" preamble from user messages in conversation view #47
• No visual distinction for user text (needs background color) — noted, kos idea captured

Copy/paste/selection

• Cannot copy from conversation area — fail (existing Text selection, copy, paste, and drag-drop file support in the TUI #40)
• Cannot copy from input field — fail (existing Text selection, copy, paste, and drag-drop file support in the TUI #40)
• Paste works — pass
• Drag image file shows as @/path — pass (useful behavior)

Large paste handling

• After ~8 lines, cursor lost in status bar, input invisible — fail → filed Large paste overflows input area — cursor lost, text invisible #48

Input navigation

• Arrow up/down navigates history instead of multiline text — fail → filed Arrow keys in multiline input should navigate text, not history #49
• Ctrl+U clears input — pass
• Escape does not clear input — fail (should it? needs design decision)

New issues filed

• Remove "You:" preamble from user messages in conversation view #47 — Remove "You:" preamble
• Large paste overflows input area — cursor lost, text invisible #48 — Large paste overflow
• Arrow keys in multiline input should navigate text, not history #49 — Arrow keys in multiline input should navigate text, not history