Skip to content

fix: classify repeated edge occurrences#2178

Open
hsneves wants to merge 5 commits into
Graphify-Labs:v8from
hsneves:fix/edge-occurrence-classification
Open

fix: classify repeated edge occurrences#2178
hsneves wants to merge 5 commits into
Graphify-Labs:v8from
hsneves:fix/edge-occurrence-classification

Conversation

@hsneves

@hsneves hsneves commented Jul 25, 2026

Copy link
Copy Markdown

Depends on #2177. The new work in this stacked PR is commit fb77da2.

Summary

  • preserve portable source occurrence evidence for Python type references and calls
  • normalize edges after endpoint remapping, separating legitimate repeated occurrences from producer duplicates
  • classify zero-node extraction outcomes and treat unrecognized JSON data as intentional skips
  • expose occurrence evidence in query/path/explain, HTML, GraphML, Cypher, Neo4j, and FalkorDB without collapsing parallel edges
  • extend diagnostics and generated skills with the new classifications

Validation

  • uv run ruff check .
  • uv run python -m tools.skillgen --check
  • uv run python -m tools.skillgen --schema-singleton
  • uv run python -m tools.skillgen --monolith-roundtrip
  • uv run pytest -q (3607 passed, 33 skipped)
  • real monorepo AST baseline: 457 duplicate occurrences reconciled as 418 legitimate + 39 producer duplicates; 14 intentional JSON skips; zero unclassified duplicates

@hsneves

hsneves commented Jul 25, 2026

Copy link
Copy Markdown
Author

Follow-up validation on the real monorepo exposed and fixed a Python 3.10 packaging gap: tomli was development-only, so installed CLI environments could not parse pyproject.toml. Commit 5430a55 moves it to runtime dependencies. Final validation: wheel build succeeded; full suite is now 3608 passed / 33 skipped; the installed Python 3.10 CLI materialized both manifests and the final corpus has zero unexpected-empty files.

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