@abaplint/transpiler-cli 2.14.12 → 2.14.14

package/README.md

@@ -98,6 +98,29 @@ The transpiler library retains flat output unless its caller supplies an output
 folder map as the third argument to `Transpiler.run()`. This layout is enabled
 automatically by the CLI.
 
+## Pinning a library
+
+A library given by `url` is cloned when the CLI runs, so without more the
+default branch of that moment is used and the same project commit can build
+differently from one day to the next. Set `libs[].ref` to a commit SHA, a tag
+or a branch to make the build reproducible:
+
+```json
+{
+  "libs": [
+    {"url": "https://github.com/open-abap/open-abap-core", "ref": "239b817e6fd05e62854fed25663f002c60dda07c"},
+    {"url": "https://github.com/example/utilities", "ref": "v1.2.0"}
+  ]
+}
+```
+
+A branch or tag is cloned shallow. A commit is fetched shallow by name; when
+the server does not serve that, or the SHA is abbreviated, the full history is
+fetched and the commit checked out from it. The CLI prints the commit of every
+library it clones, and of every `folder` library that is a git checkout, as
+`commit: <sha>` next to the file count, so the commits of a build can be
+recorded.
+
 ## Duplicate dependencies
 
 By default, two libraries defining the same ABAP object (type and name) cause

package/build/bundle.js

@@ -257,14 +257,101 @@ exports.FileOperations = FileOperations;
 /*!****************************!*\
   !*** ./build/git_clone.js ***!
   \****************************/
-(__unused_webpack_module, exports) {
+(__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.isCommitSha = isCommitSha;
 exports.buildGitCloneArguments = buildGitCloneArguments;
-function buildGitCloneArguments(url) {
-    return ["clone", "--quiet", "--depth", "1", "--", url, "."];
+exports.cloneRepository = cloneRepository;
+exports.resolveHead = resolveHead;
+const childProcess = __importStar(__webpack_require__(/*! child_process */ "child_process"));
+/** a full or abbreviated commit SHA, as opposed to a branch or tag name */
+function isCommitSha(ref) {
+    return /^[0-9a-f]{7,40}$/i.test(ref);
+}
+/** shallow clone of the default branch, or of the branch or tag `ref` */
+function buildGitCloneArguments(url, ref) {
+    const branch = ref === undefined || ref === "" ? [] : ["--branch", ref];
+    return ["clone", "--quiet", "--depth", "1", ...branch, "--", url, "."];
+}
+function git(args, cwd, stderr = "inherit") {
+    childProcess.execFileSync("git", args, { cwd, stdio: ["ignore", "inherit", stderr] });
+}
+/** Clones `url` into the empty directory `dir` and returns the commit checked out.
+ *
+ * Without `ref` this is a shallow clone of the default branch. A branch or tag
+ * is cloned shallow with --branch. A commit SHA is fetched shallow by name,
+ * which GitHub and every server with uploadpack.allowReachableSHA1InWant
+ * serve; when the server refuses that, or the SHA is abbreviated, the full
+ * history is fetched and the commit checked out from it. */
+function cloneRepository(url, dir, ref) {
+    if (ref === undefined || ref === "") {
+        git(buildGitCloneArguments(url), dir);
+    }
+    else if (ref.startsWith("-")) {
+        throw new Error("Invalid lib ref: " + JSON.stringify(ref));
+    }
+    else if (isCommitSha(ref)) {
+        git(["init", "--quiet"], dir);
+        git(["remote", "add", "origin", "--", url], dir);
+        try {
+            git(["fetch", "--quiet", "--depth", "1", "origin", ref], dir, "pipe");
+            git(["checkout", "--quiet", "--detach", "FETCH_HEAD"], dir);
+        }
+        catch {
+            console.log("\tshallow fetch of " + ref + " not served, fetching the full history");
+            git(["fetch", "--quiet", "--tags", "origin"], dir);
+            git(["checkout", "--quiet", "--detach", ref], dir);
+        }
+    }
+    else {
+        git(buildGitCloneArguments(url, ref), dir);
+    }
+    return resolveHead(dir);
+}
+/** the commit checked out in `dir`, undefined when it is not a git checkout */
+function resolveHead(dir) {
+    try {
+        return childProcess.execFileSync("git", ["rev-parse", "HEAD"], { cwd: dir, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] }).trim();
+    }
+    catch {
+        return undefined;
+    }
 }
 //# sourceMappingURL=git_clone.js.map
 
@@ -485,7 +572,6 @@ exports.loadLibraries = loadLibraries;
 exports.libraryRegistry = libraryRegistry;
 const fs = __importStar(__webpack_require__(/*! fs */ "fs"));
 const path = __importStar(__webpack_require__(/*! path */ "path"));
-const childProcess = __importStar(__webpack_require__(/*! child_process */ "child_process"));
 const os = __importStar(__webpack_require__(/*! os */ "os"));
 const abaplint = __importStar(__webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js"));
 const file_operations_1 = __webpack_require__(/*! ./file_operations */ "./build/file_operations.js");
@@ -533,13 +619,14 @@ async function loadLibraries(config) {
                     ? "Library must define a non-empty url or an existing folder"
                     : "Library folder not found: " + folder);
             }
-            console.log("Clone: " + lib.url);
+            console.log("Clone: " + lib.url + (lib.ref ? " at " + lib.ref : ""));
             dir = fs.mkdtempSync(path.join(os.tmpdir(), "abap_transpile-"));
             cleanupFolder = true;
         }
         try {
-            if (cleanupFolder) {
-                childProcess.execFileSync("git", (0, git_clone_1.buildGitCloneArguments)(lib.url), { cwd: dir, stdio: "inherit" });
+            const commit = cleanupFolder ? (0, git_clone_1.cloneRepository)(lib.url, dir, lib.ref) : (0, git_clone_1.resolveHead)(dir);
+            if (commit !== undefined) {
+                console.log("\tcommit: " + commit);
             }
             const patterns = typeof lib.files === "string" && lib.files !== "" ? [lib.files]
                 : Array.isArray(lib.files) ? lib.files : ["/src/**"];
@@ -6282,10 +6369,15 @@ class FormParamType extends combi_1.Expression {
     getRunnable() {
         const def = (0, combi_1.seq)("DEFAULT", (0, combi_1.alt)(_1.Constant, _1.FieldChain));
         const table = (0, combi_1.seq)((0, combi_1.alt)("STANDARD", "HASHED", "INDEX", "SORTED", "ANY"), "TABLE");
-        const tabseq = (0, combi_1.seq)(table, (0, combi_1.optPrio)((0, combi_1.seq)("OF", _1.TypeName)));
-        const ret = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.alt)("REF TO", "LINE OF")), _1.TypeName, (0, combi_1.opt)(def));
-        const like = (0, combi_1.seq)("LIKE", (0, combi_1.optPrio)((0, combi_1.alt)("REF TO", "LINE OF")), _1.FieldChain);
-        return (0, combi_1.alt)((0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(tabseq, (0, combi_1.ver)(version_1.Release.v916, (0, combi_1.seq)("ANY", "STRUCTURE")), ret)), like);
+        // the following are not valid FORM typings, fail instead of reading the
+        // remaining words as additional parameters
+        const inline = (0, combi_1.seq)((0, combi_1.alt)("TABLE", "RANGE"), "OF", (0, combi_1.failCombinator)());
+        const tableKey = (0, combi_1.seq)("WITH", (0, combi_1.failCombinator)());
+        const lengthDecimals = (0, combi_1.seq)((0, combi_1.alt)("LENGTH", "DECIMALS"), _1.Integer, (0, combi_1.failCombinator)());
+        const tabseq = (0, combi_1.seq)(table, (0, combi_1.optPrio)((0, combi_1.seq)("OF", (0, combi_1.altPrio)((0, combi_1.seq)("REF TO", (0, combi_1.failCombinator)()), _1.TypeName))), (0, combi_1.optPrio)(tableKey));
+        const ret = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.alt)("REF TO", "LINE OF")), _1.TypeName, (0, combi_1.optPrio)(lengthDecimals), (0, combi_1.opt)(def));
+        const like = (0, combi_1.seq)("LIKE", (0, combi_1.altPrio)(inline, (0, combi_1.seq)(table, (0, combi_1.failCombinator)()), (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.alt)("REF TO", "LINE OF")), _1.FieldChain)));
+        return (0, combi_1.alt)((0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(tabseq, (0, combi_1.ver)(version_1.Release.v916, (0, combi_1.seq)("ANY", "STRUCTURE")), inline, ret)), like);
     }
 }
 exports.FormParamType = FormParamType;
@@ -29558,6 +29650,15 @@ class TypeUtils {
         }
         return this.isAssignable(source, target);
     }
+    // character-like arguments of built-in functions must be elementary, flat character-like structures are not allowed
+    isAssignableBuiltin(source, target, node) {
+        if (target instanceof basic_1.CLikeType
+            && (source instanceof basic_1.StructureType
+                || (source instanceof basic_1.TableType && source.isWithHeader() && source.getRowType() instanceof basic_1.StructureType))) {
+            return false;
+        }
+        return this.isAssignableStrict(source, target, node);
+    }
     // Typing with the classic STRUCTURE addition, ie. "FORM foo USING bar STRUCTURE zkey",
     // is not a compatibility check: ABAP only checks the fragment view up to the end of the
     // formal structure, so the actual parameter is allowed to be longer. For TABLES parameters
@@ -33190,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) {
@@ -33237,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)) {
@@ -33269,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();
                 }
@@ -34624,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) {
@@ -34636,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) {
@@ -34847,6 +34961,7 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modu
 const constant_1 = __webpack_require__(/*! ./constant */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/constant.js");
 const _check_offset_length_1 = __webpack_require__(/*! ./_check_offset_length */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_offset_length.js");
 const _check_text_symbol_1 = __webpack_require__(/*! ./_check_text_symbol */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_text_symbol.js");
+const _builtin_1 = __webpack_require__(/*! ../_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
 class MethodCallParam {
     /** returns the type of the unnamed argument, if any */
     static runSyntax(node, input, method) {
@@ -34923,7 +35038,7 @@ class MethodCallParam {
             else if ((0, _check_text_symbol_1.checkTextSymbol)(child, targetType, input)) {
                 return;
             }
-            else if (new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(sourceType, targetType, child) === false) {
+            else if (this.isAssignable(sourceType, targetType, child, method, input) === false) {
                 const message = "Method parameter type not compatible";
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
@@ -34943,6 +35058,12 @@ class MethodCallParam {
         }
         return undefined;
     }
+    static isAssignable(sourceType, targetType, child, method, input) {
+        const typeUtils = new _type_utils_1.TypeUtils(input.scope);
+        return method instanceof _builtin_1.BuiltInMethod
+            ? typeUtils.isAssignableBuiltin(sourceType, targetType, child)
+            : typeUtils.isAssignableStrict(sourceType, targetType, child);
+    }
 }
 exports.MethodCallParam = MethodCallParam;
 //# sourceMappingURL=method_call_param.js.map
@@ -35180,6 +35301,7 @@ const field_chain_1 = __webpack_require__(/*! ./field_chain */ "./node_modules/@
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const _check_offset_length_1 = __webpack_require__(/*! ./_check_offset_length */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_offset_length.js");
 const _check_text_symbol_1 = __webpack_require__(/*! ./_check_text_symbol */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/_check_text_symbol.js");
+const _builtin_1 = __webpack_require__(/*! ../_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
 function normalizeParameterName(name) {
     return name.replace(/^!/, "").toUpperCase();
 }
@@ -35327,6 +35449,10 @@ class MethodParameters {
         if (this.requiredParameters === undefined) {
             this.requiredParameters = new Set(method.getParameters().getRequiredParameters().map(i => i.getName().toUpperCase()));
         }
+        const typeUtils = new _type_utils_1.TypeUtils(input.scope);
+        const assignable = method instanceof _builtin_1.BuiltInMethod
+            ? typeUtils.isAssignableBuiltin.bind(typeUtils)
+            : typeUtils.isAssignableStrict.bind(typeUtils);
         for (const item of items) {
             const parameter = allImporting.find(p => p.getName().toUpperCase() === item.name);
             if (parameter === undefined) {
@@ -35337,7 +35463,7 @@ class MethodParameters {
             else if ((0, _check_text_symbol_1.checkTextSymbol)(item.source, parameter.getType(), input)) {
                 return;
             }
-            else if (new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(item.sourceType, parameter.getType(), item.source) === false) {
+            else if (assignable(item.sourceType, parameter.getType(), item.source) === false) {
                 const message = "Method parameter type not compatible, " + item.name;
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
@@ -35770,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 === "#") {
@@ -38033,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;
@@ -38152,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) {
@@ -38177,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)) {
@@ -38225,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();
                 }
@@ -57740,7 +57887,7 @@ const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "./node_modules/@
 const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
 class CDSDetermineTypes {
     parseType(reg, parsedData, ddlsName) {
-        var _a;
+        var _a, _b;
         const ddic = new ddic_1.DDIC(reg);
         if ((parsedData === null || parsedData === void 0 ? void 0 : parsedData.fields.length) === 0) {
             return basic_1.VoidType.get("DDLS:todo");
@@ -57819,13 +57966,29 @@ class CDSDetermineTypes {
                 else {
                     components.push({
                         name: f.name,
-                        type: basic_1.VoidType.get("DDLS:fieldname:" + ddlsName),
+                        type: (_b = this.findInSources(ddic, parsedData, f.nameInSource)) !== null && _b !== void 0 ? _b : basic_1.VoidType.get("DDLS:fieldname:" + ddlsName),
                     });
                 }
             }
             return new basic_1.StructureType(components, parsedData.definitionName, parsedData.definitionName, parsedData.description);
         }
     }
+    /** unprefixed field, take the type from the first source containing it, void or unknown sources are skipped */
+    findInSources(ddic, parsedData, nameInSource) {
+        if (nameInSource === undefined) {
+            return undefined;
+        }
+        for (const source of parsedData.sources) {
+            const lookup = ddic.lookupTableOrView(source.name);
+            if (lookup.type instanceof basic_1.StructureType) {
+                const type = lookup.type.getComponentByName(nameInSource);
+                if (type) {
+                    return type;
+                }
+            }
+        }
+        return undefined;
+    }
 }
 exports.CDSDetermineTypes = CDSDetermineTypes;
 //# sourceMappingURL=cds_determine_types.js.map
@@ -61388,8 +61551,8 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLKind = exports.DDLParser = exports.AbstractFile = exports.ABAPFile = exports.MemoryFile = exports.Edits = exports.ReferenceType = exports.NativeSQL = exports.MacroContent = exports.MacroCall = exports.SpaghettiScopeNode = 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.FormLengthStats = exports.FunctionLengthStats = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.versionToABAPRelease = exports.releaseAtLeast = exports.Release = exports.LanguageVersion = exports.Version = exports.Config = exports.Issue = void 0;
-exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = exports.Info = exports.Diagnostics = exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = void 0;
+exports.Tokens = exports.ABAPParser = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLKind = exports.DDLParser = exports.AbstractFile = exports.ABAPFile = exports.MemoryFile = exports.Edits = exports.ReferenceType = exports.NativeSQL = exports.MacroContent = exports.MacroCall = exports.SpaghettiScopeNode = 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.FormLengthStats = exports.FunctionLengthStats = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.versionToABAPRelease = exports.releaseAtLeast = exports.Release = exports.LanguageVersion = exports.Version = exports.Config = exports.Issue = void 0;
+exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = exports.Info = exports.Diagnostics = exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = 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");
@@ -61486,6 +61649,8 @@ Object.defineProperty(exports, "DDLParser", { enumerable: true, get: function ()
 Object.defineProperty(exports, "DDLKind", { enumerable: true, get: function () { return ddl_parser_1.DDLKind; } });
 const cds_parser_1 = __webpack_require__(/*! ./cds/cds_parser */ "./node_modules/@abaplint/core/build/src/cds/cds_parser.js");
 Object.defineProperty(exports, "CDSParser", { enumerable: true, get: function () { return cds_parser_1.CDSParser; } });
+const abap_parser_1 = __webpack_require__(/*! ./abap/abap_parser */ "./node_modules/@abaplint/core/build/src/abap/abap_parser.js");
+Object.defineProperty(exports, "ABAPParser", { enumerable: true, get: function () { return abap_parser_1.ABAPParser; } });
 const rules_runner_1 = __webpack_require__(/*! ./rules_runner */ "./node_modules/@abaplint/core/build/src/rules_runner.js");
 Object.defineProperty(exports, "RulesRunner", { enumerable: true, get: function () { return rules_runner_1.RulesRunner; } });
 const _irule_1 = __webpack_require__(/*! ./rules/_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
@@ -66389,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");
@@ -66482,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) {
@@ -66508,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() {
@@ -66538,12 +66726,16 @@ class DataDefinition extends _abstract_object_1.AbstractObject {
             let found = (_a = e.findDirectExpression(expressions_1.CDSAs)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(expressions_1.CDSName);
             const list = (_b = e.findDirectExpression(expressions_1.CDSPrefixedName)) === null || _b === void 0 ? void 0 : _b.findAllExpressions(expressions_1.CDSName);
             if (list !== undefined) {
-                const redirected = list.length > 1 && e.concatTokens().toUpperCase().includes(" REDIRECTED TO ");
+                const redirected = e.concatTokens().toUpperCase().includes(" REDIRECTED TO ");
                 if (list.length > 1 && redirected === false) {
                     // "prefix.field" is looked up as "field" in "prefix", also when aliased via "as"
                     prefix = list[0].concatTokens();
                     nameInSource = list[list.length - 1].concatTokens();
                 }
+                else if (list.length === 1 && redirected === false) {
+                    // unprefixed "field" is looked up in the sources, also when aliased via "as"
+                    nameInSource = list[0].concatTokens();
+                }
                 if (found === undefined) {
                     found = redirected === true ? list[0] : list[list.length - 1];
                 }
@@ -73726,7 +73918,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.87";
+        return "2.120.91";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -76383,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 ***!
@@ -85508,6 +85843,18 @@ Make sure to test the downported code, it might not always be completely correct
                     }
                 }
             }
+            // a chained call with EXPORTING/IMPORTING/CHANGING/RECEIVING/EXCEPTIONS parses in the low
+            // version, but 702 reads the last call of a chain as a functional call,
+            // "Unexpected word CHANGING in functional method call"
+            if (ret.length === 0) {
+                for (let i = 0; i < lowStatements.length; i++) {
+                    const issue = this.outlineChainedCallParameters(lowStatements[i], highStatements[i], lowFile, highSyntax);
+                    if (issue) {
+                        ret.push(issue);
+                        break;
+                    }
+                }
+            }
             // INSERT with a 702 built-in function as the operand parses in the low version, but 702
             // reads the function as a method call, "Method SUBSTRING does not exist"
             if (ret.length === 0) {
@@ -88089,6 +88436,67 @@ ${indentation}    output = ${uniqueName}.\n`;
         }
         return undefined;
     }
+    /** a( )->b( CHANGING c = d ), the object the last method is called on is outlined into a
+     * variable typed via the RETURNING parameter of the method before it */
+    outlineChainedCallParameters(low, high, lowFile, highSyntax) {
+        var _a, _b, _c, _d;
+        if (!(high.get() instanceof Statements.Call)) {
+            return undefined;
+        }
+        else if (this.lowReg.getConfig().isOpenABAP()) {
+            return undefined;
+        }
+        else if (low.getFirstToken().getStart() instanceof virtual_position_1.VirtualPosition) {
+            return undefined;
+        }
+        const chain = high.findDirectExpression(Expressions.MethodCallChain);
+        const children = (chain === null || chain === void 0 ? void 0 : chain.getChildren()) || [];
+        const last = children[children.length - 1];
+        const arrow = children[children.length - 2];
+        const previous = children[children.length - 3];
+        if (chain === undefined
+            || !(last instanceof nodes_1.ExpressionNode)
+            || !(last.get() instanceof Expressions.MethodCall)
+            || (arrow === null || arrow === void 0 ? void 0 : arrow.getFirstToken().getStr()) !== "->"
+            || !(previous instanceof nodes_1.ExpressionNode)
+            || !(previous.get() instanceof Expressions.MethodCall)) {
+            return undefined;
+        }
+        const parameters = (_a = last.findDirectExpression(Expressions.MethodCallParam)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.MethodParameters);
+        if (parameters === undefined) {
+            return undefined;
+        }
+        // the type of the object, from the method that returns it
+        const methodToken = (_b = previous.findDirectExpression(Expressions.MethodName)) === null || _b === void 0 ? void 0 : _b.getFirstToken();
+        if (methodToken === undefined) {
+            return undefined;
+        }
+        const spag = highSyntax.spaghetti.lookupPosition(methodToken.getStart(), lowFile.getFilename());
+        let type = undefined;
+        for (const r of (spag === null || spag === void 0 ? void 0 : spag.getData().references) || []) {
+            if (r.referenceType === _reference_1.ReferenceType.MethodReference
+                && r.position.getStart().equals(methodToken.getStart())) {
+                const returning = (_d = (_c = r.resolved) === null || _c === void 0 ? void 0 : _c.getParameters().getReturning()) === null || _d === void 0 ? void 0 : _d.getType();
+                if (returning instanceof basic_1.ObjectReferenceType) {
+                    type = returning.toABAP();
+                }
+                break;
+            }
+        }
+        if (type === undefined) {
+            return undefined;
+        }
+        const object = children.slice(0, children.length - 2).map(c => c.concatTokens()).join("");
+        const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
+        const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
+        const code = `DATA ${uniqueName} TYPE ${type}.\n` +
+            indentation + `${uniqueName} = ${object}.\n` +
+            indentation;
+        const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getFirstToken().getStart(), code);
+        const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, chain.getFirstToken().getStart(), previous.getLastToken().getEnd(), uniqueName);
+        const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
+        return issue_1.Issue.atToken(lowFile, chain.getFirstToken(), "Outline object of chained call with parameter keywords", this.getMetadata().key, this.conf.severity, fix);
+    }
     /** if the source expression is the complete source of an INSERT or APPEND statement, then the
      * outlined variable must be typed via the row type of the table, returns the table name */
     findRowTypeOf(high, source) {
@@ -88491,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, ")"
@@ -88504,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) {
@@ -88557,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) {
@@ -90064,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 ***!
@@ -92960,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);
@@ -93034,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);
@@ -102915,7 +103448,7 @@ class PreferStringTemplate extends _abap_rule_1.ABAPRule {
                 continue;
             }
             const operands = this.flattenChain(source);
-            if (!operands.some(op => { const c = op.getFirstToken().getStr()[0]; return c === "'" || c === "`"; })) {
+            if (!operands.some(op => this.isLiteral(op))) {
                 continue;
             }
             const template = this.buildTemplate(operands);
@@ -102940,10 +103473,11 @@ class PreferStringTemplate extends _abap_rule_1.ABAPRule {
     buildTemplate(operands) {
         let inner = "";
         for (const op of operands) {
-            const raw = op.getFirstToken().getStr();
-            const firstChar = raw[0];
-            if (firstChar === "'" || firstChar === "`") {
-                inner += raw.slice(1, -1);
+            if (this.isLiteral(op)) {
+                const raw = op.getFirstToken().getStr();
+                const delimiter = raw[0];
+                const value = raw.slice(1, -1).split(delimiter + delimiter).join(delimiter);
+                inner += value.replace(/[\\|{}]/g, "\\$&");
             }
             else {
                 inner += `{ ${op.concatTokens()} }`;
@@ -102951,6 +103485,14 @@ class PreferStringTemplate extends _abap_rule_1.ABAPRule {
         }
         return `|${inner}|`;
     }
+    // plain character or string literal, ie. not text symbols or concatenated constants
+    isLiteral(op) {
+        if (op.getFirstToken() !== op.getLastToken()) {
+            return false;
+        }
+        const firstChar = op.getFirstToken().getStr()[0];
+        return firstChar === "'" || firstChar === "`";
+    }
 }
 exports.PreferStringTemplate = PreferStringTemplate;
 //# sourceMappingURL=prefer_string_template.js.map
@@ -113071,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
@@ -113156,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) {
@@ -113189,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) {
@@ -113369,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 = {
@@ -132417,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/build/types.d.ts

@@ -35,6 +35,8 @@ export interface ITranspilerConfig {
          */
         name?: string;
         url?: string;
+        /** commit SHA, tag or branch to check out from url; defaults to the default branch */
+        ref?: string;
         /** relative to the current working directory */
         folder?: string;
         files?: string | string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.14.12",
+  "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.87",
-    "@abaplint/transpiler": "^2.14.12",
+    "@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",