Skip to content

feat: 복잡도 분석 Worker 분리 및 테스트 추가 (#8)#22

Open
devchanki wants to merge 5 commits intomainfrom
8-big-o
Open

feat: 복잡도 분석 Worker 분리 및 테스트 추가 (#8)#22
devchanki wants to merge 5 commits intomainfrom
8-big-o

Conversation

@devchanki
Copy link
Copy Markdown
Contributor

@devchanki devchanki commented Apr 10, 2026

Summary

  • PR의 솔루션 파일에 대해 시간/공간 복잡도를 AI로 자동 분석하여 댓글로 제공
  • 유저 주석(TC/SC)이 있으면 실제 분석값과 비교 테이블로 검증 (✅/❌)
  • 멀티 풀이 지원 (<details> 접기), 개선 제안 포함
  • analyzeComplexity/internal/complexity-analysis 엔드포인트로 분리하여 독립 subrequest 예산(50) 확보
  • 복잡도 분석 핸들러 단위 테스트 24개 + subrequest 회귀 테스트 추가

변경 파일

  • handlers/complexity-analysis.js — 복잡도 분석 핸들러 (신규)
  • handlers/complexity-analysis.test.js — 단위 테스트 24개 (신규)
  • handlers/internal-dispatch.js/internal/complexity-analysis 엔드포인트 추가
  • handlers/webhooks.js — 3개 AI 핸들러 모두 self-fetch dispatch로 통합
  • handlers/webhooks.test.js — dispatch 횟수 검증 업데이트 (2→3)
  • tests/subrequest-budget.test.js — analyzeComplexity subrequest 회귀 테스트 추가

Test plan

  • bun test handlers/ 전체 통과 (51개)
  • bun test tests/ 전체 통과 (3개)
  • INTERNAL_SECRET 설정 후 PR opened 시 복잡도 분석 댓글 생성 확인
  • INTERNAL_SECRET 미설정 시 기존 방식(in-process)으로 폴백 확인

Closes #8

🤖 Generated with Claude Code

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 10, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 74009f6 Commit Preview URL Apr 16 2026, 02:14 PM

devchanki and others added 3 commits April 16, 2026 22:36
analyzeComplexity를 기존 tagPatterns, postLearningStatus와 동일하게
self-fetch + waitUntil 패턴으로 /internal/complexity-analysis 엔드포인트를
통해 별도 Worker invocation으로 디스패치하도록 변경.
기존에 원래 invocation에서 직접 실행되던 구조를 통합하여 3개 AI 핸들러
모두 독립적인 subrequest 예산(50)을 가짐.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
skip 조건, OpenAI 응답 파싱, 댓글 포맷(단일/멀티 풀이, 유저 주석 유무),
댓글 upsert(POST/PATCH), 파일 크기 제한, 에러 처리 등 24개 테스트 케이스 작성

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@devchanki devchanki changed the title 여러번 LLM 을 호출하던 방식 수정 feat: 시간/공간 복잡도 자동 분석 (#8) Apr 16, 2026
@devchanki devchanki requested a review from sounmind April 16, 2026 14:16
@devchanki devchanki changed the title feat: 시간/공간 복잡도 자동 분석 (#8) feat: 복잡도 분석 Worker 분리 및 테스트 추가 (#8) Apr 16, 2026
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.

시간/공간 복잡도 자동 분석

2 participants