Fix YouTube --show-code, Vercel deploy framework, stream poll default, --llm help#59
Merged
Merged
Conversation
- code_gen: download YouTube audio with yt-dlp before upload (raw URL fails) - init templates: ship vercel.json pinning the FastAPI framework preset so `aai deploy` doesn't hit Vercel's "services" framework error - stream: default --llm-interval 30s -> 10s - main: drop the redundant Authentication block from `aai --help` Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Four CLI surface fixes hit during manual QA:
--show-codegenerated a broken script. The generated code passed the raw YouTube URL straight totranscriber.transcribe(), which fails withRuntimeError: Download error— AssemblyAI can't fetch YouTube URLs. The real CLI downloads with yt-dlp first (transcribe_exec.py).code_gen/transcribe.pynow emits a yt-dlp download-to-temp-file block for YouTube sources and uploads the local file. Non-YouTube URLs still upload straight through.aai deploy→ Vercel"services"framework error. Templates shipped novercel.jsonand relied on Vercel's old zero-config Python detection, which now resolves theapi/+ Dockerfile layout to the multi-service"services"framework and hard-fails (no services declared). Addedvercel.json("framework": "fastapi") to all three templates so Vercel buildsapi/index.pyand routes every request to the ASGI app — and never auto-detects"services". Updated the now-false "novercel.jsonneeded" README claims.aai stream --llmpoll default lowered from 30s to 10s (--llm-interval).aai --helpno longer renders the redundantAuthenticationblock in the epilog.Tests
code_gentests: YouTube download path, config passthrough, plain-URL straight upload."framework": "fastapi"in each template'svercel.json; required-files list and scaffold test updated to expect it.--helpsnapshots (auth block gone,--llm-intervaldefault now 10.0).vercel.jsonfiles ship in the built wheel.🤖 Generated with Claude Code