@theia/plugin-ext 1.73.0-next.20 → 1.73.0-next.24

package/src/hosted/node/scanners/scanner-theia.ts

@@ -16,7 +16,7 @@
 
 /* eslint-disable @theia/localization-check */
 
-import { inject, injectable, unmanaged } from '@theia/core/shared/inversify';
+import { inject, injectable, unmanaged, named } from '@theia/core/shared/inversify';
 import {
     AutoClosingPair,
     AutoClosingPairConditional,
@@ -82,6 +82,7 @@ import { ColorDefinition } from '@theia/core/lib/common/color';
 import { CSSIcon } from '@theia/core/lib/common/markdown-rendering/icon-utilities';
 import { PluginUriFactory } from './plugin-uri-factory';
 import { PreferenceScope } from '@theia/core/lib/common/preferences/preference-scope';
+import { ILogger } from '@theia/core';
 
 const colorIdPattern = '^\\w+[.\\w+]*$';
 const iconIdPattern = `^${CSSIcon.iconNameSegment}(-${CSSIcon.iconNameSegment})+$`;
@@ -284,6 +285,9 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
         super('theiaPlugin', 'backend-init-theia');
     }
 
+    @inject(ILogger) @named('plugin-ext:TheiaPluginScanner')
+    protected readonly logger: ILogger;
+
     protected getEntryPoint(plugin: PluginPackage): PluginEntryPoint {
         const result: PluginEntryPoint = {
             frontend: plugin.theiaPlugin!.frontend,
@@ -335,7 +339,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 }
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'configuration'.`, rawPlugin.contributes.configuration, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'configuration'.`, rawPlugin.contributes.configuration, err);
         }
 
         const configurationDefaults = rawPlugin.contributes.configurationDefaults;
@@ -346,7 +350,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.submenus = this.readSubmenus(rawPlugin.contributes.submenus, rawPlugin);
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'submenus'.`, rawPlugin.contributes.submenus, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'submenus'.`, rawPlugin.contributes.submenus, err);
         }
 
         try {
@@ -355,7 +359,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.customEditors = customEditors;
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'customEditors'.`, rawPlugin.contributes.customEditors, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'customEditors'.`, rawPlugin.contributes.customEditors, err);
         }
 
         try {
@@ -374,7 +378,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 }
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'viewsContainers'.`, rawPlugin.contributes.viewsContainers, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'viewsContainers'.`, rawPlugin.contributes.viewsContainers, err);
         }
 
         try {
@@ -387,7 +391,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 }
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'views'.`, rawPlugin.contributes.views, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'views'.`, rawPlugin.contributes.views, err);
         }
 
         try {
@@ -395,7 +399,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.viewsWelcome = this.readViewsWelcome(rawPlugin.contributes!.viewsWelcome, rawPlugin.contributes.views);
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'viewsWelcome'.`, rawPlugin.contributes.viewsWelcome, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'viewsWelcome'.`, rawPlugin.contributes.viewsWelcome, err);
         }
 
         try {
@@ -405,7 +409,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.commands = commands.map(command => this.readCommand(command, rawPlugin));
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'commands'.`, rawPlugin.contributes!.commands, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'commands'.`, rawPlugin.contributes!.commands, err);
         }
 
         try {
@@ -418,7 +422,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 }
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'menus'.`, rawPlugin.contributes.menus, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'menus'.`, rawPlugin.contributes.menus, err);
         }
 
         try {
@@ -427,7 +431,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.keybindings = rawKeybindings.map(rawKeybinding => this.readKeybinding(rawKeybinding));
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'keybindings'.`, rawPlugin.contributes.keybindings, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'keybindings'.`, rawPlugin.contributes.keybindings, err);
         }
 
         try {
@@ -436,7 +440,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.debuggers = debuggers;
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'debuggers'.`, rawPlugin.contributes.debuggers, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'debuggers'.`, rawPlugin.contributes.debuggers, err);
         }
 
         try {
@@ -445,91 +449,91 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 contributions.taskDefinitions = definitions.map(definitionContribution => this.readTaskDefinition(rawPlugin.name, definitionContribution));
             }
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'taskDefinitions'.`, rawPlugin.contributes.taskDefinitions, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'taskDefinitions'.`, rawPlugin.contributes.taskDefinitions, err);
         }
 
         try {
             contributions.problemMatchers = rawPlugin.contributes.problemMatchers;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'problemMatchers'.`, rawPlugin.contributes.problemMatchers, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'problemMatchers'.`, rawPlugin.contributes.problemMatchers, err);
         }
 
         try {
             contributions.problemPatterns = rawPlugin.contributes.problemPatterns;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'problemPatterns'.`, rawPlugin.contributes.problemPatterns, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'problemPatterns'.`, rawPlugin.contributes.problemPatterns, err);
         }
 
         try {
             contributions.resourceLabelFormatters = rawPlugin.contributes.resourceLabelFormatters;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'resourceLabelFormatters'.`, rawPlugin.contributes.resourceLabelFormatters, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'resourceLabelFormatters'.`, rawPlugin.contributes.resourceLabelFormatters, err);
         }
 
         try {
             contributions.authentication = rawPlugin.contributes.authentication;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'authentication'.`, rawPlugin.contributes.authentication, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'authentication'.`, rawPlugin.contributes.authentication, err);
         }
 
         try {
             contributions.notebooks = rawPlugin.contributes.notebooks;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'notebooks'.`, rawPlugin.contributes.notebooks, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'notebooks'.`, rawPlugin.contributes.notebooks, err);
         }
 
         try {
             contributions.notebookRenderer = rawPlugin.contributes.notebookRenderer;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'notebook-renderer'.`, rawPlugin.contributes.notebookRenderer, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'notebook-renderer'.`, rawPlugin.contributes.notebookRenderer, err);
         }
 
         try {
             contributions.notebookPreload = rawPlugin.contributes.notebookPreload;
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'notebooks-preload'.`, rawPlugin.contributes.notebookPreload, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'notebooks-preload'.`, rawPlugin.contributes.notebookPreload, err);
         }
 
         try {
             contributions.snippets = this.readSnippets(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'snippets'.`, rawPlugin.contributes!.snippets, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'snippets'.`, rawPlugin.contributes!.snippets, err);
         }
 
         try {
             contributions.themes = this.readThemes(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'themes'.`, rawPlugin.contributes.themes, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'themes'.`, rawPlugin.contributes.themes, err);
         }
 
         try {
             contributions.icons = this.readIcons(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'icons'.`, rawPlugin.contributes.icons, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'icons'.`, rawPlugin.contributes.icons, err);
         }
 
         try {
             contributions.iconThemes = this.readIconThemes(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'iconThemes'.`, rawPlugin.contributes.iconThemes, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'iconThemes'.`, rawPlugin.contributes.iconThemes, err);
         }
 
         try {
             contributions.colors = this.readColors(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'colors'.`, rawPlugin.contributes.colors, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'colors'.`, rawPlugin.contributes.colors, err);
         }
 
         try {
             contributions.terminalProfiles = this.readTerminals(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'terminals'.`, rawPlugin.contributes.terminal, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'terminals'.`, rawPlugin.contributes.terminal, err);
         }
 
         try {
             contributions.localizations = this.readLocalizations(rawPlugin);
         } catch (err) {
-            console.error(`Could not read '${rawPlugin.name}' contribution 'localizations'.`, rawPlugin.contributes.localizations, err);
+            this.logger.error(`Could not read '${rawPlugin.name}' contribution 'localizations'.`, rawPlugin.contributes.localizations, err);
         }
 
         const [languagesResult, grammarsResult] = await Promise.allSettled([
@@ -541,7 +545,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             if (languagesResult.status === 'fulfilled') {
                 contributions.languages = languagesResult.value;
             } else {
-                console.error(`Could not read '${rawPlugin.name}' contribution 'languages'.`, rawPlugin.contributes.languages, languagesResult.reason);
+                this.logger.error(`Could not read '${rawPlugin.name}' contribution 'languages'.`, rawPlugin.contributes.languages, languagesResult.reason);
             }
         }
 
@@ -549,7 +553,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             if (grammarsResult.status === 'fulfilled') {
                 contributions.grammars = grammarsResult.value;
             } else {
-                console.error(`Could not read '${rawPlugin.name}' contribution 'grammars'.`, rawPlugin.contributes.grammars, grammarsResult.reason);
+                this.logger.error(`Could not read '${rawPlugin.name}' contribution 'grammars'.`, rawPlugin.contributes.grammars, grammarsResult.reason);
             }
         }
 
@@ -624,20 +628,20 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
         const result: ColorDefinition[] = [];
         for (const contribution of pck.contributes.colors) {
             if (typeof contribution.id !== 'string' || contribution.id.length === 0) {
-                console.error("'configuration.colors.id' must be defined and can not be empty");
+                this.logger.error("'configuration.colors.id' must be defined and can not be empty");
                 continue;
             }
             if (!contribution.id.match(colorIdPattern)) {
-                console.error("'configuration.colors.id' must follow the word[.word]*");
+                this.logger.error("'configuration.colors.id' must follow the word[.word]*");
                 continue;
             }
             if (typeof contribution.description !== 'string' || contribution.id.length === 0) {
-                console.error("'configuration.colors.description' must be defined and can not be empty");
+                this.logger.error("'configuration.colors.description' must be defined and can not be empty");
                 continue;
             }
             const defaults = contribution.defaults;
             if (!defaults || typeof defaults !== 'object' || typeof defaults.light !== 'string' || typeof defaults.dark !== 'string' || typeof defaults.highContrast !== 'string') {
-                console.error("'configuration.colors.defaults' must be defined and must contain 'light', 'dark' and 'highContrast'");
+                this.logger.error("'configuration.colors.defaults' must be defined and must contain 'light', 'dark' and 'highContrast'");
                 continue;
             }
             result.push({
@@ -679,11 +683,11 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
         const result: IconThemeContribution[] = [];
         for (const contribution of pck.contributes.iconThemes) {
             if (typeof contribution.id !== 'string') {
-                console.error('Expected string in `contributes.iconThemes.id`. Provided value:', contribution.id);
+                this.logger.error('Expected string in `contributes.iconThemes.id`. Provided value:', contribution.id);
                 continue;
             }
             if (typeof contribution.path !== 'string') {
-                console.error('Expected string in `contributes.iconThemes.path`. Provided value:', contribution.path);
+                this.logger.error('Expected string in `contributes.iconThemes.path`. Provided value:', contribution.path);
                 continue;
             }
             result.push({
@@ -706,13 +710,13 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
         for (const id in iconEntries) {
             if (pck.contributes.icons.hasOwnProperty(id)) {
                 if (!id.match(iconIdPattern)) {
-                    console.error("'configuration.icons' keys represent the icon id and can only contain letter, digits and minuses. " +
+                    this.logger.error("'configuration.icons' keys represent the icon id and can only contain letter, digits and minuses. " +
                         'They need to consist of at least two segments in the form `component-iconname`.', 'extension: ', pck.name, 'icon id: ', id);
                     return;
                 }
                 const iconContribution = iconEntries[id];
                 if (typeof iconContribution.description !== 'string' || iconContribution.description['length'] === 0) {
-                    console.error('configuration.icons.description must be defined and can not be empty, ', 'extension: ', pck.name, 'icon id: ', id);
+                    this.logger.error('configuration.icons.description must be defined and can not be empty, ', 'extension: ', pck.name, 'icon id: ', id);
                     return;
                 }
 
@@ -727,7 +731,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                 } else if (typeof defaultIcon === 'object' && typeof defaultIcon.fontPath === 'string' && typeof defaultIcon.fontCharacter === 'string') {
                     const format = getFileExtension(defaultIcon.fontPath);
                     if (['woff', 'woff2', 'ttf'].indexOf(format) === -1) {
-                        console.warn("Expected `contributes.icons.default.fontPath` to have file extension 'woff', woff2' or 'ttf', is '{0}'.", format);
+                        this.logger.warn("Expected `contributes.icons.default.fontPath` to have file extension 'woff', woff2' or 'ttf', is '{0}'.", format);
                         return;
                     }
 
@@ -742,7 +746,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
                         }
                     });
                 } else {
-                    console.error("'configuration.icons.default' must be either a reference to the id of an other theme icon (string) or a icon definition (object) with ",
+                    this.logger.error("'configuration.icons.default' must be either a reference to the id of an other theme icon (string) or a icon definition (object) with ",
                         'properties `fontPath` and `fontCharacter`.');
                 }
             }
@@ -777,7 +781,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             return content;
         } catch (e) {
             if (!isENOENT(e)) {
-                console.error(e);
+                this.logger.error(e);
             }
             return '';
         }
@@ -1018,7 +1022,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             return undefined;
         }
         if (!Array.isArray(source)) {
-            console.warn(`[${langId}]: language configuration: expected \`autoClosingPairs\` to be an array.`);
+            this.logger.warn(`[${langId}]: language configuration: expected \`autoClosingPairs\` to be an array.`);
             return undefined;
         }
 
@@ -1027,27 +1031,27 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             const pair = source[i];
             if (Array.isArray(pair)) {
                 if (!isCharacterPair(pair)) {
-                    console.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}]\` to be an array of two strings or an object.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}]\` to be an array of two strings or an object.`);
                     continue;
                 }
                 result = result || [];
                 result.push({ open: pair[0], close: pair[1] });
             } else {
                 if (!isObject(pair)) {
-                    console.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}]\` to be an array of two strings or an object.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}]\` to be an array of two strings or an object.`);
                     continue;
                 }
                 if (typeof pair.open !== 'string') {
-                    console.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}].open\` to be a string.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}].open\` to be a string.`);
                     continue;
                 }
                 if (typeof pair.close !== 'string') {
-                    console.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}].close\` to be a string.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}].close\` to be a string.`);
                     continue;
                 }
                 if (typeof pair.notIn !== 'undefined') {
                     if (!isStringArray(pair.notIn)) {
-                        console.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}].notIn\` to be a string array.`);
+                        this.logger.warn(`[${langId}]: language configuration: expected \`autoClosingPairs[${i}].notIn\` to be a string array.`);
                         continue;
                     }
                 }
@@ -1064,7 +1068,7 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             return undefined;
         }
         if (!Array.isArray(source)) {
-            console.warn(`[${langId}]: language configuration: expected \`surroundingPairs\` to be an array.`);
+            this.logger.warn(`[${langId}]: language configuration: expected \`surroundingPairs\` to be an array.`);
             return undefined;
         }
 
@@ -1073,22 +1077,22 @@ export class TheiaPluginScanner extends AbstractPluginScanner {
             const pair = source[i];
             if (Array.isArray(pair)) {
                 if (!isCharacterPair(pair)) {
-                    console.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}]\` to be an array of two strings or an object.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}]\` to be an array of two strings or an object.`);
                     continue;
                 }
                 result = result || [];
                 result.push({ open: pair[0], close: pair[1] });
             } else {
                 if (!isObject(pair)) {
-                    console.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}]\` to be an array of two strings or an object.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}]\` to be an array of two strings or an object.`);
                     continue;
                 }
                 if (typeof pair.open !== 'string') {
-                    console.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}].open\` to be a string.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}].open\` to be a string.`);
                     continue;
                 }
                 if (typeof pair.close !== 'string') {
-                    console.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}].close\` to be a string.`);
+                    this.logger.warn(`[${langId}]: language configuration: expected \`surroundingPairs[${i}].close\` to be a string.`);
                     continue;
                 }
                 result = result || [];

package/src/main/browser/debug/plugin-debug-service.ts

@@ -16,13 +16,13 @@
 
 import { DebuggerDescription, DebugPath, DebugService, DynamicDebugConfigurationProvider } from '@theia/debug/lib/common/debug-service';
 import debounce = require('@theia/core/shared/lodash.debounce');
-import { deepClone, Emitter, Event, nls } from '@theia/core';
+import { deepClone, Emitter, Event, nls, ILogger } from '@theia/core';
 import { Disposable, DisposableCollection } from '@theia/core/lib/common/disposable';
 import { DebugConfiguration } from '@theia/debug/lib/common/debug-configuration';
 import { IJSONSchema, IJSONSchemaSnippet } from '@theia/core/lib/common/json-schema';
 import { PluginDebugAdapterContribution } from './plugin-debug-adapter-contribution';
 import { PluginDebugConfigurationProvider } from './plugin-debug-configuration-provider';
-import { injectable, inject, postConstruct } from '@theia/core/shared/inversify';
+import { injectable, inject, postConstruct, named } from '@theia/core/shared/inversify';
 import { WebSocketConnectionProvider } from '@theia/core/lib/browser/messaging/ws-connection-provider';
 import { WorkspaceService } from '@theia/workspace/lib/browser';
 import { CommandIdVariables } from '@theia/variable-resolver/lib/common/variable-types';
@@ -64,6 +64,8 @@ export class PluginDebugService implements DebugService {
     protected readonly connectionProvider: WebSocketConnectionProvider;
     @inject(WorkspaceService)
     protected readonly workspaceService: WorkspaceService;
+    @inject(ILogger) @named('plugin-ext:PluginDebugService')
+    protected readonly logger: ILogger;
 
     @postConstruct()
     protected init(): void {
@@ -84,7 +86,7 @@ export class PluginDebugService implements DebugService {
         const { type } = contrib;
 
         if (this.contributors.has(type)) {
-            console.warn(`Debugger with type '${type}' already registered.`);
+            this.logger.warn(`Debugger with type '${type}' already registered.`);
             return Disposable.NULL;
         }
 
@@ -105,7 +107,7 @@ export class PluginDebugService implements DebugService {
         if (this.configurationProviders.has(provider.handle)) {
             const configuration = this.configurationProviders.get(provider.handle);
             if (configuration && configuration.type !== provider.type) {
-                console.warn(`Different debug configuration provider with type '${configuration.type}' already registered.`);
+                this.logger.warn(`Different debug configuration provider with type '${configuration.type}' already registered.`);
                 provider.handle = this.configurationProviders.size;
             }
         }
@@ -358,7 +360,7 @@ export class PluginDebugService implements DebugService {
                 }
                 resolved = await resolver(workspaceFolderUri, resolved);
             } catch (e) {
-                console.error(e);
+                this.logger.error(e);
             }
         }
         return resolved;

package/src/main/browser/languages-main.ts

@@ -39,7 +39,7 @@ import {
     IdentifiableInlineCompletions,
     HoverWithId
 } from '../../common/plugin-api-rpc';
-import { injectable, inject } from '@theia/core/shared/inversify';
+import { injectable, inject, named } from '@theia/core/shared/inversify';
 import {
     SerializedDocumentFilter, MarkerData, Range, RelatedInformation,
     MarkerSeverity, DocumentLink, WorkspaceSymbolParams, CodeAction, CompletionDto,
@@ -86,6 +86,7 @@ import { ITextModel } from '@theia/monaco-editor-core/esm/vs/editor/common/model
 import { CodeActionTriggerKind } from '../../plugin/types-impl';
 import { IReadonlyVSDataTransfer } from '@theia/monaco-editor-core/esm/vs/base/common/dataTransfer';
 import { FileUploadService } from '@theia/filesystem/lib/common/upload/file-upload';
+import { ILogger } from '@theia/core';
 
 @injectable()
 export class LanguagesMainImpl implements LanguagesMain, Disposable {
@@ -108,6 +109,9 @@ export class LanguagesMainImpl implements LanguagesMain, Disposable {
     @inject(FileUploadService)
     protected readonly fileUploadService: FileUploadService;
 
+    @inject(ILogger) @named('plugin-ext:LanguagesMainImpl')
+    protected readonly logger: ILogger;
+
     private readonly proxy: LanguagesExt;
     private readonly services = new Map<number, Disposable>();
     private readonly toDispose = new DisposableCollection();
@@ -958,7 +962,7 @@ export class LanguagesMainImpl implements LanguagesMain, Disposable {
                 dispose: () => this.proxy.$releaseCodeActions(handle, actions.map(a => a.cacheId))
             };
         } catch (e) {
-            console.error(e);
+            this.logger.error(e);
             return undefined;
         }
     }

package/src/main/browser/plugin-contribution-handler.ts

@@ -52,6 +52,7 @@ import { ContextKeyService } from '@theia/core/lib/browser/context-key-service';
 import { LanguageService } from '@theia/core/lib/browser/language-service';
 import { ThemeIcon } from '@theia/monaco-editor-core/esm/vs/base/common/themables';
 import { JSONObject, JSONValue } from '@theia/core/shared/@lumino/coreutils';
+import { ILogger } from '@theia/core';
 
 // The enum export is missing from `vscode-textmate@9.2.0`
 const enum StandardTokenType {
@@ -147,6 +148,9 @@ export class PluginContributionHandler {
     @inject(ContextKeyService)
     protected readonly contextKeyService: ContextKeyService;
 
+    @inject(ILogger) @named('plugin-ext:PluginContributionHandler')
+    protected readonly logger: ILogger;
+
     protected readonly commandHandlers = new Map<string, CommandHandler['execute'] | undefined>();
 
     protected readonly onDidRegisterCommandHandlerEmitter = new Emitter<string>();
@@ -164,8 +168,8 @@ export class PluginContributionHandler {
         }
         const toDispose = new DisposableCollection(Disposable.create(() => { /* mark as not disposed */ }));
         /* eslint-disable @typescript-eslint/no-explicit-any */
-        const logError = (message: string, ...args: any[]) => console.error(`[${clientId}][${plugin.metadata.model.id}]: ${message}`, ...args);
-        const logWarning = (message: string, ...args: any[]) => console.warn(`[${clientId}][${plugin.metadata.model.id}]: ${message}`, ...args);
+        const logError = (message: string, ...args: any[]) => this.logger.error(`[${clientId}][${plugin.metadata.model.id}]: ${message}`, ...args);
+        const logWarning = (message: string, ...args: any[]) => this.logger.warn(`[${clientId}][${plugin.metadata.model.id}]: ${message}`, ...args);
         const pushContribution = (id: string, contribute: () => Disposable) => {
             if (toDispose.disposed) {
                 return;
@@ -517,7 +521,7 @@ export class PluginContributionHandler {
 
     registerCommand(command: Command, enablement?: string): Disposable {
         if (this.hasCommand(command.id)) {
-            console.warn(`command '${command.id}' already registered`);
+            this.logger.warn(`command '${command.id}' already registered`);
             return Disposable.NULL;
         }
 
@@ -565,7 +569,7 @@ export class PluginContributionHandler {
 
     registerCommandHandler(id: string, execute: CommandHandler['execute']): Disposable {
         if (this.hasCommandHandler(id)) {
-            console.warn(`command handler '${id}' already registered`);
+            this.logger.warn(`command handler '${id}' already registered`);
             return Disposable.NULL;
         }
 

package/src/main/browser/webview/webview-resource-cache.ts

@@ -14,9 +14,10 @@
 // SPDX-License-Identifier: EPL-2.0 OR GPL-2.0-only WITH Classpath-exception-2.0
 // *****************************************************************************
 
-import { injectable } from '@theia/core/shared/inversify';
+import { injectable, inject, named } from '@theia/core/shared/inversify';
 import { Deferred } from '@theia/core/lib/common/promise-util';
 import { MaybePromise } from '@theia/core/lib/common/types';
+import { ILogger } from '@theia/core';
 
 export interface WebviewResourceResponse {
     eTag: string | undefined,
@@ -29,6 +30,9 @@ export interface WebviewResourceResponse {
 @injectable()
 export class WebviewResourceCache {
 
+    @inject(ILogger) @named('plugin-ext:WebviewResourceCache')
+    protected readonly logger: ILogger;
+
     protected readonly cache = new Deferred<Cache | undefined>();
 
     constructor() {
@@ -39,7 +43,7 @@ export class WebviewResourceCache {
         try {
             this.cache.resolve(await caches.open('webview:v1'));
         } catch (e) {
-            console.error('Failed to enable webview caching: ', e);
+            this.logger.error('Failed to enable webview caching: ', e);
             this.cache.resolve(undefined);
         }
     }

package/src/main/browser/webview/webview.ts

@@ -22,7 +22,7 @@
 
 import * as mime from 'mime';
 import { JSONExt } from '@theia/core/shared/@lumino/coreutils';
-import { injectable, inject, postConstruct } from '@theia/core/shared/inversify';
+import { injectable, inject, postConstruct, named } from '@theia/core/shared/inversify';
 import { WebviewPanelOptions, WebviewPortMapping } from '@theia/plugin';
 import { BaseWidget, Message, codicon } from '@theia/core/lib/browser/widgets/widget';
 import { Disposable, DisposableCollection } from '@theia/core/lib/common/disposable';
@@ -49,7 +49,7 @@ import { FileService } from '@theia/filesystem/lib/browser/file-service';
 import { FileOperationError, FileOperationResult } from '@theia/filesystem/lib/common/files';
 import { BinaryBufferReadableStream } from '@theia/core/lib/common/buffer';
 import { ExtractableWidget } from '@theia/core/lib/browser/widgets/extractable-widget';
-import { MenuPath } from '@theia/core';
+import { MenuPath, ILogger } from '@theia/core';
 import { ContextMenuRenderer } from '@theia/core/lib/browser';
 import { ContextKeyService } from '@theia/core/lib/browser/context-key-service';
 import { PluginViewWidget } from '../view/plugin-view-widget';
@@ -166,6 +166,9 @@ export class WebviewWidget extends BaseWidget implements StatefulWidget, Extract
     @inject(ContextKeyService)
     protected readonly contextKeyService: ContextKeyService;
 
+    @inject(ILogger) @named('plugin-ext:WebviewWidget')
+    protected readonly logger: ILogger;
+
     viewState: WebviewPanelViewState = {
         visible: false,
         active: false,
@@ -626,7 +629,7 @@ export class WebviewWidget extends BaseWidget implements StatefulWidget, Extract
             await this.ready.promise;
             this.postMessage(channel, data);
         } catch (e) {
-            console.error(e);
+            this.logger.error(e);
         }
     }
 

package/src/main/node/plugin-deployer-impl.ts

@@ -42,7 +42,7 @@ export class PluginDeployerImpl implements PluginDeployer {
     protected readonly onDidDeployEmitter = new Emitter<void>();
     readonly onDidDeploy = this.onDidDeployEmitter.event;
 
-    @inject(ILogger)
+    @inject(ILogger) @named('plugin-ext:PluginDeployerImpl')
     protected readonly logger: ILogger;
 
     @inject(PluginDeployerHandler)
@@ -215,7 +215,7 @@ export class PluginDeployerImpl implements PluginDeployer {
                         }
                     }
                 } catch (e) {
-                    console.error(`Failed to resolve plugins from '${entry.id}'`, e);
+                    this.logger.error(`Failed to resolve plugins from '${entry.id}'`, e);
                     errors.push(e instanceof Error ? e : new Error(String(e)));
                 }
             }));