Fix Claude Code agent format and follow subagent best practices

- Add required YAML frontmatter (name, description) to pr-testing-agent.md
- Move pr-testing-guide.md to .claude/docs/ (documentation, not an agent)
- Replace markdown links with plain file paths per subagent best practices
- Add "Additional Context" section pointing to files the agent can read
- Update cross-references for new file locations
- Remove reference to non-existent code-reviewer agent

The agent was originally created as documentation (PR #2109) and later
moved to .claude/agents/ (PR #2112) without the required frontmatter.
This aligns it with Claude Code subagent specifications.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Author: ihabadham

.claude/agents/pr-testing-agent.md

@@ -1,19 +1,22 @@
+---
+name: pr-testing-agent
+description: Validates PR testing before merge. Use when creating PRs, investigating CI failures, or verifying testing claims. Adopts a skeptical stance - assumes tests have gaps and manual verification is required.
+---
+
 # PR Testing Agent
 
 **Role:** Specialized agent for comprehensive PR testing validation before merge.
 
 **Core Principle:** Be deeply suspicious of claims that tests passed unless you have concrete evidence. Assume automated tests have gaps. Manual testing is often required.
 
-## Quick Reference
+## Additional Context
 
-**See Also:**
+When you need more details, use the Read tool to examine:
 
-- **[PR Testing Guide](pr-testing-guide.md)** - How to use this agent with Claude Code
-- [Testing Build Scripts](../docs/testing-build-scripts.md) - Build/package testing requirements
-- [CI Config Switching](../../SWITCHING_CI_CONFIGS.md) - Testing minimum vs latest dependencies
-- [Local Testing Issues](../../react_on_rails/spec/dummy/TESTING_LOCALLY.md) - Environment-specific testing issues
-- [Master Health Monitoring](../docs/master-health-monitoring.md) - Post-merge CI monitoring
-- [CLAUDE.md](../../CLAUDE.md) - Full development guide with CI debugging
+- `.claude/docs/testing-build-scripts.md` - Build/package testing requirements
+- `.claude/docs/master-health-monitoring.md` - Post-merge CI monitoring
+- `SWITCHING_CI_CONFIGS.md` - CI config switching guide
+- `react_on_rails/spec/dummy/TESTING_LOCALLY.md` - Environment-specific testing issues
 
 ## Agent Behavior
 
@@ -146,7 +149,7 @@ bundle exec rake
 
 **Why this matters:**
 
-- See [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md) for real examples of silent failures
+- See `.claude/docs/testing-build-scripts.md` for real examples of silent failures
 - Build scripts run during `npm install`, `yalc publish`, and package installation
 - Failures are often SILENT in CI but break users completely
 
@@ -211,7 +214,7 @@ gh run list --workflow="Integration Tests" --branch <pr-branch> --limit 10 --jso
 # Key question: Did MY commits break it, or was it already broken?
 ```
 
-**See [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md) "Before You Start: Check CI Status"**
+**See `.claude/docs/testing-build-scripts.md` "Before You Start: Check CI Status"**
 
 **Reproduce failures locally:**
 
@@ -231,7 +234,7 @@ bin/ci-rerun-failures
 pbpaste | bin/ci-run-failed-specs
 ```
 
-**See [../../SWITCHING_CI_CONFIGS.md](../../SWITCHING_CI_CONFIGS.md) for full details**
+**See `SWITCHING_CI_CONFIGS.md` for full details**
 
 ### 6. Generator Changes
 
@@ -381,7 +384,7 @@ cd react_on_rails/spec/dummy
 bundle exec rspec spec/system/integration_spec.rb
 ```
 
-**See [../../react_on_rails/spec/dummy/TESTING_LOCALLY.md](../../react_on_rails/spec/dummy/TESTING_LOCALLY.md) for details**
+**See `react_on_rails/spec/dummy/TESTING_LOCALLY.md` for details**
 
 ## Success Criteria: Well-Tested PR
 

.claude/agents/pr-testing-guide.md .claude/docs/pr-testing-guide.md.claude/agents/pr-testing-guide.md renamed to .claude/docs/pr-testing-guide.md

@@ -1,6 +1,6 @@
 # PR Testing Guide
 
-**Companion to:** [PR Testing Agent](pr-testing-agent.md)
+**Companion to:** [PR Testing Agent](../agents/pr-testing-agent.md)
 
 This guide shows you **how to use** the PR Testing Agent with Claude Code, including workflows, checklists, templates, and real-world examples.
 
@@ -17,8 +17,8 @@ This guide shows you **how to use** the PR Testing Agent with Claude Code, inclu
 
 **Related Documentation:**
 
-- **[PR Testing Agent](pr-testing-agent.md)** - Core agent behavior and requirements
-- [Testing Build Scripts](../docs/testing-build-scripts.md) - Build/package testing requirements
+- **[PR Testing Agent](../agents/pr-testing-agent.md)** - Core agent behavior and requirements
+- [Testing Build Scripts](testing-build-scripts.md) - Build/package testing requirements
 - [CI Config Switching](../../SWITCHING_CI_CONFIGS.md) - Testing minimum vs latest dependencies
 - [CLAUDE.md](../../CLAUDE.md) - Full development guide
 
@@ -68,7 +68,7 @@ This guide shows you **how to use** the PR Testing Agent with Claude Code, inclu
 **The easiest way to use this agent with Claude Code is to explicitly reference it in your prompts:**
 
 ```
-"Use the PR Testing Agent from .claude/agents/pr-testing-agent.md to validate my testing"
+"Validate my testing using the pr-testing-agent"
 
 "I changed package.json. According to PR Testing Agent Section 3, what testing is required?"
 
@@ -334,7 +334,7 @@ pbpaste | bin/ci-run-failed-specs
 
 **The PR Testing Agent guidelines are automatically available when:**
 
-- You reference `.claude/agents/pr-testing-agent.md` in prompts
+- You ask about PR testing validation or readiness to merge
 - You mention "PR Testing Agent" or "testing checklist"
 - You ask about testing requirements for specific file types
 - CLAUDE.md is loaded (which references this documentation)
@@ -348,7 +348,6 @@ pbpaste | bin/ci-run-failed-specs
 **This agent complements but does not replace:**
 
 - Standard code review for logic, design, and maintainability
-- The `code-reviewer` agent (focuses on code quality, security)
 - CI automated checks (provides guidance when they fail)
 
 **This agent specializes in:**
@@ -686,8 +685,8 @@ gh pr view --json statusCheckRollup
 
 **Reference documentation:**
 
-- Testing build scripts: [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md)
+- Testing build scripts: [testing-build-scripts.md](testing-build-scripts.md)
 - CI debugging: [CLAUDE.md](../../CLAUDE.md) "Replicating CI Failures Locally"
 - Config switching: [SWITCHING_CI_CONFIGS.md](../../SWITCHING_CI_CONFIGS.md)
 - Local testing issues: [react_on_rails/spec/dummy/TESTING_LOCALLY.md](../../react_on_rails/spec/dummy/TESTING_LOCALLY.md)
-- Master health: [../docs/master-health-monitoring.md](../docs/master-health-monitoring.md)
+- Master health: [master-health-monitoring.md](master-health-monitoring.md)