@abaplint/cli 2.113.141 → 2.113.143

package/build/cli.js

@@ -102,7 +102,7 @@ class FileOperations {
     static loadFileNames(arg, error = true) {
         const files = glob.sync(arg, { nodir: true, absolute: true, posix: true });
         if (files.length === 0 && error) {
-            throw "Error: No files found";
+            throw "Error: No files found, " + arg;
         }
         return files;
     }
@@ -8791,10 +8791,13 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SQLPath = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
 const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
+const association_name_1 = __webpack_require__(/*! ./association_name */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/association_name.js");
+const sql_cond_1 = __webpack_require__(/*! ./sql_cond */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_cond.js");
 class SQLPath extends combi_1.Expression {
     getRunnable() {
         // todo, only from version?
-        const ret = (0, combi_1.seq)((0, combi_1.regex)(/\\_\w+/), (0, combi_1.tok)(tokens_1.Dash), (0, combi_1.regex)(/\w+/));
+        const condition = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.BracketLeftW), "ONE TO ONE WHERE", sql_cond_1.SQLCond, (0, combi_1.tok)(tokens_1.WBracketRight));
+        const ret = (0, combi_1.seq)(association_name_1.AssociationName, (0, combi_1.optPrio)(condition), (0, combi_1.tok)(tokens_1.Dash), (0, combi_1.regex)(/\w+/));
         return ret;
     }
 }
@@ -16017,7 +16020,7 @@ class ReadEntities {
         const from = (0, combi_1.seq)("FROM", expressions_1.Source);
         const fields = (0, combi_1.seq)("FIELDS", (0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.plus)(expressions_1.SimpleName), (0, combi_1.tok)(tokens_1.WParenRightW), "WITH", expressions_1.Source);
         const all = (0, combi_1.seq)("ALL FIELDS WITH", expressions_1.Source);
-        const entity = (0, combi_1.seq)("ENTITY", expressions_1.SimpleName, (0, combi_1.opt)((0, combi_1.seq)("BY", expressions_1.AssociationName)), (0, combi_1.alt)(fields, from, all), (0, combi_1.optPrio)((0, combi_1.seq)("RESULT", expressions_1.Target)));
+        const entity = (0, combi_1.seq)("ENTITY", expressions_1.NamespaceSimpleName, (0, combi_1.opt)((0, combi_1.seq)("BY", expressions_1.AssociationName)), (0, combi_1.alt)(fields, from, all), (0, combi_1.optPrio)((0, combi_1.seq)("RESULT", expressions_1.Target)));
         const s = (0, combi_1.seq)("READ ENTITIES OF", expressions_1.NamespaceSimpleName, (0, combi_1.opt)("IN LOCAL MODE"), (0, combi_1.plus)(entity), (0, combi_1.optPrio)((0, combi_1.seq)("LINK", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.seq)("FAILED", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.seq)("REPORTED", expressions_1.Target)));
         return (0, combi_1.ver)(version_1.Version.v754, s);
     }
@@ -25742,8 +25745,9 @@ const dereference_1 = __webpack_require__(/*! ./dereference */ "./node_modules/@
 const source_field_symbol_1 = __webpack_require__(/*! ./source_field_symbol */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source_field_symbol.js");
 const source_field_1 = __webpack_require__(/*! ./source_field */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source_field.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class FieldChain {
-    static runSyntax(node, input, refType) {
+    static runSyntax(node, input, refType, allowGenericDeference = false) {
         var _a, _b, _c;
         if (((_a = node.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SourceField
             && node.findDirectExpression(Expressions.ComponentName)) {
@@ -25801,7 +25805,7 @@ class FieldChain {
                         }
                     }
                     else {
-                        const message = "Not a structure, FieldChain";
+                        const message = "Not a structure, FieldChain, " + (context === null || context === void 0 ? void 0 : context.constructor.name) + ", " + current.concatTokens();
                         input.issues.push((0, _syntax_input_1.syntaxIssue)(input, current.getFirstToken(), message));
                         return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                     }
@@ -25818,6 +25822,12 @@ class FieldChain {
             }
             else if (current.get() instanceof expressions_1.Dereference) {
                 context = dereference_1.Dereference.runSyntax(current, context, input);
+                if (allowGenericDeference === false
+                    && (context === null || context === void 0 ? void 0 : context.isGeneric()) === true
+                    && input.scope.getVersion() < version_1.Version.v756
+                    && input.scope.getVersion() !== version_1.Version.Cloud) {
+                    throw new Error("A generic reference cannot be dereferenced");
+                }
             }
             else if (current.get() instanceof Expressions.ComponentName) {
                 if (context instanceof basic_1.TableType && context.isWithHeader()) {
@@ -27982,8 +27992,9 @@ const assert_error_1 = __webpack_require__(/*! ../assert_error */ "./node_module
 * VALUE #( ... ).                 syntax error
 * DATA(bar) = VALUE #( ... ).     give error, no type can be derived
 */
+// TODO: refactor all these method parameters to objects, this is getting messy
 class Source {
-    static runSyntax(node, input, targetType, writeReference = false) {
+    static runSyntax(node, input, targetType, writeReference = false, allowGenericDeference = false) {
         if (node === undefined) {
             return undefined;
         }
@@ -28146,7 +28157,7 @@ class Source {
                     }
                 }
                 else if (get instanceof Expressions.FieldChain) {
-                    context = field_chain_1.FieldChain.runSyntax(first, input, type);
+                    context = field_chain_1.FieldChain.runSyntax(first, input, type, allowGenericDeference);
                 }
                 else if (get instanceof Expressions.StringTemplate) {
                     context = string_template_1.StringTemplate.runSyntax(first, input);
@@ -28811,7 +28822,7 @@ class Target {
                     && !(context instanceof basic_1.TableType && context.isWithHeader() && context.getRowType() instanceof basic_1.StructureType)
                     && !(context instanceof basic_1.TableType && context.isWithHeader() && context.getRowType() instanceof basic_1.VoidType)
                     && !(context instanceof basic_1.VoidType)) {
-                    const message = "Not a structure, target";
+                    const message = "Not a structure, target, " + (context === null || context === void 0 ? void 0 : context.constructor.name) + ", " + current.concatTokens();
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                     return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                 }
@@ -29818,7 +29829,7 @@ class Assign {
             sourceType = basic_1.VoidType.get("Dynamic");
         }
         else {
-            sourceType = source_1.Source.runSyntax(theSource, input);
+            sourceType = source_1.Source.runSyntax(theSource, input, undefined, false, true);
         }
         if ((assignSource === null || assignSource === void 0 ? void 0 : assignSource.getChildren().length) === 5
             && ((_a = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getFirstChild()) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase()) === "COMPONENT") {
@@ -32213,7 +32224,12 @@ class InsertInternal {
                 inline_data_1.InlineData.runSyntax(afterInto, input, new basic_1.DataReference(sourceType));
             }
             else {
-                target_1.Target.runSyntax(afterInto, input);
+                const type = target_1.Target.runSyntax(afterInto, input);
+                if (type instanceof basic_1.TableType && type.getAccessType() === basic_1.TableAccessType.hashed) {
+                    const message = "Implicit or explicit index operation on hashed table is not possible";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                    return;
+                }
             }
         }
         for (const s of node.findDirectExpressions(Expressions.Source)) {
@@ -41463,7 +41479,7 @@ const __1 = __webpack_require__(/*! ../.. */ "./node_modules/@abaplint/core/buil
 const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
 class CDSDefineProjection extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.opt)("ROOT"), "VIEW", (0, combi_1.ver)(__1.Version.v755, (0, combi_1.opt)("ENTITY")), _1.CDSName, (0, combi_1.opt)(_1.CDSProviderContract), "AS PROJECTION ON", _1.CDSName, (0, combi_1.opt)(_1.CDSAs), (0, combi_1.str)("{"), (0, combi_1.plus)(_1.CDSElement), (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.opt)("ROOT"), "VIEW", (0, combi_1.ver)(__1.Version.v755, (0, combi_1.opt)("ENTITY")), _1.CDSName, (0, combi_1.opt)(_1.CDSProviderContract), "AS PROJECTION ON", _1.CDSName, (0, combi_1.opt)(_1.CDSAs), (0, combi_1.str)("{"), (0, combi_1.plus)(_1.CDSElement), (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.str)("}"), (0, combi_1.opt)(_1.CDSWhere), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineProjection = CDSDefineProjection;
@@ -54574,7 +54590,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.141";
+        return "2.113.143";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.141",
+    "version": "2.113.143",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.141",
+        "@abaplint/core": "^2.113.143",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.12",
+        "@types/node": "^24.0.14",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
         "chalk": "^5.4.1",
-        "eslint": "^9.30.1",
+        "eslint": "^9.31.0",
         "glob": "^11.0.3",
         "json5": "^2.2.3",
         "memfs": "^4.17.2",
@@ -55,7 +55,7 @@
         "mocha": "^11.7.1",
         "progress": "^2.0.3",
         "typescript": "^5.8.3",
-        "webpack": "^5.99.9",
+        "webpack": "^5.100.1",
         "webpack-cli": "^6.0.1",
         "xml-js": "^1.6.11"
     }