Skip to content

Closes #538: save_manifest at end of --update merge step #545

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
safishamsi merged 1 commit into from
May 2, 2026
Merged

Closes #538: save_manifest at end of --update merge step #545

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions graphify/skill-copilot.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -823,6 +823,14 @@ if deleted:
# Merge: new nodes/edges into existing graph
G_existing.update(G_new)
print(f'Merged: {G_existing.number_of_nodes()} nodes, {G_existing.number_of_edges()} edges')

# Save manifest with the CURRENT full file list so the next --update
# diffs against today's filesystem state, not the prior --update's
# baseline. Without this, deleted files get reported as ghosts again
# on every subsequent --update until a full rebuild runs.
from graphify.detect import save_manifest
save_manifest(incremental['files'])
Comment on lines +831 to +832

Copilot AI Apr 25, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This snippet groups imports at the top, but from graphify.detect import save_manifest is added later. Consider moving that import up with the other imports to keep the python -c block easier to skim and maintain.

Copilot uses AI. Check for mistakes.
print('[graphify update] Manifest saved.')
"
```

Expand Down
8 changes: 8 additions & 0 deletions graphify/skill.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -931,6 +931,14 @@ merged_out = {
}
Path('graphify-out/.graphify_extract.json').write_text(json.dumps(merged_out))
print(f'[graphify update] Merged extraction written ({len(merged_out[\"nodes\"])} nodes, {len(merged_out[\"edges\"])} edges)')

# Save manifest with the CURRENT full file list so the next --update
# diffs against today's filesystem state, not the prior --update's
# baseline. Without this, deleted files get reported as ghosts again
# on every subsequent --update until a full rebuild runs.
from graphify.detect import save_manifest
save_manifest(incremental['files'])
Comment on lines +939 to +940

Copilot AI Apr 25, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The save_manifest import is introduced mid-script inside this python -c block, while other imports are grouped at the top. For readability/consistency, move the from graphify.detect import save_manifest line up into the main import section of this snippet and keep the call site here.

Copilot uses AI. Check for mistakes.
print('[graphify update] Manifest saved.')
"
```

Expand Down
Loading