Apply reviewer code suggestions from mobile #1383

Merged

timmy merged 1 commits from timmy/1382-mobile-review-suggestions into main 2026-08-25 06:19:22 +00:00

Conversation 0 Commits 1 Files Changed 6

+267 -1

timmy commented 2026-08-25 06:08:46 +00:00

Owner

Copy link

Copy source

Closes #1382

What changed

• recognize one bounded fenced suggestion on current inline review feedback
• preview the exact current line and proposed replacement before committing
• apply through existing head/blob conflict guards while keeping full-file fallback
• clear stale previews on comment navigation and keep the flow read-only offline

Verification

• RED: backend suggestion normalization failed with missing suggestion
• RED: mobile journey failed with missing suggestion action; stale preview navigation test then failed
• focused: 80 passed
• full: 2628 passed, 22 skipped
• rendered at 320x568 and 390x844 with no overflow and 44px controls

Closes #1382 ## What changed - recognize one bounded fenced suggestion on current inline review feedback - preview the exact current line and proposed replacement before committing - apply through existing head/blob conflict guards while keeping full-file fallback - clear stale previews on comment navigation and keep the flow read-only offline ## Verification - RED: backend suggestion normalization failed with missing `suggestion` - RED: mobile journey failed with missing suggestion action; stale preview navigation test then failed - focused: 80 passed - full: 2628 passed, 22 skipped - rendered at 320x568 and 390x844 with no overflow and 44px controls

timmy added 1 commit 2026-08-25 06:08:47 +00:00

feat: apply mobile review suggestions (Closes #1382) ed662c3a3a

timmy merged commit c8ffde7d96 into main 2026-08-25 06:19:22 +00:00

timmy referenced this issue from a commit 2026-08-25 06:19:23 +00:00

Merge pull request 'Apply reviewer code suggestions from mobile' (#1383) from timmy/1382-mobile-review-suggestions into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

Drop everything

P1

High priority

P2

Medium priority

agent:nightprowl

Assignments routed to the LongCat NightProwl evaluation worker

backend

Backend service

ci

Continuous integration / actions

frontend

Frontend / UI

meta

Meta / planning

pages

Gitea Pages

template

Repo template / standards

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

nightprowl timmy (Timmy · the Wizard Mind) turdbot zephyrus-sparkborn (Zephyrus Sparkborn)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#1383