Auto-fix CI failures for PR #1039

[github-actions]

CI Auto-Fix

Original PR: #1039
Failed CI Run: Non-Main Branch CI/CD

Fixes Applied

File	Fix	Type
biome.json	Updated schema version from 2.4.4 to 2.4.12	config
src/actions/users.ts	Removed unused suppression comment	lint
23 other files	Auto-fixes for optional chains, imports, unused vars	lint/format

Changes Summary

1. Biome config: Updated schema version to match CLI version (2.4.12)
2. Code fixes: Applied biome auto-fixes including:
   • Optional chain conversions (e.g.,
     !payload || !payload.length
     ->
     !payload?.length
     )
   • Unused variable prefixing with underscore
   • Import organization
   • Removed ineffective suppression comment

Remaining Warnings (Non-blocking)

7 warnings remain in:

• src/components/ui/map.tsx: useExhaustiveDependencies warnings (pre-existing, intentionally disabled via eslint-disable comments)
• src/app/[locale]/dashboard/logs/_components/usage-logs-view-virtualized.tsx: useExhaustiveDependencies warning
• tests/unit/lib/proxy-agent/agent-pool.test.ts: unused suppression comment

These are warnings only and do not block CI.

Verification

• bun run typecheck passes
• bun run lint passes (exit code 0)
• No logic changes made

---

Auto-generated by Claude Code

Greptile Summary

This is a CI auto-fix PR that applies mechanical linting and formatting corrections produced by Biome 2.4.12 to address failures from PR #1039. Changes include updating the Biome schema version, optional-chain conversions, import organization, and prefixing intentionally-unused variables with _. No business logic is altered.

Confidence Score: 5/5

Safe to merge — all changes are mechanical lint/format fixes with no logic modifications.

Every change in this PR is a Biome auto-fix: schema version bump, optional-chain conversions that are semantically equivalent, unused-variable underscore prefixing, and import organisation. No business logic, data flow, or security-sensitive code paths were altered. The constant-time comparison rename preserves the timing-attack guarantee. All remaining warnings are pre-existing and non-blocking.

No files require special attention.

Important Files Changed

Filename	Overview
biome.json	Schema URL updated from 2.4.4 to 2.4.12 to match the installed Biome CLI version — straightforward and correct.
src/lib/security/constant-time-compare.ts	Renamed loop accumulator to _dummy to satisfy the unused-variable lint rule. The XOR accumulation loop body still executes, preserving the constant-time guarantee against timing attacks.
src/actions/users.ts	Removed an ineffective suppression comment; no logic changes to the large user-management action file.
src/app/v1/_lib/proxy/provider-selector.ts	Prefixed the deprecated parameter targetProviderType → _deprecatedTargetProviderType to satisfy the unused-variable rule; no functional change.
src/lib/provider-testing/validators/content-validator.ts	Optional-chain applied to responseBody?.trim() guard; responseBody is typed as string so the chain is a no-op but harmless.
src/app/v1/_lib/proxy/message-service.ts	Import and optional-chain formatting fixes only; no logic changes.
src/app/v1/_lib/proxy/version-guard.ts	Lint/import organisation fixes only; ProxyVersionGuard logic unchanged.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Biome 2.4.12 auto-fix] --> B[biome.json\nSchema version bump]
    A --> C[Source files\n22 files]
    C --> D[Optional-chain\nconversions]
    C --> E[Unused-variable\nunderscore prefix]
    C --> F[Import\norganisation]
    C --> G[Remove ineffective\nsuppression comment]
    D --> H[No semantic change]
    E --> H
    F --> H
    G --> H
    H --> I[CI passes ✓]

Loading

_{Reviews (1): Last reviewed commit: "chore: fix CI linting errors" | Re-trigger Greptile}