Skip to content

Phase 5: Remove remaining 51 unused React imports#2709

Merged
zanesq merged 2 commits intofix/typescript-errors-phase4-complex-typesfrom
fix/typescript-errors-phase5-more-unused-react
May 29, 2025
Merged

Phase 5: Remove remaining 51 unused React imports#2709
zanesq merged 2 commits intofix/typescript-errors-phase4-complex-typesfrom
fix/typescript-errors-phase5-more-unused-react

Conversation

@zanesq
Copy link
Copy Markdown
Contributor

@zanesq zanesq commented May 28, 2025

  • Remove unused React imports from 51 files across the entire codebase

  • Categories fixed:

    • UI Components: Box, CustomRadio, Select, Send, Stop, VertDots, X, icons
    • Main Components: GooseLogo, LoadingGoose, WelcomeView, GooseMessage, etc.
    • Settings Components: Models, Providers, Extensions sections
    • Modal Components: Various configuration and setup modals
    • Utility Files: suspense-loader, toasts
  • Changes made:

    • 21 files: Removed simple 'import React from react'
    • 24 files: Converted 'import React, { ... }' to 'import { ... }'
    • 6 files: Removed various React import patterns
  • Perfect prediction accuracy: eliminated exactly 51 errors (331 → 280)

  • All files use new JSX transform, no functionality impact

  • Total progress: 120+ errors eliminated across all phases (400+ → 280)

- Remove unused React imports from 51 files across the entire codebase
- Categories fixed:
  * UI Components: Box, CustomRadio, Select, Send, Stop, VertDots, X, icons
  * Main Components: GooseLogo, LoadingGoose, WelcomeView, GooseMessage, etc.
  * Settings Components: Models, Providers, Extensions sections
  * Modal Components: Various configuration and setup modals
  * Utility Files: suspense-loader, toasts

- Changes made:
  * 21 files: Removed simple 'import React from react'
  * 24 files: Converted 'import React, { ... }' to 'import { ... }'
  * 6 files: Removed various React import patterns

- Perfect prediction accuracy: eliminated exactly 51 errors (331 → 280)
- All files use new JSX transform, no functionality impact
- Total progress: 120+ errors eliminated across all phases (400+ → 280)
@zanesq zanesq requested a review from alexhancock May 28, 2025 22:55
@zanesq zanesq merged commit b1e683c into fix/typescript-errors-phase4-complex-types May 29, 2025
1 check passed
@zanesq zanesq deleted the fix/typescript-errors-phase5-more-unused-react branch May 29, 2025 15:26
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.

2 participants