Skip to content

Commit be5016b

Browse files
stephendolanclaude
andcommitted
refactor(ai): replace counselors with Codex plugin for external reviews
Counselors CLI is superseded by the official Codex plugin (openai-codex), which provides GPT 5.4 second opinions via /codex:adversarial-review and the codex:codex-rescue agent. Removes the skill, config, Brewfile entries, DotBot symlinks, and updates all referencing skills/agents. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 15fc9c6 commit be5016b

File tree

14 files changed

+24
-228
lines changed

14 files changed

+24
-228
lines changed

CLAUDE.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@ The `ai/` directory contains configuration for Claude Code and other AI editors:
8989
- `ai/AGENTS.md` - Shared instructions (symlinked as `~/.claude/CLAUDE.md` and `~/.config/AGENTS.md`)
9090
- `ai/agents/*.md` - Agent prompts that define specialized subagent behavior
9191
- `ai/skills/*/SKILL.md` - Skill prompts that define workflows (slash commands) and domain expertise
92-
- `ai/counselors/config.json` - Counselors CLI config (symlinked to `~/.config/counselors/config.json`)
9392
- `ai/hooks/*.sh` - Hook scripts for Claude Code lifecycle events
9493
- `ai/mcp.json` - MCP server definitions (source of truth)
9594
- `ai/generate-mcp.sh` - Generates MCP configs for Claude Desktop, Claude Code CLI, and Codex CLI

ai/.claude-plugin/plugin.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "stephendolan",
3-
"version": "1.5.0",
3+
"version": "1.6.0",
44
"description": "Personal agents and skills for development workflows, code review, architecture analysis, and domain expertise.",
55
"author": {
66
"name": "Stephen Dolan",

ai/AGENTS.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ When work decomposes into independent pieces, delegate each to a sub-agent and r
2323
| -------------------------- | ----------------------------------------- |
2424
| `/commit` | Commit with conventional message (why > what) |
2525
| `/create-pr` | Create PR with concise description |
26-
| `/counselors` | Fan out a review to Claude and Codex |
2726
| `/ship` | Autonomous end-to-end feature development |
2827
| `/refine-implementation` | Multi-pass code review before commit |
2928
| `/examine-architecture` | Evaluate codebase for structural problems |
@@ -79,9 +78,8 @@ Avoid temporal references: "vs previous", "used to be X", "now uses Y", "the new
7978

8079
### External Review Gate
8180

82-
Use `/counselors` selectively when you want a true external second opinion from the configured Claude and Codex tools.
81+
Use `/codex:adversarial-review` to get a second opinion from GPT 5.4 on implementation diffs. Use the `codex:codex-rescue` agent to delegate plan reviews to Codex.
8382

84-
- `/ship` is the exception: run one `/counselors` pass on the implementation plan before building
8583
- Good targets: high-risk plans before implementation, architecture reviews with cross-cutting findings, and risky diffs before commit
8684
- Bad targets: small edits, routine refactors, or cases where local reviewers already agree and the risk is low
8785

ai/README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ ai/
3636
plugin.json Plugin manifest
3737
agents/ Subagent definitions
3838
skills/ Slash command workflows and domain expertise
39-
counselors/ Counselors CLI configuration
4039
hooks/ Event handlers
4140
AGENTS.md Shared instructions
4241
mcp.json MCP server definitions
@@ -121,7 +120,6 @@ flowchart LR
121120
| -------------------------- | -------------------------------------------------- |
122121
| `/commit` | Commit with conventional message (why > what) |
123122
| `/create-pr` | Create PR with concise description |
124-
| `/counselors` | Fan out a review to Claude and Codex in parallel |
125123
| `/ship` | Autonomous end-to-end feature development |
126124
| `/refine-implementation` | Multi-pass quality review before committing |
127125
| `/examine-architecture` | Evaluate codebase for structural problems |

ai/agents/plan-refiner.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ Suggest alternatives only with high confidence. Intervene when:
2626
3. Architecture hurts maintainability
2727
4. Missing context might change the optimal approach
2828

29-
If the plan is high-risk and uncertainty remains after reviewing local context, recommend a `/counselors` pass rather than pretending confidence.
29+
If the plan is high-risk and uncertainty remains after reviewing local context, recommend a `/codex:adversarial-review` pass rather than pretending confidence.
3030

3131
If the plan is reasonable, approve it and move on.
3232

@@ -65,7 +65,7 @@ Only suggest changes when elegance gains clearly outweigh costs. Prefer what's e
6565
For plans touching auth, migrations, public APIs, infrastructure, billing, or large cross-cutting refactors:
6666

6767
- If you have a strong view, state it plainly
68-
- If you do not, say so explicitly and recommend a `/counselors` review to gather one more independent perspective
68+
- If you do not, say so explicitly and recommend a `/codex:adversarial-review` to gather one more independent perspective
6969

7070
## Output Format
7171

ai/claude-settings.json

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,16 @@
296296
"stephendolan@dotfiles": true,
297297
"swift-lsp@claude-plugins-official": true,
298298
"clangd-lsp@claude-plugins-official": true,
299-
"ruby-lsp@claude-plugins-official": true
299+
"ruby-lsp@claude-plugins-official": true,
300+
"codex@openai-codex": true
301+
},
302+
"extraKnownMarketplaces": {
303+
"openai-codex": {
304+
"source": {
305+
"source": "github",
306+
"repo": "openai/codex-plugin-cc"
307+
}
308+
}
300309
},
301310
"spinnerTipsOverride": {
302311
"tips": [

ai/counselors/config.json

Lines changed: 0 additions & 34 deletions
This file was deleted.

ai/skills/counselors/SKILL.md

Lines changed: 0 additions & 161 deletions
This file was deleted.

ai/skills/examine-architecture/SKILL.md

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ You are examining a codebase's architecture to identify structural problems. Fol
1414
- **Analyze, don't fix**: This command identifies problems and creates implementation plans, but doesn't make changes
1515
- **Parallel analysis**: Launch multiple agents to examine different surfaces simultaneously
1616
- **Validate recommendations**: Use plan-refiner to eliminate over-engineering from proposed fixes
17-
- **Use Counselors on the top issues only**: External second opinions are valuable for high-severity or cross-cutting recommendations, not every surface
1817
- **Use TodoWrite**: Track all progress throughout
1918

2019
---
@@ -84,10 +83,9 @@ Focus area: $ARGUMENTS
8483
- Validate tractability
8584
- Suggest simpler approaches
8685
- Flag anything too complex to be worth fixing
87-
3. If the top issues are high-severity, cross-cutting, or likely to be controversial, run `/counselors` on the top 2-3 issues and proposed fixes
88-
4. Ask Counselors to challenge the root-cause analysis, spot missed risks, and call out fixes that feel overbuilt
89-
5. Incorporate plan-refiner and Counselors feedback
90-
6. Present validated findings to user
86+
3. If the top issues are high-severity, cross-cutting, or likely to be controversial, launch the `codex:codex-rescue` agent to challenge the root-cause analysis and spot missed risks
87+
4. Incorporate plan-refiner and Codex feedback
88+
5. Present validated findings to user
9189

9290
---
9391

ai/skills/refine-implementation/SKILL.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ After reviewing and discussing with agents as needed:
8686

8787
### 6. External Second Opinion (Optional)
8888

89-
Run `/counselors` after local reconciliation when the change is risky enough to justify one more pass.
89+
Run `/codex:adversarial-review` after local reconciliation when the change is risky enough to justify one more pass.
9090

9191
Good triggers:
9292

@@ -95,8 +95,6 @@ Good triggers:
9595
- Local review agents disagreeing or repeatedly escalating
9696
- A branch that feels high-consequence and worth one last independent check
9797

98-
Ask Counselors to focus on regressions, blind spots, and missing tests. Give it the key files and branch diff context, not a giant pasted prompt.
99-
10098
Incorporate valid findings, then do another local pass only if the external review changed the shape of the branch materially.
10199

102100
### 7. Check for Another Pass
@@ -127,6 +125,6 @@ Stop after:
127125

128126
- **No commits until user approves** - Changes stay uncommitted for review
129127
- **Multiple perspectives** - Different agents catch different issues
130-
- **External review is a scalpel** - Use `/counselors` for risky or disputed changes, not every pass
128+
- **External review is a scalpel** - Use `/codex:adversarial-review` for risky or disputed changes, not every pass
131129
- **Iterative improvement** - One pass rarely catches everything
132130
- **User control** - Always show changes, never auto-commit

0 commit comments

Comments
 (0)