@abaplint/transpiler-cli 2.13.88 → 2.13.90

package/build/bundle.js

@@ -12875,7 +12875,7 @@ class CallTransaction {
     getMatcher() {
         const options = (0, combi_1.seq)("OPTIONS FROM", expressions_1.Source);
         const messages = (0, combi_1.seq)("MESSAGES INTO", expressions_1.Target);
-        const auth = (0, combi_1.ver)(version_1.Release.v740sp02, (0, combi_1.seq)((0, combi_1.altPrio)("WITH", "WITHOUT"), "AUTHORITY-CHECK"));
+        const auth = (0, combi_1.ver)(version_1.Release.v740sp02, (0, combi_1.seq)((0, combi_1.altPrio)("WITH", "WITHOUT"), "AUTHORITY-CHECK"), { also: combi_1.AlsoIn.OpenABAP });
         const perm = (0, combi_1.per)((0, combi_1.seq)("UPDATE", expressions_1.Source), "AND SKIP FIRST SCREEN", options, messages, (0, combi_1.seq)("MODE", expressions_1.Source));
         const ret = (0, combi_1.seq)("CALL TRANSACTION", expressions_1.Source, (0, combi_1.optPrio)(auth), (0, combi_1.optPrio)((0, combi_1.seq)("USING", expressions_1.Source)), (0, combi_1.opt)(perm));
         return (0, combi_1.verNotLang)(version_1.LanguageVersion.Cloud, ret);
@@ -71132,7 +71132,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.54";
+        return "2.120.56";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -72071,7 +72071,7 @@ class AlignParameters extends _abap_rule_1.ABAPRule {
 * CREATE OBJECT statements
 * RAISE EVENT statements
 
-https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#align-parameters
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#align-parameters
 
 Does not take effect on non functional method calls, use https://rules.abaplint.org/functional_writing/
 
@@ -76905,7 +76905,7 @@ class ConstantClasses {
             key: "constant_classes",
             title: "Validate constant classes",
             shortDescription: `Checks that a class contains exactly the constants corresponding to a domain's fixed values.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-enumeration-classes-to-constants-interfaces`,
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-enum-to-constants-interfaces`,
             tags: [_irule_1.RuleTag.Styleguide],
         };
     }
@@ -81096,6 +81096,10 @@ ${indentation}    output = ${uniqueName}.\n`;
         if (!(high.get() instanceof Statements.CallTransaction)) {
             return undefined;
         }
+        else if (this.lowReg.getConfig().isOpenABAP()) {
+            // open-abap supports the addition
+            return undefined;
+        }
         const tokens = high.getTokens();
         // the lexer splits "AUTHORITY-CHECK" into three tokens: AUTHORITY, -, CHECK
         for (let i = 1; i < tokens.length - 3; i++) {
@@ -85523,6 +85527,7 @@ __exportStar(__webpack_require__(/*! ./prefer_inline */ "./node_modules/@abaplin
 __exportStar(__webpack_require__(/*! ./prefer_insert_into_table */ "./node_modules/@abaplint/core/build/src/rules/prefer_insert_into_table.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_is_not */ "./node_modules/@abaplint/core/build/src/rules/prefer_is_not.js"), exports);
 __exportStar(__webpack_require__(/*! ./prefer_pragmas */ "./node_modules/@abaplint/core/build/src/rules/prefer_pragmas.js"), exports);
+__exportStar(__webpack_require__(/*! ./prefer_abap_bool */ "./node_modules/@abaplint/core/build/src/rules/prefer_abap_bool.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_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);
@@ -86336,7 +86341,7 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
             key: "keyword_case",
             title: "Keyword case",
             shortDescription: `Checks that keywords have the same case. Non-keywords must be lower case.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-your-pretty-printer-team-settings`,
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-your-teams-abap-formatter-settings`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `write 'hello world'.`,
             goodExample: `WRITE 'hello world'.`,
@@ -92816,6 +92821,116 @@ exports.PragmaStyle = PragmaStyle;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/prefer_abap_bool.js"
+/*!*************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/prefer_abap_bool.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.PreferAbapBool = exports.PreferAbapBoolConf = 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 Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/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");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const FORBIDDEN_TYPES = new Set(["XFELD", "SAP_BOOL", "BOOLE_D", "FLAG"]);
+class PreferAbapBoolConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.PreferAbapBoolConf = PreferAbapBoolConf;
+class PreferAbapBool extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new PreferAbapBoolConf();
+    }
+    getMetadata() {
+        return {
+            key: "prefer_abap_bool",
+            title: "Prefer abap_bool",
+            shortDescription: `Use abap_bool instead of XFELD, SAP_BOOL, BOOLE_D, FLAG for boolean variables`,
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-abap_bool-for-booleans`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
+            badExample: `DATA foo TYPE xfeld.`,
+            goodExample: `DATA foo TYPE abap_bool.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const structure = file.getStructure();
+        if (structure === undefined) {
+            return [];
+        }
+        const statements = [
+            ...structure.findAllStatements(Statements.Data),
+            ...structure.findAllStatements(Statements.ClassData),
+            ...structure.findAllStatements(Statements.Constant),
+            ...structure.findAllStatements(Statements.Type),
+            ...structure.findAllStatements(Statements.FieldSymbol),
+        ];
+        for (const stat of statements) {
+            const typeNameExpr = stat.findFirstExpression(Expressions.TypeName);
+            if (typeNameExpr === undefined) {
+                continue;
+            }
+            const typeName = typeNameExpr.concatTokens().toUpperCase();
+            if (FORBIDDEN_TYPES.has(typeName) === false) {
+                continue;
+            }
+            const token = typeNameExpr.getFirstToken();
+            const fix = edit_helper_1.EditHelper.replaceToken(file, token, "abap_bool");
+            issues.push(issue_1.Issue.atToken(file, token, `Use abap_bool instead of ${typeName}`, this.getMetadata().key, this.conf.severity, fix));
+        }
+        return issues;
+    }
+}
+exports.PreferAbapBool = PreferAbapBool;
+//# sourceMappingURL=prefer_abap_bool.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/prefer_corresponding.js"
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/prefer_corresponding.js ***!
@@ -93262,9 +93377,8 @@ class PreferInsertIntoTable extends _abap_rule_1.ABAPRule {
             key: "prefer_insert_into_table",
             title: "Prefer INSERT INTO TABLE over APPEND",
             shortDescription: `Prefer INSERT INTO TABLE over APPEND`,
-            extendedInformation: `INSERT INTO TABLE respects the table type, while APPEND always adds to the end regardless of table type.
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md`,
+            // eslint-disable-next-line max-len
+            extendedInformation: `INSERT INTO TABLE respects the table type, while APPEND always tries to add to the end. This will dump if the sort order of a sorted table would change.\n\nhttps://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-insert-into-table-to-append-to`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
             badExample: `APPEND row TO itab.`,
             goodExample: `INSERT row INTO TABLE itab.`,
@@ -95958,6 +96072,8 @@ class SequentialBlank extends _abap_rule_1.ABAPRule {
             key: "sequential_blank",
             title: "Sequential blank lines",
             shortDescription: `Checks that code does not contain more than the configured number of blank lines in a row.`,
+            // eslint-disable-next-line max-len
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#add-a-single-blank-line-to-separate-things-but-not-more`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
         };
     }
@@ -103436,7 +103552,14 @@ class DatabaseSetup {
         return {
             schemas: {
                 sqlite: this.driver(new sqlite_database_schema_1.SQLiteDatabaseSchema(this.reg)),
-                hdb: ["todo"],
+                // HANA takes the PG schema unchanged -- measured against HANA Express
+                // on a tree of 77 tables. The one thing it needs is that identifiers
+                // reach it quoted in UPPER case, and that is not a dialect of DDL: the
+                // same rule applies to every statement, so @abaplint/database-hdb
+                // folds them on the way out rather than a second generator doing it
+                // here. (Unquoted would nearly work and then meet a column called
+                // `cross`, which is one of HANA's reserved words.)
+                hdb: this.driver(new pg_database_schema_1.PGDatabaseSchema(this.reg)),
                 pg: this.driver(new pg_database_schema_1.PGDatabaseSchema(this.reg)),
                 snowflake: this.driver(new snowflake_database_schema_1.SnowflakeDatabaseSchema(this.reg)),
             },
@@ -103591,7 +103714,7 @@ class PopulateTables {
         if (raw === undefined) {
             return "";
         }
-        return `INSERT INTO reposrc ('PROGNAME', 'DATA') VALUES ('${name.padEnd(40, " ")}', '${this.escape(raw)}');`;
+        return `INSERT INTO reposrc (progname, data) VALUES ('${name.padEnd(40, " ")}', '${this.escape(raw)}');`;
     }
     insertT100(msag) {
         // ignore if T100 is unknown

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.13.88",
+  "version": "2.13.90",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,17 +27,17 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.120.54",
-    "@abaplint/transpiler": "^2.13.88",
+    "@abaplint/core": "^2.120.56",
+    "@abaplint/transpiler": "^2.13.90",
     "@types/glob": "^9.0.0",
-    "@types/node": "^26.4.1",
+    "@types/node": "^26.6.2",
     "@types/progress": "^2.0.7",
     "glob": "^13.0.6",
     "p-limit": "^7.3.2",
     "progress": "^2.0.3",
     "ts-json-schema-generator": "^2.9.0",
     "typescript": "^6.0.3",
-    "webpack": "^5.110.3",
+    "webpack": "^5.111.1",
     "webpack-cli": "^7.2.3"
   }
 }