pi-rolecast 0.2.0 → 0.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-rolecast",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Multi-agent role framework for Pi. Groups of specialist agents (coding, video, etc.) bound to per-role models, dispatched via pi-subagents. Profile-driven, gate-verified, language-agnostic.",
   "type": "module",
   "license": "MIT",

package/scripts/install.sh

@@ -31,11 +31,9 @@ if [[ -n "$DRY_RUN" ]]; then
 fi
 mkdir -p "$PREFIX"
 
-# Framework symlink (new pi-rolecast name; keep removing the legacy name).
-if [[ ! -e "$PREFIX/pi-rolecast" ]]; then
-  ln -s "$FRAMEWORK_ROOT" "$PREFIX/pi-rolecast"
-  echo "linked $PREFIX/pi-rolecast to $FRAMEWORK_ROOT"
-fi
+# Remove legacy pi-agent-workflow framework symlink if it exists.
+# v0.2.3: no longer creates ~/.pi/agent/pi-rolecast symlink — npm install
+# path (~/.pi/agent/npm/node_modules/pi-rolecast) is canonical.
 if [[ -e "$PREFIX/pi-agent-workflow" ]]; then
   echo "removing legacy $PREFIX/pi-agent-workflow symlink"
   rm -f "$PREFIX/pi-agent-workflow"
@@ -58,9 +56,19 @@ fi
 # Global role symlinks for every role-packs/<group>/<role>.md.
 agents_dir="$PREFIX/agents"
 mkdir -p "$agents_dir"
+# Clean up ANY pre-existing symlinks pointing at the legacy framework root.
+# Old symlinks would dangle after `pi-agent-workflow` is removed; clean first.
+if [[ -d "$agents_dir" ]]; then
+  while IFS= read -r stale; do
+    target="$(readlink "$stale" 2>/dev/null || true)"
+    if [[ -n "$target" && "$target" == *pi-agent-workflow* ]]; then
+      rm -f "$stale"
+    fi
+  done < <(find "$agents_dir" -maxdepth 1 -type l -name '*.md')
+fi
 linked=0
 while IFS= read -r md; do
-  role=$(basename "$md" .md)
+  role=$(basename "$md" .md)        # e.g. "coding-architect" (full prefixed name)
   link_path="$agents_dir/$role.md"
   if [[ -L "$link_path" ]]; then
     rm -f "$link_path"

package/scripts/sync_settings.py

