TypeScript: this.m() links to an imported function m instead of the class's own m #2954

this.m() inside a class links to a function m that the file imports, not to the class's own m. This is the common case of a class wrapping a helper of the same name.

// lib.ts
export function findAll(q: string): string[] { return [q]; }
export const getFactor = (w: number): number => w;

// tools.ts
import { findAll, getFactor } from './lib';
export class Tools {
  run(q: string) { return this.findAll(q); }                               // line 3
  private findAll(q: string): string[] { return findAll(q).concat('own'); } // line 4
  size(w: number) { return this.getFactor(w); }                            // line 5
  private getFactor = (w: number): number => getFactor(w) * 2;             // line 6
}

The class's own findAll and getFactor get no callers from run/size, and the import gets callers that never call it directly. The edge's refName is the bare name (findAll), so the this. receiver is already lost by the time the import binding answers.

Reproduced on main at 2577601 with CODEGRAPH_KERNEL=0. Linux/WSL2, Node 22.23.2. 2577601 ("a call on the caller's own class reaches that class's member") doesn't change this: that path ranks same-named methods, and here the import answers first.

In this repo. ToolHandler in src/mcp/tools.ts imports groupDefinitions from ../graph/symbol-lookup and also has a private groupDefinitions(...) (line 3220). this.groupDefinitions(allMatches.nodes, fileFilter) at line 3250 links to src/graph/symbol-lookup.ts. this.findAllSymbols(cg, symbol) at lines 3245 and 3328 does the same with src/graph/named-symbol-flow.ts, ignoring the class's own findAllSymbols (line 8675). I counted 7 such edges in the index of this repo, and 1 in excalidraw (this.getFormFactor(…) in App.tsx → packages/common/src/editorInterface.ts, ignoring App's private getFormFactor = …). Found by checking every TS calls edge against the TypeScript checker.

Expected. A call with a this. (or super.) receiver should never resolve through the file's import bindings. It names a member, not a binding in scope.

Related: 2577601 / #1952 (own-class members for this. calls), #2953 (x.m() resolved through the import of x).