@theia/plugin-ext 1.29.0-next.9 → 1.30.0-next.1

package/src/plugin/types-impl.ts

@@ -20,7 +20,6 @@
 *--------------------------------------------------------------------------------------------*/
 
 /* eslint-disable no-null/no-null */
-/* eslint-disable @typescript-eslint/no-explicit-any */
 
 import { UUID } from '@theia/core/shared/@phosphor/coreutils';
 import { illegalArgument } from '../common/errors';
@@ -38,6 +37,7 @@ import { es5ClassCompat } from '../common/types';
  * A reviver that takes URI's transferred via JSON.stringify() and makes
  * instances of our local plugin API URI class (below)
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function reviver(key: string | undefined, value: any): any {
     const revived = ObjectsTransferrer.reviver(key, value);
     if (CodeURI.isUri(revived)) {
@@ -423,7 +423,7 @@ export class Position {
         return false;
     }
 
-    toJSON(): any {
+    toJSON(): unknown {
         return { line: this.line, character: this.character };
     }
 }
@@ -546,19 +546,17 @@ export class Range {
         return new Range(start, end);
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    static isRange(thing: any): thing is theia.Range {
+    static isRange(thing: unknown): thing is theia.Range {
         if (thing instanceof Range) {
             return true;
         }
-        if (!thing) {
-            return false;
-        }
-        return Position.isPosition((<Range>thing).start)
-            && Position.isPosition((<Range>thing).end);
+        const range = thing as theia.Range;
+        return !!thing && typeof thing === 'object'
+            && Position.isPosition(range.start)
+            && Position.isPosition(range.end);
     }
 
-    toJSON(): any {
+    toJSON(): unknown {
         return [this.start, this.end];
     }
 }
@@ -721,8 +719,7 @@ export class SnippetString {
 
 @es5ClassCompat
 export class ThemeColor {
-    constructor(public id: string) {
-    }
+    constructor(public id: string) { }
 }
 
 @es5ClassCompat
@@ -1100,18 +1097,18 @@ export class SignatureHelp {
 @es5ClassCompat
 export class Hover {
 
-    public contents: theia.MarkdownString[] | theia.MarkedString[];
+    public contents: Array<theia.MarkdownString | theia.MarkedString>;
     public range?: Range;
 
     constructor(
-        contents: theia.MarkdownString | theia.MarkedString | theia.MarkdownString[] | theia.MarkedString[],
+        contents: theia.MarkdownString | theia.MarkedString | Array<theia.MarkdownString | theia.MarkedString>,
         range?: Range
     ) {
         if (!contents) {
             illegalArgument('contents must be defined');
         }
         if (Array.isArray(contents)) {
-            this.contents = <theia.MarkdownString[] | theia.MarkedString[]>contents;
+            this.contents = contents;
         } else {
             this.contents = [contents];
         }
@@ -1556,8 +1553,6 @@ export class QuickInputButtons {
     static readonly Back: theia.QuickInputButton = {
         iconPath: {
             id: 'Back',
-            dark: '',
-            light: ''
         },
         tooltip: 'Back'
     };
@@ -1636,8 +1631,8 @@ export class FileSystemError extends Error {
 
         // workaround when extending builtin objects and when compiling to ES5, see:
         // https://github.com/Microsoft/TypeScript-wiki/blob/master/Breaking-Changes.md#extending-built-ins-like-error-array-and-map-may-no-longer-work
-        if (typeof (<any>Object).setPrototypeOf === 'function') {
-            (<any>Object).setPrototypeOf(this, FileSystemError.prototype);
+        if (typeof Object.setPrototypeOf === 'function') {
+            Object.setPrototypeOf(this, FileSystemError.prototype);
         }
 
         if (typeof Error.captureStackTrace === 'function' && typeof terminator === 'function') {
@@ -2555,7 +2550,7 @@ export class SemanticTokensLegend {
     }
 }
 
-function isStrArrayOrUndefined(arg: any): arg is string[] | undefined {
+function isStrArrayOrUndefined(arg: unknown): arg is string[] | undefined {
     return ((typeof arg === 'undefined') || (Array.isArray(arg) && arg.every(e => typeof e === 'string')));
 }
 
@@ -2593,7 +2588,7 @@ export class SemanticTokensBuilder {
 
     public push(line: number, char: number, length: number, tokenType: number, tokenModifiers?: number): void;
     public push(range: Range, tokenType: string, tokenModifiers?: string[]): void;
-    public push(arg0: any, arg1: any, arg2: any, arg3?: any, arg4?: any): void {
+    public push(arg0: number | Range, arg1: number | string, arg2?: number | string[], arg3?: number, arg4?: number): void {
         if (typeof arg0 === 'number' && typeof arg1 === 'number' && typeof arg2 === 'number' && typeof arg3 === 'number' &&
             (typeof arg4 === 'number' || typeof arg4 === 'undefined')) {
             if (typeof arg4 === 'undefined') {
@@ -2741,7 +2736,7 @@ export class SemanticTokensBuilder {
 
 @es5ClassCompat
 export class SemanticTokens {
-    readonly resultId?: string;
+    readonly resultId: string | undefined;
     readonly data: Uint32Array;
 
     constructor(data: Uint32Array, resultId?: string) {
@@ -2754,7 +2749,7 @@ export class SemanticTokens {
 export class SemanticTokensEdit {
     readonly start: number;
     readonly deleteCount: number;
-    readonly data?: Uint32Array;
+    readonly data: Uint32Array | undefined;
 
     constructor(start: number, deleteCount: number, data?: Uint32Array) {
         this.start = start;
@@ -2765,7 +2760,7 @@ export class SemanticTokensEdit {
 
 @es5ClassCompat
 export class SemanticTokensEdits {
-    readonly resultId?: string;
+    readonly resultId: string | undefined;
     readonly edits: SemanticTokensEdit[];
 
     constructor(edits: SemanticTokensEdit[], resultId?: string) {
@@ -2774,4 +2769,10 @@ export class SemanticTokensEdits {
     }
 }
 
+export enum InputBoxValidationSeverity {
+    Info = 1,
+    Warning = 2,
+    Error = 3
+}
+
 // #endregion

package/src/plugin/workspace.ts

@@ -243,8 +243,12 @@ export class WorkspaceExtImpl implements WorkspaceExt {
         // `file` and `untitled` schemas are reserved by `workspace.openTextDocument` API:
         // `file`-scheme for opening a file
         // `untitled`-scheme for opening a new file that should be saved
-        if (scheme === Schemes.file || scheme === Schemes.untitled || this.documentContentProviders.has(scheme)) {
+        if (scheme === Schemes.file || scheme === Schemes.untitled) {
             throw new Error(`Text Content Document Provider for scheme '${scheme}' is already registered`);
+        } else if (this.documentContentProviders.has(scheme)) {
+            // TODO: we should be able to handle multiple registrations, but for now we should ensure that it doesn't crash plugin activation.
+            console.warn(`Repeat registration of TextContentDocumentProvider for scheme '${scheme}'. This registration will be ignored.`);
+            return { dispose: () => { } };
         }
 
         this.documentContentProviders.set(scheme, provider);

package/src/plugin-ext-backend-electron-module.ts

@@ -18,7 +18,7 @@ import { ContainerModule } from '@theia/core/shared/inversify';
 import { bindElectronBackend } from './hosted/node-electron/plugin-ext-hosted-electron-backend-module';
 import { bindMainBackend } from './main/node/plugin-ext-backend-module';
 
-export default new ContainerModule(bind => {
-    bindMainBackend(bind);
+export default new ContainerModule((bind, unbind, isBound, rebind) => {
+    bindMainBackend(bind, unbind, isBound, rebind);
     bindElectronBackend(bind);
 });

package/src/plugin-ext-backend-module.ts

@@ -18,7 +18,7 @@ import { ContainerModule } from '@theia/core/shared/inversify';
 import { bindHostedBackend } from './hosted/node/plugin-ext-hosted-backend-module';
 import { bindMainBackend } from './main/node/plugin-ext-backend-module';
 
-export default new ContainerModule(bind => {
-    bindMainBackend(bind);
+export default new ContainerModule((bind, unbind, isBound, rebind) => {
+    bindMainBackend(bind, unbind, isBound, rebind);
     bindHostedBackend(bind);
 });