@codingame/monaco-vscode-api 32.0.1 → 33.0.3

package/vscode/src/vs/workbench/contrib/chat/common/promptSyntax/service/promptsServiceImpl.js

@@ -1,10 +1,9 @@
 
 import { __decorate, __param } from '../../../../../../../../../external/tslib/tslib.es6.js';
-import { CancellationToken } from '../../../../../../base/common/cancellation.js';
 import { CancellationError } from '../../../../../../base/common/errors.js';
 import { Emitter, Event } from '../../../../../../base/common/event.js';
 import { parse } from '../../../../../../base/common/json.js';
-import { Disposable, DisposableStore } from '../../../../../../base/common/lifecycle.js';
+import { Disposable } from '../../../../../../base/common/lifecycle.js';
 import { StopWatch } from '../../../../../../base/common/stopwatch.js';
 import '../../../../../../base/common/observableInternal/index.js';
 import { ResourceMap, ResourceSet } from '../../../../../../base/common/map.js';
@@ -16,11 +15,9 @@ import { localize } from '../../../../../../nls.js';
 import { IConfigurationService } from '../../../../../../platform/configuration/common/configuration.service.js';
 import { FileOperationError, FileOperationResult } from '../../../../../../platform/files/common/files.js';
 import { IFileService } from '../../../../../../platform/files/common/files.service.js';
-import { IExtensionService } from '../../../../../services/extensions/common/extensions.service.js';
 import { IInstantiationService } from '../../../../../../platform/instantiation/common/instantiation.js';
 import { ILabelService } from '../../../../../../platform/label/common/label.service.js';
 import { ILogService } from '../../../../../../platform/log/common/log.service.js';
-import { IFilesConfigurationService } from '../../../../../services/filesConfiguration/common/filesConfigurationService.service.js';
 import { StorageScope, StorageTarget } from '../../../../../../platform/storage/common/storage.js';
 import { IStorageService } from '../../../../../../platform/storage/common/storage.service.js';
 import { ITelemetryService } from '../../../../../../platform/telemetry/common/telemetry.service.js';
