CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix: resolve all CI failures blocking PR merges #4673

Merged⚡ AI-generatedXSccantynz wants to mergecopilot/fix-merge-blocking-issuemainopened Mar 22, 20265/7 tasks
ccantynzcommented Mar 22, 2026

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


Multiple build-breaking errors on main caused every PR's PR Quality Gates check to fail, blocking all merges.

What changed?

  • Small fix
  • Feature
  • Refactor
  • Infra / CI

CI / Workflow

  • d8-deploy.yml: continue-on-error: true was at 6-space indent (sibling to step list item, not inside it) — invalid YAML. GitHub failed to parse the file on every push, surfacing as a failing check on PRs.
  • protected-paths.ps1: moved param() before Set-StrictMode / $ErrorActionPreference — required by PowerShell for parameter binding.
  • tsconfig.json + .eslintignore: added all stale archive dirs (_archive, .d8_bak, _app_disabled, pages__DISABLED, etc.) to exclude lists — stale files were being compiled and failing on broken imports.

Code errors (build-blocking)

  • src/app/page.tsx: removed intentional-but-unfixed demo import of non-existent AutoRepairDemoWidget.
  • src/app/_client/D8TV.tsx: removed floating type annotations (tone?: string; etc.) inside a function body — caused ESLint parse error.
  • src/app/api/__probe__/route.ts: try: any {try {.
  • src/lib/kv.ts: removed eslint-disable @typescript-eslint/no-var-requires — plugin not loaded, caused rule-not-found error.
  • src/app/(marketing)/_d8/D8Bits.tsx: added title, body, kicker props to D8Card (pricing page was already calling it with these props).
  • src/app/admin/marketing-queue/page.tsx: typed useMemo return as Record<string, string> to satisfy HeadersInit.
  • src/lib/d8kv.ts: added missing projectVideoKey export (referenced by video project page).
  • src/app/video/_client/VideoStudioClient.tsx: captured non-null canvas context as ctx2d before passing into tick() closure (TypeScript doesn't narrow const through closures).

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

No protected paths modified. All changes are correctness fixes — no feature or design changes. The D8Card prop additions are additive and backward-compatible.


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

ccantynzcommented Mar 22, 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 22, 2026 11:21pm

Request Review

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts