@delorenj/pjangler 1.4.6 → 1.5.2

package/dist/assets/project-notebook-skill/SHA256SUMS

@@ -1,10 +1,10 @@
 dba418f3b40d2e1362e615bdfda6cb1616ec5e49e2e8bd81ca48144ef219b29f  agents/openai.yaml
 8a469caf88ed05dc466ffaad611cb45dd0c26046595d9538415d7c89287c8800  hooks/claude.settings.json
 8a469caf88ed05dc466ffaad611cb45dd0c26046595d9538415d7c89287c8800  hooks/hooks.master.json
-848d2f5ba9000c2538e21d5a21257c72a5e4799ee3a9145a4eb9d7b00179e52b  hooks/session-end.sh
-f146a81c3e6cfbbce28c85c3c2163255974abef61339638c2a0ac9902346905e  hooks/session-start.sh
-a25d1d34f9960b2b3bf17a00191d5477b81a05bffe5d4e183c2954cc3975b535  references/configuration.md
+bb37fa22701c93bc02afde2b54be7c34ebc939e52628f4b13f40d9aa55591486  hooks/session-end.sh
+dca8e974513fd3ddaba3d94cd0386f4ac277f548b2af583c7f96b86270655662  hooks/session-start.sh
+70bc3e9b084ef4f2e760987dea1f1cf97d85c6e4ad6312853fd4b9281c57184e  references/configuration.md
 6a61d9dd0d5dd32f8a7fefcb82001bbad90523704fa1500b7e0becc3a848545c  references/recovery.md
-95d1b6199b0416f7fdf3306215bc10a9a647f64654ccbb26786cc228fc35ac37  scripts/project-hooks.py
+62b15b5de6a67f2cf26d279119e1160ea82e31658446af660727643c76f9f3f4  scripts/project-hooks.py
 8567ab2567ec412eb8d673dd713340b81a45e69374d9baee3cb7e827999be437  SKILL.md
-7e16380d079277ee08b8ea64767b1596dc4e8f7ce1bd89c45d9d4821f6d493d3  tests/test_project_hooks.py
+d4dca28cb7074edce3cf1e13b85e9cd57b6509c2726a561a5d02ea8e36ca1028  tests/test_project_hooks.py

package/dist/assets/project-notebook-skill/export-manifest.json

@@ -19,17 +19,17 @@
     },
     {
       "path": "hooks/session-end.sh",
-      "sha256": "848d2f5ba9000c2538e21d5a21257c72a5e4799ee3a9145a4eb9d7b00179e52b",
+      "sha256": "bb37fa22701c93bc02afde2b54be7c34ebc939e52628f4b13f40d9aa55591486",
       "mode": "0755"
     },
     {
       "path": "hooks/session-start.sh",
-      "sha256": "f146a81c3e6cfbbce28c85c3c2163255974abef61339638c2a0ac9902346905e",
+      "sha256": "dca8e974513fd3ddaba3d94cd0386f4ac277f548b2af583c7f96b86270655662",
       "mode": "0755"
     },
     {
       "path": "references/configuration.md",
-      "sha256": "a25d1d34f9960b2b3bf17a00191d5477b81a05bffe5d4e183c2954cc3975b535",
+      "sha256": "70bc3e9b084ef4f2e760987dea1f1cf97d85c6e4ad6312853fd4b9281c57184e",
       "mode": "0644"
     },
     {
@@ -39,7 +39,7 @@
     },
     {
       "path": "scripts/project-hooks.py",
-      "sha256": "95d1b6199b0416f7fdf3306215bc10a9a647f64654ccbb26786cc228fc35ac37",
+      "sha256": "62b15b5de6a67f2cf26d279119e1160ea82e31658446af660727643c76f9f3f4",
       "mode": "0755"
     },
     {
@@ -49,7 +49,7 @@
     },
     {
       "path": "tests/test_project_hooks.py",
-      "sha256": "7e16380d079277ee08b8ea64767b1596dc4e8f7ce1bd89c45d9d4821f6d493d3",
+      "sha256": "d4dca28cb7074edce3cf1e13b85e9cd57b6509c2726a561a5d02ea8e36ca1028",
       "mode": "0644"
     }
   ]

package/dist/assets/project-notebook-skill/hooks/session-end.sh

@@ -16,7 +16,7 @@ DISPLAY_EVENT = "SessionEnd"
 HOOK_EVENT = "session-close"
 REQUEST_LIMIT_BYTES = 1_048_576
 STREAM_LIMIT_BYTES = REQUEST_LIMIT_BYTES + 1
-CHILD_TIMEOUT_SECONDS = 0.75
+CHILD_TIMEOUT_SECONDS = 4.0
 NODE_BINARY = Path("/usr/bin/node")
 AUTH_VARIABLE = "OPEN_NOTEBOOK_PASSWORD"
 
@@ -208,8 +208,27 @@ def invoke(entry: pwd.struct_passwd, launcher: Path, payload: bytes) -> int:
     return 0
 
 
+def hub_owned(home: Path) -> bool:
+    if os.environ.get("PJ_HOOK_FROM_HUB") == "1":
+        return False
+    if os.environ.get("BB_HOOK_HUB") == "off":
+        return False
+    helper = home / ".agents/hooks/hub/ownership.py"
+    if not helper.is_file():
+        return False
+    try:
+        return subprocess.run(
+            [sys.executable, "-I", str(helper), "project-notebook-end", "--cli", "claude"],
+            capture_output=True, timeout=0.5, check=False,
+        ).returncode == 0
+    except (OSError, subprocess.SubprocessError):
+        return False
+
+
 def main() -> int:
     entry = canonical_identity()
+    if hub_owned(Path(entry.pw_dir)):
+        return 0
     validate_node_binary()
     launcher = resolve_launcher(entry)
     payload = sys.stdin.buffer.read(STREAM_LIMIT_BYTES)

package/dist/assets/project-notebook-skill/hooks/session-start.sh

@@ -16,7 +16,7 @@ DISPLAY_EVENT = "SessionStart"
 HOOK_EVENT = "session-start"
 REQUEST_LIMIT_BYTES = 1_048_576
 STREAM_LIMIT_BYTES = REQUEST_LIMIT_BYTES + 1
-CHILD_TIMEOUT_SECONDS = 2.25
+CHILD_TIMEOUT_SECONDS = 4.0
 NODE_BINARY = Path("/usr/bin/node")
 AUTH_VARIABLE = "OPEN_NOTEBOOK_PASSWORD"
 
@@ -208,8 +208,27 @@ def invoke(entry: pwd.struct_passwd, launcher: Path, payload: bytes) -> int:
     return 0
 
 
+def hub_owned(home: Path) -> bool:
+    if os.environ.get("PJ_HOOK_FROM_HUB") == "1":
+        return False
+    if os.environ.get("BB_HOOK_HUB") == "off":
+        return False
+    helper = home / ".agents/hooks/hub/ownership.py"
+    if not helper.is_file():
+        return False
+    try:
+        return subprocess.run(
+            [sys.executable, "-I", str(helper), "project-notebook-start", "--cli", "claude"],
+            capture_output=True, timeout=0.5, check=False,
+        ).returncode == 0
+    except (OSError, subprocess.SubprocessError):
+        return False
+
+
 def main() -> int:
     entry = canonical_identity()
+    if hub_owned(Path(entry.pw_dir)):
+        return 0
     validate_node_binary()
     launcher = resolve_launcher(entry)
     payload = sys.stdin.buffer.read(STREAM_LIMIT_BYTES)

package/dist/assets/project-notebook-skill/references/configuration.md

@@ -91,3 +91,5 @@ the 1,048,576-byte request ceiling was exceeded. Valid-size input is sent over
 stdin with an explicit child timeout shorter than the outer hook timeout.
 Resolution, validation, input, timeout, and PJangler failures are bounded and
 fail open without creating wrapper-local state.
+
+When the Bloodbank hub owns `project-notebook-start` or `project-notebook-end`, installation removes only that event’s native notebook copy and `check` accepts the centralized owner. A paused hub concern stays owned. Without hub ownership, the standalone Claude projection remains available. Legacy wrappers also defer to the central owner, while supervised hub children execute normally.

package/dist/assets/project-notebook-skill/scripts/project-hooks.py

@@ -14,6 +14,7 @@ import posixpath
 import secrets
 import shlex
 import stat
+import subprocess
 import sys
 from collections.abc import Iterator, Sequence
 from dataclasses import dataclass
@@ -433,10 +434,32 @@ def _dedicated_owned_group(group: dict[str, Any], event: str) -> bool:
     )
 
 
+def _hub_owned(event: str) -> bool:
+    """The central registry remains the owner while a concern is paused."""
+    if os.environ.get("BB_HOOK_HUB") == "off":
+        return False
+    helper = Path.home() / ".agents/hooks/hub/ownership.py"
+    if not helper.is_file():
+        return False
+    concern = "project-notebook-start" if event == "SessionStart" else "project-notebook-end"
+    try:
+        return subprocess.run(
+            [sys.executable, str(helper), concern, "--cli", "claude"],
+            capture_output=True, timeout=1, check=False,
+        ).returncode == 0
+    except (OSError, subprocess.SubprocessError):
+        return False
+
+
 def _install_projection(live: dict[str, Any], fragment: dict[str, Any]) -> dict[str, Any]:
-    merged = copy.deepcopy(live)
+    owned_events = {event for event in EVENT_ORDER if _hub_owned(event)}
+    merged = _uninstall_projection(live, owned_events)
+    if len(owned_events) == len(EVENT_ORDER):
+        return merged
     hooks = merged.setdefault("hooks", {})
     for event in EVENT_ORDER:
+        if event in owned_events:
+            continue
         canonical_group = fragment["hooks"][event][0]
         canonical_hook = canonical_group["hooks"][0]
         groups = hooks.setdefault(event, [])
@@ -469,12 +492,14 @@ def _install_projection(live: dict[str, Any], fragment: dict[str, Any]) -> dict[
     return merged
 
 
-def _uninstall_projection(live: dict[str, Any]) -> dict[str, Any]:
+def _uninstall_projection(
+    live: dict[str, Any], events: Sequence[str] = EVENT_ORDER
+) -> dict[str, Any]:
     merged = copy.deepcopy(live)
     hooks = merged.get("hooks")
     if not isinstance(hooks, dict):
         return merged
-    for event in EVENT_ORDER:
+    for event in events:
         groups = hooks.get(event)
         if not isinstance(groups, list):
             continue
@@ -528,6 +553,11 @@ def _check_findings(live: dict[str, Any], fragment: dict[str, Any]) -> list[dict
 
     for event in EVENT_ORDER:
         entries = owned[event]
+        if _hub_owned(event):
+            if entries:
+                findings.append({"kind": "duplicate", "event": event, "count": len(entries),
+                                 "message": "central hook hub owns this concern; remove native copies"})
+            continue
         if not entries:
             findings.append(
                 {"kind": "missing", "event": event, "message": "canonical hook is absent"}

package/dist/assets/project-notebook-skill/tests/test_project_hooks.py

@@ -49,6 +49,12 @@ class ProjectHooksTests(unittest.TestCase):
         self.temporary = tempfile.TemporaryDirectory()
         self.addCleanup(self.temporary.cleanup)
         self.root = Path(self.temporary.name)
+        environment = mock.patch.dict(os.environ, {
+            "BB_HOOK_OWNERSHIP": str(self.root / "uninstalled-ownership.json"),
+            "BB_HOOK_HUB": "",
+        })
+        environment.start()
+        self.addCleanup(environment.stop)
         self.claude = self.root / ".claude"
         self.claude.mkdir(mode=0o700)
         self.target = self.claude / "settings.json"
@@ -193,6 +199,37 @@ class ProjectHooksTests(unittest.TestCase):
             env=environment,
         )
 
+    def test_hub_owned_install_retires_only_native_notebook_copies(self) -> None:
+        foreign = {"matcher": "preserved", "hooks": [{"command": "foreign-command"}]}
+        live = {"theme": "dark", "hooks": {
+            "SessionStart": [copy.deepcopy(foreign), *self.master["hooks"]["SessionStart"]],
+            "SessionEnd": copy.deepcopy(self.master["hooks"]["SessionEnd"]),
+        }}
+        with mock.patch.object(PROJECTOR, "_hub_owned", return_value=True):
+            projected = PROJECTOR._install_projection(live, self.master)
+            self.assertEqual(projected, {"theme": "dark", "hooks": {"SessionStart": [foreign]}})
+            self.assertEqual(PROJECTOR._install_projection(projected, self.master), projected)
+            self.assertEqual(PROJECTOR._check_findings(projected, self.master), [])
+            self.assertEqual({row["kind"] for row in PROJECTOR._check_findings(live, self.master)}, {"duplicate"})
+
+    def test_hub_owned_one_event_preserves_other_standalone_fallback(self) -> None:
+        with mock.patch.object(PROJECTOR, "_hub_owned", side_effect=lambda event: event == "SessionStart"):
+            projected = PROJECTOR._install_projection({}, self.master)
+        self.assertEqual(projected, {"hooks": {"SessionEnd": self.master["hooks"]["SessionEnd"]}})
+
+    def test_legacy_wrapper_defers_to_hub_but_hub_child_runs(self) -> None:
+        fake_home, _ = self.make_fake_pj()
+        helper = fake_home / ".agents/hooks/hub/ownership.py"
+        helper.parent.mkdir(parents=True)
+        helper.write_text("raise SystemExit(0)\n")
+        for wrapper in ("session-start.sh", "session-end.sh"):
+            native = self.run_wrapper(wrapper, fake_home, b"{}")
+            self.assertEqual(native.returncode, 0, native.stderr)
+            self.assertEqual(native.stdout, b"")
+            supervised = self.run_wrapper(wrapper, fake_home, b"{}", extra_environment={"BB_HOOK_HUB": "off"})
+            self.assertEqual(supervised.returncode, 0, supervised.stderr)
+            self.assertTrue(supervised.stdout, supervised.stderr)
+
     def test_master_fragment_and_wrappers_are_exact(self) -> None:
         self.assertEqual(MASTER_PATH.read_bytes(), FRAGMENT_PATH.read_bytes())
         self.assertEqual(list(self.master["hooks"]), ["SessionStart", "SessionEnd"])