후속: 모듈 컨트랙트 트리거 정밀화와 조건부 읽기 가이드#2
Merged
gkrtjd99 merged 2 commits intoMay 14, 2026
Merged
Conversation
- 모듈 컨트랙트 트리거를 좁혀 단일 파일·일회성 모듈은 생성하지 않도록 명시. - code-organization-contract.md에 Authority 섹션을 추가해 module contract가 source-of-truth, code-map은 derived index임을 명시. - starter-kit/CLAUDE.md에 Reference Map과 코드맵·컨트랙트 우선 참조 규칙을 추가. - development-rules.md에 Read Order Usage 섹션을 추가해 code-map과 module contract가 코드 변경 시점에만 필수임을 명시. - skill/ 변경을 sync-skill-targets.sh로 5개 runtime bundle에 동기화.
- PR #1의 db930ea (tests/ 행 평문 복원) 를 베이스에서 이어받음.
gkrtjd99
commented
May 14, 2026
Owner
Author
gkrtjd99
left a comment
There was a problem hiding this comment.
검토했습니다.
확인한 내용:
- 모듈 컨트랙트 생성 조건을 durable/multi-file surface 중심으로 좁힌 변경이
README.md,skill/SKILL.md,skill/references/templates.md, runtime target bundle에 일관되게 반영되어 있습니다. development-rules.md의 Read Order Usage 추가가 AGENTS read order를 실제 작업 맥락에 맞게 해석하도록 보완합니다.skill/변경 후bash scripts/sync-skill-targets.sh를 실행해도 worktree가 깨끗하게 유지되어 target bundle 동기화가 맞습니다.
로컬 검증:
bash -n대상 스크립트 전체 통과- root/starter-kit Markdown 링크 검사 통과
- root/starter-kit design-doc lint 통과
bash scripts/check-agents-doc.sh통과bash scripts/check-bundle-structure.sh통과
Merged
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.
변경 목적
PR #1(코드 조직 계약과 모듈 문서 구조 추가)의 리뷰 후속 항목 4–7번을 별도 PR로 처리합니다. PR #1 위에 쌓는 stacked PR이며, PR #1이 머지된 후 자동으로 main 대비 diff가 깔끔해집니다.
다루는 리뷰 항목
skill/SKILL.md와references/templates.md에 "단일 파일·일회성 모듈은 생성하지 않음" 음(negative) 조건 추가,README.md표 문구도 같은 의미로 갱신code-map↔module-contracts권위 명시docs/design-docs/code-organization-contract.md에 Authority 섹션 추가, contract가 source-of-truth, code-map은 derived index임을 규정starter-kit/CLAUDE.md포인터 보강development-rules.md에 Read Order Usage 섹션 추가, 코드 변경 시점에만 code-map/module-contracts 필독, 문서·설정·소규모 버그픽스에는 생략 가능주요 변경
skill/SKILL.md: module contract 생성 규칙에 음 조건 추가.skill/references/templates.md: Optional Docs 표 module-contracts 행 좁히기, development-rules.md 템플릿에 Read Order Usage 섹션 추가.docs/design-docs/code-organization-contract.md: Authority 섹션 추가.docs/references/development-rules.md,starter-kit/docs/references/development-rules.md: Read Order Usage 섹션 추가.starter-kit/CLAUDE.md: Reference Map 및 사용 규칙 추가.README.md: module-contracts 선택 문서 생성 조건 한국어 표현 좁히기.targets/*/harness-init/:bash scripts/sync-skill-targets.sh로 5개 runtime bundle 동기화.검증
bash scripts/sync-skill-targets.sh bash starter-kit/scripts/check-doc-links.sh . bash starter-kit/scripts/check-doc-links.sh starter-kit bash starter-kit/scripts/lint-architecture.sh docs/design-docs bash starter-kit/scripts/lint-architecture.sh starter-kit/docs/design-docs bash scripts/check-bundle-structure.sh bash scripts/check-agents-doc.sh모두 PASS.
관련 PR
codex/code-organization-contracts). 이 PR의 1·2·3번은 PR #1에 직접 푸시했고, 4–7번은 본 후속 PR에서 다룹니다.