@abaplint/transpiler-cli 2.13.94 → 2.13.95

package/build/bundle.js

@@ -605,9 +605,8 @@ const BUFS = new Set([
     CH_DOT, CH_COMMA, CH_COLON, 40 /* ( */, 41 /* ) */, 91 /* [ */,
     93 /* ] */, 43 /* + */, CH_AT
 ]);
-// characters that may follow the closing "'", "`" or "|" of a literal, anything else
-// is a syntax error, "There must be a space or equivalent character after ...",
-// note that "(" is allowed, ie. text elements like 'text'(001)
+// characters treated as separators after ping literals and string templates.
+// Single-quoted text literals always end at their closing quote.
 const AFTER_LITERAL = new Set([
     EOF, CH_SPACE, CH_TAB, CH_NL, CH_DOT, CH_COMMA, CH_COLON, CH_DQUOTE,
     40 /* ( */, 41 /* ) */, 91 /* [ */, 93 /* ] */
@@ -925,9 +924,7 @@ class Lexer {
                 && ahead !== CH_QUOTE
                 && this.buffer.countIsEven(CH_QUOTE)) {
                 // end of string
-                if (AFTER_LITERAL.has(ahead)) {
-                    this.add();
-                }
+                this.add();
                 if (ahead === CH_DQUOTE) {
                     this.m = ModeComment;
                 }
@@ -3646,7 +3643,8 @@ class ExpandMacros {
         for (const input of inputs) {
             const search = "&" + i;
             const reg = new RegExp(search, "g");
-            str = str.replace(reg, input);
+            // a function replacement: "$`" and "$'" in the argument are text, not patterns
+            str = str.replace(reg, () => input);
             i++;
         }
         const file = new memory_file_1.MemoryFile("expand_macros.abap.prog", str);
@@ -17390,7 +17388,7 @@ class ModifyEntities {
         const mapped = (0, combi_1.seq)("MAPPED", expressions_1.Target);
         const reported = (0, combi_1.seq)("REPORTED", expressions_1.Target);
         const end = (0, combi_1.optPrio)((0, combi_1.per)(failed, result, mapped, reported));
-        const entities = (0, combi_1.seq)((0, combi_1.optPrio)("AUGMENTING"), "ENTITIES OF", expressions_1.NamespaceSimpleName, (0, combi_1.opt)("IN LOCAL MODE"), (0, combi_1.plusPrio)((0, combi_1.seq)("ENTITY", expressions_1.NamespaceSimpleName, (0, combi_1.plus)(operation))));
+        const entities = (0, combi_1.seq)((0, combi_1.optPrio)("AUGMENTING"), "ENTITIES OF", expressions_1.NamespaceSimpleName, (0, combi_1.optPrio)((0, combi_1.altPrio)("IN LOCAL MODE", (0, combi_1.seq)((0, combi_1.opt)("FORWARDING"), "PRIVILEGED"))), (0, combi_1.plusPrio)((0, combi_1.seq)("ENTITY", expressions_1.NamespaceSimpleName, (0, combi_1.plus)(operation))));
         const dynamic = (0, combi_1.seq)("ENTITIES", (0, combi_1.optPrio)((0, combi_1.altPrio)("IN LOCAL MODE", (0, combi_1.seq)((0, combi_1.opt)("FORWARDING"), "PRIVILEGED"))), "OPERATIONS", expressions_1.Source);
         const create2 = (0, combi_1.seq)("CREATE", fieldsWith, (0, combi_1.opt)((0, combi_1.seq)("CREATE BY", expressions_1.EMLEntityPath, fieldsWith)));
         const create3 = (0, combi_1.seq)("CREATE BY", expressions_1.EMLEntityPath, fieldsWith);
@@ -71317,7 +71315,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.61";
+        return "2.120.63";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -102428,6 +102426,8 @@ class XMLConsistencyConf extends _basic_rule_config_1.BasicRuleConfig {
         this.textAndTranslationLengthSeverity = severity_1.Severity.Error;
         /** Problem severity for data element(DTEL) field checks */
         this.dataElementSeverity = severity_1.Severity.Error;
+        /** Problem severity for transparent table checks */
+        this.transparentTableSeverity = severity_1.Severity.Error;
     }
 }
 exports.XMLConsistencyConf = XMLConsistencyConf;
@@ -102715,11 +102715,11 @@ class XMLConsistency {
         if (obj.getTableCategory() === Objects.TableCategory.Transparent) {
             if (!((_c = obj.getDeliveryClass()) === null || _c === void 0 ? void 0 : _c.trim())) {
                 const message = `Transparent table must have delivery class(CONTFLAG in DD02V) set`;
-                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.severity));
+                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.transparentTableSeverity));
             }
             if (!((_d = obj.getSizeCategory()) === null || _d === void 0 ? void 0 : _d.trim())) {
                 const message = `Transparent table must have size category(TABKAT in DD09L) set`;
-                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.severity));
+                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.transparentTableSeverity));
             }
         }
         return issues;
