brighterscript 1.0.0-alpha.27 → 1.0.0-alpha.29

package/dist/Scope.js

@@ -4,23 +4,17 @@ exports.Scope = void 0;
 /* eslint-disable @typescript-eslint/dot-notation */
 const path = require("path");
 const chalk_1 = require("chalk");
-const DiagnosticMessages_1 = require("./DiagnosticMessages");
 const interfaces_1 = require("./interfaces");
-const ClassValidator_1 = require("./validators/ClassValidator");
 const Parser_1 = require("./parser/Parser");
 const util_1 = require("./util");
-const globalCallables_1 = require("./globalCallables");
 const Cache_1 = require("./Cache");
-const vscode_uri_1 = require("vscode-uri");
 const Logger_1 = require("./Logger");
 const reflection_1 = require("./astUtils/reflection");
 const SymbolTable_1 = require("./SymbolTable");
-const NamespaceType_1 = require("./types/NamespaceType");
 const ReferenceType_1 = require("./types/ReferenceType");
 const UnionType_1 = require("./types/UnionType");
 const AssociativeArrayType_1 = require("./types/AssociativeArrayType");
-const AstNode_1 = require("./parser/AstNode");
-const visitors_1 = require("./astUtils/visitors");
+const perf_hooks_1 = require("perf_hooks");
 /**
  * Assign some few factories to the SymbolTable to prevent cyclical imports. This file seems like the most intuitive place to do the linking
  * since Scope will be used by pretty much everything
@@ -36,10 +30,15 @@ class Scope {
         this.program = program;
         this._dependencyGraphKey = _dependencyGraphKey;
         this.cache = new Cache_1.Cache();
+        this.useFileCachesForFileLinkLookups = false;
         /**
          * The list of diagnostics found specifically for this scope. Individual file diagnostics are stored on the files themselves.
          */
         this.diagnostics = [];
+        this.validationMetrics = {
+            linkTime: 0,
+            validationTime: 0
+        };
         /**
          * A list of functions that will be called whenever `unlinkSymbolTable` is called
          */
@@ -104,6 +103,17 @@ class Scope {
         const nsList = lookupKeys.filter(key => key === lookupName).map(key => lookup.get(key));
         return nsList;
     }
+    /**
+     * Get a NamespaceContainer by its name, looking for a fully qualified version first, then global version next if not found
+     */
+    getFirstNamespaceWithRoot(rootName, containingNamespace) {
+        const nameLower = rootName === null || rootName === void 0 ? void 0 : rootName.toLowerCase();
+        let lookupName = nameLower;
+        if (containingNamespace) {
+            lookupName = `${containingNamespace === null || containingNamespace === void 0 ? void 0 : containingNamespace.toLowerCase()}.${nameLower}`;
+        }
+        return this.namespaceLookup.get(lookupName);
+    }
     /**
      * Get the class with the specified name.
      * @param className - The class name, including the namespace of the class if possible
@@ -131,18 +141,48 @@ class Scope {
         var _a;
         return (_a = this.getEnumFileLink(enumName, containingNamespace)) === null || _a === void 0 ? void 0 : _a.item;
     }
+    getFileLinkFromFileMap(cachedMapName, itemName, containingNamespace) {
+        var _a;
+        let result;
+        const fullNameLower = (_a = util_1.util.getFullyQualifiedClassName(itemName, containingNamespace)) === null || _a === void 0 ? void 0 : _a.toLowerCase();
+        const itemNameLower = itemName === null || itemName === void 0 ? void 0 : itemName.toLowerCase();
+        if (fullNameLower) {
+            this.enumerateBrsFilesWithBreak((file) => {
+                let stmt = file['_cachedLookups'][cachedMapName].get(fullNameLower);
+                if (stmt) {
+                    result = { item: stmt, file: file };
+                }
+                return !!stmt;
+            });
+        }
+        if (!result && itemNameLower && fullNameLower !== itemNameLower) {
+            this.enumerateBrsFilesWithBreak((file) => {
+                let stmt = file['_cachedLookups'][cachedMapName].get(itemNameLower);
+                if (stmt) {
+                    result = { item: stmt, file: file };
+                }
+                return !!stmt;
+            });
+        }
+        return result;
+    }
     /**
      * Get a class and its containing file by the class name
      * @param className - The class name, including the namespace of the class if possible
      * @param containingNamespace - The namespace used to resolve relative class names. (i.e. the namespace around the current statement trying to find a class)
      */
     getClassFileLink(className, containingNamespace) {
-        const lowerClassName = className === null || className === void 0 ? void 0 : className.toLowerCase();
+        var _a;
+        if (this.useFileCachesForFileLinkLookups) {
+            return this.getFileLinkFromFileMap('classStatementMap', className, containingNamespace);
+        }
+        const lowerName = className === null || className === void 0 ? void 0 : className.toLowerCase();
+        const fullNameLower = (_a = util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace)) === null || _a === void 0 ? void 0 : _a.toLowerCase();
         const classMap = this.getClassMap();
