Skip to content

Commit 1f22061

Browse files
kangcheolungclaude
andcommitted
docs: CodeRabbit 리뷰로 발견된 문서 부정확 항목 정정
update-design-doc 커맨드에 $ARGUMENTS는 #없이 숫자만 전달해야 함을 명시(패턴에 이미 #이 포함돼 있어 #21처럼 넘기면 매칭 실패). #21 문서의 public 메서드 개수 표기를 오버로드 포함 실제 시그니처 개수(9개)에 맞게 정정. #29 문서의 getMyCollections 테스트 관련 설명이 실제로는 존재하지 않는 테스트를 언급하고 있어 바로잡고, 테스트 커버리지 공백으로 기록한다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 27e4f74 commit 1f22061

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

.claude/commands/update-design-doc.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# 설계 문서 동기화 커맨드
22

3-
이슈 번호($ARGUMENTS)를 받아, 코드 리팩토링/수정 이후 낡아진 `docs/design/` 설계 문서를 실제 코드에 맞게 갱신합니다.
3+
이슈 번호($ARGUMENTS)를 받아, 코드 리팩토링/수정 이후 낡아진 `docs/design/` 설계 문서를 실제 코드에 맞게 갱신합니다. `$ARGUMENTS``#` 없이 숫자만 전달한다(예: `21`) — 아래 패턴 자체에 `#`이 포함돼 있어 `#21`처럼 넘기면 `##21`이 되어 매칭에 실패한다.
44

55
## 절차
66

docs/design/kangcheolung-#21-permission-query-service.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ DEPT 권한 → 매 요청마다 live 조회 (JOIN 여러 번)
3434

3535
### `domain/permission/service/query/PermissionQueryService.java`
3636

37-
이 이슈의 핵심 파일이며, 6개의 public 메서드를 제공한다(`canReadCollection`은 이후 리팩토링에서 추가됨 — 아래 참고). 컬렉션 대상 3종(`canReadCollection`/`canWriteCollection`/`canAdminCollection`)은 각각 ID로 조회하는 버전과, 이미 조회된 `DocumentCollection` 엔티티를 받는 버전 2개씩 오버로드로 제공한다 — 호출부가 이미 엔티티를 들고 있으면 중복 조회 없이 엔티티 버전을 바로 쓸 수 있다.
37+
이 이슈의 핵심 파일이며, 6개의 권한 판정 그룹(`canReadDocument`/`canWriteDocument`/`canAdminDocument`/`canReadCollection`/`canWriteCollection`/`canAdminCollection`, `canReadCollection`은 이후 리팩토링에서 추가됨 — 아래 참고)을 제공한다. 컬렉션 대상 3종은 각각 ID로 조회하는 버전과, 이미 조회된 `DocumentCollection` 엔티티를 받는 버전 2개씩 오버로드로 제공해서 — 문서 3종(오버로드 없음) + 컬렉션 3종(오버로드 2개씩)으로 공개 메서드 시그니처는 총 9개다. 호출부가 이미 엔티티를 들고 있으면 중복 조회 없이 엔티티 버전을 바로 쓸 수 있다.
3838

3939
#### `canReadDocument` (5단계)
4040

@@ -172,7 +172,7 @@ if (cacheRepository.existsValidReadCache(...)) { ... }
172172
$ ./gradlew test --tests "*PermissionQueryServiceTest*"
173173
BUILD SUCCESSFUL
174174
```
175-
`PermissionQueryServiceTest` 43개 모두 통과(현재 기준 재검증) — 이 서비스의 메서드 6개(`canReadDocument`, `canWriteDocument`, `canAdminDocument`, `canReadCollection`, `canWriteCollection`, `canAdminCollection`) 각각의 단계별 분기를 검증하는 테스트가 다수 포함되어 있다.
175+
`PermissionQueryServiceTest` 44개 모두 통과(현재 기준 재검증) — 이 서비스의 권한 판정 그룹 6개(`canReadDocument`, `canWriteDocument`, `canAdminDocument`, `canReadCollection`, `canWriteCollection`, `canAdminCollection`, 오버로드 포함 공개 메서드 시그니처 9개) 각각의 단계별 분기를 검증하는 테스트가 다수 포함되어 있다.
176176

177177
---
178178

docs/design/kangcheolung-#29-collection-management.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ GET /collections
211211
$ ./gradlew test --tests "*CollectionCommandServiceTest*" --tests "*CollectionQueryServiceTest*"
212212
BUILD SUCCESSFUL
213213
```
214-
`CollectionCommandServiceTest`(총 15개)에 `deleteCollection`/`removeDocument` 관련 케이스가 `#16``createCollection`/`addDocument` 케이스와 함께 묶여 있다(클래스 자체는 `#16`에서 이미 생성, 이 이슈에서 케이스만 추가). `CollectionQueryServiceTest`(총 4개, `#16` 문서 참고 — `canReadCollection` 도입으로 케이스 추가됨)`getMyCollections` 케이스가 `getCollection`과 함께 있다.
214+
`CollectionCommandServiceTest`(총 15개)에 `deleteCollection`/`removeDocument` 관련 케이스가 `#16``createCollection`/`addDocument` 케이스와 함께 묶여 있다(클래스 자체는 `#16`에서 이미 생성, 이 이슈에서 케이스만 추가). `CollectionQueryServiceTest`(총 4개, `#16` 문서 참고 — `canReadCollection` 도입으로 케이스 추가됨). 실제로는 4개 전부 `getCollection` 케이스이고 `getMyCollections`에 대한 단위 테스트는 없다 — 코드리뷰로 발견된 기존 테스트 커버리지 공백이며, 별도로 추가가 필요하다.
215215

216216
---
217217

0 commit comments

Comments
 (0)