Reduce unused exports #164937
Labels
debt
Code quality issues
Comments
|
I have added owners of code areas I used to own and I updated the table. |
|
I often use |
|
@hediet yeah please feel free to keep those exported 🙂 |
roblourens
added a commit
that referenced
this issue
Nov 1, 2022
|
Also types that are used implicitly (like by passing an options object literal to another method) but are not actually currently referenced by name. I have a couple of those. But this was some good cleanup, thanks for the prompt! |
roblourens
added a commit
that referenced
this issue
Nov 1, 2022
hediet
added
editor-core
|
Checked and updated those assigned to me. |
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.
editorMarkerNavigationWarningHeaderextensionRatingIconColorextensionPreReleaseIconColorInputEventIObservableLoggerLoggingObserverLoggingObservableValueTimeOffsetSchedulerScheduledTaskScheduledTaskSourceTimeTravelSchedulerAsyncSchedulerProcessororiginalGlobalValuesthrowIfDisposablesAreLeakedAsyncPartialModelCursorStatePartialViewCursorStateOutputPositionjoinSequenceDiffsshiftSequenceDiffsExtensionsBracketKindBaselengthComparelengthOfStringObjlengthHashIDenseKeyProviderILineChangeICharChangeIUnicodeCharacterSearcherTargetActiveGhostTextControllerDelegatingModelInlineCompletionsSessionTrackedInlineCompletionSuggestWidgetStateSuggestItemInfoIBannerItemTextEditSoundMergeEditorArgsIMergeEditorInputModelFactoryelementAtOrUndefinedMergeMarkersControllerIMergeDiffComputerResultLineEditsMergeEditorModelStategetOtherInputNumberAbstractModifiedBaseRangeStateModifiedBaseRangeStateBaseModifiedBaseRangeStateInput1ModifiedBaseRangeStateInput2ModifiedBaseRangeStateBothModifiedBaseRangeStateUnrecognizedProjectedDiffComputerITextModelDiffsStateIGutterItemProviderLineAlignmentMergeEditorViewZonesMergeEditorViewZoneModifiedBaseRangeGutterItemModelMergeConflictGutterItemViewCenteredViewStateICenteredViewStylesWorkbenchListMultiSelectionSIGN_SERVICE_IDICompositeBarOptionsMAX_VALUE_RENDER_LENGTH_IN_VIEWLETtwistiePixelsIRenderValueOptionsIVariableTemplateDataIPrivateBreakpointWidgetServiceVariablesDataSourceWatchExpressionsRendererDEBUG_SERVICE_IDIEnvConfigBaseSaveAllActionIFileTemplateDataMAX_OUTPUT_LENGTHCONTEXT_IN_OUTPUTcreateThe text was updated successfully, but these errors were encountered: