Conversation
Member
|
Closing. This rewrite drops the parts that define brainstorm-ideas: the six conversation principles, the mentor tone, the Phase 0–3 structure, the session picker, and the wrap-up rituals (reflection, final recommendation, "you think, I fetch"). It also inverts principle (b) and edits existing tests to fit. Entering at the current need is a fair goal, but it belongs as a short routing paragraph on top of the existing skill, not a replacement of it. |
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.
Brainstorming currently restarts a fixed conversation even when the user has already chosen a direction or asked for a concrete derivation. This PR routes by the current need: open exploration keeps Socratic guidance and optional advisor discovery, while a selected problem, resumed session, or requested report enters at that point. Advisor execution and append-only session history move to task-specific references.
The design follows OpenAI's Rethinking skills and prompts for GPT-6 Astra, while preserving the shared Claude/Codex interaction defaults validated during #62. It is the brainstorming slice split from #62 after review feedback.
Validation
python3 scripts/validate_skills.py: 16 skills passpython3 -m pytest -q: 269 passed, 5 skippedgit diff --check: clean