-
Notifications
You must be signed in to change notification settings - Fork 35
docs: add memory agent to CUSTOM-AGENTS.md #359
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Hi @WilliamBerryiii, I’ve added the memory agent to CUSTOM-AGENTS.md and included its details following the existing format. Please let me know if any changes are needed. Thanks! |
|
Thanks for pointing this out — you’re right. The Thanks for the guidance. |
|
Thanks for the guidance. I’ve restored the original content and updated the PR to only add the memory agent, following the existing structure and patterns. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #359 +/- ##
==========================================
- Coverage 52.44% 52.41% -0.04%
==========================================
Files 17 17
Lines 3110 3110
==========================================
- Hits 1631 1630 -1
- Misses 1479 1480 +1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
3cfcef6 to
74c59bb
Compare
74c59bb to
e1004f8
Compare
|
Thanks for the contribution @SrishJ23 ... merging now. |
|
@katriendg, Can you review the changes? I'm also going to file an issue that maybe consider lightening this document to just enumerate the groupings of agents and create detailed agent/workflow explanations in the /docs directory. Thoughts? |
|
|
||
| **Creates:** Architecture Decision Records: | ||
|
|
||
| * `.copilot-tracking/adrs/{{topic-name}}-draft.md` (working draft) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we keep the original, this is relevant information, same on the line 201, keep the (final location) wording
|
|
||
| **Creates:** Installation configuration and tracking: | ||
|
|
||
| * `.hve-tracking.json` (for agent copy tracking) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please keep the (for agent copy tracking) on line 294
| **Creates:** Installation configuration and tracking: | ||
|
|
||
| * `.hve-tracking.json` (for agent copy tracking) | ||
| * Settings updates in `.vscode/settings.json` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here also keep Settings updates in wording, it is relevant.
|
@WilliamBerryiii I like the idea of moving the detailed documentations into the |
|
@SrishJ23 Thanks for the changes - I did see a few changes that we should revert, small additions of wording that we want to keep. Once you revert those back we can approve and merge! |
Fixes #311
This PR documents the
memoryagent in CUSTOM-AGENTS.md.