@abaplint/cli 2.113.131 → 2.113.133

package/build/cli.js

@@ -4850,7 +4850,8 @@ class CorrespondingBody extends combi_1.Expression {
         const mapping = (0, combi_1.seq)("MAPPING", (0, combi_1.plus)((0, combi_1.seq)(_1.ComponentName, "=", component_chain_1.ComponentChain)));
         const baseParen = (0, combi_1.seq)("BASE", (0, combi_1.tok)(tokens_1.WParenLeftW), _1.Source, (0, combi_1.tok)(tokens_1.WParenRightW));
         const discarding = (0, combi_1.ver)(version_1.Version.v751, "DISCARDING DUPLICATES");
-        return (0, combi_1.seq)((0, combi_1.optPrio)("DEEP"), (0, combi_1.optPrio)(baseParen), _1.Source, (0, combi_1.optPrio)(discarding), (0, combi_1.optPrio)(mapping), (0, combi_1.optPrio)("CHANGING CONTROL"), (0, combi_1.optPrio)((0, combi_1.seq)("EXCEPT", (0, combi_1.alt)((0, combi_1.plus)(_1.Field), "*"))));
+        return (0, combi_1.seq)((0, combi_1.optPrio)("DEEP"), (0, combi_1.optPrio)(baseParen), _1.Source, (0, combi_1.optPrio)(discarding), (0, combi_1.optPrio)(mapping), (0, combi_1.optPrio)("CHANGING CONTROL"), (0, combi_1.optPrio)("MAPPING FROM ENTITY USING CONTROL"), // todo, version something?
+        (0, combi_1.optPrio)((0, combi_1.seq)("EXCEPT", (0, combi_1.alt)((0, combi_1.plus)(_1.Field), "*"))));
     }
 }
 exports.CorrespondingBody = CorrespondingBody;
@@ -8486,7 +8487,7 @@ const integer_1 = __webpack_require__(/*! ./integer */ "./node_modules/@abaplint
 const sql_function_input_1 = __webpack_require__(/*! ./sql_function_input */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_function_input.js");
 class SQLFunction extends combi_1.Expression {
     getRunnable() {
-        const castTypes = (0, combi_1.altPrio)((0, combi_1.seq)("CHAR", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), (0, combi_1.seq)("DEC", (0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, ",", integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)), (0, combi_1.seq)("NUMC", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), "FLTP", "INT2", "INT8");
+        const castTypes = (0, combi_1.altPrio)((0, combi_1.seq)("CHAR", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), (0, combi_1.seq)("DEC", (0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, ",", integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)), (0, combi_1.seq)("NUMC", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), "FLTP", "INT2", "INT4", "INT8");
         const commaParam = (0, combi_1.seq)(",", sql_function_input_1.SQLFunctionInput);
         // note: the function names are not keywords, they are usually in lower case
         const abs = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^abs$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, (0, combi_1.tok)(tokens_1.WParenRightW)));
@@ -9466,7 +9467,8 @@ class TypeStructure extends combi_1.Expression {
         const reportedEarly = (0, combi_1.seq)("REPORTED EARLY", _1.NamespaceSimpleName);
         const structure = (0, combi_1.seq)("STRUCTURE FOR", (0, combi_1.altPrio)(hier, evt, create, update, action, permissionsRequest, readResult));
         const response = (0, combi_1.seq)("RESPONSE FOR", (0, combi_1.altPrio)(failedEarly, mappedEarly, reportedEarly));
-        return (0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(structure, response));
+        const request = (0, combi_1.seq)("REQUEST FOR CHANGE", _1.NamespaceSimpleName);
+        return (0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(structure, response, request));
     }
 }
 exports.TypeStructure = TypeStructure;
@@ -11375,7 +11377,7 @@ class CommitEntities {
         const failed = (0, combi_1.seq)("FAILED", expressions_1.Target);
         const reported = (0, combi_1.seq)("REPORTED", expressions_1.Target);
         const responses = (0, combi_1.seq)("RESPONSES", failed, reported);
-        const s = (0, combi_1.seq)("COMMIT ENTITIES", (0, combi_1.optPrio)("IN SIMULATION MODE"), (0, combi_1.opt)(responses), (0, combi_1.opt)((0, combi_1.seq)("RESPONSE OF", expressions_1.NamespaceSimpleName, failed, reported)));
+        const s = (0, combi_1.seq)("COMMIT ENTITIES", (0, combi_1.optPrio)("IN SIMULATION MODE"), (0, combi_1.opt)(responses), (0, combi_1.opt)((0, combi_1.seq)("RESPONSE OF", expressions_1.NamespaceSimpleName, (0, combi_1.per)(failed, reported))));
         return (0, combi_1.ver)(version_1.Version.v754, s);
     }
 }
@@ -14204,6 +14206,7 @@ __exportStar(__webpack_require__(/*! ./modify_entities */ "./node_modules/@abapl
 __exportStar(__webpack_require__(/*! ./read_entities */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/read_entities.js"), exports);
 __exportStar(__webpack_require__(/*! ./rollback_entities */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/rollback_entities.js"), exports);
 __exportStar(__webpack_require__(/*! ./summary */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/summary.js"), exports);
+__exportStar(__webpack_require__(/*! ./input */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/input.js"), exports);
 __exportStar(__webpack_require__(/*! ./on_change */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/on_change.js"), exports);
 __exportStar(__webpack_require__(/*! ./at_user_command */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/at_user_command.js"), exports);
 __exportStar(__webpack_require__(/*! ./position */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/position.js"), exports);
@@ -14485,6 +14488,28 @@ exports.Initialization = Initialization;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/input.js":
+/*!*************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/2_statements/statements/input.js ***!
+  \*************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.Input = void 0;
+const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
+class Input {
+    getMatcher() {
+        return (0, combi_1.verNot)(version_1.Version.Cloud, "INPUT");
+    }
+}
+exports.Input = Input;
+//# sourceMappingURL=input.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/insert_database.js":
 /*!***********************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/2_statements/statements/insert_database.js ***!
@@ -14955,20 +14980,18 @@ class MethodDef {
         const def = (0, combi_1.ver)(version_1.Version.v740sp08, (0, combi_1.seq)("DEFAULT", (0, combi_1.altPrio)("FAIL", "IGNORE")));
         const parameters = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("FINAL", def, expressions_1.Abstract)), (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefChanging), (0, combi_1.optPrio)(expressions_1.MethodDefReturning), (0, combi_1.optPrio)((0, combi_1.altPrio)(expressions_1.MethodDefRaising, expressions_1.MethodDefExceptions)));
         const testing = (0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), "FOR TESTING", (0, combi_1.optPrio)((0, combi_1.altPrio)(expressions_1.MethodDefRaising, expressions_1.MethodDefExceptions)));
-        // todo, this is only from version something
-        const tableFunction = (0, combi_1.seq)("TABLE FUNCTION", expressions_1.NamespaceSimpleName);
-        // todo, this is only from version something
-        const ddl = "DDL OBJECT OPTIONS CDS SESSION CLIENT REQUIRED";
         const result = (0, combi_1.seq)("RESULT", expressions_1.MethodParamName);
         const link = (0, combi_1.seq)("LINK", expressions_1.MethodParamName);
         const full = (0, combi_1.seq)("FULL", expressions_1.MethodParamName);
         const modify = (0, combi_1.alt)((0, combi_1.seq)("FOR ACTION", expressions_1.TypeName, (0, combi_1.optPrio)(result)), (0, combi_1.seq)("FOR CREATE", (0, combi_1.alt)(expressions_1.TypeName, expressions_1.EntityAssociation)), (0, combi_1.seq)("FOR DELETE", expressions_1.TypeName), (0, combi_1.seq)("FOR UPDATE", expressions_1.TypeName));
         const forRead = (0, combi_1.seq)("FOR READ", (0, combi_1.alt)(expressions_1.TypeName, expressions_1.EntityAssociation), (0, combi_1.optPrio)(full), result, (0, combi_1.optPrio)(link));
         const forfunction = (0, combi_1.seq)("FOR FUNCTION", expressions_1.TypeName, result);
-        const behavior = (0, combi_1.altPrio)((0, combi_1.seq)("VALIDATE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("MODIFY IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("READ IMPORTING", expressions_1.MethodParamName, (0, combi_1.altPrio)(forRead, forfunction)), (0, combi_1.seq)("FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.NamespaceSimpleName, "FOR", expressions_1.NamespaceSimpleName, result), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR CREATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR UPDATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR DELETE", expressions_1.TypeName), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR READ", expressions_1.TypeName, result), (0, combi_1.seq)((0, combi_1.alt)("BEHAVIOR", "LOCK"), "IMPORTING", expressions_1.MethodParamName, "FOR LOCK", expressions_1.TypeName), (0, combi_1.seq)("DETERMINE", (0, combi_1.alt)("ON MODIFY", "ON SAVE"), "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("GLOBAL AUTHORIZATION IMPORTING REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("INSTANCE AUTHORIZATION IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("INSTANCE FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result));
+        const behavior = (0, combi_1.altPrio)("DDL OBJECT OPTIONS CDS SESSION CLIENT REQUIRED", // todo, this is only from version something
+        (0, combi_1.seq)("TABLE FUNCTION", expressions_1.NamespaceSimpleName), // todo, this is only from version something
+        (0, combi_1.seq)("VALIDATE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("MODIFY IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("PRECHECK IMPORTING", expressions_1.MethodParamName, modify), (0, combi_1.seq)("READ IMPORTING", expressions_1.MethodParamName, (0, combi_1.altPrio)(forRead, forfunction)), (0, combi_1.seq)("FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.NamespaceSimpleName, "FOR", expressions_1.NamespaceSimpleName, result), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR CREATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR UPDATE", expressions_1.TypeName, expressions_1.MethodParamName, "FOR DELETE", expressions_1.TypeName), (0, combi_1.seq)("BEHAVIOR IMPORTING", expressions_1.MethodParamName, "FOR READ", expressions_1.TypeName, result), (0, combi_1.seq)((0, combi_1.alt)("BEHAVIOR", "LOCK"), "IMPORTING", expressions_1.MethodParamName, "FOR LOCK", expressions_1.TypeName), (0, combi_1.seq)("DETERMINE", (0, combi_1.alt)("ON MODIFY", "ON SAVE"), "IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("GLOBAL AUTHORIZATION IMPORTING REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("INSTANCE AUTHORIZATION IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result), (0, combi_1.seq)("INSTANCE FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.MethodParamName, "FOR", expressions_1.TypeName, result));
         // todo, this is only from version something
         const amdp = (0, combi_1.seq)("AMDP OPTIONS", (0, combi_1.optPrio)("READ-ONLY"), "CDS SESSION CLIENT CURRENT", (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefRaising));
-        const ret = (0, combi_1.seq)((0, combi_1.altPrio)("CLASS-METHODS", "METHODS"), expressions_1.MethodName, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), (0, combi_1.optPrio)(def), expressions_1.EventHandler), parameters, testing, (0, combi_1.seq)("FOR", (0, combi_1.alt)(tableFunction, ddl, behavior)), amdp, "NOT AT END OF MODE", (0, combi_1.optPrio)(expressions_1.Redefinition)));
+        const ret = (0, combi_1.seq)((0, combi_1.altPrio)("CLASS-METHODS", "METHODS"), expressions_1.MethodName, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), (0, combi_1.optPrio)(def), expressions_1.EventHandler), parameters, testing, (0, combi_1.seq)("FOR", behavior), amdp, "NOT AT END OF MODE", (0, combi_1.optPrio)(expressions_1.Redefinition)));
         return ret;
     }
 }
@@ -24290,6 +24313,9 @@ class BasicTypes {
             if (found && this.isOccurs(node)) {
                 found = new Types.TableType(found, { withHeader: text.includes("WITH HEADER LINE"), keyType: Types.TableKeyType.default }, qualifiedName);
             }
+            else if (text.endsWith(" WITH HEADER LINE") && found instanceof Types.TableType) {
+                found = new Types.TableType(found.getRowType(), { withHeader: true, keyType: Types.TableKeyType.default }, qualifiedName);
+            }
         }
         else if (text.startsWith("TYPE LINE OF ")) {
             const sub = node.findFirstExpression(Expressions.TypeName);
@@ -27853,10 +27879,12 @@ class Select {
         var _a, _b;
         let expr = undefined;
         const ret = [];
-        expr = node.findFirstExpression(Expressions.SQLFieldList);
-        if (expr === undefined) {
+        if (node.get() instanceof Expressions.SelectLoop) {
             expr = node.findFirstExpression(Expressions.SQLFieldListLoop);
         }
+        else {
+            expr = node.findFirstExpression(Expressions.SQLFieldList);
+        }
         if (((_a = expr === null || expr === void 0 ? void 0 : expr.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
             dynamic_1.Dynamic.runSyntax(expr.getFirstChild(), input);
         }
@@ -33349,7 +33377,9 @@ class ReadTable {
             }
             const type = source_1.Source.runSyntax(s, input);
             if (s === afterKey) {
-                if (type instanceof basic_1.StringType || type instanceof basic_1.TableType || type instanceof basic_1.ObjectReferenceType) {
+                if (type instanceof basic_1.StringType
+                    || (type instanceof basic_1.TableType && type.isWithHeader() === false)
+                    || type instanceof basic_1.ObjectReferenceType) {
                     const message = "Key cannot be string or table or reference";
                     input.issues.push((0, _syntax_input_1.syntaxIssue)(input, s.getFirstToken(), message));
                     return;
@@ -47101,6 +47131,37 @@ exports.BusinessCatalogAppAssignment = BusinessCatalogAppAssignment;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/business_configuration_maintenance_object.js":
+/*!****************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/business_configuration_maintenance_object.js ***!
+  \****************************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.BusinessConfigurationMaintenanceObject = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class BusinessConfigurationMaintenanceObject extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "SMBC";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 60, // todo
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.BusinessConfigurationMaintenanceObject = BusinessConfigurationMaintenanceObject;
+//# sourceMappingURL=business_configuration_maintenance_object.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/business_configuration_set.js":
 /*!*************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/business_configuration_set.js ***!
@@ -47715,6 +47776,37 @@ exports.ConceptsOfPackage = ConceptsOfPackage;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/custom_data_browser_object.js":
+/*!*************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/custom_data_browser_object.js ***!
+  \*************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CustomDataBrowserObject = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class CustomDataBrowserObject extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "CDBO";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 32,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.CustomDataBrowserObject = CustomDataBrowserObject;
+//# sourceMappingURL=custom_data_browser_object.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/customer_enhancement_project.js":
 /*!***************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/customer_enhancement_project.js ***!
@@ -49118,6 +49210,37 @@ exports.GatewayModelMetadata = GatewayModelMetadata;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/gateway_odata_group_and_assignment.js":
+/*!*********************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/gateway_odata_group_and_assignment.js ***!
+  \*********************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.GatewayODataGroupAndAssignment = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class GatewayODataGroupAndAssignment extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "G4BA";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 32,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.GatewayODataGroupAndAssignment = GatewayODataGroupAndAssignment;
+//# sourceMappingURL=gateway_odata_group_and_assignment.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/gateway_project.js":
 /*!**************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/gateway_project.js ***!
@@ -49730,6 +49853,7 @@ __exportStar(__webpack_require__(/*! ./bsp_application */ "./node_modules/@abapl
 __exportStar(__webpack_require__(/*! ./business_add_in_implementation */ "./node_modules/@abaplint/core/build/src/objects/business_add_in_implementation.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_catalog_app_assignment */ "./node_modules/@abaplint/core/build/src/objects/business_catalog_app_assignment.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_catalog */ "./node_modules/@abaplint/core/build/src/objects/business_catalog.js"), exports);
+__exportStar(__webpack_require__(/*! ./business_configuration_maintenance_object */ "./node_modules/@abaplint/core/build/src/objects/business_configuration_maintenance_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_configuration_set */ "./node_modules/@abaplint/core/build/src/objects/business_configuration_set.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_function_assignment */ "./node_modules/@abaplint/core/build/src/objects/business_function_assignment.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_function_set_assignment */ "./node_modules/@abaplint/core/build/src/objects/business_function_set_assignment.js"), exports);
@@ -49746,6 +49870,7 @@ __exportStar(__webpack_require__(/*! ./communication_scenario */ "./node_modules
 __exportStar(__webpack_require__(/*! ./composite_enhancement_implementation */ "./node_modules/@abaplint/core/build/src/objects/composite_enhancement_implementation.js"), exports);
 __exportStar(__webpack_require__(/*! ./composite_enhancement_spot */ "./node_modules/@abaplint/core/build/src/objects/composite_enhancement_spot.js"), exports);
 __exportStar(__webpack_require__(/*! ./concepts_of_package */ "./node_modules/@abaplint/core/build/src/objects/concepts_of_package.js"), exports);
+__exportStar(__webpack_require__(/*! ./custom_data_browser_object */ "./node_modules/@abaplint/core/build/src/objects/custom_data_browser_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./customer_enhancement_project */ "./node_modules/@abaplint/core/build/src/objects/customer_enhancement_project.js"), exports);
 __exportStar(__webpack_require__(/*! ./customizing_attributes */ "./node_modules/@abaplint/core/build/src/objects/customizing_attributes.js"), exports);
 __exportStar(__webpack_require__(/*! ./customizing_img_activity */ "./node_modules/@abaplint/core/build/src/objects/customizing_img_activity.js"), exports);
@@ -49773,6 +49898,7 @@ __exportStar(__webpack_require__(/*! ./format_type */ "./node_modules/@abaplint/
 __exportStar(__webpack_require__(/*! ./function_group */ "./node_modules/@abaplint/core/build/src/objects/function_group.js"), exports);
 __exportStar(__webpack_require__(/*! ./gateway_model_metadata */ "./node_modules/@abaplint/core/build/src/objects/gateway_model_metadata.js"), exports);
 __exportStar(__webpack_require__(/*! ./gateway_model */ "./node_modules/@abaplint/core/build/src/objects/gateway_model.js"), exports);
+__exportStar(__webpack_require__(/*! ./gateway_odata_group_and_assignment */ "./node_modules/@abaplint/core/build/src/objects/gateway_odata_group_and_assignment.js"), exports);
 __exportStar(__webpack_require__(/*! ./gateway_project */ "./node_modules/@abaplint/core/build/src/objects/gateway_project.js"), exports);
 __exportStar(__webpack_require__(/*! ./gateway_service_groups_metadata */ "./node_modules/@abaplint/core/build/src/objects/gateway_service_groups_metadata.js"), exports);
 __exportStar(__webpack_require__(/*! ./gateway_service */ "./node_modules/@abaplint/core/build/src/objects/gateway_service.js"), exports);
@@ -49793,6 +49919,7 @@ __exportStar(__webpack_require__(/*! ./inbound_service */ "./node_modules/@abapl
 __exportStar(__webpack_require__(/*! ./info_area */ "./node_modules/@abaplint/core/build/src/objects/info_area.js"), exports);
 __exportStar(__webpack_require__(/*! ./info_object */ "./node_modules/@abaplint/core/build/src/objects/info_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./interface */ "./node_modules/@abaplint/core/build/src/objects/interface.js"), exports);
+__exportStar(__webpack_require__(/*! ./knowledge_transfer_document */ "./node_modules/@abaplint/core/build/src/objects/knowledge_transfer_document.js"), exports);
 __exportStar(__webpack_require__(/*! ./lock_object */ "./node_modules/@abaplint/core/build/src/objects/lock_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./maintenance_and_transport_object */ "./node_modules/@abaplint/core/build/src/objects/maintenance_and_transport_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./message_class */ "./node_modules/@abaplint/core/build/src/objects/message_class.js"), exports);
@@ -49828,6 +49955,7 @@ __exportStar(__webpack_require__(/*! ./object_characteristic */ "./node_modules/
 __exportStar(__webpack_require__(/*! ./outbound_service */ "./node_modules/@abaplint/core/build/src/objects/outbound_service.js"), exports);
 __exportStar(__webpack_require__(/*! ./package_interface */ "./node_modules/@abaplint/core/build/src/objects/package_interface.js"), exports);
 __exportStar(__webpack_require__(/*! ./package */ "./node_modules/@abaplint/core/build/src/objects/package.js"), exports);
+__exportStar(__webpack_require__(/*! ./page_format */ "./node_modules/@abaplint/core/build/src/objects/page_format.js"), exports);
 __exportStar(__webpack_require__(/*! ./parameter */ "./node_modules/@abaplint/core/build/src/objects/parameter.js"), exports);
 __exportStar(__webpack_require__(/*! ./personalization_object */ "./node_modules/@abaplint/core/build/src/objects/personalization_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./program */ "./node_modules/@abaplint/core/build/src/objects/program.js"), exports);
@@ -50020,6 +50148,37 @@ exports.Interface = Interface;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/knowledge_transfer_document.js":
+/*!**************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/knowledge_transfer_document.js ***!
+  \**************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.KnowledgeTransferDocument = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class KnowledgeTransferDocument extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "SKTD";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 60, // todo
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.KnowledgeTransferDocument = KnowledgeTransferDocument;
+//# sourceMappingURL=knowledge_transfer_document.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/lock_object.js":
 /*!**********************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/lock_object.js ***!
@@ -51360,6 +51519,37 @@ exports.PackageInterface = PackageInterface;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/page_format.js":
+/*!**********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/page_format.js ***!
+  \**********************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.PageFormat = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class PageFormat extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "SPPF";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 30,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.PageFormat = PageFormat;
+//# sourceMappingURL=page_format.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/parameter.js":
 /*!********************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/parameter.js ***!
@@ -54354,7 +54544,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.131";
+        return "2.113.133";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -67326,6 +67516,7 @@ ENDIF.
         return issues;
     }
     analyze(file, cond) {
+        var _a, _b;
         const issues = [];
         let comparator = "";
         let found = false;
@@ -67351,6 +67542,17 @@ ENDIF.
                     found = true; // dont report for the simple case that contains quick fixes
                 }
             }
+            else if (c instanceof nodes_1.ExpressionNode
+                && c.get() instanceof Expressions.Compare
+                && ((_a = c.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Source
+                && c.getChildren().length === 3) {
+                const concat = (_b = c.getFirstChild()) === null || _b === void 0 ? void 0 : _b.concatTokens();
+                if ((concat === null || concat === void 0 ? void 0 : concat.startsWith("(")) && concat.endsWith(")")) {
+                    const message = "Parentheses can be removed";
+                    const issue = issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+            }
             if (comparator === "") {
                 comparator = current;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.131",
+    "version": "2.113.133",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,21 +38,21 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.131",
+        "@abaplint/core": "^2.113.133",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.0.3",
+        "@types/node": "^24.0.7",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
         "chalk": "^5.4.1",
-        "eslint": "^9.29.0",
+        "eslint": "^9.30.0",
         "glob": "^11.0.3",
         "json5": "^2.2.3",
         "memfs": "^4.17.2",
         "minimist": "^1.2.8",
-        "mocha": "^11.6.0",
+        "mocha": "^11.7.1",
         "progress": "^2.0.3",
         "typescript": "^5.8.3",
         "webpack": "^5.99.9",