Fix some basic eslint errors in copilot

[mjbvz]

Starts working through some basic repo wide eslint rules for the copilot extension. Stuff like missing semicolons, missing super calls, and missing readonly modifiers for disposables

[Copilot]

Pull request overview

This PR incrementally addresses repo-wide ESLint violations in the Copilot extension (primarily TypeScript style/typing rules) to reduce lint noise and keep the codebase compliant with shared conventions.

Changes:

• Replace any-based casts with unknown and standardize DI/service-brand patterns.
• Add missing semicolons / type terminators across many type definitions and signatures.
• Mark long-lived DisposableStore fields as readonly and fix a few test/cleanup patterns (e.g., suite callback, super.dispose()).

Show a summary per file

File	Description
extensions/copilot/test/testVisualizationRunnerSTest.ts	Replace any cast with unknown for globals typing.
extensions/copilot/test/testVisualizationRunner.ts	Replace any cast with unknown for globals typing.
extensions/copilot/test/pipeline/pipeline.ts	Add missing type terminator ; for options type.
extensions/copilot/test/pipeline/alternativeAction/types.ts	Add missing type terminators in test pipeline types.
extensions/copilot/test/base/simulationOptions.ts	Add missing type terminator for NesDatagen.
extensions/copilot/src/util/node/worker.ts	Add missing type terminator for RpcProxy mapped type.
extensions/copilot/src/platform/thinking/common/thinking.ts	Add missing type terminator for delta type.
extensions/copilot/src/platform/telemetry/common/msftTelemetrySender.ts	Make _disposables readonly.
extensions/copilot/src/platform/review/vscode/reviewServiceImpl.ts	Make disposable stores readonly.
extensions/copilot/src/platform/notebook/common/alternativeContentEditGenerator.ts	Add missing type terminator for telemetry options type.
extensions/copilot/src/platform/networking/common/networking.ts	Add missing type terminator for telemetry properties type.
extensions/copilot/src/platform/networking/common/fetch.ts	Add missing type terminators for stream-related types.
extensions/copilot/src/platform/inlineEdits/node/inlineEditsModelService.ts	Add missing type terminator for ModelInfo.
extensions/copilot/src/platform/inlineEdits/common/workspaceEditTracker/workspaceDocumentEditTracker.ts	Add missing type terminator for history diff type.
extensions/copilot/src/platform/inlineEdits/common/workspaceEditTracker/nesXtabHistoryTracker.ts	Fix union/type terminators for history tracker event types.
extensions/copilot/src/platform/inlineEdits/common/statelessNextEditProvider.ts	Add missing terminators for generator/type definitions.
extensions/copilot/src/platform/inlineEdits/common/dataTypes/xtabPromptOptions.ts	Add missing type terminators across prompt option types.
extensions/copilot/src/platform/inlineEdits/common/dataTypes/languageContext.ts	Add missing type terminators for serialized/context types.
extensions/copilot/src/platform/inlineCompletions/common/api.ts	Add missing type terminators in proposed API types.
extensions/copilot/src/platform/git/vscode-node/gitServiceImpl.ts	Add missing semicolon for overload signature.
extensions/copilot/src/platform/endpoint/test/node/openaiCompatibleEndpoint.ts	Add missing type terminator for model config type.
extensions/copilot/src/platform/endpoint/common/endpointProvider.ts	Add missing type terminator for capabilities type.
extensions/copilot/src/platform/chat/common/commonTypes.ts	Add missing terminator for ChatFetchRetriableError.
extensions/copilot/src/platform/authentication/common/authentication.ts	Add missing semicolon to abstract method signature.
extensions/copilot/src/lib/node/chatLibMain.ts	Add missing type terminator for exported type alias.
extensions/copilot/src/extension/workspaceRecorder/vscode-node/workspaceListenerService.ts	Use declare _serviceBrand pattern for service branding.
extensions/copilot/src/extension/typescriptContext/vscode-node/types.ts	Add missing type terminators for event/result types.
extensions/copilot/src/extension/typescriptContext/vscode-node/nesRenameService.ts	Make disposables readonly and adjust injected field modifiers.
extensions/copilot/src/extension/typescriptContext/vscode-node/languageContextService.ts	Make disposables readonly and adjust injected field modifiers/order.
extensions/copilot/src/extension/typescriptContext/serverPlugin/src/node/test/nes.spec.ts	Add missing type terminators for test types.
extensions/copilot/src/extension/typescriptContext/serverPlugin/src/node/create.ts	Add missing type terminator for resolved input type.
extensions/copilot/src/extension/typescriptContext/serverPlugin/src/common/typescripts.ts	Add missing type terminator for exported type alias.
extensions/copilot/src/extension/typescriptContext/serverPlugin/src/common/protocol.ts	Add missing type terminators across protocol types.
extensions/copilot/src/extension/typescriptContext/serverPlugin/src/common/contextProvider.ts	Add missing signature/type terminators in context provider types.
extensions/copilot/src/extension/typescriptContext/common/serverProtocol.ts	Add missing type terminators across protocol mirror types.
extensions/copilot/src/extension/tools/common/virtualTools/preComputedToolEmbeddingsCache.ts	Make injected log service a private readonly field.
extensions/copilot/src/extension/search/vscode-node/commands.ts	Adjust constructor injection; potential unused-parameter issue.
extensions/copilot/src/extension/mcp/vscode-node/commands.ts	Make injected services private readonly fields.
extensions/copilot/src/extension/log/node/chatLogExport.ts	Remove unnecessary type assertion in error-log push.
extensions/copilot/src/extension/inlineEdits/vscode-node/features/diagnosticsBasedCompletions/importDiagnosticsCompletionProvider.ts	Add missing type terminator for import details type.
extensions/copilot/src/extension/conversation/vscode-node/test/languageModelAccess.test.ts	Remove async from suite callback.
extensions/copilot/src/extension/conversation/vscode-node/remoteAgents.ts	Make disposables readonly.
extensions/copilot/src/extension/conversation/vscode-node/languageModelAccess.ts	Remove unused injected field _expService from wrapper ctor.
extensions/copilot/src/extension/conversation/vscode-node/conversationFeature.ts	Make disposable stores readonly.
extensions/copilot/src/extension/completions-core/vscode-node/lib/src/prompt/similarFiles/openTabFiles.ts	Make injected doc manager private readonly field.
extensions/copilot/src/extension/completions-core/vscode-node/lib/src/experiments/featuresService.ts	Add missing type terminator for exp settings type.
extensions/copilot/src/extension/completions-core/vscode-node/extension/src/statusBar.ts	Make injected services private readonly fields.
extensions/copilot/src/extension/completions-core/vscode-node/extension/src/codeReferencing/index.ts	Make injected runtime-mode service private readonly field.
extensions/copilot/src/extension/chatSessions/vscode-node/test/claudeChatSessionContentProvider.spec.ts	Ensure dispose() calls super.dispose().
extensions/copilot/src/extension/chatSessions/copilotcli/node/permissionHelpers.ts	Add missing type terminators for permission helper types.

Copilot's findings

• Files reviewed: 50/50 changed files
• Comments generated: 1

[Copilot]

telemetryService is injected but never used (and no longer stored as a field). This likely violates the repo's unused-parameter eslint rule; either remove the ITelemetryService parameter or store/use it (e.g. emit a feedback telemetry event).

See below for a potential fix:

import { Disposable } from '../../../util/vs/base/common/lifecycle';
import { IFeedbackReporter } from '../../prompt/node/feedbackReporter';
import { SearchFeedbackKind, SemanticSearchTextSearchProvider } from '../../workspaceSemanticSearch/node/semanticSearchTextSearchProvider';

export class SearchPanelCommands extends Disposable {
	constructor(