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

Complete all open pull requests (#1–#5) #4696

Draft⚡ AI-generatedXLccantynz wants to mergecopilot/complete-all-pull-requests→main↑2 ↓9opened Feb 21, 20264/7 tasksLive: 0 editing
ccantynzcommented Feb 21, 2026

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

Consolidates all 5 open PRs into a single merge-ready branch.

What changed?

  • Small fix
  • Feature
  • Refactor
  • Infra / CI

Build fixes (PR #5)

  • d8-deploy.yml — fix continue-on-error misindented as sibling of name:, breaking YAML parse
  • D8TV.tsx — remove stray tone?/lead?/eyebrow? bare statements inside function body
  • __probe__/route.ts — try: any { → try {
  • kv.ts — remove eslint-disable @typescript-eslint/no-var-requires (rule not in next/core-web-vitals)
  • D8Bits.tsx — D8Card accepts title, body, kicker props (pricing page was passing them)
  • marketing-queue/page.tsx — useMemo typed as Record<string, string> to fix header spread type error
  • page.tsx — remove intentionally-broken AutoRepairDemoWidget import
  • VideoStudioClient.tsx — capture safeCtx: CanvasRenderingContext2D after null guard so closure doesn't widen back to null
  • d8kv.ts — add missing projectVideoKey() export (used by /projects/[id]/video/page.tsx)
  • tsconfig.json + .eslintignore — exclude _archive/** (backup files were being compiled/linted)

TV route (PR #1 + #3)

  • src/app/admin/preview/tv/page.tsx — proper App Router page replacing the blank tracked file

Cleanup (PR #2 + #3)

  • .gitignore — suppress *.ps1, agent_logs/, bundle_logs/, build-output*.txt, vercel-*, LOCK_GREEN_*, tsconfig.tsbuildinfo
  • Delete 33 root-level artifacts: build logs, dump files, stale next.config.js.{bak,off}, JSON payloads

Gallery + protected paths (PR #4)

  • gallery/page.jsx — <img> → next/image <Image fill sizes=…> inside relative container
  • protected-paths.ps1 — add $RequireLabel param; workflow passes -RequireLabel "allow-protected-paths" (decouples label name from JSON config)

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

Adding *.ps1 to .gitignore is broad — existing tracked scripts (D8_GUARD.ps1, RUN_AUTOMATION.ps1, etc.) remain tracked and unaffected, but new .ps1 files won't be committed without removing that rule first.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

ccantynzcommented Feb 21, 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 21, 2026 4:16am
dominat8com Ready Ready Preview, Comment Feb 21, 2026 4:16am

Request Review

Files changed

Side-by-side diff for copilot/complete-all-pull-requests → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
Risk score:⚠ HIGH (7/10)a35dca8
Touches 50 file(s) with 79 added and 1382 removed across 0 owner(s). Includes a new dependency. No tests were added for the new code.
See full signal breakdown
  • files changed: 50
  • lines added/removed: 79 / 1382
  • 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.
13 conflicting files between copilot/complete-all-pull-requests and main:
  • ._app_disabled/projects/[projectId]/page.tsx
  • .eslintignore
  • .gitignore
  • src/app/(marketing)/_d8/D8Bits.tsx
  • src/app/(marketing)/gallery/page.jsx
  • 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/complete-all-pull-requests
git merge origin/main      # fix the files listed above, then:
git add -A && git commit
git push origin copilot/complete-all-pull-requests
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