acstack 0.50.2 → 0.50.4

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "acstack",
   "description": "Agentic coding stack where planning grows only as far as needed: a map of decisions, tasks, checks, and handoffs behind a pluggable tracker.",
-  "version": "0.50.2",
+  "version": "0.50.4",
   "author": {
     "name": "ac8318740"
   },

package/assets/terminal-workspace/lib/apply-herdr.sh

@@ -303,6 +303,9 @@ PY
 
   local integ; integ=$(cfg_get herdr.integration none)
   if [ "$integ" != "none" ] && [ -n "$integ" ]; then
-    herdr integration install "$integ" >/dev/null 2>&1 && say "herdr $integ state hook installed"
+    if herdr integration install "$integ" >/dev/null 2>&1; then
+      say "herdr $integ state hook installed"
+      patch_herdr_pi_status
+    fi
   fi
 }

package/assets/terminal-workspace/lib/apply-yazi.sh

@@ -288,11 +288,36 @@ def drop_claimed(t):
     return "\n".join(out)
 
 
+def inspect_available(t):
+    # Preserve custom bindings, including sequences beginning with i.
+    if "i" in CLAIMED or tomllib is None:
+        return False
+    try:
+        data = tomllib.loads(t)
+    except ValueError:
+        return False
+    for section in ("mgr", "manager"):
+        for name in ("prepend_keymap", "keymap", "append_keymap"):
+            for binding in data.get(section, {}).get(name, []):
+                on = binding.get("on", [])
+                if on == "i" or isinstance(on, list) and on[:1] == ["i"]:
+                    return False
+    return True
+
+
 parts = []
 if not claimed(text):
     # A binding conceded below is one this script never writes, so the user's
     # own must survive. Only strip when the bindings are actually going in.
+    inspect = inspect_available(text)
     text = drop_claimed(text)
+    if inspect:
+        parts.append(
+            "[[mgr.prepend_keymap]]\n"
+            'on = "i"\n'
+            """run = 'shell --block -- case %h in *.[mM][dD]|*.[mM][aA][rR][kK][dD][oO][wW][nN]) frogmouth -- %h ;; esac'\n"""
+            'desc = "Inspect Markdown in Frogmouth"'
+        )
     # %h is the hovered file. $0 and $@ are not: yazi runs the template through
     # a shell, where $0 names the shell itself and $@ is empty - both measured,
     # and both would open the browser on nothing at all.

package/assets/terminal-workspace/lib/commands.sh

@@ -189,6 +189,7 @@ PY
              say "delta unset as git pager"
              say "now set delta.enabled: false in $CFG so apply does not restore it" ;;
       herdr)
+        patch_herdr_pi_status --remove
         py "$HERDR_CFG" "$BEGIN" "$END" <<'PY'
 import re, sys, os
 p, b, e = sys.argv[1:4]

package/assets/terminal-workspace/lib/patch-herdr-pi.py

@@ -0,0 +1,86 @@
+"""Keep herdr's single Pi status writer aware of background subagents."""
+import os
+import sys
+import tempfile
+from pathlib import Path
+
+path = Path(sys.argv[1])
+remove = sys.argv[2:] == ["--remove"]
+if not path.exists():
+    sys.exit(0)
+if path.is_symlink() or not path.is_file():
+    sys.exit("  pi: herdr status patch skipped: reporter is not a regular file")
+
+begin = "// >>> acstack terminal-workspace: background status >>>"
+end = "// <<< acstack terminal-workspace: background status <<<"
+
+
+def marked(text):
+    return f"  {begin}\n{text}  {end}\n"
+
+
+replacements = [
+    ("  let agentActive = false;\n", "  let agentActive = false;\n" + marked('''  // shortcut: no workflow/reload replay; extend when Tintin exposes active-run snapshots.
+  const backgroundAgents = new Set<string>();
+  let busyCount = 0;
+''')),
+    ("    if (agentActive) {\n", marked('''    // blocked > active parent or children > idle
+    if (agentActive || backgroundAgents.size > 0 || busyCount > 0) {
+''')),
+    ('  pi.events.on("herdr:blocked", (data) => {\n', marked('''  // Tintin's in-process children share this bus, but never own the pane status.
+  pi.events.on("subagents:started", (data) => {
+    if (!rootSession || typeof data?.id !== "string" || !data.id) return;
+    backgroundAgents.add(data.id);
+    publishState();
+  });
+  for (const event of ["subagents:completed", "subagents:failed"]) {
+    pi.events.on(event, (data) => {
+      if (!rootSession || typeof data?.id !== "string") return;
+      backgroundAgents.delete(data.id);
+      publishState();
+    });
+  }
+  // nicobailon/pi-subagents uses balanced active intervals instead of IDs.
+  pi.events.on("herdr:busy", (data) => {
+    if (!rootSession || typeof data?.active !== "boolean") return;
+    busyCount = Math.max(0, busyCount + (data.active ? 1 : -1));
+    publishState();
+  });
+''') + '  pi.events.on("herdr:blocked", (data) => {\n'),
+]
+
+text = path.read_text()
+if not text.startswith("// installed by herdr\n") or "HERDR_INTEGRATION_ID=pi\n" not in text:
+    sys.exit("  pi: herdr status patch skipped: unrecognised reporter")
+# These lifecycle guards must still exist before adding shared-bus listeners.
+required = [
+    '  pi.on("agent_settled", (_event, ctx) => {',
+    '    if (ctx?.mode !== "tui") {',
+    '    if (!rootSession || ctx?.isIdle?.() !== true) {',
+]
+if any(text.count(anchor) != 1 for anchor in required):
+    sys.exit("  pi: herdr status patch skipped: upstream lifecycle changed")
+patched = begin in text
+if patched == (not remove):
+    # Validate every managed block before treating an existing patch as ours.
+    if patched and any(text.count(new) != 1 for _, new in replacements):
+        sys.exit("  pi: herdr status patch skipped: managed blocks were changed")
+    sys.exit(0)
+
+for old, new in replacements:
+    if text.count(new if remove else old) != 1:
+        sys.exit("  pi: herdr status patch skipped: upstream layout changed")
+for old, new in replacements:
+    text = text.replace(new, old, 1) if remove else text.replace(old, new, 1)
+
+# Validate all matches first, then replace atomically so a failed patch leaves no partial writer.
+fd, temporary = tempfile.mkstemp(dir=path.parent, prefix=".acstack-herdr-")
+try:
+    with os.fdopen(fd, "w") as stream:
+        stream.write(text)
+    os.chmod(temporary, path.stat().st_mode)
+    os.replace(temporary, path)
+finally:
+    if os.path.exists(temporary):
+        os.unlink(temporary)
+print(f"  pi: herdr background status patch {'removed' if remove else 'applied'}")

package/assets/terminal-workspace/lib/pi-packages.sh

