@gethmy/mcp 2.23.0 → 2.24.0

package/dist/lib/config.js

@@ -17,7 +17,7 @@ var __esm = (fn, res) => () => (fn && (res = fn(fn = 0)), res);
 // src/config.ts
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
-import { join } from "node:path";
+import { dirname, join, parse, resolve } from "node:path";
 function getConfigDir() {
   return join(homedir(), ".harmony-mcp");
 }
@@ -27,6 +27,22 @@ function getConfigPath() {
 function getLocalConfigPath(cwd) {
   return join(cwd || process.cwd(), LOCAL_CONFIG_FILENAME);
 }
+function findLocalConfigPath(cwd) {
+  const home = resolve(homedir());
+  let dir = resolve(cwd || process.cwd());
+  const { root } = parse(dir);
+  for (;; ) {
+    if (dir !== home && dir !== root) {
+      const candidate = join(dir, LOCAL_CONFIG_FILENAME);
+      if (existsSync(candidate))
+        return candidate;
+    }
+    const parent = dirname(dir);
+    if (parent === dir)
+      return null;
+    dir = parent;
+  }
+}
 function emptyConfig() {
   return {
     apiKey: null,
@@ -78,8 +94,8 @@ function saveConfig(config) {
   });
 }
 function loadLocalConfig(cwd) {
-  const localConfigPath = getLocalConfigPath(cwd);
-  if (!existsSync(localConfigPath)) {
+  const localConfigPath = findLocalConfigPath(cwd);
+  if (localConfigPath === null) {
     return null;
   }
   try {
@@ -94,7 +110,7 @@ function loadLocalConfig(cwd) {
   }
 }
 function saveLocalConfig(config, cwd) {
-  const localConfigPath = getLocalConfigPath(cwd);
+  const localConfigPath = findLocalConfigPath(cwd) ?? getLocalConfigPath(cwd);
   const existingConfig = loadLocalConfig(cwd) || {
     workspaceId: null,
     projectId: null
@@ -108,7 +124,7 @@ function saveLocalConfig(config, cwd) {
   writeFileSync(localConfigPath, JSON.stringify(cleanConfig, null, 2));
 }
 function hasLocalConfig(cwd) {
-  return existsSync(getLocalConfigPath(cwd));
+  return findLocalConfigPath(cwd) !== null;
 }
 function getActiveCredential() {
   const config = loadConfig();
@@ -133,33 +149,69 @@ function getUserEmail() {
 function setUserEmail(email) {
   saveConfig({ userEmail: email });
 }
-function setActiveWorkspace(workspaceId, options) {
-  if (options?.local) {
-    saveLocalConfig({ workspaceId }, options.cwd);
-  } else {
-    saveConfig({ activeWorkspaceId: workspaceId });
+function setActiveContext(context, options) {
+  if (options?.global) {
+    saveConfig({
+      activeWorkspaceId: context.workspaceId,
+      activeProjectId: context.projectId
+    });
+    return;
   }
-}
-function setActiveProject(projectId, options) {
-  if (options?.local) {
-    saveLocalConfig({ projectId }, options.cwd);
+  const localPath = findLocalConfigPath(options?.cwd);
+  if (options?.local || localPath !== null) {
+    saveLocalConfig({ workspaceId: context.workspaceId, projectId: context.projectId }, options?.cwd);
   } else {
-    saveConfig({ activeProjectId: projectId });
+    saveConfig({
+      activeWorkspaceId: context.workspaceId,
+      activeProjectId: context.projectId
+    });
   }
 }
-function getActiveWorkspaceId(cwd) {
+function setActiveWorkspace(workspaceId, options) {
+  const currentWorkspaceId = getActiveWorkspaceId(options?.cwd);
+  const keepProject = currentWorkspaceId === workspaceId;
+  setActiveContext({
+    workspaceId,
+    projectId: keepProject ? getActiveProjectId(options?.cwd) : null
+  }, options);
+}
+function readActiveContext(cwd) {
   const localConfig = loadLocalConfig(cwd);
-  if (localConfig?.workspaceId) {
-    return localConfig.workspaceId;
+  if (localConfig) {
+    return {
+      workspaceId: localConfig.workspaceId ?? null,
+      projectId: localConfig.projectId ?? null
+    };
   }
-  return loadConfig().activeWorkspaceId;
+  const globalConfig = loadConfig();
+  return {
+    workspaceId: globalConfig.activeWorkspaceId,
+    projectId: globalConfig.activeProjectId
+  };
+}
+function getActiveWorkspaceId(cwd) {
+  return readActiveContext(cwd).workspaceId;
 }
 function getActiveProjectId(cwd) {
-  const localConfig = loadLocalConfig(cwd);
-  if (localConfig?.projectId) {
-    return localConfig.projectId;
+  return readActiveContext(cwd).projectId;
+}
+function getActiveContext(cwd) {
+  return describeActiveContext({
+    projectId: getActiveProjectId(cwd),
+    workspaceId: getActiveWorkspaceId(cwd)
+  });
+}
+function describeActiveContext(context) {
+  const { projectId, workspaceId } = context;
+  if (projectId && !workspaceId) {
+    return {
+      projectId,
+      workspaceId,
+      consistent: false,
+      note: `An active project (${projectId}) is set with no active workspace, so ` + "workspace-scoped tools cannot resolve one from it. Re-set it with " + "harmony_set_project_context, or pass workspaceId explicitly."
+    };
   }
-  return loadConfig().activeProjectId;
+  return { projectId, workspaceId, consistent: true, note: null };
 }
 function isConfigured() {
   const config = loadConfig();
@@ -214,7 +266,7 @@ init_config();
 export {
   setUserEmail,
   setActiveWorkspace,
-  setActiveProject,
+  setActiveContext,
   saveLocalConfig,
   saveConfig,
   loadLocalConfig,
@@ -232,5 +284,8 @@ export {
   getActiveWorkspaceId,
   getActiveProjectId,
   getActiveCredential,
+  getActiveContext,
+  findLocalConfigPath,
+  describeActiveContext,
   areSkillsInstalled
 };

package/dist/lib/oauth-refresh.js

@@ -17,7 +17,7 @@ var __esm = (fn, res) => () => (fn && (res = fn(fn = 0)), res);
 // src/config.ts
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
-import { join } from "node:path";
+import { dirname, join, parse, resolve } from "node:path";
 function getConfigDir() {
   return join(homedir(), ".harmony-mcp");
 }
@@ -27,6 +27,22 @@ function getConfigPath() {
 function getLocalConfigPath(cwd) {
   return join(cwd || process.cwd(), LOCAL_CONFIG_FILENAME);
 }
+function findLocalConfigPath(cwd) {
+  const home = resolve(homedir());
+  let dir = resolve(cwd || process.cwd());
+  const { root } = parse(dir);
+  for (;; ) {
+    if (dir !== home && dir !== root) {
+      const candidate = join(dir, LOCAL_CONFIG_FILENAME);
+      if (existsSync(candidate))
+        return candidate;
+    }
+    const parent = dirname(dir);
+    if (parent === dir)
+      return null;
+    dir = parent;
+  }
+}
 function emptyConfig() {
   return {
     apiKey: null,
@@ -78,8 +94,8 @@ function saveConfig(config) {
   });
 }
 function loadLocalConfig(cwd) {
-  const localConfigPath = getLocalConfigPath(cwd);
-  if (!existsSync(localConfigPath)) {
+  const localConfigPath = findLocalConfigPath(cwd);
+  if (localConfigPath === null) {
     return null;
   }
   try {
@@ -94,7 +110,7 @@ function loadLocalConfig(cwd) {
   }
 }
 function saveLocalConfig(config, cwd) {
-  const localConfigPath = getLocalConfigPath(cwd);
+  const localConfigPath = findLocalConfigPath(cwd) ?? getLocalConfigPath(cwd);
   const existingConfig = loadLocalConfig(cwd) || {
     workspaceId: null,
     projectId: null
@@ -108,7 +124,7 @@ function saveLocalConfig(config, cwd) {
   writeFileSync(localConfigPath, JSON.stringify(cleanConfig, null, 2));
 }
 function hasLocalConfig(cwd) {
-  return existsSync(getLocalConfigPath(cwd));
+  return findLocalConfigPath(cwd) !== null;
 }
 function getActiveCredential() {
   const config = loadConfig();
@@ -133,33 +149,69 @@ function getUserEmail() {
 function setUserEmail(email) {
   saveConfig({ userEmail: email });
 }
-function setActiveWorkspace(workspaceId, options) {
-  if (options?.local) {
-    saveLocalConfig({ workspaceId }, options.cwd);
-  } else {
-    saveConfig({ activeWorkspaceId: workspaceId });
+function setActiveContext(context, options) {
+  if (options?.global) {
+    saveConfig({
+      activeWorkspaceId: context.workspaceId,
+      activeProjectId: context.projectId
+    });
+    return;
   }
-}
-function setActiveProject(projectId, options) {
-  if (options?.local) {
-    saveLocalConfig({ projectId }, options.cwd);
+  const localPath = findLocalConfigPath(options?.cwd);
+  if (options?.local || localPath !== null) {
+    saveLocalConfig({ workspaceId: context.workspaceId, projectId: context.projectId }, options?.cwd);
   } else {
-    saveConfig({ activeProjectId: projectId });
+    saveConfig({
+      activeWorkspaceId: context.workspaceId,
+      activeProjectId: context.projectId
+    });
   }
 }
-function getActiveWorkspaceId(cwd) {
+function setActiveWorkspace(workspaceId, options) {
+  const currentWorkspaceId = getActiveWorkspaceId(options?.cwd);
+  const keepProject = currentWorkspaceId === workspaceId;
+  setActiveContext({
+    workspaceId,
+    projectId: keepProject ? getActiveProjectId(options?.cwd) : null
+  }, options);
+}
+function readActiveContext(cwd) {
   const localConfig = loadLocalConfig(cwd);
-  if (localConfig?.workspaceId) {
-    return localConfig.workspaceId;
+  if (localConfig) {
+    return {
+      workspaceId: localConfig.workspaceId ?? null,
+      projectId: localConfig.projectId ?? null
+    };
   }
-  return loadConfig().activeWorkspaceId;
+  const globalConfig = loadConfig();
+  return {
+    workspaceId: globalConfig.activeWorkspaceId,
+    projectId: globalConfig.activeProjectId
+  };
+}
+function getActiveWorkspaceId(cwd) {
+  return readActiveContext(cwd).workspaceId;
 }
 function getActiveProjectId(cwd) {
-  const localConfig = loadLocalConfig(cwd);
-  if (localConfig?.projectId) {
-    return localConfig.projectId;
+  return readActiveContext(cwd).projectId;
+}
+function getActiveContext(cwd) {
+  return describeActiveContext({
+    projectId: getActiveProjectId(cwd),
+    workspaceId: getActiveWorkspaceId(cwd)
+  });
+}
+function describeActiveContext(context) {
+  const { projectId, workspaceId } = context;
+  if (projectId && !workspaceId) {
+    return {
+      projectId,
+      workspaceId,
+      consistent: false,
+      note: `An active project (${projectId}) is set with no active workspace, so ` + "workspace-scoped tools cannot resolve one from it. Re-set it with " + "harmony_set_project_context, or pass workspaceId explicitly."
+    };
   }
-  return loadConfig().activeProjectId;
+  return { projectId, workspaceId, consistent: true, note: null };
 }
 function isConfigured() {
   const config = loadConfig();
@@ -238,7 +290,7 @@ function lockPath() {
   return join2(getConfigDir(), LOCK_FILENAME);
 }
 function sleep(ms) {
-  return new Promise((resolve) => setTimeout(resolve, ms));
+  return new Promise((resolve2) => setTimeout(resolve2, ms));
 }
 async function withRefreshLock(fn) {
   const path = lockPath();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gethmy/mcp",
-  "version": "2.23.0",
+  "version": "2.24.0",
   "description": "MCP server for Harmony, the shared surface for human–agent teams — agents claim cards, report progress, and move work on your board.",
   "publishConfig": {
     "access": "public"

package/src/api-client.ts

@@ -2148,6 +2148,9 @@ export class HarmonyApiClient {
     description?: string;
     steps?: unknown;
     triggerType?: string;
+    /** Auto-bind rule (#892); validated server-side, passed through as-is. */
+    autoBind?: unknown;
+    catalogId?: string;
   }): Promise<{ playbook: unknown }> {
     return this.request("POST", "/playbooks", data);
   }
@@ -2160,6 +2163,9 @@ export class HarmonyApiClient {
       steps?: unknown;
       enabled?: boolean;
       state?: string;
+      triggerType?: string;
+      /** `null` clears the rule; absent leaves it untouched (#892). */
+      autoBind?: unknown;
     },
   ): Promise<{ playbook: unknown }> {
     return this.request("PATCH", `/playbooks/${playbookId}`, updates);

package/src/cli.ts

@@ -3,10 +3,11 @@ import { createRequire } from "node:module";
 import { program } from "commander";
 import {
   areSkillsInstalled,
+  findLocalConfigPath,
+  getActiveContext,
   getActiveProjectId,
   getActiveWorkspaceId,
   getConfigPath,
-  getLocalConfigPath,
   hasLocalConfig,
   isConfigured,
   loadConfig,
@@ -85,7 +86,9 @@ program
       console.log("\nContext:");
 
       if (hasLocal) {
-        console.log(`  Local config: ${getLocalConfigPath()}`);
+        // The file that is actually READ, which since #893 may be an ancestor's
+        // — printing the cwd path would name a file that does not exist.
+        console.log(`  Local config: ${findLocalConfigPath()}`);
         console.log(
           `    Workspace: ${localConfig?.workspaceId || "(not set)"}`,
         );
@@ -103,16 +106,14 @@ program
       // Show effective (active) context
       const effectiveWorkspace = getActiveWorkspaceId();
       const effectiveProject = getActiveProjectId();
-      const wsSource = localConfig?.workspaceId
-        ? "local"
-        : globalConfig.activeWorkspaceId
-          ? "global"
-          : "";
-      const projSource = localConfig?.projectId
-        ? "local"
-        : globalConfig.activeProjectId
-          ? "global"
-          : "";
+      // The local file wins as a WHOLE file (#893), so the source is the file,
+      // not the field: with a local config in scope BOTH ids come from it, and
+      // a key it omits reads as unset rather than falling through to global.
+      // Reporting "global" for such a field would describe a fallback that no
+      // longer happens.
+      const contextSource = hasLocal ? "local" : "global";
+      const wsSource = effectiveWorkspace ? contextSource : "";
+      const projSource = effectiveProject ? contextSource : "";
 
       console.log("\n  Active (effective):");
       console.log(
@@ -121,6 +122,14 @@ program
       console.log(
         `    Project: ${effectiveProject || "(not set)"}${projSource ? ` ← ${projSource}` : ""}`,
       );
+
+      // Say it out loud when the pair does not hold together (#893). A project
+      // with no workspace leaves every workspace-scoped tool guessing, and this
+      // status output is where a person looks when one of them misbehaves.
+      const report = getActiveContext();
+      if (!report.consistent && report.note) {
+        console.log(`\n  ⚠ ${report.note}`);
+      }
     } else {
       console.log("Status: Not configured\n");
       console.log("Run: npx @gethmy/mcp setup");