stackchain/stackchain-dashboard
1
0
Fork 0
Code Issues 9 Pull Requests 1 Actions Packages Projects Releases 713 Wiki Activity

Review saved pull request diffs and draft feedback offline #355

New Issue
Closed
opened 2026-08-09 00:09:59 +00:00 by timmy · 0 comments
timmy commented 2026-08-09 00:09:59 +00:00
Owner
Copy Link

Product gap

Requested reviews can be prioritized in Today and fully reviewed on mobile while online, but the offline path rejects them even when private offline work is enabled. This breaks the highest-value read-and-draft flow during commutes and unreliable connectivity.

Scope

  • Warm an account-bound, bounded review record for requested-review Today items, including head SHA, sanitized diff previews, CI state, and prior review context.
  • Open the existing review sheet from a cold offline launch.
  • Preserve SHA-scoped file progress, summary, decision, notes, and inline-comment drafts offline.
  • Never queue or submit a review offline. Require reconnect validation; if the head changed, preserve the prior-head draft and start review state for the new head.

Acceptance

  • A saved Today review opens offline with bounded files/diffs and prior review context.
  • Offline review drafting survives close/reload and remains scoped by repository, pull number, and head SHA.
  • Submit is disabled offline with explicit reconnect guidance; no mutation is queued.
  • Reconnect reloads current review detail. Same-head drafts remain submit-ready; changed-head drafts remain retained under the old SHA and new-head review state starts clean.
  • Opt-out, expiry, account mismatch, and clear-offline-data continue to deny/remove saved review detail.
  • Focused behavioral tests prove storage allowlisting/bounds, review-aware warming, offline opening, and reconnect validation; the full suite remains green.

Non-duplication

Historical issue and recent commit review found online review work (#117–#165, #187, #217, #240), read-only offline My Work (#228), saved Today details (#345), and automatic Today warming (#349), but no issue joins cached diffs and SHA-scoped drafts into an offline review journey.

## Product gap Requested reviews can be prioritized in Today and fully reviewed on mobile while online, but the offline path rejects them even when private offline work is enabled. This breaks the highest-value read-and-draft flow during commutes and unreliable connectivity. ## Scope - Warm an account-bound, bounded review record for requested-review Today items, including head SHA, sanitized diff previews, CI state, and prior review context. - Open the existing review sheet from a cold offline launch. - Preserve SHA-scoped file progress, summary, decision, notes, and inline-comment drafts offline. - Never queue or submit a review offline. Require reconnect validation; if the head changed, preserve the prior-head draft and start review state for the new head. ## Acceptance - A saved Today review opens offline with bounded files/diffs and prior review context. - Offline review drafting survives close/reload and remains scoped by repository, pull number, and head SHA. - Submit is disabled offline with explicit reconnect guidance; no mutation is queued. - Reconnect reloads current review detail. Same-head drafts remain submit-ready; changed-head drafts remain retained under the old SHA and new-head review state starts clean. - Opt-out, expiry, account mismatch, and clear-offline-data continue to deny/remove saved review detail. - Focused behavioral tests prove storage allowlisting/bounds, review-aware warming, offline opening, and reconnect validation; the full suite remains green. ## Non-duplication Historical issue and recent commit review found online review work (#117–#165, #187, #217, #240), read-only offline My Work (#228), saved Today details (#345), and automatic Today warming (#349), but no issue joins cached diffs and SHA-scoped drafts into an offline review journey.
timmy self-assigned this 2026-08-09 00:09:59 +00:00
timmy closed this issue 2026-08-09 00:22:08 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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 Label
P0
P1
P2
agent:nightprowl
backend
ci
frontend
meta
pages
template
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
timmy
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#355
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?