Skip to content

fix: skip static prerendering on /results page for Next.js export #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
SynTechRev merged 2 commits into from
Apr 13, 2026
Merged

fix: skip static prerendering on /results page for Next.js export #18

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b7bf564
Initial plan
Copilot Apr 13, 2026
ab8a8e1
fix: add force-dynamic to results page to fix static export prerender…
Copilot Apr 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions frontend/app/results/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
'use client';
export const dynamic = 'force-dynamic';
Comment on lines 1 to +2
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

output: "export" is used for ELECTRON_BUILD (static export to local files). Marking this route dynamic = 'force-dynamic' makes it require runtime rendering, which typically isn't supported in static export and can still fail the build or produce a route that can’t be loaded directly from out/. Prefer keeping the page exportable (e.g., avoid useSearchParams() at render time by reading window.location.search in an effect / state, or move the search-param-dependent UI into a next/dynamic(..., { ssr: false }) client-only component) so next build can prerender a static shell.

Copilot uses AI. Check for mistakes.

Comment on lines 1 to 3
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file now contains a second 'use client'; later (after the header comment). Only the first directive at the very top has any effect; the later one is just an unused string expression and can be removed to avoid confusion/lint noise.

Copilot uses AI. Check for mistakes.
/**
* Results Page — displays audit findings with plain-language explanations.
Expand Down
Loading