docs: README 와 코드가 어긋난 곳을 맞춘다 - #2
Merged
Merged
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe changes update repository ignores and documentation. The README now uses the Notion MCP example and documents two validation commands. Diagram skill guidance now defines SaveFile handling and browser-rendering disclosure. ChangesRepository guidance
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
요약
README 와 코드를 대조해 어긋난 서술 두 건과, 규칙을 지키지 않던 스킬 하나를 고쳤다.
빠진 문서보다 틀린 문서가 먼저다 — 셋 다 읽는 사람을 잘못된 곳으로 보내고 있었다.
변경
fix: declare diagram-design's SaveFile fallback in compatibilityREADME 는 "빌트인에 기대는 스킬은
compatibility프론트매터로 밝힌다"고 규정한다.diagram-design만 그 서술이 본문(SKILL.md:71-72)에만 있어 프론트매터가 비어 있었다.같은 계약을 쓰는 나머지 넷(
html-report·html-wireframe·html-prototype·html-explainer)과자리를 맞췄다. 문구는 정적 산출물이라는 점이 같은
html-wireframe과 동일하다.docs: fix the extension-doc example and document the validator's testsdocument를 한 줄 description 으로 보여 주는데, 실제document.md는>폴딩을 쓴다. 게다가 바로 다음 줄이document를 폴딩 사례로지목해 예시와 설명이 서로를 부정하고 있었다. 예시를 실제로 한 줄인
notion으로 바꿨다.scripts/test_validate.py와 CI 의 unittest 단계가 README 어디에도 없었다."Validate before merging" 절에 실행 명령과 이 테스트가 무엇을 고정하는지를 넣고,
마지막 문장을
CI runs both로 고쳤다.chore: stop tracking compiled Python bytecodescripts/__pycache__/validate.cpython-312.pyc가 저장소에 커밋돼 있었고.gitignore에는
__pycache__/항목이 없었다. 둘 다 정리했다.Breaking Changes
없다.
diagram-design의compatibility는 본문이 이미 규정하던 동작을 프론트매터로옮긴 것이라 런타임 동작은 그대로다.
검증
python3 scripts/validate.py— 7 plugins, 24 skills 통과. 경고 8건은 기존private-HTTP 배포 예외로 이 PR 과 무관하다.
python3 -m unittest discover -s scripts -p 'test_*.py'— 14 tests OK.compatibility는 스펙 상한 500자 안이다.Summary by CodeRabbit
Documentation
Chores