CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix: resolve all lint/build/CI errors across workflows #4690

Open⚡ AI-generatedMccantynz wants to mergecopilot/ensure-all-jobs-greenmain↑3 ↓9opened Mar 1, 20264/7 tasksLive: 0 editing
ccantynzcommented Mar 1, 2026

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


What changed?

  • Small fix
  • Feature
  • Refactor
  • Infra / CI

CI was red on main due to accumulated syntax errors, type errors, and a malformed workflow file. Ten files fixed:

Syntax/parse errors

  • D8TV.tsx — removed stray type annotations pasted into function body
  • route.tstry: any {try {
  • kv.ts — disabled nonexistent ESLint rule @typescript-eslint/no-var-requires

Type errors

  • (marketing)/_d8/D8Bits.tsxD8Card lacked title/body/kicker props used by all 4 marketing pages
  • marketing-queue/page.tsxuseMemo return type widened to Record<string, string> for fetch headers compat
  • d8kv.ts — added missing projectVideoKey export
  • VideoStudioClient.tsx — canvas context type narrowing lost in closure; used intermediate binding

Build config

  • tsconfig.json — excluded _archive/** (old backup dir was compiled)
  • page.tsx — removed intentionally-broken AutoRepairDemoWidget import

CI workflow

  • d8-deploy.ymlcontinue-on-error was at list-item indent instead of step-property indent:
# before (invalid)
      - name: Build gate
      continue-on-error: true
        run: npm run build

# after
      - name: Build gate
        continue-on-error: true
        run: npm run build

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

  • All remaining ESLint output is warnings only (pre-existing: no-img-element, react-hooks/exhaustive-deps, no-async-client-component). No errors.
  • _archive/ exclusion is additive to tsconfig.json; no impact on app source.
  • CodeQL: 0 alerts.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

ccantynzcommented Mar 1, 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)
dominat8-engine Ready Ready Preview, Comment Mar 14, 2026 7:29am
dominat8com Ready Ready Preview, Comment Mar 14, 2026 7:29am

Request Review

gluecron[bot]🤖 botcommented 25d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.
gluecron[bot]🤖 botcommented 18d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.
gluecron[bot]🤖 botcommented 11d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.
gluecron[bot]🤖 botcommented 4d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.

Files changed

Side-by-side diff for copilot/ensure-all-jobs-green → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
Risk score: LOW (2/10)daf6501
Touches 1 file(s) with 3 added and 2 removed across 0 owner(s). Includes a new dependency. No tests were added for the new code.
See full signal breakdown
  • files changed: 1
  • lines added/removed: 3 / 2
  • 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.
9 conflicting files between copilot/ensure-all-jobs-green and main:
  • 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/ensure-all-jobs-green
git merge origin/main      # fix the files listed above, then:
git add -A && git commit
git push origin copilot/ensure-all-jobs-green
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