@theia/plugin-ext 1.28.0 → 1.29.0-next.11

package/src/plugin/languages/code-action.ts

@@ -145,20 +145,17 @@ export class CodeActionAdapter {
         return this.cacheId++;
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    private static _isCommand(smth: any): smth is theia.Command {
-        return typeof (<theia.Command>smth).command === 'string';
+    private static _isCommand(arg: unknown): arg is theia.Command {
+        return !!arg && typeof arg === 'object' && typeof (arg as theia.Command).command === 'string';
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    private static _isSelection(obj: any): obj is Selection {
-        return (
-            obj
-            && (typeof obj.selectionStartLineNumber === 'number')
-            && (typeof obj.selectionStartColumn === 'number')
-            && (typeof obj.positionLineNumber === 'number')
-            && (typeof obj.positionColumn === 'number')
-        );
+    private static _isSelection(obj: unknown): obj is Selection {
+        const selection = obj as Selection;
+        return !!obj && typeof obj === 'object'
+            && typeof selection.selectionStartLineNumber === 'number'
+            && typeof selection.selectionStartColumn === 'number'
+            && typeof selection.positionLineNumber === 'number'
+            && typeof selection.positionColumn === 'number';
     }
 
 }

package/src/plugin/languages/document-highlight.ts

@@ -55,8 +55,7 @@ export class DocumentHighlightAdapter {
         });
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    private isDocumentHighlightArray(array: any): array is types.DocumentHighlight[] {
+    private isDocumentHighlightArray(array: unknown): array is types.DocumentHighlight[] {
         return Array.isArray(array) && array.length > 0 && array[0] instanceof types.DocumentHighlight;
     }
 }

package/src/plugin/languages/evaluatable-expression.ts

@@ -0,0 +1,47 @@
+// *****************************************************************************
+// Copyright (C) 2022 STMicroelectronics and others.
+//
+// This program and the accompanying materials are made available under the
+// terms of the Eclipse Public License v. 2.0 which is available at
+// http://www.eclipse.org/legal/epl-2.0.
+//
+// This Source Code may also be made available under the following Secondary
+// Licenses when the conditions for such availability set forth in the Eclipse
+// Public License v. 2.0 are satisfied: GNU General Public License, version 2
+// with the GNU Classpath Exception which is available at
+// https://www.gnu.org/software/classpath/license.html.
+//
+// SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
+// *****************************************************************************
+
+import { URI } from '@theia/core/shared/vscode-uri';
+import * as theia from '@theia/plugin';
+import { Position } from '../../common/plugin-api-rpc';
+import { EvaluatableExpression } from '../../common/plugin-api-rpc-model';
+import { DocumentsExtImpl } from '../documents';
+import * as Converter from '../type-converters';
+
+export class EvaluatableExpressionAdapter {
+
+    constructor(
+        private readonly provider: theia.EvaluatableExpressionProvider,
+        private readonly documents: DocumentsExtImpl
+    ) { }
+
+    async provideEvaluatableExpression(resource: URI, position: Position, token: theia.CancellationToken): Promise<EvaluatableExpression | undefined> {
+        const documentData = this.documents.getDocumentData(resource);
+        if (!documentData) {
+            return Promise.reject(new Error(`There is no document data for ${resource}`));
+        }
+
+        const document = documentData.document;
+        const pos = Converter.toPosition(position);
+
+        return Promise.resolve(this.provider.provideEvaluatableExpression(document, pos, token)).then(expression => {
+            if (!expression) {
+                return undefined;
+            }
+            return Converter.fromEvaluatableExpression(expression);
+        });
+    }
+}

package/src/plugin/languages/util.ts

@@ -17,12 +17,10 @@
 import * as theia from '@theia/plugin';
 import * as types from '../types-impl';
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export function isLocationArray(array: any): array is types.Location[] {
+export function isLocationArray(array: unknown): array is types.Location[] {
     return Array.isArray(array) && array.length > 0 && array[0] instanceof types.Location;
 }
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export function isDefinitionLinkArray(array: any): array is theia.DefinitionLink[] {
+export function isDefinitionLinkArray(array: unknown): array is theia.DefinitionLink[] {
     return Array.isArray(array) && array.length > 0 && array[0].hasOwnProperty('targetUri') && array[0].hasOwnProperty('targetRange');
 }

package/src/plugin/languages.ts

@@ -62,11 +62,13 @@ import {
     CallHierarchyIncomingCall,
     CallHierarchyOutgoingCall,
     LinkedEditingRanges,
+    EvaluatableExpression
 } from '../common/plugin-api-rpc-model';
 import { CompletionAdapter } from './languages/completion';
 import { Diagnostics } from './languages/diagnostics';
 import { SignatureHelpAdapter } from './languages/signature';
 import { HoverAdapter } from './languages/hover';
+import { EvaluatableExpressionAdapter } from './languages/evaluatable-expression';
 import { DocumentHighlightAdapter } from './languages/document-highlight';
 import { DocumentFormattingAdapter } from './languages/document-formatting';
 import { RangeFormattingAdapter } from './languages/range-formatting';
@@ -100,6 +102,7 @@ import { serializeEnterRules, serializeIndentation, serializeRegExp } from './la
 type Adapter = CompletionAdapter |
     SignatureHelpAdapter |
     HoverAdapter |
+    EvaluatableExpressionAdapter |
     DocumentHighlightAdapter |
     DocumentFormattingAdapter |
     RangeFormattingAdapter |
@@ -350,6 +353,18 @@ export class LanguagesExtImpl implements LanguagesExt {
     }
     // ### Hover Provider end
 
+    // ### EvaluatableExpression Provider begin
+    registerEvaluatableExpressionProvider(selector: theia.DocumentSelector, provider: theia.EvaluatableExpressionProvider, pluginInfo: PluginInfo): theia.Disposable {
+        const callId = this.addNewAdapter(new EvaluatableExpressionAdapter(provider, this.documents));
+        this.proxy.$registerEvaluatableExpressionProvider(callId, pluginInfo, this.transformDocumentSelector(selector));
+        return this.createDisposable(callId);
+    }
+
+    $provideEvaluatableExpression(handle: number, resource: UriComponents, position: Position, token: theia.CancellationToken): Promise<EvaluatableExpression | undefined> {
+        return this.withAdapter(handle, EvaluatableExpressionAdapter, adapter => adapter.provideEvaluatableExpression(URI.revive(resource), position, token), undefined);
+    }
+    // ### EvaluatableExpression Provider end
+
     // ### Document Highlight Provider begin
     registerDocumentHighlightProvider(selector: theia.DocumentSelector, provider: theia.DocumentHighlightProvider, pluginInfo: PluginInfo): theia.Disposable {
         const callId = this.addNewAdapter(new DocumentHighlightAdapter(provider, this.documents));

package/src/plugin/plugin-context.ts

@@ -78,6 +78,7 @@ import {
     SignatureHelp,
     SignatureHelpTriggerKind,
     Hover,
+    EvaluatableExpression,
     DocumentHighlightKind,
     DocumentHighlight,
     DocumentLink,
@@ -559,7 +560,7 @@ export function createAPIFactory(
                 extHostFileSystemEvent.getOnWillDeleteFileEvent(plugin)(listener, thisArg, disposables),
             onWillRenameFiles: (listener: (e: theia.FileWillRenameEvent) => any, thisArg?: any, disposables?: theia.Disposable[]) =>
                 extHostFileSystemEvent.getOnWillRenameFileEvent(plugin)(listener, thisArg, disposables),
-            getConfiguration(section?, resource?): theia.WorkspaceConfiguration {
+            getConfiguration(section, resource): theia.WorkspaceConfiguration {
                 return preferenceRegistryExt.getConfiguration(section, resource);
             },
             onDidChangeConfiguration(listener, thisArgs?, disposables?): theia.Disposable {
@@ -731,6 +732,9 @@ export function createAPIFactory(
             registerHoverProvider(selector: theia.DocumentSelector, provider: theia.HoverProvider): theia.Disposable {
                 return languagesExt.registerHoverProvider(selector, provider, pluginToPluginInfo(plugin));
             },
+            registerEvaluatableExpressionProvider(selector: theia.DocumentSelector, provider: theia.EvaluatableExpressionProvider): theia.Disposable {
+                return languagesExt.registerEvaluatableExpressionProvider(selector, provider, pluginToPluginInfo(plugin));
+            },
             registerDocumentHighlightProvider(selector: theia.DocumentSelector, provider: theia.DocumentHighlightProvider): theia.Disposable {
                 return languagesExt.registerDocumentHighlightProvider(selector, provider, pluginToPluginInfo(plugin));
             },
@@ -991,6 +995,7 @@ export function createAPIFactory(
             SignatureHelp,
             SignatureHelpTriggerKind,
             Hover,
+            EvaluatableExpression,
             DocumentHighlightKind,
             DocumentHighlight,
             DocumentLink,

package/src/plugin/plugin-storage.ts

@@ -38,6 +38,10 @@ export class Memento implements theia.Memento {
         }
     }
 
+    keys(): string[] {
+        return Object.entries(this.cache).filter(([, value]) => value !== undefined).map(([key]) => key);
+    }
+
     get<T>(key: string): T | undefined;
     get<T>(key: string, defaultValue: T): T;
     get<T>(key: string, defaultValue?: T): T | undefined {

package/src/plugin/preference-registry.spec.ts

@@ -14,16 +14,17 @@
 // SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
 // *****************************************************************************
 
-import { PreferenceRegistryExtImpl } from './preference-registry';
+import { PreferenceRegistryExtImpl, PreferenceScope } from './preference-registry';
 import * as chai from 'chai';
 import { WorkspaceExtImpl } from '../plugin/workspace';
 import { ProxyIdentifier, RPCProtocol } from '../common/rpc-protocol';
+import { URI } from './types-impl';
 
 const expect = chai.expect;
 
 /* eslint-disable @typescript-eslint/no-explicit-any */
 describe('PreferenceRegistryExtImpl:', () => {
-
+    const workspaceRoot = URI.parse('/workspace-root');
     let preferenceRegistryExtImpl: PreferenceRegistryExtImpl;
     const getProxy = (proxyId: ProxyIdentifier<unknown>) => { };
     const set = (identifier: ProxyIdentifier<unknown>, instance: unknown) => { };
@@ -34,59 +35,266 @@ describe('PreferenceRegistryExtImpl:', () => {
         set,
         dispose
     } as RPCProtocol;
-    const mockWorkspace: WorkspaceExtImpl = {} as WorkspaceExtImpl;
+    const mockWorkspace: WorkspaceExtImpl = { workspaceFolders: [{ uri: workspaceRoot, name: 'workspace-root', index: 0 }] } as WorkspaceExtImpl;
 
     beforeEach(() => {
         preferenceRegistryExtImpl = new PreferenceRegistryExtImpl(mockRPC, mockWorkspace);
     });
 
-    it('Check parseConfigurationData', () => {
-        const value: { [key: string]: any } = {
+    it('should parse configuration data without overrides', () => {
+        const value: Record<string, any> = {
             'my.key1.foo': 'value1',
             'my.key1.bar': 'value2',
         };
-        const result: { [key: string]: any } = (preferenceRegistryExtImpl as any).parseConfigurationData(value);
-        expect(result.my).to.be.an('object');
-        expect(result.my.key1).to.be.an('object');
+        const result = preferenceRegistryExtImpl['parseConfigurationData'](value);
+        expect(result.contents.my).to.be.an('object');
+        expect(result.contents.my.key1).to.be.an('object');
 
-        expect(result.my.key1.foo).to.be.an('string');
-        expect(result.my.key1.foo).to.equal('value1');
+        expect(result.contents.my.key1.foo).to.be.an('string');
+        expect(result.contents.my.key1.foo).to.equal('value1');
 
-        expect(result.my.key1.bar).to.be.an('string');
-        expect(result.my.key1.bar).to.equal('value2');
+        expect(result.contents.my.key1.bar).to.be.an('string');
+        expect(result.contents.my.key1.bar).to.equal('value2');
+        expect(result.keys).deep.equal(['my.key1.foo', 'my.key1.bar']);
     });
 
-    it('Prototype pollution check', () => {
-        const value: { [key: string]: any } = {
-            'my.key1.foo': 'value1',
-            '__proto__.injectedParsedPrototype': true,
-            'a.__proto__.injectedParsedPrototype': true,
-            '__proto__': {},
+    it('should parse configuration with overrides', () => {
+        const value: Record<string, any> = {
+            'editor.tabSize': 2,
+            '[typescript].editor.tabSize': 4,
         };
-        const result: { [key: string]: any } = (preferenceRegistryExtImpl as any).parseConfigurationData(value);
-        expect(result.my).to.be.an('object');
-        expect(result.__proto__).to.be.an('undefined');
-        expect(result.my.key1.foo).to.equal('value1');
-        const prototypeObject = Object.prototype as any;
-        expect(prototypeObject.injectedParsedPrototype).to.be.an('undefined');
-        const rawObject = {} as any;
-        expect(rawObject.injectedParsedPrototype).to.be.an('undefined');
+        const result = preferenceRegistryExtImpl['parseConfigurationData'](value);
+        expect(result.contents.editor.tabSize).to.equal(2);
+        const tsOverride = result.overrides[0];
+        expect(tsOverride.contents.editor.tabSize).to.equal(4);
+        expect(tsOverride.identifiers).deep.equal(['typescript']);
+        expect(tsOverride.keys).deep.equal(['editor.tabSize']);
     });
 
-    it('Prototype constructor pollution check', () => {
-        const value: { [key: string]: any } = {
-            'my.key1.foo': 'value1',
-            'a.constructor.prototype.injectedParsedConstructorPrototype': true,
-            'constructor.prototype.injectedParsedConstructorPrototype': true,
+    describe('Prototype pollution', () => {
+        it('Ignores key `__proto__`', () => {
+            const value: Record<string, any> = {
+                'my.key1.foo': 'value1',
+                '__proto__.injectedParsedPrototype': true,
+                'a.__proto__.injectedParsedPrototype': true,
+                '__proto__': {},
+                '[typescript].someKey.foo': 'value',
+                '[typescript].__proto__.injectedParsedPrototype': true,
+            };
+            const result = preferenceRegistryExtImpl['parseConfigurationData'](value);
+            expect(result.contents.my).to.be.an('object');
+            expect(result.contents.__proto__).to.be.an('undefined');
+            expect(result.contents.my.key1.foo).to.equal('value1');
+            expect(result.overrides[0].contents.__proto__).to.be.an('undefined');
+            const prototypeObject = Object.prototype as any;
+            expect(prototypeObject.injectedParsedPrototype).to.be.an('undefined');
+            const rawObject = {} as any;
+            expect(rawObject.injectedParsedPrototype).to.be.an('undefined');
+        });
+
+        it('Ignores key `constructor.prototype`', () => {
+            const value: Record<string, any> = {
+                'my.key1.foo': 'value1',
+                'a.constructor.prototype.injectedParsedConstructorPrototype': true,
+                'constructor.prototype.injectedParsedConstructorPrototype': true,
+                '[python].some.key.foo': 'value',
+                '[python].a.constructor.prototype.injectedParsedConsttructorPrototype': true
+            };
+            const result = preferenceRegistryExtImpl['parseConfigurationData'](value);
+            expect(result.contents.my).to.be.an('object');
+            expect(result.contents.__proto__).to.be.an('undefined');
+            expect(result.contents.my.key1.foo).to.equal('value1');
+            const prototypeObject = Object.prototype as any;
+            expect(prototypeObject.injectedParsedConstructorPrototype).to.be.an('undefined');
+            expect(result.overrides[0].contents.__proto__).to.be.an('undefined');
+            const rawObject = {} as any;
+            expect(rawObject.injectedParsedConstructorPrototype).to.be.an('undefined');
+        });
+    });
+
+    /* eslint-disable no-unused-expressions */
+    describe('toConfigurationChangeEvent', () => {
+        // E.g. deletion of a `tasks.json`.
+        it('Handles deletion of a section', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent']([{ newValue: undefined, preferenceName: 'whole-section' }]);
+            expect(affectsChecker.affectsConfiguration('whole-section')).to.be.true;
+        });
+
+        it('Reports true of supersection if subsection changes', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent']([{ newValue: 'foo', preferenceName: 'whole-section.subsection.item' }]);
+            expect(affectsChecker.affectsConfiguration('whole-section')).to.be.true;
+        });
+
+        // This assumes that there should not exist a preference `section` and `section.*` as separate preferences.
+        // This is true in practice in all cases except `extensions` (i.e. extensions.json) and extensions.ignoreRecommendations etc.
+        // Given that, if a super-section changes (e.g. through deletion), all subsections will also be affected.
+        it('Reports true of a subsection if a supersection changes', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent']([{ newValue: 'bar', preferenceName: 'whole-section' }]);
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection')).to.be.true;
+        });
+
+        it('Does not report true if a different subsection changes:', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent']([{ newValue: 'bar', preferenceName: 'whole-section.subsection.itemA' }]);
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection.itemB')).to.be.false;
+        });
+
+        it('Reports that any URI is affected if change has no URI', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent']([{ newValue: 'bar', preferenceName: 'whole-section' }]);
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', URI.parse('/wherever'))).to.be.true;
+        });
+
+        it('Reports true if no URI is provided to check', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: 'whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection')).to.be.true;
+        });
+
+        it('Reports false if the URIs dont match.', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: 'whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', URI.parse('/other/specific/path'))).to.be.false;
+        });
+
+        it('Reports true if the URIs do match', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: 'whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', URI.parse('/very/specific/path'))).to.be.true;
+        });
+
+        it('Reports true if the checked URI is a child of the affected path', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: 'whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', URI.parse('/very/specific/path/and/its/child'))).to.be.true;
+        });
+
+        it('Reports false if the checked URI starts with the affected path but not at a directory break', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: 'whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', URI.parse('/very/specific/path-or-not/and/its/child'))).to.be.false;
+        });
+
+        it('Extracts language override and returns false if change does not include language', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: 'whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', { languageId: 'typescript' })).to.be.false;
+        });
+
+        it('Extracts language override and returns true if change does include language', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: '[typescript].whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', { languageId: 'typescript' })).to.be.true;
+        });
+
+        it("Extracts language override and URI and returns false if the URI doesn't match", () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: '[typescript].whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', { languageId: 'typescript', uri: URI.parse('/other/specific/path') })).to.be.false;
+        });
+
+        it('Extracts language override and URI and returns true if both match', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: '[typescript].whole-section', scope: 'file:///very/specific/path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subsection', { languageId: 'typescript', uri: URI.parse('/very/specific/path') })).to.be.true;
+        });
+
+        it('Reports true if no language override is provided and a language overriden preference changes', () => {
+            const affectsChecker = preferenceRegistryExtImpl['toConfigurationChangeEvent'](
+                [{ newValue: 'bar', preferenceName: '[typescript].whole-section.subitem', scope: 'file:///somewhat-specific-path' }]
+            );
+            expect(affectsChecker.affectsConfiguration('whole-section.subitem')).to.be.true;
+        });
+    });
+
+    describe('Overrides', () => {
+        const values = {
+            [PreferenceScope.Default]: {
+                'editor.fontSize': 14,
+                'editor.tabSize': 2,
+                'editor.renderWhitespace': 'selection'
+            },
+            [PreferenceScope.User]: {
+                'editor.tabSize': 4,
+            },
+            [PreferenceScope.Workspace]: {
+                'editor.renderWhitespace': 'none',
+                '[python].editor.renderWhitespace': 'all',
+            },
+            [PreferenceScope.Folder]: {
+                [workspaceRoot.toString()]: {
+                    'editor.fontSize': 12,
+                }
+            },
         };
-        const result: { [key: string]: any } = (preferenceRegistryExtImpl as any).parseConfigurationData(value);
-        expect(result.my).to.be.an('object');
-        expect(result.__proto__).to.be.an('undefined');
-        expect(result.my.key1.foo).to.equal('value1');
-        const prototypeObject = Object.prototype as any;
-        expect(prototypeObject.injectedParsedConstructorPrototype).to.be.an('undefined');
-        const rawObject = {} as any;
-        expect(rawObject.injectedParsedConstructorPrototype).to.be.an('undefined');
+        beforeEach(() => preferenceRegistryExtImpl.init(values));
+        it('Returns a scoped value when URI is provided', () => {
+            const valuesRetrieved = preferenceRegistryExtImpl.getConfiguration(undefined, workspaceRoot).get('editor') as Record<string, unknown>;
+            expect(valuesRetrieved.fontSize).equal(12);
+        });
+        it('Returns a lower-priority scope if the value is undefined in the URI-designated scope', () => {
+            const valuesRetrieved = preferenceRegistryExtImpl.getConfiguration(undefined, workspaceRoot).get('editor') as Record<string, unknown>;
+            expect(valuesRetrieved.renderWhitespace).equal('none');
+        });
+        it('Returns a language-overridden value if languageId is provided', () => {
+            const valuesRetrieved = preferenceRegistryExtImpl.getConfiguration(undefined, { uri: workspaceRoot, languageId: 'python' }).get('editor') as Record<string, unknown>;
+            expect(valuesRetrieved.renderWhitespace).equal('all');
+        });
+        it('Returns the default value if the language override is undefined', () => {
+            const valuesRetrieved = preferenceRegistryExtImpl.getConfiguration(undefined, { uri: workspaceRoot, languageId: 'python' }).get('editor') as Record<string, unknown>;
+            expect(valuesRetrieved.tabSize).equal(4);
+        });
     });
 
