nikserg/skim

Update Skim demo assets

Regenerates the product demo video and the light/dark interface screenshots that ship with the landing page. Everything is driven by the real Skim UI running against a mocked Tauri IPC layer (demo/mock/), so no real account, mailbox, or model is ever touched.

Outputs, all published into docs/:

• skim-demo.mp4 — the scripted feature tour (1280×800)
• skim-light.jpg, skim-dark.jpg — interface screenshots, 800px wide

Do not commit or push

This skill only updates files on disk. Do not run git add, git commit, or git push. The user reviews the regenerated assets themselves and decides what to keep — these are binary files that produce large diffs, and a run that looks fine by exit code can still be visually wrong. Leaving the working tree dirty is the correct outcome.

Only touch git if the user explicitly asks in that same request (e.g. "regenerate and commit them"). Otherwise, finish by telling them the files are updated and unstaged.

Running it

From the repo root:

npm run demo

That chains three steps: demo:record (Playwright drives the real UI → demo/output/raw.webm) → demo:encode (ffmpeg → MP4, copied into docs/) → demo:shots (both themes → docs/).

If only part of the UI changed, run just what's needed — it's faster and keeps unrelated assets byte-identical:

• npm run demo:shots — screenshots only (static UI, colors, layout changes)
• npm run demo:record && npm run demo:encode — video only (flows, AI behavior)

Prerequisites: npm install and ffmpeg on PATH. The whole run takes a couple of minutes; recording is the slow part.

npm run demo begins with demo:browser (playwright install chromium) — a no-op when the browser is present. If a run dies with "Executable doesn't exist at ...chromium_headless_shell-NNNN", that's Playwright's browser build being out of step with the library version (a ^ bump re-points it, and the headless shell is a separate download from Chromium). Running npm run demo:browser, or the full npm run demo, resolves it — no need to debug it as a demo problem. The individual demo:record / demo:shots scripts skip that check.

Verify before reporting success

A run can exit 0 and still produce a broken video. An unhandled mock command returns undefined rather than throwing, which renders as an empty pane instead of an error — so trust your eyes, not the exit code.

1. Confirm the three files in docs/ have fresh mtimes.
2. Check dimensions: screenshots 800px wide, MP4 1280×800.
3. Pull frames across the whole MP4 and actually look at them. Sample by fraction rather than fixed seconds — the tour's length changes whenever a scene is added or the pacing is retuned, so hardcoded timestamps rot:

   dur=$(ffprobe -v error -show_entries format=duration -of csv=p=0 docs/skim-demo.mp4)
   for k in 1 2 3 4 5 6 7; do
     ffmpeg -y -hide_banner -loglevel error \
       -ss $(echo "$dur * $k / 8" | bc -l) -i docs/skim-demo.mp4 -frames:v 1 /tmp/f$k.png
   done
   

   Every frame should show a populated pane, and the AI scenes should have their streamed text. demo/output/progress.log names each scene as it finished, which helps map a bad frame back to the step that produced it.
4. View both screenshots — check the dark one especially (email bodies render in a sandboxed iframe with their own theming logic and can go dark-on-dark).

Report which files changed and what you saw. If a scene looks wrong, fix it rather than shipping it.

When the tour breaks after a UI change

The harness leans on two contracts. The failure message tells you which one moved:

IPC command names (demo/mock/tauri-core.ts). The mock implements invoke() with a switch over command names mirroring src/lib/api.ts. If a command was renamed or added, add the matching case. Symptom: an empty pane or missing data rather than a crash.

Selectors and English labels (demo/record.mjs). The tour clicks things like .row, .quick-btn:has-text("Summarize"), .ai-input .instruction. Renamed CSS classes or changed strings in src/lib/i18n/locales/en.json break the matching step, and the timeout error names the selector that failed. The demo forces locale: "en", so English labels are what matter.

The same goes for the fixtures: demo/mock/data.ts matches the ai.prompt_* strings to decide which scripted answer a quick prompt gets, and the tour waits on distinctive words from those answers ("sync", "yours", "4.2"). Reword a fixture and the wait that depends on it times out.

All fixture data and every scripted AI response live in demo/mock/data.ts — that's the file to edit when you want the demo to show something different (different emails, a different summary, different drafted reply).

Non-obvious things that will bite you when editing record.mjs:

• The composer body is a <textarea>. Its streamed content is its value, not its text — wait with waitValue, not waitText.
• .compose-window textarea matches the AI instruction box too (it comes first in the DOM). Use .compose-window > textarea for the body.
• Compose scenes navigate by hash and must call page.reload() afterwards — a hash-only goto doesn't remount the SPA, so the composer never appears.
• Save the video with video.saveAs() before browser.close(), or it fails with "target closed".

Pacing and themes

Defaults are tuned for a readable landing video. Override per run with env vars (all ms): DEMO_READ (dwell per screen), DEMO_BEAT (between actions), DEMO_TYPE (keystroke delay), and DEMO_AI_TYPING / DEMO_AI_THINK / DEMO_AI_STEP (AI stream cadence). Useful when a scene feels rushed after you add content to it.

Themes are two-axis — "<cold|warm>-<light|dark>", e.g. warm-light (the app default). Both scripts force the theme through localStorage (skimdemo.theme), which the mock returns from get_settings:

• The video is pinned to warm-light; override with DEMO_THEME=cold-dark npm run demo:record.
• Screenshots render warm-light → docs/skim-light.jpg and warm-dark → docs/skim-dark.jpg (the SHOTS array in demo/screenshots.mjs maps theme → filename).

Watch out for legacy single-axis values: "light" and "dark" still parse, but map onto the cold palette, so passing them silently renders the wrong temperature rather than failing.

DEMO_PREBUILT=1 serves a static build from demo/dist-demo instead of the dev server — handy in constrained or headless environments.

More detail lives in demo/README.md.