Skip to content

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

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

circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/youtube-worker-deploy-fix-ci-2

Conversation

@circleci-app

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

Copy link
Copy Markdown
Contributor

Summary

  • YouTube API v3 worker deploy: Added deploy-worker job to .circleci/config.yml that deploys src/youtube-worker.ts (fully-featured YouTube API v3 Cloudflare Worker with /api/channel, /api/videos, /api/search endpoints) to Cloudflare Workers on every push to main. Requires CLOUDFLARE_API_TOKEN env var in CircleCI project settings.
  • Fix node-checks job: Added missing npm install --legacy-peer-deps step for the functions/ package before lint/typecheck; switched to local ./node_modules/.bin/tsc instead of npx tsc.
  • Fix functions/.eslintrc.js: File was minified to a single line, causing it to violate its own max-len and spacing rules (40 ESLint errors on itself). Expanded to properly formatted config.
  • Fix functions/package.json: Duplicate firebase-admin keys (^13.9.0 and ^10.3.0) — JSON last-key-wins semantics meant ^10.3.0 was active, which conflicts with firebase-functions@^7 peer dep requirement of ^11.10.0+. Removed the stale ^10.3.0 entry.
  • Fix fix_app.py & fix_index.py: Replaced bare open() calls with Path.open() to satisfy ruff PTH123 rule (the PTH rule group is explicitly selected in pyproject.toml).

Workflow failures fixed

Job Root cause Fix
python-checks ruff check . --fix exited non-zero on 4 unfixable PTH123 errors Patched fix_app.py and fix_index.py
node-checks (lint) functions/.eslintrc.js was minified — 40 self-referential lint errors Expanded to readable format
node-checks (lint) functions/node_modules/ not installed before lint step Added npm install step
node-checks (tsc) npx tsc in CI with no npm cache = wrong version or missing binary Use ./node_modules/.bin/tsc
node-checks (install) Duplicate firebase-admin keys caused peer dep conflict Removed duplicate entry

https://app.circleci.com/agents/gh/ruhdevops/chat/05fdbaf2-35a9-4f1f-9880-c827d01525a8

- Add deploy-worker job to .circleci/config.yml that deploys
  src/youtube-worker.ts (YouTube API v3) to Cloudflare Workers
- Fix node-checks: add functions npm install step and use local tsc binary
- Fix functions/.eslintrc.js: expand minified file (was violating its own rules)
- Fix functions/package.json: remove duplicate firebase-admin key (^10.3.0
  shadowed ^13.9.0, causing peer dep conflict with firebase-functions@^7)
- Fix fix_app.py and fix_index.py: replace open() with Path.open() to
  satisfy ruff PTH123 rule (PTH rule group is enabled in pyproject.toml)

AI-Generated: true
@bolt-new-by-stackblitz

Copy link
Copy Markdown

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

@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 4:26am UTC
yt-studio-production Error Error Oct 19, 2026 4:26am UTC

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 19, 2026 •

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 2038ac1 Oct 19 2026, 04:26 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 2038ac1 Oct 19 2026, 04:26 AM

@netlify

netlify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for elegant-seahorse-d9d460 ready!

Name Link
🔨 Latest commit 2038ac1
🔍 Latest deploy log https://app.netlify.com/projects/elegant-seahorse-d9d460/deploys/6ad59bed05de2d519c1b1bd3
😎 Deploy Preview https://deploy-preview-262--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.

@netlify

netlify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for ytr-studio failed.

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

This branch had an error being deployed

2 failed deployments
Preview – yt-studio — 2038ac11 Deployed Oct 19, 2026 by vercel[bot]
Preview – yt-studio-production — 2038ac11 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