Save & Share Components (requires accounts)

[michaelmagan]

Description:

Create registered Tambo components for saving and sharing songs. Users should be able to ask "save this song" or "share this with a link".

Save Component:

Save current REPL code as a named song

View/manage saved songs

Load a saved song back into the REPL

Share Component:

Generate a public share link for the current code

Anyone with the link can view/play (no account required to view)

Optional: copy link to clipboard

Notes:

Both components require user to be logged in

If user is not authenticated, prompt them to create an account

Saved songs and threads should be stored server-side and associated with user account

Register both components in `src/lib/tambo.ts

[charliecreates]

Expand this to see my work.

• Looked through generated Next.js types in .next/types/app/share/[shareId]/page.ts to confirm the PageProps shape (including params?: Promise<SegmentParams>) matches how src/app/share/[shareId]/page.tsx is currently implemented.
• Added MAX_CODE_LENGTH and MAX_TITLE_LENGTH constants in src/app/api/shares/route.ts and tightened createShareSchema to code: z.string().min(1).max(MAX_CODE_LENGTH) and title: z.string().trim().min(1).max(MAX_TITLE_LENGTH).optional() so oversized share payloads are rejected at the API boundary.
• Updated handleCreate in src/components/tambo/share-song.tsx to short-circuit with a friendly error ("Create or load a song before generating a share link.") when code.trim() is empty, avoiding pointless /api/shares requests with blank content.
• Re-ran npm run build after the schema and ShareSong guard changes to verify that the tighter validation and new pre-check compile cleanly alongside the existing SaveSong/ShareSong components and src/lib/tambo.ts registry wiring.
• Re-ran npm run lint to confirm the latest changes keep the codebase lint-clean, with only existing @next/next/no-img-element warnings in src/app/page.tsx and src/components/info-modal.tsx and no new issues introduced by the save/share work.
• Reviewed the generated code review feedback on src/app/share/[shareId]/page.tsx, src/services/song-share.ts, src/app/api/shares/route.ts, and src/app/chat/page.tsx to identify small follow-ups (like simplifying params, documenting the SQLite path, and making ensureSchema more robust) that could further harden the new share functionality.
• Updated ensureSchema in src/services/song-share.ts to wrap the Postgres/SQLite CREATE TABLE and CREATE INDEX calls in a try/catch that resets schemaReadyPromise to null on failure so transient DB errors don’t permanently poison future song_share operations.
• Added a user-facing safety note ("Only run songs you trust.") in src/app/share/[shareId]/shared-song-client.tsx below the Reset button to remind people that shared songs run arbitrary code.
• Ran npm run build again after the schema retry fix and share-page safety message to confirm the updated song-share service and shared-song-client still compile cleanly, with Next.js reporting a successful optimized production build and only existing <img> warnings.
• Confirmed a clean working tree with git status --porcelain and opened PR #18 via gh pr create, including a summary of the new SaveSong/ShareSong Tambo components, POST /api/shares + song_share persistence, user-scoped contextKey, and lint/build verification, then requested review from CharlieHelps.

[charliecreates]

Opened PR #18 with the “Save this song” + “Share this with a link” flow, gated behind accounts: #18

Changes

• Added registered Tambo components:
  • SaveSong (src/components/tambo/save-song.tsx): prompts sign-in if needed, then lets users name the current REPL + browse/load/rename/remove saved songs.
  • ShareSong (src/components/tambo/share-song.tsx): prompts sign-in if needed, then creates a public share URL (with copy-to-clipboard).
• Added server-side persistence + public viewer:
  • POST /api/shares (src/app/api/shares/route.ts) stores the current code in song_share.
  • Public page /share/[shareId] (src/app/share/[shareId]/page.tsx) lets anyone with the link view/play the shared song.
• Updated Tambo thread storage to be account-associated by making contextKey stable per signed-in user (strudel-user-<userId>) and re-initializing thread selection when it changes (src/app/chat/page.tsx).

Verification

# Lint: pass (2 existing warnings)
$ npm run lint

# Build: pass
$ npm run build

Notes:

• Local next build logs warnings about missing BETTER_AUTH_SECRET in the devbox env, but completes successfully.
• src/app/share/[shareId]/page.tsx uses params: Promise<...> because this repo’s generated Next.js PageProps expects params?: Promise<SegmentParams> (confirmed in .next/types/...).