Skip to content

Auto-fix CI failures for PR #1062#1064

Closed
github-actions[bot] wants to merge 1 commit into
feat/status-page-ui-ux-optimization-20260422from
claude-fix-pr-1062-24759380574
Closed

Auto-fix CI failures for PR #1062#1064
github-actions[bot] wants to merge 1 commit into
feat/status-page-ui-ux-optimization-20260422from
claude-fix-pr-1062-24759380574

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Apr 22, 2026

CI Auto-Fix

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

Fixes Applied

File Fix Type
src/lib/public-status/config.ts Corrected ternary operator formatting to match Biome style format

Error Details

The CI failed due to a formatting issue in src/lib/public-status/config.ts (lines 100-103). The ternary operator was incorrectly formatted across multiple lines instead of being on the same line as the variable declaration.

Before (incorrect):

const providerTypeOverride =
  isObjectEntry
    ? sanitizeProviderType(...)
    : undefined;

After (correct):

const providerTypeOverride = isObjectEntry
  ? sanitizeProviderType(...)
  : undefined;

Verification

  • bun run format:check passes
  • bun run lint passes
  • No logic changes made

Auto-generated by Claude AI

Greptile Summary

This PR applies a single formatting fix in src/lib/public-status/config.ts to satisfy Biome's ternary operator style rule: the providerTypeOverride assignment now begins its ternary on the same line as the = token rather than on the next line. No logic, behaviour, or interface changes are included.

Confidence Score: 5/5

Safe to merge — pure formatting change with no logic impact.

Only a single cosmetic ternary-operator line-break adjustment is present; all findings are P2 or below. No correctness, security, or data-integrity concerns.

No files require special attention.

Important Files Changed

Filename Overview
src/lib/public-status/config.ts Formatting-only fix: moved ternary operator for providerTypeOverride onto the same line as the variable declaration to satisfy Biome style rules. No logic changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[sanitizePublicModels called] --> B{entry is object?}
    B -- yes --> C[isObjectEntry = true]
    B -- no --> D[isObjectEntry = false]
    C --> E[extract modelKey from entry.modelKey]
    D --> F[extract modelKey from entry string]
    E & F --> G{modelKey valid and unique?}
    G -- no --> H[skip entry]
    G -- yes --> I[add to seen set]
    I --> J{isObjectEntry?}
    J -- yes --> K[providerTypeOverride = sanitizeProviderType]
    J -- no --> L[providerTypeOverride = undefined]
    K & L --> M[push to normalized result]
Loading

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

Fixed formatting issue in src/lib/public-status/config.ts:
- Corrected ternary operator formatting to match Biome style

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

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ding113 ding113 deleted the branch feat/status-page-ui-ux-optimization-20260422 April 22, 2026 04:08
@ding113 ding113 closed this Apr 22, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Claude Code Hub Roadmap Apr 22, 2026
@ding113 ding113 deleted the claude-fix-pr-1062-24759380574 branch May 13, 2026 06:23
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