fix(p10-1b): apply round-1 lang.rs doc + tests/ test case missed in 4503b5b #143

altair823 · 2026-05-20T02:29:16Z

altair823 commented

2026-05-20 02:29:16 +00:00

요약

PR #142 (p10-1B) round-1 fix commit 4503b5b 보고에는 lang.rs 의 (a) module_path_for_python doc comment 갱신 (tests/ / examples/ / benches/ 가 의도적으로 strip 안 됨 명시) 과 (b) tests/test_foo.py → tests.test_foo 단언 추가가 포함됐다고 적혔으나, 실제 commit 에 lang.rs 변경이 staging 되지 않아 main 에 안 들어감.

이는 review loop round 2 의 verify 단계에서 implementer 보고만 신뢰하고 commit 내용을 직접 확인 안 한 것이 원인. 1B 머지 후 main 동기화 단계에서 working tree 에 살아남은 stale 변경을 통해 발견.

Fix

누락된 (5)+(6) 항목만 retro 적용:

module_path_for_python doc comment 4줄 추가 — stripped vs not-stripped roots 명시
module_path_for_python_strips_src_roots_and_extensions 테스트에 tests/test_foo.py → tests.test_foo 단언 1줄 + 주석 2줄

검증

cargo test -p kebab-parse-code --lib → 20/20 pass (이전: 19, +1 새 단언).
doc-only 변경이라 다른 단위 무영향.

범위

crates/kebab-parse-code/src/lang.rs 1 파일, +9 lines (test 1 + doc 4 + 주석 2 + 빈줄 2).

🤖 Generated with Claude Code

## 요약 PR #142 (p10-1B) round-1 fix commit `4503b5b` 보고에는 `lang.rs` 의 (a) `module_path_for_python` doc comment 갱신 (`tests/` / `examples/` / `benches/` 가 의도적으로 strip 안 됨 명시) 과 (b) `tests/test_foo.py` → `tests.test_foo` 단언 추가가 포함됐다고 적혔으나, **실제 commit 에 `lang.rs` 변경이 staging 되지 않아 main 에 안 들어감**. 이는 review loop round 2 의 verify 단계에서 implementer 보고만 신뢰하고 commit 내용을 직접 확인 안 한 것이 원인. 1B 머지 후 main 동기화 단계에서 working tree 에 살아남은 stale 변경을 통해 발견. ## Fix 누락된 (5)+(6) 항목만 retro 적용: - `module_path_for_python` doc comment 4줄 추가 — stripped vs not-stripped roots 명시 - `module_path_for_python_strips_src_roots_and_extensions` 테스트에 `tests/test_foo.py` → `tests.test_foo` 단언 1줄 + 주석 2줄 ## 검증 - `cargo test -p kebab-parse-code --lib` → 20/20 pass (이전: 19, +1 새 단언). - doc-only 변경이라 다른 단위 무영향. ## 범위 `crates/kebab-parse-code/src/lang.rs` 1 파일, +9 lines (test 1 + doc 4 + 주석 2 + 빈줄 2). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

altair823 added 1 commit 2026-05-20 02:29:17 +00:00

fix(p10-1b): apply round-1 lang.rs doc + tests/ test case missed in 4503b5b 23c4ad97b9

PR #142 round-1 fix commit 4503b5b 보고에는 lang.rs 의 (a) module_path_for_python
doc comment 갱신 (tests/examples/benches 가 의도적으로 strip 안 됨 명시) 과
(b) tests/test_foo.py → tests.test_foo 단언 추가가 포함됐다고 적혔으나,
실제 commit 에는 lang.rs 변경이 staging 되지 않아 main 에 안 들어감 (review
loop round 2 이 working tree 상태만 신뢰하고 commit 검증을 안 함).

이번 PR 이 누락된 (5)+(6) 항목만 retro 적용. lang.rs +9 lines (test 1 +
doc 4 + 주석 2 + 빈줄 2). cargo test -p kebab-parse-code --lib → 20/20 pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

claude-reviewer-01 approved these changes 2026-05-20 02:30:11 +00:00

claude-reviewer-01 left a comment

회차 1 — LGTM. 누락됐던 lang.rs 변경 두 가지(module_path_for_python 독스트링 + tests/test_foo.py → tests.test_foo 단언)가 정확히 들어왔고, 이 외 다른 파일은 없음. cargo test -p kebab-parse-code --lib 20/20 통과(이전 19 → 20), clippy -D warnings 이상 없음. 독스트링이 구현(src/, lib/, crates/<crate>/src/ 제거; tests/, examples/, benches/ 유지)과 일치하며, 인라인 코멘트도 의도를 명확히 설명함. 깔끔한 레트로픽스.

회차 1 — LGTM. 누락됐던 `lang.rs` 변경 두 가지(`module_path_for_python` 독스트링 + `tests/test_foo.py` → `tests.test_foo` 단언)가 정확히 들어왔고, 이 외 다른 파일은 없음. `cargo test -p kebab-parse-code --lib` 20/20 통과(이전 19 → 20), `clippy -D warnings` 이상 없음. 독스트링이 구현(`src/`, `lib/`, `crates/<crate>/src/` 제거; `tests/`, `examples/`, `benches/` 유지)과 일치하며, 인라인 코멘트도 의도를 명확히 설명함. 깔끔한 레트로픽스.

altair823 merged commit 68ada396f3 into main

2026-05-20 02:31:16 +00:00

altair823 deleted branch fix/p10-1b-lang-doc-test-staging-miss

2026-05-20 02:31:17 +00:00

altair823 referenced this issue from a commit

2026-05-20 02:31:17 +00:00

Merge pull request 'fix(p10-1b): apply round-1 lang.rs doc + tests/ test case missed in 4503b5b' (#143) from fix/p10-1b-lang-doc-test-staging-miss into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: altair823-org/kebab#143

fix(p10-1b): apply round-1 lang.rs doc + tests/ test case missed in 4503b5b #143

요약

Fix

검증

범위

fix(p10-1b): apply round-1 lang.rs doc + tests/ test case missed in `4503b5b` #143