@@ -30,7 +27,7 @@ import { getSkillFolderName, getCleanPromptName, AGENT_MD_FILENAME, CLAUDE_MD_FI
 import { PromptsType, PromptFileSource, PROMPT_LANGUAGE_ID, getPromptsTypeForLanguageId, Target } from '../promptTypes.js';
 import { PromptFilesLocator } from '../utils/promptFilesLocator.js';
 import { PromptFileParser, evaluateApplyToPattern, PromptHeaderAttributes } from '../promptFileParser.js';
-import { PromptsStorage, SKILL_PROVIDER_ACTIVATION_EVENT, PROMPT_FILE_PROVIDER_ACTIVATION_EVENT, INSTRUCTIONS_PROVIDER_ACTIVATION_EVENT, CUSTOM_AGENT_PROVIDER_ACTIVATION_EVENT, matchesSessionType, AgentInstructionFileType } from './promptsService.js';
+import { PromptsStorage, matchesSessionType, AgentInstructionFileType } from './promptsService.js';
 import { Delayer } from '../../../../../../base/common/async.js';
 import { Schemas } from '../../../../../../base/common/network.js';
 import { parseSubagentHooksFromYaml } from '../hookSchema.js';
@@ -39,12 +36,11 @@ import { IWorkspaceContextService } from '../../../../../../platform/workspace/c
 import { IWorkspaceTrustManagementService } from '../../../../../../platform/workspace/common/workspaceTrust.service.js';
 import { IPathService } from '../../../../../services/path/common/pathService.service.js';
 import { getTarget, mapClaudeModels, mapClaudeTools } from '../languageProviders/promptFileAttributes.js';
-import { ContextKeyExpr } from '../../../../../../platform/contextkey/common/contextkey.js';
-import { IContextKeyService } from '../../../../../../platform/contextkey/common/contextkey.service.js';
 import { getCanonicalPluginCommandId } from '../../plugins/agentPluginService.js';
 import { IAgentPluginService } from '../../plugins/agentPluginService.service.js';
 import { isContributionEnabled } from '../../enablement.js';
 import { assertNever } from '../../../../../../base/common/assert.js';
+import { ExtensionPromptFileService } from './extensionPromptFileService.js';
 import { autorun } from '../../../../../../base/common/observableInternal/reactions/autorun.js';
 
 let PromptsService = class PromptsService extends Disposable {
@@ -56,13 +52,10 @@ let PromptsService = class PromptsService extends Disposable {
         userDataService,
         configurationService,
         fileService,
-        filesConfigService,
         storageService,
-        extensionService,
         telemetryService,
         workspaceService,
         pathService,
-        contextKeyService,
         agentPluginService,
         workspaceTrustService
     ) {
@@ -74,48 +67,27 @@ let PromptsService = class PromptsService extends Disposable {
         this.userDataService = userDataService;
         this.configurationService = configurationService;
         this.fileService = fileService;
-        this.filesConfigService = filesConfigService;
         this.storageService = storageService;
-        this.extensionService = extensionService;
         this.telemetryService = telemetryService;
         this.workspaceService = workspaceService;
         this.pathService = pathService;
-        this.contextKeyService = contextKeyService;
         this.agentPluginService = agentPluginService;
         this.workspaceTrustService = workspaceTrustService;
         this.cachedParsedPromptFromModels = ( new ResourceMap());
         this.cachedFileLocations = {};
         this.fileLocatorEvents = {};
-        this.contributedFiles = {
-            [PromptsType.prompt]: ( new ResourceMap()),
-            [PromptsType.instructions]: ( new ResourceMap()),
-            [PromptsType.agent]: ( new ResourceMap()),
-            [PromptsType.skill]: ( new ResourceMap()),
-            [PromptsType.hook]: ( new ResourceMap())
-        };
-        this._contributedWhenKeys = ( new Set());
-        this._contributedWhenClauses = ( new Map());
-        this._providerWhenClauses = ( new Map());
-        this._onDidContributedWhenChange = this._register(( new Emitter()));
-        this._onDidChangeInstructions = this._register(( new Emitter()));
         this._onDidPluginPromptFilesChange = this._register(( new Emitter()));
         this._onDidPluginHooksChange = this._register(( new Emitter()));
         this._pluginPromptFilesByType = ( new Map());
-        this._pendingReadonlyUris = [];
-        this._pendingReadonlyFlush = false;
-        this.promptFileProviders = [];
         this.disabledPromptsStorageKeyPrefix = "chat.disabledPromptFiles.";
         this.fileLocator = this.createPromptFilesLocator();
         this._register(this.modelService.onModelRemoved(model => {
             this.cachedParsedPromptFromModels.delete(model.uri);
         }));
-        this._register(this.contextKeyService.onDidChangeContext(e => {
-            if (e.affectsSome(this._contributedWhenKeys)) {
-                for (const type of ( Object.keys(this.cachedFileLocations))) {
-                    this.cachedFileLocations[type] = undefined;
-                }
-                this._onDidContributedWhenChange.fire();
-            }
+        this.extensionPromptFiles = this._register(this.instantiationService.createInstance(ExtensionPromptFileService));
+        const onDidChangeExtensionPromptFiles = this.extensionPromptFiles.onDidChange;
+        this._register(onDidChangeExtensionPromptFiles(e => {
+            this.cachedFileLocations[e.type] = undefined;
         }));
         const modelChangeEvent = this._register(( new ModelChangeTracker(this.modelService))).onDidPromptChange;
         this.cachedCustomAgents = this._register(( new CachedPromise(token => this.computeAgentDiscoveryInfo(token), () => Event.any(
@@ -125,8 +97,8 @@ let PromptsService = class PromptsService extends Disposable {
                 this.configurationService.onDidChangeConfiguration,
                 e => e.affectsConfiguration(PromptsConfig.USE_CHAT_HOOKS)
             ),
-            this._onDidContributedWhenChange.event,
-            this._onDidPluginPromptFilesChange.event,
+            Event.filter(onDidChangeExtensionPromptFiles, e => e.type === PromptsType.agent),
+            Event.filter(this._onDidPluginPromptFilesChange.event, t => t === PromptsType.agent),
             this.workspaceTrustService.onDidChangeTrust
         ))));
         this.cachedSlashCommands = this._register(( new CachedPromise(token => this.computeSlashCommandDiscoveryInfo(token), () => Event.any(
@@ -134,14 +106,20 @@ let PromptsService = class PromptsService extends Disposable {
             this.getFileLocatorEvent(PromptsType.skill),
             Event.filter(modelChangeEvent, e => e.promptType === PromptsType.prompt),
             Event.filter(modelChangeEvent, e => e.promptType === PromptsType.skill),
-            this._onDidContributedWhenChange.event,
-            this._onDidPluginPromptFilesChange.event
+            Event.filter(
+                onDidChangeExtensionPromptFiles,
+                e => e.type === PromptsType.prompt || e.type === PromptsType.skill
+            ),
+            Event.filter(
+                this._onDidPluginPromptFilesChange.event,
+                t => t === PromptsType.prompt || t === PromptsType.skill
+            )
         ))));
         this.cachedSkills = this._register(( new CachedPromise(token => this.computeSkillDiscovery(token), () => Event.any(
             this.getFileLocatorEvent(PromptsType.skill),
             Event.filter(modelChangeEvent, e => e.promptType === PromptsType.skill),
-            this._onDidContributedWhenChange.event,
-            this._onDidPluginPromptFilesChange.event
+            Event.filter(onDidChangeExtensionPromptFiles, e => e.type === PromptsType.skill),
+            Event.filter(this._onDidPluginPromptFilesChange.event, t => t === PromptsType.skill)
         ))));
         this.cachedHooks = this._register(( new CachedPromise(
             token => this.computeHooks(token),
@@ -152,9 +130,11 @@ let PromptsService = class PromptsService extends Disposable {
         )));
         this.cachedInstructions = this._register(( new CachedPromise(token => this.computeInstructionFiles(token), () => Event.any(
             this.getFileLocatorEvent(PromptsType.instructions),
-            this._onDidContributedWhenChange.event,
-            this._onDidChangeInstructions.event,
-            this._onDidPluginPromptFilesChange.event
+            Event.filter(onDidChangeExtensionPromptFiles, e => e.type === PromptsType.instructions),
+            Event.filter(
+                this._onDidPluginPromptFilesChange.event,
+                t => t === PromptsType.instructions
+            )
         ))));
         this._register(
             this.watchPluginPromptFilesForType(PromptsType.prompt, (plugin, reader) => plugin.commands.read(reader))
@@ -213,7 +193,7 @@ let PromptsService = class PromptsService extends Disposable {
             nextFiles.sort((a, b) => `${a.name ?? ""}|${( a.uri.toString())}`.localeCompare(`${b.name ?? ""}|${( b.uri.toString())}`));
             this._pluginPromptFilesByType.set(type, nextFiles);
             this.cachedFileLocations[type] = undefined;
-            this._onDidPluginPromptFilesChange.fire();
+            this._onDidPluginPromptFilesChange.fire(type);
         });
     }
     createPromptFilesLocator() {
@@ -277,87 +257,7 @@ let PromptsService = class PromptsService extends Disposable {
         })));
     }
     registerPromptFileProvider(extension, type, provider) {
-        const providerEntry = {
-            extension,
-            type,
-            ...provider
-        };
-        this.promptFileProviders.push(providerEntry);
-        const disposables = ( new DisposableStore());
-        if (provider.onDidChangePromptFiles) {
-            disposables.add(provider.onDidChangePromptFiles(() => {
-                this.invalidatePromptFileCache(type);
-            }));
-        }
-        this.invalidatePromptFileCache(type);
-        disposables.add({
-            dispose: () => {
-                const index = this.promptFileProviders.findIndex(p => p === providerEntry);
-                if (index >= 0) {
-                    this.promptFileProviders.splice(index, 1);
-                    this._providerWhenClauses.delete(providerEntry);
-                    this._updateContributedWhenKeys();
-                    this.invalidatePromptFileCache(type);
-                }
-            }
-        });
-        return disposables;
-    }
-    invalidatePromptFileCache(type) {
-        if (type === PromptsType.agent) {
-            this.cachedFileLocations[PromptsType.agent] = undefined;
-            this.cachedCustomAgents.refresh();
-        } else if (type === PromptsType.instructions) {
-            this.cachedFileLocations[PromptsType.instructions] = undefined;
-            this._onDidChangeInstructions.fire();
-        } else if (type === PromptsType.prompt) {
-            this.cachedFileLocations[PromptsType.prompt] = undefined;
-            this.cachedSlashCommands.refresh();
-        } else if (type === PromptsType.skill) {
-            this.cachedFileLocations[PromptsType.skill] = undefined;
-            this.cachedSkills.refresh();
-            this.cachedSlashCommands.refresh();
-        }
-    }
-    async listFromProviders(type, activationEvent, token) {
-        const result = [];
-        const readonlyUris = [];
-        await this.extensionService.activateByEvent(activationEvent);
-        const providers = this.promptFileProviders.filter(p => p.type === type);
-        if (providers.length === 0) {
-            return result;
-        }
-        for (const providerEntry of providers) {
-            try {
-                const files = await providerEntry.providePromptFiles({}, token);
-                this._providerWhenClauses.set(providerEntry, files?.flatMap(file => file.when ? [file.when] : []) ?? []);
-                this._updateContributedWhenKeys();
-                if (!files || token.isCancellationRequested) {
-                    continue;
-                }
-                for (const file of files) {
-                    readonlyUris.push(file.uri);
-                    result.push({
-                        uri: file.uri,
-                        storage: PromptsStorage.extension,
-                        type,
-                        extension: providerEntry.extension,
-                        source: PromptFileSource.ExtensionAPI,
-                        name: file.name,
-                        description: file.description,
-                        when: file.when,
-                        sessionTypes: file.sessionTypes
-                    });
-                }
-            } catch (e) {
-                this.logger.error(
-                    `[listFromProviders] Failed to get ${type} files from provider`,
-                    e instanceof Error ? e.message : String(e)
-                );
-            }
-        }
-        void this.filesConfigService.updateReadonly(readonlyUris, true);
-        return result;
+        return this.extensionPromptFiles.registerPromptFileProvider(extension, type, provider);
     }
     async listPromptFilesForStorage(type, storage, token) {
         let promptPaths;
@@ -387,39 +287,8 @@ let PromptsService = class PromptsService extends Disposable {
         }
         return promptPaths;
     }
-    async getExtensionPromptFiles(type, token) {
-        await this.extensionService.whenInstalledExtensionsRegistered();
-        const settledResults = await Promise.allSettled(( this.contributedFiles[type].values()));
-        const contributedFiles = ( settledResults.filter(result => result.status === "fulfilled").map(result => result.value));
-        const activationEvent = this.getProviderActivationEvent(type);
-        const providerFiles = activationEvent ? await this.listFromProviders(type, activationEvent, token) : [];
-        return [...contributedFiles, ...providerFiles].filter(file => {
-            if (!file.when) {
-                return true;
-            }
-            const when = ContextKeyExpr.deserialize(file.when);
-            if (!when) {
-                this.logger.warn(
-                    `[getExtensionPromptFiles] Ignoring contributed prompt file with invalid when clause: ${file.when}`
-                );
-                return false;
-            }
-            return this.contextKeyService.contextMatchesRules(when);
-        });
-    }
-    getProviderActivationEvent(type) {
-        switch (type) {
-        case PromptsType.agent:
-            return CUSTOM_AGENT_PROVIDER_ACTIVATION_EVENT;
-        case PromptsType.instructions:
-            return INSTRUCTIONS_PROVIDER_ACTIVATION_EVENT;
-        case PromptsType.prompt:
-            return PROMPT_FILE_PROVIDER_ACTIVATION_EVENT;
-        case PromptsType.skill:
-            return SKILL_PROVIDER_ACTIVATION_EVENT;
-        case PromptsType.hook:
-            return undefined;
-        }
+    getExtensionPromptFiles(type, token) {
+        return this.extensionPromptFiles.getExtensionPromptFiles(type, token);
     }
     async getSourceFolders(type) {
         const result = [];
@@ -687,95 +556,7 @@ let PromptsService = class PromptsService extends Disposable {
         return ( new PromptFileParser()).parse(uri, ( fileContent.value.toString()));
     }
     registerContributedFile(type, uri, extension, name, description, when, sessionTypes) {
-        const bucket = this.contributedFiles[type];
-        if (( bucket.has(uri))) {
-            return Disposable.None;
-        }
-        const entryPromise = (async () => {
-            if (type === PromptsType.skill) {
-                try {
-                    const validated = await this.validateAndSanitizeSkillFile(uri, CancellationToken.None);
-                    name = validated.name;
-                    description = validated.description;
-                } catch (e) {
-                    const msg = e instanceof Error ? e.message : String(e);
-                    this.logger.error(
-                        `[registerContributedFile] Extension '${extension.identifier.value}' failed to validate skill file: ${uri}`,
-                        msg
-                    );
-                    throw e;
-                }
-            }
-            return {
-                uri,
-                name,
-                description,
-                when,
-                sessionTypes,
-                storage: PromptsStorage.extension,
-                type,
-                extension,
-                source: PromptFileSource.ExtensionContribution
-            };
-        })();
-        bucket.set(uri, entryPromise);
-        this._enqueueReadonlyUpdate(uri);
-        if (when) {
-            this._contributedWhenClauses.set(`${type}/${( uri.toString())}`, when);
-        }
-        const flushCachesIfRequired = () => {
-            this._updateContributedWhenKeys();
-            this.cachedFileLocations[type] = undefined;
-            switch (type) {
-            case PromptsType.agent:
-                this.cachedCustomAgents.refresh();
-                break;
-            case PromptsType.prompt:
-                this.cachedSlashCommands.refresh();
-                break;
-            case PromptsType.skill:
-                this.cachedSkills.refresh();
-                this.cachedSlashCommands.refresh();
-                break;
-            }
-        };
-        flushCachesIfRequired();
-        return {
-            dispose: () => {
-                bucket.delete(uri);
-                this._contributedWhenClauses.delete(`${type}/${( uri.toString())}`);
-                flushCachesIfRequired();
-            }
-        };
-    }
-    _enqueueReadonlyUpdate(uri) {
-        this._pendingReadonlyUris.push(uri);
-        if (!this._pendingReadonlyFlush) {
-            this._pendingReadonlyFlush = true;
-            queueMicrotask(() => {
-                const uris = this._pendingReadonlyUris;
-                this._pendingReadonlyUris = [];
-                this._pendingReadonlyFlush = false;
-                void this.filesConfigService.updateReadonly(uris, true);
-            });
-        }
-    }
-    _updateContributedWhenKeys() {
-        this._contributedWhenKeys.clear();
-        for (const whenClause of ( this._contributedWhenClauses.values())) {
-            const expr = ContextKeyExpr.deserialize(whenClause);
-            for (const key of expr?.keys() ?? []) {
-                this._contributedWhenKeys.add(key);
-            }
-        }
-        for (const whenClauses of ( this._providerWhenClauses.values())) {
-            for (const whenClause of whenClauses) {
-                const expr = ContextKeyExpr.deserialize(whenClause);
-                for (const key of expr?.keys() ?? []) {
-                    this._contributedWhenKeys.add(key);
-                }
-            }
-        }
+        return this.extensionPromptFiles.registerContributedFile(type, uri, extension, name, description, when, sessionTypes);
     }
     getPromptLocationLabel(promptPath) {
         switch (promptPath.storage) {
@@ -784,17 +565,17 @@ let PromptsService = class PromptsService extends Disposable {
                 relative: true
             });
         case PromptsStorage.user:
-            return localize(8085, "User Data");
+            return localize(8170, "User Data");
         case PromptsStorage.extension:
             {
                 return localize(
-                    8086,
+                    8171,
                     "Extension: {0}",
                     promptPath.extension.displayName ?? promptPath.extension.id
                 );
             }
         case PromptsStorage.plugin:
-            return localize(8087, "Plugin");
+            return localize(8172, "Plugin");
         default:
             assertNever(promptPath, "Unknown prompt storage type");
         }
@@ -938,30 +719,6 @@ let PromptsService = class PromptsService extends Disposable {
     sanitizeAgentSkillText(text) {
         return text.replace(/<[^>]+>/g, "");
     }
-    async validateAndSanitizeSkillFile(uri, token) {
-        const parsedFile = await this.parseNew(uri, token);
-        const folderName = getSkillFolderName(uri);
-        let name = parsedFile.header?.name;
-        if (!name) {
-            this.logger.debug(
-                `[validateAndSanitizeSkillFile] Agent skill file missing name attribute, using folder name "${folderName}": ${uri}`
-            );
-            name = folderName;
-        }
-        const description = parsedFile.header?.description;
-        let sanitizedName = this.truncateAgentSkillName(name, uri);
-        if (sanitizedName !== folderName) {
-            this.logger.debug(
-                `[validateAndSanitizeSkillFile] Agent skill name "${sanitizedName}" does not match folder name "${folderName}", using folder name: ${uri}`
-            );
-            sanitizedName = folderName;
-        }
-        const sanitizedDescription = description ? this.truncateAgentSkillDescription(description, uri) : undefined;
-        return {
-            name: sanitizedName,
-            description: sanitizedDescription
-        };
-    }
     truncateAgentSkillName(name, uri) {
         const MAX_NAME_LENGTH = 64;
         const sanitized = this.sanitizeAgentSkillText(name);
@@ -1467,7 +1224,7 @@ let PromptsService = class PromptsService extends Disposable {
         };
     }
 };
-PromptsService = ( __decorate([( __param(0, ILogService)), ( __param(1, ILabelService)), ( __param(2, IModelService)), ( __param(3, IInstantiationService)), ( __param(4, IUserDataProfileService)), ( __param(5, IConfigurationService)), ( __param(6, IFileService)), ( __param(7, IFilesConfigurationService)), ( __param(8, IStorageService)), ( __param(9, IExtensionService)), ( __param(10, ITelemetryService)), ( __param(11, IWorkspaceContextService)), ( __param(12, IPathService)), ( __param(13, IContextKeyService)), ( __param(14, IAgentPluginService)), ( __param(15, IWorkspaceTrustManagementService))], PromptsService));
+PromptsService = ( __decorate([( __param(0, ILogService)), ( __param(1, ILabelService)), ( __param(2, IModelService)), ( __param(3, IInstantiationService)), ( __param(4, IUserDataProfileService)), ( __param(5, IConfigurationService)), ( __param(6, IFileService)), ( __param(7, IStorageService)), ( __param(8, ITelemetryService)), ( __param(9, IWorkspaceContextService)), ( __param(10, IPathService)), ( __param(11, IAgentPluginService)), ( __param(12, IWorkspaceTrustManagementService))], PromptsService));
 class CachedPromise extends Disposable {
     constructor(computeFn, getEvent, delay = 0) {
         super();

package/vscode/src/vs/workbench/contrib/chat/common/promptSyntax/utils/promptFilesLocator.js

@@ -57,7 +57,7 @@ let PromptFilesLocator = class PromptFilesLocator {
             filePattern: undefined,
             source: PromptFileSource.UserData,
             storage: PromptsStorage.user,
-            displayPath: ( localize(8088, "User Data")),
+            displayPath: ( localize(8173, "User Data")),
             isDefault: true
         };
     }

package/vscode/src/vs/workbench/contrib/chat/common/tools/builtinTools/askQuestionsTool.js

@@ -107,9 +107,9 @@ function createAskQuestionsToolData() {
         legacyToolReferenceFullNames: [AskQuestionsToolId, "vscode/askQuestions"],
         canBeReferencedInPrompt: false,
         icon: ThemeIcon.fromId(Codicon.question.id),
-        displayName: ( localize(8089, "Ask Clarifying Questions")),
+        displayName: ( localize(8174, "Ask Clarifying Questions")),
         userDescription: ( localize(
-            8090,
+            8175,
             "Ask structured clarifying questions using single select, multi-select, or freeform inputs to collect task requirements before proceeding."
         )),
         modelDescription: "Use this tool to ask the user a small number of clarifying questions before proceeding. Provide the questions array with concise headers and prompts. Use options for fixed choices, set multiSelect when multiple selections are allowed. Users can always provide a freeform text answer alongside options unless you set allowFreeformInput to false.",
@@ -135,7 +135,7 @@ let AskQuestionsTool = class AskQuestionsTool extends Disposable {
         this.logService.trace(`[AskQuestionsTool] Invoking with ${questions?.length ?? 0} question(s)`);
         if (!questions || questions.length === 0) {
             throw ( new Error(( localize(
-                8091,
+                8176,
                 "No questions provided. The questions array must contain at least one question."
             ))));
         }
@@ -210,7 +210,7 @@ let AskQuestionsTool = class AskQuestionsTool extends Disposable {
             };
         }
         progress.report({
-            message: ( localize(8092, "Analyzing your answers..."))
+            message: ( localize(8177, "Analyzing your answers..."))
         });
         const converted = this.convertCarouselAnswers(questions, answerResult?.answers, idToHeaderMap);
         const {
@@ -248,14 +248,14 @@ let AskQuestionsTool = class AskQuestionsTool extends Disposable {
         } = parameters;
         if (!questions || questions.length === 0) {
             throw ( new Error(( localize(
-                8091,
+                8176,
                 "No questions provided. The questions array must contain at least one question."
             ))));
         }
         for (const question of questions) {
             if (question.options && question.options.length === 1) {
                 throw ( new Error(( localize(
-                    8093,
+                    8178,
                     "Question \"{0}\" must have at least two options, or none for free text input.",
                     question.header
                 ))));
@@ -263,8 +263,8 @@ let AskQuestionsTool = class AskQuestionsTool extends Disposable {
         }
         const questionCount = questions.length;
         const headers = ( questions.map(q => q.header)).join(", ");
-        const message = questionCount === 1 ? ( localize(8094, "Asking a question ({0})", headers)) : ( localize(8095, "Asking {0} questions ({1})", questionCount, headers));
-        const pastMessage = questionCount === 1 ? ( localize(8096, "Asked a question ({0})", headers)) : ( localize(8097, "Asked {0} questions ({1})", questionCount, headers));
+        const message = questionCount === 1 ? ( localize(8179, "Asking a question ({0})", headers)) : ( localize(8180, "Asking {0} questions ({1})", questionCount, headers));
+        const pastMessage = questionCount === 1 ? ( localize(8181, "Asked a question ({0})", headers)) : ( localize(8182, "Asked {0} questions ({1})", questionCount, headers));
         return {
             invocationMessage: ( new MarkdownString(message)),
             pastTenseMessage: ( new MarkdownString(pastMessage))

package/vscode/src/vs/workbench/contrib/chat/common/tools/builtinTools/manageTodoListTool.js

@@ -48,8 +48,8 @@ function createManageTodoListToolData() {
         legacyToolReferenceFullNames: ["todos"],
         canBeReferencedInPrompt: true,
         icon: ThemeIcon.fromId(Codicon.checklist.id),
-        displayName: ( localize(8098, "Manage and track todo items for task planning")),
-        userDescription: ( localize(8099, "Manage and track todo items for task planning")),
+        displayName: ( localize(8183, "Manage and track todo items for task planning")),
+        userDescription: ( localize(8184, "Manage and track todo items for task planning")),
         modelDescription: "Manage a structured todo list to track progress and plan tasks throughout your coding session. Use this tool VERY frequently to ensure task visibility and proper planning.\n\nWhen to use this tool:\n- Complex multi-step work requiring planning and tracking\n- When user provides multiple tasks or requests (numbered/comma-separated)\n- After receiving new instructions that require multiple steps\n- BEFORE starting work on any todo (mark as in-progress)\n- IMMEDIATELY after completing each todo (mark completed individually)\n- When breaking down larger tasks into smaller actionable steps\n- To give users visibility into your progress and planning\n\nWhen NOT to use:\n- Single, trivial tasks that can be completed in one step\n- Purely conversational/informational requests\n- When just reading files or performing simple searches\n\nCRITICAL workflow:\n1. Plan tasks by writing todo list with specific, actionable items\n2. Mark ONE todo as in-progress before starting work\n3. Complete the work for that specific todo\n4. Mark that todo as completed IMMEDIATELY\n5. Move to next todo and repeat\n\nTodo states:\n- not-started: Todo not yet begun\n- in-progress: Currently working (limit ONE at a time)\n- completed: Finished successfully\n\nIMPORTANT: Mark todos completed as soon as they are done. Do not batch completions.",
         source: ToolDataSource.Internal,
         inputSchema: inputSchema
@@ -107,7 +107,7 @@ let ManageTodoListTool = class ManageTodoListTool extends Disposable {
         const currentTodoItems = this.chatTodoListService.getTodos(chatSessionResource);
         let message;
         if (args.operation === "read") {
-            message = ( localize(8100, "Read todo list"));
+            message = ( localize(8185, "Read todo list"));
         } else if (args.todoList) {
             message = this.generatePastTenseMessage(currentTodoItems, args.todoList);
         }
@@ -117,12 +117,12 @@ let ManageTodoListTool = class ManageTodoListTool extends Disposable {
             title: todo.title,
             status: todo.status
         })));
-        const invocationLabel = message?.replace(/^(Starting|Completed): /i, "") ?? ( localize(8101, "Updating todo list"));
+        const invocationLabel = message?.replace(/^(Starting|Completed): /i, "") ?? ( localize(8186, "Updating todo list"));
         const invocationMessage = ( new MarkdownString(invocationLabel));
         return {
             invocationMessage,
             presentation: items.length ? undefined : ToolInvocationPresentation.Hidden,
-            pastTenseMessage: ( new MarkdownString(message ?? ( localize(8102, "Updated todo list")))),
+            pastTenseMessage: ( new MarkdownString(message ?? ( localize(8187, "Updated todo list")))),
             toolSpecificData: {
                 kind: "todoList",
                 todoList: todoList
@@ -131,7 +131,7 @@ let ManageTodoListTool = class ManageTodoListTool extends Disposable {
     }
     generatePastTenseMessage(currentTodos, newTodos) {
         if (currentTodos.length === 0) {
-            return newTodos.length === 1 ? ( localize(8103, "Created 1 todo")) : ( localize(8104, "Created {0} todos", newTodos.length));
+            return newTodos.length === 1 ? ( localize(8188, "Created 1 todo")) : ( localize(8189, "Created {0} todos", newTodos.length));
         }
         const currentTodoMap = ( new Map(( currentTodos.map(todo => [todo.id, todo]))));
         const startedTodos = newTodos.filter(newTodo => {
@@ -143,7 +143,7 @@ let ManageTodoListTool = class ManageTodoListTool extends Disposable {
             const totalTodos = newTodos.length;
             const currentPosition = newTodos.findIndex(todo => todo.id === startedTodo.id) + 1;
             return localize(
-                8105,
+                8190,
                 "Starting: *{0}* ({1}/{2})",
                 startedTodo.title,
                 currentPosition,
@@ -159,7 +159,7 @@ let ManageTodoListTool = class ManageTodoListTool extends Disposable {
             const totalTodos = newTodos.length;
             const currentPosition = newTodos.findIndex(todo => todo.id === completedTodo.id) + 1;
             return localize(
-                8106,
+                8191,
                 "Completed: *{0}* ({1}/{2})",
                 completedTodo.title,
                 currentPosition,
@@ -168,9 +168,9 @@ let ManageTodoListTool = class ManageTodoListTool extends Disposable {
         }
         const addedTodos = newTodos.filter(newTodo => !( currentTodoMap.has(newTodo.id)));
         if (addedTodos.length > 0) {
-            return addedTodos.length === 1 ? ( localize(8107, "Added 1 todo")) : ( localize(8108, "Added {0} todos", addedTodos.length));
+            return addedTodos.length === 1 ? ( localize(8192, "Added 1 todo")) : ( localize(8193, "Added {0} todos", addedTodos.length));
         }
-        return localize(8109, "Updated todo list");
+        return localize(8194, "Updated todo list");
     }
     handleRead(todoItems, sessionResource) {
         if (todoItems.length === 0) {

package/vscode/src/vs/workbench/contrib/chat/common/tools/builtinTools/reviewPlanTool.js

@@ -72,8 +72,8 @@ function createReviewPlanToolData() {
         toolReferenceName: "reviewPlan",
         canBeReferencedInPrompt: false,
         icon: ThemeIcon.fromId(Codicon.checklist.id),
-        displayName: ( localize(8110, "Review Plan")),
-        userDescription: ( localize(8111, "Ask the user to review and approve a plan before proceeding.")),
+        displayName: ( localize(8195, "Review Plan")),
+        userDescription: ( localize(8196, "Ask the user to review and approve a plan before proceeding.")),
         modelDescription: "Use this tool to present a plan to the user for review. Provide the plan content as markdown, a list of approval actions (with optional default), and whether the user can provide freeform feedback. Optionally provide a URI to the backing plan file so the user can edit it. The tool returns the chosen action, whether the plan was rejected, and any feedback.",
         source: ToolDataSource.Internal,
         inputSchema
@@ -96,7 +96,7 @@ let ReviewPlanTool = class ReviewPlanTool extends Disposable {
             canProvideFeedback
         } = parameters;
         if (!actions || actions.length === 0) {
-            throw ( new Error(( localize(8112, "At least one approval action must be provided."))));
+            throw ( new Error(( localize(8197, "At least one approval action must be provided."))));
         }
         const {
             request
@@ -119,7 +119,7 @@ let ReviewPlanTool = class ReviewPlanTool extends Disposable {
                 }
             }
         }
-        const reviewData = ( new ChatPlanReviewData(title ?? ( localize(8113, "Review plan")), content, actions, canProvideFeedback, planUri?.toJSON(), generateUuid()));
+        const reviewData = ( new ChatPlanReviewData(title ?? ( localize(8198, "Review plan")), content, actions, canProvideFeedback, planUri?.toJSON(), generateUuid()));
         this.chatService.appendProgress(request, reviewData);
         const result = await raceCancellation(reviewData.completion.p, token);
         if (token.isCancellationRequested) {
@@ -132,11 +132,11 @@ let ReviewPlanTool = class ReviewPlanTool extends Disposable {
     async prepareToolInvocation(context, _token) {
         const parameters = context.parameters;
         if (!parameters.actions || parameters.actions.length === 0) {
-            throw ( new Error(( localize(8112, "At least one approval action must be provided."))));
+            throw ( new Error(( localize(8197, "At least one approval action must be provided."))));
         }
         return {
-            invocationMessage: ( new MarkdownString(( localize(8114, "Asking you to review the plan")))),
-            pastTenseMessage: ( new MarkdownString(( localize(8115, "Asked you to review the plan"))))
+            invocationMessage: ( new MarkdownString(( localize(8199, "Asking you to review the plan")))),
+            pastTenseMessage: ( new MarkdownString(( localize(8200, "Asked you to review the plan"))))
         };
     }
     toResult(result) {