Skip to content

feat: YouTube API v3 worker deploy + fix CI workflow failures - #258

Open
circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/yt-worker-deploy-and-fix-ci-workflows
Open

circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/yt-worker-deploy-and-fix-ci-workflows

Conversation

@circleci-app

@circleci-app circleci-app Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add deploy-worker CI job that deploys the Cloudflare Worker (src/youtube-worker.ts) serving YouTube API v3 data (channel stats, video listings, search) for channel UCrjJP_SHUeCmqpTSHJCXkdA. Job runs after node-checks passes on main branch only.
  • Fix duplicate JSON key in package.json — circletui appeared twice in dependencies, causing wrangler to emit a parse warning during deploy.
  • Fix functions/package.json — removed duplicate firebase-admin entry, downgraded TypeScript from ^6.0.0 → ^5.1.6 (v6 doesn't exist), relaxed node engine from "24" → ">=20" for CI compatibility.
  • Regenerate functions/package-lock.json to match the corrected functions/package.json.
  • Reformat functions/.eslintrc.js from minified single-line to readable multi-line (identical logic).
  • Upgrade node-checks CI image from cimg/node:20.18 → cimg/node:22.9 and add a dedicated functions npm install step.
  • Fix local_checks.py trailing comma for ruff linter compliance.

Test plan

  • pnpm install --frozen-lockfile succeeds (lockfile in sync)
  • cd functions && npm install && npm run lint -- --max-warnings 0 passes
  • cd functions && ./node_modules/.bin/tsc --noEmit exits 0
  • pnpm wrangler deploy --env production --dry-run shows no duplicate-key warnings
  • CircleCI python-checks job passes (ruff, mypy, pytest)
  • CircleCI node-checks job passes (lint + tsc)
  • CircleCI deploy-worker job deploys worker to Cloudflare on merge to main

https://app.circleci.com/agents/gh/ruhdevops/chat/7e98a5bb-c718-415e-b7ab-d9a880d4b579

- Add deploy-worker job to CircleCI config to deploy the Cloudflare Worker
  that serves YouTube API v3 data (channel stats, videos, search)
- Fix duplicate JSON keys in package.json (circletui appeared twice in
  dependencies) that caused wrangler to emit warnings
- Fix functions/package.json: remove duplicate firebase-admin entry,
  downgrade typescript from ^6.0.0 to ^5.1.6 (6.x doesn't exist),
  relax node engine from "24" to ">=20" for CI compatibility
- Reformat functions/.eslintrc.js from minified to readable (same config)
- Regenerate functions/package-lock.json to match corrected package.json
- Fix local_checks.py trailing comma for ruff linter compliance
- Upgrade node-checks CI image from node:20.18 to node:22.9
- Add functions npm install step to node-checks CI job

AI-Generated: true
@vercel

vercel Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
yt-studio Error Error Oct 19, 2026 12:32am UTC
yt-studio-production Error Error Oct 19, 2026 12:32am UTC

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@cloudflare-workers-and-pages

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
ytstudio 3d84191 Oct 19 2026, 12:32 AM

@cloudflare-workers-and-pages

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
ytstudio 3d84191 Oct 19 2026, 12:32 AM

@netlify

netlify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for ytr-studio failed.

Name Link
🔨 Latest commit 3d84191
🔍 Latest deploy log https://app.netlify.com/projects/ytr-studio/deploys/6ad56511aa8ae0f618dd22bd

@netlify

netlify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for elegant-seahorse-d9d460 ready!

Name Link
🔨 Latest commit 3d84191
🔍 Latest deploy log https://app.netlify.com/projects/elegant-seahorse-d9d460/deploys/6ad56511509fb402d287eeb4
😎 Deploy Preview https://deploy-preview-258--elegant-seahorse-d9d460.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

This branch had an error being deployed

2 failed deployments
Preview – yt-studio — 3d841910 Deployed Oct 19, 2026 by vercel[bot]
Preview – yt-studio-production — 3d841910 Deployed Oct 19, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants