Skip to content

feat: add codebase-memory-mcp skill 🤖🤖🤖#2326

Open
astandrik wants to merge 2 commits into
github:mainfrom
astandrik:feat/codebase-memory-mcp-skill
Open

feat: add codebase-memory-mcp skill 🤖🤖🤖#2326
astandrik wants to merge 2 commits into
github:mainfrom
astandrik:feat/codebase-memory-mcp-skill

Conversation

@astandrik

Copy link
Copy Markdown

Summary

  • add a focused codebase-memory-mcp Agent Skill for graph-backed repository discovery
  • require source verification, freshness handling, bounded queries, and safe local fallbacks
  • keep the contribution portable by requiring no scripts, bundled binaries, or installation automation

Why this is distinct

acquire-codebase-knowledge produces a fixed documentation set for repository onboarding. This skill instead guides interactive symbol lookup, architecture orientation, dependency and data-flow tracing, and impact analysis through an already configured Codebase Memory MCP server.

Validation

  • npm run skill:validate
  • npm run build
  • bash eng/fix-line-endings.sh
  • git diff --check HEAD^ HEAD
  • fresh-context RED/GREEN scenarios for structural discovery, stale indexes, literal lookup, unavailable-MCP fallback, and negative triggers

@astandrik
astandrik requested a review from aaronpowell as a code owner July 16, 2026 20:05
Copilot AI review requested due to automatic review settings July 16, 2026 20:05
@github-actions github-actions Bot added new-submission PR adds at least one new contribution skills PR touches skills labels Jul 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 2 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 1
ℹ️ Info 0
Severity Rule File Line Match
🟠 package-exec-command docs/README.skills.md 31 | [acreadiness-assess](../skills/acreadiness-assess/SKILL.md)<br />`gh skills install github/awesome-copilot acreadiness-assess` | Run the AgentRC readiness assessment on the curre

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

✅ All checks passed

Scope Checked
Skills 1
Agents 0
Total 1
Severity Count
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ npm warn EBADENGINE Unsupported engine {
ℹ️ npm warn EBADENGINE package: 'commander@15.0.0',
ℹ️ npm warn EBADENGINE required: { node: '>=22.12.0' },
ℹ️ npm warn EBADENGINE current: { node: 'v20.20.2', npm: '10.8.2' }
ℹ️ npm warn EBADENGINE }
ℹ️ npm warn deprecated prebuild-install@7.1.3: No longer maintained. Please contact the author of the relevant native addon; alternatives are available.
ℹ️ ✅ codebase-memory-mcp (2/2 checks passed)
ℹ️ ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️ ✓ spec-compliance: All spec checks passed.
ℹ️ ✓ [valid-refs] All file references across 1 skill(s) are valid.
Full linter output
### Linting skills/codebase-memory-mcp
npm warn EBADENGINE Unsupported engine {
npm warn EBADENGINE   package: 'commander@15.0.0',
npm warn EBADENGINE   required: { node: '>=22.12.0' },
npm warn EBADENGINE   current: { node: 'v20.20.2', npm: '10.8.2' }
npm warn EBADENGINE }
npm warn deprecated prebuild-install@7.1.3: No longer maintained. Please contact the author of the relevant native addon; alternatives are available.
✅ codebase-memory-mcp (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a portable Agent Skill for safe, graph-backed codebase discovery through Codebase Memory MCP.

Changes:

  • Adds workflows for bounded graph queries, freshness checks, and source verification.
  • Documents safe local fallbacks and updates the generated skill catalog.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
skills/codebase-memory-mcp/SKILL.md Defines the new MCP discovery skill.
docs/README.skills.md Adds the skill to the catalog.

Comment thread skills/codebase-memory-mcp/SKILL.md Outdated
Copilot AI review requested due to automatic review settings July 16, 2026 20:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-submission PR adds at least one new contribution skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants