@abaplint/cli 2.119.41 → 2.119.43

package/build/cli.js

@@ -29142,7 +29142,8 @@ class FieldChain {
                 if (allowGenericDeference === false
                     && (context === null || context === void 0 ? void 0 : context.isGeneric()) === true
                     && input.scope.getVersion() < version_1.Version.v756
-                    && input.scope.getVersion() !== version_1.Version.Cloud) {
+                    && input.scope.getVersion() !== version_1.Version.Cloud
+                    && input.scope.getVersion() !== version_1.Version.OpenABAP) {
                     throw new Error("A generic reference cannot be dereferenced");
                 }
             }
@@ -32154,9 +32155,23 @@ class Select {
             }
         }
     }
+    static countResultType(scope) {
+        if (scope.getVersion() >= version_1.Version.v750
+            || scope.getVersion() === version_1.Version.OpenABAP
+            || scope.getVersion() === version_1.Version.Cloud) {
+            return new basic_1.Integer8Type();
+        }
+        else {
+            return basic_1.IntegerType.get();
+        }
+    }
     static buildStructureType(fields, dbSources, scope) {
         var _a, _b, _c, _d, _e, _f;
         if (fields.length === 1 && dbSources.length === 1) {
+            // COUNT( * ) yields an integer regardless of the (possibly dynamic) table
+            if (/^count\(\s*\*\s*\)$/i.test(fields[0].code)) {
+                return this.countResultType(scope);
+            }
             const dbType = (_a = dbSources[0]) === null || _a === void 0 ? void 0 : _a.parseType(scope.getRegistry());
             if (dbType === undefined) {
                 const name = ((_b = dbSources[0]) === null || _b === void 0 ? void 0 : _b.getName()) || "buildStructureTypeError";
@@ -32176,16 +32191,6 @@ class Select {
                     if (field) {
                         return field;
                     }
-                    else if (fields[0].code === "COUNT(*)") {
-                        if (scope.getVersion() >= version_1.Version.v750
-                            || scope.getVersion() === version_1.Version.OpenABAP
-                            || scope.getVersion() === version_1.Version.Cloud) {
-                            return new basic_1.Integer8Type();
-                        }
-                        else {
-                            return basic_1.IntegerType.get();
-                        }
-                    }
                     else {
                         // todo: aggregated/calculated values
                         return basic_1.VoidType.get("SELECT_todo11");
@@ -66582,7 +66587,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.41";
+        return "2.119.43";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -96133,6 +96138,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 const minimist_1 = __importDefault(__webpack_require__(/*! minimist */ "./node_modules/minimist/index.js"));
+const core_1 = __webpack_require__(/*! @abaplint/core */ "../core/build/src/index.js");
 const _1 = __webpack_require__(/*! . */ "./build/src/index.js");
 const parsed = (0, minimist_1.default)(process.argv.slice(2), { boolean: ["p", "c", "fix", "rename"] });
 let format = "standard";
@@ -96160,9 +96166,12 @@ const arg = {
                 if (issues[0].getKey() === _1.GENERIC_ERROR) {
                     process.exit(2); // eg. "git" does not exist in system
                 }
-                else {
+                else if (issues.some(i => i.getSeverity() === core_1.Severity.Error)) {
                     process.exit(1);
                 }
+                else {
+                    process.exit(0); //only Warnings/ Infos
+                }
             }
             else {
                 process.exit();
@@ -96307,9 +96316,12 @@ class FileOperations {
         }
         fs.rmSync(dir, { recursive: true });
     }
+    /** Normalize backslashes to forward slashes, keeping the drive letter so absolute
+     * paths/glob patterns resolve on the correct drive (the path may be on a different
+     * drive than the current working directory). */
     static toUnixPath(path) {
         if (os.platform() === "win32") {
-            return path.replace(/[\\/]+/g, "/").replace(/^([a-zA-Z]+:|\.\/)/, "");
+            return path.replace(/[\\/]+/g, "/");
         }
         else {
             return path;
@@ -97355,14 +97367,13 @@ async function loadDependencies(config, compress, bar, base) {
         }
         if (d.url) {
             process.stderr.write("Clone: " + d.url + "\n");
-            let dir = fs.mkdtempSync(path.join(os.tmpdir(), "abaplint-"));
-            dir = file_operations_1.FileOperations.toUnixPath(dir);
+            const dir = fs.mkdtempSync(path.join(os.tmpdir(), "abaplint-"));
             let branch = "";
             if (d.branch) {
                 branch = "-b " + d.branch + " ";
             }
             childProcess.execSync("git clone --quiet --depth 1 " + branch + d.url + " .", { cwd: dir, stdio: "inherit" });
-            const names = file_operations_1.FileOperations.loadFileNames(dir + d.files);
+            const names = file_operations_1.FileOperations.loadFileNames(file_operations_1.FileOperations.toUnixPath(dir) + d.files);
             files = files.concat(await file_operations_1.FileOperations.loadFiles(compress, names, bar));
             file_operations_1.FileOperations.deleteFolderRecursive(dir);
         }
@@ -97589,8 +97600,8 @@ class Rename {
                 continue;
             }
             for (const f of o.getFiles()) {
-                // yea, ffs
-                const n = outputFolder + f.getFilename().replace(myBase, "").replace("//?/C:", "");
+                // strip the long-path prefix (Windows) then the drive-qualified base
+                const n = outputFolder + f.getFilename().replace("//?/", "").replace(myBase, "");
                 console.log("Write " + n);
                 fs.mkdirSync(path.dirname(n), { recursive: true });
                 fs.writeFileSync(n, f.getRaw());

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.119.41",
+    "version": "2.119.43",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -8,11 +8,12 @@
     },
     "scripts": {
         "lint": "eslint \"src/**/*.ts\" \"test/**/*.ts\"",
-        "compile": "tsc",
+        "clean": "rm -rf build",
+        "compile": "npm run clean && tsc",
         "test": "npm run compile && mocha && npm run lint && npm run webpack",
         "webpack": "webpack --progress",
-        "publish:minor": "npm --no-git-tag-version version minor && rm -rf build && npm install && npm run test && npm publish --access public",
-        "publish:patch": "npm --no-git-tag-version version patch && rm -rf build && npm install && npm run test && npm publish --access public"
+        "publish:minor": "npm --no-git-tag-version version minor && npm run clean && npm install && npm run test && npm publish --access public",
+        "publish:patch": "npm --no-git-tag-version version patch && npm run clean && npm install && npm run test && npm publish --access public"
     },
     "mocha": {
         "recursive": true,
@@ -38,7 +39,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.119.41",
+        "@abaplint/core": "^2.119.43",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",