[FEAT] 코드블록 분리 프롬프팅 추가

[coli-geonwoo]

🚩 연관 이슈

close #95

🔂 변경 내역

Summary by CodeRabbit

• 문서화
  • PR 분석 지침 강화: 코드 인용 시 언어 힌트가 있는 마크다운 코드 블록과 원본 파일 참조를 포함하도록 권장.
  • 질문 작성 규칙 추가: 각 카테고리 3개 이상, 최대 255자, 정중한 의문문 형식 및 관련 변경 내용 코드 인용 필수.
  • 출력 JSON 스키마 안내 문구의 문장부호 및 형식 정교화.
• 리팩터링
  • 내부 컴포넌트 등록 방식 일부 정리.

[coderabbitai]

Walkthrough

프롬프트/구성 YAML에 코드블록 사용과 질문 제약을 추가 및 JSON 스키마 안내 문장 끝문자 수정, 그리고 OpenAiPrAnalysisClient에서 Spring의 @Component 어노테이션을 제거해 빈 자동등록 상태가 변경되었습니다.

Changes

Cohort / File(s)	Summary of Changes
프롬프트 · 구성 파일 gss-client/gss-mcp-client/src/main/resources/application-mcp-client.yml, gss-mcp-app/src/test/resources/application.yml	PR 질문 생성 안내에 Markdown 코드블록 사용(언어 힌트 권장) 및 코드블록 내 출처 파일 표기 요구 추가; 질문 제약(최대 255자, 카테고리별 3개 이상, 정중한 의문형 등) 추가; JSON 스키마 안내 라인 끝 문구를 ")."에서 ")::"로 변경.
Spring 빈 등록 변경 gss-client/gss-mcp-client/src/main/java/com/devoops/client/openai/OpenAiPrAnalysisClient.java	클래스에서 @Component 어노테이션 및 해당 import 제거 — 클래스는 공개 상태로 남아 있으나 더 이상 Spring의 컴포넌트 스캔으로 자동 등록되지 않음.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Dev as 개발자/빌드
  participant Spring as SpringContext
  participant Client as OpenAiPrAnalysisClient

  rect rgba(60,179,113,0.08)
    Dev->>Spring: 애플리케이션 시작 (기존)
    Spring->>Client: 컴포넌트 스캔 및 빈 생성 (@Component present)
    Note right of Client: 기존: 빈으로 등록됨
  end

  rect rgba(70,130,180,0.08)
    Dev->>Spring: 애플리케이션 시작 (변경 후)
    Spring-->>Client: 더 이상 자동 스캔/빈 생성 없음 (@Component removed)
    Note right of Spring: 변경: 수동 등록 또는 다른 구성 필요
  end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Out of Scope Changes Check	⚠️ Warning	Java 클래스의 @component 어노테이션 제거와 JSON 스키마 안내 구문의 문장 부호 변경은 코드블록 가독성 이슈와 직접 관련이 없어 범위를 벗어난 변경입니다.	어노테이션 제거 및 JSON 스키마 수정은 별도 PR로 분리하거나 해당 변경의 필요성을 설명해 주세요.
Description Check	⚠️ Warning	PR 설명에 템플릿의 '# 🚩 연관 이슈'와 '# 🔂 변경 내역' 섹션은 존재하지만 '변경 내역'에 실제 수정 내용이 누락되어 있어 완전하지 않습니다.	'# 🔂 변경 내역' 섹션에 YAML 구성 변경 및 Java 클래스 어노테이션 제거 등 구체적 변경 사항을 요약해 추가해 주세요.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	타이틀 "[FEAT] 코드블록 분리 프롬프팅 추가"는 PR에서 YAML 설정에 코드블록 분리 프롬프팅 기능을 도입한 핵심 변경사항을 간결하게 요약하고 있어 적절합니다.
Linked Issues Check	✅ Passed	PR에서 YAML 설정에 코드블록 사용 가이드를 추가해 이슈 #95의 '코드블록을 활용하여 질문 가독성 올리기' 요구사항을 충족하고 있습니다.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#95-code-block

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 17a1a56 and d058ac6.

📒 Files selected for processing (1)

• gss-client/gss-mcp-client/src/main/resources/application-mcp-client.yml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• gss-client/gss-mcp-client/src/main/resources/application-mcp-client.yml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build-and-push

---

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.}

[github-actions]

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit d058ac6.

♻️ This comment has been updated with latest results.

[github-actions]

📝 Test Coverage Report

Overall Project	NaN% NaN%	🍏

There is no coverage information present for the Files changed

[coderabbitai]

Actionable comments posted: 2

🧹 Nitpick comments (1)

gss-mcp-app/src/test/resources/application.yml (1)

70-70: 구두점 변경 사항을 확인하세요.

70번째 줄에서 JSON 스키마 안내 문구의 구두점이 ")."에서 ")::"로 변경되었습니다. 이는 의도적인 변경으로 보이며, application-mcp-client.yml 파일(33번째 줄)에서도 동일한 패턴이 적용되었습니다.

이중 콜론(::)을 사용한 의도가 명확하지 않으므로, 가독성 측면에서 이 변경이 실제로 개선되었는지 확인하는 것이 좋습니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0b0b0bd and 821bca1.

📒 Files selected for processing (3)

• gss-client/gss-mcp-client/src/main/java/com/devoops/client/claude/ClaudePrAnalysisClient.java (1 hunks)
• gss-client/gss-mcp-client/src/main/resources/application-mcp-client.yml (1 hunks)
• gss-mcp-app/src/test/resources/application.yml (2 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

gss-client/gss-mcp-client/src/main/java/com/devoops/client/claude/ClaudePrAnalysisClient.java (1)

gss-mcp-app/src/main/java/com/devoops/adaptor/PrAnalysisAdapter.java (1)

• Slf4j (18-40)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build-and-push

🔇 Additional comments (1)

gss-client/gss-mcp-client/src/main/java/com/devoops/client/claude/ClaudePrAnalysisClient.java (1)

17-20: LGTM! Spring 빈 등록이 적절합니다.

@Component 어노테이션을 추가하여 ClaudePrAnalysisClient를 Spring 관리 빈으로 등록했습니다. 이를 통해 PrAnalysisAdapter의 List<PrAnalysisClient> 의존성 주입이 정상적으로 작동합니다.