@abaplint/cli 2.107.2 → 2.107.4

package/build/cli.js

@@ -245,7 +245,7 @@ class ApplyFixes {
         const edits = [];
         const appliedIssues = [];
         for (const i of issues) {
-            const edit = i.getFix();
+            const edit = i.getDefaultFix();
             if (edit === undefined) {
                 continue;
             }
@@ -41254,8 +41254,8 @@ exports.MemoryFile = MemoryFile;
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.Visibility = exports.Info = 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.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.ReferenceType = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
-exports.RuleTag = exports.Severity = void 0;
+exports.Info = exports.Diagnostics = 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.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.ReferenceType = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
+exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = 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; } }));
 const config_1 = __webpack_require__(/*! ./config */ "./node_modules/@abaplint/core/build/src/config.js");
@@ -41356,6 +41356,10 @@ const cyclomatic_complexity_stats_1 = __webpack_require__(/*! ./utils/cyclomatic
 Object.defineProperty(exports, "CyclomaticComplexityStats", ({ enumerable: true, get: function () { return cyclomatic_complexity_stats_1.CyclomaticComplexityStats; } }));
 const skip_logic_1 = __webpack_require__(/*! ./skip_logic */ "./node_modules/@abaplint/core/build/src/skip_logic.js");
 Object.defineProperty(exports, "SkipLogic", ({ enumerable: true, get: function () { return skip_logic_1.SkipLogic; } }));
+const diagnostics_1 = __webpack_require__(/*! ./lsp/diagnostics */ "./node_modules/@abaplint/core/build/src/lsp/diagnostics.js");
+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; } }));
 //# sourceMappingURL=index.js.map
 
 /***/ }),
@@ -41401,7 +41405,7 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
         });
     }
@@ -41415,7 +41419,7 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
         });
     }
@@ -41427,7 +41431,7 @@ class Issue {
             key,
             start,
             end,
-            fix,
+            defaultFix: fix,
             severity,
         });
     }
@@ -41440,7 +41444,7 @@ class Issue {
             start: token.getStart(),
             end: token.getEnd(),
             severity,
-            fix,
+            defaultFix: fix,
         });
     }
     static atIdentifier(identifier, message, key, severity, fix) {
@@ -41452,14 +41456,14 @@ class Issue {
             start: identifier.getStart(),
             end: identifier.getEnd(),
             severity,
-            fix,
+            defaultFix: fix,
         });
     }
     constructor(data) {
         this.data = data;
         if (this.data.start instanceof virtual_position_1.VirtualPosition) {
             // no quick fixes inside macros
-            this.data.fix = undefined;
+            this.data.defaultFix = undefined;
         }
         if (this.data.start.getCol() < 1) {
             throw new Error("issue, start col < 1");
@@ -41486,8 +41490,11 @@ class Issue {
     getFilename() {
         return this.data.filename;
     }
-    getFix() {
-        return this.data.fix;
+    getDefaultFix() {
+        return this.data.defaultFix;
+    }
+    getAlternativeFixes() {
+        return this.data.alternativeFixes;
     }
     getSeverity() {
         return this.data.severity;
@@ -41522,22 +41529,6 @@ class LSPEdit {
         }
         return workspace;
     }
-    /*
-      public static mapEditsDocument(edit: IEdit): LServer.WorkspaceEdit {
-        const workspace: LServer.WorkspaceEdit = {documentChanges: []};
-        for (const filename in edit) {
-          const doc: LServer.VersionedTextDocumentIdentifier = {uri: filename, version: 1};
-    
-          const e = LServer.TextDocumentEdit.create(doc, this.mapText(edit[filename]));
-          workspace.documentChanges?.push(e);
-        }
-        // @ts-ignore
-        console.dir(workspace.documentChanges![0].edits[0]);
-        // @ts-ignore
-        console.dir(workspace.documentChanges![0].edits[1]);
-        return workspace;
-      }
-    */
     static mapEdit(edit) {
         const workspace = { changes: {} };
         for (const filename in edit) {
@@ -42040,7 +42031,7 @@ class CodeActions {
         const shown = new Set();
         const ret = [];
         for (const i of issues) {
-            const fix = i.getFix();
+            const fix = i.getDefaultFix();
             if (fix === undefined) {
                 continue;
             }
@@ -42075,7 +42066,7 @@ class CodeActions {
         const diagnostics = [];
         const fixes = [];
         for (const i of issues) {
-            const fix = i.getFix();
+            const fix = i.getDefaultFix();
             if (fix === undefined) {
                 continue;
             }
@@ -51462,7 +51453,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.107.2";
+        return "2.107.4";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52583,6 +52574,14 @@ ENDINTERFACE.`,
     }
     check(fields, column, file) {
         const issues = [];
+        const rows = new Set();
+        for (const f of fields) {
+            const row = f.after.getRow();
+            if (rows.has(row)) {
+                return [];
+            }
+            rows.add(row);
+        }
         for (const f of fields) {
             if (f.after.getCol() === column) {
                 continue;
@@ -53671,7 +53670,7 @@ https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statement
             }
             let prevFix;
             if (previousRow === colon.getStart().getRow()) {
-                prevFix = (_a = issues.pop()) === null || _a === void 0 ? void 0 : _a.getFix();
+                prevFix = (_a = issues.pop()) === null || _a === void 0 ? void 0 : _a.getDefaultFix();
             }
             const fix = this.getFix(file, statement, statementNode, prevFix);
             const message = "Chain mainly declarations";

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.107.2",
+    "version": "2.107.4",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,12 +38,12 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.107.2",
-        "@types/chai": "^4.3.15",
+        "@abaplint/core": "^2.107.4",
+        "@types/chai": "^4.3.16",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.6",
-        "@types/node": "^20.12.8",
+        "@types/node": "^20.12.9",
         "@types/progress": "^2.0.7",
         "chai": "^4.4.1",
         "chalk": "^5.3.0",