Update Rules and hooks on the monitor workspace

Author: stephenlienharrell

.cursor/hooks/README.md

@@ -1,15 +1,45 @@
-# Cursor hooks (HPCPerfStats)
+# Cursor hooks (HPCPerfStats + monitor workspace)
 
 Project hooks for Cursor Agent. Cursor loads `.cursor/hooks.json` from the **workspace root**.
 
-This workspace opens at the parent folder (contains `.venv/`). Symlinks at `<workspace>/.cursor/` point here:
+## Symlink setup
+
+### Monitor-focused workspace (`.cursor/rules` → `monitor/cursor-rules/`)
+
+From workspace root (contains `.venv/` and `HPCPerfStats/`):
 
 ```bash
-# From workspace root (one-time, if symlinks are missing)
-ln -sf ../HPCPerfStats/.cursor/hooks.json .cursor/hooks.json
-ln -sf ../HPCPerfStats/.cursor/hooks .cursor/hooks
+ln -sf HPCPerfStats/.cursor/hooks.json .cursor/hooks.json
+ln -sf HPCPerfStats/.cursor/hooks .cursor/hooks
 ```
 
+### Full-stack workspace (`.cursor/rules` → `hpcperfstats/cursor-rules/`)
+
+From workspace root:
+
+```bash
+ln -sf HPCPerfStats/.cursor/hooks.json .cursor/hooks.json
+ln -sf HPCPerfStats/.cursor/hooks .cursor/hooks
+```
+
+Both layouts share the **same** hook scripts under `HPCPerfStats/.cursor/hooks/`.
+
+## Profile detection
+
+Hooks auto-detect workspace profile from the **`.cursor/rules` symlink target**:
+
+| Symlink target | Profile | Rules dir label |
+|----------------|---------|-----------------|
+| `…/monitor/cursor-rules` | `monitor` | `HPCPerfStats/monitor/cursor-rules` |
+| `…/hpcperfstats/cursor-rules` | `hpcperfstats` | `hpcperfstats/cursor-rules` |
+
+`hook_task_router.py` merges **`MONITOR_ROUTER_ENTRIES`** and **`HPCPERFSTATS_ROUTER_ENTRIES`** so authorized cross-edits can trigger rules from both trees.
+
+Plan template paths accepted for Read verification:
+
+- `HPCPerfStats/monitor/docs/plans/PLAN_TEMPLATE.md` (monitor workspace)
+- `HPCPerfStats/docs/plans/PLAN_TEMPLATE.md` (full-stack workspace)
+
 ## Hooks
 
 | Event | Script | Behavior |
@@ -22,14 +52,24 @@ ln -sf ../HPCPerfStats/.cursor/hooks .cursor/hooks
 
 1. Cursor **Settings → Hooks** tab should list both hooks after save/reload.
 2. Hooks output channel shows stdin/stdout when hooks fire.
-3. Manual smoke test:
+3. Unit tests: `cd HPCPerfStats && ../.venv/bin/python3 -m pytest hpcperfstats/tests/test_cursor_hooks.py -q`
+4. Manual smoke test:
 
 ```bash
-echo '{"status":"completed","loop_count":0,"transcript_path":"/path/to/transcript.jsonl"}' | \
+echo '{"status":"completed","loop_count":0,"transcript_path":"/path/to/transcript.jsonl","workspace_roots":["/path/to/workspace"]}' | \
   python3 HPCPerfStats/.cursor/hooks/check-close-gate.py
 ```
 
 ## Requirements
 
-- `python3` on `PATH` (macOS default)
+- `python3` on `PATH`
 - Trusted workspace (project hooks do not run in untrusted workspaces)
+
+## Dual registration
+
+New domain rules must appear in **both**:
+
+1. Profile **`agent-discipline-core.mdc`** task router table
+2. **`hook_task_router.py`** — `MONITOR_ROUTER_ENTRIES` or `HPCPERFSTATS_ROUTER_ENTRIES`
+
+See monitor **`RULES_README.md`** or hpcperfstats **`RULES_README.md`** for always-on caps.

.cursor/hooks/check-close-gate.py

@@ -17,6 +17,7 @@
     load_json_stdin,
     looks_like_task_close,
     parse_transcript_lines,
+    profile_rules_dir_label,
     turn_had_closeable_work,
     turn_had_create_plan,
     turn_had_edits,
@@ -40,6 +41,9 @@ def main() -> int:
         emit_json({})
         return 0
 
+    workspace_roots = payload.get("workspace_roots") or []
+    rules_dir = profile_rules_dir_label(workspace_roots=workspace_roots)
+
     rows = parse_transcript_lines(transcript_path)
     turn_rows = last_turn_rows(rows)
     had_edits = turn_had_edits(turn_rows)
@@ -66,18 +70,19 @@ def main() -> int:
     if had_plan:
         plan_extra = (
             "\nPlan turns also require CreatePlan body sections per "
-            "docs/plans/PLAN_TEMPLATE.md (Problem and facts, Approach, Testing, "
-            "Implementation, Cursor rules, Final code review, post-implementation-review "
-            "todo) and Read of plan-creation-contract.mdc + PLAN_TEMPLATE.md before "
-            "CreatePlan.\n"
+            f"{rules_dir}/../docs/plans/PLAN_TEMPLATE.md or "
+            "HPCPerfStats/docs/plans/PLAN_TEMPLATE.md (Problem and facts, Approach, "
+            "Testing, Implementation, Cursor rules, Final code review, "
+            "post-implementation-review todo) and Read of plan-creation-contract.mdc + "
+            "PLAN_TEMPLATE.md before CreatePlan.\n"
         )
     followup = (
         "Close gate incomplete (Cursor stop hook). This turn used %s but the "
         "final assistant message is missing required sections or listed rules were "
         "not Read via the Read tool.%s\n\n"
         "Add in order:\n"
         "1. ## Agent rule dispatch — list every triggered "
-        "hpcperfstats/cursor-rules/*.mdc you Read (or N/A only when work did not "
+        f"{rules_dir}/*.mdc you Read (or N/A only when work did not "
         "trigger domain rules). Each listed domain rule needs a Read tool call on "
         "that .mdc path BEFORE the first Write/StrReplace/CreatePlan in this turn. "
         "Auto-triggered rules from edited or plan-referenced paths must appear in "

.cursor/hooks/check-edit-triggered-rules.py

@@ -21,6 +21,7 @@
     parse_transcript_lines,
     paths_from_plan_markdown,
     plan_template_read_issues,
+    profile_rules_dir_label,
 )
 from hook_task_router import triggered_rules_for_paths  # noqa: E402
 
@@ -107,16 +108,20 @@ def main() -> int:
         emit_json({})
         return 0
 
+    rules_dir = profile_rules_dir_label(
+        workspace_roots=payload.get("workspace_roots") or [],
+    )
     action = "CreatePlan" if tool_name == "CreatePlan" else "this edit"
     emit_json(
         {
             "additional_context": (
                 "RULE DISPATCH (pre-close): %s triggers domain or plan-authoring "
                 f"rules that must be Read before further work. Issues: "
                 f"{', '.join(issues)}. "
-                "Use the Read tool on each listed hpcperfstats/cursor-rules/*.mdc "
-                "and docs/plans/PLAN_TEMPLATE.md for plan turns, then list them "
-                "under ## Agent rule dispatch at close."
+                f"Use the Read tool on each listed {rules_dir}/*.mdc "
+                "and the workspace PLAN_TEMPLATE.md "
+                "(HPCPerfStats/monitor/docs/plans/ or HPCPerfStats/docs/plans/) "
+                "for plan turns, then list them under ## Agent rule dispatch at close."
             )
             % action,
         },

.cursor/hooks/check-new-rule-router.py

@@ -77,8 +77,9 @@ def main() -> int:
                 f"MANDATORY (agent-discipline-core.mdc): new/updated rule `{basename}` "
                 f"is not fully dual-registered. Missing: {', '.join(missing)}. "
                 "In the same task, add a row to `agent-discipline-core.mdc` "
-                "(trigger paths → Read this rule), a matching `ROUTER_ENTRIES` row in "
-                "`.cursor/hooks/hook_task_router.py`, and a one-line note in "
+                "(trigger paths → Read this rule), a matching row in "
+                "`.cursor/hooks/hook_task_router.py` (`MONITOR_ROUTER_ENTRIES` or "
+                "`HPCPERFSTATS_ROUTER_ENTRIES`), and a one-line note in "
                 "`RULES_README.md` if policy changes. Verify both routers in "
                 "Final code review before close."
             ),