+    describe('Proxy Behavior', () => {
+        const deepConfig = {
+            'python.linting.enabled': true,
+            'python.linting.flake8Args': [],
+            'python.linting.flake8CategorySeverity.E': 'Error',
+            'python.linting.flake8CategorySeverity.F': 'Error',
+            'python.linting.flake8CategorySeverity.W': 'Warning',
+            'python.linting.flake8Enabled': false,
+            'python.linting.flake8Path': 'flake8',
+            'python.linting.ignorePatterns': ['.vscode/*.py', '**/site-packages/**/*.py'],
+            'python.linting.lintOnSave': true,
+            'python.linting.maxNumberOfProblems': 100,
+            'python.linting.banditArgs': [],
+            'python.linting.banditEnabled': false,
+            'python.linting.banditPath': 'bandit',
+            'python.linting.mypyArgs': [
+                '--ignore-missing-imports',
+                '--follow-imports=silent',
+                '--show-column-numbers'
+            ],
+            'python.linting.mypyCategorySeverity.error': 'Error',
+            'python.linting.mypyCategorySeverity.note': 'Information',
+            'python.linting.mypyEnabled': false,
+            'python.linting.mypyPath': 'mypy',
+        };
+        // https://github.com/eclipse-theia/theia/issues/11501
+        it("Doesn't violate proxy rules and return a proxy when the underlying object is expected.", () => {
+            preferenceRegistryExtImpl.init({
+                [PreferenceScope.Default]: deepConfig,
+                [PreferenceScope.User]: {},
+                [PreferenceScope.Workspace]: {},
+                [PreferenceScope.Folder]: {
+                    [workspaceRoot.toString()]: {},
+                }
+            });
+            const pythonConfig = preferenceRegistryExtImpl.getConfiguration('python', workspaceRoot);
+            const lintConfig = pythonConfig.get<Record<string, unknown>>('linting')!;
+            const stringDictionary = Object.create(null);
+            Object.keys(lintConfig).forEach(key => {
+                stringDictionary[key] = lintConfig[key];
+            });
+            expect(Boolean('Made it this far without throwing an error')).to.be.true;
+        });
+    });
 });