fix(web): cover studio auto-start; correct API docs - #1670
Conversation
Query ?video= starts processVideo once with the canonical watch URL, including a Strict Mode double-invoke. API docs describe the YouTube workflow (pack + video-to-actions), not the REST catalog. Co-authored-by: Hayden <groupthinking@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
Bot
added
documentation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 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. Comment |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
There was a problem hiding this comment.
🟢 Approved
The reviewed changes have regression coverage and no unresolved issues.
Pull request overview
Adds Strict Mode-safe Studio query auto-start behavior and corrects API workflow documentation.
Changes:
- Deduplicates query-driven Studio analysis starts.
- Adds regression coverage for valid and invalid queries.
- Documents the Video Pack → video-to-actions workflow.
File summaries
| File | Description |
|---|---|
apps/web/src/lib/studio-handoff.ts |
Adds the one-shot auto-start helper. |
apps/web/src/lib/__tests__/studio-handoff.test.ts |
Tests deduplication and invalid input. |
apps/web/src/lib/__tests__/home-sell-surface.test.ts |
Verifies Studio and documentation integration. |
apps/web/src/components/OneLoopStudio.tsx |
Uses the helper for query handoffs. |
apps/web/src/app/docs/api/page.tsx |
Lists and explains the workflow endpoints. |
Review details
- Files reviewed: 5/5 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Please add a See governance: #898 |
Canonical issue
Closes #1669
Outcome
Studio
?video=auto-start is one-shot (Strict Mode safe) with a behavioral test. API docs describe the end-to-end YouTube workflow instead of claiming Studio runs the REST catalog.Follow-up to merged #1668 (Copilot review landed after that squash). This PR does not change Home vs Studio routing.
Scope
applyStudioQueryAutoStart: onestart(canonicalWatchUrl)across a Strict Mode double-invoke and same-query rerender (auJzb1D-fag).?video=/?url=./docs/apiTry-it copy: YouTube workflow via/api/video/packthen/api/workflows/video-to-actions; both listed in the catalog./featuresor/playground(CoS: those stay 308 to Home/, already on main from feat(web): Home sell page + Studio workbench (Closes #1667) #1668).npm cifailure (tracked as fix(ci): resync web lockfile blocking Trivy #1657).Risk
processVideoon/studio?video=.Verification
Tied to head SHA
3e856d992d7da6c9ce1291a40cbc065cd8b2a9ce.cd apps/web && npx vitest run src/lib/__tests__/studio-handoff.test.ts src/lib/__tests__/home-sell-surface.test.ts— 2 files, 14 passedtest-frontend,lint-frontend,buildpassednpm ci/ fix(ci): resync web lockfile blocking Trivy #1657)Production evidence
Preview Ready on this SHA: https://v0-uvai-git-cursor-studio-autostart-docs-4940-garv1.vercel.app (deployment https://vercel.com/garv1/v0-uvai/5ewRypQ6X1MMiyBozLarGWmGU4Kn). Preview is behind Vercel SSO. Production already has the #1668 Home/Studio split.
Agent handoff