Code Issues Pull Requests Actions Security InsightsSettings

Fix critical build errors: remove duplicate JSX routes and async client component conflict #4672

Draft⚡ AI-generatedXLccantynz wants to mergecopilot/fix-build-errors-remove-duplicate-routes→main↑3 ↓7opened Mar 23, 20263/4 tasksLive: 0 editing

ccantynzcommented Mar 23, 2026

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

Next.js build was broken by duplicate .jsx/.tsx route files and an invalid async client component, preventing Vercel deployments.

What changed?

Small fix

Duplicate JSX routes removed (Next.js prohibits .jsx + .tsx for the same route):

src/app/page.jsx
src/app/(marketing)/faq/page.jsx
src/app/(marketing)/pricing/page.jsx
src/app/(marketing)/templates/page.jsx
src/app/(marketing)/use-cases/page.jsx

Async server component fixed — src/app/admin/projects/page.tsx had "use client" on an async function (invalid in React); directive and leading BOM removed.

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

Removed .jsx files were superseded by their .tsx counterparts — no functionality lost. The "use client" removal makes AdminProjectsPage a server component, which is correct given it uses async/await for data fetching.

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

ccantynzcommented Mar 23, 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		Mar 23, 2026 1:28am

Files changed

Side-by-side diff for copilot/fix-build-errors-remove-duplicate-routes → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →

Risk score:⚠ HIGH (5/10)bbb5559

Touches 6 file(s) with 0 added and 493 removed across 0 owner(s). Includes a new dependency. No tests were added for the new code.

See full signal breakdown

files changed: 6
lines added/removed: 0 / 493
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.

2 conflicting files between copilot/fix-build-errors-remove-duplicate-routes and main:

package-lock.json
src/app/admin/projects/page.tsx

git fetch origin
git checkout copilot/fix-build-errors-remove-duplicate-routes
git merge origin/main      # fix the files listed above, then:
git add -A && git commit
git push origin copilot/fix-build-errors-remove-duplicate-routes

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