Skip to content

Comments

Fix symbol position in ruff parser#24

Closed
ram-nadella wants to merge 1 commit intomainfrom
fix-ruff-parser-position
Closed

Fix symbol position in ruff parser#24
ram-nadella wants to merge 1 commit intomainfrom
fix-ruff-parser-position

Conversation

@ram-nadella
Copy link
Owner

Summary

  • Fixed off-by-one character position issue in ruff parser for function and class definitions
  • Changed from using the name's range to the statement's range to correctly position symbols at the def/class keyword

Test plan

  • All existing tests pass
  • Code formatted with cargo fmt
  • Code linted with cargo clippy
  • Integration tests verify correct symbol extraction

🤖 Generated with Claude Code

The ruff parser was reporting symbol positions starting at the function/class name instead of the def/class keyword. This fix uses the range of the entire statement to correctly position symbols at the beginning of their definitions.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ram-nadella
Copy link
Owner Author

Replaced by #25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant