Conversation
lkostrowski
commented
Feb 23, 2026
- Run knip, removed files marked as unused → all pass
- Update knip to not report mutations/queries
- Auto-fixed some eslint issues
|
There was a problem hiding this comment.
Pull request overview
This PR performs cleanup by removing unused files identified by the knip tool and applying automated eslint fixes for type imports. The changes focus on eliminating dead code while ensuring no breaking changes to the codebase.
Changes:
- Removed unused component files and barrel exports (index.ts files) from orders, icons, sidebar, and other components
- Updated knip configuration to use wildcard patterns for queries and mutations files
- Applied eslint auto-fixes to add
typekeyword to type-only imports across Playwright test files
Reviewed changes
Copilot reviewed 67 out of 67 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/orders/components/OrderUsedGiftCards/index.ts | Removed barrel export; component imported directly from .tsx file |
| src/orders/components/OrderTransactionsSection/index.ts | Removed barrel export; component imported directly from .tsx file |
| src/orders/components/OrderSummaryCard/*.ts | Removed entire unused OrderSummaryCard component and related files |
| src/orders/components/OrderPaymentSummaryCard/*.ts | Removed unused files; OrderPaymentStatusPill subcomponent still exists and is used |
| src/orders/components/OrderPaymentDialog/*.tsx | Removed entire unused OrderPaymentDialog component |
| src/icons/*.tsx | Removed unused icon components (Edit, ArrowDropdown) |
| src/components/Sidebar/menu/EnvironmentLink.tsx | Removed unused component |
| src/components/OverflowTooltip/*.tsx | Removed entire unused OverflowTooltip component |
| src/components/ModalFilters/ModalFiltersProvider.tsx | Removed unused provider component |
| src/components/ListItemLink/*.tsx | Removed entire unused ListItemLink component |
| src/components/Combobox/hooks/useComboboxEmptyOption.ts | Removed unused hook |
| src/attributes/components/AttributeValues/index.ts | Removed barrel export; component imported directly from .tsx file |
| .knip.json | Updated patterns from queries.ts to queries*.ts and mutations.ts to mutations*.ts for better coverage |
| playwright/**/*.ts | Applied eslint auto-fix to add type keyword to type-only imports (Page, APIRequestContext, etc.) |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6358 +/- ##
==========================================
+ Coverage 42.71% 42.85% +0.13%
==========================================
Files 2534 2516 -18
Lines 44006 43864 -142
Branches 10417 10413 -4
==========================================
Hits 18799 18799
+ Misses 23868 23737 -131
+ Partials 1339 1328 -11 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|