Skip to content

Commit d07f203

Browse files
prosdevclaude
andcommitted
docs: fix review findings — normalize file names, align commit strategy
- Normalize service file names to *-analysis.ts across all parts - Align overview commit strategy (6 commits) with part-level commits Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 3b9bb74 commit d07f203

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

.claude/da-plans/mcp/phase-2-composite-tools/2.3-cli-commands.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,8 @@ To avoid duplicating logic between CLI and MCP adapters:
6767

6868
```
6969
packages/core/src/services/
70-
review-service.ts ← analysis logic (impact, patterns, structure)
71-
research-service.ts ← analysis logic (search, enrich, format)
70+
review-analysis.ts ← analysis logic (impact, patterns, structure)
71+
research-analysis.ts ← analysis logic (search, enrich, format)
7272
```
7373

7474
Both CLI commands and MCP adapters call these services. The services depend

.claude/da-plans/mcp/phase-2-composite-tools/overview.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -231,11 +231,12 @@ When one specialist fails (e.g., patterns times out but refs succeeds):
231231
## Commit strategy
232232

233233
```
234-
1. feat(core): add review and research analysis services
234+
1. feat(core): add review analysis service
235235
2. feat(mcp): add dev_review composite adapter
236-
3. feat(mcp): add dev_research composite adapter
237-
4. feat(cli): add dev review and dev research commands
238-
5. docs: update CLAUDE.md, agents, and doc site for composite tools
236+
3. feat(core): add research analysis service
237+
4. feat(mcp): add dev_research composite adapter
238+
5. feat(cli): add dev review and dev research commands
239+
6. docs: update CLAUDE.md, agents, and doc site for composite tools
239240
```
240241

241242
---

0 commit comments

Comments
 (0)