refactor: org 라우팅 구조 수정 by seongminn · Pull Request #569 · hufscheer/web

seongminn · 2026-07-14T14:54:34Z

✅ 작업 내용

기존에는 org 정보를 쿼리 파라미터로 관리하고 있었습니다.
데이터 구조를 생각해봤을 때 조직 -> 스포츠 종목 -> 경기 목록 -> 경기 Id처럼 조직 정보는 모든 스포츠, 게임들의 최상위 정보이므로, 이에 맞게 조직 정보를 세그먼트로 변경해야 한다고 판단했습니다.
따라서 domain/soccer?org=9 구조에서 domain/org/[orgId]/soccer 구조로 마이그레이션 했습니다.
기존과 동일하게 org 정보가 없을 때는 쿠키의 정보를 이용하거나, welcome 페이지로 이동하도록 정책을 유지했습니다.

vercel · 2026-07-14T14:54:39Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
manager	Ready	Preview, Comment	Jul 18, 2026 7:44am
spectator	Ready	Preview, Comment	Jul 18, 2026 7:44am

gemini-code-assist

Code Review

This pull request refactors the application to migrate the organization ID from a query parameter to a path segment (/org/[orgId]/...), updating routes, hooks, and middleware accordingly. The review feedback highlights critical Next.js 15 compatibility issues where layout parameters are now passed as a Promise and must be awaited to prevent runtime errors. Additionally, a bug was identified in the basketball layout where the sport was hardcoded to 'SOCCER'. Finally, defensive programming improvements are suggested across several page components to validate the orgId and id parameters to prevent redirecting to invalid paths containing NaN.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

gemini-code-assist · 2026-07-14T14:57:57Z

