@abaplint/core 2.115.22 → 2.115.24

package/build/src/abap/2_statements/expressions/cast.js

@@ -9,7 +9,7 @@ const let_1 = require("./let");
 class Cast extends combi_1.Expression {
     getRunnable() {
         const rparen = (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenRightW), (0, combi_1.tok)(tokens_1.WParenRight));
-        const cast = (0, combi_1.seq)("CAST", _1.TypeNameOrInfer, (0, combi_1.tok)(tokens_1.ParenLeftW), (0, combi_1.optPrio)(let_1.Let), _1.Source, rparen);
+        const cast = (0, combi_1.seq)("CAST", _1.TypeNameOrInfer, (0, combi_1.tok)(tokens_1.ParenLeftW), (0, combi_1.optPrio)(let_1.Let), _1.Source, rparen, (0, combi_1.opt)(_1.Dereference));
         return (0, combi_1.ver)(version_1.Version.v740sp02, cast, version_1.Version.OpenABAP);
     }
 }

package/build/src/abap/2_statements/statements/move.js

@@ -14,8 +14,7 @@ class Move {
         const equals = (0, combi_1.altPrio)((0, combi_1.altPrio)(chained, "?="), calcAssign);
         // todo, move "?=" to CAST?
         const eq = (0, combi_1.seq)(expressions_1.Target, equals, expressions_1.Source);
-        const eqd = (0, combi_1.seq)(expressions_1.Target, equals, expressions_1.Source, expressions_1.Dereference);
-        return (0, combi_1.alt)(move, eq, eqd);
+        return (0, combi_1.alt)(move, eq);
     }
 }
 exports.Move = Move;

package/build/src/abap/5_syntax/statements/create_object.js

@@ -72,8 +72,9 @@ class CreateObject {
                 return;
             }
             else if (found instanceof basic_1.ObjectReferenceType) {
-                const id = found.getIdentifier();
-                if (id instanceof types_1.InterfaceDefinition && type === undefined) {
+                const identifier = found.getIdentifier();
+                const idFound = input.scope.findObjectDefinition(identifier.getName());
+                if (idFound instanceof types_1.InterfaceDefinition && type === undefined) {
                     const message = "Interface reference, cannot be instantiated";
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return;
@@ -85,11 +86,11 @@ class CreateObject {
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return;
                 }
-                else if (id instanceof types_1.ClassDefinition && cdef === undefined) {
-                    cdef = id;
+                else if (idFound instanceof types_1.ClassDefinition && cdef === undefined) {
+                    cdef = idFound;
                 }
-                if (type === undefined && id instanceof types_1.ClassDefinition && id.isAbstract() === true) {
-                    const message = id.getName() + " is abstract, cannot be instantiated";
+                if (type === undefined && idFound instanceof types_1.ClassDefinition && idFound.isAbstract() === true) {
+                    const message = identifier.getName() + " is abstract, cannot be instantiated";
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return;
                 }

package/build/src/cds/expressions/cds_annotation.js

@@ -7,7 +7,9 @@ const cds_annotation_array_1 = require("./cds_annotation_array");
 class CDSAnnotation extends combi_1.Expression {
     getRunnable() {
         const nameWithSlash = (0, combi_1.seq)((0, combi_1.regex)(/^\w+$/), (0, combi_1.star)((0, combi_1.seq)("/", (0, combi_1.regex)(/^\w+$/))));
-        return (0, combi_1.seq)((0, combi_1.regex)(/^@\w+$/), (0, combi_1.star)((0, combi_1.seq)(".", nameWithSlash)), (0, combi_1.opt)(":"), (0, combi_1.opt)((0, combi_1.alt)(cds_annotation_array_1.CDSAnnotationArray, _1.CDSAnnotationObject, _1.CDSAnnotationSimple)));
+        // Support both "@Name" (single token) and "@ Name" (two tokens with space)
+        const annotationStart = (0, combi_1.alt)((0, combi_1.regex)(/^@\w+$/), (0, combi_1.seq)("@", (0, combi_1.regex)(/^\w+$/)));
+        return (0, combi_1.seq)(annotationStart, (0, combi_1.star)((0, combi_1.seq)(".", nameWithSlash)), (0, combi_1.opt)((0, combi_1.seq)(":", (0, combi_1.alt)(cds_annotation_array_1.CDSAnnotationArray, _1.CDSAnnotationObject, _1.CDSAnnotationSimple))));
     }
 }
 exports.CDSAnnotation = CDSAnnotation;

package/build/src/cds/expressions/cds_arithmetics.js

@@ -9,11 +9,14 @@ class CDSArithmetics extends combi_1.Expression {
         const name = (0, combi_1.seq)(_1.CDSName, (0, combi_1.optPrio)((0, combi_1.seq)(".", _1.CDSName)));
         const val = (0, combi_1.altPrio)(cds_integer_1.CDSInteger, _1.CDSFunction, _1.CDSCase, _1.CDSCast, _1.CDSString, _1.CDSAggregate, name);
         const operator = (0, combi_1.altPrio)("+", "-", "*", "/");
+        // Support unary operators (e.g., "- field" in CASE expressions)
+        const unary = (0, combi_1.altPrio)("-", "+");
+        const unaryExpression = (0, combi_1.seq)(unary, val);
         const operatorValue = (0, combi_1.seq)(operator, val);
         const paren = (0, combi_1.seq)("(", val, (0, combi_1.plusPrio)(operatorValue), ")");
         const noParen = (0, combi_1.seq)(val, (0, combi_1.plusPrio)(operatorValue));
         // todo: this is pretty bad, it needs a rewrite
-        return (0, combi_1.altPrio)((0, combi_1.seq)(paren, (0, combi_1.starPrio)(operatorValue)), noParen);
+        return (0, combi_1.altPrio)(unaryExpression, (0, combi_1.seq)(paren, (0, combi_1.starPrio)(operatorValue)), noParen);
     }
 }
 exports.CDSArithmetics = CDSArithmetics;

package/build/src/objects/rename/renamer_helper.js

@@ -25,7 +25,15 @@ class RenamerHelper {
             }
         }
         // start with the last reference in the file first, if there are multiple refs per line
-        return this.replaceRefs(refs, oldName, newName).reverse();
+        // sort refs by position descending (row desc, then col desc) so edits don't corrupt positions
+        refs.sort((a, b) => {
+            const rowDiff = b.getStart().getRow() - a.getStart().getRow();
+            if (rowDiff !== 0) {
+                return rowDiff;
+            }
+            return b.getStart().getCol() - a.getStart().getCol();
+        });
+        return this.replaceRefs(refs, oldName, newName);
     }
     renameDDICCodeReferences(obj, oldName, newName) {
         const changes = [];

package/build/src/registry.js

@@ -74,7 +74,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.115.22";
+        return "2.115.24";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.115.22",
+    "version": "2.115.24",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,7 +53,7 @@
         "@microsoft/api-extractor": "^7.56.3",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.10.12",
+        "@types/node": "^24.10.13",
         "chai": "^4.5.0",
         "eslint": "^9.39.2",
         "mocha": "^11.7.5",
@@ -63,7 +63,7 @@
         "typescript": "^5.9.3"
     },
     "dependencies": {
-        "fast-xml-parser": "^5.3.4",
+        "fast-xml-parser": "^5.3.5",
         "json5": "^2.2.3",
         "vscode-languageserver-types": "^3.17.5"
     }