@@ -252,7 +252,8 @@ def _write_agents(agents_dir: Path, framework_root: Path,
                   dry_run: bool, registry) -> int:
     """Write project-local copies of role-packs/<group>/<role>.md for every
     bound role. Uses full role name (`<group>-<role>`) as the filename so
-    pi-subagents can find it via @<full-name> mention syntax."""
+    pi-subagents can find it via @<full-name> mention syntax.
+    """
     roles = available_roles(framework_root, groups=enabled_groups or None)
     if dry_run:
         for role_full_name, rd in sorted(roles.items()):
@@ -274,12 +275,13 @@ def _write_agents(agents_dir: Path, framework_root: Path,
             print("warning: role-packs file not found: " + str(rd.file_path),
                   file=sys.stderr)
             continue
-        dst = agents_dir / (role_full_name + ".md")
         body = src.read_text()
         model_id = new_overrides[role_full_name]["model"]
         provider = _provider_for_model(model_id, registry)
         full = (provider + "/" + model_id) if provider else model_id
         updated = _set_frontmatter_field(body, "model", full)
+        # v0.2.0+ canonical filename
+        dst = agents_dir / (role_full_name + ".md")
         dst.write_text(updated)
         written += 1
     print("wrote " + str(written) + " project-local agent files to " + str(agents_dir))

package/scripts/uninstall.sh

@@ -0,0 +1,150 @@
+#!/usr/bin/env bash
+# pi-rolecast uninstaller. Removes all framework artifacts created by
+# install.sh and sync_settings.py. Safe to run multiple times (idempotent).
+#
+# Removes:
+#   - ~/.pi/agent/pi-rolecast framework symlink
+#   - ~/.pi/agent/agents/*.md symlinks whose target is inside a pi-rolecast
+#     framework
+#   - .pi/agents/<framework-role>.md files matching framework role names
+#     (in cwd)
+#   - subagents.agentOverrides entries for framework roles in settings.json
+#   - empty agent directories if nothing else remains
+#
+# Does NOT remove:
+#   - ~/.pi/agent/npm/node_modules/pi-rolecast/  (run `pi uninstall npm:pi-rolecast` first)
+#   - user-created agent files in .pi/agents/ (e.g. custom-role.md)
+#   - user-created symlinks in ~/.pi/agent/agents/ (e.g. targeting other frameworks)
+#   - settings.json entries for non-framework roles
+set -euo pipefail
+
+PREFIX="${HOME}/.pi/agent"
+DRY_RUN=""
+KEEP_PROJECT_AGENTS=""
+SETTINGS_JSON=""
+GLOBAL_AGENTS_DIR=""
+PROJECT_AGENTS_DIR=""
+FRAMEWORK_ROOT=""
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --prefix)               PREFIX="$2"; shift 2 ;;
+    --framework-root)       FRAMEWORK_ROOT="$2"; shift 2 ;;
+    --dry-run)              DRY_RUN="1"; shift ;;
+    --settings)             SETTINGS_JSON="$2"; shift 2 ;;
+    --global-agents-dir)    GLOBAL_AGENTS_DIR="$2"; shift 2 ;;
+    --project-agents-dir)   PROJECT_AGENTS_DIR="$2"; shift 2 ;;
+    --keep-project-agents)  KEEP_PROJECT_AGENTS="1"; shift ;;
+    *) echo "unknown argument: $1" >&2; exit 1 ;;
+  esac
+done
+
+[[ -z "$SETTINGS_JSON" ]] && SETTINGS_JSON="$PREFIX/settings.json"
+[[ -z "$GLOBAL_AGENTS_DIR" ]] && GLOBAL_AGENTS_DIR="$PREFIX/agents"
+[[ -z "$PROJECT_AGENTS_DIR" ]] && PROJECT_AGENTS_DIR="$(pwd)/.pi/agents"
+
+# Find framework root to enumerate framework role names. Without this we
+# cannot tell framework-written project-local files from user customizations.
+if [[ -z "$FRAMEWORK_ROOT" ]]; then
+  if [[ -d "./role-packs" ]]; then
+    FRAMEWORK_ROOT="$(cd . && pwd)"
+  elif [[ -d "$PREFIX/npm/node_modules/pi-rolecast/role-packs" ]]; then
+    FRAMEWORK_ROOT="$PREFIX/npm/node_modules/pi-rolecast"
+  fi
+fi
+
+framework_roles=()
+if [[ -n "$FRAMEWORK_ROOT" ]]; then
+  while IFS= read -r role; do
+    framework_roles+=("$role")
+  done < <(find "$FRAMEWORK_ROOT/role-packs" -type f -name '*.md' \
+              -exec basename {} .md \; 2>/dev/null | sort -u)
+fi
+
+removed=0
+_action() {
+  # $1 = action (rm), $2 = path, $3 = reason
+  if [[ -n "$DRY_RUN" ]]; then
+    echo "would $1 $2 ($3)"
+  else
+    if "$1" "$2" 2>/dev/null; then
+      echo "$1 $2 ($3)"
+      removed=$((removed + 1))
+    fi
+  fi
+}
+
+# 1. Framework symlink ~/.pi/agent/pi-rolecast
+link="$PREFIX/pi-rolecast"
+if [[ -L "$link" ]]; then
+  target="$(readlink "$link")"
+  case "$target" in
+    *pi-rolecast*) _action rm "$link" "framework symlink" ;;
+  esac
+fi
+
+# 2. Global agent symlinks ~/.pi/agent/agents/*.md
+#    Only remove symlinks whose target is inside a pi-rolecast framework.
+if [[ -d "$GLOBAL_AGENTS_DIR" ]]; then
+  while IFS= read -r l; do
+    target="$(readlink "$l")"
+    case "$target" in
+      *pi-rolecast*) _action rm "$l" "global agent symlink" ;;
+    esac
+  done < <(find "$GLOBAL_AGENTS_DIR" -maxdepth 1 -type l -name '*.md' 2>/dev/null)
+fi
+
+# 3. Project-local agent files .pi/agents/<role>.md
+#    Only remove files matching framework role names. Skip symlinks (user-created).
+if [[ -z "$KEEP_PROJECT_AGENTS" && -d "$PROJECT_AGENTS_DIR" && ${#framework_roles[@]} -gt 0 ]]; then
+  for role in "${framework_roles[@]}"; do
+    f="$PROJECT_AGENTS_DIR/$role.md"
+    [[ -e "$f" || -L "$f" ]] || continue
+    [[ -L "$f" ]] && continue
+    _action rm "$f" "project-local agent file"
+  done
+elif [[ -z "$KEEP_PROJECT_AGENTS" && -d "$PROJECT_AGENTS_DIR" ]]; then
+  echo "warning: framework root not found, skipping project-local agent cleanup" >&2
+  echo "  hint: cd into a project with .pi/rolecast.yaml, or run from pi-rolecast repo" >&2
+fi
+
+# 4. Clean settings.json subagents.agentOverrides for framework roles
+if [[ -f "$SETTINGS_JSON" && ${#framework_roles[@]} -gt 0 ]]; then
+  python3 - "$SETTINGS_JSON" "${framework_roles[@]}" <<'PYEOF'
+import json, sys
+path = sys.argv[1]
+roles = set(sys.argv[2:])
+try:
+    with open(path) as f:
+        s = json.load(f)
+except (FileNotFoundError, json.JSONDecodeError):
+    sys.exit(0)
+sub = s.setdefault("subagents", {})
+ov = sub.get("agentOverrides", {}) or {}
+cleared = [k for k in list(ov) if k in roles]
+for k in cleared:
+    del ov[k]
+sub["agentOverrides"] = ov
+s["subagents"] = sub
+with open(path, "w") as f:
+    json.dump(s, f, indent=2, sort_keys=True)
+    f.write("\n")
+print(f"removed {len(cleared)} framework overrides from {path}")
+PYEOF
+fi
+
+# 5. Remove empty agent directories (after all cleanup)
+for d in "$GLOBAL_AGENTS_DIR" "$PROJECT_AGENTS_DIR"; do
+  if [[ -d "$d" && -z "$(ls -A "$d" 2>/dev/null)" ]]; then
+    if [[ -n "$DRY_RUN" ]]; then
+      echo "would remove empty dir $d"
+    else
+      if rmdir "$d" 2>/dev/null; then
+        echo "rmdir $d (empty dir)"
+        removed=$((removed + 1))
+      fi
+    fi
+  fi
+done
+
+echo "uninstall complete: removed $removed items"