Walkthrough

Introduces DoCallRangeIndex, a package-private class that pre-indexes all doCall AST nodes in a DocumentContext by their method-name or constructor-type selection range. SourceDefinedMethodCallInlayHintSupplier is refactored to resolve references through this index instead of walking the full AST per reference, removing the isRightMethod helper. New test cases cover multi-callsite, empty-argument, fewer-arguments, empty-range, and skipped-argument scenarios.

Changes

DoCallRangeIndex and InlayHint Supplier Refactor

Sequence Diagram(s)

sequenceDiagram
  participant Client
  participant Supplier as SourceDefinedMethodCallInlayHintSupplier
  participant Index as DoCallRangeIndex
  participant Doc as DocumentContext

  Client->>Supplier: getInlayHints(documentContext, range)
  Supplier->>Index: of(documentContext)
  Index->>Doc: traverse all doCall nodes
  Doc-->>Index: doCall list with name ranges
  Index-->>Supplier: DoCallRangeIndex

  loop each method reference in range
    Supplier->>Index: doCallFor(reference)
    Index-->>Supplier: Optional DoCallContext
    Supplier->>Supplier: toInlayHints(reference, doCall)
  end

  Supplier-->>Client: List of InlayHint

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 A rabbit once indexed a call-tree with care,
No more walking the AST branch by branch there.
A map of name-ranges, quick look-up for free,
Now hints pop up swiftly wherever they be.
With tests for the skipped and the empty in place,
The inlay hints hop with remarkable grace! 🌿

❌ Failed checks (1 warning)

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

• X
• Mastodon
• Reddit
• LinkedIn