Skip to content

Phase 5: Remove remaining 51 unused React imports - #2709

Merged
zanesq merged 2 commits into
fix/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 into
fix/typescript-errors-phase4-complex-typesfrom
fix/typescript-errors-phase5-more-unused-react

Conversation

@zanesq

@zanesq zanesq commented May 28, 2025

Copy link
Copy Markdown
Contributor
  • 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
@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