`Span` #52

New issue

Closed

opened 2026-03-07 19:41:20 +03:00 by kerem · 0 comments

kerem commented

2026-03-07 19:41:20 +03:00

Owner

Originally created by @mschlaipfer on GitHub (Dec 17, 2025).
Original GitHub issue: https://github.com/awslabs/iam-policy-autopilot/issues/83

Originally assigned to: @mschlaipfer on GitHub.

We have multiple occurrences of usize pairs, and structs which have both a start_position and an end_position with .0 and .1 calls. We should clean these up to make the code more readable. We currently modify the 0-indexed positions provided by TreeSitter in various places throughout the code base. It would be nicer if we stored them as-is, and have a 1-indexed view on the Position struct (and if we need it a 0-indexed one).

Originally created by @mschlaipfer on GitHub (Dec 17, 2025). Original GitHub issue: https://github.com/awslabs/iam-policy-autopilot/issues/83 Originally assigned to: @mschlaipfer on GitHub. We have multiple occurrences of `usize` pairs, and structs which have both a `start_position` and an `end_position` with `.0` and `.1` calls. We should clean these up to make the code more readable. We currently modify the 0-indexed positions provided by TreeSitter in various places throughout the code base. It would be nicer if we stored them as-is, and have a 1-indexed view on the `Position` struct (and if we need it a 0-indexed one).