@@ -301,6 +301,7 @@ apply_pi() {
   pi_state merge "$PI_SUBAGENT_CFG" subagent '{"toolActivation": "eager"}'
   pi_state md write
   patch_pi_win32_paste
+  patch_herdr_pi_status
 
   # Logging in is interactive and stores a credential, so it stays the user's.
   [ -f "$PI_DIR/auth.json" ] || say "pi: not logged in yet. Run pi, then /login"
@@ -311,6 +312,7 @@ apply_pi() {
 # and leaves the pi binary, its logins and its sessions alone.
 remove_pi() {
   find_pi
+  patch_herdr_pi_status --remove
   pi_question_bridge remove
   if pi_state parses "$PI_SETTINGS"; then
     pi_unpin "$PI_ACSTACK"

package/assets/terminal-workspace/lib/pi-state.sh

@@ -404,6 +404,12 @@ find_pi() {
 }
 
 
+# Keep herdr's own writer as the sole pane authority; reapply after it is replaced.
+patch_herdr_pi_status() {
+  python3 "$SCRIPT_DIR/lib/patch-herdr-pi.py" "$PI_DIR/extensions/herdr-agent-state.ts" "$@" \
+    || say "pi: herdr background status patch was not applied; reporter left unchanged"
+}
+
 # Pi's bundled TUI does not always receive Windows Terminal paste as plain
 # text. Through Herdr and ConPTY it can arrive as Win32 input records such as
 # ESC[13;28;13;1;0;1_. Patch the installed bundle after install or update.

package/assets/terminal-workspace/lib/versions.sh

@@ -204,8 +204,12 @@ upgrade_one() {  # upgrade_one <name>
     find_pi
   fi
   case "$1" in
-    herdr)     herdr update >/dev/null 2>&1 \
-                 && say "herdr updated" || say "herdr update failed" ;;
+    herdr)     if herdr update >/dev/null 2>&1; then
+                 say "herdr updated"
+                 patch_herdr_pi_status
+               else
+                 say "herdr update failed"
+               fi ;;
     gh-dash)   gh extension upgrade dlvhdr/gh-dash >/dev/null 2>&1 \
                  && say "gh-dash upgraded" || say "gh-dash upgrade failed" ;;
     harlequin) uv tool upgrade harlequin >/dev/null 2>&1 \

package/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "acstack-cli",
-  "version": "0.50.2",
+  "version": "0.50.4",
   "description": "CLI for the ACStack agentic coding stack",
   "type": "module",
   "license": "MIT",

package/docs/terminal-workspace-keys.md

@@ -74,6 +74,7 @@ Add `shift` to any of these to open it in a full tab instead of a popup.
 | Key | Does |
 |---|---|
 | `e` | edit in `$EDITOR` |
+| `i` | inspect Markdown in Frogmouth (install with `pipx install frogmouth`; existing `i` bindings take precedence) |
 | `o` | open by rule, which for markdown is the reader |
 | `b` | open in the browser you are sitting at |
 | `D` | send the file to your laptop over Taildrop |

package/docs/terminal-workspace.md

@@ -396,6 +396,14 @@ What `apply` does:
 - It writes the four keys under `pi.settings` into `~/.pi/agent/settings.json`
 - It patches pi's bundled TUI so Windows Terminal Win32 input records paste as text
     - `pi update` can replace the bundle, so `setup.sh upgrade pi` reapplies the patch
+- It patches an existing herdr Pi reporter to keep the parent working while background subagents run
+    - Questions still show blocked, even when children are active
+    - The patch handles `pi-subagents` busy events and `@tintinweb/pi-subagents` Agent lifecycle events
+    - Tintin workflows and children already running during an extension reload lack lifecycle replay and remain outside this tracking
+    - `apply`, herdr integration installation, and `setup.sh upgrade herdr` reapply the patch
+    - Direct `herdr integration install pi` overwrites the patch; run `apply` afterward, then restart Pi or use `/reload`
+    - Setup leaves unknown reporter layouts unchanged and reports the skipped patch
+    - `disable pi`, `disable herdr`, and `uninstall` remove unchanged managed patch blocks
 
 How it treats pi's files:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "acstack",
-  "version": "0.50.2",
+  "version": "0.50.4",
   "private": false,
   "description": "Agentic coding stack with first-class Claude Code and pi support: skills, orchestrator instructions, and verification agents.",
   "license": "MIT",

package/skills/terminal-workspace/SKILL.md

@@ -79,6 +79,7 @@ The two files this skill works with:
 - **Config**: `~/.config/acstack/terminal-workspace.yaml`, copied from `assets/terminal-workspace/config.example.yaml`
 - **Script**: `assets/terminal-workspace/setup.sh` in this plugin
 
+Yazi's `i` opens Markdown in Frogmouth (`pipx install frogmouth`); existing `i` bindings take precedence, and `e` keeps editing in `$EDITOR`.
 The keys, on one page: [docs/terminal-workspace-keys.md](../../docs/terminal-workspace-keys.md).
 
 Background and why each piece is there: