chore(foundry): JS hook test suites, fortify git commit rules

- Add 4 hook test suites: test_task_log_js.py (307L), test_statusline_js.py (152L), test_commit_guard_js.py (143L), test_agent_router_js.py (122L); conftest.py gains run_hook + state_dir fixtures; README.md documents test table and runner
- ci-tests.yml: add Node 20 setup; JS hook tests now run in CI alongside Python bin/ tests
- Fix task-log.js: replace sliding-window tool counter with fixed 30s window; add pending/ cleanup on PostToolUse(Agent); clear tools/ on UserPromptSubmit
- Distill 3 bin/ Python scripts to inline prose rules across plugins: codemap/resolve_target_module.py, develop/codemap_flags.py, oss/parse-review-args.py deleted; corresponding SKILL.md files updated to apply logic directly
- rules/git-commit.md: add two rules — no GitHub auto-links, no non-VCS paths in commit messages
- agents/shepherd.md: upgrade model sonnet → opusplan; fix cross-ref to /oss:review skill
- agents/perf-optimizer.md: correct model opusplan → opus; update py-spy to uv install form
- agents/curator.md: reflect shepherd reclassification in model-tier table
- agents/challenger.md: extend SKIP condition for codex re-review case
- agents/solution-architect.md: note research plugin requirement in description
- skills/calibrate: add --local to argument-hint; fix bare /tmp/ paths to ${TMPDIR:-/tmp}
- skills/init, setup (foundry), fix (develop): fix bare /tmp/ paths to ${TMPDIR:-/tmp}
- skills/judge (research): fix hardcoded <path_to_program.md> and <RUN_DIR> placeholders in J3 scientist spawn prompt

---
Co-authored-by: Claude Code <noreply@anthropic.com>

Author: Borda

.github/workflows/ci-tests.yml

@@ -24,10 +24,14 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
 
+      - uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+
       - name: Install pytest
         run: pip install -q pytest pytest-cov scipy
 
-      - name: Run plugin's bin tests
+      - name: Run plugin tests (Python bin/ + JS hooks)
         env:
           RUN_INTEGRATION: "1"
         run: pytest -W error::DeprecationWarning

plugins/codemap/.claude-plugin/plugin.json

@@ -17,5 +17,5 @@
   "license": "MIT",
   "name": "codemap",
   "repository": "https://github.com/Borda/AI-Rig",
-  "version": "0.4.1"
+  "version": "0.4.2"
 }

plugins/codemap/bin/resolve_target_module.py

@@ -280,15 +280,13 @@ fi
 
 For skills where target module derives from `$ARGUMENTS` (refactor, fix with module path, review), also add after `central` — **derive `TARGET_MODULE` first**; without it calls run as `scan-query rdeps ""` and return nothing:
 
+Derive `TARGET_MODULE` from `$ARGUMENTS`: strip leading `./`; strip leading `src/`; strip trailing `.py`; replace `/` → `.`. If result empty, use `Path($ARGUMENTS).stem`. Example: `src/foo/bar.py` → `foo.bar`.
+
+If `TARGET_MODULE` non-empty, substitute derived value and run:
+
 ```bash
-# Derive TARGET_MODULE from the file/path argument (e.g. src/foo/bar.py → foo.bar)
-TARGET_MODULE=$(python "${CLAUDE_PLUGIN_ROOT:-plugins/codemap}/bin/resolve_target_module.py" "$ARGUMENTS")  # timeout: 5000
-if [ -z "$TARGET_MODULE" ]; then
-    echo "⚠ TARGET_MODULE empty — skipping rdeps/deps soft-check"
-else
-    scan-query rdeps "$TARGET_MODULE" 2>/dev/null  # timeout: 5000
-    scan-query deps  "$TARGET_MODULE" 2>/dev/null  # timeout: 5000
-fi
+scan-query rdeps "<TARGET_MODULE>" 2>/dev/null  # timeout: 5000
+scan-query deps  "<TARGET_MODULE>" 2>/dev/null  # timeout: 5000
 ```
 
 **For agent `.md` files** — append to last workflow instruction paragraph, before closing section or final notes. Agents have no `$ARGUMENTS` — derive `TARGET_MODULE` from user's input prompt:

plugins/codemap/skills/integration/SKILL.md

@@ -7,5 +7,5 @@
   "license": "MIT",
   "name": "develop",
   "repository": "https://github.com/Borda/AI-Rig",
-  "version": "0.7.1"
+  "version": "0.7.2"
 }

plugins/codemap/tests/test_resolve_target_module.py

@@ -6,8 +6,8 @@
     codemap_scan.py --source=diff [--limit N]
 
 Sources:
-    find — enumerate ``.py`` files under ``<path>``, derive module names via codemap's
-        ``resolve_target_module`` rules.
+    find — enumerate ``.py`` files under ``<path>``, derive module names (strip ``./``,
+        ``src/``, ``.py``; replace ``/`` → ``.``).
     diff — derive modules from ``git diff HEAD --name-only``; flat-layout fallback when
         ``src/`` strip yields nothing.
 

plugins/develop/.claude-plugin/plugin.json

@@ -284,33 +284,9 @@ Crystallise intended API contract before any implementation. Choose form based o
 
 > **Choosing demo form**: use inline doctest for simple functions/methods with minimal setup; use example script for features requiring external state, multiple steps, or side effects.
 
-**Unit function / simple API** -> inline doctest:
-
-```python
-def predict(self, x: Tensor) -> Tensor:
-    """
-    >>> model = Classifier()
-    >>> model.predict(torch.zeros(1, 3))
-    tensor([0])
-    """
-```
-
-**Complex feature** (setup required, side effects, multi-step flow) -> minimal example script:
-
-```bash
-mkdir -p examples/
-```
-
-```python
-# examples/demo_<feature>.py  — throwaway script, run manually
-from mypackage import Classifier
-
-model = Classifier.from_pretrained("tiny")
-result = model.predict_batch(["hello", "world"])
-print(result)  # expected: [label, label]
-```
+**Unit function / simple API** -> inline doctest (doctest in method docstring; must fail against current code).
 
-Example script captures what feature should feel like to use. Becomes formal pytest test once implementation complete and API stable (end of Step 3).
+**Complex feature** (setup required, side effects, multi-step flow) -> minimal example script `examples/demo_<feature>.py`; shows intended API end-to-end; becomes formal pytest test once implementation complete and API stable (end of Step 3).
 
 Both forms must: