Skip to content

fix(installer): include codegraph_files in Claude auto-allow (#565)#566

Open
ruanchaves wants to merge 1 commit into
colbymchenry:mainfrom
ruanchaves:fix/565-claude-codegraph-files
Open

fix(installer): include codegraph_files in Claude auto-allow (#565)#566
ruanchaves wants to merge 1 commit into
colbymchenry:mainfrom
ruanchaves:fix/565-claude-codegraph-files

Conversation

@ruanchaves
Copy link
Copy Markdown

Summary

Closes #565.

The Claude installer's auto-allow list omitted mcp__codegraph__codegraph_files, so codegraph install --target=claude ... wrote a settings.json that still prompted for codegraph_files even though the documented manual Claude example already includes it.

Change

  • add mcp__codegraph__codegraph_files to getCodeGraphPermissions() in src/installer/targets/shared.ts
  • add one installer regression assertion in __tests__/installer-targets.test.ts
  • add an [Unreleased] fix note to CHANGELOG.md

This PR is intentionally narrow. It does not expand the Claude allowlist to codegraph_explore or codegraph_trace; those can be discussed separately.

Validation

  • npx vitest run __tests__/installer-targets.test.ts
  • npm run build

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Claude installer auto-allow omits codegraph_files despite the README/manual Claude example including it

1 participant