Repository navigation
feat: YouTube API v3 worker deployment + fix CI workflow failures - #260
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 functions/package.json: remove duplicate firebase-admin dependency
that was causing npm resolution issues (kept ^13.9.0, removed ^10.3.0)
- Fix app/wrangler.jsonc: replace placeholder <WORKER_NAME> and
<COMPATIBILITY_DATE> with real values (yt-studio-app, 2024-11-01)
- Update .circleci/config.yml:
- Add separate npm install step for functions/ before linting (was
missing, causing node-checks job to fail)
- Add worker-tests job: installs app/ deps and runs Cloudflare vitest
pool tests against the worker
- Add deploy-worker job: installs wrangler globally and deploys the
YouTube API v3 worker (src/youtube-worker.ts) as 'ytstudio' to
Cloudflare Workers using CLOUDFLARE_API_TOKEN + CLOUDFLARE_ACCOUNT_ID
- Add deploy workflow that gates deployment on all checks passing on
main branch (python-checks → node-checks → worker-tests → deploy-worker)
- Validate workflow runs all three check jobs in parallel on every branch
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 | f07f40a | Oct 19 2026, 02:25 AM |
Contributor
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | f07f40a | Oct 19 2026, 02:25 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
functions/package.json: Remove duplicatefirebase-adminkey (^13.9.0+^10.3.0). Duplicate JSON keys cause unpredictable resolution — kept^13.9.0which matches the lock file.app/wrangler.jsonc: Replace placeholder<WORKER_NAME>and<COMPATIBILITY_DATE>with real values (yt-studio-app,2024-11-01) so the Cloudflare Worker can actually be built/tested.node-checksCI job: Added missingcd functions && npm installstep beforenpm run lint. Without it, ESLint couldn't find its dependencies and the job always failed.worker-testsjob: Installsapp/dependencies and runs the Cloudflare vitest-pool-workers test suite against the YouTube API v3 worker.deploy-workerjob: InstallswranglerCLI globally, then deployssrc/youtube-worker.tsas theytstudioCloudflare Worker usingCLOUDFLARE_API_TOKEN+CLOUDFLARE_ACCOUNT_IDenvironment variables.deployworkflow: Gates deployment onmainonly — runspython-checks → node-checks → worker-tests → deploy-workerin sequence.validateworkflow: All three check jobs run in parallel on every branch push.Worker: YouTube API v3 (
src/youtube-worker.ts)The existing worker already implements full YouTube Data API v3 integration for channel
UCrjJP_SHUeCmqpTSHJCXkdA(Ruh Al Tarikh):GET /or/healthGET /api/channelGET /api/videos?maxResults=N&pageToken=TGET /api/search?q=queryFalls back to demo data when
YOUTUBE_API_KEYis not bound.Test plan
validateworkflow runs all three jobs on non-main branchesdeployworkflow triggers onmainand gates on tests passingworker-testspasses (Hello World unit + integration tests)node-checkspasses with the newnpm installstep infunctions/CLOUDFLARE_API_TOKENandCLOUDFLARE_ACCOUNT_IDto CircleCI project env vars before merginghttps://app.circleci.com/agents/gh/ruhdevops/chat/d4beedae-099c-4398-8d4d-ed398476c21e