Jiseoup/showmycodePublic
EN|KO
  • Code
  • Commits
  • Pull Requests
← Back to list
Merged

fix: render home at request time so CI builds without secrets

#44
JiseoupJiseoup · Jun 23, 2026fix/ci-build-without-secrets → main
fix
OverviewCommitsFiles changed

What & Why

The home page (/[lang]) was prerendered at build, which called the GitHub API and required GITHUB_PAT during next build. Fork PRs can't access repo secrets, so their build step failed.

Adding await connection() makes the home render at request time, so the build no longer calls the API. The revalidate: 60 data cache is unaffected (still 60s). With no build-time API call, the CI build no longer needs secrets, so the env: block is removed — fork PRs now pass the full pipeline and CI holds no secrets.

Related Issue

N/A

How to Verify

  1. Build with no GitHub env vars set (simulating a fork PR): npm run build — it succeeds, and /[lang] shows as ƒ (Dynamic) in the route table.
  2. npm run dev — the home page still lists repositories.

Checklist

  • PR title follows Conventional Commits (feat:, fix:, chore:, refactor:, docs:, i18n:)
  • Translations added to both locales/ko.json and locales/en.json (if UI text changed)
  • Tested on mobile viewport (if UI changed)