Updated agent to come up with a lesson each day - #729
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The updated source-priority section is internally inconsistent (missing/renumbered items and removed copyright guardrails), and the frontmatter name change may break identifier-based lookups.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates the advanced-cpp-coach agent definition to add a “daily lesson” teaching flow and revise parts of the agent’s metadata and source-citation guidance.
Changes:
- Renames the agent in YAML frontmatter.
- Revises the “Source priority” citation list (including adding a link to open-std.org).
- Adds a daily-lesson routine with objectives, quizzes, and an in-chat lesson ledger.
File summaries
| File | Description |
|---|---|
| CopilotStore/docs-cpp/agents/advanced-cpp-coach.agent.md | Updates agent metadata, citation priorities, and introduces a daily lesson/ledger coaching workflow. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
tdykstra
left a comment
tdykstra
left a comment
There was a problem hiding this comment.
Great idea! Some day I'll build an ASP.NET version...
wadepickett
left a comment
wadepickett
left a comment
There was a problem hiding this comment.
@TylerMSFT: Aproved assuming the issues are addressed that were already brought up. All 3 seem worth fixing. I am not finding any other issues than those. Very cool agent!
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Expanded the role of the Advanced C++ Learning Coach to include practical and philosophical discussions about the past and future of C++. Updated source priority and accuracy rules for clarity and detail.
|
#sign-off |
No description provided.