docs(#918): Error 표면 준수 결정문 + 세션 인계 (#921/#922 grilling 산출)#1061
Open
thxforall wants to merge 2 commits into
Open
docs(#918): Error 표면 준수 결정문 + 세션 인계 (#921/#922 grilling 산출)#1061thxforall wants to merge 2 commits into
thxforall wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요약
#918 Error page wayfinder 맵의 결정 티켓 2개(#921 경계 정책 · #922 스타일 준수)를 grilling으로 해소하고, 결정을 서면화한 docs-only PR이다. 코드 변경 0.
결정 본문은 #921/#922 코멘트에도 전문이 있지만, 발행한 6개 구현 티켓(#1055~#1060)이 결정문 파일 경로를 참조하므로 dev에 랜딩시킨다.
감사 정정 2건
맵 #918 Notes의 원본 감사(2026-07-11)에 사실 오류가 있어 코드로 재확인해 정정했다.
shared/index.ts가 export하지만 import 0건. admin=AdminEmptyState, explore=CatalogEmptyState, profile=자체EmptyState— 3계열 병존/ko/asdf는isShellRoute=false라 레거시 SmartNav+MobileHeader가 붙음 → 현재 404가 경로에 따라 두 얼굴추가로
decoded/[id]는 실재하지 않고(index 페이지뿐),cody/looks/[slug]는(shell)밖이며, 루트app/loading.tsx가 전 라우트 폴백인데 내용은 홈 전용 스켈레톤이라는 점이 새로 드러났다.결정을 가른 Next 16.2.1 제약
(shell)/error.tsx→ AppShell 안에서 렌더(shell)/not-found.tsx→ 명시적notFound()호출만 잡음[...rest]catch-all shim이 별도 필요/asdf는[locale]트리 밖이라 shim으로도 못 잡음 → 완전 통일이 구조적으로 불가결정 (D1~D9)
#921 경계 정책
(shell)/error.tsx)/asdf는 어차피 못 잡아 통일 불가cody·search·login…) 제외 — 레거시 nav가 이미 붙고, cody는 🗺️ [wayfinder] 백그라운드 잡 UX — TryOn+StyleDNA 장기 실행 (progress·완료·복귀·간섭제약) #919/🗺️ [wayfinder] TryOn 모바일 UX/UI — 진입·모달·카메라/갤러리·아이템노출 (QA-first) #920 재편 중#922 스타일 준수
not-found·loading시맨틱 토큰 마이그레이션.--primary가 이미 그 라임이라 브랜드 포기가 아니다(라이트 대비는--primary-strong).shell.mdR4가 이미 회귀로 규정. 스코프는 2파일 + 새 경계 4개까지dynamic()6곳 한정. 청크 로드 실패는error.tsx가 못 잡음. 전역 래핑 금지loading.tsx오적용 포함발행한 슬라이스 (전부
ready-for-agent)shared/ErrorState·EmptyState보강loading.tsx오적용 수정 + 토큰#1056을 #1057보다 먼저 두는 이유: R1(루트
not-found.tsx주석에 남은 "@modal parallel route + auto not-found →React.Children.only" 회귀 이력,@modal슬롯이 양쪽 트리에 존재)이 거기서 처음 드러나며, 해소책이 필요했다면 #1057 범위가 달라진다.미검증으로 남긴 것
#1059의 전제("루트
loading.tsx가 전 라우트 전환에 홈 스켈레톤을 띄운다")는 코드 구조에서 추론한 것이고 브라우저 재현은 하지 않았다. 정적/ISR 프리렌더(#942)에선 Suspense 폴백이 기대만큼 안 뜰 수 있다. 티켓 AC에 재현 검증을 걸어 뒀고, 재현이 안 되면 폐기가 아니라 "root loading이 홈 전용인 것 자체를 중립화"로 리프레임하도록 명시했다.검증
코드 변경 0 → 빌드/테스트 해당 없음. 감사 사실은 전부 코드로 재확인했다: 라우트 트리 ·
ConditionalNav/AppShell/AdminLayout구현 ·notFound()호출처 5곳 ·shared/*import 0건 ·--primary토큰 값 ·dynamic()6곳 ·react-error-boundary미설치.Closes #921
Closes #922
Refs #918