Skip to content

Commit e3764d0

Browse files
committed
feat: 리뷰 워크스루 출력 형식 개선 (변경 내용 및 의도에 포맷 규칙 추가)
1 parent a309abb commit e3764d0

3 files changed

Lines changed: 22 additions & 10 deletions

File tree

action/index.cjs

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -150875,7 +150875,9 @@ const robot = (app) => {
150875150875
parts.push("| 파일 | 변경 내용 | 의도 |");
150876150876
parts.push("|------|----------|------|");
150877150877
for (const w of review.walkthrough) {
150878-
parts.push(`| \`${w.file}\` | ${w.changes} | ${w.intent} |`);
150878+
const changesWithBr = w.changes.replace(/\n/g, "<br>");
150879+
const intentWithBr = w.intent.replace(/\n/g, "<br>");
150880+
parts.push(`| \`${w.file}\` | ${changesWithBr} | ${intentWithBr} |`);
150879150881
}
150880150882
}
150881150883
if (review.affectedAreas.length > 0) {
@@ -151177,6 +151179,10 @@ class Chat {
151177151179
- **changes**: 무엇이 바뀌었는지 (기술적 사실)
151178151180
- **intent**: 왜 바꿨는지 (추론한 의도)
151179151181

151182+
**포맷 규칙**:
151183+
- 각 항목 앞에 \`-\`를 붙여 구분
151184+
- 내용이 길면 줄바꿈(\\n) 사용
151185+
151180151186
---
151181151187

151182151188
## 예시 출력
@@ -151187,13 +151193,13 @@ class Chat {
151187151193
"walkthrough": [
151188151194
{
151189151195
"file": "src/auth/twoFactor.ts",
151190-
"changes": "TOTP 생성/검증 로직 구현",
151191-
"intent": "2FA 핵심 기능 제공"
151196+
"changes": "- TOTP 시크릿 생성 함수 추가\\n- OTP 검증 로직 구현\\n- QR 코드 URL 생성",
151197+
"intent": "- 2FA 핵심 기능 제공"
151192151198
},
151193151199
{
151194151200
"file": "src/auth/login.ts",
151195-
"changes": "로그인 플로우에 2FA 검증 단계 추가",
151196-
"intent": "기존 인증 흐름에 2FA 통합"
151201+
"changes": "- 로그인 플로우에 2FA 검증 단계 추가",
151202+
"intent": "- 기존 인증 흐름에 2FA 통합\\n- 2FA 미설정 사용자는 기존 플로우 유지"
151197151203
}
151198151204
],
151199151205
"affectedAreas": ["로그인", "사용자 설정", "세션 관리"]

src/bot.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -668,7 +668,9 @@ export const robot = (app: Probot) => {
668668
parts.push("| 파일 | 변경 내용 | 의도 |");
669669
parts.push("|------|----------|------|");
670670
for (const w of review.walkthrough) {
671-
parts.push(`| \`${w.file}\` | ${w.changes} | ${w.intent} |`);
671+
const changesWithBr = w.changes.replace(/\n/g, "<br>");
672+
const intentWithBr = w.intent.replace(/\n/g, "<br>");
673+
parts.push(`| \`${w.file}\` | ${changesWithBr} | ${intentWithBr} |`);
672674
}
673675
}
674676

src/chat.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,10 @@ export class Chat {
192192
- **changes**: 무엇이 바뀌었는지 (기술적 사실)
193193
- **intent**: 왜 바꿨는지 (추론한 의도)
194194
195+
**포맷 규칙**:
196+
- 각 항목 앞에 \`-\`를 붙여 구분
197+
- 내용이 길면 줄바꿈(\\n) 사용
198+
195199
---
196200
197201
## 예시 출력
@@ -202,13 +206,13 @@ export class Chat {
202206
"walkthrough": [
203207
{
204208
"file": "src/auth/twoFactor.ts",
205-
"changes": "TOTP 생성/검증 로직 구현",
206-
"intent": "2FA 핵심 기능 제공"
209+
"changes": "- TOTP 시크릿 생성 함수 추가\\n- OTP 검증 로직 구현\\n- QR 코드 URL 생성",
210+
"intent": "- 2FA 핵심 기능 제공"
207211
},
208212
{
209213
"file": "src/auth/login.ts",
210-
"changes": "로그인 플로우에 2FA 검증 단계 추가",
211-
"intent": "기존 인증 흐름에 2FA 통합"
214+
"changes": "- 로그인 플로우에 2FA 검증 단계 추가",
215+
"intent": "- 기존 인증 흐름에 2FA 통합\\n- 2FA 미설정 사용자는 기존 플로우 유지"
212216
}
213217
],
214218
"affectedAreas": ["로그인", "사용자 설정", "세션 관리"]

0 commit comments

Comments
 (0)