ccantynz/dominat8.com
CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
Understand
Explain codebaseAsk AIChat with repoClaude sessionsNL searchSemantic searchArchaeologyDebt mapDocs tracking
Build
Spec to PRWorkspaceGenerate testsPropose migrationDev environmentCodebase migrator
Ship
Preview URLsAI release notesAI changelog
More
Repository
CommitsBranchesTagsReleasesPackagesSymbolsStale branches
Planning
MilestonesProjectsDiscussionsWikiContributors
Delivery
GatesMerge queueMerge checksDeploymentsPipelineAgents
Activity
Health reportPulsePush log
Settings

Fix two failing CI checks: Protected Paths script crash + build errors #4693

Draft⚡ AI-generatedMccantynz wants to mergecopilot/fix-failing-tests→main↑2 ↓9opened Feb 28, 20265/7 tasksLive: 0 editing
ccantynzcommented Feb 28, 2026

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

Two CI workflows were failing on claude/setup-glossy-icon-pages-AR2NL due to pre-existing bugs in the codebase that block any PR from passing CI.

What changed?

  • Small fix
  • Feature
  • Refactor
  • Infra / CI

Protected Paths workflow — PowerShell parse error: Set-StrictMode before param() causes PS to treat the param block as a regular call, making typed defaults invalid syntax. Moved param() to be first statement.

PR Quality Gates workflow — multiple TypeScript/ESLint errors were failing the build (some masked by earlier parse errors):

File Issue
src/app/_client/D8TV.tsx tone?: string; lead?: string; eyebrow?: string; accidentally inside function body
src/app/api/__probe__/route.ts Invalid try: any { syntax
src/lib/kv.ts eslint-disable referencing non-existent rule @typescript-eslint/no-var-requires
src/app/page.tsx Intentional-demo broken import (AutoRepairDemoWidget) left in production code
src/app/(marketing)/_d8/D8Bits.tsx D8Card missing title/body/kicker props used by pricing/page.tsx
src/app/admin/marketing-queue/page.tsx useMemo inferred { "x-admin-token"?: undefined } not assignable to Record<string, string>
src/app/video/_client/VideoStudioClient.tsx Closure doesn't re-narrow CanvasRenderingContext2D | null — captured as non-null alias before inner fn
src/lib/d8kv.ts Missing projectVideoKey export consumed by video page
tsconfig.json _archive/** not excluded — archived files pulled into TS compilation

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

ALLOW_PROTECTED_PATHS added because src/app/_client/D8TV.tsx and src/app/page.tsx are in protected paths but required bug fixes. Should be removed or reviewed after merge if stricter path protection is desired going forward.

The _archive/** tsconfig exclusion is additive — archived code was always unreachable at runtime, now it's also excluded from type-checking.

🔒 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 Feb 28, 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 Feb 28, 2026 1:54pm
dominat8com Ready Ready Preview, Comment Feb 28, 2026 1:54pm

Request Review

Files changed

Side-by-side diff for copilot/fix-failing-tests → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
Risk score:⚠ HIGH (5/10)80262f6
Touches 12 file(s) with 37 added and 30 removed across 0 owner(s). Includes a new dependency. No tests were added for the new code.
See full signal breakdown
  • files changed: 12
  • lines added/removed: 37 / 30
  • 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.
12 conflicting files between copilot/fix-failing-tests and main:
  • .github/ALLOW_PROTECTED_PATHS
  • .github/scripts/protected-paths.ps1
  • 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/fix-failing-tests
git merge origin/main      # fix the files listed above, then:
git add -A && git commit
git push origin copilot/fix-failing-tests
After you push, the merge check re-runs automatically on the next page load and the Merge button unblocks once it passes.
Conflicts detected — GlueCron AI will attempt auto-resolution on merge.
c comment · e edit title · m merge · a approve · r request changes · ? shortcuts