Fix ESLint errors and cascading TypeScript build failures #4680
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 | Mar 19, 2026 9:57pm |
Files changed
Side-by-side diff for copilot/disable-eslint-rules → main.
Cross-repo impact
See what breaks downstream if this PR merges.
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 failingGateTestDisabled 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
Originally written by @Copilot on GitHub.
Imported from https://github.com/ccantynz-alt/Dominat8.com/pull/22
What changed?
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 bodyroute.ts— invalidtry: any {→try {kv.ts—eslint-disablereferencing non-existent@typescript-eslint/no-var-requiresruleTypeScript errors (cascading, previously masked):
(marketing)/_d8/D8Bits.tsx—D8Cardmissingtitle/body/kickerprops used by all callerspage.tsx— removed deadAutoRepairDemoWidgetimport (module doesn't exist)marketing-queue/page.tsx—headersunion type not assignable toHeadersInitd8kv.ts— missingprojectVideoKeyexportVideoStudioClient.tsx—CanvasRenderingContext2D | nullnot narrowed in closureConfig:
tsconfig.json/.eslintignore— exclude_archive/(stale backup files with broken imports)Quality gates (must be green)
Notes / risks
D8Cardin(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.projectVideoKeykey format (project:{id}:video) is inferred from thed8Key_*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.