Repository navigation
fix: resolve CI failures and add YouTube Worker deployment pipeline - #327
Open
circleci-app[bot] wants to merge 1 commit into
Open
circleci-app[bot] wants to merge 1 commit into
circleci-app[bot] wants to merge 1 commit into
Conversation
- Fix node-checks job: add npm install in functions/ before lint/tsc (pnpm install at root doesn't install functions' npm dependencies) - Add TypeScript type check step for Cloudflare Worker (src/) - Add deploy-worker job: deploys src/youtube-worker.ts to Cloudflare Workers as 'ytstudio' on main branch merges via cloudflare context - Fix functions/package.json: remove duplicate firebase-admin, remove stray lint dep, correct node engine 24→20, TypeScript ^6→^5.5 - Expand functions/.eslintrc.js from minified form; add .eslintrc.js to ignorePatterns so ESLint doesn't try to parse itself - Regenerate functions/package-lock.json after dependency cleanup - Remove duplicate circletui from root package.json - Apply ruff auto-fixes to fix_app.py, fix_deps.py, fix_index.py (quote style, unused import) AI-Generated: true
|
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | d212b51 | Nov 05 2026, 12:31 AM |
Contributor
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | d212b51 | Nov 05 2026, 12:31 AM |
✅ Deploy Preview for elegant-seahorse-d9d460 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
❌ Deploy Preview for ytr-studio failed.
|
This branch had an error being deployed
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
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
node-checksCI failure: The job ranpnpm installat root butfunctions/is a separate npm package — ESLint and tsc weren't available. Addednpm installinfunctions/before lint/typecheck steps.npx tsc --noEmitstep at root to validatesrc/youtube-worker.ts(the Cloudflare Worker) on every CI run.deploy-workerjob: New CircleCI job deployssrc/youtube-worker.tsto Cloudflare Workers asytstudiousing wrangler. Runs only onmainmerges via thecloudflarecontext (requiresCLOUDFLARE_API_TOKENandCLOUDFLARE_ACCOUNT_IDenv vars).functions/package.json: Removed duplicatefirebase-admin, removed straylintdep, correctednodeengine from24to20, downgraded TypeScript from^6.0.0to^5.5.0(v6 doesn't exist yet).functions/.eslintrc.js: Was minified; expanded to readable form and added.eslintrc.jstoignorePatternsto prevent ESLint from trying to parse its own config file.fix_app.py,fix_deps.py,fix_index.py.Worker Endpoints
The YouTube API v3 worker (
src/youtube-worker.ts) exposes:GET /or/healthGET /api/channelGET /api/videosmaxResultsparam)GET /api/search?q=...Requires
YOUTUBE_API_KEYsecret binding in Cloudflare; falls back to demo data if missing.Test plan
validateworkflow:python-checksandnode-checksboth passdeployworkflow triggers on merge tomainand runsdeploy-workerCLOUDFLARE_API_TOKEN,CLOUDFLARE_ACCOUNT_ID, andYOUTUBE_API_KEYin Cloudflare/CircleCI contexts before first deploymenthttps://ytstudio.<account>.workers.dev/api/videosto confirm live datahttps://app.circleci.com/agents/gh/ruhdevops/chat/90c71c74-bfe5-468b-b2b3-8bae79811414