Skip to content

fix: resolve CI workflow failures for YouTube API v3 worker - #264

Open
circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/youtube-api-v3-ci-fixes-worker
Open

circleci-app[bot] wants to merge 1 commit into
mainfrom
chunk/youtube-api-v3-ci-fixes-worker

Conversation

@circleci-app

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

Copy link
Copy Markdown
Contributor

Summary

  • TypeScript type check was failing in youtube-worker-validate job: app/tsconfig.json was missing @cloudflare/workers-types, so tsc --noEmit couldn't resolve Cloudflare Worker globals (Request, Response, fetch, ExportedHandler, URL). Fixed by adding "types": ["@cloudflare/workers-types"] — TypeScript resolves the package from the root node_modules up the directory tree.
  • Created app/worker-configuration.d.ts — the tsconfig include list referenced this file but it didn't exist. The file declares the Env interface matching wrangler.jsonc bindings (YOUTUBE_API_KEY, CHANNEL_ID, ENVIRONMENT).
  • Removed duplicate JSON keys — fix_deps.py had added "circletui" and "trigger-circleci-pipeline" to package.json dependencies (already in devDependencies), and "firebase-admin" was duplicated in functions/package.json. These caused parser warnings and potential JSON tooling issues.

Validation

All CI workflow steps verified locally:

  • ruff check . + ruff format --check . + mypy . — pass
  • pnpm install --frozen-lockfile — passes (lockfile unchanged)
  • functions ESLint + TypeScript check — pass
  • app tsc --noEmit — now passes (was failing with 11 errors)
  • app npm test -- --run — 4/4 tests pass

https://app.circleci.com/agents/gh/ruhdevops/chat/509fdf53-31da-4cfe-ba79-a1df7b3bd17c

- Add @cloudflare/workers-types to app/tsconfig.json so tsc --noEmit
  can resolve Request, Response, fetch, ExportedHandler globals
- Create app/worker-configuration.d.ts (required by tsconfig include)
  with typed Env bindings matching wrangler.jsonc vars
- Remove duplicate "circletui" key from root package.json dependencies
- Remove duplicate "firebase-admin" key from functions/package.json dependencies

These fixes unblock the youtube-worker-validate and node-checks CI jobs
that were failing due to missing type declarations and invalid JSON duplicate keys.

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 20, 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 20, 2026 2:32am UTC
yt-studio-production Error Error Oct 20, 2026 2:32am 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 4c1e0b3 Oct 20 2026, 02: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 4c1e0b3 Oct 20 2026, 02:32 AM

@netlify

netlify Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for elegant-seahorse-d9d460 ready!

Name Link
🔨 Latest commit 4c1e0b3
🔍 Latest deploy log https://app.netlify.com/projects/elegant-seahorse-d9d460/deploys/6ad6d2b2beb0d9de98ee5e3e
😎 Deploy Preview https://deploy-preview-264--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 20, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for ytr-studio failed.

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

This branch had an error being deployed

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