[WIP] Update prompt wording to align with Copilot documentation#128
Conversation
Co-authored-by: Bryan-Roe <74067792+Bryan-Roe@users.noreply.github.com>
🔴 Coverage —
|
| Metric | Value |
|---|---|
| Total coverage | 59.7% |
→ vs main |
0.0% |
| Minimum threshold | 60% |
Updated on every push · 2026-05-09
There was a problem hiding this comment.
Pull request overview
Updates Copilot-facing documentation to better describe the intent/usage of agi.prompt.md, and adds a gitignore entry for generated chat-cli logs.
Changes:
- Adjusts
.github/copilot-instructions.mdprompt listing to describeagi.prompt.mdas structured analysis and not exposing hidden chain-of-thought. - Updates
.github/COPILOT_SETUP_GUIDE.mdprompt table entry to match the revisedagi.prompt.mddescription. - Adds
ai-projects/chat-cli/logs/to.gitignore.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.gitignore |
Ignores generated chat-cli log output directory. |
.github/copilot-instructions.md |
Updates prompt catalog wording for agi.prompt.md. |
.github/COPILOT_SETUP_GUIDE.md |
Updates prompt purpose table wording for agi.prompt.md. |
| | Prompt | Purpose | | ||
| |--------|---------| | ||
| | `agi.prompt.md` | Chain-of-thought AGI reasoning | | ||
| | `agi.prompt.md` | AGI reasoning (structured analysis; do not expose hidden chain-of-thought) | |
There was a problem hiding this comment.
This table row now says agi.prompt.md is "structured analysis" and "do not expose hidden chain-of-thought", but the actual .github/prompts/agi.prompt.md content doesn’t state that restriction and includes enable_chain_of_thought=True. Also, later in this same guide the example text still tells readers to use agi.prompt for chain-of-thought reasoning. Please align the guide with the prompt’s real behavior (or update the prompt).
There was a problem hiding this comment.
Fixed in commit 0f5547b. The table entry in COPILOT_SETUP_GUIDE.md now reads "AGI reasoning with structured analysis" — removing the inaccurate "do not expose hidden chain-of-thought" constraint that contradicted agi.prompt.md's actual behavior (enable_chain_of_thought=True). The example text referencing chain-of-thought reasoning was kept as it accurately reflects the prompt's behavior.
There was a problem hiding this comment.
Hi @Bryan-Roe! 👋
Your private repo does not have access to Sourcery.
Please upgrade to continue using Sourcery ✨
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…behavior Agent-Logs-Url: https://github.com/Bryan-Roe/Aria/sessions/786cfd57-76e7-4791-8203-a66a55e32e3c Co-authored-by: Bryan-Roe <74067792+Bryan-Roe@users.noreply.github.com>
Signed-off-by: Bryan <74067792+Bryan-Roe@users.noreply.github.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.