@abaplint/cli 2.82.13 → 2.82.14

package/build/cli.js

@@ -7606,7 +7606,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.GetBadi = void 0;\r\nconst Expressions = __webpack_require__(/*! ../../2_statements/expressions */ \"./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js\");\r\nconst source_1 = __webpack_require__(/*! ../expressions/source */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js\");\r\nconst target_1 = __webpack_require__(/*! ../expressions/target */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js\");\r\nconst dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js\");\r\nclass GetBadi {\r\n    runSyntax(node, scope, filename) {\r\n        for (const s of node.findDirectExpressions(Expressions.Source)) {\r\n            new source_1.Source().runSyntax(s, scope, filename);\r\n        }\r\n        for (const t of node.findDirectExpressions(Expressions.Target)) {\r\n            new target_1.Target().runSyntax(t, scope, filename);\r\n        }\r\n        for (const d of node.findDirectExpressions(Expressions.Dynamic)) {\r\n            new dynamic_1.Dynamic().runSyntax(d, scope, filename);\r\n        }\r\n    }\r\n}\r\nexports.GetBadi = GetBadi;\r\n//# sourceMappingURL=get_badi.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_badi.js?");
+eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.GetBadi = void 0;\r\nconst Expressions = __webpack_require__(/*! ../../2_statements/expressions */ \"./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js\");\r\nconst source_1 = __webpack_require__(/*! ../expressions/source */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js\");\r\nconst target_1 = __webpack_require__(/*! ../expressions/target */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js\");\r\nconst dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dynamic.js\");\r\nclass GetBadi {\r\n    runSyntax(node, scope, filename) {\r\n        for (const s of node.findAllExpressions(Expressions.Source)) {\r\n            new source_1.Source().runSyntax(s, scope, filename);\r\n        }\r\n        for (const t of node.findDirectExpressions(Expressions.Target)) {\r\n            new target_1.Target().runSyntax(t, scope, filename);\r\n        }\r\n        for (const d of node.findDirectExpressions(Expressions.Dynamic)) {\r\n            new dynamic_1.Dynamic().runSyntax(d, scope, filename);\r\n        }\r\n    }\r\n}\r\nexports.GetBadi = GetBadi;\r\n//# sourceMappingURL=get_badi.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/abap/5_syntax/statements/get_badi.js?");
 
 /***/ }),
 
@@ -9718,7 +9718,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.ConceptsOfPackage = void 0;\r\nconst _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ \"./node_modules/@abaplint/core/build/src/objects/_abstract_object.js\");\r\nclass ConceptsOfPackage extends _abstract_object_1.AbstractObject {\r\n    getType() {\r\n        return \"SOTS\";\r\n    }\r\n    getAllowedNaming() {\r\n        return {\r\n            maxLength: 30,\r\n            allowNamespace: true,\r\n        };\r\n    }\r\n    getDescription() {\r\n        // todo\r\n        return undefined;\r\n    }\r\n}\r\nexports.ConceptsOfPackage = ConceptsOfPackage;\r\n//# sourceMappingURL=concepts_of_package.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/objects/concepts_of_package.js?");
+eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.ConceptsOfPackage = void 0;\r\nconst _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ \"./node_modules/@abaplint/core/build/src/objects/_abstract_object.js\");\r\nclass ConceptsOfPackage extends _abstract_object_1.AbstractObject {\r\n    getType() {\r\n        return \"SOTS\";\r\n    }\r\n    getAllowedNaming() {\r\n        return {\r\n            maxLength: 30,\r\n            allowNamespace: true,\r\n            customRegex: new RegExp(/^(\\/[A-Z_\\d]{3,8}\\/)?[A-Z_-\\d<> ]+$/i),\r\n        };\r\n    }\r\n    getDescription() {\r\n        // todo\r\n        return undefined;\r\n    }\r\n}\r\nexports.ConceptsOfPackage = ConceptsOfPackage;\r\n//# sourceMappingURL=concepts_of_package.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/objects/concepts_of_package.js?");
 
 /***/ }),
 
@@ -10224,7 +10224,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.Namespace = void 0;\r\nconst _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ \"./node_modules/@abaplint/core/build/src/objects/_abstract_object.js\");\r\nclass Namespace extends _abstract_object_1.AbstractObject {\r\n    getType() {\r\n        return \"NSPC\";\r\n    }\r\n    getAllowedNaming() {\r\n        return {\r\n            maxLength: 10,\r\n            allowNamespace: true,\r\n        };\r\n    }\r\n    getDescription() {\r\n        // todo\r\n        return undefined;\r\n    }\r\n}\r\nexports.Namespace = Namespace;\r\n//# sourceMappingURL=namespace.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/objects/namespace.js?");
+eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.Namespace = void 0;\r\nconst _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ \"./node_modules/@abaplint/core/build/src/objects/_abstract_object.js\");\r\nclass Namespace extends _abstract_object_1.AbstractObject {\r\n    getType() {\r\n        return \"NSPC\";\r\n    }\r\n    getAllowedNaming() {\r\n        return {\r\n            maxLength: 10,\r\n            allowNamespace: true,\r\n            customRegex: new RegExp(/^\\/[A-Z_\\d]{3,8}\\/$/i),\r\n        };\r\n    }\r\n    getDescription() {\r\n        // todo\r\n        return undefined;\r\n    }\r\n}\r\nexports.Namespace = Namespace;\r\n//# sourceMappingURL=namespace.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/objects/namespace.js?");
 
 /***/ }),
 
@@ -10708,7 +10708,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.WebMIME = void 0;\r\nconst _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ \"./node_modules/@abaplint/core/build/src/objects/_abstract_object.js\");\r\nclass WebMIME extends _abstract_object_1.AbstractObject {\r\n    getType() {\r\n        return \"W3MI\";\r\n    }\r\n    getAllowedNaming() {\r\n        return {\r\n            maxLength: 40,\r\n            allowNamespace: true,\r\n        };\r\n    }\r\n    getDescription() {\r\n        // todo\r\n        return undefined;\r\n    }\r\n}\r\nexports.WebMIME = WebMIME;\r\n//# sourceMappingURL=web_mime.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/objects/web_mime.js?");
+eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.WebMIME = void 0;\r\nconst _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ \"./node_modules/@abaplint/core/build/src/objects/_abstract_object.js\");\r\nclass WebMIME extends _abstract_object_1.AbstractObject {\r\n    getType() {\r\n        return \"W3MI\";\r\n    }\r\n    getAllowedNaming() {\r\n        return {\r\n            maxLength: 40,\r\n            allowNamespace: true,\r\n            customRegex: new RegExp(/^[A-Z_-\\d/<> ]+$/i),\r\n        };\r\n    }\r\n    getDescription() {\r\n        // todo\r\n        return undefined;\r\n    }\r\n}\r\nexports.WebMIME = WebMIME;\r\n//# sourceMappingURL=web_mime.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/objects/web_mime.js?");
 
 /***/ }),
 
