Add mcp-research agent + mcps/ output dir; add templates/agents.md; apply core principles to existing agents#7
Merged
ChrisThompsonTLDR merged 6 commits intomainfrom Apr 8, 2026
Conversation
Agent-Logs-Url: https://github.com/ChrisThompsonTLDR/agentic-programming/sessions/b8c67413-7a98-489d-a1d6-6ecd034d83c4 Co-authored-by: ChrisThompsonTLDR <348801+ChrisThompsonTLDR@users.noreply.github.com>
…in README Agent-Logs-Url: https://github.com/ChrisThompsonTLDR/agentic-programming/sessions/30d1e26c-e45b-41f8-996e-131c5767a9b3 Co-authored-by: ChrisThompsonTLDR <348801+ChrisThompsonTLDR@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ChrisThompsonTLDR/agentic-programming/sessions/f66f2504-ecae-4b0b-987a-7d1a93055988 Co-authored-by: ChrisThompsonTLDR <348801+ChrisThompsonTLDR@users.noreply.github.com>
…gents Agent-Logs-Url: https://github.com/ChrisThompsonTLDR/agentic-programming/sessions/49db6d29-c230-40da-b22e-c4e74ea37fa8 Co-authored-by: ChrisThompsonTLDR <348801+ChrisThompsonTLDR@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ChrisThompsonTLDR/agentic-programming/sessions/ff88fe47-3328-41ff-8bce-555068d4da59 Co-authored-by: ChrisThompsonTLDR <348801+ChrisThompsonTLDR@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
ChrisThompsonTLDR
April 8, 2026 07:44
View session
Owner
|
@coderabbitai full review |
✅ Actions performedFull review triggered. |
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (51)
WalkthroughReplaces the Cursor multi-phase command pipeline and MCP registry with GitHub Copilot Cloud agent workflows: adds three new agents (laravel-package, mcp-research, skill-research), skill docs and templates, setup workflow and output directories, and removes many command, support, and mcp.json files (plus a Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant LPAgent as laravel-package<br/>Agent
participant GitHub
participant Packagist
participant DeepWiki
participant Context7
participant FileSystem
User->>LPAgent: /laravel-package vendor/package
LPAgent->>LPAgent: Parse & validate slug
LPAgent->>GitHub: Fetch repo metrics & releases
LPAgent->>Packagist: Fetch package stats & downloads
LPAgent->>DeepWiki: Search Laravel News / docs
LPAgent->>Context7: (optional) fetch code attributes
LPAgent->>LPAgent: Render templated markdown
LPAgent->>FileSystem: Write .steering/laravel-packages/vendor__package.md
LPAgent-->>User: Return file path
sequenceDiagram
actor User
participant SRAgent as skill-research<br/>Agent
participant GitHub
participant DeepWiki
participant FileSystem
User->>SRAgent: /skill-research <github-skill-dir-url>
SRAgent->>SRAgent: Parse URL -> owner/repo/skill-name/path
SRAgent->>GitHub: Read SKILL.md & enumerate files
SRAgent->>GitHub: Read parent README & search for .agent.md/workflows
SRAgent->>DeepWiki: (optional) repo summary/context
SRAgent->>SRAgent: Populate skill-template.md
SRAgent->>FileSystem: Write .steering/skills/owner__skill-name.md
SRAgent-->>User: Return file path
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Actions performedFull review triggered. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 10
♻️ Duplicate comments (2)
.github/agents/skill-research.agent.md (1)
6-8:⚠️ Potential issue | 🔴 CriticalDeepWiki MCP server missing authentication configuration.
The DeepWiki SSE server lacks authentication configuration. This is the same issue present in
.github/agents/laravel-package.agent.md. TheDEEPWIKI_KEYsecret documented in the repository setup is not being passed to the server.See the review comment on
.github/agents/laravel-package.agent.mdlines 6-8 for the proposed fix.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/agents/skill-research.agent.md around lines 6 - 8, The DeepWiki SSE config block named "deepwiki" is missing authentication and must pass the DEEPWIKI_KEY secret to the server; update the "deepwiki" entry (the SSE config with keys "type: sse" and "url: https://api.deepwiki.com/sse") to include the auth header or token field that sends process/env DEEPWIKI_KEY (e.g., an "headers" or "auth" property containing Authorization: Bearer <DEEPWIKI_KEY> or a "token" field) so the SSE connection uses the repository secret; mirror the same approach used to fix the laravel-package agent config..github/agents/mcp-research.agent.md (1)
6-8:⚠️ Potential issue | 🔴 CriticalDeepWiki MCP server missing authentication configuration.
The DeepWiki SSE server lacks authentication configuration. This is the same issue present in the other agent definitions.
See the review comment on
.github/agents/laravel-package.agent.mdlines 6-8 for the proposed fix.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/agents/mcp-research.agent.md around lines 6 - 8, The DeepWiki agent definition (the deepwiki mapping with keys type and url) is missing authentication configuration; update the deepwiki entry to include an authentication block matching the other agent definitions (e.g., add an authentication/auth or headers object that specifies the auth type—such as bearer/token or apiKey—and where to read the credential from, then include the header/key name to send with requests) so SSE connections to https://api.deepwiki.com/sse are authenticated the same way as the other agents.
🧹 Nitpick comments (4)
.github/skills/agentic-programming/SKILL.md (1)
30-38: Specify language for code blocks.The code blocks should specify a language identifier for better rendering.
📝 Proposed fix
-``` +```bash /laravel-package <vendor/package>Example:
-
+bash
/laravel-package spatie/laravel-markdown-response🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/skills/agentic-programming/SKILL.md around lines 30 - 38, Update the code blocks in SKILL.md that show the "/laravel-package <vendor/package>" command and its example to include a language identifier (bash) for proper rendering: locate the fenced code blocks containing "/laravel-package <vendor/package>" and "/laravel-package spatie/laravel-markdown-response" and change their opening fences to "```bash" so both the command and the example are marked as bash code blocks.README.md (2)
102-122: Specify language for file structure code block.The file structure tree should use a language identifier.
📝 Proposed fix
-``` +```text .github/ ├── agents/🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 102 - 122, The README's fenced file-tree block is missing a language identifier; update the code block that starts with ".github/" (the file tree showing agents/, skills/, workflows/, .steering/) to include a language tag such as ```text so the tree renders correctly and preserves spacing/characters when displayed.
39-47: Specify language for code blocks.The code blocks should specify language identifiers for better rendering.
📝 Proposed fix
-``` +```bash /laravel-package spatie/laravel-markdown-responseThe agent will research the package and write a note to:
-
+text
.steering/laravel-packages/spatie__laravel-markdown-response.md... -``` +```bash /skill-research https://github.com/microsoft/skills/blob/main/.github/skills/copilot-sdkThe agent will research the skill and write a note to:
-
+text
.steering/skills/microsoft__copilot-sdk.mdAlso applies to: 134-142
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 39 - 47, Update the README.md code blocks to include explicit language identifiers for proper rendering: change the block containing "/laravel-package spatie/laravel-markdown-response" to use ```bash, the block containing ".steering/laravel-packages/spatie__laravel-markdown-response.md" to use ```text, the "/skill-research https://github.com/microsoft/skills/blob/main/.github/skills/copilot-sdk" block to use ```bash, and the ".steering/skills/microsoft__copilot-sdk.md" block to use ```text; ensure the same fix is applied to the other occurrences noted (around the 134-142 region) so every fenced code block has an appropriate language tag..github/skills/skill-research/SKILL.md (1)
9-63: Specify language for the template code block.The template code block should specify
markdownas the language identifier for better rendering and syntax highlighting.📝 Proposed fix
-``` +```markdown --- title: SKILL_NAME🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/skills/skill-research/SKILL.md around lines 9 - 63, The top template code block in SKILL.md is missing a language identifier; update the opening fence of the big template block (the triple-backtick that starts the YAML/Markdown template) to use ```markdown so the snippet is rendered/highlighted correctly in previews and editors; locate the template block in SKILL.md and add the language identifier to the opening fence (keep the existing closing ```).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/agents/laravel-package.agent.md:
- Around line 6-8: The deepwiki SSE server configuration named "deepwiki" is
missing authentication headers; update the deepwiki entry to include headers
with Authorization set to "Bearer $DEEPWIKI_KEY" (matching README and SKILL.md)
so the SSE client sends the required DEEPWIKI_KEY secret; ensure the headers key
is added alongside type and url so requests include the Authorization bearer
token.
In @.github/skills/agentic-programming/SKILL.md:
- Around line 15-18: The DeepWiki MCP server configuration is missing the
required authorization headers; update the deepwiki server entry in
`.github/agents/laravel-package.agent.md` (the deepwiki section) to include a
headers block that sets Authorization to "Bearer $DEEPWIKI_KEY" (use the
DEEPWIKI_KEY secret referenced in SKILL.md) so the SSE-type service can
authenticate correctly.
In @.github/skills/laravel-research/laravel-package-template.md:
- Line 27: Replace the malformed Markdown link "[GitHub]((github_url))" with a
valid placeholder link; locate the exact text "[GitHub]((github_url))" and
change it to "[GitHub]({{github_url}})" so the link renders correctly (use the
same placeholder token style used elsewhere in the template).
- Around line 15-28: The template
.github/skills/laravel-research/laravel-package-template.md is missing required
placeholders from the SKILL.md contract (e.g., image, description, and feature
placeholders); update the file to include explicit placeholders for an image
(e.g., IMAGE_URL), a short description (DESCRIPTION), and the Key Features
section populated with FEATURE_1 / FEATURE_2 placeholders and any required
resource placeholders so outputs conform to the "EXACT template" spec and match
the SKILL.md expectations.
In @.github/skills/laravel-research/SKILL.md:
- Line 9: In SKILL.md update the unlabeled fenced code blocks at the places
noted (the blocks starting at the current lines around 9 and 47) to specify a
language for the triple-backtick fences (for example change ``` to ```md or
```text) so the markdown linter MD040 is satisfied; locate the unnamed fences in
the file (the two triple-backtick blocks referenced) and add the appropriate
language tag to each opening fence.
In @.github/skills/skill-research/skill-template.md:
- Line 31: Replace the malformed Markdown link "[GitHub]((github_url))" with a
valid link placeholder by removing the extra parentheses so it reads
"[GitHub]({{github_url}})" (or the actual URL), ensuring the link target is
inside a single pair of parentheses; update the Markdown in the skill template
where the text "[GitHub]((github_url))" appears.
In @.github/skills/skill-research/SKILL.md:
- Line 61: Replace the malformed Markdown link "[GitHub]((github_url))" in
SKILL.md by removing the extra opening parenthesis so the link uses standard
Markdown syntax (e.g., "[GitHub](github_url)" or with the actual URL/variable
you intend to use); locate the string "[GitHub]((github_url))" and update it to
a single-pair parenthesis link format.
In @.github/workflows/copilot-setup-steps.yml:
- Line 13: The workflow step currently installs the wrong global package name
("@context7/cli") which doesn't match the agent's configured MCP server package;
update the npm install command to install "@context7/mcp-server" instead so the
agent can resolve the MCP server at runtime (replace the "@context7/cli" token
in the workflow step with "@context7/mcp-server").
In `@README.md`:
- Around line 66-87: The README shows an Authorization header for
mcpServers.deepwiki (mcpServers.deepwiki.headers.Authorization) but the agent
configs (.github/agents/laravel-package.agent.md and
.github/agents/skill-research.agent.md) don’t include that auth, causing a
mismatch; fix by updating the two agent .agent.md files to include the same
DeepWiki authentication snippet (reference DEEPWIKI_KEY env var and set the
Authorization: Bearer $DEEPWIKI_KEY header under their mcp server block),
ensuring the agents’ mcp server entries match the README, or alternatively
update the README to remove or note the missing auth if you prefer not to change
the agents (preferred: update the agent files to include the header).
In `@templates/agents.md`:
- Line 15: Replace unlabeled fenced code blocks (the raw "```" markers) with
labeled fences including a language identifier (for example "```md") in the
templates/agents.md occurrences flagged (the standalone "```" blocks around the
examples). Update each unlabeled fence (currently just "```") to "```md" so
editors and linters recognize the markdown content and MD040 is resolved.
---
Duplicate comments:
In @.github/agents/mcp-research.agent.md:
- Around line 6-8: The DeepWiki agent definition (the deepwiki mapping with keys
type and url) is missing authentication configuration; update the deepwiki entry
to include an authentication block matching the other agent definitions (e.g.,
add an authentication/auth or headers object that specifies the auth type—such
as bearer/token or apiKey—and where to read the credential from, then include
the header/key name to send with requests) so SSE connections to
https://api.deepwiki.com/sse are authenticated the same way as the other agents.
In @.github/agents/skill-research.agent.md:
- Around line 6-8: The DeepWiki SSE config block named "deepwiki" is missing
authentication and must pass the DEEPWIKI_KEY secret to the server; update the
"deepwiki" entry (the SSE config with keys "type: sse" and "url:
https://api.deepwiki.com/sse") to include the auth header or token field that
sends process/env DEEPWIKI_KEY (e.g., an "headers" or "auth" property containing
Authorization: Bearer <DEEPWIKI_KEY> or a "token" field) so the SSE connection
uses the repository secret; mirror the same approach used to fix the
laravel-package agent config.
---
Nitpick comments:
In @.github/skills/agentic-programming/SKILL.md:
- Around line 30-38: Update the code blocks in SKILL.md that show the
"/laravel-package <vendor/package>" command and its example to include a
language identifier (bash) for proper rendering: locate the fenced code blocks
containing "/laravel-package <vendor/package>" and "/laravel-package
spatie/laravel-markdown-response" and change their opening fences to "```bash"
so both the command and the example are marked as bash code blocks.
In @.github/skills/skill-research/SKILL.md:
- Around line 9-63: The top template code block in SKILL.md is missing a
language identifier; update the opening fence of the big template block (the
triple-backtick that starts the YAML/Markdown template) to use ```markdown so
the snippet is rendered/highlighted correctly in previews and editors; locate
the template block in SKILL.md and add the language identifier to the opening
fence (keep the existing closing ```).
In `@README.md`:
- Around line 102-122: The README's fenced file-tree block is missing a language
identifier; update the code block that starts with ".github/" (the file tree
showing agents/, skills/, workflows/, .steering/) to include a language tag such
as ```text so the tree renders correctly and preserves spacing/characters when
displayed.
- Around line 39-47: Update the README.md code blocks to include explicit
language identifiers for proper rendering: change the block containing
"/laravel-package spatie/laravel-markdown-response" to use ```bash, the block
containing ".steering/laravel-packages/spatie__laravel-markdown-response.md" to
use ```text, the "/skill-research
https://github.com/microsoft/skills/blob/main/.github/skills/copilot-sdk" block
to use ```bash, and the ".steering/skills/microsoft__copilot-sdk.md" block to
use ```text; ensure the same fix is applied to the other occurrences noted
(around the 134-142 region) so every fenced code block has an appropriate
language tag.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4dfcbcd2-95cc-4928-a0b6-f53b919154f5
📒 Files selected for processing (51)
.cursor/gaps/.gitkeep.github/agents/laravel-package.agent.md.github/agents/mcp-research.agent.md.github/agents/skill-research.agent.md.github/skills/agentic-programming/SKILL.md.github/skills/laravel-research/SKILL.md.github/skills/laravel-research/laravel-package-template.md.github/skills/skill-research/SKILL.md.github/skills/skill-research/skill-template.md.github/workflows/copilot-setup-steps.yml.steering/laravel-packages/README.md.steering/skills/README.mdREADME.mdcommands/00-start.mdcommands/10-planning/11-discuss.mdcommands/10-planning/12-idea.mdcommands/10-planning/13-packages.mdcommands/10-planning/14-prd.mdcommands/10-planning/15-user-stories.mdcommands/10-planning/16-mermaid.mdcommands/10-planning/17-create.mdcommands/20-roles/21-devops.mdcommands/20-roles/22-architect.mdcommands/20-roles/23-backend.mdcommands/20-roles/24-frontend.mdcommands/20-roles/25-test.mdcommands/20-roles/26-lead.mdcommands/30-process/31-expand.mdcommands/40-dev/41-open.mdcommands/40-dev/42-code.mdcommands/40-dev/43-verify.mdcommands/40-dev/44-close.mdcommands/50-final/51-docs.mdcommands/50-final/52-qa.mdcommands/50-final/53-done.mdcommands/50-final/54-pr.mdcommands/60-git/61-review.mdcommands/60-git/62-address.mdcommands/60-git/63-verify.mdcommands/60-git/64-commit.mdcommands/80-research/81-research.mdcommands/80-research/82-gap.mdcommands/99-rebake.mdmcp.jsonmcps/README.mdrules/epics/.gitkeepsupport/01-forbidden.mdsupport/02-mcp.mdsupport/03-pipeline.mdsupport/04-rules.mdtemplates/agents.md
💤 Files with no reviewable changes (36)
- support/04-rules.md
- .cursor/gaps/.gitkeep
- commands/60-git/62-address.md
- commands/50-final/51-docs.md
- commands/10-planning/16-mermaid.md
- support/03-pipeline.md
- commands/40-dev/43-verify.md
- commands/40-dev/42-code.md
- commands/10-planning/14-prd.md
- commands/50-final/54-pr.md
- commands/10-planning/15-user-stories.md
- commands/20-roles/22-architect.md
- commands/10-planning/17-create.md
- mcp.json
- commands/20-roles/21-devops.md
- commands/00-start.md
- commands/99-rebake.md
- commands/80-research/82-gap.md
- commands/60-git/61-review.md
- commands/20-roles/25-test.md
- commands/20-roles/26-lead.md
- commands/40-dev/41-open.md
- commands/20-roles/23-backend.md
- commands/30-process/31-expand.md
- commands/10-planning/11-discuss.md
- commands/20-roles/24-frontend.md
- commands/40-dev/44-close.md
- commands/10-planning/12-idea.md
- support/01-forbidden.md
- commands/60-git/64-commit.md
- commands/60-git/63-verify.md
- commands/10-planning/13-packages.md
- commands/50-final/52-qa.md
- support/02-mcp.md
- commands/80-research/81-research.md
- commands/50-final/53-done.md
| ## Usage | ||
|
|
||
| ## Resources | ||
| - [GitHub]((github_url)) |
There was a problem hiding this comment.
Fix malformed Markdown link placeholder
Line 27 uses [GitHub]((github_url)), which renders incorrectly.
Proposed fix
-- [GitHub]((github_url))
+- [GitHub](github_url)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - [GitHub]((github_url)) | |
| - [GitHub](github_url) |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.github/skills/laravel-research/laravel-package-template.md at line 27,
Replace the malformed Markdown link "[GitHub]((github_url))" with a valid
placeholder link; locate the exact text "[GitHub]((github_url))" and change it
to "[GitHub]({{github_url}})" so the link renders correctly (use the same
placeholder token style used elsewhere in the template).
|
|
||
| ## Resources | ||
|
|
||
| - [GitHub]((github_url)) |
There was a problem hiding this comment.
Fix malformed Markdown link placeholder
Line 31 has invalid link syntax: [GitHub]((github_url)).
Proposed fix
-- [GitHub]((github_url))
+- [GitHub](github_url)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - [GitHub]((github_url)) | |
| - [GitHub](github_url) |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.github/skills/skill-research/skill-template.md at line 31, Replace the
malformed Markdown link "[GitHub]((github_url))" with a valid link placeholder
by removing the extra parentheses so it reads "[GitHub]({{github_url}})" (or the
actual URL), ensuring the link target is inside a single pair of parentheses;
update the Markdown in the skill template where the text
"[GitHub]((github_url))" appears.
|
|
||
| ## Resources | ||
|
|
||
| - [GitHub]((github_url)) |
There was a problem hiding this comment.
Fix malformed link syntax.
The link has double opening parentheses ((github_url)) instead of the correct syntax.
🔗 Proposed fix
-- [GitHub]((github_url))
+- [GitHub](github_url)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - [GitHub]((github_url)) | |
| - [GitHub](github_url) |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.github/skills/skill-research/SKILL.md at line 61, Replace the malformed
Markdown link "[GitHub]((github_url))" in SKILL.md by removing the extra opening
parenthesis so the link uses standard Markdown syntax (e.g.,
"[GitHub](github_url)" or with the actual URL/variable you intend to use);
locate the string "[GitHub]((github_url))" and update it to a single-pair
parenthesis link format.
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - run: mkdir -p .steering/laravel-packages | ||
| - run: npm install -g @context7/cli |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify package/command alignment across workflow + agent config.
rg -n -C2 '@context7/(cli|mcp-server)|CONTEXT7' .github/workflows .github/agents .github/skillsRepository: ChrisThompsonTLDR/agentic-programming
Length of output: 1376
Install the MCP server package that the agent actually invokes
Line 13 installs @context7/cli, but the agent configuration at .github/agents/laravel-package.agent.md uses @context7/mcp-server. This mismatch causes runtime resolution failure when the agent tries to invoke the MCP server.
Proposed fix
- - run: npm install -g `@context7/cli`
+ - run: npm install -g `@context7/mcp-server`📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - run: npm install -g @context7/cli | |
| - run: npm install -g `@context7/mcp-server` |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.github/workflows/copilot-setup-steps.yml at line 13, The workflow step
currently installs the wrong global package name ("@context7/cli") which doesn't
match the agent's configured MCP server package; update the npm install command
to install "@context7/mcp-server" instead so the agent can resolve the MCP
server at runtime (replace the "@context7/cli" token in the workflow step with
"@context7/mcp-server").
There was a problem hiding this comment.
Actionable comments posted: 3
♻️ Duplicate comments (7)
.github/skills/skill-research/skill-template.md (1)
31-31:⚠️ Potential issue | 🟡 MinorFix malformed Markdown link placeholder.
The link has double opening parentheses:
[GitHub]((github_url))instead of the correct[GitHub](github_url).🔗 Proposed fix
-- [GitHub]((github_url)) +- [GitHub](github_url)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/skills/skill-research/skill-template.md at line 31, Replace the malformed Markdown link `[GitHub]((github_url))` with the correct single-parenthesis form `[GitHub](github_url)` in the skill template; locate the string `[GitHub]((github_url))` and update it to `[GitHub](github_url)` so the link renders properly..github/skills/skill-research/SKILL.md (1)
61-61:⚠️ Potential issue | 🟡 MinorFix malformed Markdown link placeholder.
The link has double opening parentheses:
[GitHub]((github_url))instead of the correct[GitHub](github_url).🔗 Proposed fix
-- [GitHub]((github_url)) +- [GitHub](github_url)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/skills/skill-research/SKILL.md at line 61, Replace the malformed Markdown link `[GitHub]((github_url))` with the correct syntax `[GitHub](github_url)` in SKILL.md; locate the line containing the double opening parentheses and update it so the link uses a single pair of parentheses..github/skills/laravel-research/laravel-package-template.md (1)
27-27:⚠️ Potential issue | 🟡 MinorFix malformed Markdown link placeholder.
The link has double opening parentheses:
[GitHub]((github_url))instead of the correct[GitHub](github_url).🔗 Proposed fix
-- [GitHub]((github_url)) +- [GitHub](github_url)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/skills/laravel-research/laravel-package-template.md at line 27, The Markdown link placeholder "[GitHub]((github_url))" is malformed with a double opening parenthesis; update the link to use the correct Markdown syntax by replacing "[GitHub]((github_url))" with "[GitHub](github_url)" so the link renders properly (locate and edit the line containing the "[GitHub]" link in the document)..github/agents/laravel-package.agent.md (1)
6-8:⚠️ Potential issue | 🔴 CriticalAdd DeepWiki Authorization header to avoid unauthenticated MCP calls.
Line 6-8 configures DeepWiki SSE without auth headers. This will likely fail when the server requires
DEEPWIKI_KEY.#!/bin/bash # Verify DeepWiki auth headers are present in agent configs and compare docs. rg -n -C2 '^\s*deepwiki:\s*$|Authorization|DEEPWIKI_KEY' .github/agents/*.agent.md README.md .github/skills/agentic-programming/SKILL.md🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/agents/laravel-package.agent.md around lines 6 - 8, The DeepWiki SSE agent config currently lacks auth headers; update the deepwiki block (the entry with keys "deepwiki", "type: sse", and "url: https://api.deepwiki.com/sse") to include an Authorization header that uses the DEEPWIKI_KEY (e.g., set a headers or auth subkey with "Authorization: Bearer ${DEEPWIKI_KEY}") so requests are authenticated; ensure the config key name matches other agents' header patterns and that DEEPWIKI_KEY is referenced consistently in docs and examples..github/agents/skill-research.agent.md (1)
6-8:⚠️ Potential issue | 🔴 CriticalAdd DeepWiki auth headers in this agent config as well.
Line 6-8 configures DeepWiki without
Authorization: Bearer $DEEPWIKI_KEY, so DeepWiki-backed research can fail due to authentication.#!/bin/bash # Confirm both agents consistently configure DeepWiki auth. rg -n -C3 '^\s*deepwiki:\s*$|Authorization|DEEPWIKI_KEY' .github/agents/laravel-package.agent.md .github/agents/skill-research.agent.md🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/agents/skill-research.agent.md around lines 6 - 8, The deepwiki agent block currently lacks authentication; update the deepwiki YAML block (the "deepwiki" entry with type: sse and url) to include an HTTP headers mapping that sets Authorization: "Bearer $DEEPWIKI_KEY" (matching the other agent's format) so the agent sends the DEEPWIKI_KEY env var as the Bearer token; ensure the headers key name and string quoting follow the same style as the existing agent config (so the SSE client receives Authorization: Bearer $DEEPWIKI_KEY)..github/skills/laravel-research/SKILL.md (1)
9-47:⚠️ Potential issue | 🟠 MajorFix fenced-template nesting so the markdown renders correctly.
Line 9 starts a triple-backtick block, but the inner install snippet at Line 38 also uses triple backticks, which can prematurely terminate the outer template block in markdown renderers. Use a 4-backtick outer fence.
Suggested fix
-``` +````md --- title: PACKAGE_NAME org: VENDOR package: package-name github_url: https://github.com/VENDOR/package-name docs_url: composer_require: composer require VENDOR/package-name author: announce_date: latest_release: vX.Y.Z (DATE) release_date: laravel_news_url: downloads_30d: stars: tags: [laravel, packages, FEATURE1, FEATURE2] --- # PACKAGE_NAME <img> **DESCRIPTION** ## Key Features - FEATURE1 - FEATURE2 ## Installation ```bash composer_requireUsage
CODE_EXAMPLE
Resources
- [[Laravel Packages]] (related)
-```
+````</details> <details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against the current code and only fix it if needed.
In @.github/skills/laravel-research/SKILL.md around lines 9 - 47, The outer
fenced code block that begins with triple backticks around the frontmatter and
content conflicts with the inner "bash" install snippet and breaks rendering; update the outer fence to a 4-backtick fence (e.g., "````" with optional "md") and replace its closing triple-backticks with the matching 4-backtick closing fence so the innerbash composer_require snippet stays nested correctly
(locate the outer fence surrounding the frontmatter/README-like block and the
inner "```bash" install snippet to apply the change).</details> </blockquote></details> <details> <summary>templates/agents.md (1)</summary><blockquote> `15-15`: _⚠️ Potential issue_ | _🟡 Minor_ **Add language identifiers to fenced markdown blocks.** These fences are unlabeled and will keep triggering MD040. Label them (for example `md`, `text`, or `bash` as appropriate). Also applies to: 80-80, 94-94, 190-190 <details> <summary>🤖 Prompt for AI Agents</summary> ```` Verify each finding against the current code and only fix it if needed. In `@templates/agents.md` at line 15, Several fenced code blocks in templates/agents.md are unlabeled and trigger MD040; update each plain triple-backtick fence so it includes an appropriate language identifier (e.g., ```md for markdown snippets, ```bash for shell commands, ```text for plain text) to satisfy the linter. Find the unlabeled fences around the examples and headings (the initial top block and the other occurrences mentioned) and replace the opening ``` with the correct ```md/```bash/```text tag based on the block content, leaving the closing ``` unchanged. Ensure all similar unlabeled fences in the file are updated consistently so MD040 no longer fires. ```` </details> </blockquote></details> </blockquote></details> <details> <summary>🧹 Nitpick comments (1)</summary><blockquote> <details> <summary>.github/skills/skill-research/SKILL.md (1)</summary><blockquote> `9-9`: **Consider adding language specifier to fenced code block.** The code block on line 9 contains a Markdown template but lacks a language specifier. Adding `markdown` after the opening backticks would improve syntax highlighting and satisfy linting tools. <details> <summary>📝 Proposed enhancement</summary> ```diff -``` +```markdown --- title: SKILL_NAME ``` </details> As per static analysis hints, markdownlint-cli2 recommends specifying a language for fenced code blocks (MD040). <details> <summary>🤖 Prompt for AI Agents</summary> ```` Verify each finding against the current code and only fix it if needed. In @.github/skills/skill-research/SKILL.md at line 9, The fenced code block in SKILL.md is missing a language specifier; update the opening backticks for the template block (the line with "```") to include "markdown" (i.e., change the fence to "```markdown") so the Markdown template (the front-matter starting with "---" and "title: SKILL_NAME") gets proper syntax highlighting and satisfies MD040 linting; locate the block in SKILL.md where the template begins and modify that fence only. ```` </details> </blockquote></details> </blockquote></details> <details> <summary>🤖 Prompt for all review comments with AI agents</summary>Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/copilot-setup-steps.yml:
- Line 12: Update the workflow step that currently creates only
.steering/laravel-packagesto create all required agent output directories so
agents won't fail: add creation of.steering/skillsandmcpsalongside the
existing.steering/laravel-packagesin the same step (the run command that
currently containsmkdir -p .steering/laravel-packages) so the
skill-researchagent can write to.steering/skillsand themcp-research
agent can write tomcps.In
@README.md:
- Around line 7-10: The Agents table in README.md is missing the new agent entry
for mcp-research; add a row to the table matching the style of existing entries
with Agentmcp-research, Trigger/mcp-research <github-url-to-mcp>(or the
exact CLI trigger used in the PR), and Output
.steering/mcps/<owner>__<name>.mdso the documentation reflects the introduced
mcps/__.md contract and usage; update the table between the
existinglaravel-packageandskill-researchrows (or append consistently)
ensuring the agent name, trigger, and output cell contents follow the same
backtick formatting.In
@templates/agents.md:
- Line 228: Update the stale MCP configuration reference that currently points
to.vscode/mcp.jsonin the sentence "Pre-configured Model Context Protocol
servers in.vscode/mcp.jsonprovide additional capabilities:" — replace that
path with the current documented location for GitHub Copilot Cloud agent configs
(the.github/agents/*.agent.mdpattern) and ensure the surrounding copy
mentions the new config source and filename pattern so readers are directed to
the correct, current agent configuration files.
Duplicate comments:
In @.github/agents/laravel-package.agent.md:
- Around line 6-8: The DeepWiki SSE agent config currently lacks auth headers;
update the deepwiki block (the entry with keys "deepwiki", "type: sse", and
"url: https://api.deepwiki.com/sse") to include an Authorization header that
uses the DEEPWIKI_KEY (e.g., set a headers or auth subkey with "Authorization:
Bearer ${DEEPWIKI_KEY}") so requests are authenticated; ensure the config key
name matches other agents' header patterns and that DEEPWIKI_KEY is referenced
consistently in docs and examples.In @.github/agents/skill-research.agent.md:
- Around line 6-8: The deepwiki agent block currently lacks authentication;
update the deepwiki YAML block (the "deepwiki" entry with type: sse and url) to
include an HTTP headers mapping that sets Authorization: "Bearer $DEEPWIKI_KEY"
(matching the other agent's format) so the agent sends the DEEPWIKI_KEY env var
as the Bearer token; ensure the headers key name and string quoting follow the
same style as the existing agent config (so the SSE client receives
Authorization: Bearer $DEEPWIKI_KEY).In @.github/skills/laravel-research/laravel-package-template.md:
- Line 27: The Markdown link placeholder "GitHub" is malformed
with a double opening parenthesis; update the link to use the correct Markdown
syntax by replacing "GitHub" with "GitHub" so the
link renders properly (locate and edit the line containing the "[GitHub]" link
in the document).In @.github/skills/laravel-research/SKILL.md:
- Around line 9-47: The outer fenced code block that begins with triple
backticks around the frontmatter and content conflicts with the inner "bash" install snippet and breaks rendering; update the outer fence to a 4-backtick fence (e.g., "````" with optional "md") and replace its closing triple-backticks with the matching 4-backtick closing fence so the innerbash composer_require
snippet stays nested correctly (locate the outer fence surrounding the
frontmatter/README-like block and the inner "```bash" install snippet to apply
the change).In @.github/skills/skill-research/skill-template.md:
- Line 31: Replace the malformed Markdown link
[GitHub]((github_url))with the
correct single-parenthesis form[GitHub](github_url)in the skill template;
locate the string[GitHub]((github_url))and update it to
[GitHub](github_url)so the link renders properly.In @.github/skills/skill-research/SKILL.md:
- Line 61: Replace the malformed Markdown link
[GitHub]((github_url))with the
correct syntax[GitHub](github_url)in SKILL.md; locate the line containing
the double opening parentheses and update it so the link uses a single pair of
parentheses.In
@templates/agents.md:
- Line 15: Several fenced code blocks in templates/agents.md are unlabeled and
trigger MD040; update each plain triple-backtick fence so it includes an
appropriate language identifier (e.g.,md for markdown snippets,bash for
shell commands,text for plain text) to satisfy the linter. Find the unlabeled fences around the examples and headings (the initial top block and the other occurrences mentioned) and replace the openingwith the correct
md/bash/text tag based on the block content, leaving the closing
unchanged. Ensure all similar unlabeled fences in the file are updated
consistently so MD040 no longer fires.
Nitpick comments:
In @.github/skills/skill-research/SKILL.md:
- Line 9: The fenced code block in SKILL.md is missing a language specifier;
update the opening backticks for the template block (the line with "") to include "markdown" (i.e., change the fence to "markdown") so the Markdown
template (the front-matter starting with "---" and "title: SKILL_NAME") gets
proper syntax highlighting and satisfies MD040 linting; locate the block in
SKILL.md where the template begins and modify that fence only.</details> <details> <summary>🪄 Autofix (Beta)</summary> Fix all unresolved CodeRabbit comments on this PR: - [ ] <!-- {"checkboxId": "4b0d0e0a-96d7-4f10-b296-3a18ea78f0b9"} --> Push a commit to this branch (recommended) - [ ] <!-- {"checkboxId": "ff5b1114-7d8c-49e6-8ac1-43f82af23a33"} --> Create a new PR with the fixes </details> --- <details> <summary>ℹ️ Review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: Organization UI **Review profile**: CHILL **Plan**: Pro **Run ID**: `87cd5b10-fc48-4e24-af1f-d0d2d1fd0d50` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between e08605fbff4a3ec0d0d558b00d9f845415063d65 and 4e553d8853ff9c4e7c88036d831800acc866ebe3. </details> <details> <summary>📒 Files selected for processing (51)</summary> * `.cursor/gaps/.gitkeep` * `.github/agents/laravel-package.agent.md` * `.github/agents/mcp-research.agent.md` * `.github/agents/skill-research.agent.md` * `.github/skills/agentic-programming/SKILL.md` * `.github/skills/laravel-research/SKILL.md` * `.github/skills/laravel-research/laravel-package-template.md` * `.github/skills/skill-research/SKILL.md` * `.github/skills/skill-research/skill-template.md` * `.github/workflows/copilot-setup-steps.yml` * `.steering/laravel-packages/README.md` * `.steering/skills/README.md` * `README.md` * `commands/00-start.md` * `commands/10-planning/11-discuss.md` * `commands/10-planning/12-idea.md` * `commands/10-planning/13-packages.md` * `commands/10-planning/14-prd.md` * `commands/10-planning/15-user-stories.md` * `commands/10-planning/16-mermaid.md` * `commands/10-planning/17-create.md` * `commands/20-roles/21-devops.md` * `commands/20-roles/22-architect.md` * `commands/20-roles/23-backend.md` * `commands/20-roles/24-frontend.md` * `commands/20-roles/25-test.md` * `commands/20-roles/26-lead.md` * `commands/30-process/31-expand.md` * `commands/40-dev/41-open.md` * `commands/40-dev/42-code.md` * `commands/40-dev/43-verify.md` * `commands/40-dev/44-close.md` * `commands/50-final/51-docs.md` * `commands/50-final/52-qa.md` * `commands/50-final/53-done.md` * `commands/50-final/54-pr.md` * `commands/60-git/61-review.md` * `commands/60-git/62-address.md` * `commands/60-git/63-verify.md` * `commands/60-git/64-commit.md` * `commands/80-research/81-research.md` * `commands/80-research/82-gap.md` * `commands/99-rebake.md` * `mcp.json` * `mcps/README.md` * `rules/epics/.gitkeep` * `support/01-forbidden.md` * `support/02-mcp.md` * `support/03-pipeline.md` * `support/04-rules.md` * `templates/agents.md` </details> <details> <summary>💤 Files with no reviewable changes (36)</summary> * support/02-mcp.md * .cursor/gaps/.gitkeep * commands/50-final/51-docs.md * commands/60-git/62-address.md * commands/80-research/82-gap.md * support/01-forbidden.md * commands/00-start.md * support/03-pipeline.md * mcp.json * commands/40-dev/42-code.md * commands/60-git/64-commit.md * commands/20-roles/26-lead.md * commands/20-roles/22-architect.md * commands/10-planning/12-idea.md * commands/20-roles/21-devops.md * commands/99-rebake.md * commands/40-dev/43-verify.md * commands/10-planning/17-create.md * commands/10-planning/15-user-stories.md * commands/80-research/81-research.md * commands/10-planning/16-mermaid.md * commands/30-process/31-expand.md * commands/60-git/63-verify.md * commands/50-final/53-done.md * commands/50-final/52-qa.md * commands/10-planning/11-discuss.md * commands/40-dev/41-open.md * commands/20-roles/24-frontend.md * commands/40-dev/44-close.md * commands/20-roles/23-backend.md * commands/10-planning/13-packages.md * commands/10-planning/14-prd.md * commands/60-git/61-review.md * commands/50-final/54-pr.md * support/04-rules.md * commands/20-roles/25-test.md </details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
Comment on lines
+7
to
+10
| | Agent | Trigger | Output | | ||
| |-------|---------|--------| | ||
| | `laravel-package` | `/laravel-package <vendor/package>` | `.steering/laravel-packages/<vendor>__<package>.md` | | ||
| | `skill-research` | `/skill-research <github-url-to-skill>` | `.steering/skills/<owner>__<skill-name>.md` | |
There was a problem hiding this comment.
Document the new mcp-research agent in the Agents table.
Lines 7-10 omit mcp-research, even though this PR introduces it and its mcps/<owner>__<name>.md output contract. This makes the primary usage docs incomplete.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 7 - 10, The Agents table in README.md is missing the
new agent entry for mcp-research; add a row to the table matching the style of
existing entries with Agent `mcp-research`, Trigger `/mcp-research
<github-url-to-mcp>` (or the exact CLI trigger used in the PR), and Output
`.steering/mcps/<owner>__<name>.md` so the documentation reflects the introduced
mcps/<owner>__<name>.md contract and usage; update the table between the
existing `laravel-package` and `skill-research` rows (or append consistently)
ensuring the agent name, trigger, and output cell contents follow the same
backtick formatting.
|
|
||
| ## MCP Servers | ||
|
|
||
| Pre-configured Model Context Protocol servers in `.vscode/mcp.json` provide additional capabilities: |
There was a problem hiding this comment.
Update stale MCP configuration path/reference.
Line 228 points readers to .vscode/mcp.json as the preconfigured source, but this repo now documents/uses GitHub Copilot Cloud agent configs under .github/agents/*.agent.md. Keeping this path can send users to non-existent or outdated setup steps.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@templates/agents.md` at line 228, Update the stale MCP configuration
reference that currently points to `.vscode/mcp.json` in the sentence
"Pre-configured Model Context Protocol servers in `.vscode/mcp.json` provide
additional capabilities:" — replace that path with the current documented
location for GitHub Copilot Cloud agent configs (the `.github/agents/*.agent.md`
pattern) and ensure the surrounding copy mentions the new config source and
filename pattern so readers are directed to the correct, current agent
configuration files.
Owner
|
@copilot Address all the PR feedback |
- Add DeepWiki auth headers (Bearer $DEEPWIKI_KEY) to all 3 agent files - Add .steering/skills and mcps dirs to copilot-setup-steps.yml - Add mcp-research row to README Agents table - Fix malformed [GitHub]((github_url)) links in 3 template/skill files - Add markdown language label to laravel-research/SKILL.md fenced block - Add text language labels to 4 unlabeled blocks in templates/agents.md - Align laravel-package-template.md placeholders with SKILL.md Agent-Logs-Url: https://github.com/ChrisThompsonTLDR/agentic-programming/sessions/a33672db-80c6-4c28-9f2b-35eea8932e42 Co-authored-by: ChrisThompsonTLDR <348801+ChrisThompsonTLDR@users.noreply.github.com>
Contributor
Author
Addressed all CodeRabbit feedback in 8536b43:
|
Owner
|
@coderabbitai full review |
✅ Actions performedFull review triggered. |
Owner
|
@BugBot run |
Owner
|
@BugBot run |
Owner
|
@cursor-bot review |
1 similar comment
Owner
|
@cursor-bot review |
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.
Adds three related improvements to the agent ecosystem:
templates/agents.mdVerbatim copy of
microsoft/skillsAgents.md — canonical reference for agent authoring conventions (core principles, clean architecture, do's/don'ts).Core principles applied retroactively to existing agents
Both
laravel-packageandskill-researchagents now open with a Core Principles block derived from Agents.md:New:
mcp-researchagentInvoke with
/mcp-research <url>. Accepts GitHub repos, npm, PyPI, or any MCP source URL.owner+namefrom the URL to derive the output pathmcps/<owner>__<name>.mdcovering transport type, tools/resources/prompts table, install snippet, config snippet, and env variablesmcps/README.mddocuments the directory and naming conventionSummary by CodeRabbit
New Features
Documentation
Chores
Removed