Merged
Conversation
239c09a to
8d8fa72
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Adds NPM package dependency analysis and visualizations to the reporting pipeline, extends longest-path GraphViz output with additional TypeScript project context, and improves Neo4j write stability via transaction batching.
Changes:
- Add NPM package projections and reporting (topological sort, centrality, path finding, and GraphViz visualizations).
- Enhance longest-path GraphViz output to include optional project info + level info (and ensure topological-sort properties exist before rendering).
- Add
IN TRANSACTIONS OF <n> ROWSbatching to multiple write-heavy Cypher scripts and prevent self-loops inCHANGED_TOGETHER_WITH.
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/reports/TopologicalSortCsv.sh | Adds NPM package topological sort report generation. |
| scripts/reports/PathFindingVisualization.sh | Adds NPM longest-path visualizations and triggers topological-sort property computation when needed. |
| scripts/reports/PathFindingCsv.sh | Adds NPM package path-finding algorithms run. |
| scripts/reports/InternalDependenciesVisualization.sh | Adds NPM package build-level GraphViz visualization run. |
| scripts/reports/CentralityCsv.sh | Adds NPM package directed/undirected centrality reports. |
| scripts/prepareAnalysis.sh | Extends TypeScript enrichment pipeline with NPM package-to-package dependency enrichment and counts. |
| cypher/Typescript_Enrichment/Link_npm_packages_with_depends_on_relationships.cypher | New: creates package-to-package DEPENDS_ON relationships with a computed weight. |
| cypher/Typescript_Enrichment/Link_npm_dependencies_to_npm_packages.cypher | Batches MERGE and filters template-like package names. |
| cypher/Typescript_Enrichment/Link_external_modules_to_corresponding_npm_dependency.cypher | Adds transactional batching for relationship creation. |
| cypher/Typescript_Enrichment/Enrich_npm_packages_with_dependency_counts.cypher | New: computes incoming/outgoing dependency counts on NPM:Package nodes. |
| cypher/Typescript_Enrichment/Add_IS_IMPLEMENTED_IN_relationship_for_matching_modules.cypher | Adds transactional batching. |
| cypher/Typescript_Enrichment/Add_IS_IMPLEMENTED_IN_relationship_for_matching_declarations.cypher | Adds transactional batching. |
| cypher/Typescript_Enrichment/Add_DEPENDS_ON_relationship_to_resolved_modules.cypher | Adds transactional batching. |
| cypher/Topological_Sort/Topological_Sort_Exists.cypher | New: check query used to decide whether to run topological sort write. |
| cypher/Path_Finding/Set_Parameters_Typescript_Module.cypher | New: example parameter set for TS module path finding. |
| cypher/Path_Finding/Set_Parameters_NPM.cypher | New: example parameter set for NPM package path finding. |
| cypher/Path_Finding/Path_Finding_6_Longest_paths_for_graphviz.cypher | Adds optional project + level info to node labels; documents topological-sort prerequisite. |
| cypher/Path_Finding/Path_Finding_6_Longest_paths_contributors_for_graphviz.cypher | Same visualization enrichment as above. |
| cypher/Internal_Dependencies/Typescript_Module_build_levels_for_graphviz.cypher | Refactors label construction for clearer level info formatting. |
| cypher/Internal_Dependencies/NPM_Package_build_levels_for_graphviz.cypher | New: GraphViz edges for NPM package build levels. |
| cypher/GitLog/Add_RESOLVES_TO_relationships_to_git_files_for_Typescript.cypher | Adds transactional batching. |
| cypher/GitLog/Add_RESOLVES_TO_relationships_to_git_files_for_Java.cypher | Adds transactional batching. |
| cypher/GitLog/Add_CHANGED_TOGETHER_WITH_relationships_to_git_files.cypher | Prevents self-loop edges when building file pairs. |
| cypher/GitLog/Add_CHANGED_TOGETHER_WITH_relationships_to_code_files.cypher | Prevents self-loop edges and adds transactional batching. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1bf3f1f to
be44681
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 24 out of 24 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
be44681 to
d32fbe6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 Feature
⚙️ Optimization
🛠 Fix