Skip to content

fix: reuse ready transcript instead of YouTube re-hit on deploy - #1859

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/studio-deploy-reuse-transcript-d09a
Sep 12, 2026
Merged

fix: reuse ready transcript instead of YouTube re-hit on deploy#1859
cursor[bot] merged 1 commit into
mainfrom
cursor/studio-deploy-reuse-transcript-d09a

Conversation

@groupthinking

@groupthinking groupthinking commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Canonical issue

Closes studio.deploy YouTube bot residual after #1855 (dpl_8DjoBuK8pE5MxE5L7SQaRgixivTK / XYMcBrFSJ4c, receipt er:gate:v1:wrun_01M2ACYVYXBHM0YVMX1WHMQ1PJ).

Outcome

When Video Pack / transcript is already ready, Attempt deploy does not start URL-only /videos/process (yt-dlp / “Sign in to confirm you’re not a bot”). It reuses the Studio transcript, still tries video-to-software for a verified https:// live URL + EventRelay receipt, or HOLDs with the real miss — not the YouTube bot string and not the cleared residuals (UNKNOWN checks, workflow-run, return-value, BACKEND_URL).

Scope

Risk

  • Risk level: low
  • Failure mode: video-to-software still 401 / no live URL → honest HOLD (not a YouTube re-hit)
  • Rollback: revert this squash on main

Verification

  • Focused Vitest (pipeline-async-job, studio-workflow, studio-deploy POST, studio-pipeline-status, gate-transition)
  • Required CI (0 required pending; core green; 0 failed)
  • Squash-merged

Production evidence

AXIOM still needs a signed-in XYMcBrFSJ4c dogfood. PASS only with a verified https live URL + EventRelay receipt.

Agent handoff

  • One named residual
  • No competing PR for this residual
  • Acceptance: reuse ready transcript; no YouTube bot HOLD; claim guard kept
  • Required checks pass on the merged squash
  • Human AXIOM re-dogfood XYMcBrFSJ4c on this prod READY
Open in Web Open in Cursor 

Pass Studio transcript through studio.deploy and skip URL-only
/videos/process so yt-dlp cannot HOLD on a bot check.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: ee8db93f-a17e-47a6-8928-7ddb42350bce


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

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

@vercel

vercel Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
v0-uvai Ready Ready Preview, v0 Sep 12, 2026 9:01am UTC

@github-actions github-actions Bot added documentation Improvements or additions to documentation javascript Pull requests that update javascript code tests labels Sep 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 31e741d.
Ensure 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 Files

None

@groupthinking
groupthinking marked this pull request as ready for review September 12, 2026 09:05
@cursor
cursor Bot merged commit eec86e6 into main Sep 12, 2026
40 of 41 checks passed
@cursor
cursor Bot deleted the cursor/studio-deploy-reuse-transcript-d09a branch September 12, 2026 09:05
@linear-code

linear-code Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

GRV-530

groupthinking added a commit that referenced this pull request Sep 12, 2026
* fix: return 202 from video-to-software before CF 524

Origin answers within a 12s sync budget. Slow deploys persist a
pollable job_id so Studio never waits for Cloudflare HTTP 524.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>

* fix: skip process handoff when deploy transcript is ready

Restore #1859 reuse so HTTP 524 no longer starts URL-only
/videos/process. Origin 202 remains the async path past 524.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>

* docs: record reuse-after-524 verification on PR 1880

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>

* fix: unblock test-frontend parse and persistence suite

Drop the duplicate PIPELINE_RESPONSE_BUFFER_MS import and run
dashboard store persistence under Vitest so the required check
can go green for the reuse-after-524 cut.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Hayden <groupthinking@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation javascript Pull requests that update javascript code tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants