Fix: 피드백 문체 수정 (#226)

[jangsh7]

🔗 관련 이슈

관련된 이슈 번호를 적어주세요.

closes #226

📌 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.

🧪 테스트 결과

Postman 스크린샷, 테스트 통과 여부 등을 첨부해주세요.

📸 스크린샷 (선택)

필요시 스크린샷을 첨부해주세요.

📎 참고 사항 (선택)

리뷰어에게 전달할 내용이 있다면 작성해주세요.

Summary by CodeRabbit

변경 사항

• 개선 사항
  • AI 응답의 문체가 변경되었습니다. 모든 시스템 프롬프트의 문장이 해요체(~세요, ~에요)로 통일되어 작성됩니다. 이는 응답 형식의 변화이며, 기존의 기능과 처리 로직에는 영향을 주지 않습니다.

[coderabbitai]

Walkthrough

FeedbackPromptBuilder의 SYSTEM_PROMPT에 한국어 문체 지침을 추가하여 AI 피드백 응답이 해요체(존댓글)로 작성되도록 변경했습니다. 로직 변경 없이 프롬프트 문구만 수정되었습니다.

Changes

Cohort / File(s)	Summary
AI 피드백 프롬프트 문체 조정 src/main/java/com/umc/finly/domain/record/infra/FeedbackPromptBuilder.java	SYSTEM_PROMPT에 해요체 사용 지침 추가: "모든 문장은 반드시 해요체로 작성하세요 (예: ~세요, ~에요)"

Possibly related PRs

• [refactor] AI feedback을 content와 suggestion으로 분리 #130: FeedbackPromptBuilder의 동일한 SYSTEM_PROMPT를 수정하여 JSON 구조화 지침을 추가함
• [Hotfix] 연관분석 AI 프롬프트 말투 변경 #224: AI 프롬프트 빌더를 수정하여 응답 톤을 해요체(존댓글)로 조정함

Suggested reviewers

• ye-zin
• wonee1
• na311ng
• dosp74

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR 설명이 템플릿 구조는 따르지만, 작업 내용, 테스트 결과 등 주요 섹션이 비어있습니다.	작업 내용 섹션에 SYSTEM_PROMPT 가이드라인 추가 사항을 구체적으로 작성하고, 테스트 결과를 첨부해주세요.
Linked Issues check	❓ Inconclusive	연결된 이슈 #226의 구체적인 목표가 명시되지 않았으며, 체크리스트가 미작성 상태입니다.	이슈 #226에 구체적인 작업 내용과 수용 기준을 추가하여 변경 사항이 요구사항을 충족하는지 명확히 해주세요.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경 사항과 일치합니다. 시스템 프롬프트의 문체를 수정한 것과 '피드백 문체 수정'이라는 제목이 부합합니다.
Out of Scope Changes check	✅ Passed	SYSTEM_PROMPT의 문체 가이드라인 추가는 이슈 #226의 '피드백 문체 조정'과 관련이 있으며, 범위를 벗어나지 않습니다.
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

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/record-226-feedback-refactor

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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)

src/main/java/com/umc/finly/domain/record/infra/FeedbackPromptBuilder.java (1)

30-30: 해요체 예시 범위가 기존 예문과 일부 불일치합니다.

신규 규칙(Line 30)은 해요체 예시로 ~세요, ~에요만 제시하고 있지만, 바로 아래 Line 35의 기존 예문에는 매도하셨군요, 하셨네요(~군요, ~네요 어미)가 사용되어 있습니다. ~군요, ~네요도 해요체에 해당하나, 예시가 좁게 명시되어 있어 LLM이 이 어미들을 해요체 외 표현으로 오해할 여지가 있습니다. 예시를 보완하면 프롬프트 내 일관성이 높아집니다.

✏️ 예시 보완 제안

-            - 모든 문장은 반드시 해요체로 작성하세요 (예: ~세요, ~에요)
+            - 모든 문장은 반드시 해요체로 작성하세요 (예: ~세요, ~에요, ~네요, ~군요, ~죠)