Skip to content

[pull] canary from vercel:canary#491

Merged
pull[bot] merged 1 commit intojavascript-for-kids:canaryfrom
vercel:canary
Dec 19, 2025
Merged

[pull] canary from vercel:canary#491
pull[bot] merged 1 commit intojavascript-for-kids:canaryfrom
vercel:canary

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Dec 19, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

…87312)

### What?

Simplifies the graph traversal.
Allows the GraphStore to store edges and nodes.
This no longer forces us to abuse the node type to store edge info, which actually has lead to a race condition problem which made the module graph non-deterministic.

Removes `skip_duplicated` and made the AdjacencyMap always skip duplicates. That simplifies the implementation. We never not skipped duplicates and that rarely makes sense anyway since any cycle would make it traverse forever...

Also removed a bunch of unused stuff...
@pull pull bot locked and limited conversation to collaborators Dec 19, 2025
@pull pull bot added the ⤵️ pull label Dec 19, 2025
@pull pull bot merged commit 92f7237 into javascript-for-kids:canary Dec 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant