fix(lang): track Solidity modifier invocations as decorators - #2090
fix(lang): track Solidity modifier invocations as decorators#2090sanskar717 wants to merge 1 commit into
Conversation
Signed-off-by: Sanskar gupta <sg3427195@gmail.com>
|
Thanks for opening this — it has been seen, and it is queued. This note is automated, but it is not a brush-off: it exists so you know where your PR stands instead of having to guess from silence. Current review status: working through a backlog. What that means for this PR, concretely:
Things that will genuinely speed it up whenever review does happen:
If this fixes a bug, a reproduction we can run is worth more than a description of the symptom. Thanks for contributing, and sorry in advance for the wait. |
|
The issue specifically requested Verified using |
Summary
Adds Solidity
modifier_invocationsupport to the existing Solidity language specification so modifier usages such asonlyOwnerare tracked as decorators in the code graph.Changes
solidity_decorator_typeswithmodifier_invocationCBM_LANG_SOLIDITYonlyOwnerto be tracked correctlyVerification
Verified locally with
search_codeusingonlyOwner:onlyOwnerinvocation onwithdrawRelation to #1754
The issue proposes adding Solidity parser support. Since Solidity grammar is already vendored through Tree-sitter, this change extends the existing parsing path without introducing a Node.js runtime or JavaScript dependency.
This PR specifically fixes the missing modifier/decorator tracking in the current Solidity support.