Repository navigation
fix(code-finder): match find_functions_by_argument against arg_types not parameter names - #1687
AloysJehwin wants to merge 1 commit into
Conversation
…not parameter names
|
@AloysJehwin is attempting to deploy a commit to the shashankss1205's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks @AloysJehwin — #1686 landed first for #1685 and carried the piece this one was missing: |
|
All CI failures here are pre-existing on
This PR could not be reopened (viewer lacks reopen permission despite being author). Re-submitted as #1701. |
Fixes #1685.
find_functions_by_argumentwas matching againstParameter.name(the variable name) rather than the argument type. For typed languages like Java, type information lives onFunction.arg_types, not on theParameternode, so queries for types likeOrderFilterreturned no results.The Cypher query now uses
any(t IN f.arg_types WHERE t CONTAINS $arg_type)to match against the type list, with a fallback to parameter-name matching for untyped languages.