Skip to content

feat(report): add commit log collection from all repositories (#627)#629

Merged
Kewton merged 3 commits intodevelopfrom
feature/627-commit-log-in-report
Apr 5, 2026
Merged

feat(report): add commit log collection from all repositories (#627)#629
Kewton merged 3 commits intodevelopfrom
feature/627-commit-log-in-report

Conversation

@Kewton
Copy link
Copy Markdown
Owner

@Kewton Kewton commented Apr 5, 2026

Summary

  • git-utils.tsgetCommitsByDateRange() 関数追加(全ブランチ、日付範囲指定)
  • summary-prompt-builder.ts<commit_log> セクション構築を追加
  • daily-summary-generator.ts で全リポジトリのコミットログを収集しプロンプトに含める
  • レポート生成時にAIがIssue消化数・コミット数・作業内容を正確に把握可能に
  • 23テストケース追加

Closes #627

Test plan

  • npm run lint — PASS
  • npx tsc --noEmit — PASS
  • npm run test:unit — 6131 passed
  • npm run build — PASS

🤖 Generated with Claude Code

Kewton and others added 3 commits April 5, 2026 11:09
- Add CommitLogEntry and RepositoryCommitLogs types to src/types/git.ts
- Add MAX_COMMIT_LOG_LENGTH and GIT_LOG_TOTAL_TIMEOUT_MS constants
- Add withTimeout utility with fallback support to src/lib/utils.ts
- Add getCommitsByDateRange and collectRepositoryCommitLogs to git-utils.ts
- Extend sanitizeMessage to escape <commit_log> tags via ESCAPED_TAGS array
- Extend buildSummaryPrompt with optional commitLogs parameter
- Integrate commit log collection into daily-summary-generator
- Add comprehensive tests for all new functionality (106 tests pass)
- All static analysis checks passed (tsc, eslint)

Resolves #627

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Move TimeoutError class and withTimeout function after escapeHtml to
restore the broken JSDoc-to-function association for escapeHtml. Add
section separator comment consistent with codebase conventions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Kewton Kewton merged commit c1cccdc into develop Apr 5, 2026
5 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