fix(callgraph): arrow functions as object-literal values not tracked, undercounts ref_count

[Wolfvin]

Mengapa

Ditemukan oleh worker saat mengerjakan issue #210 (sudah closed via PR #219). PR #221 (closed, konflik dengan #219 yang sudah merge duluan) menemukan gap TERPISAH yang belum ter-cover oleh fix #219: arrow function yang jadi value di object-literal pair (const service = { list: (ctx) => {...}, create: (ctx) => {...} }) tidak pernah terdaftar sebagai node, jadi calls di dalam body-nya invisible ke reference counting.

Reproduksi worker (sebelum fix, TS parser):

export const assignmentService = {
  list: (ctx) => { if (!hasPermission(ctx.user, 'read')) return null; return []; },
  create: (ctx) => { if (!hasPermission(ctx.user, 'write')) throw new Error('no'); return {}; },
};
Output parser: nodes=0, edges=0

_parse_variable_declarator di ts_backend_parser.py/js_backend_parser.py hanya handle value berupa arrow_function/function_expression/defineStore(...) call langsung — kalau value adalah object node (object literal), return None, dan arrow function di dalam pair-nya tidak pernah diregister.

Konteks

Pattern ini umum untuk "service object" / "handler map" style code (grouping related functions sebagai object daripada named exports terpisah) — mirip Pinia defineStore yang SUDAH dihandle khusus di fix #219, tapi object-literal biasa (tanpa defineStore) belum.

Tujuan

Tambah dukungan: kalau variable_declarator's value adalah object literal (object node type), walk semua property/pair yang value-nya arrow_function/function_expression, register sebagai node dengan nama <varName>.<key> (persis seperti approach worker: assignmentService.list, assignmentService.create), supaya calls di dalam body-nya ter-collect dengan benar.

Constraint

• Jangan duplikat logic module-scope-calls yang sudah ada dari PR fix(callgraph): extract module-top-level calls to fix rc undercount (closes #210) #219 — fokus HANYA pada object-literal arrow function registration
• Cek dulu behavior sekarang di main (post-fix(callgraph): extract module-top-level calls to fix rc undercount (closes #210) #219) sebelum mulai — jangan asumsikan kode dari PR fix(callgraph): collect module-scope calls + object-literal arrow fns (closes #210) #221 yang di-reference di sini masih relevan tanpa verifikasi ulang terhadap state main terbaru
• Ikuti pattern node_id yang konsisten dengan node lain (<file>:<line> format, bukan node baru dengan skema berbeda)

Definition of Done

1. Reproduksi kasus di atas (const service = {method: (ctx) => {...}}) — call di dalam method ter-extract dengan benar
2. search --mode symbol untuk fungsi yang dipanggil hanya lewat pattern ini menunjukkan rc yang benar (bukan 0)
3. Tidak ada regresi terhadap fix fix(callgraph): extract module-top-level calls to fix rc undercount (closes #210) #219 (module-scope calls, inline arrow callback di argument) — test suite terkait harus tetap pass
4. Minimal 2 unit test baru (TS + JS) yang reproduce pattern object-literal-arrow-function

Checklist Laporan Worker

• Konfirmasi state main SAAT INI (post fix(callgraph): extract module-top-level calls to fix rc undercount (closes #210) #219) — jalankan reproduksi dulu untuk pastikan gap ini masih ada
• rc sebelum/sesudah fix untuk reproduksi case
• Hasil test suite lengkap (0 regresi baru, termasuk test yang ditambahkan fix(callgraph): extract module-top-level calls to fix rc undercount (closes #210) #219)

[Wolfvin]

Mengerjakan issue ini -- 2026-07-12T03:04:45Z

[Wolfvin]

PR siap review

PR: #225 -- fix(callgraph): register object-literal arrow fns as nodes (closes #222)
Branch: fix/issue-222-object-literal-arrow-fns
Summary: Verifikasi state main post-#219 menunjukkan rc callee SUDAH benar (3) berkat module-level pass dari PR #219 — calls di object-literal arrow body di-attribute ke . Tapi arrow function-nya sendiri TIDAK terdaftar sebagai node, jadi trace/impact/search --mode symbol tidak bisa resolve method by name. Fix: tambah pass terpisah (_find_object_literal_method_decls di TS, _collect_object_literal_method_calls di JS) yang register setiap pair dengan arrow/function value sebagai node .. Update _find_module_level_calls untuk skip pair dengan arrow/function value (hindari double-count antara per-function pass baru dan module-level pass #219).
Tests: 16 baru (8 TS + 8 JS) — registration, call attribution, multi-method, ref_count end-to-end, non-arrow pair handling, no double-count, object literal in function body. Full suite: 2276 passed (naik 16 dari baseline 2260), 17 pre-existing failures (sama dengan baseline — 0 regresi baru).