Skip to content

fix(native): align sort_targets_by_confidence between native and WASM/TS resolution paths #1476

@carlos-alm

Description

@carlos-alm

In PR #1465, sort_targets_by_confidence is called in the native build_edges.rs path after resolve_call_targets, but the TS buildFileCallEdges in build-edges.ts does not sort before emitting. For multi-target calls, the stored confidence value could differ between engines.

The dedup key is (source_id, target_id) so the edge set is unaffected, but the stored confidence for duplicate multi-target calls may differ. Aligning both paths (either add sorting to TS or remove from native) would eliminate this latent difference.

Related: PR #1465 review.

Metadata

Metadata

Assignees

No one assigned

    Labels

    follow-upDeferred work from PR reviews that needs tracking

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions