fix: resolve dependency injection issues

Fixes eclipse-theia#13320

Contributed on behalf of STMicroelectronics

Signed-off-by: Olaf Lessenich <[email protected]>
Co-authored-by: Christian W. Damus <[email protected]>

Author: xai

packages/plugin-ext/src/hosted/browser/worker/worker-main.ts

@@ -20,7 +20,7 @@ import * as theia from '@theia/plugin';
 import { emptyPlugin, MAIN_RPC_CONTEXT, Plugin } from '../../../common/plugin-api-rpc';
 import { ExtPluginApi } from '../../../common/plugin-ext-api-contribution';
 import { getPluginId, PluginMetadata } from '../../../common/plugin-protocol';
-import { RPCProtocol, RPCProtocolImpl } from '../../../common/rpc-protocol';
+import { RPCProtocol } from '../../../common/rpc-protocol';
 import { ClipboardExt } from '../../../plugin/clipboard-ext';
 import { EditorsAndDocumentsExtImpl } from '../../../plugin/editors-and-documents';
 import { MessageRegistryExt } from '../../../plugin/message-registry';
@@ -56,7 +56,7 @@ function initialize(contextPath: string, pluginMetadata: PluginMetadata): void {
 const container = new Container();
 container.load(pluginHostModule);
 
-const rpc: RPCProtocol = container.get(RPCProtocolImpl);
+const rpc: RPCProtocol = container.get(RPCProtocol);
 const pluginManager = container.get(PluginManagerExtImpl);
 pluginManager.setPluginHost({
         // eslint-disable-next-line @typescript-eslint/no-explicit-any

packages/plugin-ext/src/hosted/browser/worker/worker-plugin-module.ts

@@ -23,10 +23,10 @@ import { RPCProtocol, RPCProtocolImpl } from '../../../common/rpc-protocol';
 import { ClipboardExt } from '../../../plugin/clipboard-ext';
 import { EditorsAndDocumentsExtImpl } from '../../../plugin/editors-and-documents';
 import { MessageRegistryExt } from '../../../plugin/message-registry';
-import { PluginManagerExtImpl } from '../../../plugin/plugin-manager';
-import { KeyValueStorageProxy } from '../../../plugin/plugin-storage';
+import { MinimalTerminalServiceExt, PluginManagerExtImpl } from '../../../plugin/plugin-manager';
+import { InternalStorageExt, KeyValueStorageProxy } from '../../../plugin/plugin-storage';
 import { PreferenceRegistryExtImpl } from '../../../plugin/preference-registry';
-import { SecretsExtImpl } from '../../../plugin/secrets-ext';
+import { InternalSecretsExt, SecretsExtImpl } from '../../../plugin/secrets-ext';
 import { TerminalServiceExtImpl } from '../../../plugin/terminal-ext';
 import { WebviewsExtImpl } from '../../../plugin/webviews';
 import { WorkspaceExtImpl } from '../../../plugin/workspace';
@@ -60,7 +60,9 @@ export default new ContainerModule(bind => {
     bind(EnvExtImpl).to(WorkerEnvExtImpl).inSingletonScope();
     bind(LocalizationExt).to(LocalizationExtImpl).inSingletonScope();
     bind(KeyValueStorageProxy).toSelf().inSingletonScope();
+    bind(InternalStorageExt).toService(KeyValueStorageProxy);
     bind(SecretsExtImpl).toSelf().inSingletonScope();
+    bind(InternalSecretsExt).toService(SecretsExtImpl);
     bind(PreferenceRegistryExtImpl).toSelf().inSingletonScope();
     bind(DebugExtImpl).toDynamicValue(({ container }) => createDebugExtStub(container))
         .inSingletonScope();
@@ -70,4 +72,5 @@ export default new ContainerModule(bind => {
     bind(ClipboardExt).toSelf().inSingletonScope();
     bind(WebviewsExtImpl).toSelf().inSingletonScope();
     bind(TerminalServiceExtImpl).toSelf().inSingletonScope();
+    bind(MinimalTerminalServiceExt).toService(TerminalServiceExtImpl);
 });

packages/plugin-ext/src/plugin/plugin-manager.ts

@@ -81,7 +81,7 @@ class ActivatedPlugin {
 
 export const MinimalTerminalServiceExt = Symbol('MinimalTerminalServiceExt');
 export type MinimalTerminalServiceExt = Pick<TerminalServiceExt,
-    'getEnvironmentVariableCollection'|'$initEnvironmentVariableCollections'|'$setShell'>;
+    'getEnvironmentVariableCollection' | '$initEnvironmentVariableCollections' | '$setShell'>;
 
 @injectable()
 // eslint-disable-next-line @typescript-eslint/no-explicit-any