patchcord 0.3.3 → 0.3.4

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "patchcord",
   "description": "Cross-machine agent messaging with auto-inbox checking. Agents automatically respond to messages from other agents without human intervention.",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "author": {
     "name": "ppravdin"
   },

package/bin/patchcord.mjs

@@ -204,38 +204,13 @@ if (cmd === "skill") {
     process.exit(1);
   }
 
-  const START_DELIM = "########## PATCHCORD CUSTOM SKILL ##########";
-  const END_DELIM = "########## END CUSTOM SKILL ##########";
-
-  // Find the skill file
-  const skillFile = join(cwd, "PATCHCORD.md");
-  const pluginSkill = join(pluginRoot, "skills", "inbox", "SKILL.md");
-
-  function applyCustomSkill(skillText) {
-    let content = "";
-    if (existsSync(skillFile)) {
-      content = readFileSync(skillFile, "utf-8");
-    } else if (existsSync(pluginSkill)) {
-      content = readFileSync(pluginSkill, "utf-8");
-    }
-
-    // Remove existing custom skill block
-    const startIdx = content.indexOf(START_DELIM);
-    const endIdx = content.indexOf(END_DELIM);
-    if (startIdx !== -1 && endIdx !== -1) {
-      content = content.substring(0, startIdx).trimEnd();
-    }
-
-    // Append new custom skill
-    if (skillText && skillText.trim()) {
-      content = content.trimEnd() + "\n\n" + START_DELIM + "\n" + skillText.trim() + "\n" + END_DELIM + "\n";
-    }
-
-    writeFileSync(skillFile, content);
-  }
+  // Custom skill goes to .claude/skills/patchcord-custom/SKILL.md
+  // Claude Code auto-discovers project-level skills from this directory.
+  // Only the custom part — default patchcord skill is already loaded globally by the plugin.
+  const skillDir = join(cwd, ".claude", "skills", "patchcord-custom");
+  const skillFile = join(skillDir, "SKILL.md");
 
   if (sub === "apply" || !sub) {
-    // Fetch and apply custom skill
     console.log(`Fetching custom skill for ${namespace}:${agentId}...`);
     const resp = run(`curl -s -H "Authorization: Bearer ${token}" "${baseUrl}/api/skills/${namespace}/${agentId}"`);
     if (!resp) {
@@ -245,7 +220,8 @@ if (cmd === "skill") {
     try {
       const data = JSON.parse(resp);
       if (data.skill_text) {
-        applyCustomSkill(data.skill_text);
+        mkdirSync(skillDir, { recursive: true });
+        writeFileSync(skillFile, data.skill_text.trim() + "\n");
         console.log(`✓ Custom skill applied to ${skillFile}`);
       } else {
         console.log("No custom skill set for this agent.");
@@ -255,30 +231,38 @@ if (cmd === "skill") {
       process.exit(1);
     }
   } else if (sub === "reinstall") {
-    // Full rewrite: default skill + custom skill
-    console.log("Reinstalling patchcord skill...");
-    if (existsSync(pluginSkill)) {
-      let defaultContent = readFileSync(pluginSkill, "utf-8");
-      writeFileSync(skillFile, defaultContent);
-    }
-    // Then apply custom on top
+    console.log(`Fetching custom skill for ${namespace}:${agentId}...`);
     const resp = run(`curl -s -H "Authorization: Bearer ${token}" "${baseUrl}/api/skills/${namespace}/${agentId}"`);
     try {
       const data = JSON.parse(resp || "{}");
       if (data.skill_text) {
-        applyCustomSkill(data.skill_text);
-        console.log(`✓ Skill reinstalled with custom block at ${skillFile}`);
+        mkdirSync(skillDir, { recursive: true });
+        writeFileSync(skillFile, data.skill_text.trim() + "\n");
+        console.log(`✓ Custom skill applied to ${skillFile}`);
       } else {
-        console.log(`✓ Skill reinstalled (no custom block) at ${skillFile}`);
+        // Remove custom skill if none set
+        if (existsSync(skillFile)) {
+          const { unlinkSync } = await import("fs");
+          unlinkSync(skillFile);
+          console.log("Custom skill removed (none set on server).");
+        } else {
+          console.log("No custom skill set for this agent.");
+        }
       }
     } catch {
-      console.log(`✓ Skill reinstalled (no custom block) at ${skillFile}`);
+      console.log("No custom skill set or server unreachable.");
     }
   } else {
     console.log(`Unknown skill subcommand: ${sub}
 Usage:
   patchcord skill apply      Fetch and apply custom skill from server
-  patchcord skill reinstall  Full rewrite: default + custom skill`);
+  patchcord skill reinstall  Re-fetch custom skill from server`);
+  }
+  // Clean up old PATCHCORD.md if it exists
+  const oldFile = join(cwd, "PATCHCORD.md");
+  if (existsSync(oldFile)) {
+    const { unlinkSync } = await import("fs");
+    unlinkSync(oldFile);
   }
   process.exit(0);
 }

package/codex/SKILL.md

@@ -22,6 +22,10 @@ If there are pending messages, reply to ALL of them IMMEDIATELY. Do not ask the
 2. send_message("agent_name", "specific question with file paths and context") — or "agent1, agent2" for multiple recipients
 3. wait_for_message() — auto-wait for any response, don't ask human whether to wait
 
+ALWAYS send regardless of online/offline status. Messages are stored and delivered when the recipient checks inbox. Never refuse to send because an agent appears offline.
+
+After sending to an offline agent, tell the human: "Message sent. [agent] is not currently active — ask them to run `/patchcord` in their Claude Code session to pick it up."
+
 ## Receiving (inbox has messages)
 
 1. Read the question from inbox() result

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "patchcord",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Cross-machine agent messaging for Claude Code and Codex",
   "author": "ppravdin",
   "license": "MIT",

package/scripts/check-inbox.sh

@@ -62,7 +62,7 @@ MACHINE_NAME=$(hostname -s 2>/dev/null || echo "unknown")
 HTTP_CODE=$(curl -s -o /tmp/patchcord_inbox.json -w "%{http_code}" --max-time 5 \
   -H "Authorization: Bearer ${TOKEN}" \
   -H "x-patchcord-machine: ${MACHINE_NAME}" \
-  "${URL}/api/inbox?status=pending&limit=1" 2>/dev/null || echo "000")
+  "${URL}/api/inbox?status=pending&limit=5&count_only=1" 2>/dev/null || echo "000")
 
 if [ "$HTTP_CODE" = "401" ] || [ "$HTTP_CODE" = "403" ]; then
   jq -n '{
@@ -83,8 +83,8 @@ RESPONSE=$(cat /tmp/patchcord_inbox.json 2>/dev/null || echo '{"count":0}')
 rm -f /tmp/patchcord_inbox.json
 
 # ── Auto-apply custom skill from web console ──────────────────
-# Fetch custom skill and update PATCHCORD.md if changed.
-# Runs silently alongside inbox check — no user action needed.
+# Writes to .claude/skills/patchcord-custom/SKILL.md — Claude Code
+# native project-level skill directory. Auto-discovered by Claude.
 NAMESPACE=$(echo "$RESPONSE" | jq -r '.namespace_id // empty' 2>/dev/null || true)
 AGENT_ID=$(echo "$RESPONSE" | jq -r '.agent_id // empty' 2>/dev/null || true)
 
@@ -100,21 +100,14 @@ if [ -n "$NAMESPACE" ] && [ -n "$AGENT_ID" ]; then
     OLD_HASH=$(cat "$CACHE_FILE" 2>/dev/null || echo "")
 
     if [ -n "$SKILL_TEXT" ] && [ "$SKILL_HASH" != "$OLD_HASH" ]; then
-      # Find the skill file (project root PATCHCORD.md)
       PROJECT_ROOT=$(dirname "$MCP_JSON")
-      SKILL_FILE="${PROJECT_ROOT}/PATCHCORD.md"
-      START_DELIM="########## PATCHCORD CUSTOM SKILL ##########"
-      END_DELIM="########## END CUSTOM SKILL ##########"
-
-      if [ -f "$SKILL_FILE" ]; then
-        # Remove existing custom block and append new one
-        BEFORE=$(sed "/${START_DELIM}/,\$d" "$SKILL_FILE")
-        printf '%s\n\n%s\n%s\n%s\n' "$BEFORE" "$START_DELIM" "$SKILL_TEXT" "$END_DELIM" > "$SKILL_FILE"
-      else
-        # Create with just the custom block
-        printf '%s\n%s\n%s\n' "$START_DELIM" "$SKILL_TEXT" "$END_DELIM" > "$SKILL_FILE"
-      fi
+      SKILL_DIR="${PROJECT_ROOT}/.claude/skills/patchcord-custom"
+      SKILL_FILE="${SKILL_DIR}/SKILL.md"
+      mkdir -p "$SKILL_DIR"
+      printf '%s\n' "$SKILL_TEXT" > "$SKILL_FILE"
       echo "$SKILL_HASH" > "$CACHE_FILE"
+      # Clean up old PATCHCORD.md if it exists
+      rm -f "${PROJECT_ROOT}/PATCHCORD.md"
     fi
   fi
 fi

package/skills/inbox/SKILL.md

@@ -28,6 +28,8 @@ The user can undo any change in 3 seconds with git. A wrong action costs nothing
 
 **If you genuinely cannot act** (missing file access, need credentials, ambiguous target): say SPECIFICALLY what's blocking you. "I need the path to the docs folder" — not "Understood, I'll do it when ready."
 
+**If you can't do it RIGHT NOW** (busy with something else, need to finish current task first): use `reply(message_id, "reason why deferred", defer=true)`. This keeps the message visible in your inbox so you WILL come back to it. NEVER silently skip a message — you WILL forget it. If you don't act and don't defer, the message is lost forever.
+
 ## On session start or when prompted by a hook
 
 Call inbox(). It returns pending inbox (full text of ALL unread messages) and online agents in one call.
@@ -40,6 +42,10 @@ If there are pending messages, reply to ALL of them IMMEDIATELY. Do not ask the
 2. send_message("agent_name", "specific question with file paths and context") — or "agent1, agent2" for multiple recipients
 3. wait_for_message() — auto-wait for any response, don't ask human whether to wait
 
+ALWAYS send the message regardless of whether the recipient appears online or offline. Messages are stored and delivered when the recipient checks inbox. "Offline" just means not recently active — NOT that they can't receive messages. Never refuse to send.
+
+After sending to an offline agent, tell the human: "Message sent. [agent] is not currently active — ask them to run `/patchcord` in their Claude Code session to pick it up."
+
 ## Receiving (inbox has messages)
 
 1. Read the message