@serviceme/devtools-core 0.1.7 → 0.1.9

package/dist/index.d.mts

@@ -109,7 +109,39 @@ declare function createCopilotAuthRequiredError(): _serviceme_devtools_protocol.
 
 declare function copilotPrompt(options: CopilotPromptOptions): Promise<CopilotPromptResult>;
 
+interface RunCommandOptions {
+    args?: string[];
+    cwd?: string;
+    env?: NodeJS.ProcessEnv;
+    timeoutMs?: number;
+    stdin?: string;
+    shell?: boolean;
+}
+interface RunCommandResult {
+    stdout: string;
+    stderr: string;
+    code: number;
+}
+declare function runCommand(command: string, options?: RunCommandOptions): Promise<RunCommandResult>;
+
+interface EnvironmentInspectorOptions {
+    /**
+     * Override the command runner. Production callers should leave this
+     * undefined and rely on the default `runCommand` from `../process/runCommand`;
+     * tests inject a fake to drive error/edge branches without spawning
+     * real subprocesses.
+     */
+    runCommand?: typeof runCommand;
+    /**
+     * Override the detected platform. Defaults to `process.platform`. Tests
+     * use this to exercise the Windows-specific branches of the inspector.
+     */
+    platform?: NodeJS.Platform;
+}
 declare class EnvironmentInspector {
+    private readonly runCommandFn;
+    private readonly platform;
+    constructor(options?: EnvironmentInspectorOptions);
     checkEnvironment(): Promise<EnvironmentCheckResult>;
     checkTool(toolName: KnownEnvironmentTool): Promise<ToolCheckResult>;
     private getToolPath;
@@ -194,38 +226,6 @@ declare class PidManager {
     getRunningPid(): number | null;
 }
 
-declare class SchedulerDaemon {
-    private readonly workspacePath;
-    private readonly configManager;
-    private readonly logManager;
-    private readonly pidManager;
-    private readonly logger;
-    private tickTimer;
-    private watcher;
-    private running;
-    private startTime;
-    private lastRun;
-    private taskRunning;
-    constructor(workspacePath: string);
-    start(): void;
-    stop(): void;
-    private setupConfigWatch;
-    private tick;
-    private shouldRun;
-    private executeTask;
-    getStatus(): {
-        running: boolean;
-        pid: number;
-        uptimeSeconds: number | null;
-        tasksRegistered: number;
-        tasksEnabled: number;
-        workspacePath: string;
-        pidFile: string;
-    };
-}
-declare function parseIntervalMs(schedule: string): number;
-declare function matchesCron(expression: string, date: Date): boolean;
-
 interface ExecutorResult {
     status: TaskExecutionStatus;
     output?: string;
@@ -260,6 +260,48 @@ declare class ShellExecutor implements StreamingTaskExecutor {
 
 declare function getExecutor(taskType: ScheduledTaskType): TaskExecutor;
 
+interface SchedulerDaemonOptions {
+    /**
+     * Override the executor factory. Production callers should leave this
+     * undefined and rely on the default `getExecutor` from `../executors`;
+     * tests inject a fake to drive the execution branch without spawning
+     * real subprocesses.
+     */
+    getExecutor?: typeof getExecutor;
+}
+declare class SchedulerDaemon {
+    private readonly workspacePath;
+    private readonly configManager;
+    private readonly logManager;
+    private readonly pidManager;
+    private readonly logger;
+    private readonly getExecutor;
+    private tickTimer;
+    private watcher;
+    private running;
+    private startTime;
+    private lastRun;
+    private taskRunning;
+    constructor(workspacePath: string, options?: SchedulerDaemonOptions);
+    start(): void;
+    stop(): void;
+    private setupConfigWatch;
+    private tick;
+    private shouldRun;
+    private executeTask;
+    getStatus(): {
+        running: boolean;
+        pid: number;
+        uptimeSeconds: number | null;
+        tasksRegistered: number;
+        tasksEnabled: number;
+        workspacePath: string;
+        pidFile: string;
+    };
+}
+declare function parseIntervalMs(schedule: string): number;
+declare function matchesCron(expression: string, date: Date): boolean;
+
 declare function validateTaskPayload(taskType: ScheduledTaskType, payload: TaskPayload): void;
 interface CreateTaskInput {
     name: string;
@@ -429,4 +471,4 @@ declare class SkillStore {
 declare const unzipFile: (zipPath: string, dest: string) => Promise<void>;
 declare const moveFiles: (sourceDir: string, destDir: string, overwrite?: boolean) => Promise<void>;
 
-export { type AgentCatalog, AgentCatalogClient, type AgentCatalogClientOptions, type AgentDownloadFile, type AgentInstallScope, type AgentMutateResult, AgentReconciler, type AgentReconcilerDependencies, AgentStore, type AgentStoreFileSystem, type AgentStoreOptions, type AgentsStateFile, type AppendLogInput, type CreateTaskInput, DaemonLogger, type EditTaskInput, EnvironmentInspector, type ExecutorResult, GithubCopilotCliExecutor, HttpRequestExecutor, ImageTools, type InstalledAgent, type JsonTools, type OutputEventCallback, PidManager, ProjectTools, SchedulerDaemon, type ServiceMeLogger, ShellExecutor, type SkillCatalog, SkillCatalogClient, type SkillCatalogClientOptions, type SkillDownloadFile, type SkillMutateResult, SkillReconciler, type SkillReconcilerDependencies, SkillStore, type SkillStoreFileSystem, type SkillStoreOptions, type StreamingExecutorHandle, type StreamingTaskExecutor, TOOL_RISK_MAP, TaskConfigManager, type TaskEventListener, TaskExecutionEngine, type TaskExecutor, TaskLogManager, copilotDoctor, copilotPrompt, createConsoleLogger, createCopilotAuthRequiredError, createCopilotNotInstalledError, createImageTools, createJsonTools, createProjectTools, getExecutor, isCopilotAuthenticated, isStreamingTaskExecutor, matchesCron, moveFiles, noopLogger, parseAgentToolPermissions, parseIntervalMs, resolveTaskExecutionPayload, unzipFile, validateTaskPayload };
+export { type AgentCatalog, AgentCatalogClient, type AgentCatalogClientOptions, type AgentDownloadFile, type AgentInstallScope, type AgentMutateResult, AgentReconciler, type AgentReconcilerDependencies, AgentStore, type AgentStoreFileSystem, type AgentStoreOptions, type AgentsStateFile, type AppendLogInput, type CreateTaskInput, DaemonLogger, type EditTaskInput, EnvironmentInspector, type EnvironmentInspectorOptions, type ExecutorResult, GithubCopilotCliExecutor, HttpRequestExecutor, ImageTools, type InstalledAgent, type JsonTools, type OutputEventCallback, PidManager, ProjectTools, SchedulerDaemon, type ServiceMeLogger, ShellExecutor, type SkillCatalog, SkillCatalogClient, type SkillCatalogClientOptions, type SkillDownloadFile, type SkillMutateResult, SkillReconciler, type SkillReconcilerDependencies, SkillStore, type SkillStoreFileSystem, type SkillStoreOptions, type StreamingExecutorHandle, type StreamingTaskExecutor, TOOL_RISK_MAP, TaskConfigManager, type TaskEventListener, TaskExecutionEngine, type TaskExecutor, TaskLogManager, copilotDoctor, copilotPrompt, createConsoleLogger, createCopilotAuthRequiredError, createCopilotNotInstalledError, createImageTools, createJsonTools, createProjectTools, getExecutor, isCopilotAuthenticated, isStreamingTaskExecutor, matchesCron, moveFiles, noopLogger, parseAgentToolPermissions, parseIntervalMs, resolveTaskExecutionPayload, unzipFile, validateTaskPayload };

package/dist/index.d.ts

@@ -109,7 +109,39 @@ declare function createCopilotAuthRequiredError(): _serviceme_devtools_protocol.
 
 declare function copilotPrompt(options: CopilotPromptOptions): Promise<CopilotPromptResult>;
 
+interface RunCommandOptions {
+    args?: string[];
+    cwd?: string;
+    env?: NodeJS.ProcessEnv;
+    timeoutMs?: number;
+    stdin?: string;
+    shell?: boolean;
+}
+interface RunCommandResult {
+    stdout: string;
+    stderr: string;
+    code: number;
+}
+declare function runCommand(command: string, options?: RunCommandOptions): Promise<RunCommandResult>;
+
+interface EnvironmentInspectorOptions {
+    /**
+     * Override the command runner. Production callers should leave this
+     * undefined and rely on the default `runCommand` from `../process/runCommand`;
+     * tests inject a fake to drive error/edge branches without spawning
+     * real subprocesses.
+     */
+    runCommand?: typeof runCommand;
+    /**
+     * Override the detected platform. Defaults to `process.platform`. Tests
+     * use this to exercise the Windows-specific branches of the inspector.
+     */
+    platform?: NodeJS.Platform;
+}
 declare class EnvironmentInspector {
+    private readonly runCommandFn;
+    private readonly platform;
+    constructor(options?: EnvironmentInspectorOptions);
     checkEnvironment(): Promise<EnvironmentCheckResult>;
     checkTool(toolName: KnownEnvironmentTool): Promise<ToolCheckResult>;
     private getToolPath;
@@ -194,38 +226,6 @@ declare class PidManager {
     getRunningPid(): number | null;
 }
 
-declare class SchedulerDaemon {
-    private readonly workspacePath;
-    private readonly configManager;
-    private readonly logManager;
-    private readonly pidManager;
-    private readonly logger;
-    private tickTimer;
-    private watcher;
-    private running;
-    private startTime;
-    private lastRun;
-    private taskRunning;
-    constructor(workspacePath: string);
-    start(): void;
-    stop(): void;
-    private setupConfigWatch;
-    private tick;
-    private shouldRun;
-    private executeTask;
-    getStatus(): {
-        running: boolean;
-        pid: number;
-        uptimeSeconds: number | null;
-        tasksRegistered: number;
-        tasksEnabled: number;
-        workspacePath: string;
-        pidFile: string;
-    };
-}
-declare function parseIntervalMs(schedule: string): number;
-declare function matchesCron(expression: string, date: Date): boolean;
-
 interface ExecutorResult {
     status: TaskExecutionStatus;
     output?: string;
@@ -260,6 +260,48 @@ declare class ShellExecutor implements StreamingTaskExecutor {
 
 declare function getExecutor(taskType: ScheduledTaskType): TaskExecutor;
 
+interface SchedulerDaemonOptions {
+    /**
+     * Override the executor factory. Production callers should leave this
+     * undefined and rely on the default `getExecutor` from `../executors`;
+     * tests inject a fake to drive the execution branch without spawning
+     * real subprocesses.
+     */
+    getExecutor?: typeof getExecutor;
+}
+declare class SchedulerDaemon {
+    private readonly workspacePath;
+    private readonly configManager;
+    private readonly logManager;
+    private readonly pidManager;
+    private readonly logger;
+    private readonly getExecutor;
+    private tickTimer;
+    private watcher;
+    private running;
+    private startTime;
+    private lastRun;
+    private taskRunning;
+    constructor(workspacePath: string, options?: SchedulerDaemonOptions);
+    start(): void;
+    stop(): void;
+    private setupConfigWatch;
+    private tick;
+    private shouldRun;
+    private executeTask;
+    getStatus(): {
+        running: boolean;
+        pid: number;
+        uptimeSeconds: number | null;
+        tasksRegistered: number;
+        tasksEnabled: number;
+        workspacePath: string;
+        pidFile: string;
+    };
+}
+declare function parseIntervalMs(schedule: string): number;
+declare function matchesCron(expression: string, date: Date): boolean;
+
 declare function validateTaskPayload(taskType: ScheduledTaskType, payload: TaskPayload): void;
 interface CreateTaskInput {
     name: string;
@@ -429,4 +471,4 @@ declare class SkillStore {
 declare const unzipFile: (zipPath: string, dest: string) => Promise<void>;
 declare const moveFiles: (sourceDir: string, destDir: string, overwrite?: boolean) => Promise<void>;
 
-export { type AgentCatalog, AgentCatalogClient, type AgentCatalogClientOptions, type AgentDownloadFile, type AgentInstallScope, type AgentMutateResult, AgentReconciler, type AgentReconcilerDependencies, AgentStore, type AgentStoreFileSystem, type AgentStoreOptions, type AgentsStateFile, type AppendLogInput, type CreateTaskInput, DaemonLogger, type EditTaskInput, EnvironmentInspector, type ExecutorResult, GithubCopilotCliExecutor, HttpRequestExecutor, ImageTools, type InstalledAgent, type JsonTools, type OutputEventCallback, PidManager, ProjectTools, SchedulerDaemon, type ServiceMeLogger, ShellExecutor, type SkillCatalog, SkillCatalogClient, type SkillCatalogClientOptions, type SkillDownloadFile, type SkillMutateResult, SkillReconciler, type SkillReconcilerDependencies, SkillStore, type SkillStoreFileSystem, type SkillStoreOptions, type StreamingExecutorHandle, type StreamingTaskExecutor, TOOL_RISK_MAP, TaskConfigManager, type TaskEventListener, TaskExecutionEngine, type TaskExecutor, TaskLogManager, copilotDoctor, copilotPrompt, createConsoleLogger, createCopilotAuthRequiredError, createCopilotNotInstalledError, createImageTools, createJsonTools, createProjectTools, getExecutor, isCopilotAuthenticated, isStreamingTaskExecutor, matchesCron, moveFiles, noopLogger, parseAgentToolPermissions, parseIntervalMs, resolveTaskExecutionPayload, unzipFile, validateTaskPayload };
+export { type AgentCatalog, AgentCatalogClient, type AgentCatalogClientOptions, type AgentDownloadFile, type AgentInstallScope, type AgentMutateResult, AgentReconciler, type AgentReconcilerDependencies, AgentStore, type AgentStoreFileSystem, type AgentStoreOptions, type AgentsStateFile, type AppendLogInput, type CreateTaskInput, DaemonLogger, type EditTaskInput, EnvironmentInspector, type EnvironmentInspectorOptions, type ExecutorResult, GithubCopilotCliExecutor, HttpRequestExecutor, ImageTools, type InstalledAgent, type JsonTools, type OutputEventCallback, PidManager, ProjectTools, SchedulerDaemon, type ServiceMeLogger, ShellExecutor, type SkillCatalog, SkillCatalogClient, type SkillCatalogClientOptions, type SkillDownloadFile, type SkillMutateResult, SkillReconciler, type SkillReconcilerDependencies, SkillStore, type SkillStoreFileSystem, type SkillStoreOptions, type StreamingExecutorHandle, type StreamingTaskExecutor, TOOL_RISK_MAP, TaskConfigManager, type TaskEventListener, TaskExecutionEngine, type TaskExecutor, TaskLogManager, copilotDoctor, copilotPrompt, createConsoleLogger, createCopilotAuthRequiredError, createCopilotNotInstalledError, createImageTools, createJsonTools, createProjectTools, getExecutor, isCopilotAuthenticated, isStreamingTaskExecutor, matchesCron, moveFiles, noopLogger, parseAgentToolPermissions, parseIntervalMs, resolveTaskExecutionPayload, unzipFile, validateTaskPayload };

package/dist/index.js

@@ -80,10 +80,10 @@ var AgentCatalogClient = class {
   }
   async getCatalog() {
     if (!this.baseUrl) {
-      return {
-        agents: [],
-        fetchedAt: (/* @__PURE__ */ new Date()).toISOString()
-      };
+      throw (0, import_devtools_protocol.createServicemeError)(
+        "workspace_not_found",
+        "Agent catalog baseUrl is not configured."
+      );
     }
     const response = await this.fetchImpl(
       `${this.baseUrl}/api/v1/marketplace/agents`
@@ -577,6 +577,9 @@ async function copilotPrompt(options) {
     }
     return { output, exitCode: 0 };
   } catch (error) {
+    if (error instanceof import_devtools_protocol3.ServicemeProtocolError) {
+      throw error;
+    }
     const err = error;
     if (err.code === "ETIMEDOUT") {
       throw (0, import_devtools_protocol3.createServicemeError)(
@@ -622,11 +625,13 @@ var TOOL_CHECK_TIMEOUT_MS = {
 var ERROR_CODE_NOT_FOUND = 127;
 var ERROR_CODE_TIMEOUT = "ETIMEDOUT";
 var EnvironmentInspector = class {
+  constructor(options = {}) {
+    this.runCommandFn = options.runCommand ?? runCommand;
+    this.platform = options.platform ?? process.platform;
+  }
   async checkEnvironment() {
     const results = await Promise.all(
-      import_devtools_protocol4.KNOWN_ENVIRONMENT_TOOLS.map(
-        async (tool) => [tool, await this.checkTool(tool)]
-      )
+      import_devtools_protocol4.KNOWN_ENVIRONMENT_TOOLS.map(async (tool) => [tool, await this.checkTool(tool)])
     );
     return Object.fromEntries(results);
   }
@@ -654,8 +659,8 @@ var EnvironmentInspector = class {
   }
   async getToolPath(toolName) {
     try {
-      const isWindows = process.platform === "win32";
-      const result = await runCommand(isWindows ? "where" : "which", {
+      const isWindows = this.platform === "win32";
+      const result = await this.runCommandFn(isWindows ? "where" : "which", {
         args: [toolName],
         timeoutMs: this.getToolTimeout(toolName)
       });
@@ -672,14 +677,12 @@ var EnvironmentInspector = class {
    */
   async getToolShimPath(toolName) {
     try {
-      const result = await runCommand("where", {
+      const result = await this.runCommandFn("where", {
         args: [toolName],
         timeoutMs: this.getToolTimeout(toolName)
       });
       const candidates = result.stdout.split(/\r?\n/).map((line) => line.trim()).filter((line) => line.length > 0);
-      const cmdShim = candidates.find(
-        (line) => line.toLowerCase().endsWith(".cmd")
-      );
+      const cmdShim = candidates.find((line) => line.toLowerCase().endsWith(".cmd"));
       return cmdShim ?? candidates[0];
     } catch {
       return void 0;
@@ -687,16 +690,16 @@ var EnvironmentInspector = class {
   }
   async getToolVersion(toolName) {
     const invocation = await this.getVersionInvocation(toolName);
-    const result = await runCommand(invocation.command, {
+    const result = await this.runCommandFn(invocation.command, {
       args: invocation.args,
       timeoutMs: this.getToolTimeout(toolName)
     });
     return this.parseVersion(toolName, result.stdout || result.stderr);
   }
   async checkNvm() {
-    if (process.platform === "win32") {
+    if (this.platform === "win32") {
       try {
-        const result = await runCommand("cmd.exe", {
+        const result = await this.runCommandFn("cmd.exe", {
           args: ["/c", "nvm version"],
           timeoutMs: this.getToolTimeout("nvm")
         });
@@ -713,7 +716,7 @@ var EnvironmentInspector = class {
       }
     }
     try {
-      const result = await runCommand("/bin/bash", {
+      const result = await this.runCommandFn("/bin/bash", {
         args: [
           "-lc",
           'export NVM_DIR="$HOME/.nvm"; [ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"; nvm --version'
@@ -741,7 +744,7 @@ var EnvironmentInspector = class {
       };
     }
     try {
-      const result = await runCommand("dotnet", {
+      const result = await this.runCommandFn("dotnet", {
         args: ["nuget", "list", "source"],
         timeoutMs: this.getToolTimeout("nuget")
       });
@@ -762,7 +765,7 @@ var EnvironmentInspector = class {
     }
   }
   async getVersionInvocation(toolName) {
-    const isWindows = process.platform === "win32";
+    const isWindows = this.platform === "win32";
     if (isWindows && ["npm", "pnpm", "nrm"].includes(toolName)) {
       const shim = await this.getToolShimPath(toolName);
       if (shim) {
@@ -1100,52 +1103,45 @@ var import_devtools_protocol7 = require("@serviceme/devtools-protocol");
 var import_node_fs = require("fs");
 var import_promises = require("fs/promises");
 var import_node_path = require("path");
-var import_yauzl = require("yauzl");
+var import_yauzl = __toESM(require("yauzl"));
 var unzipFile = (zipPath, dest) => {
   return new Promise((resolve, reject) => {
-    (0, import_yauzl.open)(
-      zipPath,
-      { lazyEntries: true },
-      (err, zipfile) => {
-        if (err) return reject(err);
-        if (!zipfile) return reject(new Error("Failed to open zip file."));
-        zipfile.readEntry();
-        zipfile.on("entry", (entry) => {
-          if (/\/$/.test(entry.fileName)) {
-            void (0, import_promises.mkdir)((0, import_node_path.join)(dest, entry.fileName), { recursive: true }).then(() => {
-              zipfile.readEntry();
-            }).catch(reject);
-          } else {
-            const outputPath = (0, import_node_path.join)(dest, entry.fileName);
-            void (0, import_promises.mkdir)((0, import_node_path.dirname)(outputPath), { recursive: true }).then(() => {
-              zipfile.openReadStream(
-                entry,
-                (streamError, readStream) => {
-                  if (streamError) return reject(streamError);
-                  if (!readStream)
-                    return reject(
-                      new Error("Failed to open zip entry stream.")
-                    );
-                  const writeStream = (0, import_node_fs.createWriteStream)(outputPath);
-                  readStream.on("error", reject);
-                  writeStream.on("error", reject);
-                  writeStream.on("close", () => {
-                    zipfile.readEntry();
-                  });
-                  readStream.pipe(writeStream);
-                }
-              );
-            }).catch(reject);
-          }
-        });
-        zipfile.on("end", () => {
-          resolve();
-        });
-        zipfile.on("error", (zipError) => {
-          reject(zipError);
-        });
-      }
-    );
+    import_yauzl.default.open(zipPath, { lazyEntries: true }, (err, zipfile) => {
+      if (err) return reject(err);
+      if (!zipfile) return reject(new Error("Failed to open zip file."));
+      zipfile.readEntry();
+      zipfile.on("entry", (entry) => {
+        if (/\/$/.test(entry.fileName)) {
+          void (0, import_promises.mkdir)((0, import_node_path.join)(dest, entry.fileName), { recursive: true }).then(() => {
+            zipfile.readEntry();
+          }).catch(reject);
+        } else {
+          const outputPath = (0, import_node_path.join)(dest, entry.fileName);
+          void (0, import_promises.mkdir)((0, import_node_path.dirname)(outputPath), { recursive: true }).then(() => {
+            zipfile.openReadStream(
+              entry,
+              (streamError, readStream) => {
+                if (streamError) return reject(streamError);
+                if (!readStream) return reject(new Error("Failed to open zip entry stream."));
+                const writeStream = (0, import_node_fs.createWriteStream)(outputPath);
+                readStream.on("error", reject);
+                writeStream.on("error", reject);
+                writeStream.on("close", () => {
+                  zipfile.readEntry();
+                });
+                readStream.pipe(writeStream);
+              }
+            );
+          }).catch(reject);
+        }
+      });
+      zipfile.on("end", () => {
+        resolve();
+      });
+      zipfile.on("error", (zipError) => {
+        reject(zipError);
+      });
+    });
   });
 };
 var tryLstat = async (targetPath) => {
@@ -1169,11 +1165,7 @@ var mergeEntry = async (sourcePath, destPath, overwrite) => {
     await (0, import_promises.mkdir)(destPath, { recursive: true });
     const children = await (0, import_promises.readdir)(sourcePath);
     for (const child of children) {
-      await mergeEntry(
-        (0, import_node_path.join)(sourcePath, child),
-        (0, import_node_path.join)(destPath, child),
-        overwrite
-      );
+      await mergeEntry((0, import_node_path.join)(sourcePath, child), (0, import_node_path.join)(destPath, child), overwrite);
     }
     await (0, import_promises.rm)(sourcePath, { recursive: true, force: true });
     return;
@@ -1263,23 +1255,15 @@ var ProjectTools = class {
   }
   async makeScriptsExecutable(workspacePath) {
     const isWindows = process.platform === "win32";
-    const scripts = await this.findScripts(
-      workspacePath,
-      isWindows ? [".ps1", ".bat"] : [".sh"]
-    );
+    const scripts = await this.findScripts(workspacePath, isWindows ? [".ps1", ".bat"] : [".sh"]);
     let updatedCount = 0;
     if (isWindows) {
-      for (const scriptPath of scripts.filter(
-        (script) => script.endsWith(".ps1")
-      )) {
+      for (const scriptPath of scripts.filter((script) => script.endsWith(".ps1"))) {
         try {
-          await runCommand(
-            `powershell -Command "Unblock-File -Path '${scriptPath}'"`,
-            {
-              cwd: workspacePath,
-              shell: true
-            }
-          );
+          await runCommand(`powershell -Command "Unblock-File -Path '${scriptPath}'"`, {
+            cwd: workspacePath,
+            shell: true
+          });
           updatedCount += 1;
         } catch {
         }
@@ -1344,11 +1328,7 @@ var ProjectTools = class {
     if (await this.pathExists(presetManifestPath)) {
       return;
     }
-    const projectModePath = path3.join(
-      workspacePath,
-      ".ms-scaffold",
-      "project-mode.json"
-    );
+    const projectModePath = path3.join(workspacePath, ".ms-scaffold", "project-mode.json");
     if (!await this.pathExists(projectModePath)) {
       return;
     }
@@ -1402,9 +1382,7 @@ var ProjectTools = class {
     if (directoryNames.length === 1) {
       return directoryNames[0] ?? null;
     }
-    const exactMatch = directoryNames.find(
-      (directoryName) => directoryName === expectedDirName
-    );
+    const exactMatch = directoryNames.find((directoryName) => directoryName === expectedDirName);
     if (exactMatch) {
       return exactMatch;
     }
@@ -1645,10 +1623,8 @@ var GithubCopilotCliExecutor = class {
       windowsHide: true
     });
     if (child.pid) {
-      writeDiagnostic(
-        `[GithubCopilotCliExecutor] spawned child PID=${child.pid}
-`
-      );
+      writeDiagnostic(`[GithubCopilotCliExecutor] spawned child PID=${child.pid}
+`);
     }
     const timeoutMs = execution.timeoutMs;
     const timer = timeoutMs != null ? setTimeout(() => {
@@ -1666,15 +1642,13 @@ var GithubCopilotCliExecutor = class {
     child.stdout.on("data", (chunk) => {
       const data = chunk.toString();
       stdoutBuf += data;
-      if (stdoutBuf.length > MAX_OUTPUT_BYTES)
-        stdoutBuf = stdoutBuf.slice(-MAX_OUTPUT_BYTES);
+      if (stdoutBuf.length > MAX_OUTPUT_BYTES) stdoutBuf = stdoutBuf.slice(-MAX_OUTPUT_BYTES);
       onOutput("stdout", data);
     });
     child.stderr.on("data", (chunk) => {
       const data = chunk.toString();
       stderrBuf += data;
-      if (stderrBuf.length > MAX_OUTPUT_BYTES)
-        stderrBuf = stderrBuf.slice(-MAX_OUTPUT_BYTES);
+      if (stderrBuf.length > MAX_OUTPUT_BYTES) stderrBuf = stderrBuf.slice(-MAX_OUTPUT_BYTES);
       onOutput("stderr", data);
     });
     child.on("close", (code) => {
@@ -2003,10 +1977,7 @@ function isRecord(value) {
 }
 function requireNonEmptyString(payload, field, taskType) {
   if (!isRecord(payload) || typeof payload[field] !== "string" || !payload[field].trim()) {
-    throw (0, import_devtools_protocol8.createServicemeError)(
-      "invalid_payload",
-      `${taskType} payload ${field} is required`
-    );
+    throw (0, import_devtools_protocol8.createServicemeError)("invalid_payload", `${taskType} payload ${field} is required`);
   }
 }
 function validateTaskPayload(taskType, payload) {
@@ -2169,9 +2140,7 @@ var TaskExecutionEngine = class {
     if (policy === "reject") {
       const existingIds = this.taskExecutions.get(snapshot.taskId);
       if (existingIds && existingIds.size > 0) {
-        throw new Error(
-          `TASK_ALREADY_RUNNING: Task ${snapshot.taskId} is already running`
-        );
+        throw new Error(`TASK_ALREADY_RUNNING: Task ${snapshot.taskId} is already running`);
       }
     }
     const startedAt = (/* @__PURE__ */ new Date()).toISOString();
@@ -2436,7 +2405,7 @@ var TaskLogManager = class {
 var TICK_INTERVAL = 1e3;
 var MIN_SCHEDULE_INTERVAL = 1e3;
 var SchedulerDaemon = class {
-  constructor(workspacePath) {
+  constructor(workspacePath, options = {}) {
     this.tickTimer = null;
     this.watcher = null;
     this.running = false;
@@ -2449,6 +2418,7 @@ var SchedulerDaemon = class {
     this.logManager = new TaskLogManager(workspacePath);
     this.pidManager = new PidManager(workspacePath);
     this.logger = new DaemonLogger(workspacePath);
+    this.getExecutor = options.getExecutor ?? getExecutor;
   }
   start() {
     if (this.running) return;
@@ -2554,7 +2524,7 @@ var SchedulerDaemon = class {
         this.workspacePath
       );
       validateTaskPayload(task.taskType, executionPayload);
-      const executor = getExecutor(task.taskType);
+      const executor = this.getExecutor(task.taskType);
       const result = await executor.execute(executionPayload);
       const finishedAt = (/* @__PURE__ */ new Date()).toISOString();
       const durationMs = Date.now() - startMs;
@@ -2659,10 +2629,10 @@ var SkillCatalogClient = class {
   }
   async getCatalog() {
     if (!this.baseUrl) {
-      return {
-        skills: [],
-        fetchedAt: (/* @__PURE__ */ new Date()).toISOString()
-      };
+      throw (0, import_devtools_protocol9.createServicemeError)(
+        "workspace_not_found",
+        "Skill catalog baseUrl is not configured."
+      );
     }
     const response = await this.fetchImpl(
       `${this.baseUrl}/api/v1/marketplace/skills`

package/dist/index.mjs

@@ -14,10 +14,10 @@ var AgentCatalogClient = class {
   }
   async getCatalog() {
     if (!this.baseUrl) {
-      return {
-        agents: [],
-        fetchedAt: (/* @__PURE__ */ new Date()).toISOString()
-      };
+      throw createServicemeError(
+        "workspace_not_found",
+        "Agent catalog baseUrl is not configured."
+      );
     }
     const response = await this.fetchImpl(
       `${this.baseUrl}/api/v1/marketplace/agents`
@@ -474,7 +474,8 @@ function createCopilotAuthRequiredError() {
 // src/copilot/prompt.ts
 import {
   COPILOT_ERROR_CODES as COPILOT_ERROR_CODES2,
-  createServicemeError as createServicemeError3
+  createServicemeError as createServicemeError3,
+  ServicemeProtocolError
 } from "@serviceme/devtools-protocol";
 var COPILOT_COMMAND2 = "copilot";
 var DEFAULT_TIMEOUT_MS = 12e4;
@@ -517,6 +518,9 @@ async function copilotPrompt(options) {
     }
     return { output, exitCode: 0 };
   } catch (error) {
+    if (error instanceof ServicemeProtocolError) {
+      throw error;
+    }
     const err = error;
     if (err.code === "ETIMEDOUT") {
       throw createServicemeError3(
@@ -565,11 +569,13 @@ var TOOL_CHECK_TIMEOUT_MS = {
 var ERROR_CODE_NOT_FOUND = 127;
 var ERROR_CODE_TIMEOUT = "ETIMEDOUT";
 var EnvironmentInspector = class {
+  constructor(options = {}) {
+    this.runCommandFn = options.runCommand ?? runCommand;
+    this.platform = options.platform ?? process.platform;
+  }
   async checkEnvironment() {
     const results = await Promise.all(
-      KNOWN_ENVIRONMENT_TOOLS.map(
-        async (tool) => [tool, await this.checkTool(tool)]
-      )
+      KNOWN_ENVIRONMENT_TOOLS.map(async (tool) => [tool, await this.checkTool(tool)])
     );
     return Object.fromEntries(results);
   }
@@ -597,8 +603,8 @@ var EnvironmentInspector = class {
   }
   async getToolPath(toolName) {
     try {
-      const isWindows = process.platform === "win32";
-      const result = await runCommand(isWindows ? "where" : "which", {
+      const isWindows = this.platform === "win32";
+      const result = await this.runCommandFn(isWindows ? "where" : "which", {
         args: [toolName],
         timeoutMs: this.getToolTimeout(toolName)
       });
@@ -615,14 +621,12 @@ var EnvironmentInspector = class {
    */
   async getToolShimPath(toolName) {
     try {
-      const result = await runCommand("where", {
+      const result = await this.runCommandFn("where", {
         args: [toolName],
         timeoutMs: this.getToolTimeout(toolName)
       });
       const candidates = result.stdout.split(/\r?\n/).map((line) => line.trim()).filter((line) => line.length > 0);
-      const cmdShim = candidates.find(
-        (line) => line.toLowerCase().endsWith(".cmd")
-      );
+      const cmdShim = candidates.find((line) => line.toLowerCase().endsWith(".cmd"));
       return cmdShim ?? candidates[0];
     } catch {
       return void 0;
@@ -630,16 +634,16 @@ var EnvironmentInspector = class {
   }
   async getToolVersion(toolName) {
     const invocation = await this.getVersionInvocation(toolName);
-    const result = await runCommand(invocation.command, {
+    const result = await this.runCommandFn(invocation.command, {
       args: invocation.args,
       timeoutMs: this.getToolTimeout(toolName)
     });
     return this.parseVersion(toolName, result.stdout || result.stderr);
   }
   async checkNvm() {
-    if (process.platform === "win32") {
+    if (this.platform === "win32") {
       try {
-        const result = await runCommand("cmd.exe", {
+        const result = await this.runCommandFn("cmd.exe", {
           args: ["/c", "nvm version"],
           timeoutMs: this.getToolTimeout("nvm")
         });
@@ -656,7 +660,7 @@ var EnvironmentInspector = class {
       }
     }
     try {
-      const result = await runCommand("/bin/bash", {
+      const result = await this.runCommandFn("/bin/bash", {
         args: [
           "-lc",
           'export NVM_DIR="$HOME/.nvm"; [ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"; nvm --version'
@@ -684,7 +688,7 @@ var EnvironmentInspector = class {
       };
     }
     try {
-      const result = await runCommand("dotnet", {
+      const result = await this.runCommandFn("dotnet", {
         args: ["nuget", "list", "source"],
         timeoutMs: this.getToolTimeout("nuget")
       });
@@ -705,7 +709,7 @@ var EnvironmentInspector = class {
     }
   }
   async getVersionInvocation(toolName) {
-    const isWindows = process.platform === "win32";
+    const isWindows = this.platform === "win32";
     if (isWindows && ["npm", "pnpm", "nrm"].includes(toolName)) {
       const shim = await this.getToolShimPath(toolName);
       if (shim) {
@@ -1048,62 +1052,47 @@ import {
 
 // src/utils/fileUtils.ts
 import { constants, createWriteStream } from "fs";
-import {
-  access as access2,
-  copyFile,
-  lstat,
-  mkdir,
-  readdir,
-  rename,
-  rm
-} from "fs/promises";
+import { access as access2, copyFile, lstat, mkdir, readdir, rename, rm } from "fs/promises";
 import { dirname as dirname3, join as join3 } from "path";
-import { open } from "yauzl";
+import yauzl from "yauzl";
 var unzipFile = (zipPath, dest) => {
   return new Promise((resolve, reject) => {
-    open(
-      zipPath,
-      { lazyEntries: true },
-      (err, zipfile) => {
-        if (err) return reject(err);
-        if (!zipfile) return reject(new Error("Failed to open zip file."));
-        zipfile.readEntry();
-        zipfile.on("entry", (entry) => {
-          if (/\/$/.test(entry.fileName)) {
-            void mkdir(join3(dest, entry.fileName), { recursive: true }).then(() => {
-              zipfile.readEntry();
-            }).catch(reject);
-          } else {
-            const outputPath = join3(dest, entry.fileName);
-            void mkdir(dirname3(outputPath), { recursive: true }).then(() => {
-              zipfile.openReadStream(
-                entry,
-                (streamError, readStream) => {
-                  if (streamError) return reject(streamError);
-                  if (!readStream)
-                    return reject(
-                      new Error("Failed to open zip entry stream.")
-                    );
-                  const writeStream = createWriteStream(outputPath);
-                  readStream.on("error", reject);
-                  writeStream.on("error", reject);
-                  writeStream.on("close", () => {
-                    zipfile.readEntry();
-                  });
-                  readStream.pipe(writeStream);
-                }
-              );
-            }).catch(reject);
-          }
-        });
-        zipfile.on("end", () => {
-          resolve();
-        });
-        zipfile.on("error", (zipError) => {
-          reject(zipError);
-        });
-      }
-    );
+    yauzl.open(zipPath, { lazyEntries: true }, (err, zipfile) => {
+      if (err) return reject(err);
+      if (!zipfile) return reject(new Error("Failed to open zip file."));
+      zipfile.readEntry();
+      zipfile.on("entry", (entry) => {
+        if (/\/$/.test(entry.fileName)) {
+          void mkdir(join3(dest, entry.fileName), { recursive: true }).then(() => {
+            zipfile.readEntry();
+          }).catch(reject);
+        } else {
+          const outputPath = join3(dest, entry.fileName);
+          void mkdir(dirname3(outputPath), { recursive: true }).then(() => {
+            zipfile.openReadStream(
+              entry,
+              (streamError, readStream) => {
+                if (streamError) return reject(streamError);
+                if (!readStream) return reject(new Error("Failed to open zip entry stream."));
+                const writeStream = createWriteStream(outputPath);
+                readStream.on("error", reject);
+                writeStream.on("error", reject);
+                writeStream.on("close", () => {
+                  zipfile.readEntry();
+                });
+                readStream.pipe(writeStream);
+              }
+            );
+          }).catch(reject);
+        }
+      });
+      zipfile.on("end", () => {
+        resolve();
+      });
+      zipfile.on("error", (zipError) => {
+        reject(zipError);
+      });
+    });
   });
 };
 var tryLstat = async (targetPath) => {
@@ -1127,11 +1116,7 @@ var mergeEntry = async (sourcePath, destPath, overwrite) => {
     await mkdir(destPath, { recursive: true });
     const children = await readdir(sourcePath);
     for (const child of children) {
-      await mergeEntry(
-        join3(sourcePath, child),
-        join3(destPath, child),
-        overwrite
-      );
+      await mergeEntry(join3(sourcePath, child), join3(destPath, child), overwrite);
     }
     await rm(sourcePath, { recursive: true, force: true });
     return;
@@ -1221,23 +1206,15 @@ var ProjectTools = class {
   }
   async makeScriptsExecutable(workspacePath) {
     const isWindows = process.platform === "win32";
-    const scripts = await this.findScripts(
-      workspacePath,
-      isWindows ? [".ps1", ".bat"] : [".sh"]
-    );
+    const scripts = await this.findScripts(workspacePath, isWindows ? [".ps1", ".bat"] : [".sh"]);
     let updatedCount = 0;
     if (isWindows) {
-      for (const scriptPath of scripts.filter(
-        (script) => script.endsWith(".ps1")
-      )) {
+      for (const scriptPath of scripts.filter((script) => script.endsWith(".ps1"))) {
         try {
-          await runCommand(
-            `powershell -Command "Unblock-File -Path '${scriptPath}'"`,
-            {
-              cwd: workspacePath,
-              shell: true
-            }
-          );
+          await runCommand(`powershell -Command "Unblock-File -Path '${scriptPath}'"`, {
+            cwd: workspacePath,
+            shell: true
+          });
           updatedCount += 1;
         } catch {
         }
@@ -1302,11 +1279,7 @@ var ProjectTools = class {
     if (await this.pathExists(presetManifestPath)) {
       return;
     }
-    const projectModePath = path3.join(
-      workspacePath,
-      ".ms-scaffold",
-      "project-mode.json"
-    );
+    const projectModePath = path3.join(workspacePath, ".ms-scaffold", "project-mode.json");
     if (!await this.pathExists(projectModePath)) {
       return;
     }
@@ -1360,9 +1333,7 @@ var ProjectTools = class {
     if (directoryNames.length === 1) {
       return directoryNames[0] ?? null;
     }
-    const exactMatch = directoryNames.find(
-      (directoryName) => directoryName === expectedDirName
-    );
+    const exactMatch = directoryNames.find((directoryName) => directoryName === expectedDirName);
     if (exactMatch) {
       return exactMatch;
     }
@@ -1603,10 +1574,8 @@ var GithubCopilotCliExecutor = class {
       windowsHide: true
     });
     if (child.pid) {
-      writeDiagnostic(
-        `[GithubCopilotCliExecutor] spawned child PID=${child.pid}
-`
-      );
+      writeDiagnostic(`[GithubCopilotCliExecutor] spawned child PID=${child.pid}
+`);
     }
     const timeoutMs = execution.timeoutMs;
     const timer = timeoutMs != null ? setTimeout(() => {
@@ -1624,15 +1593,13 @@ var GithubCopilotCliExecutor = class {
     child.stdout.on("data", (chunk) => {
       const data = chunk.toString();
       stdoutBuf += data;
-      if (stdoutBuf.length > MAX_OUTPUT_BYTES)
-        stdoutBuf = stdoutBuf.slice(-MAX_OUTPUT_BYTES);
+      if (stdoutBuf.length > MAX_OUTPUT_BYTES) stdoutBuf = stdoutBuf.slice(-MAX_OUTPUT_BYTES);
       onOutput("stdout", data);
     });
     child.stderr.on("data", (chunk) => {
       const data = chunk.toString();
       stderrBuf += data;
-      if (stderrBuf.length > MAX_OUTPUT_BYTES)
-        stderrBuf = stderrBuf.slice(-MAX_OUTPUT_BYTES);
+      if (stderrBuf.length > MAX_OUTPUT_BYTES) stderrBuf = stderrBuf.slice(-MAX_OUTPUT_BYTES);
       onOutput("stderr", data);
     });
     child.on("close", (code) => {
@@ -1961,10 +1928,7 @@ function isRecord(value) {
 }
 function requireNonEmptyString(payload, field, taskType) {
   if (!isRecord(payload) || typeof payload[field] !== "string" || !payload[field].trim()) {
-    throw createServicemeError8(
-      "invalid_payload",
-      `${taskType} payload ${field} is required`
-    );
+    throw createServicemeError8("invalid_payload", `${taskType} payload ${field} is required`);
   }
 }
 function validateTaskPayload(taskType, payload) {
@@ -2127,9 +2091,7 @@ var TaskExecutionEngine = class {
     if (policy === "reject") {
       const existingIds = this.taskExecutions.get(snapshot.taskId);
       if (existingIds && existingIds.size > 0) {
-        throw new Error(
-          `TASK_ALREADY_RUNNING: Task ${snapshot.taskId} is already running`
-        );
+        throw new Error(`TASK_ALREADY_RUNNING: Task ${snapshot.taskId} is already running`);
       }
     }
     const startedAt = (/* @__PURE__ */ new Date()).toISOString();
@@ -2394,7 +2356,7 @@ var TaskLogManager = class {
 var TICK_INTERVAL = 1e3;
 var MIN_SCHEDULE_INTERVAL = 1e3;
 var SchedulerDaemon = class {
-  constructor(workspacePath) {
+  constructor(workspacePath, options = {}) {
     this.tickTimer = null;
     this.watcher = null;
     this.running = false;
@@ -2407,6 +2369,7 @@ var SchedulerDaemon = class {
     this.logManager = new TaskLogManager(workspacePath);
     this.pidManager = new PidManager(workspacePath);
     this.logger = new DaemonLogger(workspacePath);
+    this.getExecutor = options.getExecutor ?? getExecutor;
   }
   start() {
     if (this.running) return;
@@ -2512,7 +2475,7 @@ var SchedulerDaemon = class {
         this.workspacePath
       );
       validateTaskPayload(task.taskType, executionPayload);
-      const executor = getExecutor(task.taskType);
+      const executor = this.getExecutor(task.taskType);
       const result = await executor.execute(executionPayload);
       const finishedAt = (/* @__PURE__ */ new Date()).toISOString();
       const durationMs = Date.now() - startMs;
@@ -2617,10 +2580,10 @@ var SkillCatalogClient = class {
   }
   async getCatalog() {
     if (!this.baseUrl) {
-      return {
-        skills: [],
-        fetchedAt: (/* @__PURE__ */ new Date()).toISOString()
-      };
+      throw createServicemeError9(
+        "workspace_not_found",
+        "Skill catalog baseUrl is not configured."
+      );
     }
     const response = await this.fetchImpl(
       `${this.baseUrl}/api/v1/marketplace/skills`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@serviceme/devtools-core",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "Reusable Node.js core capabilities powering the SERVICEME CLI and integrations.",
   "license": "SEE LICENSE IN LICENSE.md",
   "repository": {
@@ -36,7 +36,7 @@
     "comment-json": "^4.5.1",
     "json5": "^2.2.3",
     "yauzl": "^3.2.0",
-    "@serviceme/devtools-protocol": "0.1.7"
+    "@serviceme/devtools-protocol": "0.1.9"
   },
   "devDependencies": {
     "@types/node": "^24",
@@ -47,6 +47,7 @@
   },
   "scripts": {
     "watch": "tsup src/index.ts --format cjs,esm --dts --watch",
+    "lint": "biome check src test",
     "build": "tsup src/index.ts --format cjs,esm --dts",
     "typecheck": "tsc --noEmit",
     "test": "pnpm run build && node --test test/**/*.test.js && node --import tsx --test src/**/__tests__/**/*.test.ts",