Skip to content

feat(hub+web): include scratchlist in session export - #1237

Merged
tiann merged 1 commit into
tiann:mainfrom
heavygee:feat/scratchlist-in-session-export
Jul 30, 2026
Merged

feat(hub+web): include scratchlist in session export#1237
tiann merged 1 commit into
tiann:mainfrom
heavygee:feat/scratchlist-in-session-export

Conversation

@heavygee

Copy link
Copy Markdown
Collaborator

Summary

  • Bump session export schemaVersion to 2 and include per-session scratchlist text entries plus attachment metadata (id, filename, mimeType, size, path) in GET /api/sessions/:id/export.
  • Markdown export gains a readable ## Scratchlist section (omitted when empty); JSON always emits scratchlist: [] when there are no notes.
  • Attachment bytes stay out of the JSON (v1 scope). Export dialog locale copy mentions scratchlist so operators know notes are included.

Test plan

  • bun typecheck
  • Hub/web/shared unit tests (new export + markdown coverage)
  • Peer-stack Playwright: seed scratchlist → export JSON has schemaVersion: 2 + note text; dialog copy mentions scratchlist
  • Operator dogfood on peer stack: More actions → Export → Download JSON → confirm scratchlist

Issues

Fixes #1235

Bump export schema to v2 with scratchlist text and attachment metadata
so operators keep notes when they export-then-delete. Markdown gets a
Scratchlist section; attachment bytes stay out of the JSON.

Co-authored-by: Cursor <cursoragent@cursor.com>
@heavygee

Copy link
Copy Markdown
Collaborator Author

Proof (pr-attach-proof, release strategy — not user-attachments)

Peer #1235 export dialog

asset: heavygee/hapi@pr-attach-proof-1237/proof-2026-07-29T10-23-43-1235-scratchlist-export-peer.png · delete release after merge if desired

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • None.

Summary

  • Review mode: initial
  • No issues found in the added or modified lines. Residual risk: this automation did not execute PR code; assessment is based on full-diff/static context review plus the passing GitHub test check.

Testing

  • Not run (automation); GitHub test check passed.

HAPI Bot

@tiann
tiann merged commit a742fdf into tiann:main Jul 30, 2026
2 of 3 checks passed
@heavygee
heavygee deleted the feat/scratchlist-in-session-export branch July 30, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(hub+web): include scratchlist text in session export

2 participants