Skip to content

fix: YouTube API v3 worker deploy + CI workflow failures resolved - #307

Open
circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/yt-worker-ci-fixes-1793330387575
Open

circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/yt-worker-ci-fixes-1793330387575

Conversation

@circleci-app

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

Copy link
Copy Markdown
Contributor

Summary

  • YouTube API v3 + Worker deploy: Added youtube-worker-deploy CircleCI job that fetches YouTube channel data (channel UCrjJP_SHUeCmqpTSHJCXkdA) via the YouTube Data API v3 and deploys src/youtube-worker.ts to Cloudflare Workers using wrangler. Deployment is conditional on CLOUDFLARE_API_TOKEN being set in CircleCI project settings.
  • Python workflow fix: fix_app.py and fix_index.py used bare open() calls which violated ruff's PTH123 rule — replaced with Path.open() from pathlib. All ruff and mypy checks now pass.
  • Node workflow fix: Three issues in functions/:
    1. package.json had a duplicate firebase-admin key (^10.3.0 shadowed ^13.9.0), causing a peer-dep conflict with firebase-functions@7; removed the stale entry.
    2. typescript: "^6.0.0" referenced a non-existent stable release; downgraded to ^5.0.0.
    3. .eslintrc.js was minified to a single line, causing its own lint errors; expanded to properly formatted multi-line config with consistently quoted rule keys.
  • CircleCI config: Updated node image from cimg/node:20.18 → cimg/node:22.19, added npm install --legacy-peer-deps step for the functions/ directory before lint/tsc, and added the new youtube-worker-deploy job to the validate workflow.

Test plan

  • python-checks job: ruff check . --fix exits 0, mypy exits 0
  • node-checks job: npm run lint -- --max-warnings 0 exits 0, tsc --noEmit exits 0
  • youtube-worker-deploy job: prints channel info if YOUTUBE_API_KEY is set; skips deploy gracefully if CLOUDFLARE_API_TOKEN is absent
  • Set CLOUDFLARE_API_TOKEN + YOUTUBE_API_KEY in CircleCI project settings to enable live fetch and worker deploy

https://app.circleci.com/agents/gh/ruhdevops/chat/de230fb3-7716-4463-afa6-be54659fcd0f

- fix_app.py, fix_index.py: replace open() with Path.open() to satisfy
  ruff PTH123 rule (pathlib enforcement)
- functions/package.json: remove duplicate firebase-admin entry (^10.3.0
  conflicted with firebase-functions@7 peer requirement), fix TypeScript
  from non-existent ^6.0.0 to stable ^5.0.0, relax engines to >=18
- functions/.eslintrc.js: expand minified one-liner to properly formatted
  config; quote all rule keys consistently (quote-props: consistent)
- .circleci/config.yml:
  - update node image from cimg/node:20.18 to cimg/node:22.19
  - add Install functions dependencies step before lint and tsc
  - add youtube-worker-deploy job: fetches YouTube API v3 channel data
    and deploys src/youtube-worker.ts via wrangler when
    CLOUDFLARE_API_TOKEN is set

AI-Generated: true
@vercel

vercel Bot commented Oct 30, 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 30, 2026 3:28am UTC
yt-studio-production Error Error Oct 30, 2026 3:28am 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 de4c570 Oct 30 2026, 03:28 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 de4c570 Oct 30 2026, 03:28 AM

@netlify

netlify Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for ytr-studio failed.

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

@netlify

netlify Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for elegant-seahorse-d9d460 ready!

Name Link
🔨 Latest commit de4c570
🔍 Latest deploy log https://app.netlify.com/projects/elegant-seahorse-d9d460/deploys/6ae40ec9b8709c1dbaa509e9
😎 Deploy Preview https://deploy-preview-307--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 — de4c5706 Deployed Oct 30, 2026 by vercel[bot]
Preview – yt-studio-production — de4c5706 Deployed Oct 30, 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