refactor(architecture): 루트 모듈 경계 정리#104
Conversation
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 작업 기준을 분리했어요.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 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".
| const secChUa = readHeaderValue(headerStore, ['sec-ch-ua']); | ||
| const secChUaPlatform = readHeaderValue(headerStore, ['sec-ch-ua-platform']); | ||
| const secChUaPlatform = readHeaderValue(headerStore, [ | ||
| 'sec-ch-ua-infrastructure', |
There was a problem hiding this comment.
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 👍 / 👎.
변경 내용
src/app을 루트approute adapter로 이동platform을infra로 정리하고shared를ui,blog/ui/visualization,tests/support로 분리/rss.xml로 드러내고 생성 로직을blog/services/rss-feed.ts로 이동의도
영향 범위
app/**로 변경됨/rss.xml로 변경되고 기존/feed.xmlroute는 제거됨/api/og)는 유지하되 사용하지 않는@vercel/og의존성은 제거됨검증
npm run lintnpm run lint:css:syntaxnpm run verify:docsnpm run test:unitnpm run test:componentsnpm run build체크리스트
codex/<task>)에서 진행함<type>: <한국어 설명>) 준수