Skip to content

feat: 패턴 태깅 댓글에 복잡도 분석 합본 (closes #31)#43

Merged
lkhoony merged 1 commit intomainfrom
31-merger-pattern-big-o
May 1, 2026
Merged

feat: 패턴 태깅 댓글에 복잡도 분석 합본 (closes #31)#43
lkhoony merged 1 commit intomainfrom
31-merger-pattern-big-o

Conversation

@lkhoony
Copy link
Copy Markdown
Contributor

@lkhoony lkhoony commented May 1, 2026

Summary

  • 알고리즘 패턴 태깅과 시간/공간 복잡도 분석 결과를 파일별 review comment 한 곳에 합본으로 게시
  • 두 분석을 같은 invocation 안에서 병렬 실행해 디스패치 분리로 인한 타이밍 의존성을 제거
  • 구버전이 남긴 단독 복잡도 issue comment 는 통합 핸들러 첫 동작 시 자동 정리

변경 내용

  • handlers/complexity-analysis.js: callComplexityAnalysis / renderComplexitySection 만 export, 오케스트레이션·issue comment upsert 제거
  • handlers/tag-patterns.js: 모든 raw 를 사전에 한 번 다운로드해 패턴 분석 루프와 복잡도 OpenAI 1콜이 공유, 댓글 본문에 복잡도 섹션 append. 레거시 issue comment 마이그레이션 추가
  • handlers/webhooks.js / handlers/internal-dispatch.js: complexity-analysis 디스패치/라우팅 제거 (3개 → 2개 invocation)
  • 테스트: tests/tag-patterns.test.js 신규(skip 가드, 합본, 마이그레이션, 패턴 정리), tests/subrequest-budget.test.js 5파일 기준 22 → 25 갱신

subrequest 예산

5파일 기준 합본 후 25회 (50 한도 안). 12파일 이상에서는 한도 근접하지만, 기존 패턴 태깅 자체도 13파일+에서 cliff 가 있어 새로 도입된 위험은 아니다.

Test plan

  • bun test handlers/ 75/75 통과
  • bun test tests/ 17/17 통과
  • 실 PR 에 빈 커밋 push 해 합본 review comment 1개에 두 섹션 모두 표시되는지 확인
  • 기존 단독 복잡도 issue comment 가 자동 삭제되는지 확인 (구버전 PR 에서)

🤖 Generated with Claude Code

복잡도 분석 결과를 별도 issue comment 가 아니라 패턴 태깅의 파일별
review comment 한 곳에 함께 게시한다. 두 분석을 같은 invocation 안에서
병렬 실행하여 두 댓글이 시점 차이로 따로 보이던 문제와 별도 디스패치로
인한 타이밍 의존성을 모두 제거한다.

- complexity-analysis: callComplexityAnalysis / renderComplexitySection
  만 export 하고 오케스트레이션·issue comment upsert 는 제거
- tag-patterns: 모든 raw 를 사전에 한 번 다운로드해 패턴 분석 루프와
  복잡도 OpenAI 1콜이 공유, 댓글 본문에 복잡도 섹션 append
- 구버전 단독 복잡도 issue comment 는 첫 동작 시 자동 정리
- webhooks/internal-dispatch: complexity-analysis 디스패치/라우팅 제거

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@lkhoony lkhoony linked an issue May 1, 2026 that may be closed by this pull request
@cloudflare-workers-and-pages
Copy link
Copy Markdown

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 8bf34b1 Commit Preview URL May 01 2026, 03:51 AM

@lkhoony lkhoony merged commit f3c1f69 into main May 1, 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