@ts-dev-tools/core 1.9.7 → 1.9.8

package/dist/services/PackageManagerService.js

@@ -115,24 +115,47 @@ var PackageManagerService = /** @class */ (function () {
     };
     PackageManagerService.isPackageInstalled = function (packageName, dirPath) {
         return __awaiter(this, void 0, void 0, function () {
-            var packageManager, args, output, installedPackages;
+            var packageManager, args, output, error_1, installedPackages;
             var _a, _b;
             return __generator(this, function (_c) {
                 switch (_c.label) {
                     case 0:
                         packageManager = PackageManagerService.detectPackageManager(dirPath);
-                        args = [
-                            packageManager,
-                            "list",
-                            "--depth=1",
-                            "--json",
-                            "--no-progress",
-                            "--pattern=\"".concat(packageName, "\""),
-                            "--non-interactive",
-                        ];
-                        return [4 /*yield*/, PackageManagerService.execCommand(args, dirPath, true)];
+                        args = [packageManager, "list", "--depth=1", "--json"];
+                        switch (packageManager) {
+                            case PackageManagerType.yarn:
+                                args.push("--pattern=".concat(packageName));
+                                break;
+                            case PackageManagerType.npm:
+                                args.push("--no-progress", "--non-interactive", packageName);
+                                break;
+                        }
+                        _c.label = 1;
                     case 1:
+                        _c.trys.push([1, 3, , 4]);
+                        return [4 /*yield*/, PackageManagerService.execCommand(args, dirPath, true)];
+                    case 2:
                         output = _c.sent();
+                        return [3 /*break*/, 4];
+                    case 3:
+                        error_1 = _c.sent();
+                        // npm returns non-zero exit code when package is not found, but still outputs valid JSON
+                        if (typeof error_1 === "string") {
+                            try {
+                                // Try to parse as JSON - if successful, use it as output
+                                JSON.parse(error_1.trim());
+                                output = error_1;
+                            }
+                            catch (_d) {
+                                // If it's not valid JSON, the package is not installed
+                                return [2 /*return*/, false];
+                            }
+                        }
+                        else {
+                            return [2 /*return*/, false];
+                        }
+                        return [3 /*break*/, 4];
+                    case 4:
                         installedPackages = JSON.parse(output);
                         switch (packageManager) {
                             case PackageManagerType.yarn:
@@ -180,26 +203,53 @@ var PackageManagerService = /** @class */ (function () {
     };
     PackageManagerService.execCommand = function (args_1, cwd_1) {
         return __awaiter(this, arguments, void 0, function (args, cwd, silent) {
-            var cmd;
+            var cmd, cmdArgs, useShell, SHELL_OPERATORS_1, hasShellSyntax;
             if (silent === void 0) { silent = false; }
             return __generator(this, function (_a) {
-                if (!args.length) {
+                // Handle empty args check for both string and array
+                if (Array.isArray(args) && args.length === 0) {
+                    throw new Error("Command args must not be empty");
+                }
+                if (typeof args === "string" && args.trim().length === 0) {
                     throw new Error("Command args must not be empty");
                 }
                 if (cwd && !(0, fs_1.existsSync)(cwd)) {
                     throw new Error("Directory \"".concat(cwd, "\" does not exist"));
                 }
+                useShell = false;
                 if (Array.isArray(args)) {
-                    cmd = args.shift() || "";
+                    SHELL_OPERATORS_1 = [">", "|", "&&", "||", ";", "<", ">>", "2>", "&", "$("];
+                    hasShellSyntax = args.some(function (arg) {
+                        var trimmedArg = arg.trim();
+                        return SHELL_OPERATORS_1.some(function (op) {
+                            return trimmedArg.startsWith(op) ||
+                                trimmedArg.endsWith(op) ||
+                                trimmedArg.includes(" ".concat(op, " ")) ||
+                                trimmedArg.includes(" ".concat(op)) ||
+                                trimmedArg.includes("".concat(op, " "));
+                        });
+                    });
+                    if (hasShellSyntax) {
+                        // Use shell mode for commands with shell syntax
+                        cmd = args.join(" ").trim();
+                        cmdArgs = [];
+                        useShell = true;
+                    }
+                    else {
+                        // Use array mode for normal commands
+                        cmd = args[0];
+                        cmdArgs = args.slice(1);
+                    }
                 }
                 else {
                     cmd = args;
-                    args = [];
+                    cmdArgs = [];
+                    useShell = true;
                 }
                 return [2 /*return*/, new Promise(function (resolve, reject) {
-                        var child = (0, child_process_1.spawn)(cmd, args, {
+                        var child = (0, child_process_1.spawn)(cmd, cmdArgs, {
                             stdio: silent ? "pipe" : "inherit",
-                            shell: true,
+                            shell: useShell,
                             windowsVerbatimArguments: true,
                             cwd: cwd,
                         });
@@ -207,7 +257,9 @@ var PackageManagerService = /** @class */ (function () {
                         var error = "";
                         child.on("exit", function (code) {
                             if (code) {
-                                return reject(error);
+                                // For commands that output to stdout even on error (like npm list),
+                                // reject with the output so caller can handle it
+                                return reject(output.length > 0 ? output : error);
                             }
                             resolve(output);
                         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ts-dev-tools/core",
-  "version": "1.9.7",
+  "version": "1.9.8",
   "description": "TS dev tools Core",
   "keywords": [
     "linter",
@@ -36,7 +36,8 @@
     "test:unit": "npm run test -- --testPathPattern \".+spec\\.ts\"",
     "test:e2e": "npm run test -- --testPathPattern \".+e2e\\.spec\\.ts\"",
     "test:coverage": "npm run test -- --coverage",
-    "lint": "eslint \"src/**/*.{ts,tsx}\""
+    "lint": "eslint \"src/**/*.{ts,tsx}\"",
+    "lint:ci": "npm run lint -- --output-file eslint-report.json --format json"
   },
   "bugs": {
     "url": "https://github.com/escemi-tech/ts-dev-tools/issues"
@@ -45,7 +46,7 @@
     "@commitlint/cli": "^20.0.0",
     "@commitlint/config-conventional": "^20.0.0",
     "@eslint/js": "^9.25.0",
-    "@prettier/plugin-oxc": "^0.0.4",
+    "@prettier/plugin-oxc": "^0.0.5",
     "@types/jest": "^30.0.0",
     "@types/node": "^24.0.1",
     "eslint": "^9.25.0",
@@ -92,5 +93,5 @@
       "github-actions"
     ]
   },
-  "gitHead": "017cf83fb5426195d1feb11bcb97f2a173850f06"
+  "gitHead": "e49f690ba0ee84ba44a86c0631b4a2004bb77073"
 }