From 948032658570cf901ee1791d81f558888864d6e5 Mon Sep 17 00:00:00 2001 From: Justin Gordon Date: Sun, 23 Nov 2025 12:34:18 -1000 Subject: [PATCH] Reorganize: Move PR testing docs to .claude/agents/ MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Creates a clearer structure following the /commands pattern: .claude/ ├── agents/ # Agent definitions (NEW) │ ├── pr-testing-agent.md │ └── pr-testing-guide.md ├── commands/ # Slash commands └── docs/ # Supporting documentation Benefits: - Clear separation: agents vs commands vs supporting docs - Consistent with /commands structure - Easy to find agent definitions - Future-proof for more agents Changes: - Created .claude/agents/ directory - Moved pr-testing-agent.md from docs/ to agents/ - Moved pr-testing-guide.md from docs/ to agents/ - Updated all cross-references to use ../docs/ paths - Updated prompt examples to use .claude/agents/ path No duplicate files remain - clean move from docs/ to agents/ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- .claude/{docs => agents}/pr-testing-agent.md | 8 ++++---- .claude/{docs => agents}/pr-testing-guide.md | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) rename .claude/{docs => agents}/pr-testing-agent.md (97%) rename .claude/{docs => agents}/pr-testing-guide.md (97%) diff --git a/.claude/docs/pr-testing-agent.md b/.claude/agents/pr-testing-agent.md similarity index 97% rename from .claude/docs/pr-testing-agent.md rename to .claude/agents/pr-testing-agent.md index 18c130a9ec..179cd9dcb4 100644 --- a/.claude/docs/pr-testing-agent.md +++ b/.claude/agents/pr-testing-agent.md @@ -9,10 +9,10 @@ **See Also:** - **[PR Testing Guide](pr-testing-guide.md)** - How to use this agent with Claude Code -- [Testing Build Scripts](testing-build-scripts.md) - Build/package testing requirements +- [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](../../spec/dummy/TESTING_LOCALLY.md) - Environment-specific testing issues -- [Master Health Monitoring](master-health-monitoring.md) - Post-merge CI monitoring +- [Master Health Monitoring](../docs/master-health-monitoring.md) - Post-merge CI monitoring - [CLAUDE.md](../../CLAUDE.md) - Full development guide with CI debugging ## Agent Behavior @@ -146,7 +146,7 @@ bundle exec rake **Why this matters:** -- See [testing-build-scripts.md](testing-build-scripts.md) for real examples of silent failures +- See [../docs/testing-build-scripts.md](../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 +211,7 @@ gh run list --workflow="Integration Tests" --branch --limit 10 --jso # Key question: Did MY commits break it, or was it already broken? ``` -**See [testing-build-scripts.md](testing-build-scripts.md) "Before You Start: Check CI Status"** +**See [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md) "Before You Start: Check CI Status"** **Reproduce failures locally:** diff --git a/.claude/docs/pr-testing-guide.md b/.claude/agents/pr-testing-guide.md similarity index 97% rename from .claude/docs/pr-testing-guide.md rename to .claude/agents/pr-testing-guide.md index 9e1e24aa80..1cea127417 100644 --- a/.claude/docs/pr-testing-guide.md +++ b/.claude/agents/pr-testing-guide.md @@ -18,7 +18,7 @@ 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](testing-build-scripts.md) - Build/package testing requirements +- [Testing Build Scripts](../docs/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/docs/pr-testing-agent.md to validate my testing" +"Use the PR Testing Agent from .claude/agents/pr-testing-agent.md to validate my testing" "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/docs/pr-testing-agent.md` in prompts +- You reference `.claude/agents/pr-testing-agent.md` in prompts - 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) @@ -686,8 +686,8 @@ gh pr view --json statusCheckRollup **Reference documentation:** -- Testing build scripts: [testing-build-scripts.md](testing-build-scripts.md) +- Testing build scripts: [../docs/testing-build-scripts.md](../docs/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: [spec/dummy/TESTING_LOCALLY.md](../../spec/dummy/TESTING_LOCALLY.md) -- Master health: [master-health-monitoring.md](master-health-monitoring.md) +- Master health: [../docs/master-health-monitoring.md](../docs/master-health-monitoring.md)