feat: add council-v2 skill with routed advisor roster#90
Merged
Conversation
Variant of council whose roster is built from existing reasoning skills rather than ad-hoc personas. Replaces the two seats that overlapped: First Principles Thinker delegates to first-principles-mode, and the Executor is superseded by founder-thinking-mode. Contrarian, Outsider, and Expansionist survive since no new voice covers their job. Adds persona-stanier and persona-levelsio as domain seats, seated only when the question is a leadership or indie/bootstrap call, so seats with nothing to say stay benched instead of adding filler. Slash-only to avoid trigger collision with council. Peer-review anonymization dropped since the persona voices are unmaskable; chairman now dedups so repetition is not mistaken for consensus.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughAdds the ChangesCouncil v2 workflow
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant User
participant council-v2
participant Advisors
participant Reviewers
participant Chairman
participant Outputs
User->>council-v2: invoke /council-v2
council-v2->>Advisors: route and spawn seats in parallel
Advisors-->>council-v2: return advisor responses
council-v2->>Reviewers: submit responses for peer review
Reviewers-->>Chairman: return evaluations
council-v2->>Chairman: provide framed question and responses
Chairman->>Outputs: save HTML report and transcript
Outputs-->>User: return verdict and output paths
Possibly related PRs
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
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.
What
Adds
skills/council-v2/— a variant ofcouncilwhose advisor roster is built from existing reasoning skills instead of ad-hoc personas.Roster
Two v1 seats were replaced where the jobs overlapped. The Contrarian, Outsider, and Expansionist survive because no new voice covers their job. The two domain seats are seated only when the question fits, so a seat with nothing to say stays benched instead of adding filler.
The new roster also adds two productive tensions: Expansionist vs levelsio (10x thinking against stay-small) and Stanier vs Founder (build the system against just make the call).
Design decisions worth reviewing
disable-model-invocation: true).councilkeeps the natural-language triggers. Two auto-firing councils would collide nondeterministically on phrases like "pressure-test this".Verification
quick_validate.pypasses (description 923 chars, no": "terminator)check-universality.shclean across 297 files## Skillsrow added with all four dependencies declaredSummary by CodeRabbit
New Features
council-v2skill for structured multi-advisor reasoning.Documentation