@abaplint/transpiler-cli 2.14.10 → 2.14.11

package/build/bundle.js

@@ -1619,11 +1619,40 @@ exports.LexerStream = LexerStream;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.AbstractToken = void 0;
 const position_1 = __webpack_require__(/*! ../../../position */ "./node_modules/@abaplint/core/build/src/position.js");
-class AbstractToken {
+const virtual_position_1 = __webpack_require__(/*! ../../../virtual_position */ "./node_modules/@abaplint/core/build/src/virtual_position.js");
+// token strings repeat a lot across files (keywords, common names, upper cased copies), share them.
+// Longer strings are typically comments or text literals, not worth sharing
+const INTERN_MAX_LENGTH = 30;
+// bound the table, it lives for the lifetime of the process
+const INTERN_MAX_ENTRIES = 100000;
+// V8 represents substrings of this length or longer as slices that reference the full source,
+// these are flattened before being stored, so the table does not keep removed files alive
+const SLICED_MIN_LENGTH = 13;
+let interned = new Map();
+function intern(str) {
+    if (str.length > INTERN_MAX_LENGTH) {
+        return str;
+    }
+    const found = interned.get(str);
+    if (found !== undefined) {
+        return found;
+    }
+    if (interned.size >= INTERN_MAX_ENTRIES) {
+        interned = new Map();
+    }
+    const copy = str.length >= SLICED_MIN_LENGTH ? str.split("").join("") : str;
+    interned.set(copy, copy);
+    return copy;
+}
+// the token is its own start position, saves a separate Position object per token
+class AbstractToken extends position_1.Position {
     constructor(start, str) {
-        this.start = start;
-        this.str = str;
-        this.strUpper = str.toUpperCase();
+        super(start.getRow(), start.getCol());
+        this.str = intern(str);
+        this.strUpper = intern(str.toUpperCase());
+        if (start instanceof virtual_position_1.VirtualPosition) {
+            this.virtual = start;
+        }
     }
     // special function, for debugging purposes, see https://github.com/abaplint/abaplint/pull/3137
     [Symbol.for("debug.description")]() {
@@ -1635,17 +1664,12 @@ class AbstractToken {
     getUpperStr() {
         return this.strUpper;
     }
-    getRow() {
-        return this.start.getRow();
-    }
-    getCol() {
-        return this.start.getCol();
-    }
     getStart() {
-        return this.start;
+        var _a;
+        return (_a = this.virtual) !== null && _a !== void 0 ? _a : this;
     }
     getEnd() {
-        return new position_1.Position(this.start.getRow(), this.start.getCol() + this.str.length);
+        return new position_1.Position(this.getRow(), this.getCol() + this.str.length);
     }
 }
 exports.AbstractToken = AbstractToken;
@@ -12333,23 +12357,25 @@ class Result {
         return this.tokens[this.tokenIndex + offset];
     }
     shift(node) {
-        return Result.fromChain(this.tokens, this.tokenIndex + 1, { node, previous: this.nodes }, this.nodeCount + 1);
+        return Result.fromChain(this.tokens, this.tokenIndex + 1, { node, tokenCount: 1, previous: this.nodes }, this.nodeCount + 1);
     }
     wrapConsumed(consumedTokens, node) {
         let current = this.nodes;
         let currentCount = this.nodeCount;
+        let tokenCount = 0;
         const children = [];
         while (consumedTokens > 0) {
             if (current === undefined) {
                 break;
             }
             children.push(current.node);
-            consumedTokens = consumedTokens - current.node.countTokens();
+            consumedTokens = consumedTokens - current.tokenCount;
+            tokenCount = tokenCount + current.tokenCount;
             current = current.previous;
             currentCount--;
         }
         node.setChildren(children.reverse());
-        this.nodes = { node, previous: current };
+        this.nodes = { node, tokenCount, previous: current };
         this.nodeCount = currentCount + 1;
         return this;
     }
@@ -12378,7 +12404,7 @@ class Result {
         this.nodes = undefined;
         this.nodeCount = 0;
         for (const node of n) {
-            this.nodes = { node, previous: this.nodes };
+            this.nodes = { node, tokenCount: node.countTokens(), previous: this.nodes };
             this.nodeCount++;
         }
     }
@@ -20468,11 +20494,15 @@ class Submit {
         const datasetExpiration = (0, combi_1.seq)("DATASET EXPIRATION", expressions_1.Source);
         const coverText = (0, combi_1.seq)("COVER TEXT", expressions_1.Source);
         const listName = (0, combi_1.seq)("LIST NAME", expressions_1.Source);
+        const listDataset = (0, combi_1.seq)("LIST DATASET", expressions_1.Source);
+        const listAuthority = (0, combi_1.seq)("LIST AUTHORITY", expressions_1.Source);
+        const coverPage = (0, combi_1.seq)("COVER PAGE", expressions_1.Source);
+        const department = (0, combi_1.seq)("DEPARTMENT", expressions_1.Source);
         const keep = (0, combi_1.seq)("KEEP IN SPOOL", expressions_1.Source);
         const imm = (0, combi_1.seq)("IMMEDIATELY", expressions_1.Source);
         const dest = (0, combi_1.seq)("DESTINATION", expressions_1.Source);
         const language = (0, combi_1.seq)("LANGUAGE", expressions_1.Source);
-        const perm = (0, combi_1.per)((0, combi_1.plusPrio)(awith), selectionTable, (0, combi_1.plusPrio)(awith), (0, combi_1.plusPrio)(awith), spool, lineSize, lineCount, archive, archiveMode, receiver, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
+        const perm = (0, combi_1.per)((0, combi_1.plusPrio)(awith), selectionTable, (0, combi_1.plusPrio)(awith), (0, combi_1.plusPrio)(awith), spool, lineSize, lineCount, archive, archiveMode, receiver, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, listDataset, listAuthority, coverPage, department, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
         const ret = (0, combi_1.seq)("SUBMIT", prog, (0, combi_1.opt)(perm));
         return (0, combi_1.verNotLang)(version_1.LanguageVersion.Cloud, ret);
     }
@@ -27265,7 +27295,7 @@ class CurrentScope {
         else if (this.isOO() && this.current.getData().vars[upper] !== undefined) {
             throw new Error(`"${name}" already defined`);
         }
-        this.current.getData().types[upper] = type;
+        this.current.setType(upper, type);
     }
     addExtraLikeType(type) {
         if (type === undefined) {
@@ -27284,7 +27314,7 @@ class CurrentScope {
         if (this.current.getData().extraLikeTypes[upper] !== undefined) {
             throw new Error(`Type name "${name}" already defined`);
         }
-        this.current.getData().extraLikeTypes[upper] = type;
+        this.current.setExtraLikeType(upper, type);
     }
     addClassDefinition(c) {
         if (this.current === undefined) {
@@ -27294,13 +27324,13 @@ class CurrentScope {
         if (this.current.getData().cdefs[name] !== undefined) {
             throw new Error(`Class "${name}" already defined`);
         }
-        this.current.getData().cdefs[name] = c;
+        this.current.setClassDefinition(name, c);
     }
     addFormDefinitions(f) {
         if (this.current === undefined) {
             return;
         }
-        this.current.getData().forms.push(...f);
+        this.current.addFormDefinitions(f);
     }
     /** The FORM definitions are built up front, at that point the program level types are not known.
      * When the FORM statement is traversed it is replaced with one resolved in the scope of the FORM */
@@ -27325,7 +27355,7 @@ class CurrentScope {
         if (this.current.getData().cdefs[name] !== undefined) {
             throw new Error(`Interface "${name}" already defined`);
         }
-        this.current.getData().idefs[name] = i;
+        this.current.setInterfaceDefinition(name, i);
     }
     addNamedIdentifier(name, identifier) {
         if (this.current === undefined) {
@@ -27339,7 +27369,7 @@ class CurrentScope {
         else if (this.isOO() && this.current.getData().types[upper] !== undefined) {
             throw new Error(`"${name}" already defined`);
         }
-        this.current.getData().vars[upper] = identifier;
+        this.current.setVariable(upper, identifier);
     }
     addNamedIdentifierToParent(name, identifier) {
         if (this.current === undefined) {
@@ -27357,7 +27387,7 @@ class CurrentScope {
         else if (this.isOO() && parent.getData().types[upper] !== undefined) {
             throw new Error(`"${name}" already defined`);
         }
-        parent.getData().vars[upper] = identifier;
+        parent.setVariable(upper, identifier);
     }
     addIdentifier(identifier) {
         if (identifier === undefined) {
@@ -27369,7 +27399,7 @@ class CurrentScope {
         if (token === undefined) {
             return;
         }
-        this.current.getData().deferred[token.getStr().toUpperCase()] = { token, ooType: type };
+        this.current.setDeferred(token.getStr().toUpperCase(), { token, ooType: type });
     }
     addListPrefix(identifiers, prefix) {
         for (const id of identifiers) {
@@ -27389,16 +27419,16 @@ class CurrentScope {
         const position = new _identifier_1.Identifier(usage, filename);
         if (Array.isArray(type)) {
             for (const t of type) {
-                (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().references.push({ position, resolved: referencing, referenceType: t, extra });
+                (_a = this.current) === null || _a === void 0 ? void 0 : _a.addReference({ position, resolved: referencing, referenceType: t, extra });
             }
         }
         else {
-            (_b = this.current) === null || _b === void 0 ? void 0 : _b.getData().references.push({ position, resolved: referencing, referenceType: type, extra });
+            (_b = this.current) === null || _b === void 0 ? void 0 : _b.addReference({ position, resolved: referencing, referenceType: type, extra });
         }
     }
     addSQLConversion(fieldName, message, token) {
         var _a;
-        (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().sqlConversion.push({ fieldName, message, token });
+        (_a = this.current) === null || _a === void 0 ? void 0 : _a.addSQLConversion({ fieldName, message, token });
     }
     ///////////////////////////
     findFunctionModule(name) {
@@ -27434,6 +27464,9 @@ class CurrentScope {
             return undefined;
         }
         const name = (_a = this.parentObj.getClassDefinition()) === null || _a === void 0 ? void 0 : _a.behaviorDefinitionName;
+        return this.findBehaviorDefinitionByName(name);
+    }
+    findBehaviorDefinitionByName(name) {
         if (name === undefined) {
             return undefined;
         }
@@ -27763,6 +27796,84 @@ exports.CurrentScope = CurrentScope;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_eml.js"
+/*!*********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/_eml.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.EML = void 0;
+const Expressions = __importStar(__webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const _syntax_input_1 = __webpack_require__(/*! ./_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+/** checks the entities referenced in EML statements, eg. READ ENTITIES and MODIFY ENTITIES */
+class EML {
+    static checkEntities(node, input) {
+        var _a;
+        var _b;
+        const nameNode = node.findDirectExpression(Expressions.NamespaceSimpleName);
+        if (nameNode === undefined) {
+            // eg. dynamic "MODIFY ENTITIES OPERATIONS" or "ENTITY zentity\_association"
+            return;
+        }
+        const name = nameNode.concatTokens();
+        const scope = input.scope;
+        if (node.findDirectTokenByText("ENTITIES") !== undefined) {
+            // "ENTITIES OF" refers to the behavior definition
+            if (scope.findBehaviorDefinitionByName(name) === undefined && scope.getDDIC().inErrorNamespace(name)) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameNode.getFirstToken(), `Behavior definition "${name}" not found`));
+            }
+            return;
+        }
+        // single "ENTITY", inside a behavior pool the alias defined in the BDEF can be used
+        const entity = (_b = (_a = scope.findBehaviorDefinition()) === null || _a === void 0 ? void 0 : _a.findEntityNameByAlias(name)) !== null && _b !== void 0 ? _b : name;
+        if (scope.getDDIC().inErrorNamespace(entity)
+            && scope.findBehaviorDefinitionByName(entity) === undefined
+            && scope.getDDIC().lookupDDLS(entity) === undefined) {
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameNode.getFirstToken(), `Entity "${name}" not found`));
+        }
+    }
+}
+exports.EML = EML;
+//# sourceMappingURL=_eml.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_loop_table_change.js"
 /*!***********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/_loop_table_change.js ***!
@@ -28133,6 +28244,16 @@ class ObjectOriented {
         }
         return `The special method "${name}" cannot be called directly`;
     }
+    // classic exceptions cannot be raised in a method declaring class based exceptions, returns the message otherwise
+    checkClassicException() {
+        var _a;
+        const methodName = this.scope.getEnclosingMethodName();
+        const classDef = this.scope.findClassDefinition(this.scope.getEnclosingClassName());
+        if (methodName && classDef && ((_a = this.searchMethodName(classDef, methodName).method) === null || _a === void 0 ? void 0 : _a.getRaising().length)) {
+            return "Classic exceptions cannot be raised in methods with class based exceptions";
+        }
+        return undefined;
+    }
     // SUPER-> can only call the previous implementation of the method it is used in, returns the message otherwise
     // both names are resolved in the super class, so an alias and the method it stands for are the same
     checkSuperCall(superDef, methodName) {
@@ -30752,6 +30873,10 @@ class BasicTypes {
                 this.input.issues.push((0, _syntax_input_1.syntaxIssue)(this.input, chain.getFirstToken(), message));
                 return undefined;
             }
+            else if (concat === "SY-REPID" || concat === "SYST-REPID") {
+                // predefined constant in every program, not a SYST component
+                return undefined;
+            }
             return this.resolveConstantValue(chain);
         }
         throw new Error("findValue, unexpected");
@@ -38742,23 +38867,81 @@ const _typed_identifier_1 = __webpack_require__(/*! ../types/_typed_identifier *
 const _identifier_1 = __webpack_require__(/*! ../4_file_information/_identifier */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_identifier.js");
 const _reference_1 = __webpack_require__(/*! ./_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const assert_error_1 = __webpack_require__(/*! ./assert_error */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/assert_error.js");
+// most scopes only use a few of the containers, empty ones are shared and replaced on first write
+const EMPTY_OBJECT = Object.freeze({});
+const EMPTY_ARRAY = Object.freeze([]);
 class ScopeData {
     constructor() {
         this.data = {
-            vars: {},
-            cdefs: {},
-            idefs: {},
-            forms: [], // todo, refactor to object
-            types: {},
-            extraLikeTypes: {},
-            deferred: {},
-            references: [],
-            sqlConversion: [],
+            vars: EMPTY_OBJECT,
+            cdefs: EMPTY_OBJECT,
+            idefs: EMPTY_OBJECT,
+            forms: EMPTY_ARRAY, // todo, refactor to object
+            types: EMPTY_OBJECT,
+            extraLikeTypes: EMPTY_OBJECT,
+            deferred: EMPTY_OBJECT,
+            references: EMPTY_ARRAY,
+            sqlConversion: EMPTY_ARRAY,
         };
     }
     getData() {
         return this.data;
     }
+    // the shared empty containers are frozen, the methods below replace them on first write
+    setVariable(name, identifier) {
+        if (this.data.vars === EMPTY_OBJECT) {
+            this.data.vars = {};
+        }
+        this.data.vars[name] = identifier;
+    }
+    setType(name, type) {
+        if (this.data.types === EMPTY_OBJECT) {
+            this.data.types = {};
+        }
+        this.data.types[name] = type;
+    }
+    setExtraLikeType(name, type) {
+        if (this.data.extraLikeTypes === EMPTY_OBJECT) {
+            this.data.extraLikeTypes = {};
+        }
+        this.data.extraLikeTypes[name] = type;
+    }
+    setDeferred(name, deferred) {
+        if (this.data.deferred === EMPTY_OBJECT) {
+            this.data.deferred = {};
+        }
+        this.data.deferred[name] = deferred;
+    }
+    setClassDefinition(name, cdef) {
+        if (this.data.cdefs === EMPTY_OBJECT) {
+            this.data.cdefs = {};
+        }
+        this.data.cdefs[name] = cdef;
+    }
+    setInterfaceDefinition(name, idef) {
+        if (this.data.idefs === EMPTY_OBJECT) {
+            this.data.idefs = {};
+        }
+        this.data.idefs[name] = idef;
+    }
+    addFormDefinitions(forms) {
+        if (this.data.forms === EMPTY_ARRAY) {
+            this.data.forms = [];
+        }
+        this.data.forms.push(...forms);
+    }
+    addReference(reference) {
+        if (this.data.references === EMPTY_ARRAY) {
+            this.data.references = [];
+        }
+        this.data.references.push(reference);
+    }
+    addSQLConversion(conversion) {
+        if (this.data.sqlConversion === EMPTY_ARRAY) {
+            this.data.sqlConversion = [];
+        }
+        this.data.sqlConversion.push(conversion);
+    }
 }
 class SpaghettiScopeNode extends ScopeData {
     [Symbol.for("debug.description")]() {
@@ -44963,8 +45146,16 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@ab
 const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
 const message_source_1 = __webpack_require__(/*! ../expressions/message_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/message_source.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_object_oriented.js");
 class Message {
     runSyntax(node, input) {
+        const exceptionName = node.findDirectExpression(Expressions.ExceptionName);
+        const classicMessage = exceptionName ? new _object_oriented_1.ObjectOriented(input.scope).checkClassicException() : undefined;
+        if (exceptionName && classicMessage) {
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, exceptionName.getFirstToken(), classicMessage));
+            return;
+        }
         const found = node.findExpressionAfterToken("INTO");
         const inline = found === null || found === void 0 ? void 0 : found.findDirectExpression(Expressions.InlineData);
         if (inline) {
@@ -45230,8 +45421,10 @@ const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_module
 const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
 const inline_data_1 = __webpack_require__(/*! ../expressions/inline_data */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/inline_data.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const _eml_1 = __webpack_require__(/*! ../_eml */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_eml.js");
 class ModifyEntities {
     runSyntax(node, input) {
+        _eml_1.EML.checkEntities(node, input);
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             source_1.Source.runSyntax(s, input);
         }
@@ -46207,6 +46400,12 @@ class Raise {
         var _a, _b, _c, _d, _e, _f;
         const helper = new _object_oriented_1.ObjectOriented(input.scope);
         let method;
+        const exceptionName = node.findDirectExpression(Expressions.ExceptionName);
+        const classicMessage = exceptionName ? helper.checkClassicException() : undefined;
+        if (exceptionName && classicMessage) {
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, exceptionName.getFirstToken(), classicMessage));
+            return;
+        }
         const classTok = (_a = node.findDirectExpression(Expressions.ClassName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
         const className = classTok === null || classTok === void 0 ? void 0 : classTok.getStr();
         if (classTok && className) {
@@ -46541,8 +46740,10 @@ const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_module
 const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
 const inline_data_1 = __webpack_require__(/*! ../expressions/inline_data */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/inline_data.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const _eml_1 = __webpack_require__(/*! ../_eml */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_eml.js");
 class ReadEntities {
     runSyntax(node, input) {
+        _eml_1.EML.checkEntities(node, input);
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             source_1.Source.runSyntax(s, input);
         }
@@ -51695,7 +51896,6 @@ const _abstract_node_1 = __webpack_require__(/*! ./_abstract_node */ "./node_mod
 class ExpressionNode extends _abstract_node_1.AbstractNode {
     constructor(expression) {
         super();
-        this.tokenCount = 0;
         this.expression = expression;
     }
     addChild(_n) {
@@ -51706,13 +51906,15 @@ class ExpressionNode extends _abstract_node_1.AbstractNode {
     }
     setChildren(children) {
         super.setChildren(children);
-        for (const child of this.getChildren()) {
-            this.tokenCount = this.tokenCount + child.countTokens();
-        }
         return this;
     }
+    // not stored on the node, saves a field per ExpressionNode, the statement parser tracks counts itself
     countTokens() {
-        return this.tokenCount;
+        let count = 0;
+        for (const child of this.getChildren()) {
+            count = count + child.countTokens();
+        }
+        return count;
     }
     getFirstToken() {
         for (const child of this.getChildren()) {
@@ -56644,6 +56846,874 @@ exports.ArtifactsRules = ArtifactsRules;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/bdef/bdef_parser.js"
+/*!*******************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/bdef_parser.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.BDEFParser = 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_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 Expressions = __importStar(__webpack_require__(/*! ./expressions */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js"));
+class BDEFParser {
+    parse(file) {
+        if (file === undefined) {
+            return undefined;
+        }
+        // BDEF has the same comments, strings and delimiters as CDS
+        const tokens = this.removePragmas(cds_lexer_1.CDSLexer.run(file).filter(t => !(t instanceof tokens_1.Comment)));
+        const res = combi_1.Combi.run(new Expressions.BDEFDefinition(), tokens, version_1.defaultRelease);
+        if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
+            return undefined;
+        }
+        return res[0];
+    }
+    /** pragmas can occur in many places, eg. "##NOT_ASSIGNED_TO_DETACT" or "##UNMAPPED_FIELD[SingletonID]" */
+    removePragmas(tokens) {
+        var _a;
+        const ret = [];
+        for (let i = 0; i < tokens.length; i++) {
+            if (tokens[i].getStr().startsWith("##") === false) {
+                ret.push(tokens[i]);
+                continue;
+            }
+            if (((_a = tokens[i + 1]) === null || _a === void 0 ? void 0 : _a.getStr()) === "[") {
+                while (i < tokens.length - 1 && tokens[i].getStr() !== "]") {
+                    i++;
+                }
+            }
+        }
+        return ret;
+    }
+}
+exports.BDEFParser = BDEFParser;
+//# sourceMappingURL=bdef_parser.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_action.js"
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_action.js ***!
+  \*******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFAction = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** actions and functions, eg. "static factory action ( features : global ) copy parameter ZPARAM result [1] $self;" */
+class BDEFAction extends combi_1.Expression {
+    getRunnable() {
+        const modifier = (0, combi_1.altPrio)("INTERNAL", "STATIC", "FACTORY", "DEFAULT", "DRAFT", "REPEATABLE", "RECOMMENDATION", (0, combi_1.seq)("SAVE", (0, combi_1.optPrio)(_1.BDEFOptions)), (0, combi_1.seq)("KEY", _1.BDEFOptions));
+        const tail = (0, combi_1.altPrio)((0, combi_1.seq)("EXTERNAL", _1.BDEFString), (0, combi_1.seq)("AS", _1.BDEFName), _1.BDEFParameter, _1.BDEFResult, _1.BDEFCardinality, "OPTIMIZED", "EXTENSIBLE", "WITH ADDITIONAL IMPLEMENTATION");
+        return (0, combi_1.seq)((0, combi_1.starPrio)(modifier), (0, combi_1.altPrio)("ACTION", "FUNCTION"), (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFName, (0, combi_1.starPrio)(tail), (0, combi_1.altPrio)(";", _1.BDEFDefaultFunction));
+    }
+}
+exports.BDEFAction = BDEFAction;
+//# sourceMappingURL=bdef_action.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_alias.js"
+/*!******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_alias.js ***!
+  \******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFAlias = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+class BDEFAlias extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)("ALIAS", _1.BDEFName);
+    }
+}
+exports.BDEFAlias = BDEFAlias;
+//# sourceMappingURL=bdef_alias.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_association.js"
+/*!************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_association.js ***!
+  \************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFAssociation = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "association _Booking { create; with draft; }" */
+class BDEFAssociation extends combi_1.Expression {
+    getRunnable() {
+        // cross BO association, eg. "using /DMO/R_TRAVELTP_XBO \_Agency"
+        const using = (0, combi_1.seq)("USING", _1.BDEFName, (0, combi_1.optPrio)((0, combi_1.regex)(/^\\\w+$/)));
+        return (0, combi_1.seq)((0, combi_1.starPrio)((0, combi_1.altPrio)("INTERNAL", "UNMANAGED", "ANCESTOR")), "ASSOCIATION", (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFName, (0, combi_1.optPrio)(using), (0, combi_1.optPrio)((0, combi_1.seq)("ABBREVIATION", _1.BDEFName)), (0, combi_1.optPrio)("WITH HIERARCHY"), (0, combi_1.altPrio)(";", _1.BDEFAssociationBody));
+    }
+}
+exports.BDEFAssociation = BDEFAssociation;
+//# sourceMappingURL=bdef_association.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_association_body.js"
+/*!*****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_association_body.js ***!
+  \*****************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFAssociationBody = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "{ create ( features : instance ); with draft; link action linkManager; }" */
+class BDEFAssociationBody extends combi_1.Expression {
+    getRunnable() {
+        const create = (0, combi_1.seq)((0, combi_1.optPrio)("INTERNAL"), "CREATE", (0, combi_1.optPrio)(_1.BDEFOptions), (0, combi_1.altPrio)(";", _1.BDEFDefaultFunction));
+        const withh = (0, combi_1.seq)("WITH", (0, combi_1.altPrio)("DRAFT", "DEPENDENT DRAFT", "CASCADING DELETE"), ";");
+        const link = (0, combi_1.seq)((0, combi_1.altPrio)("LINK", "UNLINK"), "ACTION", _1.BDEFName, ";");
+        const inverse = (0, combi_1.seq)("INVERSE FUNCTION", _1.BDEFName, ";");
+        const item = (0, combi_1.altPrio)(create, withh, link, inverse, ";");
+        return (0, combi_1.seq)("{", (0, combi_1.starPrio)(item), "}");
+    }
+}
+exports.BDEFAssociationBody = BDEFAssociationBody;
+//# sourceMappingURL=bdef_association_body.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_authorization_context.js"
+/*!**********************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_authorization_context.js ***!
+  \**********************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFAuthorizationContext = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "define own authorization context { 'S_TABU_NAM'; }" or "define own authorization context by privileged mode;" */
+class BDEFAuthorizationContext extends combi_1.Expression {
+    getRunnable() {
+        const item = (0, combi_1.seq)((0, combi_1.altPrio)(_1.BDEFString, _1.BDEFName), ";");
+        const body = (0, combi_1.seq)("{", (0, combi_1.starPrio)(item), "}", (0, combi_1.optPrio)(";"));
+        const privileged = (0, combi_1.seq)("BY PRIVILEGED MODE", (0, combi_1.optPrio)("AND"));
+        return (0, combi_1.seq)("DEFINE", (0, combi_1.optPrio)("OWN"), "AUTHORIZATION CONTEXT", (0, combi_1.optPrio)((0, combi_1.altPrio)(privileged, _1.BDEFName)), (0, combi_1.optPrio)((0, combi_1.seq)("FOR DISABLE", _1.BDEFOptions)), (0, combi_1.altPrio)(";", body));
+    }
+}
+exports.BDEFAuthorizationContext = BDEFAuthorizationContext;
+//# sourceMappingURL=bdef_authorization_context.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_cardinality.js"
+/*!************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_cardinality.js ***!
+  \************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFCardinality = void 0;
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "[1]" or "[0..*]" */
+class BDEFCardinality extends combi_1.Expression {
+    getRunnable() {
+        const integer = (0, combi_1.regex)(/^\d+$/);
+        return (0, combi_1.seq)("[", integer, (0, combi_1.optPrio)((0, combi_1.seq)(".", ".", (0, combi_1.altPrio)(integer, "*"))), "]");
+    }
+}
+exports.BDEFCardinality = BDEFCardinality;
+//# sourceMappingURL=bdef_cardinality.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_default_function.js"
+/*!*****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_default_function.js ***!
+  \*****************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFDefaultFunction = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "{ default function GetDefaultsForCreate; }" */
+class BDEFDefaultFunction extends combi_1.Expression {
+    getRunnable() {
+        const func = (0, combi_1.seq)("DEFAULT FUNCTION", (0, combi_1.optPrio)((0, combi_1.seq)(_1.BDEFName, (0, combi_1.optPrio)((0, combi_1.seq)("EXTERNAL", _1.BDEFString)))), ";");
+        return (0, combi_1.seq)("{", (0, combi_1.plusPrio)(func), "}");
+    }
+}
+exports.BDEFDefaultFunction = BDEFDefaultFunction;
+//# sourceMappingURL=bdef_default_function.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_define_behavior.js"
+/*!****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_define_behavior.js ***!
+  \****************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFDefineBehavior = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** behavior of one entity, "define behavior for ZI_Travel alias Travel ... { ... }" */
+class BDEFDefineBehavior extends combi_1.Expression {
+    getRunnable() {
+        const fieldMapping = (0, combi_1.seq)(_1.BDEFName, "=", _1.BDEFName);
+        const lockMapping = (0, combi_1.seq)("(", fieldMapping, (0, combi_1.starPrio)((0, combi_1.seq)(",", fieldMapping)), ")");
+        const dependentBy = (0, combi_1.seq)("DEPENDENT", (0, combi_1.optPrio)((0, combi_1.seq)("BY", _1.BDEFName)));
+        const save = (0, combi_1.seq)((0, combi_1.altPrio)("WITH ADDITIONAL SAVE", "WITH UNMANAGED SAVE"), (0, combi_1.optPrio)("AND CLEANUP"), (0, combi_1.optPrio)("WITH FULL DATA"));
+        const option = (0, combi_1.altPrio)(_1.BDEFAlias, _1.BDEFImplementationIn, (0, combi_1.seq)("EXTERNAL", _1.BDEFString), (0, combi_1.seq)("PERSISTENT TABLE", _1.BDEFName), (0, combi_1.seq)("DRAFT TABLE", _1.BDEFName), (0, combi_1.seq)("QUERY", _1.BDEFName), (0, combi_1.seq)("LOCK", (0, combi_1.altPrio)((0, combi_1.seq)("MASTER", (0, combi_1.optPrio)("UNMANAGED")), (0, combi_1.seq)(dependentBy, (0, combi_1.optPrio)(lockMapping)))), (0, combi_1.seq)("AUTHORIZATION", (0, combi_1.altPrio)((0, combi_1.seq)("MASTER", _1.BDEFOptions), dependentBy)), (0, combi_1.seq)((0, combi_1.optPrio)("TOTAL"), "ETAG", (0, combi_1.altPrio)((0, combi_1.seq)("MASTER", _1.BDEFName), dependentBy, _1.BDEFName)), 
+        // eg. "( lock, authorization, etag ) dependent by _Travel"
+        (0, combi_1.seq)(_1.BDEFOptions, dependentBy), (0, combi_1.seq)((0, combi_1.altPrio)("LATE", "EARLY"), "NUMBERING", (0, combi_1.optPrio)("IN PLACE")), save, "WITH CONTROL", "USE ETAG", (0, combi_1.seq)("USING", _1.BDEFName), (0, combi_1.seq)("CHANGEDOCUMENTS", (0, combi_1.altPrio)((0, combi_1.seq)("MASTER", (0, combi_1.optPrio)("UTC"), (0, combi_1.optPrio)(_1.BDEFOptions)), dependentBy)), "EXTENSIBLE");
+        const item = (0, combi_1.altPrio)(_1.BDEFUse, _1.BDEFDetermineAction, _1.BDEFOperation, _1.BDEFAction, _1.BDEFDetermination, _1.BDEFField, _1.BDEFAssociation, _1.BDEFMapping, _1.BDEFSideEffects, _1.BDEFEvent, _1.BDEFHierarchy, _1.BDEFKey, (0, combi_1.seq)("CHANGEDOCUMENTS", _1.BDEFOptions, ";"), ";");
+        return (0, combi_1.seq)((0, combi_1.altPrio)((0, combi_1.seq)("DEFINE", (0, combi_1.optPrio)("ROOT"), "BEHAVIOR FOR"), "EXTEND BEHAVIOR FOR"), _1.BDEFName, (0, combi_1.starPrio)(option), "{", (0, combi_1.starPrio)(item), "}", (0, combi_1.optPrio)(";"));
+    }
+}
+exports.BDEFDefineBehavior = BDEFDefineBehavior;
+//# sourceMappingURL=bdef_define_behavior.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_definition.js"
+/*!***********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_definition.js ***!
+  \***********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFDefinition = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+class BDEFDefinition extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)(_1.BDEFImplementationType, (0, combi_1.starPrio)((0, combi_1.altPrio)(_1.BDEFHeader, _1.BDEFDefineBehavior, _1.BDEFAuthorizationContext)));
+    }
+}
+exports.BDEFDefinition = BDEFDefinition;
+//# sourceMappingURL=bdef_definition.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_determination.js"
+/*!**************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_determination.js ***!
+  \**************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFDetermination = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** determinations and validations, eg. "validation validateDates on save { create; field BeginDate, EndDate; }" */
+class BDEFDetermination extends combi_1.Expression {
+    getRunnable() {
+        const fields = (0, combi_1.seq)(_1.BDEFFieldPath, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.BDEFFieldPath)));
+        const trigger = (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.altPrio)("CREATE", "UPDATE", "DELETE"), ";"), (0, combi_1.seq)("FIELD", fields, ";"), (0, combi_1.seq)("ENTITY", fields, ";"), ";");
+        return (0, combi_1.seq)((0, combi_1.altPrio)("DETERMINATION", "VALIDATION"), (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFName, "ON", (0, combi_1.altPrio)("MODIFY", "SAVE"), "{", (0, combi_1.starPrio)(trigger), "}");
+    }
+}
+exports.BDEFDetermination = BDEFDetermination;
+//# sourceMappingURL=bdef_determination.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_determine_action.js"
+/*!*****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_determine_action.js ***!
+  \*****************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFDetermineAction = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "draft determine action Prepare { validation validateDate; determination ( always ) Child ~ setStatus; }" */
+class BDEFDetermineAction extends combi_1.Expression {
+    getRunnable() {
+        const item = (0, combi_1.seq)((0, combi_1.altPrio)("VALIDATION", "DETERMINATION"), (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFReference, ";");
+        const body = (0, combi_1.seq)("{", (0, combi_1.starPrio)(item), "}");
+        return (0, combi_1.seq)((0, combi_1.optPrio)("EXTEND"), (0, combi_1.optPrio)("DRAFT"), "DETERMINE ACTION", (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFName, (0, combi_1.optPrio)("EXTENSIBLE"), (0, combi_1.altPrio)(";", body));
+    }
+}
+exports.BDEFDetermineAction = BDEFDetermineAction;
+//# sourceMappingURL=bdef_determine_action.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_event.js"
+/*!******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_event.js ***!
+  \******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFEvent = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "event TravelCreated parameter ZD_TRAVEL_EVENT;" or "event StatusUpdated for side effects;" */
+class BDEFEvent extends combi_1.Expression {
+    getRunnable() {
+        const tail = (0, combi_1.altPrio)((0, combi_1.seq)("ON", _1.BDEFName), _1.BDEFParameter, "FOR SIDE EFFECTS", (0, combi_1.seq)("DESCRIPTION", _1.BDEFString), (0, combi_1.seq)("CATEGORY", _1.BDEFName));
+        return (0, combi_1.seq)((0, combi_1.optPrio)("MANAGED"), "EVENT", _1.BDEFName, (0, combi_1.starPrio)(tail), ";");
+    }
+}
+exports.BDEFEvent = BDEFEvent;
+//# sourceMappingURL=bdef_event.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_field.js"
+/*!******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_field.js ***!
+  \******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFField = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** field characteristics, eg. "field ( readonly, numbering : managed ) ID, CreatedBy;" */
+class BDEFField extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)("FIELD", _1.BDEFOptions, _1.BDEFFieldPath, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.BDEFFieldPath)), (0, combi_1.optPrio)(","), ";");
+    }
+}
+exports.BDEFField = BDEFField;
+//# sourceMappingURL=bdef_field.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_field_path.js"
+/*!***********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_field_path.js ***!
+  \***********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFFieldPath = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** field or association, possibly via associations, eg. "_Root.TotalPieces" */
+class BDEFFieldPath extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)(_1.BDEFName, (0, combi_1.starPrio)((0, combi_1.seq)(".", _1.BDEFName)));
+    }
+}
+exports.BDEFFieldPath = BDEFFieldPath;
+//# sourceMappingURL=bdef_field_path.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_header.js"
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_header.js ***!
+  \*******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFHeader = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** header statements following the implementation type, eg. "strict ( 2 );" or "with draft;" */
+class BDEFHeader extends combi_1.Expression {
+    getRunnable() {
+        const extensibleItem = (0, combi_1.seq)("WITH", (0, combi_1.plusPrio)((0, combi_1.regex)(/^\w+$/)), ";");
+        const extensible = (0, combi_1.seq)("EXTENSIBLE", (0, combi_1.optPrio)((0, combi_1.seq)("{", (0, combi_1.starPrio)(extensibleItem), "}")), (0, combi_1.optPrio)(";"));
+        return (0, combi_1.altPrio)((0, combi_1.seq)("STRICT", (0, combi_1.optPrio)((0, combi_1.seq)("(", (0, combi_1.regex)(/^\d+$/), ")")), ";"), (0, combi_1.seq)("WITH DRAFT", ";"), (0, combi_1.seq)("USE DRAFT", (0, combi_1.optPrio)("AS DEPENDENT"), ";"), (0, combi_1.seq)((0, combi_1.altPrio)("WITH", "USE"), "COLLABORATIVE DRAFT", ";"), (0, combi_1.seq)("WITH PRIVILEGED MODE", (0, combi_1.optPrio)((0, combi_1.seq)("DISABLING", _1.BDEFName)), ";"), (0, combi_1.seq)("WITH MANAGED INSTANCE FILTER", ";"), (0, combi_1.seq)("WITH HIERARCHY", ";"), (0, combi_1.seq)("WITH CROSS ASSOCIATIONS", ";"), (0, combi_1.seq)("USE SIDE EFFECTS", ";"), (0, combi_1.seq)("USE RECOMMENDATIONS", ";"), (0, combi_1.seq)("FOREIGN ENTITY", _1.BDEFName, (0, combi_1.optPrio)(_1.BDEFAlias), ";"), (0, combi_1.seq)("SCALAR ENTITY", _1.BDEFName, "FIELD", _1.BDEFName, ";"), (0, combi_1.seq)("AUXILIARY CLASS", _1.BDEFName, ";"), extensible);
+    }
+}
+exports.BDEFHeader = BDEFHeader;
+//# sourceMappingURL=bdef_header.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_hierarchy.js"
+/*!**********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_hierarchy.js ***!
+  \**********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFHierarchy = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "instance hierarchy /DMO/I_EmployeeHN { managed reorder action changeNextSibling; ascending association _Manager; }" */
+class BDEFHierarchy extends combi_1.Expression {
+    getRunnable() {
+        const action = (0, combi_1.seq)((0, combi_1.optPrio)("MANAGED"), (0, combi_1.altPrio)("REORDER", "COPY"), "ACTION", _1.BDEFName, ";");
+        const association = (0, combi_1.seq)((0, combi_1.altPrio)("ASCENDING", "DESCENDING"), "ASSOCIATION", _1.BDEFName, (0, combi_1.altPrio)(";", _1.BDEFAssociationBody));
+        const item = (0, combi_1.altPrio)(action, association, _1.BDEFField, ";");
+        return (0, combi_1.seq)("INSTANCE HIERARCHY", _1.BDEFName, "{", (0, combi_1.starPrio)(item), "}");
+    }
+}
+exports.BDEFHierarchy = BDEFHierarchy;
+//# sourceMappingURL=bdef_hierarchy.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_implementation_in.js"
+/*!******************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_implementation_in.js ***!
+  \******************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFImplementationIn = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+class BDEFImplementationIn extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)("IMPLEMENTATION IN CLASS", _1.BDEFName, (0, combi_1.optPrio)("UNIQUE"));
+    }
+}
+exports.BDEFImplementationIn = BDEFImplementationIn;
+//# sourceMappingURL=bdef_implementation_in.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_implementation_type.js"
+/*!********************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_implementation_type.js ***!
+  \********************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFImplementationType = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** first statement, eg. "managed with additional save implementation in class zbp_travel unique;" */
+class BDEFImplementationType extends combi_1.Expression {
+    getRunnable() {
+        const extension = (0, combi_1.seq)("EXTENSION", (0, combi_1.optPrio)((0, combi_1.altPrio)((0, combi_1.seq)("USING INTERFACE", _1.BDEFName), (0, combi_1.seq)("FOR", (0, combi_1.altPrio)("PROJECTION", "INTERFACE")))));
+        const type = (0, combi_1.altPrio)("MANAGED", "UNMANAGED", "ABSTRACT", "PROJECTION", "INTERFACE", extension, 
+        // old syntax, eg. "implementation unmanaged;"
+        (0, combi_1.seq)("IMPLEMENTATION", (0, combi_1.altPrio)("MANAGED", "UNMANAGED", "ABSTRACT", "PROJECTION")));
+        const save = (0, combi_1.seq)((0, combi_1.altPrio)("WITH ADDITIONAL SAVE", "WITH UNMANAGED SAVE"), (0, combi_1.optPrio)("AND CLEANUP"), (0, combi_1.optPrio)("WITH FULL DATA"));
+        const option = (0, combi_1.altPrio)(_1.BDEFImplementationIn, save);
+        return (0, combi_1.seq)(type, (0, combi_1.starPrio)(option), ";");
+    }
+}
+exports.BDEFImplementationType = BDEFImplementationType;
+//# sourceMappingURL=bdef_implementation_type.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_key.js"
+/*!****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_key.js ***!
+  \****************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFKey = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** alternative key, eg. "define pure key AlternativeKey { id; }" */
+class BDEFKey extends combi_1.Expression {
+    getRunnable() {
+        const fields = (0, combi_1.seq)(_1.BDEFName, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.BDEFName)), ";");
+        return (0, combi_1.seq)("DEFINE", (0, combi_1.optPrio)("PURE"), "KEY", _1.BDEFName, "{", (0, combi_1.starPrio)(fields), "}");
+    }
+}
+exports.BDEFKey = BDEFKey;
+//# sourceMappingURL=bdef_key.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_mapping.js"
+/*!********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_mapping.js ***!
+  \********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFMapping = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "mapping for ztravel corresponding { TravelID = travel_id; }" */
+class BDEFMapping extends combi_1.Expression {
+    getRunnable() {
+        const names = (0, combi_1.seq)(_1.BDEFName, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.BDEFName)));
+        const line = (0, combi_1.seq)(_1.BDEFName, "=", _1.BDEFName, (0, combi_1.optPrio)((0, combi_1.seq)("CONTROL", _1.BDEFName)), ";");
+        // deep mapping of a sub node, eg. "sub _Items = items;"
+        const sub = (0, combi_1.seq)("SUB", _1.BDEFName, "=", _1.BDEFName, ";");
+        const body = (0, combi_1.seq)("{", (0, combi_1.starPrio)((0, combi_1.altPrio)(sub, line, ";")), "}");
+        const option = (0, combi_1.altPrio)("CORRESPONDING", "EXTENSIBLE", (0, combi_1.seq)("EXCEPT", names), (0, combi_1.seq)("CONTROL", _1.BDEFName));
+        return (0, combi_1.seq)((0, combi_1.optPrio)("DEEP"), "MAPPING FOR", _1.BDEFName, (0, combi_1.starPrio)(option), (0, combi_1.altPrio)(";", body));
+    }
+}
+exports.BDEFMapping = BDEFMapping;
+//# sourceMappingURL=bdef_mapping.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_name.js"
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_name.js ***!
+  \*****************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFName = void 0;
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+class BDEFName extends combi_1.Expression {
+    getRunnable() {
+        const word = (0, combi_1.regex)(/^\$?\w+$/);
+        const slashName = (0, combi_1.seq)("/", (0, combi_1.regex)(/^\w+$/), "/", (0, combi_1.regex)(/^\w+$/));
+        return (0, combi_1.altPrio)(slashName, word);
+    }
+}
+exports.BDEFName = BDEFName;
+//# sourceMappingURL=bdef_name.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_operation.js"
+/*!**********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_operation.js ***!
+  \**********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFOperation = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** standard operations, eg. "internal create ( features : global );" */
+class BDEFOperation extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)((0, combi_1.starPrio)((0, combi_1.altPrio)("INTERNAL", "STATIC")), (0, combi_1.altPrio)("CREATE", "UPDATE", "DELETE"), (0, combi_1.optPrio)(_1.BDEFOptions), (0, combi_1.altPrio)(";", _1.BDEFDefaultFunction));
+    }
+}
+exports.BDEFOperation = BDEFOperation;
+//# sourceMappingURL=bdef_operation.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_options.js"
+/*!********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_options.js ***!
+  \********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFOptions = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "( features : instance, precheck )", "( read only )" or "( hierarchy-index )" */
+class BDEFOptions extends combi_1.Expression {
+    getRunnable() {
+        const name = (0, combi_1.seq)(_1.BDEFName, (0, combi_1.starPrio)((0, combi_1.altPrio)((0, combi_1.seq)("-", _1.BDEFName), _1.BDEFName)));
+        const option = (0, combi_1.seq)(name, (0, combi_1.optPrio)((0, combi_1.seq)(":", _1.BDEFName)));
+        return (0, combi_1.seq)("(", option, (0, combi_1.starPrio)((0, combi_1.seq)(",", option)), ")");
+    }
+}
+exports.BDEFOptions = BDEFOptions;
+//# sourceMappingURL=bdef_options.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_parameter.js"
+/*!**********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_parameter.js ***!
+  \**********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFParameter = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "deep parameter ZD_PARAMETER" */
+class BDEFParameter extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)((0, combi_1.optPrio)("DEEP"), "PARAMETER", _1.BDEFName);
+    }
+}
+exports.BDEFParameter = BDEFParameter;
+//# sourceMappingURL=bdef_parameter.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_reference.js"
+/*!**********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_reference.js ***!
+  \**********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFReference = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** reference to a determination, validation or action, optionally prefixed with an entity, eg. "Child ~ validatePercentage" */
+class BDEFReference extends combi_1.Expression {
+    getRunnable() {
+        const namespace = (0, combi_1.seq)("/", (0, combi_1.regex)(/^\w+$/), "/");
+        const tildeEnd = (0, combi_1.regex)(/^\w+~$/);
+        return (0, combi_1.altPrio)((0, combi_1.seq)(_1.BDEFName, "~", _1.BDEFName), (0, combi_1.seq)(_1.BDEFName, (0, combi_1.regex)(/^~\w+$/)), (0, combi_1.seq)((0, combi_1.optPrio)(namespace), tildeEnd, _1.BDEFName), (0, combi_1.seq)((0, combi_1.optPrio)(namespace), (0, combi_1.regex)(/^\w+~\w+$/)), _1.BDEFName);
+    }
+}
+exports.BDEFReference = BDEFReference;
+//# sourceMappingURL=bdef_reference.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_result.js"
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_result.js ***!
+  \*******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFResult = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "result [1] $self", "deep result ZD_RESULT" or "result selective [0..*] entity ZI_Travel" */
+class BDEFResult extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)((0, combi_1.optPrio)("DEEP"), "RESULT", (0, combi_1.optPrio)("SELECTIVE"), (0, combi_1.optPrio)(_1.BDEFCardinality), (0, combi_1.optPrio)("ENTITY"), _1.BDEFName);
+    }
+}
+exports.BDEFResult = BDEFResult;
+//# sourceMappingURL=bdef_result.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_side_effects.js"
+/*!*************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_side_effects.js ***!
+  \*************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFSideEffects = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** eg. "side effects { field Price affects field TotalPrice, messages; }" */
+class BDEFSideEffects extends combi_1.Expression {
+    getRunnable() {
+        const fieldList = (0, combi_1.seq)("(", _1.BDEFName, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.BDEFName)), ")");
+        // eg. "field *", "field ( A, B )", "field _Item.*" or "field _Root.( A, B )"
+        const field = (0, combi_1.seq)("FIELD", (0, combi_1.altPrio)("*", fieldList, (0, combi_1.seq)(_1.BDEFName, (0, combi_1.starPrio)((0, combi_1.seq)(".", (0, combi_1.altPrio)("*", fieldList, _1.BDEFName))))));
+        const permission = (0, combi_1.altPrio)((0, combi_1.seq)("ACTION", _1.BDEFReference), field, (0, combi_1.seq)((0, combi_1.altPrio)("CREATE", "UPDATE", "DELETE"), (0, combi_1.optPrio)(_1.BDEFName)));
+        const permissions = (0, combi_1.seq)("PERMISSIONS", "(", permission, (0, combi_1.starPrio)((0, combi_1.seq)(",", permission)), ")");
+        const target = (0, combi_1.altPrio)(field, (0, combi_1.seq)("ENTITY", _1.BDEFFieldPath), "MESSAGES", permissions, "$SELF");
+        const targets = (0, combi_1.seq)(target, (0, combi_1.starPrio)((0, combi_1.seq)(",", target)));
+        const trigger = (0, combi_1.altPrio)(field, (0, combi_1.seq)("ENTITY", _1.BDEFFieldPath), "$SELF");
+        const triggers = (0, combi_1.seq)(trigger, (0, combi_1.starPrio)((0, combi_1.seq)(",", trigger)));
+        const source = (0, combi_1.altPrio)(field, (0, combi_1.seq)("ACTION", _1.BDEFReference), (0, combi_1.seq)("DETERMINE ACTION", _1.BDEFReference, "EXECUTED ON", triggers), (0, combi_1.seq)("ENTITY", _1.BDEFFieldPath), (0, combi_1.seq)("EVENT", _1.BDEFName), "$SELF");
+        const recommendation = (0, combi_1.seq)("RECOMMENDATION FUNCTION", _1.BDEFName, "EXECUTED ON", triggers, ";");
+        const item = (0, combi_1.altPrio)((0, combi_1.seq)(source, "AFFECTS", targets, ";"), recommendation, ";");
+        return (0, combi_1.seq)("SIDE EFFECTS", "{", (0, combi_1.starPrio)(item), "}");
+    }
+}
+exports.BDEFSideEffects = BDEFSideEffects;
+//# sourceMappingURL=bdef_side_effects.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_string.js"
+/*!*******************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_string.js ***!
+  \*******************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFString = void 0;
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+class BDEFString extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.regex)(/^'(?:[^']|'')*'$/);
+    }
+}
+exports.BDEFString = BDEFString;
+//# sourceMappingURL=bdef_string.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_use.js"
+/*!****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_use.js ***!
+  \****************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFUse = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+/** reuse in projections and interfaces, eg. "use action acceptTravel as accept;" */
+class BDEFUse extends combi_1.Expression {
+    getRunnable() {
+        const end = (0, combi_1.altPrio)(";", _1.BDEFDefaultFunction);
+        const operation = (0, combi_1.seq)((0, combi_1.altPrio)("CREATE", "UPDATE", "DELETE"), (0, combi_1.optPrio)(_1.BDEFOptions), end);
+        const actionTail = (0, combi_1.altPrio)((0, combi_1.seq)("AS", _1.BDEFName), (0, combi_1.seq)("EXTERNAL", _1.BDEFString), _1.BDEFParameter, _1.BDEFResult);
+        const action = (0, combi_1.seq)((0, combi_1.altPrio)("ACTION", "FUNCTION"), (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFName, (0, combi_1.starPrio)(actionTail), end);
+        const association = (0, combi_1.seq)("ASSOCIATION", (0, combi_1.optPrio)(_1.BDEFOptions), _1.BDEFName, (0, combi_1.optPrio)((0, combi_1.seq)("ABBREVIATION", _1.BDEFName)), (0, combi_1.altPrio)(";", _1.BDEFAssociationBody));
+        return (0, combi_1.seq)("USE", (0, combi_1.altPrio)(operation, action, association, (0, combi_1.seq)("ETAG", ";"), (0, combi_1.seq)("SIDE EFFECTS", ";"), (0, combi_1.seq)("KEY", _1.BDEFName, ";"), (0, combi_1.seq)("EVENT", _1.BDEFName, (0, combi_1.optPrio)(_1.BDEFParameter), ";")));
+    }
+}
+exports.BDEFUse = BDEFUse;
+//# sourceMappingURL=bdef_use.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js"
+/*!*************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/bdef/expressions/index.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 __exportStar = (this && this.__exportStar) || function(m, exports) {
+    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
+};
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+__exportStar(__webpack_require__(/*! ./bdef_action */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_action.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_alias */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_alias.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_association_body */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_association_body.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_association */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_association.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_authorization_context */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_authorization_context.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_cardinality */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_cardinality.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_default_function */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_default_function.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_define_behavior */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_define_behavior.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_definition */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_definition.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_determination */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_determination.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_determine_action */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_determine_action.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_event */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_event.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_field_path */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_field_path.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_field */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_field.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_header */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_header.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_hierarchy */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_hierarchy.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_implementation_in */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_implementation_in.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_implementation_type */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_implementation_type.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_key */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_key.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_mapping */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_mapping.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_name */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_name.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_operation */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_operation.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_options */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_options.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_parameter */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_parameter.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_reference */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_reference.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_result */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_result.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_side_effects */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_side_effects.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_string */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_string.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_use */ "./node_modules/@abaplint/core/build/src/bdef/expressions/bdef_use.js"), exports);
+//# sourceMappingURL=index.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/cds/cds_determine_types.js"
 /*!**************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/cds/cds_determine_types.js ***!
@@ -64286,6 +65356,8 @@ exports.BADIDefinition = BADIDefinition;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.BehaviorDefinition = void 0;
+const bdef_parser_1 = __webpack_require__(/*! ../bdef/bdef_parser */ "./node_modules/@abaplint/core/build/src/bdef/bdef_parser.js");
+const expressions_1 = __webpack_require__(/*! ../bdef/expressions */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
 const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
 class BehaviorDefinition extends _abstract_object_1.AbstractObject {
     constructor() {
@@ -64308,8 +65380,30 @@ class BehaviorDefinition extends _abstract_object_1.AbstractObject {
     findSourceFile() {
         return this.getFiles().find(f => f.getFilename().endsWith(".asbdef"));
     }
+    parse() {
+        if (this.isDirty() === false && this.parsedData !== undefined) {
+            return { updated: false, runtime: 0 };
+        }
+        const start = Date.now();
+        const tree = new bdef_parser_1.BDEFParser().parse(this.findSourceFile());
+        this.parsedData = {
+            entities: tree ? this.findEntities(tree) : this.findEntitiesFallback(),
+            tree,
+        };
+        this.dirty = false;
+        return { updated: true, runtime: Date.now() - start };
+    }
+    getTree() {
+        var _a;
+        this.parse();
+        return (_a = this.parsedData) === null || _a === void 0 ? void 0 : _a.tree;
+    }
+    hasParserError() {
+        return this.findSourceFile() !== undefined && this.getTree() === undefined;
+    }
     listEntities() {
-        return this.parseSource().entities;
+        this.parse();
+        return this.parsedData.entities;
     }
     /** finds the entity name for a given alias, undefined if the alias is not defined */
     findEntityNameByAlias(alias) {
@@ -64323,15 +65417,29 @@ class BehaviorDefinition extends _abstract_object_1.AbstractObject {
         return undefined;
     }
     /////////////////////////
-    parseSource() {
+    findEntities(tree) {
         var _a;
-        if (this.parsedData !== undefined) {
-            return this.parsedData;
+        const ret = [];
+        for (const define of tree.findDirectExpressions(expressions_1.BDEFDefineBehavior)) {
+            const name = define.findDirectExpression(expressions_1.BDEFName);
+            if (name === undefined) {
+                continue;
+            }
+            const alias = (_a = define.findDirectExpression(expressions_1.BDEFAlias)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(expressions_1.BDEFName);
+            ret.push({
+                name: name.concatTokens().replace(/ /g, ""),
+                alias: alias === null || alias === void 0 ? void 0 : alias.concatTokens().replace(/ /g, ""),
+            });
         }
-        this.parsedData = { entities: [] };
+        return ret;
+    }
+    /** used if the source cannot be parsed */
+    findEntitiesFallback() {
+        var _a;
+        const ret = [];
         const raw = (_a = this.findSourceFile()) === null || _a === void 0 ? void 0 : _a.getRaw();
         if (raw === undefined) {
-            return this.parsedData;
+            return ret;
         }
         // BDEF uses "//" for line comments
         const stripped = raw.replace(/\/\/.*$/gm, "");
@@ -64339,10 +65447,10 @@ class BehaviorDefinition extends _abstract_object_1.AbstractObject {
         const regex = /\bdefine\s+(?:\w+\s+)*?behavior\s+for\s+([\w/]+)(?:\s+alias\s+(\w+))?/gi;
         let match = regex.exec(stripped);
         while (match !== null) {
-            this.parsedData.entities.push({ name: match[1], alias: match[2] });
+            ret.push({ name: match[1], alias: match[2] });
             match = regex.exec(stripped);
         }
-        return this.parsedData;
+        return ret;
     }
 }
 exports.BehaviorDefinition = BehaviorDefinition;
@@ -72606,7 +73714,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.83";
+        return "2.120.86";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -75263,6 +76371,104 @@ exports.AbapdocParameterNames = AbapdocParameterNames;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/abapdoc_shorttext_length.js"
+/*!*********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/abapdoc_shorttext_length.js ***!
+  \*********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.AbapdocShorttextLength = exports.AbapdocShorttextLengthConf = void 0;
+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");
+class AbapdocShorttextLengthConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.AbapdocShorttextLengthConf = AbapdocShorttextLengthConf;
+const MAX_LENGTH = 60;
+class AbapdocShorttextLength extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new AbapdocShorttextLengthConf();
+    }
+    getMetadata() {
+        return {
+            key: "abapdoc_shorttext_length",
+            title: "ABAP Doc shorttext too long",
+            shortDescription: `Reports ABAP Doc shorttexts longer than 60 characters.`,
+            extendedInformation: `A shorttext is written as <p class="shorttext">...</p> or
+<p class="shorttext synchronized">...</p>, the system warns "Shorttext must not exceed 60 characters."
+if the text is longer.
+
+The text may span several "! lines, line breaks count as one blank. Leading and trailing blanks are not counted,
+an entity like &lt; counts as one character.`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `INTERFACE zif_foo PUBLIC.
+  "! <p class="shorttext synchronized">Returns the name of the selected row in the table of the view</p>
+  METHODS name RETURNING VALUE(result) TYPE string.
+ENDINTERFACE.`,
+            goodExample: `INTERFACE zif_foo PUBLIC.
+  "! <p class="shorttext synchronized">Returns the name of the selected row</p>
+  METHODS name RETURNING VALUE(result) TYPE string.
+ENDINTERFACE.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        let block = [];
+        for (const statement of file.getStatements()) {
+            const token = statement.getFirstToken();
+            const doc = statement.get() instanceof _statement_1.Comment && token.getStr().startsWith(`"!`);
+            const previous = block[block.length - 1];
+            if (doc === false || (previous !== undefined && token.getStart().getRow() !== previous.getStart().getRow() + 1)) {
+                issues.push(...this.checkBlock(file, block));
+                block = [];
+            }
+            if (doc === true) {
+                block.push(token);
+            }
+        }
+        issues.push(...this.checkBlock(file, block));
+        return issues;
+    }
+    checkBlock(file, block) {
+        const issues = [];
+        if (block.length === 0) {
+            return issues;
+        }
+        const text = block.map(t => t.getStr().substring(2)).join("\n");
+        const regex = /<p\s[^>]*class\s*=\s*"shorttext(?:\s+synchronized)?"[^>]*>([\s\S]*?)<\/p>/gi;
+        let match;
+        while ((match = regex.exec(text)) !== null) {
+            const length = this.decode(match[1].replace(/\s+/g, " ").trim()).length;
+            if (length <= MAX_LENGTH) {
+                continue;
+            }
+            const token = block[text.substring(0, match.index).split("\n").length - 1];
+            const message = `Shorttext must not exceed ${MAX_LENGTH} characters, found ${length}`;
+            issues.push(issue_1.Issue.atToken(file, token, message, this.getMetadata().key, this.conf.severity));
+        }
+        return issues;
+    }
+    decode(text) {
+        return text.replace(/&(#\d+|#x[0-9a-fA-F]+|[a-zA-Z]+);/g, "_");
+    }
+}
+exports.AbapdocShorttextLength = AbapdocShorttextLength;
+//# sourceMappingURL=abapdoc_shorttext_length.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/add_test_attributes.js"
 /*!****************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/add_test_attributes.js ***!
@@ -76716,8 +77922,28 @@ TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md
 
 BREAK points`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
-            badExample: `DESCRIBE TABLE items LINES count.`,
-            goodExample: `count = lines( items ).`,
+            badExample: `DEFINE _hello.
+  WRITE 'hello'.
+END-OF-DEFINITION.
+
+STATICS count TYPE i.
+
+TYPES ty_items TYPE STANDARD TABLE OF i WITH DEFAULT KEY.
+
+BREAK-POINT.
+
+TEST-SEAM seam_name.
+  count = 1.
+END-TEST-SEAM.
+
+DESCRIBE TABLE items LINES count.
+
+EXPORT data = items TO MEMORY ID 'ZFOO'.
+
+EXPORT data = items TO DATABASE indx(zz) ID 'ZFOO'.`,
+            goodExample: `count = lines( items ).
+
+TYPES ty_items TYPE STANDARD TABLE OF i WITH EMPTY KEY.`,
         };
     }
     getDescription(statement) {
@@ -76814,6 +78040,161 @@ exports.AvoidUse = AvoidUse;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/bdef_check_syntax.js"
+/*!**************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/bdef_check_syntax.js ***!
+  \**************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFCheckSyntax = exports.BDEFCheckSyntaxConf = void 0;
+const expressions_1 = __webpack_require__(/*! ../bdef/expressions */ "./node_modules/@abaplint/core/build/src/bdef/expressions/index.js");
+const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.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");
+class BDEFCheckSyntaxConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.BDEFCheckSyntaxConf = BDEFCheckSyntaxConf;
+class BDEFCheckSyntax {
+    constructor() {
+        this.conf = new BDEFCheckSyntaxConf();
+    }
+    getMetadata() {
+        return {
+            key: "bdef_check_syntax",
+            title: "BDEF Check Syntax",
+            shortDescription: "Checks behavior definition references to entities and implementation classes",
+            extendedInformation: `Performs semantic checks which require other objects to be available.
+
+Missing objects are only reported when their names match the configured errorNamespace.`,
+            tags: [_irule_1.RuleTag.Syntax],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(object) {
+        if (!(object instanceof objects_1.BehaviorDefinition)) {
+            return [];
+        }
+        const tree = object.getTree();
+        const file = object.findSourceFile();
+        if (tree === undefined || file === undefined) {
+            return [];
+        }
+        const issues = [];
+        this.checkImplementationClasses(tree, issues, file);
+        this.checkEntities(tree, issues, file);
+        return issues;
+    }
+    checkImplementationClasses(tree, issues, file) {
+        for (const implementation of tree.findAllExpressions(expressions_1.BDEFImplementationIn)) {
+            const nameNode = implementation.findDirectExpression(expressions_1.BDEFName);
+            if (nameNode === undefined) {
+                continue;
+            }
+            const name = this.name(nameNode);
+            if (this.reg.getObject("CLAS", name) === undefined && this.reg.inErrorNamespace(name)) {
+                issues.push(issue_1.Issue.atToken(file, nameNode.getFirstToken(), `Implementation class "${name}" not found`, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    checkEntities(tree, issues, file) {
+        const ddic = new ddic_1.DDIC(this.reg);
+        for (const define of tree.findDirectExpressions(expressions_1.BDEFDefineBehavior)) {
+            if (define.getFirstToken().getStr().toUpperCase() !== "DEFINE") {
+                // "extend behavior for" refers to entities of the extended BDEF
+                continue;
+            }
+            const nameNode = define.findDirectExpression(expressions_1.BDEFName);
+            if (nameNode === undefined) {
+                continue;
+            }
+            const name = this.name(nameNode);
+            if (this.reg.inErrorNamespace(name)
+                && this.reg.getObject("DDLS", name) === undefined
+                && ddic.lookupDDLS(name) === undefined) {
+                issues.push(issue_1.Issue.atToken(file, nameNode.getFirstToken(), `Entity "${name}" not found`, this.getMetadata().key, this.conf.severity));
+            }
+        }
+    }
+    name(node) {
+        return node.concatTokens().replace(/ /g, "");
+    }
+}
+exports.BDEFCheckSyntax = BDEFCheckSyntax;
+//# sourceMappingURL=bdef_check_syntax.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/rules/bdef_parser_error.js"
+/*!**************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/bdef_parser_error.js ***!
+  \**************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BDEFParserError = exports.BDEFParserErrorConf = void 0;
+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");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+class BDEFParserErrorConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.BDEFParserErrorConf = BDEFParserErrorConf;
+class BDEFParserError {
+    constructor() {
+        this.conf = new BDEFParserErrorConf();
+    }
+    getMetadata() {
+        return {
+            key: "bdef_parser_error",
+            title: "BDEF Parser Error",
+            shortDescription: `BDEF parsing`,
+            extendedInformation: `Parses behavior definitions and issues parser errors`,
+            tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.SingleFile],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(_reg) {
+        return this;
+    }
+    run(object) {
+        const issues = [];
+        if (object instanceof objects_1.BehaviorDefinition) {
+            const file = object.findSourceFile();
+            if (object.hasParserError() === true && file) {
+                const empty = file.getRaw().length === 0 ? ", empty file" : "";
+                const message = "BDEF Parser error" + empty;
+                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.getConfig().severity));
+            }
+        }
+        return issues;
+    }
+}
+exports.BDEFParserError = BDEFParserError;
+//# sourceMappingURL=bdef_parser_error.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/begin_end_names.js"
 /*!************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/begin_end_names.js ***!
@@ -91515,6 +92896,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_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);
 __exportStar(__webpack_require__(/*! ./alias_never_reassigned */ "./node_modules/@abaplint/core/build/src/rules/alias_never_reassigned.js"), exports);
@@ -91525,6 +92907,8 @@ __exportStar(__webpack_require__(/*! ./allowed_object_naming */ "./node_modules/
 __exportStar(__webpack_require__(/*! ./allowed_object_types */ "./node_modules/@abaplint/core/build/src/rules/allowed_object_types.js"), exports);
 __exportStar(__webpack_require__(/*! ./ambiguous_statement */ "./node_modules/@abaplint/core/build/src/rules/ambiguous_statement.js"), exports);
 __exportStar(__webpack_require__(/*! ./avoid_use */ "./node_modules/@abaplint/core/build/src/rules/avoid_use.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_check_syntax */ "./node_modules/@abaplint/core/build/src/rules/bdef_check_syntax.js"), exports);
+__exportStar(__webpack_require__(/*! ./bdef_parser_error */ "./node_modules/@abaplint/core/build/src/rules/bdef_parser_error.js"), exports);
 __exportStar(__webpack_require__(/*! ./begin_end_names */ "./node_modules/@abaplint/core/build/src/rules/begin_end_names.js"), exports);
 __exportStar(__webpack_require__(/*! ./begin_single_include */ "./node_modules/@abaplint/core/build/src/rules/begin_single_include.js"), exports);
 __exportStar(__webpack_require__(/*! ./boolean_flag_via_if */ "./node_modules/@abaplint/core/build/src/rules/boolean_flag_via_if.js"), exports);
@@ -91667,6 +93051,7 @@ __exportStar(__webpack_require__(/*! ./prefer_pragmas */ "./node_modules/@abapli
 __exportStar(__webpack_require__(/*! ./prefer_abap_bool */ "./node_modules/@abaplint/core/build/src/rules/prefer_abap_bool.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_abap_bool_values */ "./node_modules/@abaplint/core/build/src/rules/prefer_abap_bool_values.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_raise_exception_new */ "./node_modules/@abaplint/core/build/src/rules/prefer_raise_exception_new.js"), exports);
+__exportStar(__webpack_require__(/*! ./prefer_raise_message */ "./node_modules/@abaplint/core/build/src/rules/prefer_raise_message.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_string_template */ "./node_modules/@abaplint/core/build/src/rules/prefer_string_template.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_returning_to_exporting */ "./node_modules/@abaplint/core/build/src/rules/prefer_returning_to_exporting.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_xsdbool */ "./node_modules/@abaplint/core/build/src/rules/prefer_xsdbool.js"), exports);
@@ -100798,40 +102183,269 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.PreferIsNot = exports.PreferIsNotConf = void 0;
+exports.PreferIsNot = exports.PreferIsNotConf = void 0;
+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 Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+class PreferIsNotConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.PreferIsNotConf = PreferIsNotConf;
+class PreferIsNot extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new PreferIsNotConf();
+    }
+    getMetadata() {
+        return {
+            key: "prefer_is_not",
+            title: "Prefer IS NOT to NOT IS",
+            shortDescription: `Prefer IS NOT to NOT IS`,
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-is-not-to-not-is
+
+"if not is_valid( )." examples are skipped`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
+            goodExample: `IF variable IS NOT INITIAL.
+IF variable NP 'TODO*'.
+IF variable <> 42.
+IF variable CO 'hello'.`,
+            badExample: `IF NOT variable IS INITIAL.
+IF NOT variable CP 'TODO*'.
+IF NOT variable = 42.
+IF NOT variable CA 'hello'.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        for (const s of file.getStatements()) {
+            for (const c of s.findAllExpressions(Expressions.Compare)) {
+                if (c.concatTokens().toUpperCase().startsWith("NOT ") === false) {
+                    continue;
+                }
+                else if (c.getChildren().length === 2 && c.getChildren()[1].get() instanceof Expressions.MethodCallChain) {
+                    continue;
+                }
+                const message = "Prefer IS NOT to NOT IS";
+                const fix = this.getFix(file, c);
+                issues.push(issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.conf.severity, fix));
+            }
+        }
+        return issues;
+    }
+    getFix(file, c) {
+        let insertFix;
+        if (c.getChildren()[2].getFirstToken().getStr().toUpperCase() === "IS") {
+            const tokenPositionBeforeDelete = c.getChildren()[2].getLastToken().getEnd();
+            const tokenPosition = new position_1.Position(tokenPositionBeforeDelete.getRow(), tokenPositionBeforeDelete.getCol() + 1);
+            insertFix = edit_helper_1.EditHelper.insertAt(file, tokenPosition, "NOT ");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr().toUpperCase() === "IN" || c.getChildren()[2].getFirstToken().getStr().toUpperCase() === "BETWEEN") {
+            const tokenPositionBeforeDelete = c.getChildren()[1].getLastToken().getEnd();
+            const tokenPosition = new position_1.Position(tokenPositionBeforeDelete.getRow(), tokenPositionBeforeDelete.getCol() + 1);
+            insertFix = edit_helper_1.EditHelper.insertAt(file, tokenPosition, "NOT ");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr() === "=") {
+            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "<>");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr() === "<>") {
+            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "=");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr() === "<") {
+            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), ">");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr() === ">") {
+            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "<");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr() === "<=") {
+            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), ">=");
+        }
+        else if (c.getChildren()[2].getFirstToken().getStr() === ">=") {
+            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "<=");
+        }
+        else {
+            return;
+        }
+        const endCol = c.getChildren()[0].getFirstToken().getEnd().getCol() + 1;
+        const endPosition = new position_1.Position(c.getChildren()[0].getFirstToken().getEnd().getRow(), endCol);
+        const deleteFix = edit_helper_1.EditHelper.deleteRange(file, c.getChildren()[0].getFirstToken().getStart(), endPosition);
+        const finalFix = edit_helper_1.EditHelper.merge(insertFix, deleteFix);
+        return finalFix;
+    }
+}
+exports.PreferIsNot = PreferIsNot;
+//# sourceMappingURL=prefer_is_not.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/rules/prefer_pragmas.js"
+/*!***********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/prefer_pragmas.js ***!
+  \***********************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.PreferPragmas = exports.PreferPragmasConf = void 0;
+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");
+class PreferPragmasConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.check = [
+            {
+                pseudo: "#EC CI_SUBRC",
+                pragma: "SUBRC_OK",
+            },
+            {
+                pseudo: "#EC NEEDED",
+                pragma: "NEEDED",
+            },
+            {
+                pseudo: "#EC NOTEXT",
+                pragma: "NO_TEXT",
+            },
+            {
+                pseudo: "#EC NO_HANDLER",
+                pragma: "NO_HANDLER",
+            },
+        ];
+    }
+}
+exports.PreferPragmasConf = PreferPragmasConf;
+class PreferPragmas extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new PreferPragmasConf();
+    }
+    getMetadata() {
+        return {
+            key: "prefer_pragmas",
+            title: "prefer pragmas over pseudo comments ",
+            shortDescription: `prefer pragmas over pseudo comments`,
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-pragmas-to-pseudo-comments`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
+            badExample: `DATA foo1 TYPE i. "#EC NEEDED`,
+            goodExample: `DATA foo2 TYPE i ##NEEDED.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const config = this.getConfig();
+        for (const statement of file.getStatements()) {
+            if (!(statement.get() instanceof _statement_1.Comment)) {
+                continue;
+            }
+            const concat = statement.concatTokens().toUpperCase();
+            if (concat.includes("#EC") === false) {
+                continue;
+            }
+            for (const check of config.check) {
+                if (concat.includes(check.pseudo.toUpperCase())) {
+                    const message = `Prefer pragma ${check.pragma}`;
+                    issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.getConfig().severity));
+                }
+            }
+        }
+        return issues;
+    }
+}
+exports.PreferPragmas = PreferPragmas;
+//# sourceMappingURL=prefer_pragmas.js.map
+
+/***/ },
+
+/***/ "./node_modules/@abaplint/core/build/src/rules/prefer_raise_exception_new.js"
+/*!***********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/prefer_raise_exception_new.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.PreferRaiseExceptionNew = exports.PreferRaiseExceptionNewConf = void 0;
 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 Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
-const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
-class PreferIsNotConf extends _basic_rule_config_1.BasicRuleConfig {
+const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
+class PreferRaiseExceptionNewConf extends _basic_rule_config_1.BasicRuleConfig {
 }
-exports.PreferIsNotConf = PreferIsNotConf;
-class PreferIsNot extends _abap_rule_1.ABAPRule {
+exports.PreferRaiseExceptionNewConf = PreferRaiseExceptionNewConf;
+class PreferRaiseExceptionNew extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
-        this.conf = new PreferIsNotConf();
+        this.conf = new PreferRaiseExceptionNewConf();
     }
     getMetadata() {
         return {
-            key: "prefer_is_not",
-            title: "Prefer IS NOT to NOT IS",
-            shortDescription: `Prefer IS NOT to NOT IS`,
+            key: "prefer_raise_exception_new",
+            title: "Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE",
+            shortDescription: `Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE`,
             extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-is-not-to-not-is
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-raise-exception-new-to-raise-exception-type
 
-"if not is_valid( )." examples are skipped`,
-            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            goodExample: `IF variable IS NOT INITIAL.
-IF variable NP 'TODO*'.
-IF variable <> 42.
-IF variable CO 'hello'.`,
-            badExample: `IF NOT variable IS INITIAL.
-IF NOT variable CP 'TODO*'.
-IF NOT variable = 42.
-IF NOT variable CA 'hello'.`,
+From 752 and up`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Upport],
+            goodExample: `RAISE EXCEPTION NEW cx_generation_error( previous = exception ).`,
+            badExample: `RAISE EXCEPTION TYPE cx_generation_error
+  EXPORTING
+    previous = exception.`,
         };
     }
     getConfig() {
@@ -100841,158 +102455,42 @@ IF NOT variable CA 'hello'.`,
         this.conf = conf;
     }
     runParsed(file) {
+        if (!(0, version_1.releaseAtLeast)(this.reg.getConfig().getRelease(), version_1.Release.v752)) {
+            return [];
+        }
         const issues = [];
-        for (const s of file.getStatements()) {
-            for (const c of s.findAllExpressions(Expressions.Compare)) {
-                if (c.concatTokens().toUpperCase().startsWith("NOT ") === false) {
+        for (const statement of file.getStatements()) {
+            if (statement.get() instanceof Statements.Raise) {
+                const concat = statement.concatTokens().toUpperCase();
+                if (concat.includes(" MESSAGE")) {
                     continue;
                 }
-                else if (c.getChildren().length === 2 && c.getChildren()[1].get() instanceof Expressions.MethodCallChain) {
-                    continue;
+                if (concat.startsWith("RAISE EXCEPTION TYPE ")) {
+                    const message = "Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE";
+                    const fix = this.getFix(file, statement, concat.includes(" EXPORTING") ? true : false);
+                    issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, fix));
                 }
-                const message = "Prefer IS NOT to NOT IS";
-                const fix = this.getFix(file, c);
-                issues.push(issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.conf.severity, fix));
             }
         }
         return issues;
     }
-    getFix(file, c) {
-        let insertFix;
-        if (c.getChildren()[2].getFirstToken().getStr().toUpperCase() === "IS") {
-            const tokenPositionBeforeDelete = c.getChildren()[2].getLastToken().getEnd();
-            const tokenPosition = new position_1.Position(tokenPositionBeforeDelete.getRow(), tokenPositionBeforeDelete.getCol() + 1);
-            insertFix = edit_helper_1.EditHelper.insertAt(file, tokenPosition, "NOT ");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr().toUpperCase() === "IN" || c.getChildren()[2].getFirstToken().getStr().toUpperCase() === "BETWEEN") {
-            const tokenPositionBeforeDelete = c.getChildren()[1].getLastToken().getEnd();
-            const tokenPosition = new position_1.Position(tokenPositionBeforeDelete.getRow(), tokenPositionBeforeDelete.getCol() + 1);
-            insertFix = edit_helper_1.EditHelper.insertAt(file, tokenPosition, "NOT ");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr() === "=") {
-            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "<>");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr() === "<>") {
-            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "=");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr() === "<") {
-            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), ">");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr() === ">") {
-            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "<");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr() === "<=") {
-            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), ">=");
-        }
-        else if (c.getChildren()[2].getFirstToken().getStr() === ">=") {
-            insertFix = edit_helper_1.EditHelper.replaceToken(file, c.getChildren()[2].getLastToken(), "<=");
-        }
-        else {
-            return;
-        }
-        const endCol = c.getChildren()[0].getFirstToken().getEnd().getCol() + 1;
-        const endPosition = new position_1.Position(c.getChildren()[0].getFirstToken().getEnd().getRow(), endCol);
-        const deleteFix = edit_helper_1.EditHelper.deleteRange(file, c.getChildren()[0].getFirstToken().getStart(), endPosition);
-        const finalFix = edit_helper_1.EditHelper.merge(insertFix, deleteFix);
-        return finalFix;
-    }
-}
-exports.PreferIsNot = PreferIsNot;
-//# sourceMappingURL=prefer_is_not.js.map
-
-/***/ },
-
-/***/ "./node_modules/@abaplint/core/build/src/rules/prefer_pragmas.js"
-/*!***********************************************************************!*\
-  !*** ./node_modules/@abaplint/core/build/src/rules/prefer_pragmas.js ***!
-  \***********************************************************************/
-(__unused_webpack_module, exports, __webpack_require__) {
-
-"use strict";
-
-Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.PreferPragmas = exports.PreferPragmasConf = void 0;
-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");
-class PreferPragmasConf extends _basic_rule_config_1.BasicRuleConfig {
-    constructor() {
-        super(...arguments);
-        this.check = [
-            {
-                pseudo: "#EC CI_SUBRC",
-                pragma: "SUBRC_OK",
-            },
-            {
-                pseudo: "#EC NEEDED",
-                pragma: "NEEDED",
-            },
-            {
-                pseudo: "#EC NOTEXT",
-                pragma: "NO_TEXT",
-            },
-            {
-                pseudo: "#EC NO_HANDLER",
-                pragma: "NO_HANDLER",
-            },
-        ];
-    }
-}
-exports.PreferPragmasConf = PreferPragmasConf;
-class PreferPragmas extends _abap_rule_1.ABAPRule {
-    constructor() {
-        super(...arguments);
-        this.conf = new PreferPragmasConf();
-    }
-    getMetadata() {
-        return {
-            key: "prefer_pragmas",
-            title: "prefer pragmas over pseudo comments ",
-            shortDescription: `prefer pragmas over pseudo comments`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-pragmas-to-pseudo-comments`,
-            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
-            badExample: `DATA foo1 TYPE i. "#EC NEEDED`,
-            goodExample: `DATA foo2 TYPE i ##NEEDED.`,
-        };
-    }
-    getConfig() {
-        return this.conf;
-    }
-    setConfig(conf) {
-        this.conf = conf;
-    }
-    runParsed(file) {
-        const issues = [];
-        const config = this.getConfig();
-        for (const statement of file.getStatements()) {
-            if (!(statement.get() instanceof _statement_1.Comment)) {
-                continue;
-            }
-            const concat = statement.concatTokens().toUpperCase();
-            if (concat.includes("#EC") === false) {
-                continue;
-            }
-            for (const check of config.check) {
-                if (concat.includes(check.pseudo.toUpperCase())) {
-                    const message = `Prefer pragma ${check.pragma}`;
-                    issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.getConfig().severity));
-                }
-            }
-        }
-        return issues;
+    getFix(file, statement, withExporting) {
+        const children = statement.getChildren();
+        const fixText = withExporting ? "( " + children[5].concatTokens() + " )." : "( ).";
+        const contentFix = edit_helper_1.EditHelper.replaceRange(file, children[3].getLastToken().getEnd(), statement.getEnd(), fixText);
+        const replaceType = edit_helper_1.EditHelper.replaceToken(file, children[2].getFirstToken(), "NEW");
+        return edit_helper_1.EditHelper.merge(contentFix, replaceType);
     }
 }
-exports.PreferPragmas = PreferPragmas;
-//# sourceMappingURL=prefer_pragmas.js.map
+exports.PreferRaiseExceptionNew = PreferRaiseExceptionNew;
+//# sourceMappingURL=prefer_raise_exception_new.js.map
 
 /***/ },
 
-/***/ "./node_modules/@abaplint/core/build/src/rules/prefer_raise_exception_new.js"
-/*!***********************************************************************************!*\
-  !*** ./node_modules/@abaplint/core/build/src/rules/prefer_raise_exception_new.js ***!
-  \***********************************************************************************/
+/***/ "./node_modules/@abaplint/core/build/src/rules/prefer_raise_message.js"
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/prefer_raise_message.js ***!
+  \*****************************************************************************/
 (__unused_webpack_module, exports, __webpack_require__) {
 
 "use strict";
@@ -101031,36 +102529,40 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.PreferRaiseExceptionNew = exports.PreferRaiseExceptionNewConf = void 0;
+exports.PreferRaiseMessage = exports.PreferRaiseMessageConf = void 0;
 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 edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
-const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
 const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
-class PreferRaiseExceptionNewConf extends _basic_rule_config_1.BasicRuleConfig {
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+class PreferRaiseMessageConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** List of exception class names to ignore, case insensitive */
+        this.ignoreExceptionClasses = [];
+    }
 }
-exports.PreferRaiseExceptionNewConf = PreferRaiseExceptionNewConf;
-class PreferRaiseExceptionNew extends _abap_rule_1.ABAPRule {
+exports.PreferRaiseMessageConf = PreferRaiseMessageConf;
+class PreferRaiseMessage extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
-        this.conf = new PreferRaiseExceptionNewConf();
+        this.conf = new PreferRaiseMessageConf();
     }
     getMetadata() {
         return {
-            key: "prefer_raise_exception_new",
-            title: "Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE",
-            shortDescription: `Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-raise-exception-new-to-raise-exception-type
+            key: "prefer_raise_message",
+            title: "Prefer RAISE EXCEPTION with MESSAGE",
+            shortDescription: `Prefer RAISE EXCEPTION with MESSAGE instead of supplying parameter TEXT or TEXTID`,
+            extendedInformation: `Reports RAISE EXCEPTION statements where parameter TEXT or TEXTID is supplied.
 
-From 752 and up`,
-            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Upport],
-            goodExample: `RAISE EXCEPTION NEW cx_generation_error( previous = exception ).`,
-            badExample: `RAISE EXCEPTION TYPE cx_generation_error
-  EXPORTING
-    previous = exception.`,
+Use a message class instead, so the text is translatable and can be found via where-used.
+
+Exception classes can be excluded via configuration "ignoreExceptionClasses".`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            goodExample: `RAISE EXCEPTION TYPE zcx_error MESSAGE e001(zclass).`,
+            badExample: `RAISE EXCEPTION TYPE zcx_error
+  EXPORTING text = 'hello world'.`,
         };
     }
     getConfig() {
@@ -101070,35 +102572,46 @@ From 752 and up`,
         this.conf = conf;
     }
     runParsed(file) {
-        if (!(0, version_1.releaseAtLeast)(this.reg.getConfig().getRelease(), version_1.Release.v752)) {
-            return [];
-        }
+        var _a, _b, _c, _d;
+        var _e;
         const issues = [];
+        const ignore = ((_e = this.conf.ignoreExceptionClasses) !== null && _e !== void 0 ? _e : []).map(c => c.toUpperCase());
         for (const statement of file.getStatements()) {
-            if (statement.get() instanceof Statements.Raise) {
-                const concat = statement.concatTokens().toUpperCase();
-                if (concat.includes(" MESSAGE")) {
-                    continue;
-                }
-                if (concat.startsWith("RAISE EXCEPTION TYPE ")) {
-                    const message = "Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE";
-                    const fix = this.getFix(file, statement, concat.includes(" EXPORTING") ? true : false);
-                    issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, fix));
-                }
+            if (!(statement.get() instanceof Statements.Raise)) {
+                continue;
+            }
+            const concat = statement.concatTokens().toUpperCase();
+            if (!concat.startsWith("RAISE EXCEPTION ") && !concat.startsWith("RAISE RESUMABLE EXCEPTION ")) {
+                continue;
+            }
+            else if (concat.includes(" MESSAGE")) {
+                continue;
+            }
+            let className;
+            let parameters;
+            const newObject = (_b = (_a = statement.findDirectExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.MethodCallChain)) === null || _b === void 0 ? void 0 : _b.findDirectExpression(Expressions.NewObject);
+            if (newObject) {
+                className = (_c = newObject.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+                parameters = newObject.findDirectExpression(Expressions.ParameterListS);
+            }
+            else {
+                className = (_d = statement.findDirectExpression(Expressions.ClassName)) === null || _d === void 0 ? void 0 : _d.concatTokens();
+                parameters = statement.findDirectExpression(Expressions.ParameterListS);
+            }
+            if (className === undefined || ignore.includes(className.toUpperCase())) {
+                continue;
+            }
+            const name = parameters === null || parameters === void 0 ? void 0 : parameters.findDirectExpressions(Expressions.ParameterS).map(p => { var _a; return (_a = p.findDirectExpression(Expressions.ParameterName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase(); }).find(n => n === "TEXT" || n === "TEXTID");
+            if (name) {
+                const message = "Prefer RAISE EXCEPTION with MESSAGE, " + className + " raised with parameter " + name;
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
             }
         }
         return issues;
     }
-    getFix(file, statement, withExporting) {
-        const children = statement.getChildren();
-        const fixText = withExporting ? "( " + children[5].concatTokens() + " )." : "( ).";
-        const contentFix = edit_helper_1.EditHelper.replaceRange(file, children[3].getLastToken().getEnd(), statement.getEnd(), fixText);
-        const replaceType = edit_helper_1.EditHelper.replaceToken(file, children[2].getFirstToken(), "NEW");
-        return edit_helper_1.EditHelper.merge(contentFix, replaceType);
-    }
 }
-exports.PreferRaiseExceptionNew = PreferRaiseExceptionNew;
-//# sourceMappingURL=prefer_raise_exception_new.js.map
+exports.PreferRaiseMessage = PreferRaiseMessage;
+//# sourceMappingURL=prefer_raise_message.js.map
 
 /***/ },
 
@@ -126437,11 +127950,33 @@ class FieldSymbolTranspiler {
         if (found === undefined) {
             throw new Error("FieldSymbolTranspiler, var not found, \"" + token.getStr() + "\"");
         }
+        let type = found.getType();
+        // abaplint types TYPE STANDARD TABLE, SORTED TABLE, HASHED TABLE, INDEX TABLE and
+        // ANY TABLE alike, a generic table without a primary key; the kind is in the statement,
+        // and the runtime needs it to refuse an ASSIGN of a table of another kind
+        const typeTable = node.findDirectExpression(abaplint.Expressions.TypeTable);
+        if (typeTable !== undefined
+            && type instanceof abaplint.BasicTypes.TableType
+            && type.getOptions().primaryKey === undefined) {
+            const kind = /^TYPE (STANDARD|SORTED|HASHED|INDEX|ANY) TABLE$/i.exec(typeTable.concatTokens());
+            if (kind !== null) {
+                const options = {
+                    ...type.getOptions(),
+                    primaryKey: {
+                        name: "primary_key",
+                        type: kind[1].toUpperCase(),
+                        keyFields: [],
+                        isUnique: false,
+                    },
+                };
+                type = new abaplint.BasicTypes.TableType(type.getRowType(), options, type.getQualifiedName());
+            }
+        }
         if (name) {
             return new chunk_1.Chunk()
                 .appendString("let ")
                 .appendString(new field_symbol_1.FieldSymbolTranspiler().transpile(name, traversal).getCode())
-                .appendString(" = new abap.types.FieldSymbol(" + traversal.toType(found.getType()) + ");");
+                .appendString(" = new abap.types.FieldSymbol(" + traversal.toType(type) + ");");
         }
         throw new Error("FieldSymbolTranspiler, name not found");
     }