@abaplint/cli 2.113.111 → 2.113.113

package/build/cli.js

@@ -269,7 +269,7 @@ class ApplyFixes {
           }
         }
         */
-        const changed = (0, core_1.applyEditList)(reg, edits);
+        const changed = core_1.Edits.applyEditList(reg, edits);
         for (const filename of changed) {
             this.changedFiles.add(filename);
         }
@@ -43065,7 +43065,7 @@ exports.MemoryFile = MemoryFile;
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.ABAPFile = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLParser = exports.NativeSQL = exports.MacroContent = exports.MacroCall = exports.applyEditList = exports.applyEditSingle = exports.SpaghettiScopeNode = exports.AbstractFile = exports.Token = exports.ScopeType = exports.BasicTypes = exports.TypedIdentifier = exports.AbstractType = exports.VirtualPosition = exports.Comment = exports.Unknown = exports.Empty = exports.Identifier = exports.Nodes = exports.Types = exports.Expressions = exports.Statements = exports.Structures = exports.SkipLogic = exports.Objects = exports.ArtifactsRules = exports.ArtifactsObjects = exports.ArtifactsABAP = exports.BuiltIn = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
+exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLParser = exports.AbstractFile = exports.ABAPFile = exports.MemoryFile = exports.Edits = exports.ReferenceType = exports.NativeSQL = exports.MacroContent = exports.MacroCall = exports.SpaghettiScopeNode = exports.Token = exports.ScopeType = exports.BasicTypes = exports.TypedIdentifier = exports.AbstractType = exports.VirtualPosition = exports.Comment = exports.Unknown = exports.Empty = exports.Identifier = exports.Nodes = exports.Types = exports.Expressions = exports.Statements = exports.Structures = exports.SkipLogic = exports.Objects = exports.ArtifactsRules = exports.ArtifactsObjects = exports.ArtifactsABAP = exports.BuiltIn = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.Version = exports.Config = exports.Issue = void 0;
 exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = exports.Info = exports.Diagnostics = void 0;
 const issue_1 = __webpack_require__(/*! ./issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 Object.defineProperty(exports, "Issue", ({ enumerable: true, get: function () { return issue_1.Issue; } }));
@@ -43138,8 +43138,6 @@ Object.defineProperty(exports, "MacroCall", ({ enumerable: true, get: function (
 Object.defineProperty(exports, "MacroContent", ({ enumerable: true, get: function () { return _statement_1.MacroContent; } }));
 Object.defineProperty(exports, "NativeSQL", ({ enumerable: true, get: function () { return _statement_1.NativeSQL; } }));
 const edit_helper_1 = __webpack_require__(/*! ./edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
-Object.defineProperty(exports, "applyEditSingle", ({ enumerable: true, get: function () { return edit_helper_1.applyEditSingle; } }));
-Object.defineProperty(exports, "applyEditList", ({ enumerable: true, get: function () { return edit_helper_1.applyEditList; } }));
 const _builtin_1 = __webpack_require__(/*! ./abap/5_syntax/_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
 Object.defineProperty(exports, "BuiltIn", ({ enumerable: true, get: function () { return _builtin_1.BuiltIn; } }));
 const abap_file_1 = __webpack_require__(/*! ./abap/abap_file */ "./node_modules/@abaplint/core/build/src/abap/abap_file.js");
@@ -43172,6 +43170,15 @@ const diagnostics_1 = __webpack_require__(/*! ./lsp/diagnostics */ "./node_modul
 Object.defineProperty(exports, "Diagnostics", ({ enumerable: true, get: function () { return diagnostics_1.Diagnostics; } }));
 const _edit_1 = __webpack_require__(/*! ./lsp/_edit */ "./node_modules/@abaplint/core/build/src/lsp/_edit.js");
 Object.defineProperty(exports, "LSPEdit", ({ enumerable: true, get: function () { return _edit_1.LSPEdit; } }));
+const _reference_1 = __webpack_require__(/*! ./abap/5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+Object.defineProperty(exports, "ReferenceType", ({ enumerable: true, get: function () { return _reference_1.ReferenceType; } }));
+// do not include this file from anywhere within abaplint
+// https://github.com/abaplint/abaplint/issues/873
+const Edits = {
+    applyEditSingle: edit_helper_1.applyEditSingle,
+    applyEditList: edit_helper_1.applyEditList,
+};
+exports.Edits = Edits;
 //# sourceMappingURL=index.js.map
 
 /***/ }),
@@ -53918,7 +53925,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.111";
+        return "2.113.113";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -54677,7 +54684,7 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
-const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class AlignParametersConf extends _basic_rule_config_1.BasicRuleConfig {
 }
@@ -54885,9 +54892,9 @@ DATA(sdf) = VALUE type(
     }
     raiseAndCreateCandidates(stru) {
         const candidates = [];
-        const statements = stru.findAllStatements(__1.Statements.Raise);
-        statements.push(...stru.findAllStatements(__1.Statements.CreateObject));
-        statements.push(...stru.findAllStatements(__1.Statements.RaiseEvent));
+        const statements = stru.findAllStatements(Statements.Raise);
+        statements.push(...stru.findAllStatements(Statements.CreateObject));
+        statements.push(...stru.findAllStatements(Statements.RaiseEvent));
         for (const raise of statements) {
             const parameters = [];
             const param = raise.findDirectExpression(Expressions.ParameterListS);
@@ -57825,7 +57832,8 @@ exports.ConstantClasses = exports.ConstantClassesConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
-const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
+const Objects = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const visibility_1 = __webpack_require__(/*! ../abap/4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 /** Checks that constants classes are in sync with domain fixed values */
 class ConstantClassesConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
@@ -57861,7 +57869,7 @@ class ConstantClasses {
     run(obj) {
         if (this.conf
             && this.conf.mapping
-            && obj instanceof __1.Objects.Domain) {
+            && obj instanceof Objects.Domain) {
             const configEntry = this.conf.mapping.find(x => x.domain.toUpperCase() === obj.getName().toUpperCase());
             if (!configEntry) {
                 return [];
@@ -57905,7 +57913,7 @@ class ConstantClasses {
                     issues.push(this.issueAtConstant(constant, `Use exact type ${configEntry.domain} instead of ${constant.typeName}`));
                     // quickfix will change the type
                 }
-                if (constant.visibility !== __1.Visibility.Public) {
+                if (constant.visibility !== visibility_1.Visibility.Public) {
                     issues.push(this.issueAtConstant(constant, `Constant ${constant.name} should be public`));
                     // quickfix will move constant
                 }
@@ -63441,8 +63449,8 @@ exports.FullyTypeITabs = FullyTypeITabs;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.FunctionModuleRecommendations = exports.FunctionModuleRecommendationsConf = void 0;
-const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
 const expressions_1 = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
@@ -63525,7 +63533,7 @@ class FunctionModuleRecommendations extends _abap_rule_1.ABAPRule {
             funcName = funcName.slice(1, funcName.length - 1);
             const index = this.conf.recommendations.findIndex(i => i.name.toUpperCase() === funcName && (i.from === undefined || configVersion >= i.from));
             if (index >= 0) {
-                issues.push(__1.Issue.atToken(file, token, this.getMessage(index), this.getMetadata().key, this.conf.severity));
+                issues.push(issue_1.Issue.atToken(file, token, this.getMessage(index), this.getMetadata().key, this.conf.severity));
             }
         }
         return issues;
@@ -67480,8 +67488,8 @@ exports.MethodLength = MethodLength;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.MethodOverwritesBuiltIn = exports.MethodOverwritesBuiltInConf = void 0;
-const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
 const _builtin_1 = __webpack_require__(/*! ../abap/5_syntax/_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
@@ -67531,7 +67539,7 @@ ENDCLASS.`,
         for (const method of methods) {
             if (builtIn.searchBuiltin(method.name.toUpperCase())) {
                 const message = `Method name "${method.name}" overwrites built-in SAP function name`;
-                issues.push(__1.Issue.atIdentifier(method.identifier, message, this.getMetadata().key));
+                issues.push(issue_1.Issue.atIdentifier(method.identifier, message, this.getMetadata().key));
             }
         }
         return issues;
@@ -70926,7 +70934,8 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
-const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
+const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
 class PreferRaiseExceptionNewConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.PreferRaiseExceptionNewConf = PreferRaiseExceptionNewConf;
@@ -70958,12 +70967,12 @@ From 752 and up`,
         this.conf = conf;
     }
     runParsed(file) {
-        if (this.reg.getConfig().getVersion() < __1.Version.v752) {
+        if (this.reg.getConfig().getVersion() < version_1.Version.v752) {
             return [];
         }
         const issues = [];
         for (const statement of file.getStatements()) {
-            if (statement.get() instanceof __1.Statements.Raise) {
+            if (statement.get() instanceof Statements.Raise) {
                 const concat = statement.concatTokens().toUpperCase();
                 if (concat.includes(" MESSAGE")) {
                     continue;
@@ -72142,11 +72151,15 @@ exports.SelectPerformance = SelectPerformance;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SelectSingleFullKey = exports.SelectSingleFullKeyConf = void 0;
-const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
-const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
-const __1 = __webpack_require__(/*! .. */ "./node_modules/@abaplint/core/build/src/index.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js");
+const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class SelectSingleFullKeyConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -72196,10 +72209,10 @@ If the statement contains a JOIN it is not checked`,
     }
     run(obj) {
         var _a, _b;
-        if (!(obj instanceof __1.ABAPObject)) {
+        if (!(obj instanceof _abap_object_1.ABAPObject)) {
             return [];
         }
-        const syntax = new __1.SyntaxLogic(this.reg, obj).run();
+        const syntax = new syntax_1.SyntaxLogic(this.reg, obj).run();
         if (syntax.issues.length > 0) {
             return [];
         }
@@ -72209,21 +72222,21 @@ If the statement contains a JOIN it is not checked`,
             const statements = file.getStatements();
             for (let i = 0; i < statements.length; i++) {
                 const s = statements[i];
-                if (!(s.get() instanceof __1.Statements.Select)) {
+                if (!(s.get() instanceof Statements.Select)) {
                     continue;
                 }
-                else if (s.findFirstExpression(__1.Expressions.SQLJoin)) {
+                else if (s.findFirstExpression(Expressions.SQLJoin)) {
                     continue;
                 }
                 else if (s.findTokenSequencePosition("SELECT", "SINGLE") === undefined) {
                     continue;
                 }
-                const databaseTable = s.findFirstExpression(__1.Expressions.DatabaseTable);
+                const databaseTable = s.findFirstExpression(Expressions.DatabaseTable);
                 if (databaseTable === undefined) {
                     continue;
                 }
                 const next = statements[i + 1];
-                if ((next === null || next === void 0 ? void 0 : next.get()) instanceof __1.Comment
+                if ((next === null || next === void 0 ? void 0 : next.get()) instanceof _statement_1.Comment
                     && next.concatTokens().includes(this.getMetadata().pseudoComment + "")) {
                     if (this.getConfig().allowPseudo !== true) {
                         issues.push(issue_1.Issue.atStatement(file, s, "Pseudo comment not allowed", this.getMetadata().key, this.getConfig().severity));
@@ -72236,7 +72249,7 @@ If the statement contains a JOIN it is not checked`,
                     continue;
                 }
                 const keys = table.listKeys(this.reg);
-                const cond = s.findFirstExpression(__1.Expressions.SQLCond);
+                const cond = s.findFirstExpression(Expressions.SQLCond);
                 const set = new Set();
                 for (const key of keys) {
                     if (key === "MANDT") {
@@ -72245,10 +72258,10 @@ If the statement contains a JOIN it is not checked`,
                     }
                     set.add(key);
                 }
-                for (const compare of (cond === null || cond === void 0 ? void 0 : cond.findAllExpressionsRecursive(__1.Expressions.SQLCompare)) || []) {
+                for (const compare of (cond === null || cond === void 0 ? void 0 : cond.findAllExpressionsRecursive(Expressions.SQLCompare)) || []) {
                     if (compare.getChildren().length === 3) {
-                        const fname = (_a = compare.findDirectExpression(__1.Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
-                        const operator = (_b = compare.findDirectExpression(__1.Expressions.SQLCompareOperator)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
+                        const fname = (_a = compare.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+                        const operator = (_b = compare.findDirectExpression(Expressions.SQLCompareOperator)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
                         if (fname && (operator === "=" || operator === "EQ")) {
                             set.delete(fname);
                         }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.111",
+    "version": "2.113.113",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.111",
+        "@abaplint/core": "^2.113.113",
         "@types/chai": "^4.3.20",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",