Phase 2: Enforcement nits

phase-pal-e-agency-2-enforcement-nits Phase

completed phase
Goal: Fix broken/incomplete hooks and agent configs in claude-custom.
Owner: Dev agent
Repo: forgejo_admin/claude-custom
Depends on: None

Scope

Six open TODOs/bugs targeting claude-custom hooks and agent configs. All are small, independent fixes that can ship together as one PR.
  • todo-dottie-agent-type-missing — Add "dottie" to agent-spawn-requirements.json so the spawn gate recognizes Dottie as a valid agent type.
  • todo-dottie-config-nits — Dottie agent config wording fix + add PreToolUse hook to block code writes (Write/Edit/Bash on repos).
  • todo-fix-remind-mcp-review-loop-paldocs-ref — Fix stale pal-e-docs reference in remind-mcp-review-loop.sh.
  • bug-plan-template-hook-large-content — Plan template hook fails on large HTML content. Likely a shell argument length issue.
  • todo-delete-note-warning-hook — Add PreToolUse hook that warns before delete_note calls (per sop-note-deletion).
  • todo-worktree-cleanup — Automate post-merge worktree cleanup (stale worktrees break ruff).

Deliverables

  • PR #85 merged (claude-custom) — 4 of 6 scoped items shipped
  • schemas/agent-spawn-requirements.json — added "dottie" agent type + SubagentStart matcher + context injection case
  • hooks/remind-mcp-review-loop.sh — removed stale mcp__pal-e-docs reference
  • hooks/check-note-template.sh — temp file fix for large HTML truncation
  • hooks/warn-delete-note.sh — new advisory hook for note deletion (registered in settings.json)
  • Forgejo issue #84 closed
  • Deferred: Dottie PreToolUse hook for code write blocking (todo-dottie-config-nits), worktree cleanup automation (todo-worktree-cleanup)
  • QA nits: block-docs-writes.sh missing board tools (pre-existing), dottie/general-purpose schema duplication
  • plan-pal-e-agency — parent plan
  • sop-note-deletion — governs delete_note warning hook
  • agent-dottie — Dottie agent definition