Back up graph.html on both rebuild paths (CLI and skill) - #2512
Back up graph.html on both rebuild paths (CLI and skill)#2512GUGU75006 wants to merge 3 commits into
Graphify — looks good
Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).
Details
Graphify reviewed this change.
Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).
Graphify review — findings
This PR adds graph.html to the list of artifacts snapshotted by backup_if_protected in graphify/export.py, so the interactive viewer is included in the dated pre-overwrite backup folder alongside the other graph artifacts. It also updates the various agent skill files (amp, claw, codex, copilot, droid, kilo, and others) to import and call backup_if_protected in the build/cluster step before to_json overwrites the previous run. Corresponding changelog entries, skillgen fragments/expected outputs, and export tests are updated to match.
No blocking issues surfaced. 2 lower-confidence candidates did not survive cross-model review.
Analysis details — impact, health, verification
Impact & health
Graphify review
Impact — 1353 functions depend on the 1006 functions this change touches.
Health — grade B; 10 existing hotspot(s) in the area this change touches (pre-existing, not introduced here):
_rebuild_code()— 93 callers, 51 callees (high)build_merge()— 41 callers, 14 callees (high)to_obsidian()— 24 callers, 11 callees (high)to_json()— 42 callers, 6 callees (high)dispatch_command()— 2 callers, 117 callees (high)_make_graph()— 30 callers, 6 callees (high)run_pipeline()— 8 callers, 13 callees (high)to_canvas()— 14 callers, 3 callees (high)- …and 2 more
Verification — 1353 functions in the blast radius were not formally verified this run (proofs are advisory here).
Gate & verification
graphify gate
PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.
Advisory (not blocking):
- verification_scope: 1213 function(s) in the blast radius were not formally verified this run