Skip to content

Prompting - #20

Merged
dl-tpdnjs merged 4 commits into
mainfrom
prompting
Jun 6, 2026
Merged

dl-tpdnjs merged 4 commits into
mainfrom
prompting

Conversation

@dl-tpdnjs

@dl-tpdnjs dl-tpdnjs commented Jun 6, 2026 •

Copy link
Copy Markdown
Contributor

✨ 관련된 이슈

  • close #

🙌 작업 내용

  • 맞춤형 면접 질문 사용자 답변에 대한 improvement 반환 기준 수정
  • answer-feedback-user.txt 수정

📸 스크린샷 (선택)

🤔 리뷰 요구사항(선택)

Summary by CodeRabbit

  • Bug Fixes
    • AI 피드백 판정 기준이 조정되었습니다. 기존에는 모든 항목이 3점 이상일 때 개선 불필요로 판단했으나, 이제는 모든 항목이 4점 이상일 때만 개선이 불필요하다고 처리됩니다. 이 변경으로 고품질 평가 기준이 강화되어, 더 엄격한 조건에서만 개선이 생략됩니다.

@coderabbitai

coderabbitai Bot commented Jun 6, 2026 •

Copy link
Copy Markdown

Worried about impact? Review this PR in Change Stack to explore blast radius before you approve or request changes.

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 4c4c87c0-0795-4a80-91df-26c1975fe6aa

📥 Commits

Reviewing files that changed from the base of the PR and between 62da573 and c4112af.

📒 Files selected for processing (1)
  • src/main/resources/prompts/answer-feedback-user.txt
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/main/resources/prompts/answer-feedback-user.txt

📝 Walkthrough

Walkthrough

프롬프트 한 줄이 바뀌어, improvement 필드의 JSON null 반환 조건이 모든 항목 3점 이상에서 모든 항목 4점 이상으로 상향 조정되었습니다.

Changes

Improvement 임계값 조정

Layer / File(s) Summary
Improvement null 반환 임계값 상향
src/main/resources/prompts/answer-feedback-user.txt
improvement 필드의 JSON null 반환 조건이 "모든 점수 3점 이상"에서 "모든 점수 4점 이상"으로 변경됨.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

세 칸의 기준을 하나 올려, 나는 깡충 뛰네 🐇
네 점이 되어야 문이 닫히니, 깔끔해진 길 위에
한 줄의 규칙이 마음을 다듬고,
불필요한 말은 사라지고,
진짜 개선만 남겠구나. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive PR 제목 'Prompting'은 매우 모호하고 구체적이지 않아 실제 변경 사항을 명확하게 전달하지 못합니다. 제목을 더 구체적으로 변경하세요. 예: 'improvement 반환 기준을 3점 이상에서 4점 이상으로 변경' 또는 'answer-feedback-user 프롬프트 개선 기준 상향 조정'
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch prompting

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
src/main/resources/prompts/answer-feedback-user.txt (1)

57-57: 프롬프트 임계값 상충 주장 수정 및 정리 필요

answer-feedback-system.txt(3점 이상이면 improvement null)과 answer-feedback-user.txt의 핵심 규칙(- 3점 미만 항목이 있을 때만 작성)은 모든 점수가 3점인 경우에도 improvement를 작성하지 않게 되어 있어, “3점에서 상충 지시”로 보긴 어렵습니다.

다만 answer-feedback-user.txt의 - 모든 점수가 4점 이상이면 ... JSON null 규칙(57행)은 조건(4점)과 목적(“null의 타입”)이 겹쳐 혼선을 줄 수 있으니 3점 이상으로 맞추거나(또는 중복 제거) 해당 bullet의 범위를 명확히 하는 개선이 필요합니다.

🤖 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 `@src/main/resources/prompts/answer-feedback-user.txt` at line 57, The bullet
"- 모든 점수가 4점 이상이면 문자열 "null"이 아니라 JSON null 리터럴을 반환하세요." conflicts with the
3-point threshold used in answer-feedback-system.txt and the surrounding rule
about only writing improvements when any score < 3, so update this bullet to
match the established threshold or clarify its scope: either change "4점 이상" to
"3점 이상" so it aligns with the `improvement`-null rule, or explicitly state this
rule applies only to a different field/condition and remove the overlapping
intent to avoid ambiguity.
🤖 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 `@src/main/resources/prompts/answer-feedback-user.txt`:
- Around line 55-57: The two threshold statements conflict: change the sentence
"3점 미만 항목이 있을 때만 작성하세요." to use the same 4-point threshold as the other rule by
updating it to "4점 미만 항목이 있을 때만 작성하세요." so both lines consistently require 4점
기준; locate the strings "3점 미만 항목이 있을 때만 작성하세요." and "모든 점수가 4점 이상이면 문자열
\"null\"이 아니라 JSON null 리터럴을 반환하세요." and make the first one match the 4-point
wording.

---

Nitpick comments:
In `@src/main/resources/prompts/answer-feedback-user.txt`:
- Line 57: The bullet "- 모든 점수가 4점 이상이면 문자열 "null"이 아니라 JSON null 리터럴을 반환하세요."
conflicts with the 3-point threshold used in answer-feedback-system.txt and the
surrounding rule about only writing improvements when any score < 3, so update
this bullet to match the established threshold or clarify its scope: either
change "4점 이상" to "3점 이상" so it aligns with the `improvement`-null rule, or
explicitly state this rule applies only to a different field/condition and
remove the overlapping intent to avoid ambiguity.
🪄 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.yml

Review profile: CHILL

Plan: Pro

Run ID: 22026c89-d91f-42c2-936d-b5638b71f75a

📥 Commits

Reviewing files that changed from the base of the PR and between d5e83e4 and 62da573.

📒 Files selected for processing (1)
  • src/main/resources/prompts/answer-feedback-user.txt

Comment thread src/main/resources/prompts/answer-feedback-user.txt Outdated
@dl-tpdnjs
dl-tpdnjs merged commit be3a8b9 into main Jun 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant