Skip to content

Auto-fix CI failures for PR #803#805

Closed
github-actions[bot] wants to merge 1 commit intofeat/security-auth-overhaulfrom
claude-fix-pr-803-22132410171
Closed

Auto-fix CI failures for PR #803#805
github-actions[bot] wants to merge 1 commit intofeat/security-auth-overhaulfrom
claude-fix-pr-803-22132410171

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Feb 18, 2026

CI Auto-Fix

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

Fixes Applied

File Fix Type
tests/unit/settings/providers/adaptive-thinking-editor.test.tsx Removed duplicate test cases outside describe block Syntax error
tests/unit/settings/providers/thinking-budget-editor.test.tsx Removed duplicate test cases outside describe block Syntax error

Issue Description

Both test files had duplicate test cases that were accidentally placed outside the describe block after it was closed. This caused parse errors:

  • Expected a statement but instead found '})'

The fix removes the duplicate test code that was outside the proper scope.

Not Auto-Fixable

N/A - all issues were safe to fix mechanically.

Verification

  • bun run format passes
  • bunx biome check passes on affected files
  • No logic changes made (only removed duplicate/unreachable code)

Auto-generated by Claude AI

Greptile Summary

This PR fixes CI failures by removing duplicate test cases that were accidentally placed outside the describe block in two test files, and applying code formatting fixes.

  • Removed orphaned test cases after the closing }); of the describe block in both adaptive-thinking-editor.test.tsx and thinking-budget-editor.test.tsx, which were causing parse errors (Expected a statement but instead found '})')
  • Applied automated formatting: trailing whitespace cleanup, line wrapping for long strings, and consistent arrow function parameter style
  • No logic changes — all test assertions inside the describe blocks are preserved

Confidence Score: 5/5

  • This PR is safe to merge — it only removes duplicate/unreachable test code and applies formatting fixes.
  • The changes are strictly mechanical: removing orphaned test code that was outside the describe block (and therefore unreachable/invalid) and reformatting for style consistency. No test logic was altered, added, or removed from the valid test suite. Both files are test-only, so there is zero production code risk.
  • No files require special attention.

Important Files Changed

Filename Overview
tests/unit/settings/providers/adaptive-thinking-editor.test.tsx Removed duplicate test cases that were outside the describe block (causing parse errors), plus formatting cleanup (trailing whitespace, line wrapping, arrow function style).
tests/unit/settings/providers/thinking-budget-editor.test.tsx Removed duplicate test cases outside the describe block (causing parse errors), reformatted long lines, and adjusted code style for consistency. No logic changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Original test files<br/>(PR #803)"] --> B["Duplicate test cases placed<br/>outside describe block"]
    B --> C["CI fails: parse error<br/>'Expected statement, found })'"]
    C --> D["Auto-fix commit 902b4e37<br/>Remove orphaned test code"]
    D --> E["Auto-fix commit 208193c6<br/>Format cleanup (whitespace, line wrapping)"]
    E --> F["CI passes:<br/>bun run format ✓<br/>bunx biome check ✓"]
Loading

Last reviewed commit: 208193c

Fixed syntax errors in test files that caused CI format check to fail:
- tests/unit/settings/providers/adaptive-thinking-editor.test.tsx
- tests/unit/settings/providers/thinking-budget-editor.test.tsx

Both files had duplicate test cases placed outside the describe block
after it was closed, causing parse errors.

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

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

Comments