@ikon85/agent-workflow-kit 0.24.0 → 0.26.0

package/.agents/skills/setup-workflow/SKILL.md

@@ -26,7 +26,7 @@ This is a prompt-driven skill, not a deterministic script. Explore, present what
 | `## Agent skills` + `## Prod` in CLAUDE.md **and** AGENTS.md | one-line pointers + deploy target (Sections C/G + Write) |
 | `.github/workflows/agent-workflow-kit-update.yml` | optional tested Kit update pull request for GitHub consumers (Section A2) |
 | `docs/agents/census.md` | optional-census choice, paths, state, and safe disable contract seeded from [census.md](./census.md) (Section A3) |
-| `docs/agents/workflow-capabilities.json` + capability assets | consumer-owned Worktree Lifecycle, Memory Lifecycle, and Workflow Advisories choices (Sections A4–A6) |
+| `docs/agents/workflow-capabilities.json` + capability assets | consumer-owned Worktree Lifecycle, Memory Lifecycle, Workflow Advisories, and Safety Guardrails choices (Sections A4–A7) |
 
 ## Idempotency contract — read before writing anything
 
@@ -57,7 +57,7 @@ Read the current state; don't assume. For every target file, read its first line
 - `CONTEXT.md`, `CONTEXT-MAP.md`, `docs/adr/` — domain-doc layout.
 - `docs/agents/`, `docs/agents/skills/`, `docs/conventions/` — prior output of this skill.
 - `docs/agents/census.md`, `.census/profile.json`, `.census/active.json` — an existing census choice or consumer-owned census to adopt.
-- `docs/agents/workflow-capabilities.json`, `.claude/settings.json` — existing capability choices/profile plus Worktree Lifecycle and Workflow Advisories hook wiring to adopt.
+- `docs/agents/workflow-capabilities.json`, `.claude/settings.json`, `git config --get core.hooksPath` — existing capability choices/profile plus Worktree Lifecycle, Workflow Advisories, and Safety Guardrails wiring to adopt.
 - `.memory/active/`, `.memory/archive/`, and `.memory/receipts/` — consumer-owned Memory Lifecycle policies or recovery evidence to adopt without normalization.
 - `gh auth status` (if GitHub) — is `gh` authenticated, and with which scopes?
 
@@ -218,6 +218,31 @@ repository?"* Offer exactly **Yes**, **Later**, and **No**.
 - **Disable** — remove exact kit-owned commands first, then set `enabled:
   false`; preserve every profile value and unknown key.
 
+### 2f. Section A7 — Optional Safety Guardrails
+
+> Safety Guardrails is a counted group of seven independently selectable
+> protections: four agent-tool guards and three repository-security
+> capabilities. Setup stages and validates the chosen profile plus wiring
+> before activation; disabled rows remain exact no-ops.
+
+Read [safety-guardrails.md](./safety-guardrails.md) in full. Reconcile its
+complete `yes / later / no / existing / disable` matrix. If no choice exists,
+ask: *"Should setup configure the portable Safety Guardrails for this
+repository?"* Offer exactly **Yes**, **Later**, and **No**.
+
+- **Yes** — ask which of the seven counted rows to enable, then reconcile only
+  `safetyGuardrails`, exact selected Agent-hook commands, and the selected
+  native Git-hook path. Report `N of 7 enabled`.
+- **Later / No** — record the choice without Agent- or Git-hook wiring.
+- **Existing** — adopt profile and wiring byte-safely; preserve unknown keys
+  and consumer-specific policies.
+- **Disable** — remove only exact kit-owned Agent commands and Git wiring
+  first, then set `enabled: false`.
+
+Testreporter uses the parity-on profile from the seed. Other consumers are
+opt-in/manual: never copy its pnpm, path, shim, surface, or outage decisions
+without repository evidence and user confirmation.
+
 ### 3. Section B — Triage labels
 
 > When `triage` processes an incoming issue it applies labels (or your tracker's equivalent). It needs strings you've actually configured, or it creates duplicates.
@@ -305,10 +330,11 @@ request changes a recorded choice.
 
 For `docs/agents/workflow-capabilities.json`, reconcile only each selected
 capability section. Never replace the whole file: Project Release, Memory
-Lifecycle, Worktree Lifecycle, Workflow Advisories, future sections, and
+Lifecycle, Worktree Lifecycle, Workflow Advisories, Safety Guardrails, future sections, and
 unknown consumer keys share this profile. Apply the transition and hook
-ownership rules from [worktree-lifecycle.md](./worktree-lifecycle.md) and
-[workflow-advisories.md](./workflow-advisories.md) transactionally.
+ownership rules from [worktree-lifecycle.md](./worktree-lifecycle.md),
+[workflow-advisories.md](./workflow-advisories.md), and
+[safety-guardrails.md](./safety-guardrails.md) transactionally.
 
 For Memory Lifecycle, use only the deterministic setup helper from Section A5.
 Do not copy templates with shell commands or edit the capability profile by

package/.agents/skills/setup-workflow/safety-guardrails.md

@@ -0,0 +1,155 @@
+# Safety Guardrails setup contract
+
+Safety Guardrails is a counted, opt-in capability group backed by the
+consumer-owned `docs/agents/workflow-capabilities.json` profile. Its seven
+rows are selected independently: four protect agent tool calls and three
+bootstrap repository security. No repository name, package manager, hook
+surface, threshold, path, or outage policy is inferred from the kit.
+
+## Choice matrix
+
+| State | Setup action |
+|---|---|
+| `missing` | Ask `yes / later / no`; do not infer or write before the answer. |
+| `yes` | Ask which rows to enable, stage profile plus selected wiring, validate, then activate. |
+| `later` | Record the retryable deferral; do not activate any wiring. |
+| `no` | Record the opt-out; do not activate any wiring. |
+| `existing` | Adopt the consumer profile and existing wiring byte-safely. |
+| `disable` | Remove exact kit-owned Agent and Git wiring first, then set `enabled: false`. |
+
+```json safety-guardrails-setup-effects
+[
+  {"state":"missing","choice":null,"operations":[]},
+  {"state":"yes","choice":"yes","operations":["record-choice","stage-enabled-profile","stage-agent-hook-wiring","stage-git-hook-wiring","activate-staged-profile"]},
+  {"state":"later","choice":"later","operations":["record-choice"]},
+  {"state":"no","choice":"no","operations":["record-choice"]},
+  {"state":"existing","choice":"yes","operations":["adopt-existing"]},
+  {"state":"disable","choice":"yes","operations":["remove-agent-hook-wiring","remove-git-hook-wiring","update-profile-disabled"]}
+]
+```
+
+## Counted capability map
+
+Every historical Safety row maps to one profile decision and one shipped
+artifact. Setup reports the result as `N of 7 enabled`, never as one
+all-or-nothing switch.
+
+```json safety-guardrails-capabilities
+[
+  {"id":"secrets","profilePath":"safetyGuardrails.secrets.enabled","artifact":".claude/hooks/block-secrets.py"},
+  {"id":"packageManager","profilePath":"safetyGuardrails.packageManager.enabled","artifact":".claude/hooks/block-npm-install-in-pnpm.py"},
+  {"id":"doubleBackground","profilePath":"safetyGuardrails.doubleBackground.enabled","artifact":".claude/hooks/block-bg-double-background.py"},
+  {"id":"searchShim","profilePath":"safetyGuardrails.searchShim.enabled","artifact":".claude/hooks/grep-shim-guard.py"},
+  {"id":"gitHooks","profilePath":"safetyGuardrails.repositorySecurity.gitHooks.enabled","artifact":"scripts/security/install-git-hooks.mjs"},
+  {"id":"gitleaks","profilePath":"safetyGuardrails.repositorySecurity.gitleaks.enabled","artifact":"scripts/security/ensure-gitleaks.mjs"},
+  {"id":"dependencyAudit","profilePath":"safetyGuardrails.repositorySecurity.dependencyAudit.enabled","artifact":"scripts/security/audit-gate.mjs"}
+]
+```
+
+## Profile shape
+
+Reconcile only `safetyGuardrails`; preserve every sibling section, unknown key,
+and consumer-owned value. The following is the Testreporter parity-on example,
+not a universal default. Other consumers choose each `enabled` value and fill
+only facts proven in their repository.
+
+```json
+{
+  "safetyGuardrails": {
+    "choice": "yes",
+    "enabled": true,
+    "secrets": {
+      "enabled": true,
+      "sensitiveNames": [".env", ".npmrc", "credentials.json"],
+      "sensitivePathFragments": ["secrets/", ".ssh/"],
+      "safeTemplateSuffixes": [".example", ".sample", ".template", ".dist"]
+    },
+    "packageManager": {
+      "enabled": true,
+      "lockfiles": {
+        "pnpm-lock.yaml": "pnpm",
+        "package-lock.json": "npm",
+        "yarn.lock": "yarn",
+        "bun.lockb": "bun"
+      }
+    },
+    "doubleBackground": {"enabled": true, "surfaces": ["claude"]},
+    "searchShim": {
+      "enabled": true,
+      "detected": true,
+      "commandNames": ["grep", "rg"]
+    },
+    "repositorySecurity": {
+      "gitHooks": {"enabled": true, "hooksPath": ".githooks"},
+      "gitleaks": {"enabled": true, "required": false},
+      "dependencyAudit": {
+        "enabled": true,
+        "packageManager": "pnpm",
+        "outagePolicy": "warn",
+        "blockSeverities": ["high", "critical"]
+      }
+    }
+  }
+}
+```
+
+An empty or disabled row is an honest no-op. `searchShim.detected` is a
+consumer fact, not an installation request. `gitleaks.required: false` keeps
+the verified optional provisioner non-blocking when the platform or network is
+unsupported. Dependency-audit outage policy must be explicit.
+
+## Agent-hook ownership
+
+Wire only enabled rows, using the exact kit-owned commands:
+
+- `secrets` — `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/block-secrets.py"`
+- `packageManager` —
+  `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/block-npm-install-in-pnpm.py"`
+- `doubleBackground` —
+  `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/block-bg-double-background.py"`
+- `searchShim` —
+  `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/grep-shim-guard.py"`
+
+All four are PreToolUse adapters. Preserve unrelated matchers and commands in
+`.claude/settings.json`. Disable removes only these exact commands.
+
+## Repository wiring
+
+When `gitHooks.enabled` is true, run:
+
+```bash
+node scripts/security/install-git-hooks.mjs <hooksPath>
+```
+
+only after confirming the configured directory exists and contains the
+consumer's intended hooks. Never create or replace consumer hook bodies.
+
+When `gitleaks.enabled` is true, the pinned profile may be provisioned through
+`scripts/security/ensure-gitleaks.mjs`; checksum mismatch, unsupported platform,
+offline state, and unwritable destination remain distinct visible outcomes.
+Never read a secret file to test the provisioner.
+
+When `dependencyAudit.enabled` is true, use
+`scripts/security/audit-gate.mjs` with the configured package manager and
+outage policy. High/Critical findings block; Low/Moderate findings pass.
+
+## Transaction and failure contract
+
+Prepare the next profile and `.claude/settings.json` in same-directory staging
+files. Validate JSON, all selected artifact paths, the seven-row count, the
+configured Git-hook directory, and package-manager facts before mutation.
+Then:
+
+1. reconcile selected Agent commands in the staged settings;
+2. remember the prior `core.hooksPath` and apply the selected Git wiring;
+3. atomically rename staged settings and profile into place;
+4. on any failure, restore the prior Git config and leave active files
+   byte-for-byte unchanged.
+
+Disable reverses the safety order: remove exact Agent commands and restore or
+unset only the kit-owned Git wiring before committing `enabled: false`.
+Repeated activation with the same choices is byte-idempotent.
+
+Setup and verification may inspect paths, profile metadata, command exit
+status, advisory counts, and hashes. They must never read secret contents or
+print environment/config values.

package/.agents/skills/setup-workflow/workflow-advisories.md

@@ -1,10 +1,11 @@
 # Workflow Advisories setup contract
 
 Workflow Advisories is one opt-in capability backed by the consumer-owned
-`docs/agents/workflow-capabilities.json` profile. Enabling it activates
-non-blocking large-read, baseline, pre-refactor, and affected-surface Stop
-adapters. Every threshold, glob, branch rule, command, timeout, and output
-budget stays consumer-owned.
+`docs/agents/workflow-capabilities.json` profile. Enabling it activates seven
+non-blocking rows: large-read, baseline, pre-refactor, affected-surface Stop,
+convention freshness, migration-artifact reminder, and LoC forewarning. Every
+threshold, glob, branch rule, source map, command, timeout, and output budget
+stays consumer-owned.
 
 ## Choice matrix
 
@@ -52,7 +53,20 @@ key. Consumer values are never normalized on adoption.
       "timeoutSeconds": 15,
       "outputBudget": 1000
     },
-    "stopChecks": {"surfaces": [], "timeoutSeconds": 30, "outputBudget": 1000}
+    "stopChecks": {"surfaces": [], "timeoutSeconds": 30, "outputBudget": 1000},
+    "freshness": {"documents": [], "outputBudget": 1000},
+    "migration": {
+      "commandMatchers": [],
+      "artifact": "",
+      "refreshCommand": [],
+      "outputBudget": 500
+    },
+    "locForewarn": {
+      "branchRegex": "^(?:feat|fix)/(\\d+)-",
+      "issueCommand": ["gh", "issue", "view", "{issue}", "--json", "body", "-q", ".body"],
+      "timeoutSeconds": 5,
+      "outputBudget": 500
+    }
   }
 }
 ```
@@ -70,6 +84,14 @@ project commands from the tools already present, then asks before activating.
   `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/pre-refactor-sweep.py"`
 - Stop:
   `"$CLAUDE_PROJECT_DIR/.claude/hooks/typecheck-on-stop.sh"`
+- SessionStart convention freshness:
+  `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/convention-drift-hint.py"`
+- PostToolUse on Bash migration commands:
+  `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/migration-snapshot-reminder.py"`
+- SessionStart LoC forewarning:
+  `python3 "$CLAUDE_PROJECT_DIR/.claude/hooks/loc-offender-forewarn.py"`
 
 All adapters are non-blocking. A failed or timed-out configured command remains
