@abaplint/transpiler-cli 2.14.13 → 2.14.14

package/build/bundle.js

@@ -33291,6 +33291,7 @@ class FieldChain {
         const children = node.getChildren();
         context = this.findTop(children[0], input, refType);
         let prev = children[0].concatTokens();
+        let afterTableExpression = false;
         for (let i = 1; i < children.length; i++) {
             const current = children[i];
             if (current === undefined) {
@@ -33338,6 +33339,9 @@ class FieldChain {
                 }
             }
             else if (current.get() instanceof tokens_1.InstanceArrow) {
+                if (afterTableExpression && table_expression_1.TableExpression.checkArrowAfter(input, current.getFirstToken())) {
+                    return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                }
                 if (!(context instanceof basic_1.ObjectReferenceType)
                     && !(context instanceof basic_1.DataReference)
                     && !(context instanceof basic_1.VoidType)) {
@@ -33370,6 +33374,7 @@ class FieldChain {
                     return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                 }
                 table_expression_1.TableExpression.runSyntax(current, input, context);
+                afterTableExpression = true;
                 if (!(context instanceof basic_1.VoidType) && !(context instanceof basic_1.UnknownType)) {
                     context = context.getRowType();
                 }
@@ -34725,6 +34730,7 @@ const method_call_param_1 = __webpack_require__(/*! ./method_call_param */ "./no
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const component_name_1 = __webpack_require__(/*! ./component_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_name.js");
 const attribute_name_1 = __webpack_require__(/*! ./attribute_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/attribute_name.js");
+const table_expression_1 = __webpack_require__(/*! ./table_expression */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/table_expression.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class MethodCallChain {
     static runSyntax(node, input, targetType) {
@@ -34737,6 +34743,13 @@ class MethodCallChain {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
         }
+        // the field chain is followed by "->", eg. "itab[ 1 ]-ref->meth( )"
+        if (first instanceof nodes_1.ExpressionNode
+            && first.get() instanceof Expressions.FieldChain
+            && first.findDirectExpression(Expressions.TableExpression) !== undefined
+            && table_expression_1.TableExpression.checkArrowAfter(input, children[1].getFirstToken())) {
+            return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+        }
         let currentIndex = 1;
         let context = this.findTop(first, input, targetType);
         if (first.get() instanceof Expressions.MethodCall) {
@@ -35883,6 +35896,8 @@ class NewObject {
             }
         }
         else if (typeName === "#" && targetType) {
+            const tid = new _typed_identifier_1.TypedIdentifier(typeToken, input.filename, targetType);
+            input.scope.addReference(typeToken, tid, _reference_1.ReferenceType.InferredType, input.filename);
             ret = targetType;
         }
         else if (typeName === "#") {
@@ -38146,9 +38161,23 @@ exports.TableExpression = void 0;
 const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const component_chain_1 = __webpack_require__(/*! ./component_chain */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_chain.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class TableExpression {
+    /** below v750 the object component selector "->" cannot follow a table expression, eg. "itab[ 1 ]-ref->meth( )",
+     *  returns true and adds an issue if not allowed */
+    static checkArrowAfter(input, arrow) {
+        if ((0, version_1.releaseAtLeast)(input.scope.getRelease(), version_1.Release.v750)
+            || input.scope.getLanguageVersion() === version_1.LanguageVersion.Cloud
+            || input.scope.getOpenABAP()) {
+            return false;
+        }
+        const message = "Object component selector \"->\" after table expression not possible below v750";
+        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, arrow, message));
+        return true;
+    }
     static runSyntax(node, input, rowType) {
         if (node === undefined) {
             return;
@@ -38265,6 +38294,7 @@ class Target {
             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
         }
         let currentIndex = 1;
+        let afterTableExpression = false;
         while (currentIndex <= children.length) {
             const current = children[currentIndex];
             if (current === undefined) {
@@ -38290,6 +38320,9 @@ class Target {
                 }
             }
             else if (current.get() instanceof tokens_1.InstanceArrow) {
+                if (afterTableExpression && table_expression_1.TableExpression.checkArrowAfter(input, current.getFirstToken())) {
+                    return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                }
                 if (!(context instanceof basic_1.ObjectReferenceType)
                     && !(context instanceof basic_1.DataReference)
                     && !(context instanceof basic_1.VoidType)) {
@@ -38338,6 +38371,7 @@ class Target {
                     return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
                 }
                 table_expression_1.TableExpression.runSyntax(current, input, context);
+                afterTableExpression = true;
                 if (!(context instanceof basic_1.VoidType) && !(context instanceof unknown_type_1.UnknownType)) {
                     context = context.getRowType();
                 }
@@ -66520,8 +66554,12 @@ exports.DataControl = DataControl;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DataDefinition = void 0;
+const tokens_1 = __webpack_require__(/*! ../abap/1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
+const combi_1 = __webpack_require__(/*! ../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
 const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const cds_determine_types_1 = __webpack_require__(/*! ../cds/cds_determine_types */ "./node_modules/@abaplint/core/build/src/cds/cds_determine_types.js");
+const cds_lexer_1 = __webpack_require__(/*! ../cds/cds_lexer */ "./node_modules/@abaplint/core/build/src/cds/cds_lexer.js");
+const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
 const cds_parser_1 = __webpack_require__(/*! ../cds/cds_parser */ "./node_modules/@abaplint/core/build/src/cds/cds_parser.js");
 const expressions_1 = __webpack_require__(/*! ../cds/expressions */ "./node_modules/@abaplint/core/build/src/cds/expressions/index.js");
 const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
@@ -66613,8 +66651,8 @@ class DataDefinition extends _abstract_object_1.AbstractObject {
             associations: [],
             tree: undefined,
         };
-        this.findSQLViewName();
         this.parsedData.tree = new cds_parser_1.CDSParser().parse(this.findSourceFile());
+        this.findSQLViewName(this.parsedData.tree);
         if (this.parsedData.tree) {
             for (const c of ((_a = this.parsedData.tree) === null || _a === void 0 ? void 0 : _a.getChildren()) || []) {
                 if (c.get() instanceof expressions_1.CDSAnnotation) {
@@ -66639,11 +66677,30 @@ class DataDefinition extends _abstract_object_1.AbstractObject {
         return (_a = this.parsedData) === null || _a === void 0 ? void 0 : _a.tree;
     }
     //////////
-    findSQLViewName() {
+    findSQLViewName(tree) {
         var _a;
-        const match = (_a = this.findSourceFile()) === null || _a === void 0 ? void 0 : _a.getRaw().match(/@AbapCatalog\.sqlViewName: '([\w/]+)'/);
-        if (match) {
-            this.parsedData.sqlViewName = match[1].toUpperCase();
+        let annotations = tree === null || tree === void 0 ? void 0 : tree.findDirectExpressions(expressions_1.CDSAnnotation);
+        if (annotations === undefined) {
+            const file = this.findSourceFile();
+            if (file === undefined) {
+                return;
+            }
+            // Keep SQL view lookup available even when the view body cannot be parsed.
+            const tokens = cds_lexer_1.CDSLexer.run(file).filter(t => !(t instanceof tokens_1.Comment));
+            const leadingAnnotations = (0, combi_1.seq)((0, combi_1.starPrio)(expressions_1.CDSAnnotation), (0, combi_1.starPrio)((0, combi_1.regex)(/.*/)));
+            annotations = (_a = combi_1.Combi.run(leadingAnnotations, tokens, version_1.defaultRelease)) === null || _a === void 0 ? void 0 : _a.filter((n) => n instanceof nodes_1.ExpressionNode && n.get() instanceof expressions_1.CDSAnnotation);
+        }
+        for (const annotation of annotations || []) {
+            const tokens = annotation.getTokens().map(t => t.getStr());
+            const colon = tokens.indexOf(":");
+            if (tokens.slice(0, colon).join("") !== "@AbapCatalog.sqlViewName") {
+                continue;
+            }
+            const match = tokens.slice(colon + 1).join(" ").match(/^'([\w/]+)'$/);
+            if (match) {
+                this.parsedData.sqlViewName = match[1].toUpperCase();
+                return;
+            }
         }
     }
     findDescription() {
@@ -73861,7 +73918,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.89";
+        return "2.120.91";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -76518,6 +76575,149 @@ exports.AbapdocParameterNames = AbapdocParameterNames;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/abapdoc_short_link.js"
+/*!***************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/abapdoc_short_link.js ***!
+  \***************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.AbapdocShortLink = exports.AbapdocShortLinkConf = void 0;
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
+const Expressions = __importStar(__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");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+class AbapdocShortLinkConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.AbapdocShortLinkConf = AbapdocShortLinkConf;
+class AbapdocShortLink extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new AbapdocShortLinkConf();
+    }
+    getMetadata() {
+        return {
+            key: "abapdoc_short_link",
+            title: "Use short form of ABAP Doc links",
+            shortDescription: `Reports ABAP Doc links to components of the current global class or interface which repeat the object name.`,
+            extendedInformation: `Links to components of the same repository object can omit the object name and start with a period,
+{@link zif_foo.data:field} can be written as {@link .data:field}.
+
+Only checks ABAP Doc inside the definition of the global class or interface.
+
+https://github.com/SAP/abap-file-formats/issues/880`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
+            badExample: `INTERFACE zif_foo PUBLIC.
+  "! Same as {@link zif_foo.data:bar}
+  DATA foo TYPE string.
+  DATA bar TYPE string.
+ENDINTERFACE.`,
+            goodExample: `INTERFACE zif_foo PUBLIC.
+  "! Same as {@link .data:bar}
+  DATA foo TYPE string.
+  DATA bar TYPE string.
+ENDINTERFACE.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        var _a;
+        const issues = [];
+        const objectType = obj.getType();
+        if (objectType !== "CLAS" && objectType !== "INTF") {
+            return [];
+        }
+        const objectName = obj.getName().toLowerCase();
+        let definition = false;
+        for (const statement of file.getStatements()) {
+            const type = statement.get();
+            if (type instanceof Statements.ClassDefinition || type instanceof Statements.Interface) {
+                const name = (_a = statement.findFirstExpression(Expressions.ClassName)) !== null && _a !== void 0 ? _a : statement.findFirstExpression(Expressions.InterfaceName);
+                definition = (name === null || name === void 0 ? void 0 : name.getFirstToken().getStr().toLowerCase()) === objectName;
+                continue;
+            }
+            else if (type instanceof Statements.EndClass || type instanceof Statements.EndInterface) {
+                definition = false;
+                continue;
+            }
+            else if (definition === false || !(type instanceof _statement_1.Comment)) {
+                continue;
+            }
+            const token = statement.getFirstToken();
+            const str = token.getStr();
+            if (str.startsWith(`"!`) === false) {
+                continue;
+            }
+            const regex = /\{@link\s+((?:(\w+):)?([\w/]+))\./gi;
+            let match;
+            while ((match = regex.exec(str)) !== null) {
+                const kind = match[2];
+                if (match[3].toLowerCase() !== objectName
+                    || (kind !== undefined && kind.toLowerCase() !== objectType.toLowerCase())) {
+                    continue;
+                }
+                const row = token.getStart().getRow();
+                const col = token.getStart().getCol() + match.index + match[0].length - 1 - match[1].length;
+                const start = new position_1.Position(row, col);
+                const end = new position_1.Position(row, col + match[1].length);
+                const fix = edit_helper_1.EditHelper.deleteRange(file, start, end);
+                const message = `Use short form of ABAP Doc link, remove "${match[1]}"`;
+                issues.push(issue_1.Issue.atRange(file, start, end, message, this.getMetadata().key, this.conf.severity, fix));
+            }
+        }
+        return issues;
+    }
+}
+exports.AbapdocShortLink = AbapdocShortLink;
+//# sourceMappingURL=abapdoc_short_link.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/abapdoc_shorttext_length.js"
 /*!*********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/abapdoc_shorttext_length.js ***!
@@ -88699,12 +88899,9 @@ ${indentation}    output = ${uniqueName}.\n`;
         }
     }
     /** NEW into a data reference creates data, CREATE OBJECT cannot: "target = NEW #|type( [value] )"
-     *  with a variable typed REF TO a data type becomes CREATE DATA plus the assignment of the value */
+     *  with a target typed REF TO a data type becomes CREATE DATA plus the assignment of the value */
     newDataReference(high, target, found, lowFile, highSyntax) {
         var _a, _b, _c;
-        if (target.getChildren().length !== 1 || !(((_a = target.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.TargetField)) {
-            return undefined;
-        }
         const children = found.getChildren();
         const value = found.findDirectExpression(Expressions.Source);
         // NEW, the type, "(", optionally one value, ")"
@@ -88712,12 +88909,23 @@ ${indentation}    output = ${uniqueName}.\n`;
             return undefined;
         }
         const name = target.concatTokens();
+        const typeExpr = found.findDirectExpression(Expressions.TypeNameOrInfer);
+        const typeName = typeExpr === null || typeExpr === void 0 ? void 0 : typeExpr.concatTokens();
         const spag = highSyntax.spaghetti.lookupPosition(high.getFirstToken().getStart(), lowFile.getFilename());
-        const type = (_b = spag === null || spag === void 0 ? void 0 : spag.findVariable(name)) === null || _b === void 0 ? void 0 : _b.getType();
+        let type = undefined;
+        if (typeName === "#") {
+            // the type of the target, as inferred by the syntax check
+            const typeToken = typeExpr.getFirstToken();
+            const resolved = (_a = spag === null || spag === void 0 ? void 0 : spag.getData().references.find(r => r.referenceType === _reference_1.ReferenceType.InferredType
+                && r.position.getStart().equals(typeToken.getStart()))) === null || _a === void 0 ? void 0 : _a.resolved;
+            type = resolved instanceof _typed_identifier_1.TypedIdentifier ? resolved.getType() : undefined;
+        }
+        else if (target.getChildren().length === 1 && ((_b = target.getFirstChild()) === null || _b === void 0 ? void 0 : _b.get()) instanceof Expressions.TargetField) {
+            type = (_c = spag === null || spag === void 0 ? void 0 : spag.findVariable(name)) === null || _c === void 0 ? void 0 : _c.getType();
+        }
         if (!(type instanceof basic_1.DataReference)) {
             return undefined;
         }
-        const typeName = (_c = found.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _c === void 0 ? void 0 : _c.concatTokens();
         const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
         let abap = `CREATE DATA ${name}` + (typeName === "#" ? "" : ` TYPE ${typeName}`) + ".";
         if (value === undefined) {
@@ -88765,7 +88973,7 @@ ${indentation}    output = ${uniqueName}.\n`;
                         }
                     }
                     else {
-                        throw new Error("newParameters, downport, unexpected");
+                        return undefined; // not an object, CREATE OBJECT does not apply
                     }
                 }
                 if (cdef && cdef.getMethodDefinitions === undefined) {
@@ -90272,6 +90480,121 @@ exports.ForbiddenIdentifier = ForbiddenIdentifier;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/forbidden_kernel_write.js"
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/forbidden_kernel_write.js ***!
+  \*******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ForbiddenKernelWrite = exports.ForbiddenKernelWriteConf = void 0;
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.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 issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class ForbiddenKernelWriteConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Objects allowed to use WRITE '@KERNEL', array of string regex matched against the object name.
+         * Empty, the default, allows every object
+         * @uniqueItems true
+        */
+        this.allow = [];
+    }
+}
+exports.ForbiddenKernelWriteConf = ForbiddenKernelWriteConf;
+class ForbiddenKernelWrite extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new ForbiddenKernelWriteConf();
+        this.allow = [];
+    }
+    getMetadata() {
+        return {
+            key: "forbidden_kernel_write",
+            title: "Forbidden kernel WRITE",
+            shortDescription: `Forbid WRITE '@KERNEL' outside the listed objects`,
+            extendedInformation: `The transpiler emits the literal of WRITE '@KERNEL ...' as raw JavaScript,
+which runs with the privileges of the host. Open-abap uses this for its kernel classes, code compiled by a host it did not write should not.
+
+allow is a list of regex, case insensitive, matched against the object name, eg. ["^CL_", "^KERNEL_"].
+An empty list, the default, allows every object. To forbid WRITE '@KERNEL' everywhere use a regex matching no name, eg. ["^$"].`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
+            badExample: `WRITE '@KERNEL process.exit(1);'.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+        if (this.conf.allow === undefined) {
+            this.conf.allow = [];
+        }
+        this.allow = this.conf.allow.map(a => new RegExp(a, "i"));
+    }
+    runParsed(file, obj) {
+        var _a;
+        if (this.allow.length === 0 || this.allow.some(a => a.test(obj.getName()))) {
+            return [];
+        }
+        const ret = [];
+        for (const s of file.getStatements()) {
+            if (!(s.get() instanceof Statements.Write)) {
+                continue;
+            }
+            // same check as the transpiler, statements/write.ts
+            const source = (_a = s.findDirectExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens();
+            if ((source === null || source === void 0 ? void 0 : source.startsWith("'@KERNEL ")) === true) {
+                const message = "WRITE '@KERNEL' not allowed in " + obj.getName();
+                ret.push(issue_1.Issue.atStatement(file, s, message, this.getMetadata().key, this.conf.severity));
+            }
+        }
+        return ret;
+    }
+}
+exports.ForbiddenKernelWrite = ForbiddenKernelWrite;
+//# sourceMappingURL=forbidden_kernel_write.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/forbidden_pseudo_and_pragma.js"
 /*!************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/forbidden_pseudo_and_pragma.js ***!
@@ -93168,6 +93491,7 @@ __exportStar(__webpack_require__(/*! ./abapdoc */ "./node_modules/@abaplint/core
 __exportStar(__webpack_require__(/*! ./abapdoc_html_tag */ "./node_modules/@abaplint/core/build/src/rules/abapdoc_html_tag.js"), exports);
 __exportStar(__webpack_require__(/*! ./abapdoc_leading_at */ "./node_modules/@abaplint/core/build/src/rules/abapdoc_leading_at.js"), exports);
 __exportStar(__webpack_require__(/*! ./abapdoc_parameter_names */ "./node_modules/@abaplint/core/build/src/rules/abapdoc_parameter_names.js"), exports);
+__exportStar(__webpack_require__(/*! ./abapdoc_short_link */ "./node_modules/@abaplint/core/build/src/rules/abapdoc_short_link.js"), exports);
 __exportStar(__webpack_require__(/*! ./abapdoc_shorttext_length */ "./node_modules/@abaplint/core/build/src/rules/abapdoc_shorttext_length.js"), exports);
 __exportStar(__webpack_require__(/*! ./add_test_attributes */ "./node_modules/@abaplint/core/build/src/rules/add_test_attributes.js"), exports);
 __exportStar(__webpack_require__(/*! ./aff_and_xml */ "./node_modules/@abaplint/core/build/src/rules/aff_and_xml.js"), exports);
@@ -93242,6 +93566,7 @@ __exportStar(__webpack_require__(/*! ./exporting */ "./node_modules/@abaplint/co
 __exportStar(__webpack_require__(/*! ./final_newline */ "./node_modules/@abaplint/core/build/src/rules/final_newline.js"), exports);
 __exportStar(__webpack_require__(/*! ./fm_global_parameters_obsolete */ "./node_modules/@abaplint/core/build/src/rules/fm_global_parameters_obsolete.js"), exports);
 __exportStar(__webpack_require__(/*! ./forbidden_identifier */ "./node_modules/@abaplint/core/build/src/rules/forbidden_identifier.js"), exports);
+__exportStar(__webpack_require__(/*! ./forbidden_kernel_write */ "./node_modules/@abaplint/core/build/src/rules/forbidden_kernel_write.js"), exports);
 __exportStar(__webpack_require__(/*! ./forbidden_pseudo_and_pragma */ "./node_modules/@abaplint/core/build/src/rules/forbidden_pseudo_and_pragma.js"), exports);
 __exportStar(__webpack_require__(/*! ./forbidden_void_type */ "./node_modules/@abaplint/core/build/src/rules/forbidden_void_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./form_tables_obsolete */ "./node_modules/@abaplint/core/build/src/rules/form_tables_obsolete.js"), exports);
@@ -113288,6 +113613,7 @@ class XMLConsistency {
             extendedInformation: `Checks:
 * XML is well-formed and parseable
 * Naming for CLAS and INTF objects
+* CLAS and INTF objects do not contain fields that abapGit clears, eg. AUTHOR, CREATEDON and VERSION
 * QUAN fields in TABL objects have reference table and field values
 * Transparent TABL objects have delivery class and size category set
 * TABL fields with a built-in type carry the INTTYPE abapGit serializes for it, for the confirmed pairs
@@ -113373,6 +113699,22 @@ class XMLConsistency {
         else if (((_a = obj.getMainABAPFile()) === null || _a === void 0 ? void 0 : _a.getStructure()) !== undefined && obj.getClassDefinition() === undefined) {
             issues.push(issue_1.Issue.atRow(file, 1, "Class matching XML name not found in ABAP file", this.getMetadata().key, this.conf.severity));
         }
+        issues.push(...this.checkClearedFields(obj, file, "VSEOCLASS", XMLConsistency.clearedClassFields));
+        return issues;
+    }
+    checkClearedFields(obj, file, structure, fields) {
+        var _a, _b;
+        const issues = [];
+        const block = (_b = (_a = obj.getXML()) === null || _a === void 0 ? void 0 : _a.match(new RegExp(`<${structure}>([\\s\\S]*?)</${structure}>`))) === null || _b === void 0 ? void 0 : _b[1];
+        if (block === undefined) {
+            return issues;
+        }
+        for (const field of fields) {
+            if (block.match(new RegExp(`<${field}[\\s/>]`)) !== null) {
+                const message = `Field ${field} in ${structure} is cleared by abapGit and should not be in the XML`;
+                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.severity));
+            }
+        }
         return issues;
     }
     runTextPool(obj, file) {
@@ -113406,6 +113748,7 @@ class XMLConsistency {
         else if (obj.getDefinition() !== undefined && ((_a = obj.getDefinition()) === null || _a === void 0 ? void 0 : _a.getName().toUpperCase()) !== name.toUpperCase()) {
             issues.push(issue_1.Issue.atRow(file, 1, "Interface matching XML name not found in ABAP file", this.getMetadata().key, this.conf.severity));
         }
+        issues.push(...this.checkClearedFields(obj, file, "VSEOINTERF", XMLConsistency.clearedInterfaceFields));
         return issues;
     }
     checkTextLength(file, fieldName, text, maxLength, lang) {
@@ -113586,6 +113929,14 @@ class XMLConsistency {
     }
 }
 exports.XMLConsistency = XMLConsistency;
+// fields abapGit clears before serializing, see ZCL_ABAPGIT_OO_CLASS and ZCL_ABAPGIT_OO_INTERFACE;
+// abapGit omits initial fields, so any occurrence shows as a diff on every pull
+XMLConsistency.clearedInterfaceFields = [
+    "UUID", "AUTHOR", "CREATEDON", "CHANGEDBY", "CHANGEDON", "CHGDANYBY", "CHGDANYON", "R3RELEASE", "VERSION",
+];
+XMLConsistency.clearedClassFields = [
+    ...XMLConsistency.clearedInterfaceFields, "CLSFINAL", "CLSABSTRCT", "EXPOSURE",
+];
 // INTTYPE that abapGit serializes for a built-in DATATYPE, confirmed by exported tables;
 // a different value activates, but shows as a diff on every pull
 XMLConsistency.intTypes = {
@@ -132634,6 +132985,9 @@ class SelectTranspiler {
         else if (node.findDirectTokenByText("DISTINCT") !== undefined) {
             throw new Error("SelectTranspiler, UNION DISTINCT todo");
         }
+        else if (node.findDirectExpression(abaplint.Expressions.Select)?.findDirectTokenByText("UPDATE") !== undefined) {
+            throw new Error("SELECT SINGLE FOR UPDATE, not supported, transpiler");
+        }
         let target = "undefined";
         let intoCorrespondingStructure = false;
         if (targetOverride) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.14.13",
+  "version": "2.14.14",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.120.89",
-    "@abaplint/transpiler": "^2.14.13",
+    "@abaplint/core": "^2.120.91",
+    "@abaplint/transpiler": "^2.14.14",
     "@types/glob": "^9.0.0",
     "@types/node": "^26.6.5",
     "@types/progress": "^2.0.7",