CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix: resolve all CI/workflow failures for green PR builds #4675

Merged⚡ AI-generatedXSccantynz wants to mergecopilot/fix-prs-and-workflowsmainopened Mar 20, 20265/7 tasks
ccantynzcommented Mar 20, 2026

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


What changed?

  • Small fix
  • Feature
  • Refactor
  • Infra / CI

Multiple independent failures prevented CI from passing on any PR: broken workflow YAML, PowerShell parse errors, TypeScript syntax/type errors, missing exports, and stale archive directories polluting compilation.

Workflow / CI

  • d8-deploy.yml: fixed continue-on-error indentation (was at wrong YAML nesting level)
  • protected-paths.ps1: moved param() before Set-StrictMode (PowerShell requirement)

Source fixes

  • D8TV.tsx: removed stale type-declaration lines (tone?: string; etc.) inside function body
  • __probe__/route.ts: try: any {try {
  • kv.ts: removed eslint-disable @typescript-eslint/no-var-requires — rule doesn't exist (no @typescript-eslint plugin in config)
  • page.tsx: removed intentionally broken AutoRepairDemoWidget import
  • d8kv.ts: added missing projectVideoKey export consumed by video page + embed component
  • VideoStudioClient.tsx: non-null assertion on canvas ctx in closure (narrowing doesn't propagate)
  • marketing-queue/page.tsx: typed useMemo return as Record<string, string> to satisfy HeadersInit
  • (marketing)/_d8/D8Bits.tsx: added title/body/kicker props to D8Card (callers already passed them)

Build config

  • tsconfig.json + .eslintignore: excluded 18 stale/archive dirs (~130 orphaned .ts/.tsx files)

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 a stripped-down copy missing props the marketing pages already relied on. Aligned it with src/app/_d8/D8Bits.tsx.
  • .github/ALLOW_PROTECTED_PATHS added to unblock this PR since D8TV.tsx and marketing D8Bits.tsx live under protected paths.
  • 4 pre-existing ESLint warnings remain (missing useEffect deps, <img> vs <Image>, async client component) — all non-blocking, out of scope.

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

ccantynzcommented Mar 20, 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 Ready Ready Preview, Comment Mar 22, 2026 11:42pm

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