You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Redesigns the generated graph.html explorer with a compact canvas-style layout.
Adds a tabbed sidebar for node info, hubs, and community controls.
Adds confidence/evidence filters for EXTRACTED, INFERRED, and AMBIGUOUS edges.
Improves large graph usability with an explore-first seed view, bounded expansion, cached node/edge lookups, and a bottom command/search bar.
Adds dark/light theme support while preserving the existing embedded, no-server HTML export model.
Why
The previous HTML view loads every node and sidebar control into one dense panel, which becomes hard to scan on larger code graphs. This update keeps the same data contract and export path while making the output easier to navigate and less visually noisy.
I also ran the full local suite after installing CI extras. It collected and ran, with unrelated local/environment failures from a stale installed skill version, the Windows case-collided sample.F90 fixture in my checkout, and local DNS resolution for arxiv.org.
Notes
This PR only changes graphify/export.py. No generated graphify-out artifacts are included.
The HTML explorer redesign looks great conceptually — the tabbed sidebar, command bar, and evidence filters are all useful additions.
However the diff has a UTF-8 encoding problem that blocks the merge. Your editor saved the file in latin-1, which corrupted every non-ASCII character in export.py — including em-dashes and arrows in functions like to_obsidian, safe_name, and to_svg that your PR doesn't intend to touch. Merging as-is would silently break unrelated exporters.
To fix: re-create the changes from a fresh checkout of v7, make sure your editor is set to UTF-8, and restrict the diff strictly to the HTML explorer sections (_html_styles, _html_script, to_html). The unrelated to_obsidian / to_svg context lines in the current diff show the corruption clearly.
Happy to merge once the encoding is clean — the feature itself is worth having.
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
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.
Summary
graph.htmlexplorer with a compact canvas-style layout.EXTRACTED,INFERRED, andAMBIGUOUSedges.Why
The previous HTML view loads every node and sidebar control into one dense panel, which becomes hard to scan on larger code graphs. This update keeps the same data contract and export path while making the output easier to navigate and less visually noisy.
Validation
python -m py_compile graphify/export.pypython -m pytest tests/test_export.py tests/test_pipeline.py -qI also ran the full local suite after installing CI extras. It collected and ran, with unrelated local/environment failures from a stale installed skill version, the Windows case-collided
sample.F90fixture in my checkout, and local DNS resolution forarxiv.org.Notes
This PR only changes
graphify/export.py. No generatedgraphify-outartifacts are included.