Skip to content

Commit 79e334c

Browse files
justin808claude
andauthored
Reorganize PR testing docs into .claude/agents/ (#2112)
### Summary Reorganizes the PR testing documentation into a clearer directory structure, following the established pattern of `.claude/commands/`. ### Changes **New structure:** ``` .claude/ ├── agents/ # Agent definitions (NEW) │ ├── pr-testing-agent.md │ └── pr-testing-guide.md ├── commands/ # Slash commands └── docs/ # Supporting documentation ├── analysis/ ├── testing-build-scripts.md ├── master-health-monitoring.md └── ... ``` **What changed:** - Created `.claude/agents/` directory - Moved `pr-testing-agent.md` and `pr-testing-guide.md` from `docs/` to `agents/` - Updated all cross-references in both files (now point to `../docs/` for supporting docs) - Updated prompt examples to use new path (`.claude/agents/pr-testing-agent.md`) ### Benefits - **Clear separation**: Agents vs commands vs supporting documentation - **Consistent structure**: Follows the `.claude/commands/` pattern - **Easy to find**: Agent definitions have their own directory - **Future-proof**: Ready for additional agents - **No functionality changes**: Only file moves and reference updates ### Pull Request checklist - [x] Add/update test to cover these changes - N/A, documentation reorganization - [x] Update documentation - Updated all cross-references - [x] Update CHANGELOG file - N/A, documentation reorganization <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated PR Testing Agent and PR Testing Guide: fixed several relative links and references to point to the new documentation locations and corrected prompt/example references. No functional changes. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude <[email protected]>
1 parent 20d6969 commit 79e334c

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@
99
**See Also:**
1010

1111
- **[PR Testing Guide](pr-testing-guide.md)** - How to use this agent with Claude Code
12-
- [Testing Build Scripts](testing-build-scripts.md) - Build/package testing requirements
12+
- [Testing Build Scripts](../docs/testing-build-scripts.md) - Build/package testing requirements
1313
- [CI Config Switching](../../SWITCHING_CI_CONFIGS.md) - Testing minimum vs latest dependencies
1414
- [Local Testing Issues](../../spec/dummy/TESTING_LOCALLY.md) - Environment-specific testing issues
15-
- [Master Health Monitoring](master-health-monitoring.md) - Post-merge CI monitoring
15+
- [Master Health Monitoring](../docs/master-health-monitoring.md) - Post-merge CI monitoring
1616
- [CLAUDE.md](../../CLAUDE.md) - Full development guide with CI debugging
1717

1818
## Agent Behavior
@@ -146,7 +146,7 @@ bundle exec rake
146146

147147
**Why this matters:**
148148

149-
- See [testing-build-scripts.md](testing-build-scripts.md) for real examples of silent failures
149+
- See [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md) for real examples of silent failures
150150
- Build scripts run during `npm install`, `yalc publish`, and package installation
151151
- Failures are often SILENT in CI but break users completely
152152

@@ -211,7 +211,7 @@ gh run list --workflow="Integration Tests" --branch <pr-branch> --limit 10 --jso
211211
# Key question: Did MY commits break it, or was it already broken?
212212
```
213213

214-
**See [testing-build-scripts.md](testing-build-scripts.md) "Before You Start: Check CI Status"**
214+
**See [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md) "Before You Start: Check CI Status"**
215215

216216
**Reproduce failures locally:**
217217

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

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ This guide shows you **how to use** the PR Testing Agent with Claude Code, inclu
1818
**Related Documentation:**
1919

2020
- **[PR Testing Agent](pr-testing-agent.md)** - Core agent behavior and requirements
21-
- [Testing Build Scripts](testing-build-scripts.md) - Build/package testing requirements
21+
- [Testing Build Scripts](../docs/testing-build-scripts.md) - Build/package testing requirements
2222
- [CI Config Switching](../../SWITCHING_CI_CONFIGS.md) - Testing minimum vs latest dependencies
2323
- [CLAUDE.md](../../CLAUDE.md) - Full development guide
2424

@@ -68,7 +68,7 @@ This guide shows you **how to use** the PR Testing Agent with Claude Code, inclu
6868
**The easiest way to use this agent with Claude Code is to explicitly reference it in your prompts:**
6969

7070
```
71-
"Use the PR Testing Agent from .claude/docs/pr-testing-agent.md to validate my testing"
71+
"Use the PR Testing Agent from .claude/agents/pr-testing-agent.md to validate my testing"
7272
7373
"I changed package.json. According to PR Testing Agent Section 3, what testing is required?"
7474
@@ -334,7 +334,7 @@ pbpaste | bin/ci-run-failed-specs
334334

335335
**The PR Testing Agent guidelines are automatically available when:**
336336

337-
- You reference `.claude/docs/pr-testing-agent.md` in prompts
337+
- You reference `.claude/agents/pr-testing-agent.md` in prompts
338338
- You mention "PR Testing Agent" or "testing checklist"
339339
- You ask about testing requirements for specific file types
340340
- CLAUDE.md is loaded (which references this documentation)
@@ -686,8 +686,8 @@ gh pr view --json statusCheckRollup
686686

687687
**Reference documentation:**
688688

689-
- Testing build scripts: [testing-build-scripts.md](testing-build-scripts.md)
689+
- Testing build scripts: [../docs/testing-build-scripts.md](../docs/testing-build-scripts.md)
690690
- CI debugging: [CLAUDE.md](../../CLAUDE.md) "Replicating CI Failures Locally"
691691
- Config switching: [SWITCHING_CI_CONFIGS.md](../../SWITCHING_CI_CONFIGS.md)
692692
- Local testing issues: [spec/dummy/TESTING_LOCALLY.md](../../spec/dummy/TESTING_LOCALLY.md)
693-
- Master health: [master-health-monitoring.md](master-health-monitoring.md)
693+
- Master health: [../docs/master-health-monitoring.md](../docs/master-health-monitoring.md)

0 commit comments

Comments
 (0)