mirror of
https://github.com/awslabs/iam-policy-autopilot.git
synced 2026-04-26 00:15:57 +03:00
[GH-ISSUE #83] Refactor: (usize, usize) -> Position, start_position: Position and end_position: Position -> Span #150
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/iam-policy-autopilot#150
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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
usizepairs, and structs which have both astart_positionand anend_positionwith.0and.1calls. 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 thePositionstruct (and if we need it a 0-indexed one).@mschlaipfer commented on GitHub (Dec 18, 2025):
This should happen after https://github.com/awslabs/iam-policy-autopilot/pull/84 is merged
@mschlaipfer commented on GitHub (Jan 13, 2026):
https://github.com/awslabs/iam-policy-autopilot/pull/84 introduces a
struct Locationwhich accomplishes this.