fix(trace,impact): module-level callers invisible due to synthetic source_id, inconsistent with rc

[Wolfvin]

Mengapa

Follow-up dari PR #219 (fix issue #210) — didokumentasikan eksplisit di PR body sebagai "Found, not fixed" item #1, BOS diminta putuskan apakah perlu issue terpisah.

Verifikasi konkret di KDS backend real: search "requirePermission" --mode symbol menunjukkan rc:49 (setelah fix #219 — module-level calls sekarang ter-track), tapi context --check trace --name requirePermission --direction up menunjukkan HANYA 1 entry (dirinya sendiri) — 0 real caller ditemukan. Inkonsistensi ini berbahaya: orang yang pakai trace untuk cek "aman gak kalau symbol ini diubah/dihapus" akan salah simpul "aman, tidak ada caller" padahal rc bilang dipakai 49 kali.

Konteks

Root cause (dari PR #219): module-level call edges pakai synthetic source_id = "<file>:0:<module>" yang TIDAK punya entry di graph_nodes. trace/impact JOIN graph_edges.source_id ke graph_nodes.node_id — synthetic source_id ini selalu gagal JOIN, jadi caller dari module-level calls tidak pernah muncul di hasil trace/impact walau edge-nya ada di database.

Tujuan

trace/impact harus bisa menampilkan module-level caller (misal "called at module level in src/routes/accounting.ts", tanpa perlu nama fungsi spesifik karena memang tidak ada fungsi pembungkus) — supaya konsisten dengan rc yang sudah benar dari fix #219.

Constraint

• Jangan buat node_graphs entry palsu untuk <module> (PR fix(callgraph): extract module-top-level calls to fix rc undercount (closes #210) #219 sengaja menghindari ini supaya tidak polusi list/search output dengan entry fake) — cari cara lain, misal: fallback query yang follow dangling source_id dan render sebagai "module-level caller" tanpa perlu match ke node asli
• Baca dulu trace_engine.py dan impact_engine.py untuk paham query JOIN yang dipakai sebelum ubah apapun

Definition of Done

1. trace --direction up --name requirePermission di KDS backend menunjukkan minimal beberapa module-level caller (bukan cuma 1/dirinya sendiri)
2. impact --name requirePermission menunjukkan dependent count yang konsisten dengan rc dari search
3. Tidak ada regresi di list/search --mode symbol (tidak ada entry <module> palsu muncul di situ)
4. Test suite 0 regresi baru

Checklist Laporan Worker

• Before/after trace output untuk requirePermission di KDS backend
• Konfirmasi rc dan trace caller count sekarang konsisten (atau dijelaskan kenapa masih beda by design)
• Hasil test suite lengkap

[Wolfvin]

Mengerjakan issue ini -- 2026-07-12T12:42:30Z

[Wolfvin]

PR siap review

PR: #248 -- fix(trace,impact): render module-level callers instead of silently dropping (closes #223)
Branch: fix/issue-223-module-level-callers-trace-impact
Summary: graph_model._bfs sekarang emit synthesized entry untuk synthetic source_id :0: (sebelumnya silent drop karena JOIN graph_nodes.node_id return no row). trace_engine + impact_engine ikut mempropagasi marker module_level. Tidak membuat fake node di graph_nodes (constraint dari issue terpenuhi — list/search tetap bersih).
Tests: 16 new tests (tests/test_issue223_module_level_callers.py) all pass + 128 broader sanity tests pass. 3 failures di test_compact_format.py terverifikasi pre-existing di clean main (root cause: tree-sitter-typescript grammar tidak terinstall di worker env).

Evidence (synthetic fixture: 1 module-level caller + 1 regular caller):

• Before fix: trace --direction up hanya return 1 entry (requirePermission itself); impact direct dependents = 1 (handler saja).
• After fix: trace --direction up return 3 entries (requirePermission + + handler); impact direct dependents = 2 ( + handler).
• Constraint check: find_nodes_by_name("") → []; graph_nodes tidak ada row dengan name="" atau node_id LIKE "%:0:".

Verifikasi pada KDS backend real (rc:49 case dari issue body) diserahkan ke BOS saat review — worker env tidak punya akses ke KDS workspace.