fix: credit import-type usages as exports consumers

[carlos-alm]

Summary

• domain/analysis/exports.ts's consumer query only matched kind = 'calls' edges, so interfaces/types consumed only via import type { X } + type-annotation usage showed consumerCount: 0 even though codegraph deps correctly tracked the file-level import.
• The builder already emits symbol-level imports-type edges (emitTypeOnlySymbolEdges) — widened the consumer query to kind IN ('calls', 'imports-type'), matching the edge-kind set already used elsewhere (structure.ts, graph-enrichment.ts, boundaries.ts).

Closes #1724

Test plan

• npm test — full suite green (3359 passed, 0 failed)
• npm run lint — clean
• Repro: codegraph exports src/domain/graph/builder/cha.ts -T --json now shows ChaContext with consumerCount: 2 (was 0)
• Negative case: genuinely-unused exports still show consumerCount: 0

Note: deliberately did NOT extend this to extends/implements edges — investigation found those resolve by symbol name with no file scoping, producing real false positives (e.g. unrelated same-named Repository/UserRepository fixtures across languages). Filed #1812 for that, and #1813 for a separate gap (import { type X } inline modifier not tracked as type-only in either engine's extractor).

Stacked on #1811 (base branch fix/issue-1723-roles-dead-params-interfaces) — only the exports.ts/test diff is this issue's change.

[greptile-apps]

Greptile Summary

This PR fixes a false-negative in exportsData where exported interfaces and types consumed only via import type { X } were incorrectly reported as having zero consumers. The fix widens the SQL consumer query from kind = 'calls' to kind IN ('calls', 'imports-type'), matching the edge-kind set already used consistently in structure.ts, graph-enrichment.ts, and boundaries.ts. Three focused integration tests cover the happy path, the --unused filter exclusion, and the negative case (genuinely unreferenced exports still report zero consumers).

• Core change (exports.ts): One-line SQL predicate extension with thorough inline documentation explaining why extends/implements edges are deliberately excluded.
• Test coverage (exports.test.ts): New describe block adds a self-contained fixture DB (own tmpDir2/dbPath2) so it doesn't affect existing tests; assertions include the intentional file-level name/line shape of imports-type consumers, addressing the previous review thread.

Confidence Score: 5/5

Safe to merge — a minimal, well-contained query predicate extension backed by targeted regression tests and aligned with the edge-kind set already in use throughout the codebase.

The change is a single SQL predicate line in one function. The fix is consistent with how imports-type edges are consumed everywhere else. The exclusion of extends/implements is well-reasoned and documented. The new test suite is isolated, covers the primary scenario, the --unused boundary case, and the true-negative case.

No files require special attention.

Important Files Changed

Filename	Overview
src/domain/analysis/exports.ts	Single SQL predicate widened from kind = 'calls' to kind IN ('calls', 'imports-type'); well-commented with deliberate exclusion of extends/implements edges documented inline.
tests/integration/exports.test.ts	Adds a new isolated describe block with three tests covering the credited-consumer, --unused-exclusion, and genuinely-unused cases; assertions for consumers[0].name/line document the file-level shape of imports-type edges.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["exportsFileImpl()"] --> B["consumersStmt.all(symbolId)"]
    B --> C{{"edges WHERE target_id = symbol\nAND kind IN ('calls', 'imports-type')"}}
    C -->|"calls edge"| D["Source: caller function node\nname = function name\nline = call-site line"]
    C -->|"imports-type edge (NEW)"| E["Source: importing file node\nname = filename\nline = 0"]
    D --> F["consumers[] entry"]
    E --> F
    F --> G{noTests filter?}
    G -->|yes| H["Filter out test-file consumers"]
    G -->|no| I["Keep all consumers"]
    H --> J["consumerCount = consumers.length"]
    I --> J
    J -->|"count > 0"| K["Export credited as used"]
    J -->|"count = 0"| L["Export flagged as dead / unused"]

Loading

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["exportsFileImpl()"] --> B["consumersStmt.all(symbolId)"]
    B --> C{{"edges WHERE target_id = symbol\nAND kind IN ('calls', 'imports-type')"}}
    C -->|"calls edge"| D["Source: caller function node\nname = function name\nline = call-site line"]
    C -->|"imports-type edge (NEW)"| E["Source: importing file node\nname = filename\nline = 0"]
    D --> F["consumers[] entry"]
    E --> F
    F --> G{noTests filter?}
    G -->|yes| H["Filter out test-file consumers"]
    G -->|no| I["Keep all consumers"]
    H --> J["consumerCount = consumers.length"]
    I --> J
    J -->|"count > 0"| K["Export credited as used"]
    J -->|"count = 0"| L["Export flagged as dead / unused"]

Loading

_{Reviews (3): Last reviewed commit: "fix: assert file-level name/line for imp..." | Re-trigger Greptile}

[github-actions]

Codegraph Impact Analysis

1 functions changed → 3 callers affected across 3 files

• exportsFileImpl in src/domain/analysis/exports.ts:134 (3 transitive callers)

[carlos-alm]

@greptileai