Skip to content

Remove unnecessary exports#2020

Open
GregorShear wants to merge 1 commit into
mainfrom
claude/unexport-private
Open

Remove unnecessary exports#2020
GregorShear wants to merge 1 commit into
mainfrom
claude/unexport-private

Conversation

@GregorShear

Copy link
Copy Markdown
Contributor

No description provided.

@GregorShear GregorShear requested a review from a team as a code owner July 1, 2026 18:59
@GregorShear GregorShear changed the base branch from main to claude/code-health-checks July 1, 2026 19:00
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

✅ Code Health

Removes $\textcolor{green}{169}$ dead-code issues, introduces none.

49 Unused files

File imported nowhere — delete (or import) it.

     src/hooks/useDelay.ts
     src/hooks/useDraft.ts
     src/pages/NoGrants.tsx
     src/pages/OAuth.tsx
     src/services/encryption.ts
     src/types/global.ts
     src/types/vitest.ts
     src/components/graphs/TaskHoursByMonthGraph.tsx
     src/components/shared/RelativeAppBar.tsx
     src/components/tables/Link.tsx
…and 39 more

64 Unused exports $\textcolor{green}{-93}$

Exported symbol with no references outside its own file — un-export it, or delete it if unused entirely

$\textcolor{green}{-}$ src/context/Theme.tsx : sample_blue
$\textcolor{green}{-}$ src/context/Theme.tsx : successDark
$\textcolor{green}{-}$ src/context/Theme.tsx : infoMain
$\textcolor{green}{-}$ src/context/Theme.tsx : chipDraggableIndex
$\textcolor{green}{-}$ src/context/Theme.tsx : headerLinkIndex
$\textcolor{green}{-}$ src/context/Theme.tsx : alternativeDataGridHeader
$\textcolor{green}{-}$ src/context/Theme.tsx : connectorCardLogoBackground
$\textcolor{green}{-}$ src/context/Theme.tsx : wrappingTableCell
$\textcolor{green}{-}$ src/utils/connector-utils.ts : DEKAF_IMAGE_PREFIX
$\textcolor{green}{-}$ src/utils/dataPlane-utils.ts : dataPlaneFetcher_list
…and 147 more

26 Unused exported types $\textcolor{green}{-76}$

Exported type with no references outside its own file — un-export it, or delete it if unused entirely

$\textcolor{green}{-}$ src/utils/dataPlane-utils.ts : ErrorFlags
$\textcolor{green}{-}$ src/test/test-utils.tsx : AllTheProvidersProps
$\textcolor{green}{-}$ src/stores/Tables/Store.ts : StatsSchema
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : CaptureQueryWithSpec
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : MaterializationQuery
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : MaterializationQueryWithSpec
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : CollectionQuery
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : LiveSpecsExtQuery_DetailsForm
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : LiveSpecsExtQuery_DataPlaneAuthReq
$\textcolor{green}{-}$ src/api/liveSpecsExt.ts : LiveSpecsExtQuery_Latest
…and 92 more

14 Unused exported enum members

An enum member referenced nowhere

     src/services/supabase.ts : CONNECTOR_TAGS
     src/services/supabase.ts : DRAFTS_EXT
     src/services/supabase.ts : TASKS_BY_DAY
     src/stores/Tables/hooks.ts : accessGrants
     src/stores/Tables/hooks.ts : accessLinks
     src/stores/Tables/hooks.ts : billing
     src/stores/Tables/hooks.ts : connectors
     src/stores/Tables/hooks.ts : entitySelector
     src/stores/Tables/hooks.ts : prefixes
     src/stores/Tables/hooks.ts : prefixAlerts
…and 4 more

5 Unused dependencies

In package.json but never imported

     package.json : @mui/lab
     package.json : @testing-library/jest-dom
     package.json : @urql/exchange-retry
     package.json : logrocket-react
     package.json : stripe

3 Unused devDependencies

In package.json devDependencies but never used

     package.json : @types/logrocket-react
     package.json : @types/react-inspector
     package.json : sharp

@GregorShear GregorShear force-pushed the claude/unexport-private branch 2 times, most recently from e916d98 to c2dceea Compare July 1, 2026 20:33
@GregorShear GregorShear force-pushed the claude/code-health-checks branch from 4b402dc to f34b363 Compare July 1, 2026 20:37
@GregorShear GregorShear force-pushed the claude/unexport-private branch from c2dceea to ae37332 Compare July 1, 2026 20:44
@GregorShear GregorShear force-pushed the claude/code-health-checks branch 4 times, most recently from a4152a1 to 604416d Compare July 2, 2026 20:20
Base automatically changed from claude/code-health-checks to main July 2, 2026 21:22
@GregorShear GregorShear force-pushed the claude/unexport-private branch from ae37332 to 9755fab Compare July 2, 2026 21:38
@GregorShear GregorShear requested a review from jshearer July 2, 2026 21:41
@GregorShear

Copy link
Copy Markdown
Contributor Author

What's special about this diff?

It represents most of the unnecessarily-exported-but-still-used-privately symbols - i.e. we can just drop the export keyword without running into the linter.

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.

1 participant