From 3d8419102e12fc6cf8a7752f68b22a1409692648 Mon Sep 17 00:00:00 2001 From: "circleci-app[bot]" <127350680+circleci-app[bot]@users.noreply.github.com> Date: Mon, 19 Oct 2026 00:32:01 +0000 Subject: [PATCH] feat: add YouTube API v3 worker deploy and fix CI workflow failures - Add deploy-worker job to CircleCI config to deploy the Cloudflare Worker that serves YouTube API v3 data (channel stats, videos, search) - Fix duplicate JSON keys in package.json (circletui appeared twice in dependencies) that caused wrangler to emit warnings - Fix functions/package.json: remove duplicate firebase-admin entry, downgrade typescript from ^6.0.0 to ^5.1.6 (6.x doesn't exist), relax node engine from "24" to ">=20" for CI compatibility - Reformat functions/.eslintrc.js from minified to readable (same config) - Regenerate functions/package-lock.json to match corrected package.json - Fix local_checks.py trailing comma for ruff linter compliance - Upgrade node-checks CI image from node:20.18 to node:22.9 - Add functions npm install step to node-checks CI job AI-Generated: true --- .circleci/config.yml | 35 +- fix_app.py | 17 +- fix_deps.py | 10 +- fix_index.py | 8 +- functions/.eslintrc.js | 28 +- functions/package-lock.json | 21831 ++++++---------------------------- functions/package.json | 5 +- local_checks.py | 2 +- package.json | 3 +- 9 files changed, 3501 insertions(+), 18438 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index bb176b7..aec0753 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -25,14 +25,19 @@ jobs: node-checks: docker: - - image: cimg/node:20.18 + - image: cimg/node:22.9 steps: - checkout - run: - name: Install dependencies + name: Install root dependencies command: | sudo npm install -g pnpm pnpm install --frozen-lockfile + - run: + name: Install functions dependencies + command: | + cd functions + npm install --no-audit --no-fund - run: name: Run linting command: | @@ -42,14 +47,38 @@ jobs: name: TypeScript type check command: | cd functions - npx tsc --noEmit + ./node_modules/.bin/tsc --noEmit - run: name: Quality GC check command: npx quality-gc run || echo "Quality GC issues found" + deploy-worker: + docker: + - image: cimg/node:22.9 + steps: + - checkout + - run: + name: Install dependencies + command: | + sudo npm install -g pnpm + pnpm install --frozen-lockfile + - run: + name: Deploy YouTube API Worker to Cloudflare + command: | + pnpm wrangler deploy --env production + environment: + CLOUDFLARE_API_TOKEN: $CLOUDFLARE_API_TOKEN + CLOUDFLARE_ACCOUNT_ID: $CLOUDFLARE_ACCOUNT_ID + workflows: version: 2 validate: jobs: - python-checks - node-checks + - deploy-worker: + requires: + - node-checks + filters: + branches: + only: main diff --git a/fix_app.py b/fix_app.py index 247998d..8dd1092 100644 --- a/fix_app.py +++ b/fix_app.py @@ -1,13 +1,16 @@ -import sys +from pathlib import Path -filepath = 'js/app.js' -with open(filepath, 'r') as f: +filepath = Path("js/app.js") +with filepath.open() as f: lines = f.readlines() # Part 1: DOM elements for i, line in enumerate(lines): if "themeMenu: document.getElementById('themeMenu')," in line: - lines.insert(i + 1, " episodesNavBtn: document.querySelector('[data-action=\"scroll-to-episodes\"]'),\n") + lines.insert( + i + 1, + " episodesNavBtn: document.querySelector('[data-action=\"scroll-to-episodes\"]'),\n", + ) lines.insert(i + 2, " episodesSection: document.getElementById('episodesSection'),\n") break @@ -20,16 +23,16 @@ " DOM.episodesNavBtn.addEventListener('click', () => {\n", " DOM.episodesSection.scrollIntoView({ behavior: 'smooth' });\n", " if (document.body.classList.contains('mobile-nav-active')) {\n", - " document.body.classList.remove('mobile-nav-active');\n", + " document.body.classList.remove('mobile-nav-active');\n", " if (DOM.menuToggle) DOM.menuToggle.setAttribute('aria-expanded', 'false');\n", " }\n", " });\n", " }\n", - "\n" + "\n", ] for idx, new_line in enumerate(scroll_listener): lines.insert(i + idx, new_line) break -with open(filepath, 'w') as f: +with filepath.open("w") as f: f.writelines(lines) diff --git a/fix_deps.py b/fix_deps.py index 34cbd97..f59b27b 100644 --- a/fix_deps.py +++ b/fix_deps.py @@ -15,12 +15,12 @@ # Maybe the CI is running against a different state of the PR? # Let's just make sure trigger-circleci-pipeline and circletui are in package.json if they are in the lockfile. -if 'dependencies' not in pkg: - pkg['dependencies'] = {} +if "dependencies" not in pkg: + pkg["dependencies"] = {} -pkg['dependencies']['trigger-circleci-pipeline'] = '^1.12.1' -pkg['dependencies']['circletui'] = '^1.0.3' -pkg['dependencies']['@circleci/circleci-config-sdk'] = '^0.12.5' +pkg["dependencies"]["trigger-circleci-pipeline"] = "^1.12.1" +pkg["dependencies"]["circletui"] = "^1.0.3" +pkg["dependencies"]["@circleci/circleci-config-sdk"] = "^0.12.5" with Path("package.json").open("w") as f: json.dump(pkg, f, indent=2) diff --git a/fix_index.py b/fix_index.py index 353b69c..5e73c76 100644 --- a/fix_index.py +++ b/fix_index.py @@ -1,10 +1,10 @@ -with open('index.html', 'r') as f: +with open("index.html") as f: content = f.read() # Add title attribute to search button -old_search = '