@@ -10774,7 +10774,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\nexports.Registry = void 0;\nconst config_1 = __webpack_require__(/*! ./config */ \"./node_modules/@abaplint/core/build/src/config.js\");\nconst artifacts_objects_1 = __webpack_require__(/*! ./artifacts_objects */ \"./node_modules/@abaplint/core/build/src/artifacts_objects.js\");\nconst artifacts_rules_1 = __webpack_require__(/*! ./artifacts_rules */ \"./node_modules/@abaplint/core/build/src/artifacts_rules.js\");\nconst skip_logic_1 = __webpack_require__(/*! ./skip_logic */ \"./node_modules/@abaplint/core/build/src/skip_logic.js\");\nconst _abap_object_1 = __webpack_require__(/*! ./objects/_abap_object */ \"./node_modules/@abaplint/core/build/src/objects/_abap_object.js\");\nconst find_global_definitions_1 = __webpack_require__(/*! ./abap/5_syntax/global_definitions/find_global_definitions */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/global_definitions/find_global_definitions.js\");\nconst syntax_1 = __webpack_require__(/*! ./abap/5_syntax/syntax */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js\");\nconst excludeHelper_1 = __webpack_require__(/*! ./utils/excludeHelper */ \"./node_modules/@abaplint/core/build/src/utils/excludeHelper.js\");\nconst ddic_references_1 = __webpack_require__(/*! ./ddic_references */ \"./node_modules/@abaplint/core/build/src/ddic_references.js\");\n// todo, this should really be an instance in case there are multiple Registry'ies\nclass ParsingPerformance {\n    static clear() {\n        this.results = [];\n        this.lexing = 0;\n        this.statements = 0;\n        this.structure = 0;\n    }\n    static push(obj, result) {\n        if (result.runtimeExtra) {\n            this.lexing += result.runtimeExtra.lexing;\n            this.statements += result.runtimeExtra.statements;\n            this.structure += result.runtimeExtra.structure;\n        }\n        if (result.runtime < 100) {\n            return;\n        }\n        if (this.results === undefined) {\n            this.results = [];\n        }\n        let extra = \"\";\n        if (result.runtimeExtra) {\n            extra = `\\t(lexing: ${result.runtimeExtra.lexing}ms, statements: ${result.runtimeExtra.statements}ms, structure: ${result.runtimeExtra.structure}ms)`;\n        }\n        this.results.push({\n            runtime: result.runtime,\n            extra,\n            name: obj.getType() + \" \" + obj.getName(),\n        });\n    }\n    static output() {\n        const MAX = 10;\n        this.results.sort((a, b) => { return b.runtime - a.runtime; });\n        for (let i = 0; i < MAX; i++) {\n            const row = this.results[i];\n            if (row === undefined) {\n                break;\n            }\n            process.stderr.write(`\\t${row.runtime}ms\\t${row.name} ${row.extra}\\n`);\n        }\n        process.stderr.write(`\\tTotal lexing:     ${this.lexing}ms\\n`);\n        process.stderr.write(`\\tTotal statements: ${this.statements}ms\\n`);\n        process.stderr.write(`\\tTotal structure:  ${this.structure}ms\\n`);\n    }\n}\n///////////////////////////////////////////////////////////////////////////////////////////////\nclass Registry {\n    constructor(conf) {\n        this.objects = {};\n        this.objectsByType = {};\n        /** object containing filenames of dependencies */\n        this.dependencies = {};\n        this.issues = [];\n        this.conf = conf ? conf : config_1.Config.getDefault();\n        this.references = new ddic_references_1.DDICReferences();\n    }\n    static abaplintVersion() {\n        // magic, see build script \"version.sh\"\n        return \"2.82.13\";\n    }\n    getDDICReferences() {\n        return this.references;\n    }\n    *getObjects() {\n        for (const name in this.objects) {\n            for (const type in this.objects[name]) {\n                yield this.objects[name][type];\n            }\n        }\n    }\n    *getObjectsByType(type) {\n        for (const name in this.objectsByType[type] || []) {\n            yield this.objectsByType[type][name];\n        }\n    }\n    *getFiles() {\n        for (const obj of this.getObjects()) {\n            for (const file of obj.getFiles()) {\n                yield file;\n            }\n        }\n    }\n    getFirstObject() {\n        for (const name in this.objects) {\n            for (const type in this.objects[name]) {\n                return this.objects[name][type];\n            }\n        }\n        return undefined;\n    }\n    getObjectCount(skipDependencies = true) {\n        let res = 0;\n        for (const o of this.getObjects()) {\n            if (skipDependencies === true && this.isDependency(o)) {\n                continue;\n            }\n            res = res + 1;\n        }\n        return res;\n    }\n    getFileByName(filename) {\n        const upper = filename.toUpperCase();\n        for (const o of this.getObjects()) {\n            for (const f of o.getFiles()) {\n                if (f.getFilename().toUpperCase() === upper) {\n                    return f;\n                }\n            }\n        }\n        return undefined;\n    }\n    getObject(type, name) {\n        if (type === undefined || name === undefined) {\n            return undefined;\n        }\n        const searchName = name.toUpperCase();\n        if (this.objects[searchName]) {\n            return this.objects[searchName][type];\n        }\n        return undefined;\n    }\n    getConfig() {\n        return this.conf;\n    }\n    // assumption: Config is immutable, and can only be changed via this method\n    setConfig(conf) {\n        for (const obj of this.getObjects()) {\n            obj.setDirty();\n        }\n        this.conf = conf;\n        return this;\n    }\n    inErrorNamespace(name) {\n        const reg = new RegExp(this.getConfig().getSyntaxSetttings().errorNamespace, \"i\");\n        return reg.test(name);\n    }\n    addFile(file) {\n        return this.addFiles([file]);\n    }\n    updateFile(file) {\n        const obj = this.find(file.getObjectName(), file.getObjectType());\n        obj.updateFile(file);\n        return this;\n    }\n    removeFile(file) {\n        const obj = this.find(file.getObjectName(), file.getObjectType());\n        obj.removeFile(file);\n        if (obj.getFiles().length === 0) {\n            this.references.clear(obj);\n            this.removeObject(obj);\n        }\n        return this;\n    }\n    addFiles(files) {\n        var _a;\n        const globalExclude = ((_a = this.conf.getGlobal().exclude) !== null && _a !== void 0 ? _a : [])\n            .map(pattern => new RegExp(pattern, \"i\"));\n        for (const f of files) {\n            const filename = f.getFilename();\n            const isNotAbapgitFile = filename.split(\".\").length <= 2;\n            if (isNotAbapgitFile || excludeHelper_1.ExcludeHelper.isExcluded(filename, globalExclude)) {\n                continue;\n            }\n            const found = this.findOrCreate(f.getObjectName(), f.getObjectType());\n            found.addFile(f);\n        }\n        return this;\n    }\n    addDependencies(files) {\n        for (const f of files) {\n            this.dependencies[f.getFilename().toUpperCase()] = true;\n        }\n        return this.addFiles(files);\n    }\n    addDependency(file) {\n        this.dependencies[file.getFilename().toUpperCase()] = true;\n        this.addFile(file);\n        return this;\n    }\n    isDependency(obj) {\n        const filename = obj.getFiles()[0].getFilename().toUpperCase();\n        return this.dependencies[filename] === true;\n    }\n    isFileDependency(filename) {\n        return this.dependencies[filename.toUpperCase()] === true;\n    }\n    // assumption: the file is already in the registry\n    findObjectForFile(file) {\n        const filename = file.getFilename();\n        for (const obj of this.getObjects()) {\n            for (const ofile of obj.getFiles()) {\n                if (ofile.getFilename() === filename) {\n                    return obj;\n                }\n            }\n        }\n        return undefined;\n    }\n    // todo, this will be changed to async sometime\n    findIssues(input) {\n        if (this.isDirty() === true) {\n            this.parse();\n        }\n        return this.runRules(input);\n    }\n    // todo, this will be changed to async sometime\n    findIssuesObject(iobj) {\n        if (this.isDirty() === true) {\n            this.parse();\n        }\n        return this.runRules(undefined, iobj);\n    }\n    // todo, this will be changed to async sometime\n    parse() {\n        if (this.isDirty() === false) {\n            return this;\n        }\n        ParsingPerformance.clear();\n        this.issues = [];\n        for (const o of this.getObjects()) {\n            this.parsePrivate(o);\n            this.issues.push(...o.getParsingIssues());\n        }\n        new find_global_definitions_1.FindGlobalDefinitions(this).run();\n        return this;\n    }\n    async parseAsync(input) {\n        var _a, _b;\n        if (this.isDirty() === false) {\n            return this;\n        }\n        ParsingPerformance.clear();\n        (_a = input === null || input === void 0 ? void 0 : input.progress) === null || _a === void 0 ? void 0 : _a.set(this.getObjectCount(false), \"Lexing and parsing\");\n        this.issues = [];\n        for (const o of this.getObjects()) {\n            await ((_b = input === null || input === void 0 ? void 0 : input.progress) === null || _b === void 0 ? void 0 : _b.tick(\"Lexing and parsing(\" + this.conf.getVersion() + \") - \" + o.getType() + \" \" + o.getName()));\n            this.parsePrivate(o);\n            this.issues.push(...o.getParsingIssues());\n        }\n        if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {\n            ParsingPerformance.output();\n        }\n        new find_global_definitions_1.FindGlobalDefinitions(this).run(input === null || input === void 0 ? void 0 : input.progress);\n        return this;\n    }\n    //////////////////////////////////////////\n    // todo, refactor, this is a mess, see where-used, a lot of the code should be in this method instead\n    parsePrivate(input) {\n        const config = this.getConfig();\n        const result = input.parse(config.getVersion(), config.getSyntaxSetttings().globalMacros);\n        ParsingPerformance.push(input, result);\n    }\n    isDirty() {\n        for (const o of this.getObjects()) {\n            const dirty = o.isDirty();\n            if (dirty === true) {\n                return true;\n            }\n        }\n        return false;\n    }\n    runRules(input, iobj) {\n        var _a, _b, _c, _d, _e, _f;\n        const rulePerformance = {};\n        const issues = this.issues.slice(0);\n        const objects = iobj ? [iobj] : this.getObjects();\n        const rules = this.conf.getEnabledRules();\n        const skipLogic = new skip_logic_1.SkipLogic(this);\n        (_a = input === null || input === void 0 ? void 0 : input.progress) === null || _a === void 0 ? void 0 : _a.set(iobj ? 1 : this.getObjectCount(false), \"Run Syntax\");\n        const check = [];\n        for (const obj of objects) {\n            (_b = input === null || input === void 0 ? void 0 : input.progress) === null || _b === void 0 ? void 0 : _b.tick(\"Run Syntax - \" + obj.getName());\n            if (skipLogic.skip(obj) || this.isDependency(obj)) {\n                continue;\n            }\n            if (obj instanceof _abap_object_1.ABAPObject) {\n                new syntax_1.SyntaxLogic(this, obj).run();\n            }\n            check.push(obj);\n        }\n        (_c = input === null || input === void 0 ? void 0 : input.progress) === null || _c === void 0 ? void 0 : _c.set(rules.length, \"Initialize Rules\");\n        for (const rule of rules) {\n            (_d = input === null || input === void 0 ? void 0 : input.progress) === null || _d === void 0 ? void 0 : _d.tick(\"Initialize Rules - \" + rule.getMetadata().key);\n            if (rule.initialize === undefined) {\n                throw new Error(rule.getMetadata().key + \" missing initialize method\");\n            }\n            rule.initialize(this);\n            rulePerformance[rule.getMetadata().key] = 0;\n        }\n        (_e = input === null || input === void 0 ? void 0 : input.progress) === null || _e === void 0 ? void 0 : _e.set(check.length, \"Finding Issues\");\n        for (const obj of check) {\n            (_f = input === null || input === void 0 ? void 0 : input.progress) === null || _f === void 0 ? void 0 : _f.tick(\"Finding Issues - \" + obj.getType() + \" \" + obj.getName());\n            for (const rule of rules) {\n                const before = Date.now();\n                issues.push(...rule.run(obj));\n                const runtime = Date.now() - before;\n                rulePerformance[rule.getMetadata().key] = rulePerformance[rule.getMetadata().key] + runtime;\n            }\n        }\n        if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {\n            const perf = [];\n            for (const p in rulePerformance) {\n                if (rulePerformance[p] > 100) { // ignore rules if it takes less than 100ms\n                    perf.push({ name: p, time: rulePerformance[p] });\n                }\n            }\n            perf.sort((a, b) => { return b.time - a.time; });\n            for (const p of perf) {\n                process.stderr.write(\"\\t\" + p.time + \"ms\\t\" + p.name + \"\\n\");\n            }\n        }\n        return this.excludeIssues(issues);\n    }\n    excludeIssues(issues) {\n        var _a;\n        const ret = issues;\n        // exclude issues, as now we know both the filename and issue key\n        for (const rule of artifacts_rules_1.ArtifactsRules.getRules()) {\n            const key = rule.getMetadata().key;\n            const ruleExclude = ((_a = this.conf.readByKey(key, \"exclude\")) !== null && _a !== void 0 ? _a : []);\n            const ruleExcludePatterns = ruleExclude.map(x => new RegExp(x, \"i\"));\n            for (let i = ret.length - 1; i >= 0; i--) {\n                if (ret[i].getKey() !== key) {\n                    continue;\n                }\n                const filename = ret[i].getFilename();\n                if (excludeHelper_1.ExcludeHelper.isExcluded(filename, ruleExcludePatterns)) {\n                    ret.splice(i, 1);\n                }\n            }\n        }\n        return ret;\n    }\n    findOrCreate(name, type) {\n        try {\n            return this.find(name, type);\n        }\n        catch (_a) {\n            const newName = name.toUpperCase();\n            const newType = type ? type : \"UNKNOWN\";\n            const add = artifacts_objects_1.ArtifactsObjects.newObject(newName, newType);\n            if (this.objects[newName] === undefined) {\n                this.objects[newName] = {};\n            }\n            this.objects[newName][newType] = add;\n            if (this.objectsByType[newType] === undefined) {\n                this.objectsByType[newType] = {};\n            }\n            this.objectsByType[newType][newName] = add;\n            return add;\n        }\n    }\n    removeObject(remove) {\n        if (remove === undefined) {\n            return;\n        }\n        if (this.objects[remove.getName()][remove.getType()] === undefined) {\n            throw new Error(\"removeObject: object not found\");\n        }\n        if (Object.keys(this.objects[remove.getName()]).length === 1) {\n            delete this.objects[remove.getName()];\n        }\n        else {\n            delete this.objects[remove.getName()][remove.getType()];\n        }\n        if (Object.keys(this.objectsByType[remove.getType()]).length === 1) {\n            delete this.objectsByType[remove.getType()];\n        }\n        else {\n            delete this.objectsByType[remove.getType()][remove.getName()];\n        }\n    }\n    find(name, type) {\n        const searchType = type ? type : \"UNKNOWN\";\n        const searchName = name.toUpperCase();\n        if (this.objects[searchName] !== undefined\n            && this.objects[searchName][searchType]) {\n            return this.objects[searchName][searchType];\n        }\n        throw new Error(\"find: object not found, \" + type + \" \" + name);\n    }\n}\nexports.Registry = Registry;\n//# sourceMappingURL=registry.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/registry.js?");
+eval("\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\nexports.Registry = void 0;\nconst config_1 = __webpack_require__(/*! ./config */ \"./node_modules/@abaplint/core/build/src/config.js\");\nconst artifacts_objects_1 = __webpack_require__(/*! ./artifacts_objects */ \"./node_modules/@abaplint/core/build/src/artifacts_objects.js\");\nconst artifacts_rules_1 = __webpack_require__(/*! ./artifacts_rules */ \"./node_modules/@abaplint/core/build/src/artifacts_rules.js\");\nconst skip_logic_1 = __webpack_require__(/*! ./skip_logic */ \"./node_modules/@abaplint/core/build/src/skip_logic.js\");\nconst _abap_object_1 = __webpack_require__(/*! ./objects/_abap_object */ \"./node_modules/@abaplint/core/build/src/objects/_abap_object.js\");\nconst find_global_definitions_1 = __webpack_require__(/*! ./abap/5_syntax/global_definitions/find_global_definitions */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/global_definitions/find_global_definitions.js\");\nconst syntax_1 = __webpack_require__(/*! ./abap/5_syntax/syntax */ \"./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js\");\nconst excludeHelper_1 = __webpack_require__(/*! ./utils/excludeHelper */ \"./node_modules/@abaplint/core/build/src/utils/excludeHelper.js\");\nconst ddic_references_1 = __webpack_require__(/*! ./ddic_references */ \"./node_modules/@abaplint/core/build/src/ddic_references.js\");\n// todo, this should really be an instance in case there are multiple Registry'ies\nclass ParsingPerformance {\n    static clear() {\n        this.results = [];\n        this.lexing = 0;\n        this.statements = 0;\n        this.structure = 0;\n    }\n    static push(obj, result) {\n        if (result.runtimeExtra) {\n            this.lexing += result.runtimeExtra.lexing;\n            this.statements += result.runtimeExtra.statements;\n            this.structure += result.runtimeExtra.structure;\n        }\n        if (result.runtime < 100) {\n            return;\n        }\n        if (this.results === undefined) {\n            this.results = [];\n        }\n        let extra = \"\";\n        if (result.runtimeExtra) {\n            extra = `\\t(lexing: ${result.runtimeExtra.lexing}ms, statements: ${result.runtimeExtra.statements}ms, structure: ${result.runtimeExtra.structure}ms)`;\n        }\n        this.results.push({\n            runtime: result.runtime,\n            extra,\n            name: obj.getType() + \" \" + obj.getName(),\n        });\n    }\n    static output() {\n        const MAX = 10;\n        this.results.sort((a, b) => { return b.runtime - a.runtime; });\n        for (let i = 0; i < MAX; i++) {\n            const row = this.results[i];\n            if (row === undefined) {\n                break;\n            }\n            process.stderr.write(`\\t${row.runtime}ms\\t${row.name} ${row.extra}\\n`);\n        }\n        process.stderr.write(`\\tTotal lexing:     ${this.lexing}ms\\n`);\n        process.stderr.write(`\\tTotal statements: ${this.statements}ms\\n`);\n        process.stderr.write(`\\tTotal structure:  ${this.structure}ms\\n`);\n    }\n}\n///////////////////////////////////////////////////////////////////////////////////////////////\nclass Registry {\n    constructor(conf) {\n        this.objects = {};\n        this.objectsByType = {};\n        /** object containing filenames of dependencies */\n        this.dependencies = {};\n        this.issues = [];\n        this.conf = conf ? conf : config_1.Config.getDefault();\n        this.references = new ddic_references_1.DDICReferences();\n    }\n    static abaplintVersion() {\n        // magic, see build script \"version.sh\"\n        return \"2.82.14\";\n    }\n    getDDICReferences() {\n        return this.references;\n    }\n    *getObjects() {\n        for (const name in this.objects) {\n            for (const type in this.objects[name]) {\n                yield this.objects[name][type];\n            }\n        }\n    }\n    *getObjectsByType(type) {\n        for (const name in this.objectsByType[type] || []) {\n            yield this.objectsByType[type][name];\n        }\n    }\n    *getFiles() {\n        for (const obj of this.getObjects()) {\n            for (const file of obj.getFiles()) {\n                yield file;\n            }\n        }\n    }\n    getFirstObject() {\n        for (const name in this.objects) {\n            for (const type in this.objects[name]) {\n                return this.objects[name][type];\n            }\n        }\n        return undefined;\n    }\n    getObjectCount(skipDependencies = true) {\n        let res = 0;\n        for (const o of this.getObjects()) {\n            if (skipDependencies === true && this.isDependency(o)) {\n                continue;\n            }\n            res = res + 1;\n        }\n        return res;\n    }\n    getFileByName(filename) {\n        const upper = filename.toUpperCase();\n        for (const o of this.getObjects()) {\n            for (const f of o.getFiles()) {\n                if (f.getFilename().toUpperCase() === upper) {\n                    return f;\n                }\n            }\n        }\n        return undefined;\n    }\n    getObject(type, name) {\n        if (type === undefined || name === undefined) {\n            return undefined;\n        }\n        const searchName = name.toUpperCase();\n        if (this.objects[searchName]) {\n            return this.objects[searchName][type];\n        }\n        return undefined;\n    }\n    getConfig() {\n        return this.conf;\n    }\n    // assumption: Config is immutable, and can only be changed via this method\n    setConfig(conf) {\n        for (const obj of this.getObjects()) {\n            obj.setDirty();\n        }\n        this.conf = conf;\n        return this;\n    }\n    inErrorNamespace(name) {\n        const reg = new RegExp(this.getConfig().getSyntaxSetttings().errorNamespace, \"i\");\n        return reg.test(name);\n    }\n    addFile(file) {\n        return this.addFiles([file]);\n    }\n    updateFile(file) {\n        const obj = this.find(file.getObjectName(), file.getObjectType());\n        obj.updateFile(file);\n        return this;\n    }\n    removeFile(file) {\n        const obj = this.find(file.getObjectName(), file.getObjectType());\n        obj.removeFile(file);\n        if (obj.getFiles().length === 0) {\n            this.references.clear(obj);\n            this.removeObject(obj);\n        }\n        return this;\n    }\n    addFiles(files) {\n        var _a;\n        const globalExclude = ((_a = this.conf.getGlobal().exclude) !== null && _a !== void 0 ? _a : [])\n            .map(pattern => new RegExp(pattern, \"i\"));\n        for (const f of files) {\n            const filename = f.getFilename();\n            const isNotAbapgitFile = filename.split(\".\").length <= 2;\n            if (isNotAbapgitFile || excludeHelper_1.ExcludeHelper.isExcluded(filename, globalExclude)) {\n                continue;\n            }\n            const found = this.findOrCreate(f.getObjectName(), f.getObjectType());\n            found.addFile(f);\n        }\n        return this;\n    }\n    addDependencies(files) {\n        for (const f of files) {\n            this.dependencies[f.getFilename().toUpperCase()] = true;\n        }\n        return this.addFiles(files);\n    }\n    addDependency(file) {\n        this.dependencies[file.getFilename().toUpperCase()] = true;\n        this.addFile(file);\n        return this;\n    }\n    isDependency(obj) {\n        const filename = obj.getFiles()[0].getFilename().toUpperCase();\n        return this.dependencies[filename] === true;\n    }\n    isFileDependency(filename) {\n        return this.dependencies[filename.toUpperCase()] === true;\n    }\n    // assumption: the file is already in the registry\n    findObjectForFile(file) {\n        const filename = file.getFilename();\n        for (const obj of this.getObjects()) {\n            for (const ofile of obj.getFiles()) {\n                if (ofile.getFilename() === filename) {\n                    return obj;\n                }\n            }\n        }\n        return undefined;\n    }\n    // todo, this will be changed to async sometime\n    findIssues(input) {\n        if (this.isDirty() === true) {\n            this.parse();\n        }\n        return this.runRules(input);\n    }\n    // todo, this will be changed to async sometime\n    findIssuesObject(iobj) {\n        if (this.isDirty() === true) {\n            this.parse();\n        }\n        return this.runRules(undefined, iobj);\n    }\n    // todo, this will be changed to async sometime\n    parse() {\n        if (this.isDirty() === false) {\n            return this;\n        }\n        ParsingPerformance.clear();\n        this.issues = [];\n        for (const o of this.getObjects()) {\n            this.parsePrivate(o);\n            this.issues.push(...o.getParsingIssues());\n        }\n        new find_global_definitions_1.FindGlobalDefinitions(this).run();\n        return this;\n    }\n    async parseAsync(input) {\n        var _a, _b;\n        if (this.isDirty() === false) {\n            return this;\n        }\n        ParsingPerformance.clear();\n        (_a = input === null || input === void 0 ? void 0 : input.progress) === null || _a === void 0 ? void 0 : _a.set(this.getObjectCount(false), \"Lexing and parsing\");\n        this.issues = [];\n        for (const o of this.getObjects()) {\n            await ((_b = input === null || input === void 0 ? void 0 : input.progress) === null || _b === void 0 ? void 0 : _b.tick(\"Lexing and parsing(\" + this.conf.getVersion() + \") - \" + o.getType() + \" \" + o.getName()));\n            this.parsePrivate(o);\n            this.issues.push(...o.getParsingIssues());\n        }\n        if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {\n            ParsingPerformance.output();\n        }\n        new find_global_definitions_1.FindGlobalDefinitions(this).run(input === null || input === void 0 ? void 0 : input.progress);\n        return this;\n    }\n    //////////////////////////////////////////\n    // todo, refactor, this is a mess, see where-used, a lot of the code should be in this method instead\n    parsePrivate(input) {\n        const config = this.getConfig();\n        const result = input.parse(config.getVersion(), config.getSyntaxSetttings().globalMacros);\n        ParsingPerformance.push(input, result);\n    }\n    isDirty() {\n        for (const o of this.getObjects()) {\n            const dirty = o.isDirty();\n            if (dirty === true) {\n                return true;\n            }\n        }\n        return false;\n    }\n    runRules(input, iobj) {\n        var _a, _b, _c, _d, _e, _f;\n        const rulePerformance = {};\n        const issues = this.issues.slice(0);\n        const objects = iobj ? [iobj] : this.getObjects();\n        const rules = this.conf.getEnabledRules();\n        const skipLogic = new skip_logic_1.SkipLogic(this);\n        (_a = input === null || input === void 0 ? void 0 : input.progress) === null || _a === void 0 ? void 0 : _a.set(iobj ? 1 : this.getObjectCount(false), \"Run Syntax\");\n        const check = [];\n        for (const obj of objects) {\n            (_b = input === null || input === void 0 ? void 0 : input.progress) === null || _b === void 0 ? void 0 : _b.tick(\"Run Syntax - \" + obj.getName());\n            if (skipLogic.skip(obj) || this.isDependency(obj)) {\n                continue;\n            }\n            if (obj instanceof _abap_object_1.ABAPObject) {\n                new syntax_1.SyntaxLogic(this, obj).run();\n            }\n            check.push(obj);\n        }\n        (_c = input === null || input === void 0 ? void 0 : input.progress) === null || _c === void 0 ? void 0 : _c.set(rules.length, \"Initialize Rules\");\n        for (const rule of rules) {\n            (_d = input === null || input === void 0 ? void 0 : input.progress) === null || _d === void 0 ? void 0 : _d.tick(\"Initialize Rules - \" + rule.getMetadata().key);\n            if (rule.initialize === undefined) {\n                throw new Error(rule.getMetadata().key + \" missing initialize method\");\n            }\n            rule.initialize(this);\n            rulePerformance[rule.getMetadata().key] = 0;\n        }\n        (_e = input === null || input === void 0 ? void 0 : input.progress) === null || _e === void 0 ? void 0 : _e.set(check.length, \"Finding Issues\");\n        for (const obj of check) {\n            (_f = input === null || input === void 0 ? void 0 : input.progress) === null || _f === void 0 ? void 0 : _f.tick(\"Finding Issues - \" + obj.getType() + \" \" + obj.getName());\n            for (const rule of rules) {\n                const before = Date.now();\n                issues.push(...rule.run(obj));\n                const runtime = Date.now() - before;\n                rulePerformance[rule.getMetadata().key] = rulePerformance[rule.getMetadata().key] + runtime;\n            }\n        }\n        if ((input === null || input === void 0 ? void 0 : input.outputPerformance) === true) {\n            const perf = [];\n            for (const p in rulePerformance) {\n                if (rulePerformance[p] > 100) { // ignore rules if it takes less than 100ms\n                    perf.push({ name: p, time: rulePerformance[p] });\n                }\n            }\n            perf.sort((a, b) => { return b.time - a.time; });\n            for (const p of perf) {\n                process.stderr.write(\"\\t\" + p.time + \"ms\\t\" + p.name + \"\\n\");\n            }\n        }\n        return this.excludeIssues(issues);\n    }\n    excludeIssues(issues) {\n        var _a;\n        const ret = issues;\n        // exclude issues, as now we know both the filename and issue key\n        for (const rule of artifacts_rules_1.ArtifactsRules.getRules()) {\n            const key = rule.getMetadata().key;\n            const ruleExclude = ((_a = this.conf.readByKey(key, \"exclude\")) !== null && _a !== void 0 ? _a : []);\n            const ruleExcludePatterns = ruleExclude.map(x => new RegExp(x, \"i\"));\n            for (let i = ret.length - 1; i >= 0; i--) {\n                if (ret[i].getKey() !== key) {\n                    continue;\n                }\n                const filename = ret[i].getFilename();\n                if (excludeHelper_1.ExcludeHelper.isExcluded(filename, ruleExcludePatterns)) {\n                    ret.splice(i, 1);\n                }\n            }\n        }\n        return ret;\n    }\n    findOrCreate(name, type) {\n        try {\n            return this.find(name, type);\n        }\n        catch (_a) {\n            const newName = name.toUpperCase();\n            const newType = type ? type : \"UNKNOWN\";\n            const add = artifacts_objects_1.ArtifactsObjects.newObject(newName, newType);\n            if (this.objects[newName] === undefined) {\n                this.objects[newName] = {};\n            }\n            this.objects[newName][newType] = add;\n            if (this.objectsByType[newType] === undefined) {\n                this.objectsByType[newType] = {};\n            }\n            this.objectsByType[newType][newName] = add;\n            return add;\n        }\n    }\n    removeObject(remove) {\n        if (remove === undefined) {\n            return;\n        }\n        if (this.objects[remove.getName()][remove.getType()] === undefined) {\n            throw new Error(\"removeObject: object not found\");\n        }\n        if (Object.keys(this.objects[remove.getName()]).length === 1) {\n            delete this.objects[remove.getName()];\n        }\n        else {\n            delete this.objects[remove.getName()][remove.getType()];\n        }\n        if (Object.keys(this.objectsByType[remove.getType()]).length === 1) {\n            delete this.objectsByType[remove.getType()];\n        }\n        else {\n            delete this.objectsByType[remove.getType()][remove.getName()];\n        }\n    }\n    find(name, type) {\n        const searchType = type ? type : \"UNKNOWN\";\n        const searchName = name.toUpperCase();\n        if (this.objects[searchName] !== undefined\n            && this.objects[searchName][searchType]) {\n            return this.objects[searchName][searchType];\n        }\n        throw new Error(\"find: object not found, \" + type + \" \" + name);\n    }\n}\nexports.Registry = Registry;\n//# sourceMappingURL=registry.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/registry.js?");
 
 /***/ }),
 
@@ -10862,7 +10862,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.AllowedObjectNaming = exports.AllowedObjectNamingConf = void 0;\r\nconst issue_1 = __webpack_require__(/*! ../issue */ \"./node_modules/@abaplint/core/build/src/issue.js\");\r\nconst _irule_1 = __webpack_require__(/*! ./_irule */ \"./node_modules/@abaplint/core/build/src/rules/_irule.js\");\r\nconst _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ \"./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js\");\r\nclass AllowedObjectNamingConf extends _basic_rule_config_1.BasicRuleConfig {\r\n}\r\nexports.AllowedObjectNamingConf = AllowedObjectNamingConf;\r\nclass AllowedObjectNaming {\r\n    constructor() {\r\n        this.conf = new AllowedObjectNamingConf();\r\n    }\r\n    getMetadata() {\r\n        return {\r\n            key: \"allowed_object_naming\",\r\n            title: \"Allowed object naming\",\r\n            shortDescription: `Enforces basic name length and namespace restrictions, see note SAP 104010`,\r\n            tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],\r\n        };\r\n    }\r\n    initialize(_reg) {\r\n        return this;\r\n    }\r\n    getConfig() {\r\n        return this.conf;\r\n    }\r\n    setConfig(conf) {\r\n        this.conf = conf;\r\n    }\r\n    run(obj) {\r\n        const allowed = obj.getAllowedNaming();\r\n        let message = \"\";\r\n        if (obj.getName().length > allowed.maxLength) {\r\n            message = \"Name exceeds max length\";\r\n        }\r\n        else if (allowed.allowNamespace === false && obj.getName().indexOf(\"/\") >= 0) {\r\n            message = \"Namespace not allowed for object type\";\r\n        }\r\n        else if (obj.getType() === \"NSPC\") {\r\n            if (obj.getName().match(/^\\/[A-Z_\\d]{3,8}\\/$/i) === null) {\r\n                message = \"Name not allowed\";\r\n            }\r\n        }\r\n        else if (obj.getName().match(/^(\\/[A-Z_\\d]{3,8}\\/)?[A-Z_-\\d<> ]+$/i) === null) {\r\n            message = \"Name not allowed\";\r\n        }\r\n        if (message.length > 0) {\r\n            return [issue_1.Issue.atRow(obj.getFiles()[0], 1, message, this.getMetadata().key, this.conf.severity)];\r\n        }\r\n        return [];\r\n    }\r\n}\r\nexports.AllowedObjectNaming = AllowedObjectNaming;\r\n//# sourceMappingURL=allowed_object_naming.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/rules/allowed_object_naming.js?");
+eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.AllowedObjectNaming = exports.AllowedObjectNamingConf = void 0;\r\nconst issue_1 = __webpack_require__(/*! ../issue */ \"./node_modules/@abaplint/core/build/src/issue.js\");\r\nconst _irule_1 = __webpack_require__(/*! ./_irule */ \"./node_modules/@abaplint/core/build/src/rules/_irule.js\");\r\nconst _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ \"./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js\");\r\nclass AllowedObjectNamingConf extends _basic_rule_config_1.BasicRuleConfig {\r\n}\r\nexports.AllowedObjectNamingConf = AllowedObjectNamingConf;\r\nclass AllowedObjectNaming {\r\n    constructor() {\r\n        this.conf = new AllowedObjectNamingConf();\r\n    }\r\n    getMetadata() {\r\n        return {\r\n            key: \"allowed_object_naming\",\r\n            title: \"Allowed object naming\",\r\n            shortDescription: `Enforces basic name length and namespace restrictions, see note SAP 104010`,\r\n            tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],\r\n        };\r\n    }\r\n    initialize(_reg) {\r\n        return this;\r\n    }\r\n    getConfig() {\r\n        return this.conf;\r\n    }\r\n    setConfig(conf) {\r\n        this.conf = conf;\r\n    }\r\n    run(obj) {\r\n        const allowed = obj.getAllowedNaming();\r\n        const name = obj.getName();\r\n        let message = \"\";\r\n        if (name.length > allowed.maxLength) {\r\n            message = \"Name exceeds max length\";\r\n        }\r\n        else if (allowed.allowNamespace === false && name.indexOf(\"/\") >= 0) {\r\n            message = \"Namespace not allowed for object type\";\r\n        }\r\n        else if (allowed.customRegex !== undefined) {\r\n            if (name.match(allowed.customRegex) === null) {\r\n                message = \"Name not allowed\";\r\n            }\r\n        }\r\n        else if (name.match(/^(\\/[A-Z_\\d]{3,8}\\/)?[A-Z_\\d<> ]+$/i) === null) {\r\n            message = \"Name not allowed\";\r\n        }\r\n        if (message.length > 0) {\r\n            return [issue_1.Issue.atRow(obj.getFiles()[0], 1, message, this.getMetadata().key, this.conf.severity)];\r\n        }\r\n        return [];\r\n    }\r\n}\r\nexports.AllowedObjectNaming = AllowedObjectNaming;\r\n//# sourceMappingURL=allowed_object_naming.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/rules/allowed_object_naming.js?");
 
 /***/ }),
 
@@ -12193,7 +12193,7 @@ eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\n
 /***/ ((__unused_webpack_module, exports, __webpack_require__) => {
 
 "use strict";
-eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.UnnecessaryChaining = exports.UnnecessaryChainingConf = void 0;\r\nconst issue_1 = __webpack_require__(/*! ../issue */ \"./node_modules/@abaplint/core/build/src/issue.js\");\r\nconst _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ \"./node_modules/@abaplint/core/build/src/rules/_abap_rule.js\");\r\nconst _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ \"./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js\");\r\nconst _irule_1 = __webpack_require__(/*! ./_irule */ \"./node_modules/@abaplint/core/build/src/rules/_irule.js\");\r\nclass UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {\r\n}\r\nexports.UnnecessaryChainingConf = UnnecessaryChainingConf;\r\nclass UnnecessaryChaining extends _abap_rule_1.ABAPRule {\r\n    constructor() {\r\n        super(...arguments);\r\n        this.conf = new UnnecessaryChainingConf();\r\n    }\r\n    getMetadata() {\r\n        return {\r\n            key: \"unnecessary_chaining\",\r\n            title: \"Unnecessary Chaining\",\r\n            shortDescription: `Find unnecessary chaining, all statements are checked`,\r\n            extendedInformation: ``,\r\n            tags: [_irule_1.RuleTag.SingleFile],\r\n            badExample: `WRITE: bar.`,\r\n            goodExample: `WRITE bar.`,\r\n        };\r\n    }\r\n    getConfig() {\r\n        return this.conf;\r\n    }\r\n    setConfig(conf) {\r\n        this.conf = conf;\r\n    }\r\n    runParsed(file) {\r\n        var _a, _b;\r\n        const issues = [];\r\n        const statements = file.getStatements();\r\n        for (let i = 0; i < statements.length; i++) {\r\n            const colon = statements[i].getColon();\r\n            if (colon === undefined) {\r\n                continue;\r\n            }\r\n            const next = (_a = statements[i + 1]) === null || _a === void 0 ? void 0 : _a.getColon();\r\n            const prev = (_b = statements[i - 1]) === null || _b === void 0 ? void 0 : _b.getColon();\r\n            if (next !== undefined && colon.getStart().equals(next.getStart())) {\r\n                continue;\r\n            }\r\n            else if (prev !== undefined && colon.getStart().equals(prev.getStart())) {\r\n                continue;\r\n            }\r\n            const message = \"Unnecessary chaining\";\r\n            const issue = issue_1.Issue.atStatement(file, statements[i], message, this.getMetadata().key);\r\n            issues.push(issue);\r\n        }\r\n        return issues;\r\n    }\r\n}\r\nexports.UnnecessaryChaining = UnnecessaryChaining;\r\n//# sourceMappingURL=unnecessary_chaining.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/rules/unnecessary_chaining.js?");
+eval("\r\nObject.defineProperty(exports, \"__esModule\", ({ value: true }));\r\nexports.UnnecessaryChaining = exports.UnnecessaryChainingConf = void 0;\r\nconst issue_1 = __webpack_require__(/*! ../issue */ \"./node_modules/@abaplint/core/build/src/issue.js\");\r\nconst _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ \"./node_modules/@abaplint/core/build/src/rules/_abap_rule.js\");\r\nconst _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ \"./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js\");\r\nconst _irule_1 = __webpack_require__(/*! ./_irule */ \"./node_modules/@abaplint/core/build/src/rules/_irule.js\");\r\nconst edit_helper_1 = __webpack_require__(/*! ../edit_helper */ \"./node_modules/@abaplint/core/build/src/edit_helper.js\");\r\nclass UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {\r\n}\r\nexports.UnnecessaryChainingConf = UnnecessaryChainingConf;\r\nclass UnnecessaryChaining extends _abap_rule_1.ABAPRule {\r\n    constructor() {\r\n        super(...arguments);\r\n        this.conf = new UnnecessaryChainingConf();\r\n    }\r\n    getMetadata() {\r\n        return {\r\n            key: \"unnecessary_chaining\",\r\n            title: \"Unnecessary Chaining\",\r\n            shortDescription: `Find unnecessary chaining, all statements are checked`,\r\n            extendedInformation: ``,\r\n            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],\r\n            badExample: `WRITE: bar.`,\r\n            goodExample: `WRITE bar.`,\r\n        };\r\n    }\r\n    getConfig() {\r\n        return this.conf;\r\n    }\r\n    setConfig(conf) {\r\n        this.conf = conf;\r\n    }\r\n    runParsed(file) {\r\n        var _a, _b;\r\n        const issues = [];\r\n        const statements = file.getStatements();\r\n        for (let i = 0; i < statements.length; i++) {\r\n            const colon = statements[i].getColon();\r\n            if (colon === undefined) {\r\n                continue;\r\n            }\r\n            const next = (_a = statements[i + 1]) === null || _a === void 0 ? void 0 : _a.getColon();\r\n            const prev = (_b = statements[i - 1]) === null || _b === void 0 ? void 0 : _b.getColon();\r\n            if (next !== undefined && colon.getStart().equals(next.getStart())) {\r\n                continue;\r\n            }\r\n            else if (prev !== undefined && colon.getStart().equals(prev.getStart())) {\r\n                continue;\r\n            }\r\n            const fix = edit_helper_1.EditHelper.deleteRange(file, colon.getStart(), colon.getEnd());\r\n            const message = \"Unnecessary chaining\";\r\n            const issue = issue_1.Issue.atStatement(file, statements[i], message, this.getMetadata().key, this.conf.severity, fix);\r\n            issues.push(issue);\r\n        }\r\n        return issues;\r\n    }\r\n}\r\nexports.UnnecessaryChaining = UnnecessaryChaining;\r\n//# sourceMappingURL=unnecessary_chaining.js.map\n\n//# sourceURL=webpack://@abaplint/cli/./node_modules/@abaplint/core/build/src/rules/unnecessary_chaining.js?");
 
 /***/ }),
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.82.13",
+    "version": "2.82.14",
     "description": "abaplint - Command Line Interface",
     "bin": {
         "abaplint": "./abaplint"
@@ -39,7 +39,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.82.13",
+        "@abaplint/core": "^2.82.14",
         "@types/chai": "^4.2.22",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
@@ -48,7 +48,7 @@
         "@types/progress": "^2.0.5",
         "chai": "^4.3.4",
         "chalk": "=4.1.2",
-        "eslint": "^8.3.0",
+        "eslint": "^8.4.0",
         "glob": "^7.2.0",
         "json5": "^2.2.0",
         "memfs": "^3.4.0",