[ORG-004] ComposableArchitecture 외부모듈 추가

[devJungwonLee]

✨️ 작업 내용

• ComposableArchitecture, Dependencies 외부 모듈 추가 (1.25.5 버전)
  • Presentation -> ComposableArchitecture
  • Domain -> Dependencies

💭 변경 사항

• Makefile 수정 (명령어 console 출력 hide)
• AccentColor를 추가하여 build시 생성되는 warning 제거

Summary by CodeRabbit

릴리스 노트

• New Features

  • 앱의 새로운 강조색(Accent Color) 디자인 에셋이 추가되었습니다.

• Chores

  • 빌드 시스템 및 프로젝트 종속성이 업데이트되었습니다.

[coderabbitai]

전체 요약

빌드 설정 파일의 명령어 출력을 억제하도록 Makefile을 수정하고, AccentColor 애셋을 추가했습니다. Tuist 패키지 관리를 확장하여 swift-composable-architecture 및 관련 의존성을 추가하고, 모듈 의존성 매핑을 업데이트했습니다.

변경 사항

Cohort / File(s)	요약
빌드 설정 명령어 억제 Makefile	generate, clean, reset 대상의 명령어 앞에 @ 접두사를 추가하여 명령어 에코 출력 억제
리소스 애셋 추가 Projects/App/Resources/Assets.xcassets/AccentColor.colorset/Contents.json	AccentColor.colorset에 대한 새로운 색상 애셋 메타데이터 파일 추가
의존성 핀 업데이트 Tuist/Package.resolved	Alamofire를 5.11.2로 업그레이드하고, swift-composable-architecture, swift-dependencies 등 다수의 새로운 의존성을 핀 목록에 추가
패키지 의존성 선언 Tuist/Package.swift	swift-composable-architecture(v1.25.5 이상) 외부 의존성을 패키지 매니페스트에 추가
모듈 의존성 매핑 Tuist/ProjectDescriptionHelpers/Dependency.swift, Tuist/ProjectDescriptionHelpers/Module/ExternalModule.swift	.presentation 모듈에 .composableArchitecture 의존성, .domain 모듈에 .dependencies 의존성을 추가하고, ExternalModule 열거형에 해당 경우를 선언

예상 코드 리뷰 난이도

🎯 3 (보통) | ⏱️ ~20분

시

🐇 의존성들이 모여 춤을 춘다네,
버전을 맞추고 구조를 다지며,
빌드는 조용해 명령도 숨겨,
새 색상 하나 추가되고,
우리의 프로젝트는 더욱 단단해진다! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	PR 설명은 작업 내용과 변경 사항 섹션을 포함하고 있으나, 이슈 번호와 체크리스트 항목이 누락되어 있습니다.	이슈 번호 섹션을 추가하고 체크리스트를 작성하여 템플릿을 완성해주시기 바랍니다.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 ComposableArchitecture 외부모듈 추가라는 주요 변경사항을 명확하게 요약하고 있습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/tca-setup

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

Projects/App/Resources/Assets.xcassets/AccentColor.colorset/Contents.json (1)

1-11: 빌드 경고 제거를 위한 AccentColor 추가 확인

빌드 경고를 제거하기 위해 AccentColor를 추가한 것은 좋은 접근입니다. 다만 현재는 실제 색상 값이 정의되지 않은 플레이스홀더 형태입니다. 앱에서 시스템 accent color를 사용할 계획이 있다면, 실제 색상 값을 정의하는 것을 권장합니다.

🎨 실제 색상 값을 포함한 예시

 {
   "colors" : [
     {
-      "idiom" : "universal"
+      "idiom" : "universal",
+      "color" : {
+        "color-space" : "srgb",
+        "components" : {
+          "red" : "0.000",
+          "green" : "0.478",
+          "blue" : "1.000",
+          "alpha" : "1.000"
+        }
+      }
     }
   ],
   "info" : {
     "author" : "xcode",
     "version" : 1
   }
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Projects/App/Resources/Assets.xcassets/AccentColor.colorset/Contents.json`
around lines 1 - 11, AccentColor.colorset currently contains only a placeholder
without concrete color values; update the Contents.json for AccentColor to
include actual color entries under "colors" (e.g., add a "color" object with
"color-space" and "components" for red/green/blue/alpha) and, if needed, add
"appearances" entries for light/dark variants so the system accent color is
defined; modify the JSON inside the AccentColor.colorset Contents.json (look for
the existing "colors" array and "idiom":"universal" entry) to include the proper
color dictionaries for the app's intended accent colors.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@Projects/App/Resources/Assets.xcassets/AccentColor.colorset/Contents.json`:
- Around line 1-11: AccentColor.colorset currently contains only a placeholder
without concrete color values; update the Contents.json for AccentColor to
include actual color entries under "colors" (e.g., add a "color" object with
"color-space" and "components" for red/green/blue/alpha) and, if needed, add
"appearances" entries for light/dark variants so the system accent color is
defined; modify the JSON inside the AccentColor.colorset Contents.json (look for
the existing "colors" array and "idiom":"universal" entry) to include the proper
color dictionaries for the app's intended accent colors.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 22d0e6b5-3d7d-4cea-9887-e83e96382264

📥 Commits

Reviewing files that changed from the base of the PR and between dfdb32c and bb4532e.

📒 Files selected for processing (6)

• Makefile
• Projects/App/Resources/Assets.xcassets/AccentColor.colorset/Contents.json
• Tuist/Package.resolved
• Tuist/Package.swift
• Tuist/ProjectDescriptionHelpers/Dependency.swift
• Tuist/ProjectDescriptionHelpers/Module/ExternalModule.swift

[devJungwonLee]

coderabbitai 친구가 실제 accent color 값을 지정하지 않았다고 댓글을 남겼는데,
아직 미정이고, 추후에도 추가되지 않을 것 같아 냅두겠습니다.
빈 값이어도 상관 없을 것 같아요.

[J0onYEong]

고생하셨습니다! Dependencies 의존성 관련해서 AI리뷰가 피드백을 주었지만, Tuist graph 및 빌드 실행을 통해 확인했을 때 문제 없습니다!