Refactor code to remove unused JSDoc types (#24300)

Fixes #24077

Removes redundant type annotations from JSDocs in TypeScript files where
the type is already inferred by TypeScript.

For example:

![image](https://github.com/user-attachments/assets/9ee1f0b6-f36f-4f4f-81dc-5178d46808cb)

Here I removed JSDoc types but I still kept all the useful information. 
I tried to be on the liberal side to avoid removing any comments that
provide useful context or clarify behavior. If I missed any or if more
can be removed, I’m happy to go over it again.

Additionally, if this issue only applies to a specific folder or scope,
please let me know so I can make the necessary adjustments.

pythonExtensionApi/src/main.ts

@@ -25,7 +25,6 @@ export interface PythonExtension {
 
         /**
          * Gets the path to the debugger package used by the extension.
-         * @returns {Promise<string>}
          */
         getDebuggerPackagePath(): Promise<string | undefined>;
     };

src/client/activation/types.ts

@@ -12,27 +12,14 @@ import { PythonEnvironment } from '../pythonEnvironments/info';
 export const IExtensionActivationManager = Symbol('IExtensionActivationManager');
 /**
  * Responsible for activation of extension.
- *
- * @export
- * @interface IExtensionActivationManager
- * @extends {IDisposable}
  */
 export interface IExtensionActivationManager extends IDisposable {
-    /**
-     * Method invoked when extension activates (invoked once).
-     *
-     * @returns {Promise<void>}
-     * @memberof IExtensionActivationManager
-     */
+    // Method invoked when extension activates (invoked once).
     activate(startupStopWatch: StopWatch): Promise<void>;
     /**
      * Method invoked when a workspace is loaded.
      * This is where we place initialization scripts for each workspace.
      * (e.g. if we need to run code for each workspace, then this is where that happens).
-     *
-     * @param {Resource} resource
-     * @returns {Promise<void>}
-     * @memberof IExtensionActivationManager
      */
     activateWorkspace(resource: Resource): Promise<void>;
 }
@@ -43,8 +30,6 @@ export const IExtensionActivationService = Symbol('IExtensionActivationService')
  * invoked for every workspace folder (in multi-root workspace folders) during the activation of the extension.
  * This is a great hook for extension activation code, i.e. you don't need to modify
  * the `extension.ts` file to invoke some code when extension gets activated.
- * @export
- * @interface IExtensionActivationService
  */
 export interface IExtensionActivationService {
     supportedWorkspaceTypes: { untrustedWorkspace: boolean; virtualWorkspace: boolean };
@@ -100,8 +85,6 @@ export interface ILanguageServerProxy extends IDisposable {
      * Sends a request to LS so as to load other extensions.
      * This is used as a plugin loader mechanism.
      * Anyone (such as intellicode) wanting to interact with LS, needs to send this request to LS.
-     * @param {{}} [args]
-     * @memberof ILanguageServerProxy
      */
     loadExtension(args?: unknown): void;
 }
@@ -110,9 +93,6 @@ export const ILanguageServerOutputChannel = Symbol('ILanguageServerOutputChannel
 export interface ILanguageServerOutputChannel {
     /**
      * Creates output channel if necessary and returns it
-     *
-     * @type {ILogOutputChannel}
-     * @memberof ILanguageServerOutputChannel
      */
     readonly channel: ILogOutputChannel;
 }
@@ -123,8 +103,6 @@ export const IExtensionSingleActivationService = Symbol('IExtensionSingleActivat
  * invoked during the activation of the extension.
  * This is a great hook for extension activation code, i.e. you don't need to modify
  * the `extension.ts` file to invoke some code when extension gets activated.
- * @export
- * @interface IExtensionSingleActivationService
  */
 export interface IExtensionSingleActivationService {
     supportedWorkspaceTypes: { untrustedWorkspace: boolean; virtualWorkspace: boolean };

src/client/api.ts

@@ -80,7 +80,6 @@ export function buildApi(
              * @param {Resource} [resource] A resource for which the setting is asked for.
              * * When no resource is provided, the setting scoped to the first workspace folder is returned.
              * * If no folder is present, it returns the global setting.
-             * @returns {({ execCommand: string[] | undefined })}
              */
             getExecutionDetails(
                 resource?: Resource,

src/client/api/types.ts

@@ -25,7 +25,6 @@ export interface PythonExtension {
 
         /**
          * Gets the path to the debugger package used by the extension.
-         * @returns {Promise<string>}
          */
         getDebuggerPackagePath(): Promise<string | undefined>;
     };

src/client/application/diagnostics/base.ts

@@ -73,11 +73,6 @@ export abstract class BaseDiagnosticsService implements IDiagnosticsService, IDi
     /**
      * Returns a key used to keep track of whether a diagnostic was handled or not.
      * So as to prevent handling/displaying messages multiple times for the same diagnostic.
-     *
-     * @protected
-     * @param {IDiagnostic} diagnostic
-     * @returns {string}
-     * @memberof BaseDiagnosticsService
      */
     protected getDiagnosticsKey(diagnostic: IDiagnostic): string {
         if (diagnostic.scope === DiagnosticScope.Global) {

src/client/application/types.ts

@@ -11,8 +11,6 @@ export interface IApplicationDiagnostics {
     /**
      * Perform pre-extension activation health checks.
      * E.g. validate user environment, etc.
-     * @returns {Promise<void>}
-     * @memberof IApplicationDiagnostics
      */
     performPreStartupHealthCheck(resource: Resource): Promise<void>;
     register(): void;

src/client/common/application/commands.ts

@@ -14,7 +14,6 @@ export type CommandsWithoutArgs = keyof ICommandNameWithoutArgumentTypeMapping;
 /**
  * Mapping between commands and list or arguments.
  * These commands do NOT have any arguments.
- * @interface ICommandNameWithoutArgumentTypeMapping
  */
 interface ICommandNameWithoutArgumentTypeMapping {
     [Commands.InstallPythonOnMac]: [];
@@ -52,9 +51,6 @@ export type AllCommands = keyof ICommandNameArgumentTypeMapping;
 /**
  * Mapping between commands and list of arguments.
  * Used to provide strong typing for command & args.
- * @export
- * @interface ICommandNameArgumentTypeMapping
- * @extends {ICommandNameWithoutArgumentTypeMapping}
  */
 export interface ICommandNameArgumentTypeMapping extends ICommandNameWithoutArgumentTypeMapping {
     [Commands.Create_Environment]: [CreateEnvironmentOptions];

src/client/common/application/types.ts

@@ -818,9 +818,6 @@ export interface IWorkspaceService {
 
     /**
      * Generate a key that's unique to the workspace folder (could be fsPath).
-     * @param {(Uri | undefined)} resource
-     * @returns {string}
-     * @memberof IWorkspaceService
      */
     getWorkspaceFolderIdentifier(resource: Uri | undefined, defaultValue?: string): string;
     /**

src/client/common/cancellation.ts

@@ -16,11 +16,6 @@ export class CancellationError extends Error {
 }
 /**
  * Create a promise that will either resolve with a default value or reject when the token is cancelled.
- *
- * @export
- * @template T
- * @param {({ defaultValue: T; token: CancellationToken; cancelAction: 'reject' | 'resolve' })} options
- * @returns {Promise<T>}
  */
 export function createPromiseFromCancellation<T>(options: {
     defaultValue: T;
@@ -50,10 +45,6 @@ export function createPromiseFromCancellation<T>(options: {
 
 /**
  * Create a single unified cancellation token that wraps multiple cancellation tokens.
- *
- * @export
- * @param {(...(CancellationToken | undefined)[])} tokens
- * @returns {CancellationToken}
  */
 export function wrapCancellationTokens(...tokens: (CancellationToken | undefined)[]): CancellationToken {
     const wrappedCancellantionToken = new CancellationTokenSource();
@@ -117,15 +108,13 @@ export namespace Cancellation {
 
     /**
      * isCanceled returns a boolean indicating if the cancel token has been canceled.
-     * @param cancelToken
      */
     export function isCanceled(cancelToken?: CancellationToken): boolean {
         return cancelToken ? cancelToken.isCancellationRequested : false;
     }
 
     /**
      * throws a CancellationError if the token is canceled.
-     * @param cancelToken
      */
     export function throwIfCanceled(cancelToken?: CancellationToken): void {
         if (isCanceled(cancelToken)) {

src/client/common/constants.ts

@@ -111,8 +111,6 @@ export function isTestExecution(): boolean {
 /**
  * Whether we're running unit tests (*.unit.test.ts).
  * These tests have a special meaning, they run fast.
- * @export
- * @returns {boolean}
  */
 export function isUnitTestExecution(): boolean {
     return process.env.VSC_PYTHON_UNIT_TEST === '1';

src/client/common/extensions.ts

@@ -28,7 +28,6 @@ declare interface String {
 /**
  * Appropriately formats a string so it can be used as an argument for a command in a shell.
  * E.g. if an argument contains a space, then it will be enclosed within double quotes.
- * @param {String} value.
  */
 String.prototype.toCommandArgumentForPythonExt = function (this: string): string {
     if (!this) {

src/client/common/installer/types.ts

@@ -18,11 +18,6 @@ export interface IModuleInstaller {
      * If a cancellation token is provided, then a cancellable progress message is dispalyed.
      *  At this point, this method would resolve only after the module has been successfully installed.
      * If cancellation token is not provided, its not guaranteed that module installation has completed.
-     * @param {string} name
-     * @param {InterpreterUri} [resource]
-     * @param {CancellationToken} [cancel]
-     * @returns {Promise<void>}
-     * @memberof IModuleInstaller
      */
     installModule(
         productOrModuleName: Product | string,

src/client/common/terminal/shellDetector.ts

@@ -33,10 +33,6 @@ export class ShellDetector {
      * 3. Try to identify the type of the shell based on the user environment (OS).
      * 4. If all else fail, use defaults hardcoded (cmd for windows, bash for linux & mac).
      * More information here: https://github.com/microsoft/vscode/issues/74233#issuecomment-497527337
-     *
-     * @param {Terminal} [terminal]
-     * @returns {TerminalShellType}
-     * @memberof TerminalHelper
      */
     public identifyTerminalShell(terminal?: Terminal): TerminalShellType {
         let shell: TerminalShellType | undefined;

src/client/common/terminal/shellDetectors/settingsShellDetector.ts

@@ -13,10 +13,6 @@ import { BaseShellDetector } from './baseShellDetector';
 
 /**
  * Identifies the shell based on the user settings.
- *
- * @export
- * @class SettingsShellDetector
- * @extends {BaseShellDetector}
  */
 @injectable()
 export class SettingsShellDetector extends BaseShellDetector {

src/client/common/terminal/shellDetectors/terminalNameShellDetector.ts

@@ -11,10 +11,6 @@ import { BaseShellDetector } from './baseShellDetector';
 
 /**
  * Identifies the shell, based on the display name of the terminal.
- *
- * @export
- * @class TerminalNameShellDetector
- * @extends {BaseShellDetector}
  */
 @injectable()
 export class TerminalNameShellDetector extends BaseShellDetector {

src/client/common/terminal/shellDetectors/userEnvironmentShellDetector.ts

@@ -13,10 +13,6 @@ import { BaseShellDetector } from './baseShellDetector';
 
 /**
  * Identifies the shell based on the users environment (env variables).
- *
- * @export
- * @class UserEnvironmentShellDetector
- * @extends {BaseShellDetector}
  */
 @injectable()
 export class UserEnvironmentShellDetector extends BaseShellDetector {

src/client/common/terminal/shellDetectors/vscEnvironmentShellDetector.ts

@@ -12,10 +12,6 @@ import { BaseShellDetector } from './baseShellDetector';
 
 /**
  * Identifies the shell, based on the VSC Environment API.
- *
- * @export
- * @class VSCEnvironmentShellDetector
- * @extends {BaseShellDetector}
  */
 export class VSCEnvironmentShellDetector extends BaseShellDetector {
     constructor(@inject(IApplicationEnvironment) private readonly appEnv: IApplicationEnvironment) {

src/client/common/terminal/syncTerminalService.ts

@@ -92,11 +92,6 @@ class ExecutionState implements Disposable {
  * - Send text to a terminal that executes our python file, passing in the original text as args
  * - The pthon file will execute the commands as a subprocess
  * - At the end of the execution a file is created to singal completion.
- *
- * @export
- * @class SynchronousTerminalService
- * @implements {ITerminalService}
- * @implements {Disposable}
  */
 export class SynchronousTerminalService implements ITerminalService, Disposable {
     private readonly disposables: Disposable[] = [];

src/client/common/terminal/types.ts

@@ -100,10 +100,6 @@ export interface ITerminalServiceFactory {
     /**
      * Gets a terminal service.
      * If one exists with the same information, that is returned else a new one is created.
-     *
-     * @param {TerminalCreationOptions}
-     * @returns {ITerminalService}
-     * @memberof ITerminalServiceFactory
      */
     getTerminalService(options: TerminalCreationOptions & { newTerminalPerFile?: boolean }): ITerminalService;
     createTerminalService(resource?: Uri, title?: string): ITerminalService;
@@ -132,11 +128,7 @@ export type TerminalActivationOptions = {
     resource?: Resource;
     preserveFocus?: boolean;
     interpreter?: PythonEnvironment;
-    /**
-     * When sending commands to the terminal, do not display the terminal.
-     *
-     * @type {boolean}
-     */
+    // When sending commands to the terminal, do not display the terminal.
     hideFromUser?: boolean;
 };
 export interface ITerminalActivator {
@@ -170,16 +162,10 @@ export const IShellDetector = Symbol('IShellDetector');
 /**
  * Used to identify a shell.
  * Each implemenetion will provide a unique way of identifying the shell.
- *
- * @export
- * @interface IShellDetector
  */
 export interface IShellDetector {
     /**
      * Classes with higher priorities will be used first when identifying the shell.
-     *
-     * @type {number}
-     * @memberof IShellDetector
      */
     readonly priority: number;
     identify(telemetryProperties: ShellIdentificationTelemetry, terminal?: Terminal): TerminalShellType | undefined;

src/client/common/utils/async.ts

@@ -245,12 +245,6 @@ export async function flattenIterable<T>(iterableItem: AsyncIterable<T>): Promis
 
 /**
  * Wait for a condition to be fulfilled within a timeout.
- *
- * @export
- * @param {() => Promise<boolean>} condition
- * @param {number} timeoutMs
- * @param {string} errorMessage
- * @returns {Promise<void>}
  */
 export async function waitForCondition(
     condition: () => Promise<boolean>,

src/client/common/utils/cacheUtils.ts

@@ -5,11 +5,7 @@
 
 const globalCacheStore = new Map<string, { expiry: number; data: any }>();
 
-/**
- * Gets a cache store to be used to store return values of methods or any other.
- *
- * @returns
- */
+// Gets a cache store to be used to store return values of methods or any other.
 export function getGlobalCacheStore() {
     return globalCacheStore;
 }

src/client/common/utils/misc.ts

@@ -27,10 +27,6 @@ type NonFunctionPropertyNames<T> = { [K in keyof T]: T[K] extends Function ? nev
  * Checking whether something is a Resource (Uri/undefined).
  * Using `instanceof Uri` doesn't always work as the object is not an instance of Uri (at least not in tests).
  * That's why VSC too has a helper method `URI.isUri` (though not public).
- *
- * @export
- * @param {InterpreterUri} [resource]
- * @returns {resource is Resource}
  */
 export function isResource(resource?: InterpreterUri): resource is Resource {
     if (!resource) {
@@ -44,9 +40,6 @@ export function isResource(resource?: InterpreterUri): resource is Resource {
  * Checking whether something is a Uri.
  * Using `instanceof Uri` doesn't always work as the object is not an instance of Uri (at least not in tests).
  * That's why VSC too has a helper method `URI.isUri` (though not public).
- *
- * @param {InterpreterUri} [resource]
- * @returns {resource is Uri}
  */
 
 function isUri(resource?: Uri | any): resource is Uri {

src/client/debugger/extension/hooks/childProcessAttachHandler.ts

@@ -14,9 +14,6 @@ import { DebuggerTypeName } from '../../constants';
 /**
  * This class is responsible for automatically attaching the debugger to any
  * child processes launched. I.e. this is the class responsible for multi-proc debugging.
- * @export
- * @class ChildProcessAttachEventHandler
- * @implements {IDebugSessionEventHandlers}
  */
 @injectable()
 export class ChildProcessAttachEventHandler implements IDebugSessionEventHandlers {

src/client/debugger/extension/hooks/childProcessAttachService.ts

@@ -17,9 +17,6 @@ import { getWorkspaceFolders } from '../../../common/vscodeApis/workspaceApis';
 /**
  * This class is responsible for attaching the debugger to any
  * child processes launched. I.e. this is the class responsible for multi-proc debugging.
- * @export
- * @class ChildProcessAttachEventHandler
- * @implements {IChildProcessAttachService}
  */
 @injectable()
 export class ChildProcessAttachService implements IChildProcessAttachService {

src/client/deprecatedProposedApiTypes.ts

@@ -57,7 +57,6 @@ export interface DeprecatedProposedAPI {
          * @param {Resource} [resource] A resource for which the setting is asked for.
          * * When no resource is provided, the setting scoped to the first workspace folder is returned.
          * * If no folder is present, it returns the global setting.
-         * @returns {({ execCommand: string[] | undefined })}
          */
         getExecutionDetails(
             resource?: Resource,

src/client/interpreter/autoSelection/types.ts

@@ -14,9 +14,6 @@ export const IInterpreterAutoSelectionProxyService = Symbol('IInterpreterAutoSel
  * However, the class that reads python Path, must first give preference to selected interpreter.
  * But all classes everywhere make use of python settings!
  * Solution - Use a proxy that does nothing first, but later the real instance is injected.
- *
- * @export
- * @interface IInterpreterAutoSelectionProxyService
  */
 export interface IInterpreterAutoSelectionProxyService {
     readonly onDidChangeAutoSelectedInterpreter: Event<void>;