Skip to content

fix(ingestion): align CALLS edge sourceId with node ID format#194

Merged
abhigyanpatwari merged 1 commit intomainfrom
fix/process-detection-ids
Mar 6, 2026
Merged

fix(ingestion): align CALLS edge sourceId with node ID format#194
abhigyanpatwari merged 1 commit intomainfrom
fix/process-detection-ids

Conversation

@abhigyanpatwari
Copy link
Owner

Summary

  • findEnclosingFunctionId generated IDs as generateId(label, 'path:name') but node creation uses generateId(label, 'path:name:startLine')
  • This mismatch meant every CALLS edge had a sourceId pointing to a non-existent node
  • The process detector found 0 entry points with outgoing calls → 0 execution flows
  • Fix: include :startLine in findEnclosingFunctionId to match node ID format

Test plan

  • 749 unit tests pass
  • Re-run npx gitnexus analyze on a repo and verify flows > 0

🤖 Generated with Claude Code

findEnclosingFunctionId generated IDs without :startLine suffix,
but node creation includes it. This caused every CALLS edge to
reference a non-existent source node, making the process detector
find 0 entry points and produce 0 execution flows.

Bumps to 1.3.9.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gitnexus Building Building Preview, Comment Mar 6, 2026 8:27am

Request Review

@abhigyanpatwari abhigyanpatwari merged commit 3db47f7 into main Mar 6, 2026
7 of 8 checks passed
CrazyBunQnQ pushed a commit to CrazyBunQnQ/GitNexus that referenced this pull request Mar 8, 2026
…anpatwari#194)

findEnclosingFunctionId generated IDs without :startLine suffix,
but node creation includes it. This caused every CALLS edge to
reference a non-existent source node, making the process detector
find 0 entry points and produce 0 execution flows.

Bumps to 1.3.9.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant