Skip to content

feat: batch-score commits after L1 scan and fix duplicate board assigment - #240

Merged
rmyers merged 1 commit into
mainfrom
239-batch-commit-scoring
Jul 9, 2026
Merged

rmyers merged 1 commit into
mainfrom
239-batch-commit-scoring

Conversation

@rmyers

@rmyers rmyers commented Jul 8, 2026

Copy link
Copy Markdown
Member

Refactor webhook handler to collect created commits and score them in batch after L1 scan completes, replacing per-commit AddCommit calls during the processCommits loop.

Webhook handler changes:

  • scheduleL1Scan now receives createdCommits and calls AddCommit for each after the scan finishes (instead of during the per-commit loop)
  • If scanner is not configured, commits are scored synchronously (same pre-existing behavior, no regression)
  • L1 scan timeout reduced from 5min to 1min
  • Removed inline AddCommit calls from processCommits

Scanner changes:

  • Removed gameService dependency (no longer needed after removing RefreshPlayerAfterScan at end of RunScan)
  • Simplified NewScanner constructor

Game service:

  • Fixed assignPlayerBoards to skip re-assigning a board that's already in one of the player's 3 slots (previously, multiple commits from the same project could fill slots 1, 2, and 3 with the same board)
  • Removed RefreshPlayerAfterScan (now handled by webhook handler)
  • Added test for duplicate board assignment fix

Infrastructure:

  • Added .kilo/ config (Kilo AI skills symlink)

Closes: #239

…nment

Refactor webhook handler to collect created commits and score them
in batch after L1 scan completes, replacing per-commit AddCommit
calls during the processCommits loop.

Webhook handler changes:
- scheduleL1Scan now receives createdCommits and calls AddCommit for
  each after the scan finishes (instead of during the per-commit loop)
- If scanner is not configured, commits are scored synchronously (same
  pre-existing behavior, no regression)
- L1 scan timeout reduced from 5min to 1min
- Removed inline AddCommit calls from processCommits

Scanner changes:
- Removed gameService dependency (no longer needed after removing
  RefreshPlayerAfterScan at end of RunScan)
- Simplified NewScanner constructor

Game service:
- Fixed assignPlayerBoards to skip re-assigning a board that's
  already in one of the player's 3 slots (previously, multiple
  commits from the same project could fill slots 1, 2, and 3 with
  the same board)
- Removed RefreshPlayerAfterScan (now handled by webhook handler)
- Added test for duplicate board assignment fix

Infrastructure:
- Added .kilo/ config (Kilo AI skills symlink)

Closes: #239
@rmyers
rmyers merged commit 5571a7e into main Jul 9, 2026
1 check passed
@rmyers
rmyers deleted the 239-batch-commit-scoring branch July 9, 2026 00:06
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.

Batch commit game scoring in AddCommit

1 participant