Skip to content

ci: AI 리뷰 설정 초안 - #4

Merged
wantkdd merged 5 commits into
mainfrom
ci/ai-review
Aug 9, 2026
Merged

ci: AI 리뷰 설정 초안#4
wantkdd merged 5 commits into
mainfrom
ci/ai-review

Conversation

@wantkdd

@wantkdd wantkdd commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

작업내용

  • 루트 CI에서 AI·Server·Web 품질 검사를 변경 영역 기준으로 실행하도록 통합
  • 기계 검사 통과 후 실행되는 AI 리뷰 job과 인라인 리뷰·PR별 문서 기록 하네스 추가
  • 파트별 중복 .github 설정 제거
  • 월별 decisions 경로, 공통 커밋 규칙, 브랜치 이름 규칙 정리

포함된 영역

  • FE
  • BE
  • AI
  • docs
  • infra
  • 직접 작성

예측 (작업 초기 PR 생성 시 작성)

  • 예상 소요: 최초 CI 10~20분
  • 막힐 것 같은 곳: 기존 파트별 빌드 환경 차이, AI API 응답 및 토큰 권한
  • 전문 밖 영역 중 걱정되는 것: AI·Server·Web 전체 품질 검사가 main 기준에서도 안정적으로 통과하는지

결과 (머지 전 작성)

  • 실제 소요: 최종 CI 약 2분
  • 실제로 막힌 곳: 로컬 Docker 데몬 미실행, secret-bearing AI job이 trusted base 스크립트를 실행하는 부트스트랩 특성
  • 배운 것: 공통 CI 경로 변경 시 세 파트 검사가 모두 실행되며, 새 인라인·문서 게시 기능의 실환경 확인은 이 설정을 main에 반영한 다음 PR에서 가능

리뷰 요청

  • quality gate를 Branch protection required check로 사용할지
  • AI 인라인 리뷰와 docs/pr-reviews/PR-{번호}.md 누적 방식
  • 파트별 중복 .github 삭제 범위
  • 브랜치 이름을 /으로 통일한 규칙

메모

  • 로컬 검증: AI 리뷰 도구 47 tests, AI Ruff, Server Gradle build, Web Biome/build 통과
  • GitHub CI: detect changes, AI review tooling tests, AI·Server·Web quality, AI review, quality gate 모두 통과
  • GitHub의 AI Docker 이미지 빌드도 통과
  • PR #4에서는 보안상 main의 기존 AI 리뷰 스크립트를 실행하므로 새 인라인·문서 게시 기능은 단위 테스트로 검증

wantkdd added 3 commits August 9, 2026 22:33
Constraint: GitHub 공통 자동화와 템플릿은 루트 .github에서 관리
Rejected: 파트별 .github 유지 | 중복 설정이 모노레포 정본을 흐림
Confidence: high
Scope-risk: moderate
Directive: 파트 자동화와 템플릿은 루트 .github에만 추가
Tested: 파트별 .github 제거와 루트 CI 및 템플릿 존재 확인
Not-tested: 통합 CD 워크플로는 아직 없음
Constraint: 무료 모델을 사용하되 secret-bearing job은 신뢰된 base 스크립트만 실행
Rejected: GITHUB_TOKEN으로 리뷰 문서 푸시 | 후속 CI가 실행되지 않아 최신 SHA 검사가 누락됨
Confidence: high
Scope-risk: broad
Directive: AI_REVIEW_PUSH_TOKEN은 WhyLog 저장소의 Contents read-write로만 제한
Tested: 단위 테스트 47개, Ruff lint-format, Python compile, CI YAML 파싱
Not-tested: 실제 PR 인라인 위치와 PAT synchronize 재실행은 smoke PR에서 확인 필요
Constraint: 결정 기록은 월 단위로 누적하고 커밋 메시지는 모노레포 공통 형식을 사용
Rejected: AGENTS에서 최신 월 파일 직접 참조 | 매달 공통 링크를 수정해야 함
Confidence: high
Scope-risk: narrow
Directive: 새 월 decisions 파일을 만들 때 docs/decisions/README.md 인덱스도 갱신
Tested: 2026-07 결정 내용 동일성, 문서 링크, AGENTS 150줄 제한, diff 무결성
@vercel

vercel Bot commented Aug 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
whylog Ready Ready Preview Aug 9, 2026 1:52pm

@wantkdd wantkdd self-assigned this Aug 9, 2026
Constraint: PR 체크 목록에서 서비스 AI 검사와 코드 리뷰 도구 검사를 구분해야 함
Confidence: high
Scope-risk: narrow
Tested: AI 리뷰 하네스 단위 테스트 47개 통과
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

WhyLog AI 리뷰

결과: ✅ 차단 항목 없음 · 모델: Google gemini-3.6-flash

루트 CI에 AI 리뷰 하네스(인라인 리뷰 및 PR 리뷰 문서 관리) 통합, Decisions 연/월별 구조 분리, 중복 .github 설정 제거 및 브랜치/커밋 규칙 정비가 정상적으로 적용되었습니다.

차단

없음

제안

없음

이 코멘트는 새 실행 때 갱신됩니다. 차단 항목은 사람이 타당성을 확인한 뒤 수정하세요.

Constraint: 팀이 Issue를 생성하지 않으므로 번호 기반 브랜치 규칙을 사용할 수 없음
Rejected: PR 번호를 브랜치명에 사용 | PR 생성 전 번호를 알 수 없고 열린 PR의 head rename은 안전하지 않음
Confidence: high
Scope-risk: narrow
Directive: 작업 브랜치는 type/영문-kebab-case 형식만 사용
Tested: 루트 규칙과 서버 README 참조 일치, AGENTS.md 150줄 이하, diff 무결성
@wantkdd
wantkdd marked this pull request as ready for review August 9, 2026 13:55
@wantkdd
wantkdd merged commit b239259 into main Aug 9, 2026
9 checks passed
@wantkdd
wantkdd deleted the ci/ai-review branch August 9, 2026 15:43
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