fix(code-critic): fix remaining set -e dead code in all examples

Triple-agent review found the set -e + $? dead code pattern was
only fixed in the pre-commit example but persisted in:
- GIT_HOOKS.md pre-push hook example
- GIT_HOOKS.md selective review example
- USAGE.md quick setup example (also fixed trailing -p flag)

All now use set +e / set -e with REVIEW_EXIT capture.
Also softened Config/IaC self-review note for accuracy.

AI review: code-critic:adversarial-reviewer (auto via hook)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Claude Code Bot

plugins/code-critic/README.md

@@ -153,7 +153,7 @@ model: opus
 
 To use a different model, edit `agents/adversarial-reviewer.md` and change the model field.
 
-Note: The agent's Configuration/prompts/IaC domain awareness means it can review its own prompt file — this is intentional and useful for self-improvement workflows.
+Note: The agent's Configuration/prompts/IaC domain covers prompt and config files, which includes its own prompt file — useful for self-review workflows where the agent evaluates changes to its own definition.
 
 ## Contributing
 

plugins/code-critic/docs/GIT_HOOKS.md

@@ -73,7 +73,8 @@ if [ -z "$DIFF" ]; then
   exit 0
 fi
 
-# Run review
+# Run review (temporarily allow non-zero exit to capture result)
+set +e
 echo "$DIFF" | claude --agent adversarial-reviewer \
   --no-session-persistence \
   -p "Review these changes before push. Focus on:
@@ -83,8 +84,8 @@ echo "$DIFF" | claude --agent adversarial-reviewer \
 - Maintenance burden
 
 Provide specific, actionable feedback."
-
 REVIEW_EXIT=$?
+set -e
 
 if [ $REVIEW_EXIT -ne 0 ]; then
   echo "${RED}❌ Code Critic found issues${NC}"
@@ -182,6 +183,8 @@ fi
 echo "🔒 Security-critical files detected, running adversarial review..."
 echo "$CRITICAL_FILES"
 
+# Run review (temporarily allow non-zero exit to capture result)
+set +e
 git diff origin/main...HEAD | claude --agent adversarial-reviewer \
   --no-session-persistence \
   -p "Review these security-critical changes:
@@ -195,8 +198,10 @@ Focus on:
 - XSS vulnerabilities
 - Race conditions
 - Data exposure"
+REVIEW_EXIT=$?
+set -e
 
-if [ $? -ne 0 ]; then
+if [ $REVIEW_EXIT -ne 0 ]; then
   echo "❌ Security review failed"
   exit 1
 fi

plugins/code-critic/docs/USAGE.md

@@ -190,9 +190,14 @@ if [ -z "$FILES" ]; then
 fi
 
 echo "Running adversarial code review..."
-git diff origin/main...HEAD | claude --agent adversarial-reviewer -p "Review these changes before push" --no-session-persistence -p
+set +e
+git diff origin/main...HEAD | claude --agent adversarial-reviewer \
+  --no-session-persistence \
+  -p "Review these changes before push"
+REVIEW_EXIT=$?
+set -e
 
-if [ $? -ne 0 ]; then
+if [ $REVIEW_EXIT -ne 0 ]; then
   echo "❌ Adversarial review found critical issues"
   exit 1
 fi