CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

Fix ESLint errors and cascading TypeScript build failures #4680

Draft⚡ AI-generatedMccantynz wants to mergecopilot/disable-eslint-rulesmain↑3 ↓9opened Mar 19, 20263/4 tasksLive: 0 editing
ccantynzcommented Mar 19, 2026

Originally written by @Copilot on GitHub.
Imported from https://github.com/ccantynz-alt/Dominat8.com/pull/22


What changed?

  • Small fix

CI build exited with code 1 due to three ESLint errors. Fixing those unmasked five additional TypeScript errors that were previously unreachable.

ESLint errors (root cause):

  • D8TV.tsx — stray type annotations (tone?: string; etc.) inside function body
  • route.ts — invalid try: any {try {
  • kv.tseslint-disable referencing non-existent @typescript-eslint/no-var-requires rule

TypeScript errors (cascading, previously masked):

  • (marketing)/_d8/D8Bits.tsxD8Card missing title/body/kicker props used by all callers
  • page.tsx — removed dead AutoRepairDemoWidget import (module doesn't exist)
  • marketing-queue/page.tsxheaders union type not assignable to HeadersInit
  • d8kv.ts — missing projectVideoKey export
  • VideoStudioClient.tsxCanvasRenderingContext2D | null not narrowed in closure

Config:

  • tsconfig.json / .eslintignore — exclude _archive/ (stale backup files with broken imports)

Quality gates (must be green)

  • CI build is green
  • No unexpected UI/design changes on Dominat8.com
  • If protected paths changed: override label OR ALLOW_PROTECTED_PATHS file added with reason

Notes / risks

  • D8Card in (marketing)/_d8/ was rewritten to match the canonical version in _d8/D8Bits.tsx. Minor style deltas (boxShadow, overflow) come from aligning with that source of truth.
  • projectVideoKey key format (project:{id}:video) is inferred from the d8Key_* naming convention in the same file — verify against any existing KV data.

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

ccantynzcommented Mar 19, 2026

Originally written by @vercel[bot] on GitHub.


The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dominat8com Canceled Canceled Mar 19, 2026 9:57pm

Request Review

Files changed

Side-by-side diff for copilot/disable-eslint-rules → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
Risk score:⚠ HIGH (5/10)f677514
Touches 10 file(s) with 52 added and 17 removed across 0 owner(s). Includes a new dependency. No tests were added for the new code.
See full signal breakdown
  • files changed: 10
  • lines added/removed: 52 / 17
  • distinct owners touched: 0
  • schema migration touched: no
  • locked / sensitive path touched: no
  • adds new dependency: yes
  • bumps major dependency: no
  • tests added for new code: no
  • diff-minus-test ratio: 1.00
How is this calculated? The score is a transparent sum of weighted signals — see src/lib/pr-risk.ts computePrRiskScore.

Gate checks

1 of 6 failing
GateTestDisabled in settingsSkipped
Secret scanNo secrets detectedPassed
Security scanStatic rules ran (8 checks, no AI required) — no static findings. AI review unavailable: The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored.Skipped
Merge checkMerge conflicts detected — AI auto-resolution will be attempted on mergeFailing
CINo CI workflows are configured for this repositorySkipped
AI ReviewAIAI review approvedPassed

Resolve conflicts manually

GlueCron AI will attempt auto-resolution when you press Merge. If it cannot, or you would rather not wait on it, the manual path below always works.
11 conflicting files between copilot/disable-eslint-rules and main:
  • .eslintignore
  • package-lock.json
  • src/app/(marketing)/_d8/D8Bits.tsx
  • src/app/_client/D8TV.tsx
  • src/app/admin/marketing-queue/page.tsx
  • src/app/api/__probe__/route.ts
  • src/app/page.tsx
  • src/app/video/_client/VideoStudioClient.tsx
  • src/lib/d8kv.ts
  • src/lib/kv.ts
  • tsconfig.json
git fetch origin
git checkout copilot/disable-eslint-rules
git merge origin/main      # fix the files listed above, then:
git add -A && git commit
git push origin copilot/disable-eslint-rules
After you push, the merge check re-runs automatically on the next page load and the Merge button unblocks once it passes.
c comment · e edit title · m merge · a approve · r request changes · ? shortcuts