Add configurable review agents via .local.md settings#124
Open
kieranklaassen wants to merge 10 commits intomainfrom
Open
Add configurable review agents via .local.md settings#124kieranklaassen wants to merge 10 commits intomainfrom
kieranklaassen wants to merge 10 commits intomainfrom
Conversation
Adds a new setup command that allows users to configure which review agents to use instead of hardcoding them in workflows. This enables: - Multi-step onboarding with AskUserQuestion for easy setup - Auto-detection of project type (Rails, Python, TypeScript, etc.) - Three setup modes: Quick (smart defaults), Advanced, and Minimal - Configuration stored in .claude/compound-engineering.json - Support for both global (~/.claude/) and project-specific config Updated workflows to read from config: - /workflows:review - reads reviewAgents from config - /plan_review - reads planReviewAgents from config - /workflows:work - references config for reviewer agents - /workflows:compound - references config for specialized agents 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Workflows now detect missing config and offer inline quick setup: - "Quick Setup" - auto-detect project type, create config, continue - "Full Setup" - run /compound-engineering-setup for customization - "Skip" - use defaults just this once This ensures users get onboarded automatically when running any workflow for the first time, without needing to know about the setup command beforehand. Co-Authored-By: Claude Opus 4.5 <[email protected]>
Extended /workflows:review to invoke conditional agents for: - migrations (existing) - frontend (new): JS/TS/Stimulus changes - architecture (new): structural changes, 10+ files - data (new): model/ActiveRecord changes Each category reads from conditionalAgents.* config key and runs appropriate specialized agents when file patterns match. Resolves: todos/001-ready-p2-conditional-agents-not-invoked.md Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Auto-detect custom agents in .claude/agents/ and ~/.claude/agents/ - Add modify existing config flow (add/remove agents, view config) - Include guide for creating custom review agents - Add customAgents mapping in config to track agent file paths - Update changelog with new config schema including customAgents Co-Authored-By: Claude Opus 4.5 <[email protected]>
Co-Authored-By: Claude Opus 4.5 <[email protected]>
kieranklaassen
force-pushed
the
feat/compound-engineering-setup
branch
from
3e765f7 to
a1413c9
Compare
- Add "Ask more questions" option at handoff phase - Clarify that Claude should ask the user questions (not wait for user) - Require resolving ALL open questions before offering to proceed Co-Authored-By: Claude Opus 4.5 <[email protected]>
This was referenced Feb 8, 2026
Collaborator
Author
|
@tmchow curious what you think of this one |
Resolve conflicts favoring main's versions for workflow files that will be reworked with simplified settings approach. Co-Authored-By: Claude Opus 4.6 <[email protected]>
- Rewrite setup.md (486 → 95 lines): detect project type, create .claude/compound-engineering.local.md with smart defaults - Make review.md and work.md config-aware: read agents from .local.md frontmatter, fall back to auto-detected defaults - Wire schema-drift-detector into review.md migrations conditional block - Delete technical_review.md (duplicated /plan_review) - Add disable-model-invocation to setup.md - Bump to v2.32.0 Co-Authored-By: Claude Opus 4.6 <[email protected]>
kieranklaassen
changed the title
kieranklaassen
changed the title
…rkflow - Converters now rewrite .claude/ → .opencode/ (OpenCode) and .codex/ (Codex) in command bodies and agent bodies so .local.md settings work cross-platform - Apply transformContentForCodex to agent bodies (was only commands before) - Add GitHub Action to auto-publish to npm on version bump merge to main - Bump to v0.4.0 Co-Authored-By: Claude Opus 4.6 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds per-project review agent configuration using the
.local.mdplugin settings pattern. Run/compound-engineering-setupto generate a settings file, or let workflows auto-detect your project type and pick sensible defaults.Settings file
Setup command
/compound-engineering-setupdetects your project type (Rails, Python, TypeScript, or General) and creates.claude/compound-engineering.local.mdwith appropriate defaults. Edit the file anytime to add/remove agents or add review context that gets passed to all agents.Config-aware workflows
/workflows:reviewand/workflows:workread agents from the settings file when present. Without it, they auto-detect project type and use defaults — no config required.Conditional agents (migrations, frontend, architecture, data integrity) stay hardcoded — they trigger based on file patterns in the PR, not user config.
Schema drift detection
Wires the existing
schema-drift-detectoragent into/workflows:review— runs first on any PR touchingdb/migrate/ordb/schema.rbto catch unrelated schema changes before other DB reviewers run.Test plan
/compound-engineering-setupin a Rails project — verify it detects Rails and creates.local.mdwith Rails defaults/compound-engineering-setupwhen.local.mdalready exists — verify it asks before overwriting/workflows:reviewwith.local.mdpresent — verify it uses configured agents/workflows:reviewwithout.local.md— verify it falls back to auto-detected defaults/workflows:reviewon a PR withdb/migrate/files — verify schema-drift-detector runs/workflows:work— verify it reads from.local.mdwhen present🤖 Generated with Claude Code