@@ -118068,7 +118068,10 @@ class LoopTranspiler {
             extra.push(`atLast: (last) => { ${this.atLast} = last; }`);
         }
         const topEquals = {};
-        for (const compare of whereNode?.findDirectExpressions(abaplint.Expressions.ComponentCompare) || []) {
+        // only a pure conjunction makes each top-level `=` a condition every row
+        // has to meet: `a = 1 OR b = 2` narrows by neither
+        const onlyAnd = whereNode?.getChildren().every(c => !(c instanceof abaplint.Nodes.TokenNode) || c.concatTokens().toUpperCase() !== "OR") ?? true;
+        for (const compare of onlyAnd ? whereNode?.findDirectExpressions(abaplint.Expressions.ComponentCompare) || [] : []) {
             const op = compare.findDirectExpression(abaplint.Expressions.CompareOperator)?.concatTokens().toUpperCase();
             if (op !== "=" && op !== "EQ") {
                 continue;
@@ -122169,8 +122172,12 @@ class UpdateDatabaseTranspiler {
             }
             options.push(`"set": [${s.join(",")}]`);
         }
+        const fromTable = node.findExpressionAfterToken("TABLE");
+        if (fromTable && fromTable.get() instanceof abaplint.Expressions.SQLSource) {
+            options.push(`"table": ` + traversal.traverse(fromTable).getCode());
+        }
         const sourceExpression = node.findDirectExpression(abaplint.Expressions.SQLSource)?.findDirectExpression(abaplint.Expressions.SimpleSource3);
-        if (sourceExpression) {
+        if (sourceExpression && fromTable === undefined) {
             const sqlSource = new expressions_1.SourceTranspiler(true).transpile(sourceExpression, traversal).getCode();
             const tableName = node.findDirectExpression(abaplint.Expressions.DatabaseTable)?.concatTokens();
             const tabl = traversal.findTable(tableName);
@@ -131543,7 +131550,23 @@ function pLimit(concurrency) {
 		},
 		map: {
 			async value(iterable, function_) {
-				const promises = Array.from(iterable, (value, index) => generator(function_, value, index));
+				const promises = [];
+
+				try {
+					Array.from(iterable, (value, index) => {
+						const promise = generator(function_, value, index);
+						promises.push(promise);
+						return promise;
+					});
+				} catch (error) {
+					// The iterable threw, so `Promise.all` never observes the already scheduled promises. Observe them here to avoid unhandled rejections.
+					for (const promise of promises) {
+						promise.catch(() => {}); // eslint-disable-line promise/prefer-await-to-then
+					}
+
+					throw error;
+				}
+
 				return Promise.all(promises);
 			},
 		},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.13.94",
+  "version": "2.13.95",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,13 +27,13 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.120.61",
-    "@abaplint/transpiler": "^2.13.94",
+    "@abaplint/core": "^2.120.63",
+    "@abaplint/transpiler": "^2.13.95",
     "@types/glob": "^9.0.0",
     "@types/node": "^26.6.2",
     "@types/progress": "^2.0.7",
     "glob": "^13.0.6",
-    "p-limit": "^7.3.2",
+    "p-limit": "^7.3.3",
     "progress": "^2.0.3",
     "ts-json-schema-generator": "^2.9.0",
     "typescript": "^6.0.3",