The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 4, 2026 6:47pm

PR Summary

Low Risk
Low risk UI-state change: it only adds defensive checks and error handling around reading persisted drafts, but could cause some drafts/attachments to be dropped if the stored data is malformed.

Overview
Prevents the /home user input from crashing when restoring a persisted mothership draft from localStorage.

Draft text is now coerced to a string during initial state setup, and the mount-time draft restore (text caret positioning, contexts, and file attachments) is wrapped in a try/catch that logs and calls clearDraft if the stored draft payload is corrupt.

^{Reviewed by Cursor Bugbot for commit b375d99. Configure here.}

Greptile Summary

This PR hardens the draft restore path in UserInput against corrupt mothership-drafts:v1 localStorage entries. The mount-time effect now builds all restored state into locals before applying any setters, logs errors via logger.error, and calls clearDraft on throw; the useState initializer adds a typeof guard so a non-string draft text no longer crashes render. Both concerns raised in the previous review round have been properly addressed.

Confidence Score: 5/5

Safe to merge — targeted, well-scoped bug fix with no new risks introduced.

No P0 or P1 findings. All concerns from the previous review round are addressed: state setters are only called after the entire read succeeds, and errors are surfaced via the project-standard logger. The typeof guard in the initializer and the try/catch in the effect together cover both failure modes described in the PR.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant C as Component Mount
    participant S as MothershipDraftsStore
    participant R as React State Setters
    participant L as Logger

    C->>C: useState initializer (typeof guard)
    C->>S: getState().drafts[key]?.text
    S-->>C: text (string | undefined | unknown)
    C->>C: typeof text === 'string' ? text : ''

    C->>C: useEffect (mount-only)
    C->>S: getState().drafts[key]
    alt draft exists & readable
        S-->>C: draft object
        C->>C: build restoredContexts / restoredFiles / caretText locally
        C->>R: setSelectedContexts(restoredContexts)
        C->>R: restoreAttachedFiles(restoredFiles)
        C->>C: textarea.setSelectionRange(caretText.length, ...)
    else draft missing
        S-->>C: undefined
        C->>C: return early (no state applied)
    else read throws (corrupt localStorage)
        S-->>C: throws
        C->>L: logger.error('Failed to read draft, clearing', { err })
        C->>S: clearDraft(draftScopeKey)
        C->>C: return early (no state applied)
    end

_{Reviews (2): Last reviewed commit: "fix(mothership): defer state writes and ..." | Re-trigger Greptile}

@greptile

@cursor review