cursordoctrine 0.1.0 → 0.1.1

package/linux/hooks/final-review.sh

@@ -83,8 +83,17 @@ if [ -z "$body" ]; then
      drop premature abstraction, unneeded deps, redundant comments, dead helpers.
 Fix now, re-run the scan + tests, then stop. If an axis is clean, say so in one line.'
 fi
-
-file_list="$(printf '%s\n' "$edited" | head -n 30 | sed 's/^/  /')"
+body="$(expand_agent_paths "$body")"
+
+file_list=""
+while IFS= read -r p; do
+    [ -n "$p" ] || continue
+    rp="$(resolve_agent_path "$p")"
+    file_list="${file_list}  ${rp}"$'\n'
+done <<EOF
+$edited
+EOF
+file_list="$(printf '%s' "$file_list" | head -n 30)"
 msg="FINAL REVIEW (end of implementation) - correctness, reliability, coverage, anti-slop.
 
 Files you changed this session:

package/linux/hooks/hook-common.sh

@@ -87,6 +87,24 @@ is_cursor_config_path() {
     esac
 }
 
+# Expand ~/ in agent-facing text to an absolute profile path (bash expands ~,
+# but stop-hook followups should still emit literals agents can copy-paste).
+expand_agent_paths() {
+    local text="$1"
+    local home="${HOME%/}"
+    text="${text//\~\//$home/}"
+    printf '%s' "$text"
+}
+
+# Normalize a file path for agent prompts (expand ~).
+resolve_agent_path() {
+    local p="$1"
+    case "$p" in
+        "~/"*) printf '%s' "$HOME/${p#~/}" ;;
+        *) printf '%s' "$p" ;;
+    esac
+}
+
 # merge_subagent_edit_markers <json> <parent_cid> -> 0 if anything was folded
 #
 # Subagent edits fire afterFileEdit under the SUBAGENT's conversation_id, so

package/linux/hooks/post-tool-use.sh

@@ -24,7 +24,8 @@ cid="$(safe_conversation_id "$input")"
 
 fold_note=""
 if merge_subagent_edit_markers "$input" "$cid"; then
-    fold_note="SUBAGENT WORK DETECTED - a subagent of this conversation edited files (its edits fired hooks in ITS context, not yours). YOU are the auditor of its work: audit its diff (git status / git diff on the files it touched) against ~/.agents/hooks/self-review.md. Fix real bugs; stay silent otherwise. Its files are folded into this conversation's end-of-implementation review."
+    self_review="$(expand_agent_paths "$HOME/.agents/hooks/self-review.md")"
+    fold_note="SUBAGENT WORK DETECTED - a subagent of this conversation edited files (its edits fired hooks in ITS context, not yours). YOU are the auditor of its work: audit its diff (git status / git diff on the files it touched) against $self_review. Fix real bugs; stay silent otherwise. Its files are folded into this conversation's end-of-implementation review."
 fi
 
 pending_file="$(hooks_pending_dir)/feedback-$cid.txt"

package/linux/hooks/subagent-stop-review.sh

@@ -73,12 +73,22 @@ behaviour the task asked for):
   1. Correctness - logic, edge cases (null/empty/zero/boundary), language traps, security.
   2. Reliability - error paths handled, no swallowed errors, resources released.
   3. Coverage - behaviour-bearing changes have real tests; RUN the suite if present.
-  4. Anti-slop - no duplicate helpers, premature abstraction, unneeded deps,
-     redundant comments, dead code.
+  4. Anti-slop - if ~/.cursor/skills/anti-slop/scripts/scan_slop.py exists, run
+     `python ~/.cursor/skills/anti-slop/scripts/scan_slop.py --all`; otherwise
+     apply ~/.agents/hooks/anti-slop.md to the session diff.
 If an axis is clean, say so in one line. Then stop.'
 fi
+body="$(expand_agent_paths "$body")"
 
-file_list="$(printf '%s\n' "$edited" | head -n 30 | sed 's/^/  /')"
+file_list=""
+while IFS= read -r p; do
+    [ -n "$p" ] || continue
+    rp="$(resolve_agent_path "$p")"
+    file_list="${file_list}  ${rp}"$'\n'
+done <<EOF
+$edited
+EOF
+file_list="$(printf '%s' "$file_list" | head -n 30)"
 msg="SUBAGENT FINAL REVIEW - you just finished delegated implementation work. Before your result returns to the parent agent, audit it.
 
 Files you changed this run:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cursordoctrine",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Thin self-review hooks for Cursor — the model is the auditor. One command installs the doctrine, the hook pack, and the anti-slop skill.",
   "bin": {
     "cursordoctrine": "bin/cli.mjs"

package/windows/hooks/final-review.ps1

@@ -94,12 +94,14 @@ FINAL REVIEW - audit everything you changed this session and FIX what fails
 Fix now, re-run the scan + tests, then stop. If an axis is clean, say so in one line.
 '@
 }
+$body = Expand-AgentPaths $body
 
-$fileList = ($edited | Select-Object -First 30) -join "`n  "
+$resolved = @($edited | ForEach-Object { Resolve-AgentPath $_ })
+$fileList = ($resolved | Select-Object -First 30) -join "`n  "
 $msg = "FINAL REVIEW (end of implementation) - correctness, reliability, coverage, anti-slop.`n`nFiles you changed this session:`n  $fileList`n`n$body"
 
 # Arm the one-shot brake BEFORE emitting, so a crash after emit can't re-fire.
-try { New-Item -ItemType File -Path $flag -Force | Out-Null } catch { }
+New-Item -ItemType File -Path $flag -Force -ErrorAction SilentlyContinue | Out-Null
 
 Write-HookJson @{ followup_message = $msg }
 exit 0

package/windows/hooks/hook-common.ps1

@@ -51,6 +51,28 @@ function ConvertTo-FwdPath([string]$p) {
     return $p.Replace('\', '/')
 }
 
+# Expand ~/ in agent-facing text to an absolute profile path. pwsh and many
+# agent tools do not resolve ~ on Windows; stop-hook followups must be literal.
+function Expand-AgentPaths([string]$text) {
+    if (-not $text) { return $text }
+    $homeFwd = $HOME.TrimEnd('\', '/').Replace('\', '/')
+    return $text.Replace('~/', "$homeFwd/")
+}
+
+# Normalize a file path for agent prompts (expand ~, forward slashes).
+function Resolve-AgentPath([string]$p) {
+    if (-not $p) { return $p }
+    $p = $p.Trim()
+    if ($p -match '^~[\\/]') {
+        $p = Join-Path $HOME ($p.Substring(2))
+    }
+    if (Test-Path -LiteralPath $p -ErrorAction SilentlyContinue) {
+        $resolved = Resolve-Path -LiteralPath $p -ErrorAction SilentlyContinue
+        if ($resolved) { return ConvertTo-FwdPath $resolved.Path }
+    }
+    return ConvertTo-FwdPath $p
+}
+
 # Subagent edits fire afterFileEdit under the SUBAGENT's conversation_id, so
 # their session-edits markers are invisible to the parent's stop-hook review.
 # Subagent transcripts live at <transcripts>/<parent-cid>/subagents/<sub-cid>.jsonl,

package/windows/hooks/post-tool-use.ps1

@@ -23,7 +23,8 @@ $cid = Get-SafeConversationId $obj
 
 $foldNote = ''
 if (Merge-SubagentEditMarkers $obj $cid) {
-    $foldNote = "SUBAGENT WORK DETECTED - a subagent of this conversation edited files (its edits fired hooks in ITS context, not yours). YOU are the auditor of its work: audit its diff (git status / git diff on the files it touched) against ~/.agents/hooks/self-review.md. Fix real bugs; stay silent otherwise. Its files are folded into this conversation's end-of-implementation review."
+    $selfReview = Expand-AgentPaths (Join-Path $HOME '.agents/hooks/self-review.md')
+    $foldNote = "SUBAGENT WORK DETECTED - a subagent of this conversation edited files (its edits fired hooks in ITS context, not yours). YOU are the auditor of its work: audit its diff (git status / git diff on the files it touched) against $selfReview. Fix real bugs; stay silent otherwise. Its files are folded into this conversation's end-of-implementation review."
 }
 
 $pendingFile = Join-Path (Get-HooksPendingDir) "feedback-$cid.txt"

package/windows/hooks/subagent-stop-review.ps1

@@ -73,13 +73,16 @@ behaviour the task asked for):
   1. Correctness - logic, edge cases (null/empty/zero/boundary), language traps, security.
   2. Reliability - error paths handled, no swallowed errors, resources released.
   3. Coverage - behaviour-bearing changes have real tests; RUN the suite if present.
-  4. Anti-slop - no duplicate helpers, premature abstraction, unneeded deps,
-     redundant comments, dead code.
+  4. Anti-slop - if ~/.cursor/skills/anti-slop/scripts/scan_slop.py exists, run
+     `python ~/.cursor/skills/anti-slop/scripts/scan_slop.py --all`; otherwise
+     apply ~/.agents/hooks/anti-slop.md to the session diff.
 If an axis is clean, say so in one line. Then stop.
 '@
 }
+$body = Expand-AgentPaths $body
 
-$fileList = ($edited | Select-Object -First 30) -join "`n  "
+$resolved = @($edited | ForEach-Object { Resolve-AgentPath $_ })
+$fileList = ($resolved | Select-Object -First 30) -join "`n  "
 $msg = "SUBAGENT FINAL REVIEW - you just finished delegated implementation work. Before your result returns to the parent agent, audit it.`n`nFiles you changed this run:`n  $fileList`n`n$body"
 
 # Arm the one-shot brake BEFORE emitting, so a crash after emit can't re-fire.