patchcord 0.5.87 → 0.5.89

package/package.json

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

package/per-project-skills/kimi/inbox/SKILL.md

@@ -4,6 +4,8 @@ description: Read Patchcord inbox and reply to messages
 type: flow
 ---
 
+> Use `/flow:patchcord:inbox` to run this. `/skill:` only loads it as docs (Kimi design).
+
 ```mermaid
 flowchart TD
   A([BEGIN]) --> B[Call the inbox MCP tool. For each pending message: do the work it asks for, then call reply with the message_id and a concrete summary of what you did.]

package/per-project-skills/kimi/subscribe/SKILL.md

@@ -4,6 +4,8 @@ description: Start the Patchcord background polling listener
 type: flow
 ---
 
+> Use `/flow:patchcord:subscribe` to run this. `/skill:` only loads it as docs (Kimi design).
+
 ```mermaid
 flowchart TD
   A([BEGIN]) --> B[First, call the inbox MCP tool. If there are pending messages, do the work each message asks for and call reply with the message_id and a concrete summary of what you did. Do NOT skip this step — the listener exits immediately if messages are already waiting, so you must drain them before starting it.]

package/per-project-skills/kimi/wait/SKILL.md

@@ -4,6 +4,8 @@ description: Wait for one incoming Patchcord message
 type: flow
 ---
 
+> Use `/flow:patchcord:wait` to run this. `/skill:` only loads it as docs (Kimi design).
+
 ```mermaid
 flowchart TD
   A([BEGIN]) --> B[Call the wait_for_message MCP tool to block until a message arrives or 5 minutes elapse.]

package/scripts/kimi-session-start.sh

@@ -24,9 +24,8 @@ if command -v jq >/dev/null 2>&1; then
   fi
 fi
 
-# --force tells kimi-subscribe.sh to kill any orphan listener (from a prior
-# Kimi session for this same project) before claiming the pidfile. Subscribe
-# resolves the project-specific pidfile name from .kimi/mcp.json itself, so
-# concurrent sessions for OTHER projects are untouched.
-nohup bash "$KIMI_SUB" --force 5 >/dev/null 2>&1 &
+# kimi-subscribe.sh always replaces any existing listener for this project,
+# so SessionStart starts a fresh listener even if a previous Kimi session
+# left one running.
+nohup bash "$KIMI_SUB" 5 >/dev/null 2>&1 &
 exit 0

package/scripts/kimi-subscribe.sh

@@ -14,19 +14,6 @@ set -euo pipefail
 
 command -v jq >/dev/null 2>&1 || { echo "jq required" >&2; exit 1; }
 
-# Parse --force flag (kills any existing listener for THIS project first).
-# SessionStart hook passes --force so fresh sessions always get a fresh listener.
-FORCE=0
-ARGS=()
-for arg in "$@"; do
-  if [ "$arg" = "--force" ]; then
-    FORCE=1
-  else
-    ARGS+=("$arg")
-  fi
-done
-set -- "${ARGS[@]+"${ARGS[@]}"}"
-
 # Resolve MCP config: per-project first, then global
 KIMI_MCP=""
 dir="$PWD"
@@ -73,29 +60,21 @@ fi
 PIDFILE="/tmp/patchcord_subscribe_${NAMESPACE_ID}_${AGENT_ID}.pid"
 NOTIFY_FILE="${HOME}/.kimi/patchcord-subscribe-notify.txt"
 
-# Pidfile guard.
-# --force: kill the existing listener (if any) and take over. Used by SessionStart
-#   so a fresh Kimi session always gets a fresh listener for this project.
-# default: if another instance is alive, exit 0 with a clear message so user-
-#   invoked flow skill calls are idempotent (no "background task failed").
+# Always replace any existing listener for this project. Otherwise the agent
+# sees an instant exit ("already running") and narrates it as "listener failed
+# to start" / "exited cleanly because no messages" — confusing UX. By killing
+# the old one, the new task stays in "running" state in Kimi's task browser,
+# which is what the user expects when they invoke /flow:patchcord:subscribe.
 if [ -f "$PIDFILE" ]; then
   OLD_PID=$(cat "$PIDFILE" 2>/dev/null || echo "")
   if [ -n "$OLD_PID" ] && kill -0 "$OLD_PID" 2>/dev/null; then
-    if [ "$FORCE" = "1" ]; then
-      kill "$OLD_PID" 2>/dev/null || true
-      # Wait briefly for the old process to clean up its pidfile
-      for _ in 1 2 3 4 5; do
-        kill -0 "$OLD_PID" 2>/dev/null || break
-        sleep 0.2
-      done
-      rm -f "$PIDFILE" 2>/dev/null || true
-    else
-      echo "Patchcord listener active."
-      exit 0
-    fi
-  else
-    rm -f "$PIDFILE" 2>/dev/null || true
+    kill "$OLD_PID" 2>/dev/null || true
+    for _ in 1 2 3 4 5; do
+      kill -0 "$OLD_PID" 2>/dev/null || break
+      sleep 0.2
+    done
   fi
+  rm -f "$PIDFILE" 2>/dev/null || true
 fi
 echo $$ > "$PIDFILE"