Skip to content

Pages live 문서 검증을 정본 공개 계약과 일치시킨다 - #718

Merged
developer-1px merged 1 commit into
mainfrom
issue-717-pages-public-contract
Sep 7, 2026
Merged

Pages live 문서 검증을 정본 공개 계약과 일치시킨다#718
developer-1px merged 1 commit into
mainfrom
issue-717-pages-public-contract

Conversation

@developer-1px

Copy link
Copy Markdown
Owner

Goal Anchor

  • Outcome: 정상적으로 배포된 현재 공개 문서를 오래된 기준으로 거절하는 Pages live 검증 실패를 해결합니다.
  • Done: 문서 원천·Pages 산출물·live 응답이 정본 공개 계약을 같은 owner에서 검사하고, 현재 문서 통과/구계약·삭제된 surface 거절 회귀 및 실제 live 재검증을 갖춘 수정 PR을 엽니다.
  • Don't: Core 공개 API·wire, 제품 UI, 정상 문서 내용을 구계약에 맞춰 변경하지 않습니다. 검사를 생략하거나 재시도 증가로 실패를 숨기지 않습니다. Database saved-view와 편집 표현 책임을 정본화한다 #587·refactor(annotation): 선택·gesture·출력의 정본 경계 통합 #589 및 npm 배포·배포 설정 변경은 포함하지 않으며 merge는 별도 승인 대상으로 둡니다.

Closes #717

원인과 수정

실제 배포 /llms.txt는 HTTP 200이며 저장소 원문과 완전히 같습니다. 문서는 현재 23개 Root symbol을 설명하지만 live evaluator가 21개를 요구해 정상 배포 후 검증만 실패했습니다. 이 실패는 #715/#716 이전 main에도 존재했습니다.

  • docs/public-contract-checks.mjs가 공개 문서 내용 검사를 소유합니다. symbol 수는 기존 Core public-contract.json, package 집합은 기존 api-reference/packages.mjs에서 읽습니다.
  • docs 원천·Pages artifact·live evaluator 세 곳이 같은 검사기를 소비합니다. 수동 companion 목록 2개와 live 전용 구계약 조건을 제거했습니다.
  • malformed llms는 이제 artifact 단계에서 거절됩니다. HTTP/retry, route/meta/manifest 검증과 별도 HTTP smoke는 보존합니다.
  • owner README와 현재/구계약·등록/삭제 package·consumer 위임 회귀를 추가했습니다. README의 중복 21-symbol 설명도 정본 참조로 바꿨습니다.

Repository 내부 검증 module이며 SDK public API·site UI·새 package를 만들지 않습니다. docs/public/llms.txt, package runtime/lockfile 및 배포 workflow는 변경하지 않습니다.

완료 조건·검증

조건 증거
1. 정본 데이터 소비 Core의 23개 Root export와 문서 owner 등록 29 packages를 사용, 수동 목록 제거
2. 같은 내용 검사 docs/source·Pages artifact·live 세 evaluator의 명시적 import/호출, 소비 guard
3. 거절 의미 보존 현재 원문 통과, 21-symbol·필수 설명 누락·삭제된 package/subpath/lab 참조 거절 — 14 tests
4. 실제 실행 경로 Site 전체 tests, docs 평가, Pages build/evaluation/HTTP smoke, malformed artifact 거절과 실제 live 성공
  • npm run build: 전체 package build 통과.
  • Site 전체 25 files / 137 tests passed, 신규 회귀 14 passed.
  • docs:evaluate, workspace:check, Site typecheck·정본/스타일 guards 통과.
  • standard:check: Core 1 entrypoint / 23 exports / 6 members / 0 runtime peers 유지, 347 passed / 4 skipped.
  • site:verify:pages: build·evaluation·HTTP smoke 통과.
  • 생성된 site/dist/llms.txt만 21-symbol로 변형한 실제 site:evaluate:pages 실행은 exit 1 / site dist llms.txt is missing 23 Root symbols.로 거절했습니다. 즉시 복원했고 원천과 cmp 일치 및 evaluator 재통과를 확인했습니다.
  • 현재 배포 사이트에서 기존 코드 SITE_LIVE_ATTEMPTS=1 npm run site:evaluate:live 실패 → 수정 코드 동일 명령 live site evaluation ok.
  • git diff --check 통과. UI 변경이 없어 show-me는 non-applicable이며 Chrome profile·로그인 상태를 변경하지 않았습니다.

상태

현재 PR을 통한 수정은 아직 main에 반영하지 않았습니다. 과거 Pages 실패 run 자체가 성공으로 바뀐 것은 아니며, 수정 코드로 현재 사이트의 read-only live 검증이 통과한 결과입니다. npm 배포와 재배포 설정 변경은 하지 않았습니다.

@developer-1px

Copy link
Copy Markdown
Owner Author

최종 PR CI 통과

CI run, head 0af5b04a558e330dbf3833f512fcca2699005ad4:

  • Plan 통과.
  • Package 통과.
  • Site 통과: 문서·Pages artifact/HTTP 및 선택된 browser 검증 포함.
  • External kit은 repository planner가 이번 문서 검증 도구 변경에 불필요하다고 분류하여 skip했습니다. 실행 성공으로 계산하지 않습니다.

원래 1~4 완료 조건에 대한 로컬 검증과 현재 배포 사이트 read-only live 검사도 모두 통과했습니다. 상태는 pr-open입니다. 아직 main에 merge하거나 배포 workflow를 재실행하지 않았고, 사용자 개인 자료와 기존 미병합 PR은 보존했습니다.

@developer-1px

Copy link
Copy Markdown
Owner Author

merge 단계 시작 — 승인된 contract delta

사용자가 PR #718에 대해 “merge”를 명시적으로 승인했습니다.

원격 issue 본문/진행 기록, PR 본문·실제 diff·head·base·리뷰와 repository gate를 다시 확인했습니다. head는 0af5b04a558e330dbf3833f512fcca2699005ad4, base는 5df3b687555eb9aa9cd6a2e93db7ec02dc9c200f이고 local과 remote가 일치합니다. 작업트리는 clean입니다.

PR CI의 Plan·Package·Site는 통과했고 External kit은 planner에 따른 skip입니다. 외부 review/inline comment와 blocking review는 없으며 관리자 우회 없이 해당 head만 squash merge합니다.

@developer-1px
developer-1px merged commit 08de2da into main Sep 7, 2026
4 checks passed
@developer-1px
developer-1px deleted the issue-717-pages-public-contract branch September 7, 2026 12:02
@developer-1px

Copy link
Copy Markdown
Owner Author

merge·배포 검증·정리 완료 — landed-clean

사용자가 승인한 merge 단계를 완료했습니다.

  • PR Pages live 문서 검증을 정본 공개 계약과 일치시킨다 #718: MERGED, merge commit 08de2da6f45dd42714c89046b95e18963763a62f.
  • 이슈 Pages live 문서 검증을 정본 공개 계약과 일치시킨다 #717: CLOSED.
  • 이번 merge의 Pages workflow: build·deploy 모두 성공했고 Evaluate live site 단계도 성공했습니다. 기존의 21-symbol 오판 실패가 실제 배포 workflow에서 해소됐습니다.
  • 로컬 mainorigin/main은 위 merge commit으로 일치합니다. 최종 PR 검증 head 0af5b04a558e330dbf3833f512fcca2699005ad4와 merge 후 파일 tree가 동일하며 main의 docs 평가·build도 통과했습니다.
  • 이번 작업의 issue-717-pages-public-contract 로컬/원격 branch와 worktree만 제거했습니다. 구현은 merge commit에 보존됩니다.
  • 기존 untracked 개인 자료 4개와 다른 branch·미병합 PR은 보존했습니다. tracked/staged 변경은 없습니다.
  • npm 배포나 배포 workflow 설정 변경은 하지 않았습니다.

과거 실패 run을 재실행해 성공으로 덮어쓴 것이 아니라, 수정 commit의 새 자동 Pages run이 정상 완료된 결과입니다. PR CI의 검증 결과와 별개로 실제 Pages 배포 후 검증까지 확인했습니다.

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.

Pages live 문서 검증을 정본 공개 계약과 일치시킨다

1 participant