@abaplint/cli 2.113.247 → 2.114.1

package/build/cli.js

@@ -20677,33 +20677,21 @@ exports.Identifier = Identifier;
 /*!************************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/4_file_information/abap_file_information.js ***!
   \************************************************************************************************/
-/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+/***/ ((__unused_webpack_module, exports) => {
 
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ABAPFileInformation = void 0;
-const Structures = __webpack_require__(/*! ../3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
-const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
-const Statements = __webpack_require__(/*! ../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
-const _abap_file_information_1 = __webpack_require__(/*! ./_abap_file_information */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_abap_file_information.js");
-const _identifier_1 = __webpack_require__(/*! ./_identifier */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_identifier.js");
-const Tokens = __webpack_require__(/*! ../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
-const visibility_1 = __webpack_require__(/*! ./visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 class ABAPFileInformation {
-    constructor(structure, filename) {
-        this.forms = [];
-        this.implementations = [];
-        this.interfaces = [];
-        this.classes = [];
-        this.filename = filename;
-        this.parse(structure);
+    constructor(parsed) {
+        this.parsed = parsed;
     }
     listClassImplementations() {
-        return this.implementations;
+        return this.parsed.implementations;
     }
     listInterfaceDefinitions() {
-        return this.interfaces;
+        return this.parsed.interfaces;
     }
     getInterfaceDefinitionByName(name) {
         const upper = name.toUpperCase();
@@ -20715,7 +20703,7 @@ class ABAPFileInformation {
         return undefined;
     }
     listClassDefinitions() {
-        return this.classes;
+        return this.parsed.classes;
     }
     getClassDefinitionByName(name) {
         const upper = name.toUpperCase();
@@ -20736,16 +20724,54 @@ class ABAPFileInformation {
         return undefined;
     }
     listFormDefinitions() {
-        return this.forms;
+        return this.parsed.forms;
+    }
+}
+exports.ABAPFileInformation = ABAPFileInformation;
+//# sourceMappingURL=abap_file_information.js.map
+
+/***/ }),
+
+/***/ "./node_modules/@abaplint/core/build/src/abap/4_file_information/abap_file_information_parser.js":
+/*!*******************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/4_file_information/abap_file_information_parser.js ***!
+  \*******************************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ABAPFileInformationParser = void 0;
+const Structures = __webpack_require__(/*! ../3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
+const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const Statements = __webpack_require__(/*! ../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const _identifier_1 = __webpack_require__(/*! ./_identifier */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_identifier.js");
+const Tokens = __webpack_require__(/*! ../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
+const visibility_1 = __webpack_require__(/*! ./visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
+const _abap_file_information_1 = __webpack_require__(/*! ./_abap_file_information */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_abap_file_information.js");
+class ABAPFileInformationParser {
+    constructor(filename) {
+        this.filename = filename;
     }
-    ///////////////////////
     parse(structure) {
-        var _a;
+        const result = {
+            interfaces: [],
+            classes: [],
+            forms: [],
+            implementations: [],
+        };
         if (structure === undefined) {
-            return;
+            return result;
         }
-        this.parseClasses(structure);
-        this.parseInterfaces(structure);
+        result.classes = this.parseClasses(structure);
+        result.interfaces = this.parseInterfaces(structure);
+        result.implementations = this.parseClassImplementations(structure);
+        result.forms = this.parseForms(structure);
+        return result;
+    }
+    parseClassImplementations(structure) {
+        var _a;
+        const implementations = [];
         for (const found of structure.findAllStructures(Structures.ClassImplementation)) {
             const methods = [];
             for (const method of found.findAllStructures(Structures.Method)) {
@@ -20755,24 +20781,30 @@ class ABAPFileInformation {
                 }
             }
             const name = found.findFirstStatement(Statements.ClassImplementation).findFirstExpression(Expressions.ClassName).getFirstToken();
-            this.implementations.push({
+            implementations.push({
                 name: name.getStr(),
                 identifier: new _identifier_1.Identifier(name, this.filename),
                 methods,
             });
         }
+        return implementations;
+    }
+    parseForms(structure) {
+        const forms = [];
         for (const statement of structure.findAllStructures(Structures.Form)) {
             // FORMs can contain a dash in the name
             const pos = statement.findFirstExpression(Expressions.FormName).getFirstToken().getStart();
             const name = statement.findFirstExpression(Expressions.FormName).concatTokens();
             const nameToken = new Tokens.Identifier(pos, name);
-            this.forms.push({
+            forms.push({
                 name: nameToken.getStr(),
                 identifier: new _identifier_1.Identifier(nameToken, this.filename),
             });
         }
+        return forms;
     }
     parseInterfaces(structure) {
+        const interfaces = [];
         for (const found of structure.findDirectStructures(Structures.Interface)) {
             const i = found.findFirstStatement(Statements.Interface);
             if (i === undefined) {
@@ -20784,7 +20816,7 @@ class ABAPFileInformation {
             const aliases = this.parseAliases(found, visibility_1.Visibility.Public);
             const constants = this.parseConstants(found, visibility_1.Visibility.Public);
             const g = i.findDirectExpression(Expressions.ClassGlobal);
-            this.interfaces.push({
+            interfaces.push({
                 name: interfaceName.getStr(),
                 identifier: new _identifier_1.Identifier(interfaceName, this.filename),
                 isLocal: g === undefined,
@@ -20796,9 +20828,11 @@ class ABAPFileInformation {
                 attributes,
             });
         }
+        return interfaces;
     }
     parseClasses(structure) {
         var _a;
+        const classes = [];
         for (const found of structure.findAllStructures(Structures.ClassDefinition)) {
             const className = found.findFirstStatement(Statements.ClassDefinition).findFirstExpression(Expressions.ClassName).getFirstToken();
             const methods = this.parseMethodDefinition(found.findFirstStructure(Structures.PublicSection), visibility_1.Visibility.Public);
@@ -20837,7 +20871,7 @@ class ABAPFileInformation {
             else if (concat.includes("DURATION MEDIUM")) {
                 duration = _abap_file_information_1.Duration.medium;
             }
-            this.classes.push({
+            classes.push({
                 name: className.getStr(),
                 identifier: new _identifier_1.Identifier(className, this.filename),
                 isLocal: containsGlobal === undefined,
@@ -20856,6 +20890,7 @@ class ABAPFileInformation {
                 constants,
             });
         }
+        return classes;
     }
     ///////////////////
     getImplementing(input) {
@@ -21054,8 +21089,8 @@ class ABAPFileInformation {
         return ret;
     }
 }
-exports.ABAPFileInformation = ABAPFileInformation;
-//# sourceMappingURL=abap_file_information.js.map
+exports.ABAPFileInformationParser = ABAPFileInformationParser;
+//# sourceMappingURL=abap_file_information_parser.js.map
 
 /***/ }),
 
@@ -28026,7 +28061,7 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modu
 const isSimple = /^\w+$/;
 class Select {
     static runSyntax(node, input, skipImplicitInto = false) {
-        var _a, _b;
+        var _a;
         const token = node.getFirstToken();
         let from = node.findDirectExpression(Expressions.SQLFrom);
         if (from === undefined) {
@@ -28047,10 +28082,8 @@ class Select {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
-        const isSingle = ((_a = node.getChildren()[1]) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase()) === "SINGLE"
-            || node.get() instanceof Expressions.SelectLoop;
         this.checkFields(fields, dbSources, input, node);
-        this.handleInto(node, input, fields, dbSources, isSingle);
+        const intoExpression = this.handleInto(node, input, fields, dbSources);
         const fae = node.findDirectExpression(Expressions.SQLForAllEntries);
         if (fae) {
             input.scope.push(_scope_type_1.ScopeType.OpenSQL, "SELECT", token.getStart(), input.filename);
@@ -28064,7 +28097,7 @@ class Select {
             && node.findDirectExpression(Expressions.SQLIntoTable) === undefined
             && node.findDirectExpression(Expressions.SQLIntoList) === undefined
             && node.findDirectExpression(Expressions.SQLIntoStructure) === undefined) {
-            const fields = (_b = node.findFirstExpression(Expressions.SQLAggregation)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+            const fields = (_a = node.findFirstExpression(Expressions.SQLAggregation)) === null || _a === void 0 ? void 0 : _a.concatTokens();
             const c = new RegExp(/^count\(\s*\*\s*\)$/, "i");
             if (fields === undefined || c.test(fields) === false) {
                 const nameToken = from === null || from === void 0 ? void 0 : from.findDirectExpression(Expressions.SQLFromSource);
@@ -28098,8 +28131,18 @@ class Select {
         for (const s of node.findAllExpressions(Expressions.SQLCompare)) {
             sql_compare_1.SQLCompare.runSyntax(s, input, dbSources);
         }
-        for (const s of node.findDirectExpressions(Expressions.SQLOrderBy)) {
-            sql_order_by_1.SQLOrderBy.runSyntax(s, input);
+        const orderBy = node.findDirectExpression(Expressions.SQLOrderBy);
+        if (orderBy) {
+            sql_order_by_1.SQLOrderBy.runSyntax(orderBy, input);
+            const where = node.findDirectExpression(Expressions.SQLCond);
+            if (intoExpression
+                && where
+                && intoExpression.getFirstToken().getStart().isBefore(orderBy.getFirstToken().getStart())
+                && where.getFirstToken().getStart().isBefore(orderBy.getFirstToken().getStart())
+                && where.getFirstToken().getStart().isBefore(intoExpression.getFirstToken().getStart())) {
+                const message = `ORDER BY must be before INTO, after WHERE`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, orderBy.getFirstToken(), message));
+            }
         }
         if (this.isStrictMode(node)) {
             this.strictModeChecks(node, input);
@@ -28150,13 +28193,14 @@ class Select {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
         }
     }
-    static handleInto(node, input, fields, dbSources, _isSingle) {
+    static handleInto(node, input, fields, dbSources) {
         const intoTable = node.findDirectExpression(Expressions.SQLIntoTable);
         if (intoTable) {
             const inline = intoTable.findFirstExpression(Expressions.InlineData);
             if (inline) {
                 inline_data_1.InlineData.runSyntax(inline, input, this.buildTableType(fields, dbSources, input.scope));
             }
+            return intoTable;
         }
         const intoStructure = node.findDirectExpression(Expressions.SQLIntoStructure);
         if (intoStructure) {
@@ -28170,6 +28214,7 @@ class Select {
                     inline_data_1.InlineData.runSyntax(inline, input, basic_1.VoidType.get("SELECT_todo1"));
                 }
             }
+            return intoStructure;
         }
         const intoList = node.findDirectExpression(Expressions.SQLIntoList);
         if (intoList) {
@@ -28209,7 +28254,9 @@ class Select {
                     inline_data_1.InlineData.runSyntax(inline, input, type);
                 }
             }
+            return intoList;
         }
+        return undefined;
     }
     static checkFields(fields, dbSources, input, node) {
         if (dbSources.length > 1) {
@@ -36617,12 +36664,12 @@ class ABAPFile extends _abstract_file_1.AbstractFile {
     getRaw() {
         return this.file.getRaw();
     }
-    getInfo() {
-        return this.info;
-    }
     getRawRows() {
         return this.file.getRawRows();
     }
+    getInfo() {
+        return this.info;
+    }
     getStructure() {
         return this.structure;
     }
@@ -36659,12 +36706,13 @@ exports.ABAPFile = ABAPFile;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ABAPParser = void 0;
-const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const abap_file_1 = __webpack_require__(/*! ./abap_file */ "./node_modules/@abaplint/core/build/src/abap/abap_file.js");
+const abap_file_information_1 = __webpack_require__(/*! ./4_file_information/abap_file_information */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/abap_file_information.js");
+const abap_file_information_parser_1 = __webpack_require__(/*! ./4_file_information/abap_file_information_parser */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/abap_file_information_parser.js");
 const lexer_1 = __webpack_require__(/*! ./1_lexer/lexer */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/lexer.js");
 const statement_parser_1 = __webpack_require__(/*! ./2_statements/statement_parser */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statement_parser.js");
 const structure_parser_1 = __webpack_require__(/*! ./3_structures/structure_parser */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structure_parser.js");
-const abap_file_information_1 = __webpack_require__(/*! ./4_file_information/abap_file_information */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/abap_file_information.js");
-const abap_file_1 = __webpack_require__(/*! ./abap_file */ "./node_modules/@abaplint/core/build/src/abap/abap_file.js");
+const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class ABAPParser {
     constructor(version, globalMacros, reg) {
         this.version = version ? version : version_1.defaultVersion;
@@ -36689,7 +36737,9 @@ class ABAPParser {
         for (const f of statementResult) {
             const result = structure_parser_1.StructureParser.run(f);
             // 4: file information
-            const info = new abap_file_information_1.ABAPFileInformation(result.node, f.file.getFilename());
+            const parser = new abap_file_information_parser_1.ABAPFileInformationParser(f.file.getFilename());
+            const parsed = parser.parse(result.node);
+            const info = new abap_file_information_1.ABAPFileInformation(parsed);
             output.push(new abap_file_1.ABAPFile(f.file, f.tokens, f.statements, result.node, info));
             issues.push(...result.issues);
         }
@@ -48878,6 +48928,37 @@ exports.EcattTestScript = EcattTestScript;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/email_template.js":
+/*!*************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/email_template.js ***!
+  \*************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.EmailTemplate = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class EmailTemplate extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "SMTG";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 60, // todo
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.EmailTemplate = EmailTemplate;
+//# sourceMappingURL=email_template.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/enhancement_implementation.js":
 /*!*************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/enhancement_implementation.js ***!
@@ -49311,6 +49392,8 @@ class FunctionGroup extends _abap_object_1.ABAPObject {
         super.setDirty();
         this.includes = undefined;
         this.modules = undefined;
+        this.description = undefined;
+        this.dynpros = undefined;
     }
     getAllowedNaming() {
         return {
@@ -50167,7 +50250,6 @@ __exportStar(__webpack_require__(/*! ./behavior_definition */ "./node_modules/@a
 __exportStar(__webpack_require__(/*! ./brf_plus_system_application */ "./node_modules/@abaplint/core/build/src/objects/brf_plus_system_application.js"), exports);
 __exportStar(__webpack_require__(/*! ./bsp_application */ "./node_modules/@abaplint/core/build/src/objects/bsp_application.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_add_in_implementation */ "./node_modules/@abaplint/core/build/src/objects/business_add_in_implementation.js"), exports);
-__exportStar(__webpack_require__(/*! ./cds_entity_buffer */ "./node_modules/@abaplint/core/build/src/objects/cds_entity_buffer.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);
@@ -50176,6 +50258,7 @@ __exportStar(__webpack_require__(/*! ./business_function_assignment */ "./node_m
 __exportStar(__webpack_require__(/*! ./business_function_set_assignment */ "./node_modules/@abaplint/core/build/src/objects/business_function_set_assignment.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_object_model */ "./node_modules/@abaplint/core/build/src/objects/business_object_model.js"), exports);
 __exportStar(__webpack_require__(/*! ./business_object_type */ "./node_modules/@abaplint/core/build/src/objects/business_object_type.js"), exports);
+__exportStar(__webpack_require__(/*! ./cds_entity_buffer */ "./node_modules/@abaplint/core/build/src/objects/cds_entity_buffer.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_metadata_extension */ "./node_modules/@abaplint/core/build/src/objects/cds_metadata_extension.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_type */ "./node_modules/@abaplint/core/build/src/objects/cds_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./change_document */ "./node_modules/@abaplint/core/build/src/objects/change_document.js"), exports);
@@ -50201,6 +50284,7 @@ __exportStar(__webpack_require__(/*! ./domain */ "./node_modules/@abaplint/core/
 __exportStar(__webpack_require__(/*! ./ecatt_test_configuration */ "./node_modules/@abaplint/core/build/src/objects/ecatt_test_configuration.js"), exports);
 __exportStar(__webpack_require__(/*! ./ecatt_test_data_container */ "./node_modules/@abaplint/core/build/src/objects/ecatt_test_data_container.js"), exports);
 __exportStar(__webpack_require__(/*! ./ecatt_test_script */ "./node_modules/@abaplint/core/build/src/objects/ecatt_test_script.js"), exports);
+__exportStar(__webpack_require__(/*! ./email_template */ "./node_modules/@abaplint/core/build/src/objects/email_template.js"), exports);
 __exportStar(__webpack_require__(/*! ./enhancement_implementation */ "./node_modules/@abaplint/core/build/src/objects/enhancement_implementation.js"), exports);
 __exportStar(__webpack_require__(/*! ./enhancement_spot */ "./node_modules/@abaplint/core/build/src/objects/enhancement_spot.js"), exports);
 __exportStar(__webpack_require__(/*! ./entity_type */ "./node_modules/@abaplint/core/build/src/objects/entity_type.js"), exports);
@@ -53256,6 +53340,13 @@ class Table extends _abstract_object_1.AbstractObject {
         }
         return (_a = this.parsedData) === null || _a === void 0 ? void 0 : _a.description;
     }
+    getSecondaryIndexes() {
+        var _a;
+        if (this.parsedData === undefined) {
+            this.parseXML();
+        }
+        return (_a = this.parsedData) === null || _a === void 0 ? void 0 : _a.secondaryIndexes;
+    }
     getAllowedNaming() {
         let length = 30;
         const regex = /^((\/[A-Z_\d]{3,8}\/)|[a-zA-Z0-9]{3}|CI_)\w+$/;
@@ -53483,7 +53574,7 @@ class Table extends _abstract_object_1.AbstractObject {
     }
     ///////////////
     parseXML() {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
         const parsed = super.parseRaw2();
         if (parsed === undefined) {
             return;
@@ -53522,6 +53613,25 @@ class Table extends _abstract_object_1.AbstractObject {
                 DDTEXT: field.DDTEXT,
             });
         }
+        // secondary indexes
+        const indexes = (_m = parsed.abapGit["asx:abap"]["asx:values"]) === null || _m === void 0 ? void 0 : _m.DD12V;
+        ;
+        this.parsedData.secondaryIndexes = [];
+        for (const index of (0, xml_utils_1.xmlToArray)(indexes === null || indexes === void 0 ? void 0 : indexes.DD12V)) {
+            const indexName = index.INDEXNAME;
+            const indexFields = [];
+            const indexFieldsXml = (_o = parsed.abapGit["asx:abap"]["asx:values"]) === null || _o === void 0 ? void 0 : _o.DD17V;
+            for (const indexField of (0, xml_utils_1.xmlToArray)(indexFieldsXml === null || indexFieldsXml === void 0 ? void 0 : indexFieldsXml.DD17V)) {
+                if (indexField.INDEXNAME === indexName) {
+                    // assumption: fields are listed by POSITION in the xml
+                    indexFields.push(indexField.FIELDNAME);
+                }
+            }
+            this.parsedData.secondaryIndexes.push({
+                name: indexName,
+                fields: indexFields,
+            });
+        }
     }
 }
 exports.Table = Table;
@@ -54879,7 +54989,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.247";
+        return "2.114.1";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -57792,6 +57902,10 @@ const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/c
 const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const severity_1 = __webpack_require__(/*! ../severity */ "./node_modules/@abaplint/core/build/src/severity.js");
 class CheckIncludeConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.allowUnused = false;
+    }
 }
 exports.CheckIncludeConf = CheckIncludeConf;
 class CheckInclude {
@@ -57819,7 +57933,7 @@ class CheckInclude {
     }
     initialize(reg) {
         this.reg = reg;
-        this.graph = new include_graph_1.IncludeGraph(this.reg, this.getConfig().severity || severity_1.Severity.Error);
+        this.graph = new include_graph_1.IncludeGraph(this.reg, this.getConfig().severity || severity_1.Severity.Error, this.getConfig().allowUnused || false);
         return this;
     }
     run(obj) {
@@ -65785,8 +65899,7 @@ https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapstart-of-selectio
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `REPORT zfoo.
 WRITE 'hello'.`,
-            goodExample: `
-START-OF-SELECTION.
+            goodExample: `START-OF-SELECTION.
   WRITE 'hello'.`,
         };
     }
@@ -66239,6 +66352,7 @@ __exportStar(__webpack_require__(/*! ./implement_methods */ "./node_modules/@aba
 __exportStar(__webpack_require__(/*! ./implicit_start_of_selection */ "./node_modules/@abaplint/core/build/src/rules/implicit_start_of_selection.js"), exports);
 __exportStar(__webpack_require__(/*! ./in_statement_indentation */ "./node_modules/@abaplint/core/build/src/rules/in_statement_indentation.js"), exports);
 __exportStar(__webpack_require__(/*! ./indentation */ "./node_modules/@abaplint/core/build/src/rules/indentation.js"), exports);
+__exportStar(__webpack_require__(/*! ./index_completely_contained */ "./node_modules/@abaplint/core/build/src/rules/index_completely_contained.js"), exports);
 __exportStar(__webpack_require__(/*! ./inline_data_old_versions */ "./node_modules/@abaplint/core/build/src/rules/inline_data_old_versions.js"), exports);
 __exportStar(__webpack_require__(/*! ./intf_referencing_clas */ "./node_modules/@abaplint/core/build/src/rules/intf_referencing_clas.js"), exports);
 __exportStar(__webpack_require__(/*! ./invalid_table_index */ "./node_modules/@abaplint/core/build/src/rules/invalid_table_index.js"), exports);
@@ -66345,6 +66459,83 @@ __exportStar(__webpack_require__(/*! ./xml_consistency */ "./node_modules/@abapl
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/index_completely_contained.js":
+/*!***********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/index_completely_contained.js ***!
+  \***********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.IndexCompletelyContained = exports.IndexCompletelyContainedConf = void 0;
+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 issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
+const Objects = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+class IndexCompletelyContainedConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.IndexCompletelyContainedConf = IndexCompletelyContainedConf;
+class IndexCompletelyContained {
+    constructor() {
+        this.conf = new IndexCompletelyContainedConf();
+    }
+    getMetadata() {
+        return {
+            key: "index_completely_contained",
+            title: "Check if database table indexes are completely contained",
+            shortDescription: `If indexes are completely contained in other indexes, they can be removed to improve performance.`,
+            tags: [_irule_1.RuleTag.Performance, _irule_1.RuleTag.SingleFile],
+        };
+    }
+    initialize() {
+        return this;
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    run(obj) {
+        if (!(obj instanceof Objects.Table)) {
+            return [];
+        }
+        const indexes = obj.getSecondaryIndexes();
+        if (indexes === undefined || indexes.length === 0) {
+            return [];
+        }
+        const issues = [];
+        for (let i = 0; i < indexes.length; i++) {
+            const indexA = indexes[i];
+            for (let j = 0; j < indexes.length; j++) {
+                if (i === j) {
+                    continue;
+                }
+                const indexB = indexes[j];
+                let contained = true;
+                for (const field of indexA.fields) {
+                    if (indexB.fields.indexOf(field) === -1) {
+                        contained = false;
+                        break;
+                    }
+                }
+                if (contained) {
+                    const position = new position_1.Position(1, 1);
+                    const message = `Index "${indexA.name}" is completely contained in index "${indexB.name}"`;
+                    issues.push(issue_1.Issue.atPosition(obj.getFiles()[0], position, message, this.getMetadata().key, this.conf.severity));
+                }
+            }
+        }
+        return issues;
+    }
+}
+exports.IndexCompletelyContained = IndexCompletelyContained;
+//# sourceMappingURL=index_completely_contained.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/inline_data_old_versions.js":
 /*!*********************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/inline_data_old_versions.js ***!
@@ -78517,11 +78708,12 @@ class Graph {
     }
 }
 class IncludeGraph {
-    constructor(reg, severity = severity_1.Severity.Error) {
+    constructor(reg, severity = severity_1.Severity.Error, allowUnused = false) {
         this.reg = reg;
         this.issues = [];
         this.graph = new Graph();
         this.severity = severity;
+        this.allowUnused = allowUnused;
         this.build();
     }
     listMainForInclude(filename) {
@@ -78545,6 +78737,7 @@ class IncludeGraph {
         }
         const v = this.graph.findVertexByFilename(file.getFilename());
         if (v !== undefined
+            && this.allowUnused === false
             && v.include === true
             && this.listMainForInclude(v.filename).length === 0) {
             const f = this.reg.getFileByName(v.filename);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.247",
+    "version": "2.114.1",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.247",
+        "@abaplint/core": "^2.114.1",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",