@delorenj/pjangler 1.4.5 → 1.5.2

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"])