Skip to content

fix(cli): include source for file-pinned nodes#1289

Open
Inference1 wants to merge 1 commit into
colbymchenry:mainfrom
Inference1:fix/node-file-filter-source-1284
Open

fix(cli): include source for file-pinned nodes#1289
Inference1 wants to merge 1 commit into
colbymchenry:mainfrom
Inference1:fix/node-file-filter-source-1284

Conversation

@Inference1

Copy link
Copy Markdown

Summary

  • enable source output when codegraph node <symbol> -f <file> disambiguates a symbol
  • preserve existing file-only and bare-symbol behavior
  • add end-to-end regression coverage with two same-named definitions

Root cause

The CLI passed symbol and file for a file-pinned lookup but omitted includeCode: true, so the MCP handler correctly selected the definition while rendering it without its body.

Testing

  • npm run build
  • full Vitest suite: 139 files passed, 2,375 tests passed, 4 skipped

Fixes #1284

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.

problem with showing node content in c++

1 participant