Skip to content

fix(update): reconcile removed and renamed sources#1623

Closed
oleksii-tumanov wants to merge 1 commit into
Graphify-Labs:v8from
oleksii-tumanov:fix/stale-nodes-after-source-removal
Closed

fix(update): reconcile removed and renamed sources#1623
oleksii-tumanov wants to merge 1 commit into
Graphify-Labs:v8from
oleksii-tumanov:fix/stale-nodes-after-source-removal

Conversation

@oleksii-tumanov

@oleksii-tumanov oleksii-tumanov commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix #1622

Reconcile stored extractor-backed sources with the current corpus during full and incremental rebuilds. Source-path compatibility and reconciliation are isolated from _rebuild_code; semantic and out-of-scope records remain preserved.

Result

graph.json no longer retains only.py after deletion or old.py after a destination-only rename. Coverage includes code and document sources, subdirectory roots, legacy markers, symlinks, and hyperedges.

Testing

  • python -m pytest tests/test_watch.py -q (52 passed, 2 skipped; optional watchdog unavailable)
  • ruff check graphify/watch.py tests/test_watch.py (passed)
  • python -m tools.skillgen --check (134 artifacts matched)

@oleksii-tumanov oleksii-tumanov force-pushed the fix/stale-nodes-after-source-removal branch from e6797d5 to 3b3106f Compare July 3, 2026 02:32
@oleksii-tumanov oleksii-tumanov force-pushed the fix/stale-nodes-after-source-removal branch from 3b3106f to e85ad4b Compare July 3, 2026 02:47
safishamsi added a commit that referenced this pull request Jul 3, 2026
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@safishamsi

Copy link
Copy Markdown
Collaborator

Merged into v8 as 8d8d2b8 (your authorship). Verified: watch tests (54) pass including your coverage that semantic/out-of-scope records are preserved while stale extractor-backed sources are reconciled away, and full suite green (2866). Good fix — same incremental-staleness family as #1571/#1574, and keeping the reconciliation isolated from _rebuild_code is clean. Ships next release. Thanks!

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.

Update leaves stale graph sources after renames or final-file deletion

2 participants