-visibly failed in hook context; it is never reported as green.
+visibly failed in hook context; it is never reported as green. Issue lookup
+failures silence only the LoC forewarning: the existing pre-push gate remains
+the enforcing authority and stays fail-closed.

package/.claude/hooks/_hook_utils.py

@@ -31,6 +31,7 @@ from pathlib import Path
 LOG_DIR = Path(".claude/logs")
 _WORKTREE_CORE_MODULE = "_agent_workflow_kit_worktree_lifecycle"
 _ADVISORY_CORE_MODULE = "_agent_workflow_kit_workflow_advisories"
+_LOC_OFFENDER_GATE_MODULE = "_agent_workflow_kit_loc_offender_gate"
 
 
 def rotate_log_if_needed(log_path: Path, max_bytes: int = 100_000, generations: int = 3) -> None:
@@ -221,6 +222,24 @@ def load_workflow_advisories_core():
     return module
 
 
+def load_loc_offender_gate():
+    """Load the shipped LoC gate so advisories consume its contracts directly."""
+    existing = sys.modules.get(_LOC_OFFENDER_GATE_MODULE)
+    if existing is not None:
+        return existing
+    path = Path(__file__).resolve().parents[2] / "scripts" / "loc_offender_gate.py"
+    module_dir = str(path.parent)
+    if module_dir not in sys.path:
+        sys.path.insert(0, module_dir)
+    spec = importlib.util.spec_from_file_location(_LOC_OFFENDER_GATE_MODULE, path)
+    if spec is None or spec.loader is None:
+        raise ImportError(f"cannot load LoC offender gate from {path}")
+    module = importlib.util.module_from_spec(spec)
+    sys.modules[_LOC_OFFENDER_GATE_MODULE] = module
+    spec.loader.exec_module(module)
+    return module
+
+
 def hook_event_output(event_name: str, context: str) -> dict:
     """Canonical non-blocking context payload for Claude hook events."""
     return {

package/.claude/hooks/_safety_guard.py

@@ -0,0 +1,24 @@
+"""Load the shipped Safety Guardrails core for thin hook adapters."""
+
+import importlib.util
+import sys
+from pathlib import Path
+
+_CORE_MODULE = "_agent_workflow_kit_safety_guardrails"
+
+
+def load_core():
+    existing = sys.modules.get(_CORE_MODULE)
+    if existing is not None:
+        return existing
+    path = Path(__file__).resolve().parents[2] / "scripts" / "safety-guardrails" / "core.py"
+    module_dir = str(path.parent)
+    if module_dir not in sys.path:
+        sys.path.insert(0, module_dir)
+    spec = importlib.util.spec_from_file_location(_CORE_MODULE, path)
+    if spec is None or spec.loader is None:
+        raise ImportError(f"cannot load Safety Guardrails core from {path}")
+    module = importlib.util.module_from_spec(spec)
+    sys.modules[_CORE_MODULE] = module
+    spec.loader.exec_module(module)
+    return module

package/.claude/hooks/block-bg-double-background.py

@@ -0,0 +1,30 @@
+#!/usr/bin/env python3
+"""Thin PreToolUse adapter for profile-driven double-background prevention."""
+
+import json
+import sys
+from pathlib import Path
+
+from _hook_utils import log
+from _safety_guard import load_core
+
+HOOK_NAME = "block-bg-double-background"
+
+
+def main() -> int:
+    try:
+        payload = json.load(sys.stdin)
+        core = load_core()
+        profile = core.load_profile(Path("docs/agents/workflow-capabilities.json"))
+        decision = core.evaluate("double-background", profile, payload)
+    except Exception:
+        return 0
+    if decision.action == "block":
+        print(decision.message, file=sys.stderr)
+        log(HOOK_NAME, decision.log_message)
+        return 2
+    return 0
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

package/.claude/hooks/block-npm-install-in-pnpm.py

@@ -0,0 +1,30 @@
+#!/usr/bin/env python3
+"""Thin PreToolUse adapter for profile-driven package-manager consistency."""
+
+import json
+import sys
+from pathlib import Path
+
+from _hook_utils import log
+from _safety_guard import load_core
+
+HOOK_NAME = "block-package-manager"
+
+
+def main() -> int:
+    try:
+        payload = json.load(sys.stdin)
+        core = load_core()
+        profile = core.load_profile(Path("docs/agents/workflow-capabilities.json"))
+        decision = core.evaluate("package-manager", profile, payload)
+    except Exception:
+        return 0
+    if decision.action == "block":
+        print(decision.message, file=sys.stderr)
+        log(HOOK_NAME, decision.log_message)
+        return 2
+    return 0
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

package/.claude/hooks/block-secrets.py

@@ -0,0 +1,30 @@
+#!/usr/bin/env python3
+"""Thin PreToolUse adapter for the profile-driven secret guard."""
+
+import json
+import sys
+from pathlib import Path
+
+from _hook_utils import log
+from _safety_guard import load_core
+
+HOOK_NAME = "block-secrets"
+
+
+def main() -> int:
+    try:
+        payload = json.load(sys.stdin)
+        core = load_core()
+        profile = core.load_profile(Path("docs/agents/workflow-capabilities.json"))
+        decision = core.evaluate("secrets", profile, payload)
+    except Exception:
+        return 0
+    if decision.action == "block":
+        print(decision.message, file=sys.stderr)
+        log(HOOK_NAME, decision.log_message)
+        return 2
+    return 0
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

package/.claude/hooks/convention-drift-hint.py

@@ -0,0 +1,27 @@
+#!/usr/bin/env python3
+"""SessionStart convention freshness advisory over profile-owned source maps."""
+import json
+import sys
+from pathlib import Path
+
+from _hook_utils import hook_event_output, load_workflow_advisories_core
+
+
+def main() -> int:
+    try:
+        json.load(sys.stdin)
+    except Exception:
+        pass
+    try:
+        core = load_workflow_advisories_core()
+        profile = core.load_profile(Path("docs/agents/workflow-capabilities.json"))
+        decision = core.convention_freshness_decision(profile, Path.cwd())
+        if decision.context:
+            print(json.dumps(hook_event_output(decision.event_name, decision.context)))
+    except Exception:
+        pass
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/.claude/hooks/grep-shim-guard.py

@@ -0,0 +1,30 @@
+#!/usr/bin/env python3
+"""Thin PreToolUse adapter for configured search-shim breakers."""
+
+import json
+import sys
+from pathlib import Path
+
+from _hook_utils import log
+from _safety_guard import load_core
+
+HOOK_NAME = "grep-shim-guard"
+
+
+def main() -> int:
+    try:
+        payload = json.load(sys.stdin)
+        core = load_core()
+        profile = core.load_profile(Path("docs/agents/workflow-capabilities.json"))
+        decision = core.evaluate("search-shim", profile, payload)
+    except Exception:
+        return 0
+    if decision.action == "block":
+        print(decision.message, file=sys.stderr)
+        log(HOOK_NAME, decision.log_message)
+        return 2
+    return 0
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

package/.claude/hooks/loc-offender-forewarn.py

@@ -0,0 +1,56 @@
+#!/usr/bin/env python3
+"""SessionStart forewarning over the existing LoC gate and issue marker contracts."""
+import json
+import re
+import subprocess
+import sys
+from pathlib import Path
+
+from _hook_utils import (
+    current_branch,
+    hook_event_output,
+    load_loc_offender_gate,
+    load_workflow_advisories_core,
+)
+
+
+def main() -> int:
+    try:
+        json.load(sys.stdin)
+    except Exception:
+        pass
+    try:
+        root = Path.cwd()
+        core = load_workflow_advisories_core()
+        profile = core.load_profile(root / "docs/agents/workflow-capabilities.json")
+        config = profile.get("locForewarn", {})
+        match = re.search(config.get("branchRegex", r"$^"), current_branch())
+        command = config.get("issueCommand", [])
+        if not match or not command:
+            return 0
+        issue = match.group(1)
+        argv = [str(part).replace("{issue}", issue) for part in command]
+        result = subprocess.run(
+            argv,
+            cwd=root,
+            capture_output=True,
+            text=True,
+            timeout=float(config.get("timeoutSeconds", 5)),
+        )
+        if result.returncode != 0:
+            return 0
+        gate = load_loc_offender_gate()
+        max_lines, offenders = gate.load_max_lines(root / "max-lines-allowlist.json")
+        context = gate.forewarning_context(result.stdout, max_lines, offenders)
+        if context:
+            budget = int(config.get("outputBudget", 500))
+            if budget > 0 and len(context) > budget:
+                context = context[: max(0, budget - 1)] + "…"
+            print(json.dumps(hook_event_output("SessionStart", context)))
+    except Exception:
+        pass
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/.claude/hooks/migration-snapshot-reminder.py

@@ -0,0 +1,24 @@
+#!/usr/bin/env python3
+"""PostToolUse migration artifact refresh reminder over consumer profile data."""
+import json
+import sys
+from pathlib import Path
+
+from _hook_utils import hook_event_output, load_workflow_advisories_core
+
+
+def main() -> int:
+    try:
+        payload = json.load(sys.stdin)
+        core = load_workflow_advisories_core()
+        profile = core.load_profile(Path("docs/agents/workflow-capabilities.json"))
+        decision = core.migration_reminder_decision(profile, payload)
+        if decision.context:
+            print(json.dumps(hook_event_output(decision.event_name, decision.context)))
+    except Exception:
+        pass
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/.claude/hooks/skill-drift-hint.py

@@ -21,52 +21,21 @@ import json
 import sys
 from pathlib import Path
 
-from _hook_utils import log, run, repo_root
+from _hook_utils import (
+    hook_event_output,
+    load_workflow_advisories_core,
+    log,
+    repo_root,
+)
 
 HOOK_NAME = "skill-drift-hint"
 SKILLS_REL = ".claude/skills"
 
 
-def git_commit_time(root: str, rel_path: str) -> int | None:
-    """Unix ctime of the last commit touching rel_path; None if untracked/unknown."""
-    out = run(["git", "-C", root, "log", "-1", "--format=%ct", "--", rel_path])
-    return int(out) if out.isdigit() else None
-
-
-def read_sources(sources_file: Path) -> list[str]:
-    """Repo-relative source paths from a SOURCES.txt — skips blanks and `#` comments."""
-    try:
-        lines = sources_file.read_text(encoding="utf-8").splitlines()
-    except Exception:
-        return []
-    out: list[str] = []
-    for line in lines:
-        s = line.strip()
-        if not s or s.startswith("#"):
-            continue
-        out.append(s)
-    return out
-
-
 def collect_stale(root: str) -> list[tuple[str, list[str]]]:
     """For each skill with a SOURCES.txt, list sources newer (in git) than its SKILL.md."""
-    skills_dir = Path(root) / SKILLS_REL
-    results: list[tuple[str, list[str]]] = []
-    if not skills_dir.is_dir():
-        return results
-    for sources_file in sorted(skills_dir.glob("*/SOURCES.txt")):
-        skill_name = sources_file.parent.name
-        skill_ct = git_commit_time(root, f"{SKILLS_REL}/{skill_name}/SKILL.md")
-        if skill_ct is None:
-            continue  # skill not committed yet → nothing to compare against
-        stale = [
-            src
-            for src in read_sources(sources_file)
-            if (src_ct := git_commit_time(root, src)) is not None and src_ct > skill_ct
-        ]
-        if stale:
-            results.append((skill_name, stale))
-    return results
+    core = load_workflow_advisories_core()
+    return core.collect_skill_stale(Path(root), SKILLS_REL)
 
 
 def build_context(root: str) -> str | None:
@@ -107,12 +76,7 @@ def main() -> int:
     if context is None:
         return 0
 
-    payload = {
-        "hookSpecificOutput": {
-            "hookEventName": "SessionStart",
-            "additionalContext": context,
-        }
-    }
+    payload = hook_event_output("SessionStart", context)
     print(json.dumps(payload))
     log(HOOK_NAME, f"emitted: {context.count(chr(0x26A0))} skill(s) flagged")
     return 0