--update runbook: graph.json backup instruction comes after the diff that needs it, so the graph diff never runs

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#4,539 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@sunildivakar96-cpu đang làm issue này rồi.

Từ ngày 18/10/2026.

  • #4540 của @sunildivakar96-cpu — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
markdown
Lĩnh vực
documentation

Hướng nghiên cứu

Kiểm tra 14 runbook cho các nền tảng tách riêng trong references/update.md và các monolith aider và devin được nêu trong issue. Theo dõi thứ tự các hướng dẫn về merge của Step 4, graph diff, backup và cleanup; thực hiện backup trước merge và thêm thông báo rõ ràng khi thiếu backup. Hoàn tất khi mỗi runbook trình bày các bước đó theo thứ tự hoạt động đúng và việc thiếu backup không bị bỏ qua.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Version: 0.9.92 (v8 at 86c57be); also present in 0.9.91. All 14 split-platform references/update.md runbooks, plus the aider and devin monoliths.

Problem

The --update runbook ends with a "show the graph diff" block that runs after Step 4. It compares graphify-out/.graphify_old.json, a backup of the pre-update graph.json, with the merged extraction. The instruction that creates that backup sits after the diff block, at the end of the section:

After Step 4, show the graph diff:
  ...diff block reads .graphify_old.json...
Before the merge step, save the old graph: `cp graphify-out/graph.json graphify-out/.graphify_old.json`
Clean up after: `rm -f graphify-out/.graphify_old.json`

An agent that follows the runbook from top to bottom reaches the backup sentence only after Step 4 has rewritten graph.json. By then it's too late:

  • If the agent never takes the backup, the diff block's if old_data: has no else, so the diff prints nothing at all. The user gets no diff and no sign that one was skipped.
  • If it takes the backup on reaching the sentence, the "old" graph is already the new one, and the diff reports no changes.

Repro

  1. Build a graph, change a code file, and run /graphify . --update, following references/update.md in order.
  2. After Step 4 the diff block runs before any .graphify_old.json exists, and it prints nothing.

Expected

The backup is taken before the merge, and the diff block says so when the backup is missing instead of skipping silently.

Ngôn ngữ chính
Python
Star
124k
Fork
11.9k
Merge trung bình
2 giờ 35 phút
Pull request đã merge (30 ngày)
6

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Graphify-Labs/graphify

Tất cả issue của Graphify-Labs/graphify

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.