pi-rolecast 0.2.1 → 0.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-rolecast",
-  "version": "0.2.1",
+  "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"
@@ -70,26 +68,15 @@ if [[ -d "$agents_dir" ]]; then
 fi
 linked=0
 while IFS= read -r md; do
-  role=$(basename "$md" .md)        # e.g. "coding-architect"
-  bare="${role#coding-}"           # legacy compat name e.g. "architect" (for coding group)
+  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"
   fi
   ln -s "$md" "$link_path"
   linked=$((linked + 1))
-  # Legacy shim: create <bare-role>.md symlink for extensions that key
-  # off filename (e.g. bundled pi-cc-extensions). Removed in v0.3.0.
-  if [[ "$role" == "$bare" ]]; then
-    continue                        # safety: if role had no group prefix, skip
-  fi
-  legacy_link="$agents_dir/$bare.md"
-  if [[ ! -L "$legacy_link" && ! -e "$legacy_link" ]]; then
-    ln -s "$md" "$legacy_link"
-    linked=$((linked + 1))
-  fi
 done < <(find "$FRAMEWORK_ROOT/role-packs" -type f -name '*.md' 2>/dev/null | sort)
-echo "linked $linked role agents from role-packs/ (incl. legacy compat shims)"
+echo "linked $linked role agents from role-packs/"
 
 # Warn if pi-subagents is missing — required for actual role dispatch.
 SETTINGS_JSON="$PREFIX/settings.json"

package/scripts/sync_settings.py

@@ -253,11 +253,6 @@ def _write_agents(agents_dir: Path, framework_root: Path,
     """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.
-
-    Also writes a legacy `<bare-role>.md` shim for extensions that key off
-    filename (e.g. the bundled pi-cc-extensions subagent reference provider).
-    Both files contain the same body, just different filenames. The legacy
-    shim is removed in v0.3.0.
     """
     roles = available_roles(framework_root, groups=enabled_groups or None)
     if dry_run:
@@ -269,10 +264,6 @@ def _write_agents(agents_dir: Path, framework_root: Path,
             full = (provider + "/" + model_id) if provider else model_id
             print("would write " + str(agents_dir / (role_full_name + ".md")) +
                   " model=" + full)
-            legacy = rd.role
-            if legacy and legacy != role_full_name:
-                print("would write " + str(agents_dir / (legacy + ".md")) +
-                      " model=" + full + " (legacy shim)")
         return 0
     agents_dir.mkdir(parents=True, exist_ok=True)
     written = 0
@@ -293,17 +284,7 @@ def _write_agents(agents_dir: Path, framework_root: Path,
         dst = agents_dir / (role_full_name + ".md")
         dst.write_text(updated)
         written += 1
-        # Legacy shim: bare role name for filename-keyed extensions.
-        # Only created for the coding group; other groups have no legacy.
-        legacy = rd.role
-        if legacy and legacy != role_full_name and rd.group == "coding":
-            legacy_dst = agents_dir / (legacy + ".md")
-            if legacy_dst.exists() or legacy_dst.is_symlink():
-                legacy_dst.unlink()
-            legacy_dst.write_text(updated)
-            written += 1
-    print("wrote " + str(written) + " project-local agent files to " + str(agents_dir) +
-          " (incl. legacy compat shims)")
+    print("wrote " + str(written) + " project-local agent files to " + str(agents_dir))
     return 0
 
 

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"