Skip to content

refactor(architecture): 루트 모듈 경계 정리#104

Merged
dev-wooyeon merged 3 commits into
masterfrom
codex/move-next-app-root
May 8, 2026
Merged

refactor(architecture): 루트 모듈 경계 정리#104
dev-wooyeon merged 3 commits into
masterfrom
codex/move-next-app-root

Conversation

@dev-wooyeon

Copy link
Copy Markdown
Owner

변경 내용

  • src/app을 루트 app route adapter로 이동
  • platforminfra로 정리하고 sharedui, blog/ui/visualization, tests/support로 분리
  • RSS 공개 route를 /rss.xml로 드러내고 생성 로직을 blog/services/rss-feed.ts로 이동
  • Agentation overlay, webhook route, dev script, 미사용 의존성 제거
  • 유지 기준이 약한 문서와 research dump를 삭제하고 ADR 0013에 이번 구조 결정을 기록

의도

  • DDD 전략적 설계 개념을 차용한 domain-first modular monolith 경계를 루트 디렉토리에서 드러내기 위함
  • Next.js route adapter, 도메인, 인프라, UI 지원 모듈의 책임을 리뷰 가능한 단위로 분리하기 위함
  • 유지되지 않는 문서와 도구를 줄여 다음 작업자가 구조 의도를 더 쉽게 파악하도록 하기 위함

영향 범위

  • App Router route 경로는 유지하되 파일 위치가 app/**로 변경됨
  • RSS canonical URL이 /rss.xml로 변경되고 기존 /feed.xml route는 제거됨
  • Agentation 관련 로컬 개발 엔드포인트와 script는 더 이상 제공하지 않음
  • OG route(/api/og)는 유지하되 사용하지 않는 @vercel/og 의존성은 제거됨

검증

  • npm run lint
  • npm run lint:css:syntax
  • npm run verify:docs
  • npm run test:unit
  • npm run test:components
  • npm run build
  • 다크/라이트 모드 확인
  • 모바일/데스크톱 확인

체크리스트

  • 작업 단위 브랜치(codex/<task>)에서 진행함
  • 커밋 메시지 규칙(<type>: <한국어 설명>) 준수
  • 관련 이슈/PR 링크를 본문에 연결함

src/app route adapter를 루트 app으로 이동하고 platform/shared 경계를 infra, ui, blog visualization, tests support로 재배치했어요.

RSS 생성 로직은 blog 서비스로 분리하고 공개 route는 rss.xml로 드러나게 정리했어요.
Agentation overlay, route handler, dev server script를 제거했어요.

사용하지 않는 Agentation, @vercel/og, @next/mdx 의존성과 일회성 research generation script도 정리했어요.
루트 모듈 경계와 공개 route 경계를 ADR 0013에 정리했어요.

유지되지 않는 실행 계획, research dump, product/design 초안 문서를 제거하고 사람용 문서와 AI 작업 기준을 분리했어요.
@vercel

vercel Bot commented May 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
eunu-log Ready Ready Preview, Comment May 8, 2026 4:40am

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 372f2c30ce

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread blog/api/view.ts
const secChUa = readHeaderValue(headerStore, ['sec-ch-ua']);
const secChUaPlatform = readHeaderValue(headerStore, ['sec-ch-ua-platform']);
const secChUaPlatform = readHeaderValue(headerStore, [
'sec-ch-ua-infrastructure',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Read the platform client hint header

When trackView builds a fingerprint for browsers that send the standard sec-ch-ua-platform client hint, this new header name is never present, so the secChUaPlatform component is always dropped from the dedupe signature. For users sharing the same IP/user-agent/language/brand hints, this collapses otherwise distinct viewers into the same 24-hour fingerprint and can undercount post views; this should keep reading sec-ch-ua-platform.

Useful? React with 👍 / 👎.

@dev-wooyeon dev-wooyeon merged commit e710428 into master May 8, 2026
3 checks passed
@dev-wooyeon dev-wooyeon deleted the codex/move-next-app-root branch June 15, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant