@serviceme/devtools-cli 0.2.0 → 0.3.0

package/dist/bridgeServer.js

@@ -68,6 +68,7 @@ var BRIDGE_METHODS = [
   "repo.remove",
   "repo.enable",
   "repo.disable",
+  "repo.update",
   "repo.sync",
   "repo.syncAll",
   "auth.status",
@@ -212,7 +213,7 @@ function normalizeServicemeError(error, fallbackCode = "internal_error") {
 }
 
 // src/version.ts
-var SERVICEME_CLI_VERSION = "0.2.0";
+var SERVICEME_CLI_VERSION = "0.3.0";
 
 // src/bridge/handlers/AuthBridgeHandlers.ts
 var import_auth2 = require("@serviceme/devtools-core/auth");
@@ -597,6 +598,13 @@ var BridgeServer = class {
           this.writeSuccess(request.id, result);
           return;
         }
+        case "repo.update": {
+          const r = request;
+          if (!this.skillRepoHandler) return this.unsupportedV2(request.id);
+          const result = await this.skillRepoHandler.repoUpdate(r.params);
+          this.writeSuccess(request.id, result);
+          return;
+        }
         case "repo.sync": {
           const r = request;
           if (!this.skillRepoHandler) return this.unsupportedV2(request.id);

package/dist/cli.js

@@ -112,6 +112,7 @@ var BRIDGE_METHODS = [
   "repo.remove",
   "repo.enable",
   "repo.disable",
+  "repo.update",
   "repo.sync",
   "repo.syncAll",
   "auth.status",
@@ -776,7 +777,7 @@ var readline = __toESM(require("readline"));
 
 // src/version.ts
 var SERVICEME_CLI_NAME = "serviceme";
-var SERVICEME_CLI_VERSION = "0.2.0";
+var SERVICEME_CLI_VERSION = "0.3.0";
 
 // src/bridge/handlers/AuthBridgeHandlers.ts
 var import_auth3 = require("@serviceme/devtools-core/auth");
@@ -1161,6 +1162,13 @@ var BridgeServer = class {
           this.writeSuccess(request.id, result);
           return;
         }
+        case "repo.update": {
+          const r = request;
+          if (!this.skillRepoHandler) return this.unsupportedV2(request.id);
+          const result = await this.skillRepoHandler.repoUpdate(r.params);
+          this.writeSuccess(request.id, result);
+          return;
+        }
         case "repo.sync": {
           const r = request;
           if (!this.skillRepoHandler) return this.unsupportedV2(request.id);
@@ -1335,10 +1343,7 @@ var SkillRepoBridgeHandler = class {
       detail = await store.get(params.repoId, params.name);
     } catch (err) {
       if (err instanceof import_skill_store.SkillNotFoundError) {
-        throw createServicemeError(
-          "not_found",
-          `Skill ${params.repoId}/${params.name} not found`
-        );
+        throw createServicemeError("not_found", `Skill ${params.repoId}/${params.name} not found`);
       }
       throw err;
     }
@@ -1360,11 +1365,7 @@ var SkillRepoBridgeHandler = class {
    */
   async install(params) {
     const kind = params.kind ?? "skill";
-    const source = await this.resolveEntrySource(
-      params.repoId,
-      params.name,
-      kind
-    );
+    const source = await this.resolveEntrySource(params.repoId, params.name, kind);
     try {
       const result = await (0, import_skill_linker.installSkillToWorkspace)({
         repoId: params.repoId,
@@ -1399,11 +1400,7 @@ var SkillRepoBridgeHandler = class {
    */
   async convertToSymlink(params) {
     const kind = params.kind ?? "skill";
-    const source = await this.resolveEntrySource(
-      params.repoId,
-      params.name,
-      kind
-    );
+    const source = await this.resolveEntrySource(params.repoId, params.name, kind);
     try {
       const result = await (0, import_skill_linker.convertToSymlink)({
         repoId: params.repoId,
@@ -1464,9 +1461,7 @@ var SkillRepoBridgeHandler = class {
     const kind = params.kind ?? "skill";
     const scope = params.scope ?? "workspace";
     const scopesToScan = scope === "all" ? ["workspace", "user"] : [scope];
-    const linked = (await Promise.all(
-      scopesToScan.map((s) => (0, import_skill_linker.listLinkedSkills)(params.workspaceDir, kind, s))
-    )).flat();
+    const linked = (await Promise.all(scopesToScan.map((s) => (0, import_skill_linker.listLinkedSkills)(params.workspaceDir, kind, s)))).flat();
     return {
       links: linked.map((l) => ({
         repoId: l.repoId,
@@ -1557,15 +1552,9 @@ var SkillRepoBridgeHandler = class {
    * literal, so we can't carry the "wasn't there" case in the result.
    */
   async draftDelete(params) {
-    const existing = await this.draftsStore.tryReadSummary(
-      params.kind,
-      params.id
-    );
+    const existing = await this.draftsStore.tryReadSummary(params.kind, params.id);
     if (!existing) {
-      throw createServicemeError(
-        "not_found",
-        `Draft ${params.kind}/${params.id} not found`
-      );
+      throw createServicemeError("not_found", `Draft ${params.kind}/${params.id} not found`);
     }
     await this.draftsStore.delete(params.kind, params.id);
     return { deleted: true };
@@ -1594,7 +1583,8 @@ var SkillRepoBridgeHandler = class {
     const result = await repoManager.addUserRepo({
       url: params.url,
       branch: params.branch,
-      name: params.name
+      name: params.name,
+      useProxy: params.useProxy
     });
     return {
       repo: toBridgeRepoEntry(result.repo),
@@ -1607,7 +1597,8 @@ var SkillRepoBridgeHandler = class {
    * repos (the Store enforces this).
    */
   async repoRemove(params) {
-    await this.reposStore.removeUserRepo(params.repoId);
+    const repoManager = this.buildRepoManager();
+    await repoManager.removeUserRepo(params.repoId);
     return { removed: true };
   }
   /** Enable a repo (sets `enabled = true`). */
@@ -1620,6 +1611,27 @@ var SkillRepoBridgeHandler = class {
     const repo = await this.reposStore.disableRepo(params.repoId);
     return { repo: toBridgeRepoEntry(repo) };
   }
+  /** Update editable repo fields (url/branch/name/enabled). */
+  async repoUpdate(params) {
+    const patch = {};
+    if (typeof params.url === "string") {
+      patch.url = params.url;
+    }
+    if (typeof params.branch === "string") {
+      patch.branch = params.branch;
+    }
+    if (typeof params.name === "string") {
+      patch.name = params.name;
+    }
+    if (typeof params.enabled === "boolean") {
+      patch.enabled = params.enabled;
+    }
+    if (typeof params.useProxy === "boolean") {
+      patch.useProxy = params.useProxy;
+    }
+    const repo = await this.reposStore.updateRepo(params.repoId, patch);
+    return { repo: toBridgeRepoEntry(repo) };
+  }
   /** Sync (pull) a single repo. Returns the per-repo outcome. */
   async repoSync(params) {
     const repoManager = this.buildRepoManager();
@@ -1679,18 +1691,12 @@ var SkillRepoBridgeHandler = class {
       entry = await store.get(repoId, name);
     } catch (err) {
       if (err instanceof import_skill_store.SkillNotFoundError) {
-        throw createServicemeError(
-          "not_found",
-          `Skill ${repoId}/${name} not present on disk`
-        );
+        throw createServicemeError("not_found", `Skill ${repoId}/${name} not present on disk`);
       }
       throw err;
     }
     if (entry.kind !== kind) {
-      throw createServicemeError(
-        "not_found",
-        `Skill ${repoId}/${name} not present on disk`
-      );
+      throw createServicemeError("not_found", `Skill ${repoId}/${name} not present on disk`);
     }
     const sourceIsFile = entry.dir === entry.manifestPath;
     return { sourcePath: entry.dir, sourceIsFile };
@@ -1721,6 +1727,7 @@ function toBridgeRepoEntry(repo) {
     url: repo.url,
     branch: repo.branch,
     enabled: repo.enabled,
+    useProxy: repo.useProxy,
     writeEnabled: repo.writeEnabled,
     source: repo.source,
     description: repo.description,
@@ -1857,7 +1864,7 @@ async function handleEnroll(parsed) {
 async function handleRotateSecret(parsed) {
   const gracePeriodDaysArg = getStringFlag(parsed, "gracePeriodDays");
   const gracePeriodDays = gracePeriodDaysArg ? parseInt(gracePeriodDaysArg, 10) : 7;
-  if (isNaN(gracePeriodDays) || gracePeriodDays < 0 || gracePeriodDays > 365) {
+  if (Number.isNaN(gracePeriodDays) || gracePeriodDays < 0 || gracePeriodDays > 365) {
     throw createServicemeError("invalid_params", "Expected --gracePeriodDays <0-365>.");
   }
   const core = makeDeviceCore();
@@ -2439,7 +2446,7 @@ async function handleCreate(parsed) {
   writeSuccess({ task });
 }
 function handleList3(parsed) {
-  const wp = requireWorkspace(parsed);
+  const _wp = requireWorkspace(parsed);
   const mgr = new import_devtools_core11.TaskConfigManager();
   const tasks = mgr.listTasks();
   const fields = getStringFlag(parsed, "fields");
@@ -2454,7 +2461,7 @@ function handleList3(parsed) {
   writeSuccess({ tasks: result, total: tasks.length });
 }
 function handleGet(parsed) {
-  const wp = requireWorkspace(parsed);
+  const _wp = requireWorkspace(parsed);
   const id = requireId(parsed);
   const mgr = new import_devtools_core11.TaskConfigManager();
   const task = mgr.getTask(id);
@@ -2466,7 +2473,7 @@ function handleGet(parsed) {
   writeSuccess({ task: result });
 }
 async function handleEdit(parsed) {
-  const wp = requireWorkspace(parsed);
+  const _wp = requireWorkspace(parsed);
   const id = requireId(parsed);
   const mgr = new import_devtools_core11.TaskConfigManager();
   const existing = mgr.getTask(id);
@@ -2510,7 +2517,7 @@ async function handleEdit(parsed) {
   writeSuccess({ task });
 }
 function handleDelete(parsed) {
-  const wp = requireWorkspace(parsed);
+  const _wp = requireWorkspace(parsed);
   const id = requireId(parsed);
   requireConfirmation(parsed);
   const mgr = new import_devtools_core11.TaskConfigManager();
@@ -2526,7 +2533,7 @@ function handleDelete(parsed) {
   writeSuccess({ id, name: existing.name });
 }
 function handleToggle(parsed) {
-  const wp = requireWorkspace(parsed);
+  const _wp = requireWorkspace(parsed);
   const id = requireId(parsed);
   const enabledStr = getStringFlag(parsed, "enabled");
   if (enabledStr === void 0) {
@@ -2568,7 +2575,7 @@ async function handleTrigger(parsed) {
   writeSuccess({ log });
 }
 function handleLogs(parsed) {
-  const wp = requireWorkspace(parsed);
+  const _wp = requireWorkspace(parsed);
   const taskId = getStringFlag(parsed, "task-id");
   const limitStr = getStringFlag(parsed, "limit");
   const limit = limitStr ? Math.min(Number(limitStr), 200) : 20;
@@ -2819,7 +2826,7 @@ function getSchedulerStatus() {
     pidFile: pidMgr.getPidPath()
   };
 }
-function handleStart(parsed) {
+function handleStart(_parsed) {
   const pidMgr = new import_devtools_core12.PidManager("", { pidPath: (0, import_devtools_core12.getSchedulerPidPath)() });
   const existingPid = pidMgr.getRunningPid();
   if (existingPid !== null) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@serviceme/devtools-cli",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Unified SERVICEME CLI for automation, project scaffolding, and bridge-based tooling.",
   "license": "SEE LICENSE IN LICENSE.md",
   "repository": {
@@ -43,12 +43,13 @@
     "tsup": "^8.5.1",
     "tsx": "^4.22.0",
     "typescript": "^5.9.3",
-    "@serviceme/devtools-core": "0.2.0",
-    "@serviceme/devtools-protocol": "0.2.0"
+    "@serviceme/devtools-core": "0.3.0",
+    "@serviceme/devtools-protocol": "0.3.0"
   },
   "scripts": {
     "build": "tsup --config tsup.config.ts",
     "lint": "biome check src test",
+    "format": "biome format --write src test",
     "release:check": "pnpm run test && npm pack --dry-run",
     "test": "pnpm run build && node --test test/*.test.js && node --import tsx --test 'src/**/__tests__/**/*.test.ts'",
     "pack": "npm pack",