-        let cls = classMap.get(util_1.util.getFullyQualifiedClassName(lowerClassName, containingNamespace === null || containingNamespace === void 0 ? void 0 : containingNamespace.toLowerCase()));
+        let cls = classMap.get(fullNameLower);
         //if we couldn't find the class by its full namespaced name, look for a global class with that name
-        if (!cls) {
-            cls = classMap.get(lowerClassName);
+        if (!cls && lowerName && lowerName !== fullNameLower) {
+            cls = classMap.get(lowerName);
         }
         return cls;
     }
@@ -152,11 +192,16 @@ class Scope {
      * @param containingNamespace - The namespace used to resolve relative interface names. (i.e. the namespace around the current statement trying to find a interface)
      */
     getInterfaceFileLink(ifaceName, containingNamespace) {
+        var _a;
+        if (this.useFileCachesForFileLinkLookups) {
+            return this.getFileLinkFromFileMap('interfaceStatementMap', ifaceName, containingNamespace);
+        }
         const lowerName = ifaceName === null || ifaceName === void 0 ? void 0 : ifaceName.toLowerCase();
+        const fullNameLower = (_a = util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace)) === null || _a === void 0 ? void 0 : _a.toLowerCase();
         const ifaceMap = this.getInterfaceMap();
-        let iface = ifaceMap.get(util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace === null || containingNamespace === void 0 ? void 0 : containingNamespace.toLowerCase()));
+        let iface = ifaceMap.get(fullNameLower);
         //if we couldn't find the iface by its full namespaced name, look for a global class with that name
-        if (!iface) {
+        if (!iface && lowerName && lowerName !== fullNameLower) {
             iface = ifaceMap.get(lowerName);
         }
         return iface;
@@ -167,11 +212,16 @@ class Scope {
      * @param containingNamespace - The namespace used to resolve relative enum names. (i.e. the namespace around the current statement trying to find a enum)
      */
     getEnumFileLink(enumName, containingNamespace) {
+        var _a;
+        if (this.useFileCachesForFileLinkLookups) {
+            return this.getFileLinkFromFileMap('enumStatementMap', enumName, containingNamespace);
+        }
         const lowerName = enumName === null || enumName === void 0 ? void 0 : enumName.toLowerCase();
+        const fullNameLower = (_a = util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace)) === null || _a === void 0 ? void 0 : _a.toLowerCase();
         const enumMap = this.getEnumMap();
-        let enumeration = enumMap.get(util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace === null || containingNamespace === void 0 ? void 0 : containingNamespace.toLowerCase()));
+        let enumeration = enumMap.get(fullNameLower);
         //if we couldn't find the enum by its full namespaced name, look for a global enum with that name
-        if (!enumeration) {
+        if (!enumeration && lowerName && lowerName !== fullNameLower) {
             enumeration = enumMap.get(lowerName);
         }
         return enumeration;
@@ -185,7 +235,7 @@ class Scope {
         var _a, _b;
         let lowerNameParts = (_a = enumMemberName === null || enumMemberName === void 0 ? void 0 : enumMemberName.toLowerCase()) === null || _a === void 0 ? void 0 : _a.split('.');
         let memberName = (_b = lowerNameParts === null || lowerNameParts === void 0 ? void 0 : lowerNameParts.splice(lowerNameParts.length - 1, 1)) === null || _b === void 0 ? void 0 : _b[0];
-        let lowerName = lowerNameParts === null || lowerNameParts === void 0 ? void 0 : lowerNameParts.join('.');
+        let lowerName = lowerNameParts === null || lowerNameParts === void 0 ? void 0 : lowerNameParts.join('.').toLowerCase();
         const enumMap = this.getEnumMap();
         let enumeration = enumMap.get(util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace === null || containingNamespace === void 0 ? void 0 : containingNamespace.toLowerCase()));
         //if we couldn't find the enum by its full namespaced name, look for a global enum with that name
@@ -203,30 +253,37 @@ class Scope {
      * @param containingNamespace - The namespace used to resolve relative constant names. (i.e. the namespace around the current statement trying to find a constant)
      */
     getConstFileLink(constName, containingNamespace) {
+        var _a;
+        if (this.useFileCachesForFileLinkLookups) {
+            return this.getFileLinkFromFileMap('constStatementMap', constName, containingNamespace);
+        }
         const lowerName = constName === null || constName === void 0 ? void 0 : constName.toLowerCase();
+        const fullNameLower = (_a = util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace)) === null || _a === void 0 ? void 0 : _a.toLowerCase();
         const constMap = this.getConstMap();
-        let result = constMap.get(util_1.util.getFullyQualifiedClassName(lowerName, containingNamespace === null || containingNamespace === void 0 ? void 0 : containingNamespace.toLowerCase()));
+        let result = constMap.get(fullNameLower);
         //if we couldn't find the constant by its full namespaced name, look for a global constant with that name
-        if (!result) {
+        if (!result && lowerName !== fullNameLower) {
             result = constMap.get(lowerName);
         }
         return result;
     }
-    getAllFileLinks(name, containingNamespace) {
-        var _a, _b, _c, _d, _e, _f;
+    getAllFileLinks(name, containingNamespace, includeNamespaces = false, includeNameShadowsOutsideNamespace = false) {
+        var _a;
         let links = [];
-        links.push((_a = this.getClassFileLink(name)) !== null && _a !== void 0 ? _a : this.getClassFileLink(name, containingNamespace), (_b = this.getInterfaceFileLink(name)) !== null && _b !== void 0 ? _b : this.getInterfaceFileLink(name, containingNamespace), (_c = this.getConstFileLink(name)) !== null && _c !== void 0 ? _c : this.getConstFileLink(name, containingNamespace), (_d = this.getEnumFileLink(name)) !== null && _d !== void 0 ? _d : this.getEnumFileLink(name, containingNamespace));
-        const nameSpaces = this.getNamespacesWithRoot(name, containingNamespace);
-        if ((nameSpaces === null || nameSpaces === void 0 ? void 0 : nameSpaces.length) > 0) {
-            const nsContainersWithStatement = (_e = nameSpaces.filter(nsContainer => { var _a; return ((_a = nsContainer === null || nsContainer === void 0 ? void 0 : nsContainer.namespaceStatements) === null || _a === void 0 ? void 0 : _a.length) > 0; })) === null || _e === void 0 ? void 0 : _e[0];
-            if (nsContainersWithStatement) {
-                links.push({ item: (_f = nsContainersWithStatement === null || nsContainersWithStatement === void 0 ? void 0 : nsContainersWithStatement.namespaceStatements) === null || _f === void 0 ? void 0 : _f[0], file: nsContainersWithStatement === null || nsContainersWithStatement === void 0 ? void 0 : nsContainersWithStatement.file });
+        links.push(this.getClassFileLink(name, containingNamespace), this.getInterfaceFileLink(name, containingNamespace), this.getConstFileLink(name, containingNamespace), this.getEnumFileLink(name, containingNamespace));
+        if (includeNameShadowsOutsideNamespace && containingNamespace) {
+            links.push(this.getClassFileLink(name), this.getInterfaceFileLink(name), this.getConstFileLink(name), this.getEnumFileLink(name));
+        }
+        if (includeNamespaces) {
+            const nameSpaceContainer = this.getFirstNamespaceWithRoot(name, containingNamespace);
+            if (nameSpaceContainer) {
+                links.push({ item: (_a = nameSpaceContainer.namespaceStatements) === null || _a === void 0 ? void 0 : _a[0], file: nameSpaceContainer === null || nameSpaceContainer === void 0 ? void 0 : nameSpaceContainer.file });
             }
         }
         const fullNameLower = (containingNamespace ? `${containingNamespace}.${name}` : name).toLowerCase();
         const callable = this.getCallableByName(name);
         if (callable) {
-            if (!callable.hasNamespace || callable.getName(Parser_1.ParseMode.BrighterScript).toLowerCase() === fullNameLower) {
+            if ((!callable.hasNamespace && includeNameShadowsOutsideNamespace) || callable.getName(Parser_1.ParseMode.BrighterScript).toLowerCase() === fullNameLower) {
                 // this callable has no namespace, or has same namespace
                 links.push({ item: callable.functionStatement, file: callable.file });
             }
@@ -446,7 +503,7 @@ class Scope {
             for (let dependency of dependencies) {
                 //load components by their name
                 if (dependency.startsWith('component:')) {
-                    let comp = this.program.getComponent(dependency.replace(/$component:/, ''));
+                    let comp = this.program.getComponent(dependency.replace(/^component:/, ''));
                     if (comp) {
                         result.push(comp.file);
                     }
@@ -462,6 +519,36 @@ class Scope {
             return result;
         });
     }
+    /**
+     * Gets a list of all files in this scope, but not imported files, and not from ancestor scopes
+     */
+    getImmediateFiles() {
+        return this.cache.getOrAdd('getImmediateFiles', () => {
+            let result = [];
+            if ((0, reflection_1.isXmlScope)(this)) {
+                result.push(this.xmlFile);
+            }
+            let dependencies = this.dependencyGraph.getImmediateDependencies(this.dependencyGraphKey);
+            for (let dependency of dependencies) {
+                //load components by their name
+                if (dependency.startsWith('component:')) {
+                    let comp = this.program.getComponent(dependency.replace(/^component:/, ''));
+                    if (comp) {
+                        result.push(...comp.scope.getImmediateFiles());
+                        result.push(comp.file);
+                    }
+                }
+                else {
+                    let file = this.program.getFile(dependency);
+                    if (file) {
+                        result.push(file);
+                    }
+                }
+            }
+            this.logDebug('getImmediateFiles', () => result.map(x => x.destPath));
+            return result;
+        });
+    }
     /**
      * Get the list of errors for this scope. It's calculated on the fly, so call this sparingly.
      */
@@ -516,6 +603,13 @@ class Scope {
             return result;
         });
     }
+    getCallableContainerMap() {
+        return this.cache.getOrAdd('callableContainerMap', () => {
+            let callables = this.getAllCallables();
+            //get a list of all callables, indexed by their lower case names
+            return util_1.util.getCallableContainersByLowerName(callables);
+        });
+    }
     /**
      * Iterate over Brs files not shadowed by typedefs
      */
@@ -528,6 +622,20 @@ class Scope {
             }
         }
     }
+    /**
+     * Iterate over Brs files not shadowed by typedefs
+     */
+    enumerateBrsFilesWithBreak(callback) {
+        const files = this.getAllFiles();
+        for (const file of files) {
+            //only brs files without a typedef
+            if ((0, reflection_1.isBrsFile)(file) && !file.hasTypedef) {
+                if (callback(file)) {
+                    break;
+                }
+            }
+        }
+    }
     /**
      * Call a function for each file directly included in this scope (excluding files found only in parent scopes).
      */
@@ -600,11 +708,28 @@ class Scope {
         this.program.logger.debug(this._debugLogComponentName, ...args);
     }
     validate(validationOptions = { force: false }) {
+        var _a, _b;
         //if this scope is already validated, no need to revalidate
         if (this.isValidated === true && !validationOptions.force) {
             this.logDebug('validate(): already validated');
-            return;
+            return false;
+        }
+        const hasChangedSymbols = ((_a = validationOptions.changedSymbols) === null || _a === void 0 ? void 0 : _a.get(1 /* SymbolTypeFlag.runtime */).size) > 0 || ((_b = validationOptions.changedSymbols) === null || _b === void 0 ? void 0 : _b.get(2 /* SymbolTypeFlag.typetime */).size) > 0;
+        let immediateFileChanged = false;
+        if (!validationOptions.initialValidation && validationOptions.changedSymbols && !hasChangedSymbols && validationOptions.changedFiles) {
+            for (let file of this.getImmediateFiles()) {
+                if (validationOptions.changedFiles.includes(file)) {
+                    immediateFileChanged = true;
+                    break;
+                }
+            }
+            if (!immediateFileChanged) {
+                // There was no need to validate this scope.
+                this.isValidated = true;
+                return false;
+            }
         }
+        this.useFileCachesForFileLinkLookups = !validationOptions.initialValidation;
         this.program.logger.time(Logger_1.LogLevel.debug, [this._debugLogComponentName, 'validate()'], () => {
             let parentScope = this.getParentScope();
             //validate our parent before we validate ourself
@@ -614,63 +739,26 @@ class Scope {
             }
             //clear the scope's errors list (we will populate them from this method)
             this.clearScopeLevelDiagnostics();
-            let callables = this.getAllCallables();
-            //sort the callables by filepath and then method name, so the errors will be consistent
-            // eslint-disable-next-line prefer-arrow-callback
-            callables = callables.sort((a, b) => {
-                const pathA = a.callable.file.srcPath;
-                const pathB = b.callable.file.srcPath;
-                //sort by path
-                if (pathA < pathB) {
-                    return -1;
-                }
-                else if (pathA > pathB) {
-                    return 1;
-                }
-                //sort by function name
-                const funcA = b.callable.name;
-                const funcB = b.callable.name;
-                if (funcA < funcB) {
-                    return -1;
-                }
-                else if (funcA > funcB) {
-                    return 1;
-                }
-                return 0;
-            });
-            //get a list of all callables, indexed by their lower case names
-            let callableContainerMap = util_1.util.getCallableContainersByLowerName(callables);
             //Since statements from files are shared across multiple scopes, we need to link those statements to the current scope
+            let t0 = perf_hooks_1.performance.now();
             this.linkSymbolTable();
+            this.validationMetrics.linkTime = perf_hooks_1.performance.now() - t0;
             const scopeValidateEvent = {
                 program: this.program,
                 scope: this,
                 changedFiles: validationOptions === null || validationOptions === void 0 ? void 0 : validationOptions.changedFiles,
                 changedSymbols: validationOptions === null || validationOptions === void 0 ? void 0 : validationOptions.changedSymbols
             };
+            t0 = perf_hooks_1.performance.now();
             this.program.plugins.emit('beforeScopeValidate', scopeValidateEvent);
             this.program.plugins.emit('onScopeValidate', scopeValidateEvent);
-            this._validate(callableContainerMap);
+            this.validationMetrics.validationTime = perf_hooks_1.performance.now() - t0;
             this.program.plugins.emit('afterScopeValidate', scopeValidateEvent);
             //unlink all symbol tables from this scope (so they don't accidentally stick around)
             this.unlinkSymbolTable();
             this.isValidated = true;
         });
-    }
-    _validate(callableContainerMap) {
-        //find all duplicate function declarations
-        this.diagnosticFindDuplicateFunctionDeclarations(callableContainerMap);
-        //detect missing and incorrect-case script imports
-        this.diagnosticValidateScriptImportPaths();
-        //enforce a series of checks on the bodies of class methods
-        this.validateClasses();
-        //do many per-file checks
-        this.enumerateBrsFiles((file) => {
-            this.diagnosticDetectShadowedLocalVars(file, callableContainerMap);
-            this.diagnosticDetectFunctionCollisions(file);
-            this.detectVariableNamespaceCollisions(file);
-            this.detectNameCollisions(file);
-        });
+        return true;
     }
     clearAstSegmentDiagnostics(astSegment) {
         this.diagnostics = this.diagnostics.filter(diag => !(diag.origin === interfaces_1.DiagnosticOrigin.ASTSegment && diag.astSegment === astSegment));
@@ -703,6 +791,9 @@ class Scope {
             return result;
         });
     }
+    get allNamespaceTypeTable() {
+        return this._allNamespaceTypeTable;
+    }
     /**
      * Builds the current symbol table for the scope, by merging the tables for all the files in this scope.
      * Also links all file symbols tables to this new table
@@ -723,64 +814,29 @@ class Scope {
      * ```
      */
     linkSymbolTable() {
-        var _a;
         SymbolTable_1.SymbolTable.cacheVerifier.generateToken();
+        this._allNamespaceTypeTable = new SymbolTable_1.SymbolTable(`Scope NamespaceTypes ${this.name}`);
         for (const file of this.getAllFiles()) {
             if ((0, reflection_1.isBrsFile)(file)) {
                 this.linkSymbolTableDisposables.push(file.parser.symbolTable.pushParentProvider(() => this.symbolTable));
             }
         }
-        //Add namespace aggregates to namespace member tables
-        const namespaceTypesKnown = new Map();
-        // eslint-disable-next-line no-bitwise
-        let getTypeOptions = { flags: 1 /* SymbolTypeFlag.runtime */ | 2 /* SymbolTypeFlag.typetime */ };
-        for (const [nsName, nsContainer] of this.namespaceLookup) {
-            let currentNSType = null;
-            let parentNSType = null;
-            const existingNsStmt = (_a = nsContainer.namespaceStatements) === null || _a === void 0 ? void 0 : _a[0];
-            if (!nsContainer.isTopLevel) {
-                parentNSType = namespaceTypesKnown.get(nsContainer.parentNameLower);
-                if (!parentNSType) {
-                    // we don't know about the parent namespace... uh, oh!
-                    this.program.logger.error(`Unable to find parent namespace type for namespace ${nsName}`);
-                    break;
-                }
-                currentNSType = parentNSType.getMemberType(nsContainer.fullNameLower, getTypeOptions);
-            }
-            else {
-                currentNSType = this.symbolTable.getSymbolType(nsContainer.fullNameLower, getTypeOptions);
-            }
-            if (!(0, reflection_1.isNamespaceType)(currentNSType)) {
-                if (!currentNSType || (0, reflection_1.isReferenceType)(currentNSType) || (0, reflection_1.isCallableType)(currentNSType)) {
-                    currentNSType = existingNsStmt
-                        ? existingNsStmt.getType(getTypeOptions)
-                        : new NamespaceType_1.NamespaceType(nsName);
-                    if (parentNSType) {
-                        // adding as a member of existing NS
-                        parentNSType.addMember(nsContainer.lastPartName, { definingNode: existingNsStmt }, currentNSType, getTypeOptions.flags);
-                        this.symbolsAddedDuringLinking.push({ symbolTable: parentNSType.getMemberTable(), name: nsContainer.lastPartName, flags: getTypeOptions.flags });
-                    }
-                    else {
-                        this.symbolTable.addSymbol(nsContainer.lastPartName, { definingNode: existingNsStmt }, currentNSType, getTypeOptions.flags);
-                        this.symbolsAddedDuringLinking.push({ symbolTable: this.symbolTable, name: nsContainer.lastPartName, flags: getTypeOptions.flags });
-                    }
-                }
-                else {
-                    this.program.logger.error(`Invalid existing type for namespace ${nsName} - ${currentNSType.toString()}`);
-                    continue;
-                }
-            }
-            else {
-                // Existing known namespace
-            }
-            if (!namespaceTypesKnown.has(nsName)) {
-                namespaceTypesKnown.set(nsName, currentNSType);
+        const directFiles = this.getImmediateFiles();
+        for (const file of directFiles) {
+            if ((0, reflection_1.isBrsFile)(file)) {
+                const namespaceTypes = file.getNamespaceSymbolTable();
+                this.linkSymbolTableDisposables.push(...this._allNamespaceTypeTable.mergeNamespaceSymbolTables(namespaceTypes));
+                /* this.linkSymbolTableDisposables.push(
+                     () => file.unlinkNamespaceSymbolTables()
+                 );*/
             }
+        }
+        for (const [_, nsContainer] of this.namespaceLookup) {
             for (let nsStmt of nsContainer.namespaceStatements) {
                 this.linkSymbolTableDisposables.push(nsStmt === null || nsStmt === void 0 ? void 0 : nsStmt.getSymbolTable().addSibling(nsContainer.symbolTable));
             }
-            this.linkSymbolTableDisposables.push(currentNSType.memberTable.addSibling(nsContainer.symbolTable));
         }
+        this.linkSymbolTableDisposables.push(this.symbolTable.addSibling(this._allNamespaceTypeTable));
     }
     unlinkSymbolTable() {
         for (const symbolToRemove of this.symbolsAddedDuringLinking) {
@@ -792,220 +848,6 @@ class Scope {
         }
         this.linkSymbolTableDisposables = [];
     }
-    detectVariableNamespaceCollisions(file) {
-        var _a, _b;
-        //find all function parameters
-        for (let func of file['_cachedLookups'].functionExpressions) {
-            for (let param of func.parameters) {
-                let lowerParamName = param.tokens.name.text.toLowerCase();
-                let namespace = this.getNamespace(lowerParamName, (_a = param.findAncestor(reflection_1.isNamespaceStatement)) === null || _a === void 0 ? void 0 : _a.getName(Parser_1.ParseMode.BrighterScript).toLowerCase());
-                //see if the param matches any starting namespace part
-                if (namespace) {
-                    this.diagnostics.push(Object.assign(Object.assign({ origin: interfaces_1.DiagnosticOrigin.Scope, file: file }, DiagnosticMessages_1.DiagnosticMessages.parameterMayNotHaveSameNameAsNamespace(param.tokens.name.text)), { range: param.tokens.name.range, relatedInformation: [{
-                                message: 'Namespace declared here',
-                                location: util_1.util.createLocation(vscode_uri_1.URI.file(namespace.file.srcPath).toString(), namespace.nameRange)
-                            }] }));
-                }
-            }
-        }
-        for (let assignment of file['_cachedLookups'].assignmentStatements) {
-            let lowerAssignmentName = assignment.tokens.name.text.toLowerCase();
-            let namespace = this.getNamespace(lowerAssignmentName, (_b = assignment.findAncestor(reflection_1.isNamespaceStatement)) === null || _b === void 0 ? void 0 : _b.getName(Parser_1.ParseMode.BrighterScript).toLowerCase());
-            //see if the param matches any starting namespace part
-            if (namespace) {
-                this.diagnostics.push(Object.assign(Object.assign({ origin: interfaces_1.DiagnosticOrigin.Scope, file: file }, DiagnosticMessages_1.DiagnosticMessages.variableMayNotHaveSameNameAsNamespace(assignment.tokens.name.text)), { range: assignment.tokens.name.range, relatedInformation: [{
-                            message: 'Namespace declared here',
-                            location: util_1.util.createLocation(vscode_uri_1.URI.file(namespace.file.srcPath).toString(), namespace.nameRange)
-                        }] }));
-            }
-        }
-    }
-    /**
-     * Find various function collisions
-     */
-    diagnosticDetectFunctionCollisions(file) {
-        for (let func of file.callables) {
-            const funcName = func.getName(Parser_1.ParseMode.BrighterScript);
-            const lowerFuncName = funcName === null || funcName === void 0 ? void 0 : funcName.toLowerCase();
-            if (lowerFuncName) {
-                //find function declarations with the same name as a stdlib function
-                if (globalCallables_1.globalCallableMap.has(lowerFuncName)) {
-                    this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.scopeFunctionShadowedByBuiltInFunction()), { range: func.nameRange, file: file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                }
-                //find any functions that have the same name as a class
-                const klassLink = this.getClassFileLink(lowerFuncName);
-                if (klassLink) {
-                    this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.functionCannotHaveSameNameAsClass(funcName)), { range: func.nameRange, file: file, origin: interfaces_1.DiagnosticOrigin.Scope, relatedInformation: [{
-                                location: util_1.util.createLocation(vscode_uri_1.URI.file(klassLink.file.srcPath).toString(), klassLink.item.tokens.name.range),
-                                message: 'Original class declared here'
-                            }] }));
-                }
-            }
-        }
-    }
-    detectNameCollisions(file) {
-        file.ast.walk((0, visitors_1.createVisitor)({
-            NamespaceStatement: (nsStmt) => {
-                var _a;
-                this.validateNameCollision(file, nsStmt, (_a = nsStmt.getNameParts()) === null || _a === void 0 ? void 0 : _a[0]);
-            },
-            ClassStatement: (classStmt) => {
-                this.validateNameCollision(file, classStmt, classStmt.tokens.name);
-            },
-            InterfaceStatement: (ifaceStmt) => {
-                this.validateNameCollision(file, ifaceStmt, ifaceStmt.tokens.name);
-            },
-            ConstStatement: (constStmt) => {
-                this.validateNameCollision(file, constStmt, constStmt.tokens.name);
-            },
-            EnumStatement: (enumStmt) => {
-                this.validateNameCollision(file, enumStmt, enumStmt.tokens.name);
-            }
-        }), {
-            walkMode: visitors_1.WalkMode.visitStatements
-        });
-    }
-    validateNameCollision(file, node, nameIdentifier) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
-        const name = nameIdentifier === null || nameIdentifier === void 0 ? void 0 : nameIdentifier.text;
-        if (!name || !node) {
-            return;
-        }
-        const nameRange = nameIdentifier.range;
-        const containingNamespace = (_a = node.findAncestor(reflection_1.isNamespaceStatement)) === null || _a === void 0 ? void 0 : _a.getName(Parser_1.ParseMode.BrighterScript);
-        const links = this.getAllFileLinks(name, containingNamespace);
-        for (let link of links) {
-            if (!link || link.item === node) {
-                // refers to same node
-                continue;
-            }
-            if ((0, reflection_1.isNamespaceStatement)(link.item) && (0, reflection_1.isNamespaceStatement)(node)) {
-                // namespace can be declared multiple times
-                continue;
-            }
-            if ((0, reflection_1.isFunctionStatement)(link.item) || ((_b = link.file) === null || _b === void 0 ? void 0 : _b.destPath) === 'global') {
-                // the thing found is a function OR from global (which is also a function)
-                if ((0, reflection_1.isNamespaceStatement)(node)) {
-                    // these are not callable functions in transpiled code - ignore them
-                    continue;
-                }
-            }
-            const thisNodeKindName = util_1.util.getAstNodeFriendlyName(node);
-            const thatNodeKindName = link.file.srcPath === 'global' ? 'Global Function' : (_c = util_1.util.getAstNodeFriendlyName(link.item)) !== null && _c !== void 0 ? _c : '';
-            let thatNameRange = (_g = (_f = (_e = (_d = link.item) === null || _d === void 0 ? void 0 : _d.tokens) === null || _e === void 0 ? void 0 : _e.name) === null || _f === void 0 ? void 0 : _f.range) !== null && _g !== void 0 ? _g : (_h = link.item) === null || _h === void 0 ? void 0 : _h.range;
-            // eslint-disable-next-line @typescript-eslint/switch-exhaustiveness-check
-            switch ((_j = link.item) === null || _j === void 0 ? void 0 : _j.kind) {
-                case AstNode_1.AstNodeKind.ClassStatement: {
-                    thatNameRange = (_k = link.item.tokens.name) === null || _k === void 0 ? void 0 : _k.range;
-                    break;
-                }
-                case AstNode_1.AstNodeKind.NamespaceStatement: {
-                    thatNameRange = (_m = (_l = link.item.getNameParts()) === null || _l === void 0 ? void 0 : _l[0]) === null || _m === void 0 ? void 0 : _m.range;
-                    break;
-                }
-            }
-            const relatedInformation = thatNameRange ? [{
-                    message: `${thatNodeKindName} declared here`,
-                    location: util_1.util.createLocation(vscode_uri_1.URI.file((_o = link.file) === null || _o === void 0 ? void 0 : _o.srcPath).toString(), thatNameRange)
-                }] : undefined;
-            this.diagnostics.push(Object.assign(Object.assign({ file: file }, DiagnosticMessages_1.DiagnosticMessages.nameCollision(thisNodeKindName, thatNodeKindName, name)), { origin: interfaces_1.DiagnosticOrigin.Scope, range: nameRange, relatedInformation: relatedInformation }));
-        }
-    }
-    validateClasses() {
-        let validator = new ClassValidator_1.BsClassValidator(this);
-        validator.validate();
-        this.diagnostics.push(...validator.diagnostics.map(diag => {
-            return Object.assign(Object.assign({}, diag), { origin: interfaces_1.DiagnosticOrigin.Scope });
-        }));
-    }
-    /**
-     * Detect local variables (function scope) that have the same name as scope calls
-     */
-    diagnosticDetectShadowedLocalVars(file, callableContainerMap) {
-        var _a;
-        const classMap = this.getClassMap();
-        //loop through every function scope
-        for (let funcScope of file.functionScopes) {
-            //every var declaration in this function scope
-            for (let varDeclaration of funcScope.variableDeclarations) {
-                const varName = varDeclaration.name;
-                const lowerVarName = varName.toLowerCase();
-                const varIsFunction = () => {
-                    return (0, reflection_1.isCallableType)(varDeclaration.getType());
-                };
-                if (
-                //has same name as stdlib
-                globalCallables_1.globalCallableMap.has(lowerVarName)) {
-                    //local var function with same name as stdlib function
-                    if (varIsFunction()) {
-                        this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.localVarFunctionShadowsParentFunction('stdlib')), { range: varDeclaration.nameRange, file: file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                    }
-                }
-                else if (callableContainerMap.has(lowerVarName)) {
-                    //is same name as a callable
-                    if (varIsFunction()) {
-                        this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.localVarFunctionShadowsParentFunction('scope')), { range: varDeclaration.nameRange, file: file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                    }
-                    else {
-                        this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.localVarShadowedByScopedFunction()), { range: varDeclaration.nameRange, file: file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                    }
-                    //has the same name as an in-scope class
-                }
-                else if (classMap.has(lowerVarName)) {
-                    this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.localVarSameNameAsClass((_a = classMap.get(lowerVarName)) === null || _a === void 0 ? void 0 : _a.item.getName(Parser_1.ParseMode.BrighterScript))), { range: varDeclaration.nameRange, file: file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                }
-            }
-        }
-    }
-    /**
-     * Create diagnostics for any duplicate function declarations
-     */
-    diagnosticFindDuplicateFunctionDeclarations(callableContainersByLowerName) {
-        //for each list of callables with the same name
-        for (let [lowerName, callableContainers] of callableContainersByLowerName) {
-            let globalCallables = [];
-            let nonGlobalCallables = [];
-            let ownCallables = [];
-            let ancestorNonGlobalCallables = [];
-            for (let container of callableContainers) {
-                if (container.scope === this.program.globalScope) {
-                    globalCallables.push(container);
-                }
-                else {
-                    nonGlobalCallables.push(container);
-                    if (container.scope === this) {
-                        ownCallables.push(container);
-                    }
-                    else {
-                        ancestorNonGlobalCallables.push(container);
-                    }
-                }
-            }
-            //add info diagnostics about child shadowing parent functions
-            if (ownCallables.length > 0 && ancestorNonGlobalCallables.length > 0) {
-                for (let container of ownCallables) {
-                    //skip the init function (because every component will have one of those){
-                    if (lowerName !== 'init') {
-                        let shadowedCallable = ancestorNonGlobalCallables[ancestorNonGlobalCallables.length - 1];
-                        if (!!shadowedCallable && shadowedCallable.callable.file === container.callable.file) {
-                            //same file: skip redundant imports
-                            continue;
-                        }
-                        this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.overridesAncestorFunction(container.callable.name, container.scope.name, shadowedCallable.callable.file.destPath, 
-                        //grab the last item in the list, which should be the closest ancestor's version
-                        shadowedCallable.scope.name)), { range: container.callable.nameRange, file: container.callable.file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                    }
-                }
-            }
-            //add error diagnostics about duplicate functions in the same scope
-            if (ownCallables.length > 1) {
-                for (let callableContainer of ownCallables) {
-                    let callable = callableContainer.callable;
-                    this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.duplicateFunctionImplementation(callable.name, callableContainer.scope.name)), { range: util_1.util.createRange(callable.nameRange.start.line, callable.nameRange.start.character, callable.nameRange.start.line, callable.nameRange.end.character), file: callable.file, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                }
-            }
-        }
-    }
     /**
      * Get the list of all script imports for this scope
      */
@@ -1021,35 +863,6 @@ class Scope {
         });
         return result;
     }
-    /**
-     * Verify that all of the scripts imported by each file in this scope actually exist
-     */
-    diagnosticValidateScriptImportPaths() {
-        let scriptImports = this.getOwnScriptImports();
-        //verify every script import
-        for (let scriptImport of scriptImports) {
-            let referencedFile = this.getFileByRelativePath(scriptImport.destPath);
-            //if we can't find the file
-            if (!referencedFile) {
-                //skip the default bslib file, it will exist at transpile time but should not show up in the program during validation cycle
-                if (scriptImport.destPath === this.program.bslibPkgPath) {
-                    continue;
-                }
-                let dInfo;
-                if (scriptImport.text.trim().length === 0) {
-                    dInfo = DiagnosticMessages_1.DiagnosticMessages.scriptSrcCannotBeEmpty();
-                }
-                else {
-                    dInfo = DiagnosticMessages_1.DiagnosticMessages.referencedFileDoesNotExist();
-                }
-                this.diagnostics.push(Object.assign(Object.assign({}, dInfo), { range: scriptImport.filePathRange, file: scriptImport.sourceFile, origin: interfaces_1.DiagnosticOrigin.Scope }));
-                //if the character casing of the script import path does not match that of the actual path
-            }
-            else if (scriptImport.destPath !== referencedFile.destPath) {
-                this.diagnostics.push(Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.scriptImportCaseMismatch(referencedFile.destPath)), { range: scriptImport.filePathRange, file: scriptImport.sourceFile, origin: interfaces_1.DiagnosticOrigin.Scope }));
-            }
-        }
-    }
     /**
      * Find the file with the specified relative path
      */