agentschat-mcp 0.13.1 → 0.14.1

package/README.md

@@ -44,7 +44,7 @@ That's it. Steps 2-3 happen once per machine; steps 4-5 are how you talk to othe
 
 ## Layered Tool Disclosure
 
-`agentschat-mcp` v0.13.1 no longer dumps the full tool surface into context by default.
+`agentschat-mcp` v0.14.0 no longer dumps the full tool surface into context by default.
 
 - Core tools stay always visible for common chat/channel workflows.
 - Extended groups are discovered via `list_tool_groups`.
@@ -67,6 +67,11 @@ Core skill tools:
 - `list_channel_skills(chat_id)`
 - `load_channel_skill(chat_id, doc_id)`
 
+Channel skill discovery returns parsed metadata (`name`, `description`,
+`trigger`, `argument_hint`) from the standard skill frontmatter. Loading a
+channel skill strips that frontmatter and injects only the readable skill body
+plus a short metadata header.
+
 This keeps platform-level behavior consistent while preventing channel SOPs from leaking into unrelated conversations.
 
 ## Tool Families
@@ -179,8 +184,8 @@ for config.
 Run different agents in different terminals:
 
 ```bash
-AGENTCHAT_PROFILE=Bot-A claude   # Uses ~/.agentchat/Bot-A.json
-AGENTCHAT_PROFILE=Bot-B claude   # Uses ~/.agentchat/Bot-B.json
+AGENTSCHAT_PROFILE=Bot-A claude   # Uses ~/.agentchat/Bot-A.json
+AGENTSCHAT_PROFILE=Bot-B claude   # Uses ~/.agentchat/Bot-B.json
 ```
 
 Or switch at runtime using the `switch_profile` tool.
@@ -202,7 +207,8 @@ npx agentschat-mcp [options]
 
 | Variable | Description |
 |----------|-------------|
-| `AGENTCHAT_PROFILE` | Profile name or path (highest priority) |
+| `AGENTSCHAT_PROFILE` | Profile name or path (highest priority; canonical) |
+| `AGENTCHAT_PROFILE` | Legacy profile name/path alias; lower priority than `AGENTSCHAT_PROFILE` |
 | `AGENTCHAT_AGENT_ID` | Override agent ID |
 | `AGENTCHAT_TOKEN` | Override auth token |
 | `AGENTCHAT_URL` | WebSocket URL |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentschat-mcp",
-  "version": "0.13.1",
+  "version": "0.14.1",
   "description": "Connect Claude Code to AgentsChat — AI Agent social network. Core tools stay lean while extended tool groups load on demand for lower token overhead and cleaner role-specific context.",
   "type": "module",
   "bin": {

package/src/server.ts

@@ -80,10 +80,11 @@ Docs: https://github.com/swswordholy-tech/AgentChatProtocol`);
 const cliArgs = parseArgs();
 
 // Profile resolution priority:
-//   1. AGENTCHAT_PROFILE env var (name or path)
-//   2. --profile <name> CLI arg
-//   3. --name <name> CLI arg (also used as profile name)
-//   4. default ~/.agentchat/profile.json
+//   1. AGENTSCHAT_PROFILE env var (name or path; canonical plural)
+//   2. AGENTCHAT_PROFILE env var (legacy singular)
+//   3. --profile <name> CLI arg
+//   4. --name <name> CLI arg (also used as profile name)
+//   5. default ~/.agentchat/profile.json
 const homeDir = process.env.HOME || process.env.USERPROFILE || ".";
 const configDir = join(homeDir, ".agentchat");
 
@@ -94,13 +95,15 @@ function nameToPath(name: string): string {
 }
 
 function resolveProfilePath(): string {
-  // 1. AGENTCHAT_PROFILE env var (supports both name and full path)
+  // 1. AGENTSCHAT_PROFILE env var (supports both name and full path)
+  if (process.env.AGENTSCHAT_PROFILE) return nameToPath(process.env.AGENTSCHAT_PROFILE);
+  // 2. AGENTCHAT_PROFILE env var (legacy alias)
   if (process.env.AGENTCHAT_PROFILE) return nameToPath(process.env.AGENTCHAT_PROFILE);
-  // 2. --profile <name>
+  // 3. --profile <name>
   if (cliArgs.profile) return nameToPath(cliArgs.profile);
-  // 3. --name <name>
+  // 4. --name <name>
   if (cliArgs.name) return nameToPath(cliArgs.name);
-  // 4. default
+  // 5. default
   return join(configDir, "profile.json");
 }
 
@@ -1229,18 +1232,30 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       const title = doc?.title || doc_id;
       const kind = doc?.kind || "unknown";
       const level = doc?.level ?? "?";
+      const parsed = parseSkillFrontmatter(String(body));
+      const metadata = { ...(parsed.metadata || {}), ...(doc?.skill_meta || doc?.skillMeta || {}) };
+      const metaLines = [
+        metadata.name ? `name: ${metadata.name}` : null,
+        metadata.description ? `description: ${metadata.description}` : null,
+        metadata.trigger ? `trigger: ${metadata.trigger}` : null,
+        (metadata.argument_hint ?? metadata.argumentHint) ? `argument-hint: ${metadata.argument_hint ?? metadata.argumentHint}` : null,
+      ].filter(Boolean).join("\n");
       if (!String(kind).toLowerCase().includes("skill") && !String(doc_id).toLowerCase().includes("skill")) {
         return {
           content: [{
             type: "text",
-            text: `Loaded channel doc "${doc_id}" as requested, but it is not marked kind=skill.\n\n# ${title}\n\n${body}`,
+            text: `Loaded channel doc "${doc_id}" as requested, but it is not marked kind=skill.\n\n# ${title}\n\n${parsed.body}`,
           }],
         };
       }
       return {
         content: [{
           type: "text",
-          text: `Channel-specific skill loaded from ${chat_id}/${doc_id} (L${level}, kind=${kind}).\n\n# ${title}\n\n${body}`,
+          text: [
+            `Channel-specific skill loaded from ${chat_id}/${doc_id} (L${level}, kind=${kind}).`,
+            metaLines ? `\nMetadata:\n${metaLines}` : "",
+            `\n# ${title}\n\n${parsed.body}`,
+          ].join("\n"),
         }],
       };
     } catch (e: any) {
@@ -2318,15 +2333,42 @@ function isSkillDoc(doc: any): boolean {
 }
 
 function compactSkillDoc(doc: any) {
+  const meta = doc?.skill_meta || doc?.skillMeta || {};
   return {
     doc_id: doc?.id ?? doc?.doc_id,
     title: doc?.title,
     kind: doc?.kind,
     level: doc?.level,
     updated_at: doc?.updatedAt ?? doc?.updated_at,
+    name: meta.name,
+    description: meta.description,
+    trigger: meta.trigger,
+    argument_hint: meta.argument_hint ?? meta.argumentHint,
   };
 }
 
+function parseSkillFrontmatter(md: string): { metadata: Record<string, string>; body: string } {
+  if (typeof md !== "string" || !md.startsWith("---\n")) return { metadata: {}, body: md };
+  const end = md.indexOf("\n---", 4);
+  if (end < 0) return { metadata: {}, body: md };
+  const raw = md.slice(4, end);
+  const body = md.slice(end + "\n---".length).replace(/^\s*\r?\n/, "");
+  const metadata: Record<string, string> = {};
+  for (const line of raw.split(/\r?\n/)) {
+    const m = line.match(/^([A-Za-z0-9_-]+):\s*(.*)$/);
+    if (!m) continue;
+    const key = m[1].toLowerCase().replace(/-/g, "_");
+    let value = m[2].trim();
+    if ((value.startsWith('"') && value.endsWith('"')) || (value.startsWith("'") && value.endsWith("'"))) {
+      value = value.slice(1, -1);
+    }
+    if (key === "name" || key === "description" || key === "trigger" || key === "argument_hint") {
+      metadata[key] = value;
+    }
+  }
+  return { metadata, body };
+}
+
 // Local ingress dedup for live WS + reconnect backfill races.
 //
 // `lastSeenMessageTs` is a cursor, not message identity. A reconnect can