+interface MainLayoutProps extends PropsWithChildren {
+  params: {
+    orgId: string;
+  };
+}
+
+const MainLayout = ({ children, params: { orgId } }: MainLayoutProps) => {
  return (
    <div className="relative flex min-h-dvh flex-col">
-      <Header.Root left={<LeftSlot />} right={<OrgSwitcher />} />
+      <Header.Root
+        left={<LeftSlot sport="SOCCER" orgId={Number(orgId)} />}
+        right={<OrgSwitcher />}
+      />


Next.js 15에서는 layout의 params도 Promise로 전달되므로, 동기적으로 구조 분해 할당을 하면 런타임 에러가 발생할 수 있습니다. MainLayout을 async 함수로 변경하고 params를 await 하도록 수정해야 합니다. 또한, 농구(basketball) 레이아웃임에도 LeftSlot에 sport="SOCCER"가 하드코딩되어 있어 로고 클릭 시 축구 페이지로 이동하는 버그가 있습니다. 이를 sport={SPORT_TYPE}으로 수정해야 합니다.

Suggested change

interface MainLayoutProps extends PropsWithChildren {

params: {

orgId: string;

};

}

const MainLayout = ({ children, params: { orgId } }: MainLayoutProps) => {

return (

<div className="relative flex min-h-dvh flex-col">

<Header.Root left={<LeftSlot />} right={<OrgSwitcher />} />

<Header.Root

left={<LeftSlot sport="SOCCER" orgId={Number(orgId)} />}

right={<OrgSwitcher />}

/>

interface MainLayoutProps extends PropsWithChildren {

params: Promise<{

orgId: string;

}>;

}

const MainLayout = async ({ children, params }: MainLayoutProps) => {

const { orgId } = await params;

return (

<div className="relative flex min-h-dvh flex-col">

<Header.Root

left={<LeftSlot sport={SPORT_TYPE} orgId={Number(orgId)} />}

right={<OrgSwitcher />}

/>

gemini-code-assist · 2026-07-14T14:57:57Z

+interface MainLayoutProps extends PropsWithChildren {
+  params: {
+    orgId: string;
+  };
+}
+
+const MainLayout = ({ children, params: { orgId } }: MainLayoutProps) => {
  return (
    <div className="relative flex min-h-dvh flex-col">
-      <Header.Root left={<LeftSlot />} right={<OrgSwitcher />} />
+      <Header.Root
+        left={<LeftSlot sport="SOCCER" orgId={Number(orgId)} />}
+        right={<OrgSwitcher />}
+      />


Next.js 15에서는 layout의 params도 Promise로 전달되므로, 동기적으로 구조 분해 할당을 하면 런타임 에러가 발생할 수 있습니다. MainLayout을 async 함수로 변경하고 params를 await 하도록 수정해야 합니다. 일관성을 위해 sport prop도 SPORT_TYPE을 사용하도록 변경하는 것이 좋습니다.

Suggested change

interface MainLayoutProps extends PropsWithChildren {

params: {

orgId: string;

};

}

const MainLayout = ({ children, params: { orgId } }: MainLayoutProps) => {

return (

<div className="relative flex min-h-dvh flex-col">

<Header.Root left={<LeftSlot />} right={<OrgSwitcher />} />

<Header.Root

left={<LeftSlot sport="SOCCER" orgId={Number(orgId)} />}

right={<OrgSwitcher />}

/>

interface MainLayoutProps extends PropsWithChildren {

params: Promise<{

orgId: string;

}>;

}

const MainLayout = async ({ children, params }: MainLayoutProps) => {

const { orgId } = await params;

return (

<div className="relative flex min-h-dvh flex-col">

<Header.Root

left={<LeftSlot sport={SPORT_TYPE} orgId={Number(orgId)} />}

right={<OrgSwitcher />}

/>

gemini-code-assist · 2026-07-14T14:57:57Z

+  const { orgId: _orgId, id: _id } = await params;
+  const orgId = Number(_orgId);
  const id = Number(_id);

  if (Number.isNaN(id) || id <= 0) {
-    redirect(routes.home({ sport: SPORT_TYPE }));
+    redirect(routes.home({ orgId, sport: SPORT_TYPE }));
  }


orgId가 유효한 숫자가 아닐 경우(예: NaN 또는 <= 0), routes.home으로 리다이렉트할 때 /org/NaN/... 경로로 이동하게 됩니다. 방어적 프로그래밍 관점에서 orgId에 대한 유효성 검증을 추가하여 잘못된 경로일 경우 routes.welcome으로 안전하게 리다이렉트하는 것이 좋습니다.

Suggested change

const { orgId: _orgId, id: _id } = await params;

const orgId = Number(_orgId);

const id = Number(_id);

if (Number.isNaN(id) || id <= 0) {

redirect(routes.home({ sport: SPORT_TYPE }));

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

}

const { orgId: _orgId, id: _id } = await params;

const orgId = Number(_orgId);

const id = Number(_id);

if (Number.isNaN(orgId) || orgId <= 0) {

redirect(routes.welcome);

}

if (Number.isNaN(id) || id <= 0) {

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

}

gemini-code-assist · 2026-07-14T14:57:57Z

+  const { orgId: _orgId, id: _id } = await params;
+  const orgId = Number(_orgId);
  const id = Number(_id);

  if (Number.isNaN(id) || id <= 0) {
-    redirect(routes.home({ sport: SPORT_TYPE }));
+    redirect(routes.home({ orgId, sport: SPORT_TYPE }));
  }


orgId가 유효한 숫자가 아닐 경우(예: NaN 또는 <= 0), routes.home으로 리다이렉트할 때 /org/NaN/... 경로로 이동하게 됩니다. 방어적 프로그래밍 관점에서 orgId에 대한 유효성 검증을 추가하여 잘못된 경로일 경우 routes.welcome으로 안전하게 리다이렉트하는 것이 좋습니다.

Suggested change

const { orgId: _orgId, id: _id } = await params;

const orgId = Number(_orgId);

const id = Number(_id);

if (Number.isNaN(id) || id <= 0) {

redirect(routes.home({ sport: SPORT_TYPE }));

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

}

const { orgId: _orgId, id: _id } = await params;

const orgId = Number(_orgId);

const id = Number(_id);

if (Number.isNaN(orgId) || orgId <= 0) {

redirect(routes.welcome);

}

if (Number.isNaN(id) || id <= 0) {

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

}

gemini-code-assist · 2026-07-14T14:57:57Z

+  const { orgId: _orgId, id: _id } = await params;
+  const orgId = Number(_orgId);
  const id = Number(_id);

  if (!_id || Number.isNaN(id) || id <= 0) {
-    redirect(routes.home({ sport: SPORT_TYPE }));
+    redirect(routes.home({ orgId, sport: SPORT_TYPE }));
  }


orgId가 유효한 숫자가 아닐 경우(예: NaN 또는 <= 0), routes.home으로 리다이렉트할 때 /org/NaN/... 경로로 이동하게 됩니다. 방어적 프로그래밍 관점에서 orgId에 대한 유효성 검증을 추가하여 잘못된 경로일 경우 routes.welcome으로 안전하게 리다이렉트하는 것이 좋습니다. 또한 기존의 !_id 조건은 Number.isNaN(id)에 의해 이미 커버되므로 중복 검사를 제거하여 단순화할 수 있습니다.

Suggested change

const { orgId: _orgId, id: _id } = await params;

const orgId = Number(_orgId);

const id = Number(_id);

if (!_id || Number.isNaN(id) || id <= 0) {

redirect(routes.home({ sport: SPORT_TYPE }));

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

}

const { orgId: _orgId, id: _id } = await params;

const orgId = Number(_orgId);

const id = Number(_id);

if (Number.isNaN(orgId) || orgId <= 0) {

redirect(routes.welcome);

}

if (Number.isNaN(id) || id <= 0) {

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

}

gemini-code-assist · 2026-07-14T14:57:57Z

+  const { orgId: _orgId, id: _id } = await params;
+  const orgId = Number(_orgId);
  const id = Number(_id);


다른 상세 페이지(games, leagues)와 달리 teams/[id] 페이지에서는 orgId와 id에 대한 유효성 검증이 누락되어 있습니다. 잘못된 id나 orgId가 들어올 경우 API 요청 실패나 잘못된 경로 렌더링이 발생할 수 있으므로, 일관성 있게 유효성 검증 및 리다이렉트 로직을 추가하는 것이 좋습니다.

const { orgId: _orgId, id: _id } = await params; const orgId = Number(_orgId); const id = Number(_id); if (Number.isNaN(orgId) || orgId <= 0) { redirect(routes.welcome); } if (Number.isNaN(id) || id <= 0) { redirect(routes.home({ orgId, sport: SPORT_TYPE })); }

gemini-code-assist · 2026-07-14T14:57:58Z

+  const { orgId: _orgId, id: _id } = await params;
+  const orgId = Number(_orgId);
  const id = Number(_id);


다른 상세 페이지(games, leagues)와 달리 teams/[id] 페이지에서는 orgId와 id에 대한 유효성 검증이 누락되어 있습니다. 잘못된 id나 orgId가 들어올 경우 API 요청 실패나 잘못된 경로 렌더링이 발생할 수 있으므로, 일관성 있게 유효성 검증 및 리다이렉트 로직을 추가하는 것이 좋습니다.

const { orgId: _orgId, id: _id } = await params; const orgId = Number(_orgId); const id = Number(_id); if (Number.isNaN(orgId) || orgId <= 0) { redirect(routes.welcome); } if (Number.isNaN(id) || id <= 0) { redirect(routes.home({ orgId, sport: SPORT_TYPE })); }

gemini-code-assist · 2026-07-14T14:57:58Z

+const Page = async ({ params }: Props) => {
+  const { orgId } = await params;
+  redirect(routes.home({ orgId: Number(orgId), sport: DEFAULT_SPORT }));
+};


orgId가 유효한 숫자가 아닐 경우(예: NaN 또는 <= 0), routes.home으로 리다이렉트할 때 /org/NaN/... 경로로 이동하게 됩니다. 방어적 프로그래밍 관점에서 orgId에 대한 유효성 검증을 추가하여 잘못된 경로일 경우 routes.welcome으로 안전하게 리다이렉트하는 것이 좋습니다.

Suggested change

const Page = async ({ params }: Props) => {

const { orgId } = await params;

redirect(routes.home({ orgId: Number(orgId), sport: DEFAULT_SPORT }));

};

const Page = async ({ params }: Props) => {

const { orgId: _orgId } = await params;

const orgId = Number(_orgId);

if (Number.isNaN(orgId) || orgId <= 0) {

redirect(routes.welcome);

}

redirect(routes.home({ orgId, sport: DEFAULT_SPORT }));

};

gemini-code-assist · 2026-07-14T14:57:58Z

+const Page = async ({ params }: Props) => {
+  const { orgId } = await params;
+  redirect(routes.home({ orgId: Number(orgId), sport: SPORT_TYPE }));
+};


orgId가 유효한 숫자가 아닐 경우(예: NaN 또는 <= 0), routes.home으로 리다이렉트할 때 /org/NaN/... 경로로 이동하게 됩니다. 방어적 프로그래밍 관점에서 orgId에 대한 유효성 검증을 추가하여 잘못된 경로일 경우 routes.welcome으로 안전하게 리다이렉트하는 것이 좋습니다.

Suggested change

const Page = async ({ params }: Props) => {

const { orgId } = await params;

redirect(routes.home({ orgId: Number(orgId), sport: SPORT_TYPE }));

};

const Page = async ({ params }: Props) => {

const { orgId: _orgId } = await params;

const orgId = Number(_orgId);

if (Number.isNaN(orgId) || orgId <= 0) {

redirect(routes.welcome);

}

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

};

gemini-code-assist · 2026-07-14T14:57:58Z

+const Page = async ({ params }: Props) => {
+  const { orgId } = await params;
+  redirect(routes.home({ orgId: Number(orgId), sport: SPORT_TYPE }));
+};


orgId가 유효한 숫자가 아닐 경우(예: NaN 또는 <= 0), routes.home으로 리다이렉트할 때 /org/NaN/... 경로로 이동하게 됩니다. 방어적 프로그래밍 관점에서 orgId에 대한 유효성 검증을 추가하여 잘못된 경로일 경우 routes.welcome으로 안전하게 리다이렉트하는 것이 좋습니다.

Suggested change

const Page = async ({ params }: Props) => {

const { orgId } = await params;

redirect(routes.home({ orgId: Number(orgId), sport: SPORT_TYPE }));

};

const Page = async ({ params }: Props) => {

const { orgId: _orgId } = await params;

const orgId = Number(_orgId);

if (Number.isNaN(orgId) || orgId <= 0) {

redirect(routes.welcome);

}

redirect(routes.home({ orgId, sport: SPORT_TYPE }));

};

sungwonnoh

라우팅 구조가 더 깔끔해진 거 같아요! 수고하셨습니다

refactor: org 라우팅 구조 수정

8e9424c

gemini-code-assist Bot reviewed Jul 14, 2026

View reviewed changes

seongminn added 2 commits July 15, 2026 00:13

feat: add not-found

3ffbfef

feat: redirect 정책 추가

deba1e1

vercel Bot temporarily deployed to Preview – manager July 14, 2026 15:25 Inactive

vercel Bot had a problem deploying to Preview – spectator July 14, 2026 15:26 Failure

fix: orgId 에러 수정

a522925

vercel Bot temporarily deployed to Preview – manager July 14, 2026 15:37 Inactive

vercel Bot had a problem deploying to Preview – spectator July 14, 2026 15:38 Failure

fix: add organizationId params

7ab63b8

vercel Bot temporarily deployed to Preview – manager July 14, 2026 15:46 Inactive

vercel Bot deployed to Preview – spectator July 14, 2026 15:47 View deployment

Merge branch 'feat/sperate-sport' into feat/org-segment

090f0ef

vercel Bot deployed to Preview – spectator July 18, 2026 07:43 View deployment

vercel Bot deployed to Preview – manager July 18, 2026 07:44 View deployment

sungwonnoh approved these changes Jul 19, 2026

View reviewed changes

Uh oh!

Conversation

seongminn commented Jul 14, 2026

✅ 작업 내용

Uh oh!

vercel Bot commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

sungwonnoh left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel Bot commented Jul 14, 2026 •

edited

Loading