Code Issues Pull Requests Actions Security InsightsSettings

Fix critical build errors and remove duplicate JSX routes #4674

Open⚡ AI-generatedXLccantynz wants to mergeclaude/fix-dominat8-404-error-ukbBD→main↑12 ↓7opened Mar 22, 20264/4 tasksLive: 0 editing

ccantynzcommented Mar 22, 2026

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

What changed?

Small fix

Summary

This PR fixes 6 critical build-breaking errors that were preventing Vercel deployments and causing 404/403 errors on dominat8.com:

Build Errors Fixed

Removed duplicate JSX route files — Next.js cannot have both .tsx and .jsx for the same route. Removed:
- src/app/page.jsx (kept page.tsx)
- src/app/(marketing)/faq/page.jsx (kept page.tsx)
- src/app/(marketing)/pricing/page.jsx (kept page.tsx)
- src/app/(marketing)/templates/page.jsx (kept page.tsx)
- src/app/(marketing)/use-cases/page.jsx (kept page.tsx)
Removed intentional broken import — src/app/page.tsx had a demo import of non-existent AutoRepairDemoWidget that was breaking the build.
Fixed syntax error in probe API — src/app/api/__probe__/route.ts had invalid try: any { syntax, changed to try {.
Fixed syntax error in D8TV component — src/app/_client/D8TV.tsx had orphaned type annotations inside function body, removed them.
Fixed async client component — src/app/admin/projects/page.tsx was marked "use client" but used async function, removed the client directive.
Fixed ESLint rule reference — src/lib/kv.ts referenced non-existent ESLint rule @typescript-eslint/no-var-requires, changed to no-var.

Supporting Changes

Updated tsconfig.json to exclude archived/disabled directories (_archive/, ._app_disabled/, .d8_bak/, etc.)
Updated .eslintignore to match excluded directories
Enhanced D8Card component with optional title, body, kicker props
Fixed type annotations in VideoStudioClient.tsx and marketing-queue/page.tsx
Added documentation files: PLAN.md (rebuild roadmap) and CLAUDE.md (project reference)

Quality gates

CI build is green (all syntax/import errors fixed)
No unexpected UI/design changes (only removed duplicate routes and fixed broken imports)
No protected paths changed

Notes / risks

This is a critical fix — the site was completely down due to build failures. These changes restore the build to a passing state and allow the site to deploy. All changes are error corrections with no feature modifications.

The removed .jsx files were exact duplicates of their .tsx counterparts, so no functionality is lost.

https://claude.ai/code/session_01BqF6wwgt6BCsRXVb7zJekX

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	Ready	Preview, Comment	Mar 23, 2026 1:24am

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 claude/fix-dominat8-404-error-ukbBD → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →

Risk score: LOW (2/10)bcdb2b6

Touches 2 file(s) with 2 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: 2
lines added/removed: 2 / 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.

19 conflicting files between claude/fix-dominat8-404-error-ukbBD and main:

.eslintignore
.github/ALLOW_PROTECTED_PATHS
.gitignore
CLAUDE.md
next-env.d.ts
package-lock.json
src/app/(marketing)/_d8/D8Bits.tsx
src/app/(marketing)/pricing/page.tsx
src/app/(marketing)/templates/page.tsx
src/app/admin/marketing-queue/page.tsx
src/app/admin/page.tsx
src/app/admin/projects/page.tsx
src/app/not-found.tsx
src/app/page.tsx
src/app/sitemap.ts
src/app/video/_client/VideoStudioClient.tsx
src/lib/d8kv.ts
src/lib/kv.ts
tsconfig.json

git fetch origin
git checkout claude/fix-dominat8-404-error-ukbBD
git merge origin/main      # fix the files listed above, then:
git add -A && git commit
git push origin claude/fix-dominat8-404-error-ukbBD

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