Reduce unused exports #164935
Labels
debt
Code quality issues
Comments
|
Created #165070 for this. The other ones that are listed in search are actually used in some code for the web, so I'll keep them exported. |
chrmarti
added a commit
that referenced
this issue
Nov 1, 2022
|
|
|
The lint rule does not seem to be good at identifying things that are transitively used. E.g. if I have something like this type A = foo;
type B = bar;
type C = A | Bif type C is used, I want both A and B to be exported so consumers can refer to them if needed (even if no one is doing so at the moment). Anyhow, I'll clean up others... |
connor4312
added a commit
that referenced
this issue
Dec 12, 2022
connor4312
added a commit
that referenced
this issue
Jan 6, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
@rebornix and I were wondering if there is any dead code in the VS Code codebase, and one idea was to detect unused exports across the codebase, since unused variables are flagged by the compiler but unused exports are not.
I ran
ts-unused-exportson our sources and generated the following list. I've already done a pass to remove some false positives but apologies if several remain.NB: The primary purpose of this issue is to identify unused functions and classes which aren't being picked up by the compiler because they're exported and which can be safely deleted from our sources to reduce our bundle size. A secondary benefit is to remove unused types and interfaces, e.g. anything which was exported at one point and then ceased to be referenced from other files (which more impacts compile time for dev workflows and pipeline runs, and maybe TS server analysis time, but has no direct impact on bundle size since types are erased).
If you export a type or interface that is not currently actively imported, but is used in the public API of a class that you expose, then it's quite likely that you want to keep that type/interface exported and need not un-export it as part of this issue.
Please feel free to remove your assignment if you find that this isn't valuable.
notebookEditorBackgroundIWorkerFileSystemFileHandlecreateLocalFileSearchSimpleWorkerIQuickInputWidgetStylesIChangeEOLEntryWalkThroughModelIObjectTreeViewStateINotebookRendererContributionMirroredTestCollectionICustomWebviewViewDescriptorISuggestEnabledHistoryOptionsIMemoryRangeIValidMemoryRangeIUnreadableMemoryRangeIErrorMemoryRangeEnablementBaseBreakpointMemoryINotebookRendererContributionisTenativeCharacterPredictionPredictionTimelineterminalsExtPointHideTestActionUnhideTestActionDebugActionRunUsingProfileActionRunActionSelectDefaultTestProfilesConfigureTestProfilesActionRunSelectedActionDebugSelectedActionRunAllActionDebugAllActionCancelTestRunActionTestingViewAsListActionTestingViewAsTreeActionTestingSortByStatusActionTestingSortByLocationActionTestingSortByDurationActionShowMostRecentOutputActionCollapseAllActionClearTestResultsActionGoToTestRunAtCursorDebugAtCursorRunCurrentFileDebugCurrentFileReRunFailedTestsDebugFailedTestsReRunLastRunDebugLastRunSearchForTestExtensionOpenOutputPeekToggleInlineTestOutputRefreshTestsActionCancelTestRefreshActionTestingExplorerViewModelTestResultElementTestCaseElementtestingColorIconFailedtestingColorIconErroredtestingColorIconPassedtestingColorIconQueuedtestingColorIconUnsettestingColorIconSkippedListElementTypetestIdentityProviderpruneNodesWithParentsNotInTreeAutoRunModeITestingConfigurationTestExplorerStateFiltertestStateNamesisDurationAccessorgetComputedStategetComputedDurationIStoredValueSerializationICoverageAccessorFileCoverageallTestFilterTermsITestItemUpsertChildITestItemUpdateCanResolveChildrenITestItemSetTagsITestItemRemoveChildITestItemSetPropITestItemBulkReplaceITestItemDocumentSyncedITestItemCollectionOptionsDuplicateTestItemErrorMixedTestItemControllerITestRunTaskResultssumCountsallChangedResultsisRunningTestsBaseTestResultStorageTestDiffListenergetCollectionItemParentsgetAllTestsInHierarchywaitForTestToBeIdleITestTagDisplayInfoITestCoverageIBranchCoverageIFunctionCoverageIStatementCoverageTreeStateNodestateNodesisRunningStateTestObjectTreeUnhideAllTestsActionThe text was updated successfully, but these errors were encountered: