Skip to content

fix: include symbol references in file dependents - #351

Closed
bohe76 wants to merge 1 commit into
colbymchenry:mainfrom
bohe76:fix-affected-semantic-dependents
Closed

bohe76 wants to merge 1 commit into
colbymchenry:mainfrom
bohe76:fix-affected-semantic-dependents

Conversation

@bohe76

@bohe76 bohe76 commented May 23, 2026

Copy link
Copy Markdown

Summary

  • include resolved symbol edges (calls, references, type/instantiation edges) when computing file dependencies and dependents
  • normalize changed file paths in codegraph affected so Windows-style paths match indexed file paths
  • add regression coverage for cross-file dependents and Windows path input

Validation

  • npm run build
  • npx vitest run __tests__/graph.test.ts --minWorkers=1 --maxWorkers=1 --testTimeout=30000 --hookTimeout=30000 --no-file-parallelism
  • manual codegraph affected checks against TypeScript fixtures returned expected test files
  • full serial npx vitest run ... mostly passed, with existing Windows EBUSY temp cleanup failures in mcp-roots and mcp-initialize

@colbymchenry

Copy link
Copy Markdown
Owner

Thanks for this, @bohe76, and sorry for the long wait! Both parts are on main and released: file dependents and dependencies follow the full symbol graph since #708 (1.0.0), and codegraph affected accepts back-slash, ./ and absolute paths since #865 (1.0.1). On your fixture, codegraph affected 'src\utils.ts' now lists src/utils.test.ts, so we're closing this as already fixed.

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.

2 participants