@miller-tech/uap 1.148.11 → 1.148.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miller-tech/uap",
-  "version": "1.148.11",
+  "version": "1.148.12",
   "description": "Autonomous AI agent memory system with CLAUDE.md protocol enforcement",
   "type": "module",
   "main": "dist/index.js",

package/templates/hooks/deliver_autoroute.py

@@ -47,21 +47,36 @@ def decide(out: dict, tool: str, args: dict, autoroute_on: bool, seen_files: set
     m = re.fullmatch(r'uap deliver "(.+)"', hint.strip())
     if m:
         hint = m.group(1)
-    file_path = args.get("file_path") or args.get("path") or args.get("target") or ""
+    # Accept the file-path key under ANY agent spelling. The enforcer was fixed
+    # for this long ago; autoroute was not — so for opencode (which sends
+    # `filePath`) file_path was always "", `spawn` was always False, and autoroute
+    # was INERT: the gate blocked the edit, logged the intent, told the model to
+    # call deliver… and deliver never ran. Observed live: 3 routed intents, 0
+    # deliver runs, 0 files changed — work blocked but never delivered.
+    file_path = (
+        args.get("file_path") or args.get("filePath") or args.get("path")
+        or args.get("target") or args.get("filename") or args.get("file") or ""
+    )
 
     if route != "deliver":
         return {"message": reason, "route": route, "spawn": False,
-                "file_path": file_path, "hint": hint, "intent": None}
+                "file_path": file_path, "hint": hint, "dedup_key": "", "intent": None}
 
     intent = {"ts": int(time.time()), "tool": tool, "file_path": file_path, "hint": hint}
-    spawn = bool(autoroute_on and hint and file_path and file_path not in seen_files)
+    # Dedup on the file when we have one, else on the hint itself. Requiring a
+    # file_path made an entire class unspawnable: a BASH-routed source-write
+    # (`cat > app.js <<EOF`) carries a `command`, not a path — so those intents
+    # were blocked and then silently dropped. The hint is what deliver actually
+    # runs, so it is the correct spawn key.
+    dedup_key = file_path or hint
+    spawn = bool(autoroute_on and hint and dedup_key and dedup_key not in seen_files)
     message = reason
     if spawn:
         message = reason + " [auto-routed to `uap deliver` — running in the background]"
-    elif autoroute_on and file_path in seen_files:
-        message = reason + " [already auto-routed to `uap deliver` for this file — see .uap/autoroute.log / pending-deliver.jsonl]"
+    elif autoroute_on and dedup_key and dedup_key in seen_files:
+        message = reason + " [already auto-routed to `uap deliver` for this change — see .uap/autoroute.log / pending-deliver.jsonl]"
     return {"message": message, "route": route, "spawn": spawn,
-            "file_path": file_path, "hint": hint, "intent": intent}
+            "file_path": file_path, "hint": hint, "dedup_key": dedup_key, "intent": intent}
 
 
 def _seen_path(root: Path) -> Path:
@@ -160,8 +175,12 @@ def main() -> None:
 
     if d["spawn"]:
         try:
+            # Dedup on the SAME key `decide` gated on (file when present, else the
+            # hint) — writing file_path here would record "" for a bash-routed
+            # intent and never dedup it.
+            key = d.get("dedup_key") or d["file_path"] or d["hint"]
             with _seen_path(root).open("a") as f:
-                f.write(d["file_path"].replace("\n", " ").replace("\r", " ") + "\n")
+                f.write(key.replace("\n", " ").replace("\r", " ") + "\n")
         except Exception:
             pass
         _spawn_deliver(root, d["hint"])

package/tools/agents/tests/test_deliver_autoroute.py

@@ -69,5 +69,53 @@ class LoggingTest(unittest.TestCase):
             self.assertEqual(rec["file_path"], "/repo/src/foo.ts")
 
 
+class AgentKeySpellingTest(unittest.TestCase):
+    """autoroute must accept EVERY agent's file-path key.
+
+    The enforcer was fixed for this long ago; autoroute was not — so for opencode
+    (which sends `filePath`) file_path was always "", `spawn` was always False, and
+    autoroute was INERT: the gate blocked the edit, logged the intent, told the
+    model to call deliver… and deliver never ran. Observed live: 3 routed intents,
+    0 deliver runs, 0 files changed — work blocked but never delivered. The old
+    tests only ever passed snake_case, which is why this went unnoticed.
+    """
+
+    def test_opencode_filePath_spawns(self):
+        d = mod.decide(BLOCK_OUT, "Write", {"filePath": "/repo/src/foo.ts"}, True, set())
+        self.assertTrue(d["spawn"])
+        self.assertEqual(d["dedup_key"], "/repo/src/foo.ts")
+
+    def test_claude_file_path_still_spawns(self):
+        d = mod.decide(BLOCK_OUT, "Write", {"file_path": "/repo/src/foo.ts"}, True, set())
+        self.assertTrue(d["spawn"])
+
+    def test_other_spellings(self):
+        for key in ("path", "target", "filename", "file"):
+            d = mod.decide(BLOCK_OUT, "Write", {key: "/repo/src/foo.ts"}, True, set())
+            self.assertTrue(d["spawn"], key)
+
+
+class BashRoutedIntentTest(unittest.TestCase):
+    """A BASH-routed source-write carries a `command`, not a path — requiring
+    file_path made that whole class unspawnable (blocked, then silently dropped).
+    The hint is what deliver actually runs, so it is the correct spawn key."""
+
+    def test_bash_intent_spawns_on_hint_alone(self):
+        d = mod.decide(BLOCK_OUT, "Bash", {"command": "cat > src/app.js <<EOF"}, True, set())
+        self.assertTrue(d["spawn"])
+        self.assertEqual(d["file_path"], "")           # genuinely has no path
+        self.assertEqual(d["dedup_key"], d["hint"])    # …so it dedups on the hint
+
+    def test_bash_intent_dedupes_on_hint(self):
+        d1 = mod.decide(BLOCK_OUT, "Bash", {"command": "cat > a.js <<EOF"}, True, set())
+        d2 = mod.decide(BLOCK_OUT, "Bash", {"command": "cat > a.js <<EOF"}, True, {d1["dedup_key"]})
+        self.assertTrue(d1["spawn"])
+        self.assertFalse(d2["spawn"])  # no double-spawn for the same change
+
+    def test_still_never_spawns_without_a_hint(self):
+        d = mod.decide({"reason": "x", "route": "deliver"}, "Bash", {"command": "cat > a.js"}, True, set())
+        self.assertFalse(d["spawn"])
+
+
 if __name__ == "__main__":
     unittest.main()