Skip to content

fix(code-review): log repo size#3918

Merged
alex-alecu merged 9 commits into
mainfrom
fix/code-review-repository-size-logging
Jun 11, 2026
Merged

fix(code-review): log repo size#3918
alex-alecu merged 9 commits into
mainfrom
fix/code-review-repository-size-logging

Conversation

@alex-alecu

Copy link
Copy Markdown
Contributor

Summary

This adds a safe size check before a cloud code review starts. When the review session is ready, the logs now show the review id, both session ids, and the repo size if the provider gave one. If the size check fails, the review still starts and logs null. Retry attempts keep the same size value so later logs can be matched.

Verification

N/A - this is a backend log-only change with no screen to click through.

Visual Changes

N/A

Reviewer Notes

The size is the value GitHub or GitLab reports for stored repo data. It is not the exact amount cloned over the network.

@kilo-code-bot

kilo-code-bot Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The incremental changes since the last review (commits 66623fd and 713679f) cleanly inline the lookupRepositorySize helper into both the GitHub and GitLab branches and update tests accordingly — no regressions or new issues introduced.

Files Reviewed (10 files)
  • apps/web/src/lib/code-reviews/triggers/prepare-review-payload.ts — refactored (helper inlined, behavior unchanged)
  • apps/web/src/lib/code-reviews/triggers/prepare-review-payload.test.ts — updated to match refactored code
  • apps/web/src/lib/integrations/platforms/github/adapter.ts — no change since last review
  • apps/web/src/lib/integrations/platforms/github/fetch-root-text-file-at-ref.test.ts — no change since last review
  • apps/web/src/lib/integrations/platforms/gitlab/adapter.ts — no change since last review
  • apps/web/src/lib/integrations/platforms/gitlab/adapter.test.ts — no change since last review
  • services/code-review-infra/src/code-review-orchestrator.ts — no change since last review
  • services/code-review-infra/src/index.ts — no change since last review
  • services/code-review-infra/src/types.ts — no change since last review
  • services/code-review-infra/test/integration/code-review-orchestrator.test.ts — no change since last review

Reviewed by claude-4.6-sonnet-20260217 · 1,127,914 tokens

Review guidance: REVIEW.md from base branch main

Comment thread apps/web/src/lib/integrations/platforms/github/adapter.ts Outdated
Comment thread apps/web/src/lib/integrations/platforms/gitlab/adapter.ts Outdated
Comment thread apps/web/src/lib/code-reviews/triggers/prepare-review-payload.ts Outdated
Comment thread apps/web/src/lib/code-reviews/triggers/prepare-review-payload.ts Outdated
@alex-alecu alex-alecu merged commit 336fb66 into main Jun 11, 2026
16 checks passed
@alex-alecu alex-alecu deleted the fix/code-review-repository-size-logging branch June 11, 2026 12:04
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