Skip to content

fix: resolve CI workflow failures and add YouTube API worker deploy - #320

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

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

Conversation

@circleci-app

@circleci-app circleci-app Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • functions/package.json — Removed duplicate firebase-admin@^10.3.0 entry that caused npm install to fail with a peer dependency conflict (firebase-functions@7 requires ^13.x, but npm was resolving to the older ^10.3.0 duplicate)
  • functions/.eslintrc.js — Reformatted from a minified one-liner (483 chars) to proper multi-line JS; the minified form caused ESLint to report max-len, key-spacing, comma-spacing, and quote-props errors when linting itself
  • .circleci/config.yml — Three improvements:
    1. python-checks: now targets .github/scripts/ where the actual Python files live (previously ruff reported "No Python files found")
    2. node-checks: updated to Node 22 image, uses npm install --legacy-peer-deps for functions, and invokes tsc via its local node_modules/.bin binary
    3. New deploy-worker job + deploy workflow: builds and deploys src/youtube-worker.ts (the YouTube API v3 Cloudflare Worker for channel UCrjJP_SHUeCmqpTSHJCXkdA) via wrangler deploy on pushes to main, gated on CLOUDFLARE_API_TOKEN being set

Test plan

  • validate workflow: both python-checks and node-checks jobs pass
  • deploy workflow on main: node-checks passes, deploy-worker runs and either deploys (if CLOUDFLARE_API_TOKEN is set) or exits cleanly with a warning
  • Add CLOUDFLARE_API_TOKEN to CircleCI project environment variables to enable live Cloudflare Worker deployment

https://app.circleci.com/agents/gh/ruhdevops/chat/a698d16b-33a9-4c59-9838-6de3f65ae42a

- Fix functions/package.json: remove duplicate firebase-admin@^10.3.0
  entry that conflicted with firebase-functions@7 (requires ^13.x)
- Fix functions/.eslintrc.js: reformat from minified one-liner to proper
  multi-line JS to pass eslint max-len and spacing rules
- Update .circleci/config.yml:
  - Fix node-checks: use Node 22, npm install --legacy-peer-deps, and
    invoke tsc via local node_modules binary
  - Fix python-checks: target .github/scripts/ where Python files live
  - Add deploy-worker job: runs wrangler deploy for src/youtube-worker.ts
    (YouTube API v3 Cloudflare Worker) when CLOUDFLARE_API_TOKEN is set
  - Add deploy workflow: triggers node-checks then deploy-worker on main

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 Nov 3, 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 Nov 3, 2026 2:25am UTC
yt-studio-production Error Error Nov 3, 2026 2:25am UTC

@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 1af637f Nov 03 2026, 02:25 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 1af637f Nov 03 2026, 02:25 AM

@netlify

netlify Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for elegant-seahorse-d9d460 ready!

Name Link
🔨 Latest commit 1af637f
🔍 Latest deploy log https://app.netlify.com/projects/elegant-seahorse-d9d460/deploys/6ae9460a1e5576fe892211e0
😎 Deploy Preview https://deploy-preview-320--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 Nov 3, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for ytr-studio failed.

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

This branch had an error being deployed

2 failed deployments
Preview – yt-studio — 1af637fa Deployed Nov 3, 2026 by vercel[bot]
Preview – yt-studio-production — 1af637fa Deployed Nov 3, 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