feat(dependency-graph): transitive closure and deep chain analysis — closes #111 #247

Rockachopa · 2026-04-26T09:08:44Z

Rockachopa commented

2026-04-26 09:08:44 +00:00

5.5: Transitive Dependency Analyzer

What

Enhances scripts/dependency_graph.py to meet all acceptance criteria:

Accepts:

✅ Builds transitive dep tree
✅ Identifies deep chains and circular deps
✅ Outputs transitive dependency graph

New Features

transitive_closure(graph) — Computes full transitive dependency tree for every node (all indirect dependencies). Excludes self-references.
find_deep_chains(graph) — DFS-based path finder that returns all longest simple paths in the dependency graph (ignoring cycles). Returns a list of node sequences.
JSON output now includes:
- transitive: {node: [sorted list of all deps]}
- deep_chains: [[node sequence...]]
- summary.longest_chain_length
- summary.transitive_pairs
Markdown renderers: format_transitive_markdown() and format_deep_chains_markdown() for human-readable reports.

Tests Added

New scripts/test_dependency_graph.py — 9 comprehensive test cases:

Simple chain A→B→C
Diamond A→{B,C}→D
Cycles (A→B→C→A) handling
Multiple longest chains
Empty graph edge case
Cycle detection via existing detect_cycles()
Longest chain length reporting

All tests pass locally:

$ python3 scripts/test_dependency_graph.py
✅ All dependency graph tests passed

Verification

The existing repo test suite still passes (89 tests in tests/):

$ pytest tests/ -v
============================== 89 passed =============================

Usage Example

# JSON output includes transitive closure + deep chains
python3 scripts/dependency_graph.py --repos-dir ~/repos/ --format json

Sample output:

{
  "repos": { ... },
  "cycles": [],
  "transitive": {
    "hermes-agent": ["timmy-config", "timmy-home", ...],
    ...
  },
  "deep_chains": [["a", "b", "c", "d"]],
  "summary": {
    "longest_chain_length": 4,
    "transitive_pairs": 142,
    ...
  }
}

Closes #111

## 5.5: Transitive Dependency Analyzer ### What Enhances `scripts/dependency_graph.py` to meet all acceptance criteria: **Accepts:** - ✅ Builds transitive dep tree - ✅ Identifies deep chains and circular deps - ✅ Outputs transitive dependency graph ### New Features 1. **`transitive_closure(graph)`** — Computes full transitive dependency tree for every node (all indirect dependencies). Excludes self-references. 2. **`find_deep_chains(graph)`** — DFS-based path finder that returns all longest simple paths in the dependency graph (ignoring cycles). Returns a list of node sequences. 3. **JSON output** now includes: - `transitive`: `{node: [sorted list of all deps]}` - `deep_chains`: `[[node sequence...]]` - `summary.longest_chain_length` - `summary.transitive_pairs` 4. **Markdown renderers:** `format_transitive_markdown()` and `format_deep_chains_markdown()` for human-readable reports. ### Tests Added New `scripts/test_dependency_graph.py` — 9 comprehensive test cases: - Simple chain A→B→C - Diamond A→{B,C}→D - Cycles (A→B→C→A) handling - Multiple longest chains - Empty graph edge case - Cycle detection via existing `detect_cycles()` - Longest chain length reporting All tests pass locally: ``` $ python3 scripts/test_dependency_graph.py ✅ All dependency graph tests passed ``` ### Verification The existing repo test suite still passes (89 tests in `tests/`): ``` $ pytest tests/ -v ============================== 89 passed ============================= ``` ### Usage Example ```bash # JSON output includes transitive closure + deep chains python3 scripts/dependency_graph.py --repos-dir ~/repos/ --format json ``` Sample output: ```json { "repos": { ... }, "cycles": [], "transitive": { "hermes-agent": ["timmy-config", "timmy-home", ...], ... }, "deep_chains": [["a", "b", "c", "d"]], "summary": { "longest_chain_length": 4, "transitive_pairs": 142, ... } } ``` Closes #111

Rockachopa added 1 commit 2026-04-26 09:08:45 +00:00

feat(dependency-graph): add transitive closure and deep chain analysis

Test / pytest (pull_request) Failing after 7s

Details

832b23286b

- Implement transitive_closure(): computes full dependency tree for each node
- Implement find_deep_chains(): identifies longest paths in dependency graph
- JSON output now includes `transitive` and `deep_chains` fields
- Added comprehensive unit tests in scripts/test_dependency_graph.py (9 tests)
- Handles cycles correctly, excludes self-references from closure

Meets acceptance criteria for #111:
  ✅ Builds transitive dep tree
  ✅ Identifies deep chains and circular deps
  ✅ Output: transitive dependency graph (via --format json)

Closes #111

Test / pytest (pull_request) Failing after 7s

Details

Checking for merge conflicts…

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin step35/111-5-5-transitive-dependency-an:step35/111-5-5-transitive-dependency-an

git checkout step35/111-5-5-transitive-dependency-an

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Timmy_Foundation/compounding-intelligence#247