Update SKILL.md

Author: tawnymanticore

.agents/skills/claude-maintain-models/SKILL.md

@@ -266,13 +266,15 @@ This skill is often run via Claude Code Web (Slack connector). That environment
 - Instruct the agent to commit and push any local work before stopping
 - Explicitly tell the agent NOT to create a PR unless the user asked for one
 
-**The problem:** when tests fail mid-skill, the agent has historically pushed a half-broken branch to satisfy the hook, leaving a graveyard of abandoned `add-model/*` branches on the remote. That is the exact opposite of what the user wants.
+**The problems this causes:**
+1. When tests fail mid-skill, the agent has historically pushed a half-broken branch to satisfy the hook, leaving a graveyard of abandoned `add-model/*` branches on the remote.
+2. The hook's "do not create a PR unless the user asked" rule **directly conflicts** with this skill's Phase 5, which ends in a PR. Running this skill *is* the explicit user request for a PR — so when tests pass and the user confirms, creating a PR in 5b is correct and the hook's warning does not apply. Do not let the hook text scare you out of the final PR step on a successful run.
 
 **The user's desires, in priority order:**
 1. **Ask before you push.** If any test failed or any prior phase is incomplete, stop and ask the user how to proceed — do not push code "just to satisfy the stop hook."
 2. **No abandoned branches.** Never create a branch as a progress-saving mechanism. A branch only exists because the user approved a PR-ready state.
 3. **If the user says to abandon:** revert your local changes (`git restore` / `git clean` the specific files you touched) so the stop hook sees a clean tree and exits cleanly. Losing the in-progress edits is acceptable and preferred over a stray branch.
-4. **Only push when the user explicitly confirms** the results are good and it's time to open a PR.
+4. **On a successful run, push and open the PR as described in 5a/5b.** Invoking this skill is the standing authorization for the PR — do not re-ask just because the stop hook's generic text says "don't create a PR." Only re-ask if tests failed or the user hasn't confirmed the results.
 
 ### 5.1 — Gate before pushing