Add autofix suggestion for method calls mistaken as field access#42
Open
Add autofix suggestion for method calls mistaken as field access#42
Conversation
When a user writes `some_list.len` instead of `some_list.len()`, the type checker now detects that the accessed name matches a method on the receiver type and suggests adding parentheses. This applies to all types with methods, not just List. https://claude.ai/code/session_014eWiDph7M3enBMha1hczaQ
cf2eb8e to
21c2cde
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds an autofix suggestion when a user attempts to access a field that doesn't exist but matches a method name on the type. For example, accessing
items.lenwill now suggestitems.len()with an automatic fix.Key Changes
type_checker.rsto detect when a field access name matches an available method on the typemethod_name()?"Autofixthat automatically converts the field access to a method call (e.g.,items.len→items.len())list_len_method.gdndemonstrating the autofix in actionImplementation Details
https://claude.ai/code/session_014eWiDph7M3enBMha1hczaQ