Add name elicitation, session wrap-up, and maxTurns limit#14
Merged
Conversation
- Add name elicitation to Level Detection & Initialization flow: ask for name alongside level if neither is given; ask for name alone if level is declared but name is absent; use name naturally throughout session - Add Session Wrap-up section triggered by end-of-session signals; covers what was built, levels used, engagement quality, and recommendation for next session - Add maxTurns: 40 to frontmatter to create a natural session checkpoint Closes #2 Closes #8 Closes #10 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
Summary
maxTurns: 40frontmatter field to create a natural session checkpointIssues
Closes #2
Closes #8
Closes #10
Changes in detail
Name elicitation (#2): The initialization flow now has an explicit step 2 that checks whether the user's name is known. If no level and no name were given, the "no level detected" response asks for both together. If a level was declared but name is absent, the confirmation opens with a name request. Once known, the name is used naturally throughout the session. The confirmation template now reads: "Great to meet you, [Name]. Working at Level N."
Session wrap-up (#8): New
## Session Wrap-upsection before Meta Reminder. Triggers on end-of-session signals ("thanks", "done for today", "end session", etc.). Produces a concise 5–8 line retrospective covering: what was built, levels used and their appropriateness, engagement quality, and a level recommendation for next time. Also notes open work if the session ends mid-task.maxTurns (#10): Added
maxTurns: 40to frontmatter. 40 turns supports a substantive session (~20 human + 20 agent turns) while preventing unbounded runaway sessions.Test plan
maxTurns: 40appears in frontmattermarkdownlint agents/coach.md— zero warnings🤖 Generated with Claude Code