Skip to content

fix: resolve CI workflow failures and add YouTube API worker-build job - #251

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

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

Conversation

@circleci-app

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

Copy link
Copy Markdown
Contributor

Summary

  • Fix functions/package.json: removed duplicate firebase-admin entry, removed stray lint package from dependencies, upgraded firebase-admin to ^13.9.0, corrected typescript from non-existent ^6.0.0 to stable ^5.8.3, and set engines.node to 20 to match the CI Docker image
  • Fix node-checks CI job: added an explicit npm install step inside functions/ before running ESLint and tsc — both were failing because the subdirectory's node_modules were never populated
  • Add worker-build CI job: installs root pnpm dependencies, type-checks src/youtube-worker.ts (the YouTube API v3 Cloudflare Worker) via tsc --noEmit, then deploys to Cloudflare Workers with Wrangler when CLOUDFLARE_API_TOKEN is available in the project environment
  • Wire worker-build into the validate workflow alongside python-checks and node-checks

Worker overview

src/youtube-worker.ts is a Cloudflare Worker that proxies YouTube Data API v3 for channel UCrjJP_SHUeCmqpTSHJCXkdA. It exposes:

Endpoint Description
GET / or /health Health check + endpoint listing
GET /api/channel Channel statistics from YouTube API
GET /api/videos Paginated uploads playlist
GET /api/search?q=… Channel-scoped video search

All endpoints fall back to demo data when YOUTUBE_API_KEY is not bound as a Cloudflare secret.

Test plan

  • Confirm validate workflow runs all three jobs (python-checks, node-checks, worker-build) without hard failures
  • Confirm node-checks ESLint and tsc steps complete after npm install in functions/
  • Confirm worker-build type-check passes on src/youtube-worker.ts
  • Set CLOUDFLARE_API_TOKEN in CircleCI project env to enable live worker deploy

https://app.circleci.com/agents/gh/ruhdevops/chat/e56b71bf-d6d2-42f6-9fa0-55e42642cf71

- Fix functions/package.json: remove duplicate firebase-admin key,
  remove stray `lint` package from dependencies, upgrade firebase-admin
  to ^13.9.0, fix typescript from invalid ^6.0.0 to stable ^5.8.3,
  downgrade engines.node from 24 to 20 to match CI image

- Fix node-checks CI job: add `npm install` step in functions/ before
  running ESLint and tsc (both were failing because node_modules were
  not installed in that subdirectory)

- Add worker-build CI job: installs pnpm deps, runs `tsc --noEmit`
  against src/youtube-worker.ts (the YouTube API v3 Cloudflare Worker),
  then deploys to Cloudflare Workers via wrangler when CLOUDFLARE_API_TOKEN
  is set in the project environment

- Add worker-build to validate workflow alongside python-checks and
  node-checks

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 17, 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 17, 2026 3:25am UTC
yt-studio-production Error Error Oct 17, 2026 3: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 02a93e1 Oct 17 2026, 03:24 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 02a93e1 Oct 17 2026, 03:24 AM

@netlify

netlify Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for elegant-seahorse-d9d460 ready!

Name Link
🔨 Latest commit 02a93e1
🔍 Latest deploy log https://app.netlify.com/projects/elegant-seahorse-d9d460/deploys/6ad2ea87b2b18b34c54fc6b4
😎 Deploy Preview https://deploy-preview-251--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 17, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for ytr-studio failed.

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

This branch had an error being deployed

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