Add test intelligence graph edges#38
Conversation
Signed-off-by: Mir Sameer <mirsameer0304@gmail.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8295801934
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| for (const edge of dataFlowEdges) store.insertEdge(edge); | ||
| for (const edge of httpEdges) store.insertEdge(edge); | ||
| for (const edge of typeRelationEdges) store.insertEdge(edge); | ||
| for (const edge of testEdges) store.insertEdge(edge); |
There was a problem hiding this comment.
Delete stale TESTS edges during incremental rebuilds
Because TESTS is inserted here as a derived, whole-graph edge but store.deleteDerivedEdges() does not remove TESTS, incremental rebuilds can retain obsolete test coverage. For example, after an initial test links to a unique createOrder, adding another createOrder in a different unchanged endpoint file makes addTestEdges suppress the ambiguous edge, but the old row remains because neither the test nor original target file was deleted.
Useful? React with 👍 / 👎.
Summary
Validation