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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 28, 2026 9:15pm

PR Summary

Medium Risk
Medium risk because it introduces interactive editing from the markdown preview via index-based parsing/regex replacement, which could toggle the wrong checkbox or mutate content unexpectedly; it’s gated off during streaming and when editing is disabled.

Overview
Adds interactive task-list checkboxes to the markdown preview: task list items are rendered with the app’s Checkbox component and clicking a checkbox updates the underlying markdown content (and therefore autosave) without switching to raw-text editing.

Preview behavior is adjusted so checkbox interactivity is only enabled when canEdit and not streaming, and streaming previews continue to use the revealed/animated content path.

^{Written by Cursor Bugbot for commit 51163c6. Configure here.}

Greptile Summary

This PR adds interactive markdown checkbox toggling to the file preview panel — users can click checkboxes in the rendered preview to check/uncheck them without touching the raw editor. The changes integrate cleanly with the existing streaming/committed/incoming architecture.

Key changes:

• toggleMarkdownCheckbox in file-viewer.tsx — a pure string-replace utility using the ordered-list-aware regex /^(\\s*(?:[-*+]|\\d+[.)]) +)\\[([ xX])\\]/gm to rewrite the target checkbox at a 0-based index
• handleCheckboxToggle callback — reads from contentRef (always current) and calls handleContentChange only when the content actually changed
• buildMarkdownComponents factory in preview-panel.tsx — creates custom ul/ol/li/input renderers; the input renderer increments a shared checkboxCounterRef so each checkbox gets a stable sequential index
• MarkdownPreview interactive fast-path — when onCheckboxToggle is provided, renders content directly in a single ReactMarkdown call, bypassing the streaming split (safe because onCheckboxToggle is only passed when !isStreaming)
• The Checkbox emcn component is used correctly with size="sm", disabled, checked, and onCheckedChange; all props are valid per the component definition

Confidence Score: 5/5

Safe to merge — all previously reported counter/regex issues are resolved, no new bugs found.

Both previously raised P1 concerns (ordered-list regex gap and counter index drift) have been addressed in follow-up commits. The interactive path correctly short-circuits the streaming reveal logic, contentRef is always up-to-date at toggle time, and the Checkbox component API is used correctly. No remaining P0/P1 issues were found during this review.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant U as User
    participant CB as Checkbox (preview-panel)
    participant MP as MarkdownPreview
    participant HCT as handleCheckboxToggle
    participant TMC as toggleMarkdownCheckbox
    participant HCC as handleContentChange

    U->>CB: click checkbox (index N)
    CB->>MP: onCheckedChange(newChecked)
    MP->>HCT: onCheckboxToggle(N, Boolean(newChecked))
    HCT->>TMC: toggleMarkdownCheckbox(contentRef.current, N, checked)
    TMC-->>HCT: updated markdown string
    HCT->>HCC: handleContentChange(toggled) [only if changed]
    HCC-->>MP: setContent(value) + contentRef.current = value
    MP-->>CB: re-render with updated checked prop from parsed markdown

_{Reviews (5): Last reviewed commit: "fix(files): move useMemo above condition..." | Re-trigger Greptile}

@greptile

@cursor review

@cursor review

@greptile

@greptile

@cursor review

@greptile

@cursor review

@cursor review