Remove dead live-preview path from tinyskills

[Hotragn]

Refs #86. Found by the conformance check in #244.

The bug

hooks/use-generation.ts had a branch for a source_streaming event that nothing sends:

emitted by app/api/scrape-sources/route.ts:
  source_start, source_step, source_error, source_complete,
  scrape_complete, error, scrape_start

consumed by hooks/use-generation.ts:
  source_start, source_step, source_streaming, source_complete,
  source_error, scrape_complete, error
                ^^^^^^^^^^^^^^^^ never emitted

grep -rn source_streaming tinyskills/ returns exactly one line — the consumer. So the branch never ran, ScrapeProgress.streamingUrl was never assigned, and both UI blocks gated on it were unreachable:

• app/page.tsx:366 — external-link icon on a scraping row
• components/skillforge/source-progress.tsx:105 — the "Watch live" tooltip link

Why removing it is the right fix, not emitting it

This is left over from before the recipe moved to TinyFish Fetch. The route now calls client.fetch.getContents({ urls, format: "markdown" }) — a batch request/response with no browser session, so there is no streaming URL to surface. grep -rn "agent\.\|\.stream(" tinyskills/ finds nothing; the recipe never touches the Agent endpoint.

Making the "Watch live" link work again would mean moving this recipe back to the Agent endpoint, which is metered where Fetch is free. That's a product decision about the recipe's cost profile, not a bug fix, so it isn't mine to make in this PR. Removing the vestigial path leaves the recipe honest about what it actually does.

Changes

• types/index.ts — drop ScrapeProgress.streamingUrl
• app/api/scrape-sources/route.ts — drop the field copy into finalResults
• hooks/use-generation.ts — drop the source_streaming branch
• app/page.tsx, components/skillforge/source-progress.tsx — drop the two unreachable links
• drop imports orphaned by the above (ExternalLink, and Tooltip* in source-progress.tsx)

Net −47/+1. No behaviour change: every removed path was already unreachable. components/ui/tooltip.tsx is left in place as an unused shadcn primitive.

Verification

cd tinyskills && npm ci
npx tsc --noEmit        # clean
npx eslint . --ext .ts,.tsx   # clean

Note npm run lint fails on its own — the script is next lint, which Next removed after 14. Unrelated to this change, but worth knowing if you try it.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b1c9d1d0-99ab-43a0-8846-c2f54805daf9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}