Skip to content

Fix/#149 1์ฐจ QA #154

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sum130 merged 7 commits into from
Jan 22, 2026
Merged

Fix/#149 1์ฐจ QA #154

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
ce593b1
feat: #149 ์ฑŒ๋ฆฐ์ง€ ์ข…๋ฃŒ ๋ถ„๊ธฐ์ฒ˜๋ฆฌ
sum130 Jan 21, 2026
1406d9d
fix: #149 ์ฑŒ๋ฆฐ์ง€๋ทฐ ํ…์ŠคํŠธ ๋‚ด์šฉ ์ˆ˜์ •
sum130 Jan 21, 2026
540c737
style: #149 ์ฒด๋ฆฌ png๋กœ ๋ณ€๊ฒฝ
sum130 Jan 21, 2026
e2ce49c
fix: #149 ํ…์ŠคํŠธ ํŒจ๋”ฉ๊ฐ’ ์ˆ˜์ •
sum130 Jan 21, 2026
a59ff84
fix: #149 1์ฐจQA ์™„๋ฃŒ
sum130 Jan 21, 2026
967c20a
fix: #149 ์ฑŒ๋ฆฐ์ง€ ํฐํŠธ ํ”„๋ ˆ์ž„ ์ˆ˜์ •
sum130 Jan 22, 2026
2183fc2
Merge remote-tracking branch 'origin/develop' into feat/#149-challengโ€ฆ
sum130 Jan 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions ...Presentation/Feature/ChallengeView/Coordinator/ViewModel/ChallengeProgressViewModel.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ final class ChallengeProgressViewModel: ObservableObject {
@Published private(set) var remainMissions: Int = 0
@Published private(set) var progressRate = 0
@Published private(set) var currentDay: Int = 1
@Published private(set) var challengeTitle: String = "์ฑŒ๋ฆฐ์ง€"
@Published private(set) var challengeTitle: String = ""
@Published private(set) var todayRoutines: [ProgressRoutineEntity] = []

private let fetchChallengeUseCase: FetchChallengeUseCase
Expand Down Expand Up @@ -94,14 +94,18 @@ final class ChallengeProgressViewModel: ObservableObject {
}

extension ChallengeProgressViewModel {
var isChallengeCompleted: Bool {
currentDay == 7
}
Comment on lines +97 to +99
Copy link

@coderabbitai coderabbitai bot Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

๐Ÿงน Nitpick | ๐Ÿ”ต Trivial

์™„๋ฃŒ ๊ธฐ์ค€(7์ผ) ํ•˜๋“œ์ฝ”๋”ฉ ํ™•์ธ ํ•„์š”.
์ฑŒ๋ฆฐ์ง€ ๊ธฐ๊ฐ„์ด ๊ณ ์ • 7์ผ์ด ์•„๋‹ˆ๋ผ๋ฉด ๋ฐ์ดํ„ฐ/์ƒ์ˆ˜๋กœ ๋ถ„๋ฆฌํ•ด ๊ธฐ์ค€์„ ๋งž์ถ”๋Š” ํŽธ์ด ์•ˆ์ „ํ•ฉ๋‹ˆ๋‹ค.

๐Ÿค– Prompt for AI Agents 
In
`@Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/Coordinator/ViewModel/ChallengeProgressViewModel.swift`
around lines 97 - 99, The isChallengeCompleted property currently hardcodes the
7-day completion check; change it to use a configurable duration value (e.g., a
Challenge.duration / totalDays constant or a value from the Challenge
model/repository) instead of the literal 7. Update ChallengeProgressViewModel to
reference that configurable property (for example challengeDuration or
challenge.totalDays) when evaluating currentDay in isChallengeCompleted so
different challenge lengths are supported. Ensure the new source is injected or
accessible in the ViewModel constructor or via its existing challenge/currentDay
data so you don't introduce globals.


@MainActor
private func updateInfo() {
guard let challengeData else { return }
cherryLevel = CherryLevel.from(progressRate: Double(challengeData.progressPercentage))
remainMissions = challengeData.remainingRoutinesToNextLevel
progressRate = challengeData.progressPercentage
currentDay = challengeData.currentDay
challengeTitle = challengeData.title
challengeTitle = challengeData.title + " ์ฑŒ๋ฆฐ์ง€"
todayRoutines = challengeData.todayRoutines
}
}
7 changes: 5 additions & 2 deletions Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/ChallengeLoadingView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ struct ChallengeLoadingView: View {
VStack {
VStack(spacing: 4.adjustedH) {
highlight(highlightText: viewModel.selectedRoutine?.description ?? "", normalText: "๋ฐฉํ–ฅ์„ ๋ฐ”ํƒ•์œผ๋กœ")
.padding(.top, 113.adjustedH)
.frame(height: 27.adjustedH)
.padding(.top, 94.adjustedH)
TypographyText("TO-DO ๋ฏธ์…˜์„ ๋งŒ๋“ค๊ณ  ์žˆ์–ด์š”.", style: .title1_sb_18, color: .gray800)

LottieView(animationName: "splash", loopMode: .loop)
Expand All @@ -26,12 +27,14 @@ struct ChallengeLoadingView: View {
Spacer()
.frame(height: 80.adjustedH)

TypographyText("์ž ์‹œ๋งŒ ๊ธฐ๋‹ค๋ ค์ฃผ์„ธ์š”!", style: .title2_sb_16, color: .gray800)
TypographyText("์ž ์‹œ๋งŒ ๊ธฐ๋‹ค๋ ค์ฃผ์„ธ์š”!", style: .title2_m_16, color: .gray800)
.frame(height: 24.adjustedH)
.padding(.top, 17.adjustedH)

Spacer()

TypographyText("AI๊ฐ€ ๋งž์ถคํ˜• ๋ฃจํ‹ด์„ ์ œ์ž‘ํ•˜๊ณ  ์žˆ์–ด์š”.", style: .body3_m_12, color: .gray600)
.frame(height: 17.adjustedH)
.padding(.bottom, 30.adjustedH)
}
}
Expand Down
46 changes: 35 additions & 11 deletions ...ChallengeView/ChallengeProgressView.swift โ†’ ...engeView/View/ChallengeProgressView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,17 +48,20 @@ enum CherryLevel: Int {
}

struct ChallengeProgressView: View {
@EnvironmentObject private var challengeCoordinator: ChallengeCoordinator
@StateObject var viewModel: ChallengeProgressViewModel

Comment on lines +51 to 53
Copy link

@coderabbitai coderabbitai bot Jan 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

๐Ÿงน Nitpick | ๐Ÿ”ต Trivial

EnvironmentObject ์ฃผ์ž… ๋ณด์žฅ ๋ฐ @StateObject ๊ฐ€์‹œ์„ฑ ์ •๋ฆฌ ํ•„์š”

์ƒˆ๋กœ ์ถ”๊ฐ€๋œ EnvironmentObject๊ฐ€ ์ƒ์œ„์—์„œ ์ฃผ์ž…๋˜์ง€ ์•Š์œผ๋ฉด ๋Ÿฐํƒ€์ž„ ํฌ๋ž˜์‹œ๊ฐ€ ๋‚ฉ๋‹ˆ๋‹ค. ์ƒ์„ฑ ์œ„์น˜์—์„œ ์ฃผ์ž… ์—ฌ๋ถ€๋ฅผ ํ™•์ธํ•ด ์ฃผ์„ธ์š”. ๋˜ํ•œ @StateObject๋Š” ๋ณดํ†ต private๋กœ ๋‘๊ณ  init์—์„œ ์ฃผ์ž…ํ•˜๋Š” ํŒจํ„ด์ด ๊ถŒ์žฅ๋ฉ๋‹ˆ๋‹ค(SwiftLint ๊ฒฝ๊ณ ).

โ™ป๏ธ `@StateObject` ์ ‘๊ทผ์„ฑ ์ •๋ฆฌ ์ œ์•ˆ 
-    `@StateObject` var viewModel: ChallengeProgressViewModel
+    `@StateObject` private var viewModel: ChallengeProgressViewModel
+
+    init(viewModel: ChallengeProgressViewModel) {
+        _viewModel = StateObject(wrappedValue: viewModel)
+    }
#!/bin/bash
# ChallengeProgressView ์ƒ์„ฑ ์œ„์น˜์™€ EnvironmentObject ์ฃผ์ž… ์—ฌ๋ถ€ ํ™•์ธ
rg -n --type=swift -C3 '\bChallengeProgressView\b'
rg -n --type=swift -C3 '\.environmentObject\([^)]*ChallengeCoordinator'
๐Ÿงฐ Tools
๐Ÿช› SwiftLint (0.57.0)

[Warning] 52-52: SwiftUI state properties should be private

(private_swiftui_state)

๐Ÿค– Prompt for AI Agents 
In
`@Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/ChallengeProgressView.swift`
around lines 51 - 53, The ChallengeProgressView currently declares
`@EnvironmentObject` private var challengeCoordinator: ChallengeCoordinator which
can crash if not injected and declares `@StateObject` var viewModel:
ChallengeProgressViewModel with non-private visibility; update the view to
ensure safe EnvironmentObject usage by either (1) asserting or providing a
fallback when ChallengeCoordinator is missing at creation sites (verify all
ChallengeProgressView initializers call
.environmentObject(ChallengeCoordinator()) or wrap uses in a parent that injects
it) or (2) change the property to `@EnvironmentObject` private var
challengeCoordinator: ChallengeCoordinator? and guard/handle the nil case to
avoid runtime crashes; also make viewModel private (change to `@StateObject`
private var viewModel) and adopt the init-injection pattern for
ChallengeProgressView by adding an init(_ viewModel: ChallengeProgressViewModel)
that assigns _viewModel = StateObject(wrappedValue: viewModel) to satisfy
SwiftLint and visibility expectations, and then update all call sites to pass
the viewModel and ensure .environmentObject(ChallengeCoordinator) is applied
where the view is created.

let buttonState: ButtonState = .active

var body: some View {
ScrollView {
VStack {
HStack {
TypographyText(viewModel.challengeTitle, style: .title1_sb_18, color: .gray1000)
.frame(height: 27.adjustedH)
.padding(.trailing, 12.adjustedW)
TypographyText("7์ผ ํ”Œ๋žœ", style: .body3_m_12, color: .gray700)
.frame(height: 17.adjustedH)
.padding(.horizontal, 8.adjustedW)
.padding(.vertical, 3.adjustedH)
.overlay(
Expand All @@ -82,25 +85,38 @@ struct ChallengeProgressView: View {

extension ChallengeProgressView {
private var CherryGrowthView: some View {
VStack {
VStack {
VStack(spacing: 0) {
VStack(spacing: 0) {
HStack {
TypographyText("Lv.\(viewModel.cherryLevel.levelNumber) \(viewModel.cherryLevel.name)", style: .body1_m_14, color: .gray900)
TypographyText("Lv.\(viewModel.cherryLevel.levelNumber)", style: .body1_m_14, color: .gray900)
.frame(height: 20.adjustedH)
TypographyText("\(viewModel.cherryLevel.name)", style: .body1_m_14, color: .gray900)
.frame(height: 20.adjustedH)
.padding(.leading, 6.adjustedW)
Spacer()
}
viewModel.cherryLevel.cherryImage
.padding(.top, 14.adjustedH)
TypographyText("์ฒด๋ฆฌ๊ฐ€ ํฌ๋ ค๋ฉด \(viewModel.remainMissions)๊ฐœ์˜ ๋ฏธ์…˜์„ ์ˆ˜ํ–‰ํ•ด์•ผ ํ•ด์š”!", style: .body2_r_13, color: .gray800)
.padding(.top, 14.adjustedH)
.frame(width: 154.adjustedW, height: 154.adjustedW)
.padding(.bottom, 5.adjustedH)
if viewModel.cherryLevel.levelNumber == 4 {
TypographyText("์ฑŒ๋ฆฐ์ง€ ์™„๋ฃŒ๊นŒ์ง€ \(viewModel.remainMissions)๊ฐœ์˜ ๋ฏธ์…˜์„ ์ˆ˜ํ–‰ํ•ด์•ผ ํ•ด์š”!", style: .body2_r_13, color: .gray800)
.frame(height: 18.adjustedH)
.padding(.bottom, 14.adjustedH)
}else {
TypographyText("์ฒด๋ฆฌ๊ฐ€ ํฌ๋ ค๋ฉด \(viewModel.remainMissions)๊ฐœ์˜ ๋ฏธ์…˜์„ ์ˆ˜ํ–‰ํ•ด์•ผ ํ•ด์š”!", style: .body2_r_13, color: .gray800)
.frame(height: 18.adjustedH)
.padding(.bottom, 14.adjustedH)
}
}
.padding(.horizontal, 25.adjustedW)
Rectangle()
.fill(.gray300)
.frame(height: 1)
.padding(.vertical, 14.adjustedH)
.padding(.bottom, 14.adjustedH)
VStack {
HStack {
TypographyText("์ฑŒ๋ฆฐ์ง€ ๋‹ฌ์„ฑ๋ฅ  \(viewModel.progressRate)%", style: .body1_m_14, color: .gray900)
.frame(height: 20.adjustedH)
Spacer()
}
.padding(.bottom, 12.adjustedH)
Expand Down Expand Up @@ -129,6 +145,7 @@ extension ChallengeProgressView {
VStack {
HStack {
TypographyText("\(viewModel.currentDay)์ผ์ฐจ TO-DO ๋ฏธ์…˜", style: .body1_sb_14, color: .gray1000)
.frame(height: 20.adjustedH)
Spacer()
}
Spacer()
Expand All @@ -149,14 +166,21 @@ extension ChallengeProgressView {
}

CherrishButton(
title: "์˜ค๋Š˜ ๋ฏธ์…˜ ์ข…๋ฃŒํ•˜๊ธฐ",
title: viewModel.isChallengeCompleted ? "์ฑŒ๋ฆฐ์ง€ ์ข…๋ฃŒํ•˜๊ธฐ":"์˜ค๋Š˜ ๋ฏธ์…˜ ์ข…๋ฃŒํ•˜๊ธฐ",
type: .small,
state: .constant(buttonState),
leadingIcon: nil,
trailingIcon: nil
) {
Task {
await viewModel.advanceDay()
if viewModel.isChallengeCompleted {
Task {
await viewModel.advanceDay()
challengeCoordinator.push(.startChallenge)
}
}else {
Task {
await viewModel.advanceDay()
}
Comment on lines 168 to +183
Copy link

@coderabbitai coderabbitai bot Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

โš ๏ธ Potential issue | ๐ŸŸ  Major

์™„๋ฃŒ ๋ถ„๊ธฐ์—์„œ advanceDay ์‹คํŒจ ์‹œ์—๋„ ๋„ค๋น„๊ฒŒ์ด์…˜๋ฉ๋‹ˆ๋‹ค.
advanceDay()๊ฐ€ ์‹คํŒจํ•ด๋„ ๊ณง๋ฐ”๋กœ ์ƒˆ ์ฑŒ๋ฆฐ์ง€๋กœ ์ด๋™ํ•˜๋ฉด ์„œ๋ฒ„ ์ƒํƒœ์™€ UI๊ฐ€ ์–ด๊ธ‹๋‚  ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค. ์„ฑ๊ณต ํ™•์ธ ํ›„ ์ด๋™ํ•˜๋„๋ก ๊ฐ€๋“œ๊ฐ€ ํ•„์š”ํ•ฉ๋‹ˆ๋‹ค.

โœ… ์ˆ˜์ • ์ œ์•ˆ 
-                if viewModel.isChallengeCompleted {
-                    Task {
-                        await viewModel.advanceDay()
-                        challengeCoordinator.push(.startChallenge)
-                    }
-                }else {
-                    Task {
-                        await viewModel.advanceDay()
-                    }
-                }
+                if viewModel.isChallengeCompleted {
+                    Task {
+                        await viewModel.advanceDay()
+                        guard viewModel.errorMessage == nil else { return }
+                        challengeCoordinator.push(.startChallenge)
+                    }
+                } else {
+                    Task {
+                        await viewModel.advanceDay()
+                    }
+                }
๐Ÿค– Prompt for AI Agents 
In
`@Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/ChallengeProgressView.swift`
around lines 160 - 175, The completion branch calls
challengeCoordinator.push(.startChallenge) regardless of whether
viewModel.advanceDay() succeeded; modify the branch handling in the
CherrishButton action so you await and verify success before navigating: call
viewModel.advanceDay() and check its result (or catch errors if it throws) and
only call challengeCoordinator.push(.startChallenge) when advanceDay reports
success; update viewModel.advanceDay() to return a Bool or throw if it currently
doesn't so you can gate navigation in the completion path (symbols:
CherrishButton, viewModel.isChallengeCompleted, viewModel.advanceDay(),
challengeCoordinator.push(.startChallenge)).

}
}
.padding(.top, 10.adjustedH)
Expand Down
11 changes: 4 additions & 7 deletions ...iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/ChallengeSelectMissionView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,20 +17,17 @@ struct ChallengeSelectMissionView: View {
VStack {
VStack {
HStack {
VStack(alignment: .leading) {
TypographyText("์ฑŒ๋ฆฐ์ง€ ๊ธฐ๊ฐ„ ๋™์•ˆ",
style: .title1_sb_18,
color: .gray1000
)
TypographyText("์ง„ํ–‰ํ•  ๋ฏธ์…˜์„ ์„ ํƒํ•ด์ฃผ์„ธ์š”.",
VStack(alignment: .leading, spacing: 4.adjustedH) {
TypographyText("์ฑŒ๋ฆฐ์ง€ ๊ธฐ๊ฐ„ ๋™์•ˆ\n์ง„ํ–‰ํ•  ๋ฏธ์…˜์„ ์„ ํƒํ•ด์ฃผ์„ธ์š”.",
style: .title1_sb_18,
color: .gray1000
)
.frame(height: 54.adjustedH)
TypographyText("๋ณต์ˆ˜ ์„ ํƒ์ด ๊ฐ€๋Šฅํ•ด์š”.",
style: .body1_r_14,
color: .gray700
)
.padding(.top, 4.adjustedH)
.frame(height: 20.adjustedH)
}
Spacer()
}
Expand Down
10 changes: 8 additions & 2 deletions ...iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/ChallengeSelectRoutineView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,17 @@ struct ChallengeSelectRoutineView: View {
var body: some View {
VStack {
HStack{
VStack(alignment: .leading){
TypographyText("์ง€๊ธˆ ๋‚˜์—๊ฒŒ ๊ฐ€์žฅ ํ•„์š”ํ•œ\n๊ด€๋ฆฌ ๋ฃจํ‹ด์„ ์„ ํƒํ•ด์ฃผ์„ธ์š”.",
VStack(alignment: .leading, spacing: 0){
TypographyText("์ง€๊ธˆ ๋‚˜์—๊ฒŒ ๊ฐ€์žฅ ํ•„์š”ํ•œ",
style: .title1_sb_18,
color: .gray1000
)
.frame(height: 27.adjustedH)
TypographyText("๊ด€๋ฆฌ ๋ฃจํ‹ด์„ ์„ ํƒํ•ด์ฃผ์„ธ์š”.",
style: .title1_sb_18,
color: .gray1000
)
.frame(height: 27.adjustedH)
Comment on lines +21 to +31
Copy link

@coderabbitai coderabbitai bot Jan 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

โš ๏ธ Potential issue | ๐ŸŸก Minor

๐Ÿงฉ Analysis chain

๐Ÿ Script executed:

# ํŒŒ์ผ ๊ตฌ์กฐ ํƒ์ƒ‰
find . -type f -name "*.swift" | grep -i typography | head -20

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 196

๐Ÿ Script executed:

# ChallengeSelectRoutineView.swift ์ „์ฒด ๋‚ด์šฉ ํ™•์ธ
find . -path "*ChallengeSelectRoutineView.swift" -type f

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 196

๐Ÿ Script executed:

# TypographyText ์ •์˜ ์ฐพ๊ธฐ
rg "struct TypographyText|class TypographyText" -A 10

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 1081

๐Ÿ Script executed:

# adjustedH ์ •์˜ ์ฐพ๊ธฐ
rg "adjustedH" -B 2 -A 2

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 50383

๐Ÿ Script executed:

# adjustedH ํ™•์žฅ ํ•จ์ˆ˜ ์ •์˜ ์ฐพ๊ธฐ
rg "adjustedH" -B 5 -A 2 | grep -A 7 "func adjustedH\|extension.*adjustedH\|var adjustedH"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 2224

๐Ÿ Script executed:

# TypographyText ์ „์ฒด ๊ตฌํ˜„ ํ™•์ธ
cat -n Cherrish-iOS/Cherrish-iOS/Presentation/Global/Extension/Font+.swift | head -100

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 3963

๐Ÿ Script executed:

# ๋‹ค์ด๋‚˜๋ฏน ํƒ€์ž… ์ง€์› ์—ฌ๋ถ€ ํ™•์ธ
rg "dynamicTypeSize|preferredContentSizeCategory|@Environment.*dynamicTypeSize" -A 3

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 52

๊ณ ์ • ๋†’์ด๋กœ ๋‹ค์ค‘ ์–ธ์–ด ํ™•๋Œ€ ๋ฐ ๋‹ค์ด๋‚˜๋ฏน ํƒ€์ž…์—์„œ ํ…์ŠคํŠธ ์ž˜๋ฆผ ์œ„ํ—˜

.frame(height: 27.adjustedH)๋Š” ํ™”๋ฉด ๋น„์œจ๋งŒ ์กฐ์ •ํ•˜๊ณ  ๋‹ค์ด๋‚˜๋ฏน ํƒ€์ž…(์‹œ์Šคํ…œ ํฐํŠธ ์„ค์ •)์„ ๊ณ ๋ คํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค. ๊ธด ๋ฒˆ์—ญ์ด ํ•„์š”ํ•œ ๋กœ์ผ€์ผ์ด๋‚˜ ์‚ฌ์šฉ์ž๊ฐ€ ํ…์ŠคํŠธ ํฌ๊ธฐ๋ฅผ ํ™•๋Œ€ํ–ˆ์„ ๋•Œ ๋‘ ์ค„์ด ์ง€์ •๋œ ๋†’์ด๋ฅผ ์ดˆ๊ณผํ•˜์—ฌ ์ž˜๋ฆด ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค.

๐Ÿค– Prompt for AI Agents 
In
`@Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/ChallengeSelectRoutineView.swift`
around lines 21 - 31, The two TypographyText views inside the VStack are forced
to a fixed height via .frame(height: 27.adjustedH) which can cause truncation
for longer translations or Dynamic Type; remove the fixed height or replace it
with a dynamic-friendly approach for the TypographyText components used here
(e.g., allow flexible height, remove .frame(height:), and if necessary add
modifiers like lineLimit(nil) / fixedSize(horizontal: false, vertical: true) or
use .multilineTextAlignment and a minHeight constraint) so the text can grow
with accessibility font sizes and long locales while keeping the existing VStack
and TypographyText usages intact.

}
Spacer()
}
Expand Down
7 changes: 5 additions & 2 deletions ...re/ChallengeView/StartChallengeView.swift โ†’ ...allengeView/View/StartChallengeView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,9 @@ struct ChallengeStartChallengeView: View {
HStack {
VStack(alignment: .leading, spacing: 4) {
TypographyText("์ด๋ฒˆ์—” ์–ด๋–ค ๋ฃจํ‹ด์œผ๋กœ ๊ด€๋ฆฌํ• ๊นŒ์š”?", style: .headline_sb_20, color: .gray1000)
.frame(height: 30.adjustedH)
TypographyText("๋ฃจํ‹ด์„ ์ง€ํ‚ฌ์ˆ˜๋ก ์ฒด๋ฆฌ๊ฐ€ ์„ฑ์žฅํ•ด์š”.", style: .title2_m_16, color: .gray800)
.frame(height: 24.adjustedH)
Comment on lines 20 to +23
Copy link

@coderabbitai coderabbitai bot Jan 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

โš ๏ธ Potential issue | ๐ŸŸก Minor

๐Ÿงฉ Analysis chain

๐Ÿ Script executed:

fd "StartChallengeView.swift" --type f

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 160

๐Ÿ Script executed:

find . -name "StartChallengeView.swift" -type f

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 162

๐Ÿ Script executed:

git ls-files | grep -i "startchallenge"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 121

๐Ÿ Script executed:

wc -l "Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/StartChallengeView.swift"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 163

๐Ÿ Script executed:

cat -n "Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/StartChallengeView.swift"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 2345

๐Ÿ Script executed:

rg "\.adjustedH" --type swift | head -20

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 2610

๐Ÿ Script executed:

rg "extension.*Int\|extension.*CGFloat" --type swift -A 3 | grep -A 5 "adjustedH"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 52

๐Ÿ Script executed:

fd "TypographyText" --type swift

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 242

๐Ÿ Script executed:

rg "adjustedH" --type swift -B 2 -A 2 | grep -E "extension|computed|var" | head -20

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 2215

๐Ÿ Script executed:

find . -name "*TypographyText*" -o -name "*typography*" | grep -i swift

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 52

๐Ÿ Script executed:

rg "struct TypographyText|class TypographyText" --type swift

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 167

๐Ÿ Script executed:

cat -n "Cherrish-iOS/Cherrish-iOS/Presentation/Global/Extension/Adjust+.swift"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 1218

๐Ÿ Script executed:

cat -n "Cherrish-iOS/Cherrish-iOS/Presentation/Global/Extension/Font+.swift"

Repository: TEAM-Cherrish/Cherrish-iOS

Length of output: 3963

๊ณ ์ • ๋†’์ด๊ฐ€ ํ…์ŠคํŠธ ๋ž˜ํ•‘๊ณผ ๋กœ์ปฌ๋ผ์ด์ œ์ด์…˜ ์‹œ ํ…์ŠคํŠธ ์ž˜๋ฆผ ์œ„ํ—˜ ๊ฐ€๋Šฅ

ํ…์ŠคํŠธ์— ๊ณ ์ • height๋ฅผ ์ง€์ •ํ•˜๋ฉด ๋‹ค์ค‘ ๋ผ์ธ ํ…์ŠคํŠธ๋‚˜ ๊ธด ์–ธ์–ด ๋ฌธ์ž์—ด(ํ•œ๊ตญ์–ด ๋“ฑ)์—์„œ ๋†’์ด ์ดˆ๊ณผ ์‹œ ์ž˜๋ฆด ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค. ํŠนํžˆ 30.adjustedH(line 21)์™€ 24.adjustedH(line 23, 38)๋Š” ๋‹จ์ผ ๋ผ์ธ ๊ธฐ์ค€์œผ๋กœ ์„ค์ •๋˜์—ˆ์œผ๋‚˜, ํ…์ŠคํŠธ๊ฐ€ ๋ž˜ํ•‘๋  ๊ฒฝ์šฐ ์ฝ˜ํ…์ธ ๋ฅผ ๋‹ด๊ธฐ ๋ถ€์กฑํ•  ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค. ํ•„์š”์‹œ lineLimit(nil) ์ถ”๊ฐ€๋‚˜ ๋™์  ๋†’์ด ์กฐ์ •์„ ๊ฒ€ํ† ํ•ด ์ฃผ์„ธ์š”.

๐Ÿค– Prompt for AI Agents 
In
`@Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/View/StartChallengeView.swift`
around lines 20 - 23, The two TypographyText instances (the one with "์ด๋ฒˆ์—” ์–ด๋–ค
๋ฃจํ‹ด์œผ๋กœ ๊ด€๋ฆฌํ• ๊นŒ์š”?" using .frame(height: 30.adjustedH) and the one with "๋ฃจํ‹ด์„ ์ง€ํ‚ฌ์ˆ˜๋ก ์ฒด๋ฆฌ๊ฐ€
์„ฑ์žฅํ•ด์š”." using .frame(height: 24.adjustedH)) use fixed heights that can truncate
wrapped or localized text; remove the fixed .frame(height: ...) or replace it
with a flexible layout approach and enable multiline by adding lineLimit(nil)
and allowing vertical expansion (e.g., fixedSize(horizontal: false, vertical:
true) or no explicit height constraint) so the text can grow for
wrapping/localization while keeping any intended spacing via surrounding
containers or Spacer.

}
Spacer()
}
Expand All @@ -30,11 +32,12 @@ struct ChallengeStartChallengeView: View {
.padding(.top, 10.adjustedH)
.padding(.horizontal, 24.adjustedW)

HStack(spacing: 12) {
HStack(spacing: 0) {
Image("info")
TypographyText("์ด ์ฑŒ๋ฆฐ์ง€๋Š” ์„ค์ • ์‹œ์ ๋ถ€ํ„ฐ 7์ผ๊ฐ„ ์ง„ํ–‰๋ฉ๋‹ˆ๋‹ค.", style: .body3_m_12, color: .gray600)
.frame(height: 24.adjustedH)
}
.padding(.bottom, 12.adjustedH)
.padding(.top, 20.adjustedH)

CherrishButton(
title: "์ฑŒ๋ฆฐ์ง€ ์‹œ์ž‘ํ•˜๊ธฐ",
Expand Down
12 changes: 0 additions & 12 deletions ...-iOS/Resources/Assets.xcassets/ChallengeImage/challenge_gaugebar_0.imageset/Contents.json
View file
Open in desktop

This file was deleted.

10 changes: 0 additions & 10 deletions ....xcassets/ChallengeImage/challenge_gaugebar_0.imageset/challenge_gaugebar_0.svg
View file
Open in desktop

This file was deleted.

15 changes: 13 additions & 2 deletions ...-iOS/Resources/Assets.xcassets/ChallengeImage/challenge_gaugebar_1.imageset/Contents.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,19 @@
{
"images" : [
{
"filename" : "challenge_gaugebar_1.svg",
"idiom" : "universal"
"filename" : "challenge_gaugebar_1.png",
"idiom" : "universal",
"scale" : "1x"
},
{
"filename" : "challenge_gaugebar_1@2x.png",
"idiom" : "universal",
"scale" : "2x"
},
{
"filename" : "challenge_gaugebar_1@3x.png",
"idiom" : "universal",
"scale" : "3x"
}
],
"info" : {
Expand Down
Binary file added ....xcassets/ChallengeImage/challenge_gaugebar_1.imageset/challenge_gaugebar_1.png
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading