문서 역할 재배치와 주석·테스트 규칙 보강 - #11
Merged
Merged
Conversation
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (41)
📝 WalkthroughSummary by CodeRabbit
Walkthrough이 PR은 에이전트 가이드 문서(AGENTS.md, CLAUDE.md, docs/*)를 재구성하고 각 모듈에 README를 추가한다. 또한 여러 Swift 소스 파일의 주석과 다수 테스트 파일의 메서드명을 영어에서 한국어로 변경한다. 테스트 로직과 실행 동작은 변경되지 않는다. Changes에이전트 문서 및 모듈 README 재구성
주석 및 테스트 메서드명 한국어화
Estimated code review effort: 2 (Simple) | ~15 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 5
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CLAUDE.md`:
- Line 1: CLAUDE.md의 첫 제목을 파일 역할에 맞게 변경하세요. 현재 # AGENTS를 # CLAUDE 또는 # Mozi
Agent Entry 중 하나로 교체해 별도 진입 문서임을 명확히 하세요.
In `@docs/AGENT_WORKFLOW.md`:
- Line 72: Update the four Markdown code fences in the workflow documentation at
the referenced sections to use the text language tag on each opening fence,
resolving MD040 while preserving the existing plain-text flowchart contents.
In `@docs/CONVENTIONS.md`:
- Line 6: docs/CONVENTIONS.md의 문서 섹션 제목을 한국어로 변경하세요. `Coding`, `Comments`,
`Tests`, `Schemes`, `Git Format`, `Language` 제목만 각각 `코딩`, `주석`, `테스트`, `스킴`,
`Git 형식`, `언어`로 바꾸고, API 이름·옵션명·식별자 및 기타 내용은 그대로 유지하세요.
In `@Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift`:
- Line 10: 테스트 이름과 주석의 영어 설명어를 한국어로 변경하고 HTTP, URL, Bearer, 상태 코드는 유지하세요.
Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift 10-10의
authorization을 권한으로, 39-39의 refresh와 재refresh를 토큰_갱신으로, 63-63 주석의 generation을
세대로 변경하세요. Projects/Core/Network/Tests/DefaultNetworkClientTests.swift 41-41의
void를 반환값_없는 표현으로, 53-53의 decodingFailed를 디코딩_실패로, 99-99의 refresh를 토큰_갱신으로,
138-138의 refresh_single_flight를 토큰_갱신_단일_실행으로, 169-169의 single-flight refresh를
단일 갱신으로, 188-188의 refresh와 unauthorized를 한국어 상황 및 기대 결과로, 219-219의 unauthorized를
인증_실패로 변경하세요. Projects/Core/Network/Tests/NetworkLogTests.swift 5-5의 query와
fragment를 쿼리와 프래그먼트로, Projects/Core/Network/Tests/URLRequestBuildingTests.swift
26-26의 accessToken과 refresh를 액세스_토큰과 갱신으로 변경하고, 모든 테스트 이름은 test_<상황>_<기대결과> 형식을
유지하세요.
In `@Projects/Shared/Util/Tests/Concurrency/LockedTests.swift`:
- Line 5: 테스트 이름이 동작뿐 아니라 상황과 기대 결과를 함께 나타내도록 수정하세요. LockedTests.swift의
test_withLock에서_상태_변경은 상태 변경 결과를, ArraySharedUtilsTests.swift의 두 테스트는 각각 첫 요소
유지와 순서 유지 결과를, CollectionSharedUtilsTests.swift의 테스트는 기대 청크 배열과의 일치를,
StringSharedUtilsTests.swift의 두 테스트는 각각 isBlank == true와 검색 결과 일치를 이름에 명시하세요.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 5d82afe6-c0c4-44f4-8bd8-a9950486608f
📒 Files selected for processing (41)
AGENTS.mdCLAUDE.mdCLAUDE.mdProjects/App/Sources/DI/Dependencies.swiftProjects/Core/Network/README.mdProjects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swiftProjects/Core/Network/Tests/DefaultNetworkClientTests.swiftProjects/Core/Network/Tests/NetworkErrorMappingTests.swiftProjects/Core/Network/Tests/NetworkLogTests.swiftProjects/Core/Network/Tests/URLRequestBuildingTests.swiftProjects/Data/README.mdProjects/Domain/README.mdProjects/Feature/README.mdProjects/Feature/Sources/AppCoordinator/AppCoordinatorFeature.swiftProjects/Feature/Tests/AppCoordinator/AppCoordinatorFeatureTests.swiftProjects/Feature/Tests/AppCoordinator/DeepLinkRouterTests.swiftProjects/Shared/DesignSystem/Sources/Components/Button/DesignButton.swiftProjects/Shared/DesignSystem/Sources/Components/Button/DesignButtonStyle.swiftProjects/Shared/DesignSystem/Sources/Components/Social/SocialLoginButton.swiftProjects/Shared/DesignSystem/Sources/Foundation/FontRegistration.swiftProjects/Shared/DesignSystem/Sources/Preview/DesignSystemGallery.swiftProjects/Shared/DesignSystem/Sources/Tokens/Semantic/SemanticColor.swiftProjects/Shared/DesignSystem/Tests/Components/DesignButtonMetricsTests.swiftProjects/Shared/DesignSystem/Tests/Components/SocialLoginProviderTests.swiftProjects/Shared/DesignSystem/Tests/Foundation/FontRegistrationTests.swiftProjects/Shared/DesignSystem/Tests/Smoke/SharedDesignSystemSmokeTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/PrimitiveColorTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/SemanticColorTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/SemanticNumberTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/TypographyTests.swiftProjects/Shared/Util/Tests/Bundle/AppInfoTests.swiftProjects/Shared/Util/Tests/Concurrency/LockedTests.swiftProjects/Shared/Util/Tests/Foundation/ArraySharedUtilsTests.swiftProjects/Shared/Util/Tests/Foundation/CollectionSharedUtilsTests.swiftProjects/Shared/Util/Tests/Foundation/StringSharedUtilsTests.swiftProjects/Shared/Util/Tests/Smoke/SharedUtilsSmokeTests.swiftProjects/ThirdParty/ThirdPartyCore/Sources/Exports.swiftdocs/AGENT_WORKFLOW.mddocs/ARCHITECTURE.mddocs/CONVENTIONS.mddocs/skills.md
There was a problem hiding this comment.
Caution
Inline review comments failed to post. This is likely due to GitHub's internal server error or limits when posting large numbers of comments. If you are seeing this consistently it is likely a permissions issue. Please check "Moderation" -> "Code review limits" under your organization settings.
Actionable comments posted: 5
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CLAUDE.md`:
- Line 1: CLAUDE.md의 첫 제목을 파일 역할에 맞게 변경하세요. 현재 # AGENTS를 # CLAUDE 또는 # Mozi
Agent Entry 중 하나로 교체해 별도 진입 문서임을 명확히 하세요.
In `@docs/AGENT_WORKFLOW.md`:
- Line 72: Update the four Markdown code fences in the workflow documentation at
the referenced sections to use the text language tag on each opening fence,
resolving MD040 while preserving the existing plain-text flowchart contents.
In `@docs/CONVENTIONS.md`:
- Line 6: docs/CONVENTIONS.md의 문서 섹션 제목을 한국어로 변경하세요. `Coding`, `Comments`,
`Tests`, `Schemes`, `Git Format`, `Language` 제목만 각각 `코딩`, `주석`, `테스트`, `스킴`,
`Git 형식`, `언어`로 바꾸고, API 이름·옵션명·식별자 및 기타 내용은 그대로 유지하세요.
In `@Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift`:
- Line 10: 테스트 이름과 주석의 영어 설명어를 한국어로 변경하고 HTTP, URL, Bearer, 상태 코드는 유지하세요.
Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift 10-10의
authorization을 권한으로, 39-39의 refresh와 재refresh를 토큰_갱신으로, 63-63 주석의 generation을
세대로 변경하세요. Projects/Core/Network/Tests/DefaultNetworkClientTests.swift 41-41의
void를 반환값_없는 표현으로, 53-53의 decodingFailed를 디코딩_실패로, 99-99의 refresh를 토큰_갱신으로,
138-138의 refresh_single_flight를 토큰_갱신_단일_실행으로, 169-169의 single-flight refresh를
단일 갱신으로, 188-188의 refresh와 unauthorized를 한국어 상황 및 기대 결과로, 219-219의 unauthorized를
인증_실패로 변경하세요. Projects/Core/Network/Tests/NetworkLogTests.swift 5-5의 query와
fragment를 쿼리와 프래그먼트로, Projects/Core/Network/Tests/URLRequestBuildingTests.swift
26-26의 accessToken과 refresh를 액세스_토큰과 갱신으로 변경하고, 모든 테스트 이름은 test_<상황>_<기대결과> 형식을
유지하세요.
In `@Projects/Shared/Util/Tests/Concurrency/LockedTests.swift`:
- Line 5: 테스트 이름이 동작뿐 아니라 상황과 기대 결과를 함께 나타내도록 수정하세요. LockedTests.swift의
test_withLock에서_상태_변경은 상태 변경 결과를, ArraySharedUtilsTests.swift의 두 테스트는 각각 첫 요소
유지와 순서 유지 결과를, CollectionSharedUtilsTests.swift의 테스트는 기대 청크 배열과의 일치를,
StringSharedUtilsTests.swift의 두 테스트는 각각 isBlank == true와 검색 결과 일치를 이름에 명시하세요.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 5d82afe6-c0c4-44f4-8bd8-a9950486608f
📒 Files selected for processing (41)
AGENTS.mdCLAUDE.mdCLAUDE.mdProjects/App/Sources/DI/Dependencies.swiftProjects/Core/Network/README.mdProjects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swiftProjects/Core/Network/Tests/DefaultNetworkClientTests.swiftProjects/Core/Network/Tests/NetworkErrorMappingTests.swiftProjects/Core/Network/Tests/NetworkLogTests.swiftProjects/Core/Network/Tests/URLRequestBuildingTests.swiftProjects/Data/README.mdProjects/Domain/README.mdProjects/Feature/README.mdProjects/Feature/Sources/AppCoordinator/AppCoordinatorFeature.swiftProjects/Feature/Tests/AppCoordinator/AppCoordinatorFeatureTests.swiftProjects/Feature/Tests/AppCoordinator/DeepLinkRouterTests.swiftProjects/Shared/DesignSystem/Sources/Components/Button/DesignButton.swiftProjects/Shared/DesignSystem/Sources/Components/Button/DesignButtonStyle.swiftProjects/Shared/DesignSystem/Sources/Components/Social/SocialLoginButton.swiftProjects/Shared/DesignSystem/Sources/Foundation/FontRegistration.swiftProjects/Shared/DesignSystem/Sources/Preview/DesignSystemGallery.swiftProjects/Shared/DesignSystem/Sources/Tokens/Semantic/SemanticColor.swiftProjects/Shared/DesignSystem/Tests/Components/DesignButtonMetricsTests.swiftProjects/Shared/DesignSystem/Tests/Components/SocialLoginProviderTests.swiftProjects/Shared/DesignSystem/Tests/Foundation/FontRegistrationTests.swiftProjects/Shared/DesignSystem/Tests/Smoke/SharedDesignSystemSmokeTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/PrimitiveColorTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/SemanticColorTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/SemanticNumberTests.swiftProjects/Shared/DesignSystem/Tests/Tokens/TypographyTests.swiftProjects/Shared/Util/Tests/Bundle/AppInfoTests.swiftProjects/Shared/Util/Tests/Concurrency/LockedTests.swiftProjects/Shared/Util/Tests/Foundation/ArraySharedUtilsTests.swiftProjects/Shared/Util/Tests/Foundation/CollectionSharedUtilsTests.swiftProjects/Shared/Util/Tests/Foundation/StringSharedUtilsTests.swiftProjects/Shared/Util/Tests/Smoke/SharedUtilsSmokeTests.swiftProjects/ThirdParty/ThirdPartyCore/Sources/Exports.swiftdocs/AGENT_WORKFLOW.mddocs/ARCHITECTURE.mddocs/CONVENTIONS.mddocs/skills.md
🛑 Comments failed to post (5)
CLAUDE.md (1)
1-1: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
CLAUDE.md의 제목을 파일 역할과 일치시키세요.Line 1은 새 진입 문서인
CLAUDE.md를# AGENTS로 표시합니다. 문서 참조 시 두 진입점의 식별이 모호해집니다.# CLAUDE또는# Mozi Agent Entry로 변경하세요.As per PR objectives,
CLAUDE.md는 별도 진입 문서입니다.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@CLAUDE.md` at line 1, CLAUDE.md의 첫 제목을 파일 역할에 맞게 변경하세요. 현재 # AGENTS를 # CLAUDE 또는 # Mozi Agent Entry 중 하나로 교체해 별도 진입 문서임을 명확히 하세요.docs/AGENT_WORKFLOW.md (1)
72-72: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Markdown 코드 블록에 언어 태그를 추가하세요.
정적 분석이 Lines 72, 84, 92, 103에서 MD040 경고를 보고합니다. 모두 일반 텍스트 흐름도이므로 각 시작 fence를 ````text`로 변경하세요.
수정 예시
-``` +```textAlso applies to: 84-84, 92-92, 103-103
🧰 Tools
🪛 markdownlint-cli2 (0.23.1)
[warning] 72-72: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/AGENT_WORKFLOW.md` at line 72, Update the four Markdown code fences in the workflow documentation at the referenced sections to use the text language tag on each opening fence, resolving MD040 while preserving the existing plain-text flowchart contents.Source: Linters/SAST tools
docs/CONVENTIONS.md (1)
6-6: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
문서 섹션 제목을 한국어로 통일하세요.
Lines 6, 21, 49, 57, 64, 91은
Coding,Comments,Tests,Schemes,Git Format,Language를 사용합니다. 로컬 문서는 한국어를 기본으로 작성해야 합니다. API 이름, 옵션명, 식별자는 영어로 유지하되 설명용 제목은코딩,주석,테스트,스킴,Git 형식,언어로 변경하세요.As per coding guidelines, 로컬 문서와 에이전트 문서는 한국어를 기본으로 작성해야 합니다.
As per path instructions, docs 문서의 로컬 설명은 한국어를 사용해야 합니다.Also applies to: 21-21, 49-49, 57-57, 64-64, 91-91
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/CONVENTIONS.md` at line 6, docs/CONVENTIONS.md의 문서 섹션 제목을 한국어로 변경하세요. `Coding`, `Comments`, `Tests`, `Schemes`, `Git Format`, `Language` 제목만 각각 `코딩`, `주석`, `테스트`, `스킴`, `Git 형식`, `언어`로 바꾸고, API 이름·옵션명·식별자 및 기타 내용은 그대로 유지하세요.Sources: Coding guidelines, Path instructions
Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift (1)
10-10: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
테스트명과 주석의 한국어화를 완료하세요.
일부 설명어가 영어로 남아 있습니다. 표준 기술 표기인
HTTP,URL,Bearer, 상태 코드는 유지하고, 설명어는 한국어로 변경하세요.
Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift#L10-L10:authorization를권한으로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift#L39-L39:refresh,재refresh를토큰_갱신표현으로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift#L63-L63: 주석의generation을세대로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L41-L41:void를반환값_없는과 같은 한국어 표현으로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L53-L53:decodingFailed를디코딩_실패로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L99-L99:refresh를토큰_갱신으로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L138-L138:refresh_single_flight를토큰_갱신_단일_실행과 같은 표현으로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L169-L169:single-flight refresh를단일 갱신으로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L188-L188:refresh,unauthorized를 한국어 상황과 기대 결과로 변경하세요.Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L219-L219:unauthorized를인증_실패와 같은 한국어 표현으로 변경하세요.Projects/Core/Network/Tests/NetworkLogTests.swift#L5-L5:query,fragment를쿼리,프래그먼트로 변경하세요.Projects/Core/Network/Tests/URLRequestBuildingTests.swift#L26-L26:accessToken,refresh를액세스_토큰,갱신으로 변경하세요.As per coding guidelines, 테스트 이름은 한국어
test_<상황>_<기대결과>형식을 사용해야 합니다. As per path instructions, 소스 주석은 한국어를 기본으로 작성해야 합니다.📍 Affects 4 files
Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift#L10-L10(this comment)Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift#L39-L39Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift#L63-L63Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L41-L41Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L53-L53Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L99-L99Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L138-L138Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L169-L169Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L188-L188Projects/Core/Network/Tests/DefaultNetworkClientTests.swift#L219-L219Projects/Core/Network/Tests/NetworkLogTests.swift#L5-L5Projects/Core/Network/Tests/URLRequestBuildingTests.swift#L26-L26🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift` at line 10, 테스트 이름과 주석의 영어 설명어를 한국어로 변경하고 HTTP, URL, Bearer, 상태 코드는 유지하세요. Projects/Core/Network/Tests/DefaultNetworkClientReviewFixTests.swift 10-10의 authorization을 권한으로, 39-39의 refresh와 재refresh를 토큰_갱신으로, 63-63 주석의 generation을 세대로 변경하세요. Projects/Core/Network/Tests/DefaultNetworkClientTests.swift 41-41의 void를 반환값_없는 표현으로, 53-53의 decodingFailed를 디코딩_실패로, 99-99의 refresh를 토큰_갱신으로, 138-138의 refresh_single_flight를 토큰_갱신_단일_실행으로, 169-169의 single-flight refresh를 단일 갱신으로, 188-188의 refresh와 unauthorized를 한국어 상황 및 기대 결과로, 219-219의 unauthorized를 인증_실패로 변경하세요. Projects/Core/Network/Tests/NetworkLogTests.swift 5-5의 query와 fragment를 쿼리와 프래그먼트로, Projects/Core/Network/Tests/URLRequestBuildingTests.swift 26-26의 accessToken과 refresh를 액세스_토큰과 갱신으로 변경하고, 모든 테스트 이름은 test_<상황>_<기대결과> 형식을 유지하세요.Sources: Coding guidelines, Path instructions
Projects/Shared/Util/Tests/Concurrency/LockedTests.swift (1)
5-5: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
테스트 이름에 상황과 기대 결과를 모두 포함하세요.
다음 테스트 이름은 동작만 설명하거나 결과가 모호합니다.
Projects/Shared/Util/Tests/Concurrency/LockedTests.swift#L5-L5: 상태 변경 결과를 명시하세요.Projects/Shared/Util/Tests/Foundation/ArraySharedUtilsTests.swift#L16-L16: 첫 요소 유지 결과를 명시하세요.Projects/Shared/Util/Tests/Foundation/ArraySharedUtilsTests.swift#L28-L28: 순서 유지 결과를 명시하세요.Projects/Shared/Util/Tests/Foundation/CollectionSharedUtilsTests.swift#L16-L16: 기대 청크 배열과의 일치를 명시하세요.Projects/Shared/Util/Tests/Foundation/StringSharedUtilsTests.swift#L9-L9:isBlank == true결과를 명시하세요.Projects/Shared/Util/Tests/Foundation/StringSharedUtilsTests.swift#L24-L24: 검색 결과 일치를 명시하세요.📍 Affects 4 files
Projects/Shared/Util/Tests/Concurrency/LockedTests.swift#L5-L5(this comment)Projects/Shared/Util/Tests/Foundation/ArraySharedUtilsTests.swift#L16-L16Projects/Shared/Util/Tests/Foundation/ArraySharedUtilsTests.swift#L28-L28Projects/Shared/Util/Tests/Foundation/CollectionSharedUtilsTests.swift#L16-L16Projects/Shared/Util/Tests/Foundation/StringSharedUtilsTests.swift#L9-L9Projects/Shared/Util/Tests/Foundation/StringSharedUtilsTests.swift#L24-L24🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@Projects/Shared/Util/Tests/Concurrency/LockedTests.swift` at line 5, 테스트 이름이 동작뿐 아니라 상황과 기대 결과를 함께 나타내도록 수정하세요. LockedTests.swift의 test_withLock에서_상태_변경은 상태 변경 결과를, ArraySharedUtilsTests.swift의 두 테스트는 각각 첫 요소 유지와 순서 유지 결과를, CollectionSharedUtilsTests.swift의 테스트는 기대 청크 배열과의 일치를, StringSharedUtilsTests.swift의 두 테스트는 각각 isBlank == true와 검색 결과 일치를 이름에 명시하세요.Sources: Coding guidelines, Path instructions
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
1 participant
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.
📌 변경 요약
📌 변경 내용
Docs
ARCHITECTURE전역 지도 축약 및 모듈 README 링크 추가CONVENTIONS에 주석/테스트 네이밍/형식 규칙 정리AGENT_WORKFLOW에 hard gate와 Agent Behavior 정리skills인덱스 유지 및 중복 규칙 축소AGENTS/CLAUDE진입점 요약으로 축소Module README
Tests
test_<상황>_<기대결과>한국어 통일Style
///////TODO 역할 정렬