@abaplint/cli 2.105.2 → 2.105.4

package/build/cli.js

@@ -11662,7 +11662,8 @@ class DeleteMemory {
     getMatcher() {
         const memory = (0, combi_1.seq)("MEMORY ID", expressions_1.Source);
         const id = (0, combi_1.seq)("ID", expressions_1.Source);
-        const shared = (0, combi_1.seq)("SHARED MEMORY", expressions_1.Field, "(", expressions_1.Field, ")", id);
+        const client = (0, combi_1.seq)("CLIENT", expressions_1.Source);
+        const shared = (0, combi_1.seq)("SHARED MEMORY", expressions_1.Field, "(", expressions_1.Field, ")", (0, combi_1.optPrio)(client), id);
         const ret = (0, combi_1.seq)("DELETE FROM", (0, combi_1.alt)(memory, shared));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
@@ -22548,6 +22549,9 @@ class TypeUtils {
                 }
                 return true;
             }
+            else if (target instanceof basic_1.CLikeType) {
+                return this.isCharLikeStrict(source);
+            }
             else if (target instanceof basic_1.VoidType || target instanceof basic_1.AnyType) {
                 return true;
             }
@@ -22582,6 +22586,8 @@ class TypeUtils {
         else if (source instanceof basic_1.ObjectReferenceType) {
             if (target instanceof basic_1.XSequenceType
                 || target instanceof basic_1.IntegerType
+                || target instanceof basic_1.StructureType
+                || target instanceof basic_1.TableType
                 || target instanceof basic_1.XStringType) {
                 return false;
             }
@@ -46695,6 +46701,29 @@ __exportStar(__webpack_require__(/*! ./message_class */ "./node_modules/@abaplin
 __exportStar(__webpack_require__(/*! ./messaging_channel */ "./node_modules/@abaplint/core/build/src/objects/messaging_channel.js"), exports);
 __exportStar(__webpack_require__(/*! ./mime_object */ "./node_modules/@abaplint/core/build/src/objects/mime_object.js"), exports);
 __exportStar(__webpack_require__(/*! ./namespace */ "./node_modules/@abaplint/core/build/src/objects/namespace.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_api */ "./node_modules/@abaplint/core/build/src/objects/neptune_api.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_app */ "./node_modules/@abaplint/core/build/src/objects/neptune_app.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_custom_color */ "./node_modules/@abaplint/core/build/src/objects/neptune_custom_color.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_custom_js_helper */ "./node_modules/@abaplint/core/build/src/objects/neptune_custom_js_helper.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_documentation */ "./node_modules/@abaplint/core/build/src/objects/neptune_documentation.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_enhancement */ "./node_modules/@abaplint/core/build/src/objects/neptune_enhancement.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_launchpad_layout */ "./node_modules/@abaplint/core/build/src/objects/neptune_launchpad_layout.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_launchpad */ "./node_modules/@abaplint/core/build/src/objects/neptune_launchpad.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_login_page */ "./node_modules/@abaplint/core/build/src/objects/neptune_login_page.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_media_library */ "./node_modules/@abaplint/core/build/src/objects/neptune_media_library.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_media_pack */ "./node_modules/@abaplint/core/build/src/objects/neptune_media_pack.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_metadata */ "./node_modules/@abaplint/core/build/src/objects/neptune_metadata.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_mobile_client */ "./node_modules/@abaplint/core/build/src/objects/neptune_mobile_client.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_odata */ "./node_modules/@abaplint/core/build/src/objects/neptune_odata.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_policy */ "./node_modules/@abaplint/core/build/src/objects/neptune_policy.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_rfc_mapping */ "./node_modules/@abaplint/core/build/src/objects/neptune_rfc_mapping.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_rich_text_template */ "./node_modules/@abaplint/core/build/src/objects/neptune_rich_text_template.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_splash_screen */ "./node_modules/@abaplint/core/build/src/objects/neptune_splash_screen.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_sticky_banner */ "./node_modules/@abaplint/core/build/src/objects/neptune_sticky_banner.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_tile_group */ "./node_modules/@abaplint/core/build/src/objects/neptune_tile_group.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_tile_layout */ "./node_modules/@abaplint/core/build/src/objects/neptune_tile_layout.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_tile */ "./node_modules/@abaplint/core/build/src/objects/neptune_tile.js"), exports);
+__exportStar(__webpack_require__(/*! ./neptune_url_alias */ "./node_modules/@abaplint/core/build/src/objects/neptune_url_alias.js"), exports);
 __exportStar(__webpack_require__(/*! ./number_range */ "./node_modules/@abaplint/core/build/src/objects/number_range.js"), exports);
 __exportStar(__webpack_require__(/*! ./oauth2_profile */ "./node_modules/@abaplint/core/build/src/objects/oauth2_profile.js"), exports);
 __exportStar(__webpack_require__(/*! ./object_characteristic */ "./node_modules/@abaplint/core/build/src/objects/object_characteristic.js"), exports);
@@ -46719,6 +46748,7 @@ __exportStar(__webpack_require__(/*! ./service_definition */ "./node_modules/@ab
 __exportStar(__webpack_require__(/*! ./shared_memory */ "./node_modules/@abaplint/core/build/src/objects/shared_memory.js"), exports);
 __exportStar(__webpack_require__(/*! ./smart_form */ "./node_modules/@abaplint/core/build/src/objects/smart_form.js"), exports);
 __exportStar(__webpack_require__(/*! ./smart_style */ "./node_modules/@abaplint/core/build/src/objects/smart_style.js"), exports);
+__exportStar(__webpack_require__(/*! ./standard_task */ "./node_modules/@abaplint/core/build/src/objects/standard_task.js"), exports);
 __exportStar(__webpack_require__(/*! ./switch_assignment_hierarchy */ "./node_modules/@abaplint/core/build/src/objects/switch_assignment_hierarchy.js"), exports);
 __exportStar(__webpack_require__(/*! ./switch_assignments */ "./node_modules/@abaplint/core/build/src/objects/switch_assignments.js"), exports);
 __exportStar(__webpack_require__(/*! ./table_type */ "./node_modules/@abaplint/core/build/src/objects/table_type.js"), exports);
@@ -46736,29 +46766,6 @@ __exportStar(__webpack_require__(/*! ./web_dynpro_application */ "./node_modules
 __exportStar(__webpack_require__(/*! ./web_dynpro_component_configuration */ "./node_modules/@abaplint/core/build/src/objects/web_dynpro_component_configuration.js"), exports);
 __exportStar(__webpack_require__(/*! ./web_dynpro_component */ "./node_modules/@abaplint/core/build/src/objects/web_dynpro_component.js"), exports);
 __exportStar(__webpack_require__(/*! ./web_mime */ "./node_modules/@abaplint/core/build/src/objects/web_mime.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_metadata */ "./node_modules/@abaplint/core/build/src/objects/neptune_metadata.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_app */ "./node_modules/@abaplint/core/build/src/objects/neptune_app.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_api */ "./node_modules/@abaplint/core/build/src/objects/neptune_api.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_launchpad */ "./node_modules/@abaplint/core/build/src/objects/neptune_launchpad.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_tile_group */ "./node_modules/@abaplint/core/build/src/objects/neptune_tile_group.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_tile */ "./node_modules/@abaplint/core/build/src/objects/neptune_tile.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_policy */ "./node_modules/@abaplint/core/build/src/objects/neptune_policy.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_launchpad_layout */ "./node_modules/@abaplint/core/build/src/objects/neptune_launchpad_layout.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_tile_layout */ "./node_modules/@abaplint/core/build/src/objects/neptune_tile_layout.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_enhancement */ "./node_modules/@abaplint/core/build/src/objects/neptune_enhancement.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_login_page */ "./node_modules/@abaplint/core/build/src/objects/neptune_login_page.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_sticky_banner */ "./node_modules/@abaplint/core/build/src/objects/neptune_sticky_banner.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_splash_screen */ "./node_modules/@abaplint/core/build/src/objects/neptune_splash_screen.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_custom_color */ "./node_modules/@abaplint/core/build/src/objects/neptune_custom_color.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_rich_text_template */ "./node_modules/@abaplint/core/build/src/objects/neptune_rich_text_template.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_custom_js_helper */ "./node_modules/@abaplint/core/build/src/objects/neptune_custom_js_helper.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_documentation */ "./node_modules/@abaplint/core/build/src/objects/neptune_documentation.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_rfc_mapping */ "./node_modules/@abaplint/core/build/src/objects/neptune_rfc_mapping.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_media_library */ "./node_modules/@abaplint/core/build/src/objects/neptune_media_library.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_media_pack */ "./node_modules/@abaplint/core/build/src/objects/neptune_media_pack.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_url_alias */ "./node_modules/@abaplint/core/build/src/objects/neptune_url_alias.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_odata */ "./node_modules/@abaplint/core/build/src/objects/neptune_odata.js"), exports);
-__exportStar(__webpack_require__(/*! ./neptune_mobile_client */ "./node_modules/@abaplint/core/build/src/objects/neptune_mobile_client.js"), exports);
 //# sourceMappingURL=index.js.map
 
 /***/ }),
@@ -49463,6 +49470,37 @@ exports.SmartStyle = SmartStyle;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/objects/standard_task.js":
+/*!************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/objects/standard_task.js ***!
+  \************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.StandardTask = void 0;
+const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node_modules/@abaplint/core/build/src/objects/_abstract_object.js");
+class StandardTask extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "PDTS";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 30,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.StandardTask = StandardTask;
+//# sourceMappingURL=standard_task.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/objects/switch_assignment_hierarchy.js":
 /*!**************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/objects/switch_assignment_hierarchy.js ***!
@@ -51042,7 +51080,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.2";
+        return "2.105.4";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52338,7 +52376,9 @@ STATICS: use CLASS-DATA instead
 
 DESCRIBE TABLE LINES: use lines() instead (quickfix exists)
 
-TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-test-seams-as-temporary-workaround`,
+TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-test-seams-as-temporary-workaround
+
+BREAK points`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
     }
@@ -64593,17 +64633,17 @@ class NoPrefixesConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
         /** DATA, CLASS-DATA, DATA BEGIN OF, CLASS-DATA BEGIN OF, FINAL(), DATA(), case insensitive regex */
-        this.data = "^[lg]._";
+        this.data = "^[lg].?_";
         /** STATICS, STATICS BEGIN OF, case insensitive regex */
         this.statics = "";
         /** FIELD-SYMBOLS and inline FIELD-SYMBOLS(), case insensitive regex */
-        this.fieldSymbols = "^<l._";
+        this.fieldSymbols = "^<l.?_";
         /** CONSTANTS, CONSTANTS BEGIN OF, case insensitive regex */
         this.constants = "^[lg]c_";
         /** TYPES, ENUM, MESH, case insensitive regex */
         this.types = "^ty_";
         /** importing, exporting, returning and changing parameters, case insensitive regex */
-        this.methodParameters = "^[ierc]._";
+        this.methodParameters = "^[ierc].?_";
         // todo, public localClass: string = "";
         // todo, public localInterface: string = "";
         // todo, public functionModuleParameters: string = "";
@@ -64625,6 +64665,9 @@ class NoPrefixes extends _abap_rule_1.ABAPRule {
             title: "No Prefixes",
             shortDescription: `Dont use hungarian notation`,
             extendedInformation: `
+Note: not prefixing TYPES will require changing the errorNamespace in the abaplint configuration,
+allowing all types to become voided, abaplint will then provide less precise syntax errors.
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-encodings-esp-hungarian-notation-and-prefixes
 
 https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodings.md`,
@@ -67430,6 +67473,7 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class ReduceProceduralCodeConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -67449,7 +67493,9 @@ class ReduceProceduralCode extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks FORM and FUNCTION-MODULE have few statements`,
             extendedInformation: `Delegate logic to a class method instead of using FORM or FUNCTION-MODULE.
 
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-object-orientation-to-procedural-programming`,
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-object-orientation-to-procedural-programming
+
+Comments are not counted as statements.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
             badExample: `FORM foo.
   DATA lv_bar TYPE i.
@@ -67495,6 +67541,9 @@ ENDFORM.`,
                 }
                 doCount = undefined;
             }
+            else if (statement.get() instanceof _statement_1.Comment) {
+                continue;
+            }
             else if (doCount !== undefined) {
                 count = count + 1;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.105.2",
+    "version": "2.105.4",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.105.2",
+        "@abaplint/core": "^2.105.4",
         "@types/chai": "^4.3.11",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.6",
-        "@types/node": "^20.10.4",
+        "@types/node": "^20.10.5",
         "@types/progress": "^2.0.7",
         "chai": "^4.3.10",
         "chalk": "^5.3.0",
-        "eslint": "^8.55.0",
+        "eslint": "^8.56.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
         "memfs": "^4.6.0",