Skip to content

fix: auto-fix CI formatting failures for PR #1402 - #1421

Closed
github-actions[bot] wants to merge 1 commit into
devfrom
claude-fix-pr-1402-31575987931
Closed

fix: auto-fix CI formatting failures for PR #1402#1421
github-actions[bot] wants to merge 1 commit into
devfrom
claude-fix-pr-1402-31575987931

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

CI Auto-Fix

Original PR: #1402
Failed CI Run: PR Build Check

Root Cause

The Code Quality Check job failed at bun run format:check. PR #1416 (availability monitoring) merged two files with non-Biome-compliant formatting into dev. Any PR merging against the updated dev inherits these formatting errors, including PR #1402.

Fixes Applied

File Fix Type
src/lib/availability/availability-service.ts Biome line-wrapping in floorToUtcMinute format
tests/unit/lib/availability/projection-worker.test.ts Biome line-wrapping in mock declarations and expect calls format

Changes are purely line-wrapping (whitespace/indentation) produced by biome format --write. No logic changes.

Not Auto-Fixable

The following pre-existing issues are unrelated to this CI failure (the failed job only ran typecheck + format:check, not lint) and were left untouched:

File Error Reason
biome.json Schema version mismatch (2.5.6 vs CLI 2.5.8) Config change; out of scope for this format fix
src/lib/availability/index.ts organizeImports sort order Lint (not format) issue; pre-existing, separate concern

Verification

  • bun run typecheck passes
  • bun run format:check passes
  • No logic changes made (diff is whitespace-only line wrapping)

Auto-generated by Claude AI

Greptile Summary

This PR applies Biome-generated line wrapping to availability service code and its projection-worker tests without changing runtime or test behavior.

  • Reformats the Date.UTC call in floorToUtcMinute.
  • Reformats transaction mocks and an assertion in projection-worker tests.

Confidence Score: 5/5

The PR appears safe to merge because its changes are behavior-preserving formatting updates.

The rewritten expressions retain the same calls, arguments, callbacks, and assertions, with no functional changes or accepted review findings.

Important Files Changed

Filename Overview
src/lib/availability/availability-service.ts Reformats the existing Date.UTC expression without changing its arguments or behavior.
tests/unit/lib/availability/projection-worker.test.ts Reformats existing mock declarations and an assertion without changing test semantics.

Reviews (1): Last reviewed commit: "fix: auto-fix CI formatting failures" | Re-trigger Greptile

Fixed:
- src/lib/availability/availability-service.ts: Biome line-wrapping in floorToUtcMinute
- tests/unit/lib/availability/projection-worker.test.ts: Biome line-wrapping in mock declarations and expect calls

CI Run: https://github.com/ding113/claude-code-hub/actions/runs/31575625118

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant