@faviovazquez/deliberate 0.2.9 → 0.2.10

package/BRAINSTORM.md

@@ -302,7 +302,7 @@ The visual companion server is still running at http://localhost:{port}.
 Stop it now? (Y/n)
 ```
 
-- If **Y** or Enter: run `scripts/stop-server.sh` to shut it down cleanly.
+- If **Y** or Enter: run `scripts/stop-server.sh {session_dir}` where `{session_dir}` is the `session_dir` value returned by `start-server.sh` in Phase 2. This stops the exact session that was started.
 - If **N**: leave it running. Remind the user it will auto-shutdown after 30 minutes of inactivity.
 
 If the visual companion was NOT active during this session, skip this phase.

package/README.md

@@ -490,7 +490,10 @@ Stop it now? (Y/n)
 
 If you say yes (or press Enter), it runs `scripts/stop-server.sh` to shut it down cleanly. If you say no, the server keeps running — useful if you want to review the session in the browser after the deliberation ends. **It will auto-shutdown after 30 minutes of inactivity regardless.**
 
-If you ever need to manage the server manually:
+If you ever need to manage the server manually, run these commands **from your project root**.
+
+- `start-server.sh --project-dir .` stores the session under `.deliberate/companion/` in your project. This is what allows `stop-server.sh` to find and kill it automatically.
+- `stop-server.sh` takes no arguments — it scans `.deliberate/companion/` and `/tmp/` to find running sessions and kills them.
 
 **Claude Code — global install:**
 ```bash

package/SKILL.md

@@ -368,7 +368,7 @@ The visual companion server is still running at http://localhost:{port}.
 Stop it now? (Y/n)
 ```
 
-- If **Y** or Enter: run `scripts/stop-server.sh` to shut it down cleanly.
+- If **Y** or Enter: run `scripts/stop-server.sh {session_dir}` where `{session_dir}` is the `session_dir` value returned by `start-server.sh` in Step 5c. This stops the exact session that was started.
 - If **N**: leave it running. Remind the user it will auto-shutdown after 30 minutes of inactivity.
 
 If the visual companion was NOT active during this session, skip this step.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@faviovazquez/deliberate",
-  "version": "0.2.9",
+  "version": "0.2.10",
   "description": "Multi-agent deliberation skill for AI coding assistants. Agreement is a bug.",
   "license": "MIT",
   "author": "Favio Vazquez",

package/scripts/stop-server.sh

@@ -1,42 +1,90 @@
 #!/usr/bin/env bash
 set -euo pipefail
 
-# Stop a deliberate visual companion server session
+# Stop a deliberate visual companion server session.
+#
+# Usage:
+#   stop-server.sh                  Auto-discover and stop all running sessions
+#   stop-server.sh SESSION_DIR      Stop a specific session (coordinator use)
 
-if [[ $# -lt 1 ]]; then
-  echo "Usage: stop-server.sh SESSION_DIR" >&2
-  exit 1
-fi
+stop_session() {
+  local SESSION_DIR="$1"
+  local STATE_DIR="$SESSION_DIR/state"
+  local SERVER_JS="$SESSION_DIR/server.js"
+
+  if [[ ! -d "$SESSION_DIR" ]]; then
+    echo "Session directory not found: $SESSION_DIR" >&2
+    return 1
+  fi
+
+  local PIDS
+  PIDS=$(pgrep -f "$SERVER_JS" 2>/dev/null || true)
+
+  if [[ -n "$PIDS" ]]; then
+    echo "$PIDS" | xargs kill 2>/dev/null || true
+    echo "Server stopped (session: $SESSION_DIR)"
+  else
+    echo "No running server found for session: $SESSION_DIR"
+  fi
+
+  # Mark as stopped
+  mkdir -p "$STATE_DIR"
+  echo "$(date +%s)" > "$STATE_DIR/server-stopped"
 
-SESSION_DIR="$1"
-STATE_DIR="$SESSION_DIR/state"
+  # Clean up /tmp sessions only
+  if [[ "$SESSION_DIR" == /tmp/* ]]; then
+    rm -rf "$SESSION_DIR"
+    echo "Temporary session cleaned up."
+  fi
+}
 
-if [[ ! -d "$SESSION_DIR" ]]; then
-  echo "Session directory not found: $SESSION_DIR" >&2
-  exit 1
+# ── With explicit SESSION_DIR arg (coordinator use) ────────────────────────
+if [[ $# -ge 1 ]]; then
+  stop_session "$1"
+  exit 0
 fi
 
-# Read server info to find the process
-if [[ -f "$STATE_DIR/server-info" ]]; then
-  # Find the node process serving this session
-  SERVER_JS="$SESSION_DIR/server.js"
+# ── No args: auto-discover all running deliberate companion sessions ────────
+STOPPED=0
+
+# Search project-local sessions (.deliberate/companion/)
+for SERVER_JS in .deliberate/companion/*/server.js; do
+  [[ -f "$SERVER_JS" ]] || continue
+  SESSION_DIR="$(dirname "$SERVER_JS")"
   PIDS=$(pgrep -f "$SERVER_JS" 2>/dev/null || true)
+  if [[ -n "$PIDS" ]]; then
+    echo "$PIDS" | xargs kill 2>/dev/null || true
+    echo "Server stopped (session: $SESSION_DIR)"
+    mkdir -p "$SESSION_DIR/state"
+    echo "$(date +%s)" > "$SESSION_DIR/state/server-stopped"
+    STOPPED=$((STOPPED + 1))
+  fi
+done
 
+# Search /tmp sessions
+for SERVER_JS in /tmp/deliberate-companion/*/server.js; do
+  [[ -f "$SERVER_JS" ]] || continue
+  SESSION_DIR="$(dirname "$SERVER_JS")"
+  PIDS=$(pgrep -f "$SERVER_JS" 2>/dev/null || true)
+  if [[ -n "$PIDS" ]]; then
+    echo "$PIDS" | xargs kill 2>/dev/null || true
+    echo "Server stopped (session: $SESSION_DIR)"
+    rm -rf "$SESSION_DIR"
+    echo "Temporary session cleaned up."
+    STOPPED=$((STOPPED + 1))
+  fi
+done
+
+# Fallback: kill any node process running a deliberate server.js
+if [[ "$STOPPED" -eq 0 ]]; then
+  PIDS=$(pgrep -f "deliberate-companion.*/server.js" 2>/dev/null || true)
   if [[ -n "$PIDS" ]]; then
     echo "$PIDS" | xargs kill 2>/dev/null || true
     echo "Server stopped."
-  else
-    echo "No running server found for this session."
+    STOPPED=$((STOPPED + 1))
   fi
 fi
 
-# Mark as stopped
-echo "$(date +%s)" > "$STATE_DIR/server-stopped"
-
-# Clean up /tmp sessions only
-if [[ "$SESSION_DIR" == /tmp/* ]]; then
-  rm -rf "$SESSION_DIR"
-  echo "Temporary session cleaned up."
-else
-  echo "Session files preserved at: $SESSION_DIR"
+if [[ "$STOPPED" -eq 0 ]]; then
+  echo "No running deliberate visual companion server found."
 fi