[PR #15] [CLOSED] Fix some low-hanging performance issues #174

New issue

Closed

opened 2026-03-15 11:51:29 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 11:51:29 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/awslabs/iam-policy-autopilot/pull/15
Author: @mschlaipfer
Created: 11/21/2025
Status: ❌ Closed

Base: prerelease ← Head: optimize-python-extraction

📝 Commits (1)

00aea26 Fix some low-hanging performance issues

📊 Changes

4 files changed (+275 additions, -240 deletions)

View changed files

📝 iam-policy-autopilot-policy-generation/src/extraction/extractor.rs (+13 -0)
📝 iam-policy-autopilot-policy-generation/src/extraction/python/boto3_resources_model.rs (+20 -59)
📝 iam-policy-autopilot-policy-generation/src/extraction/python/extractor.rs (+42 -4)
📝 iam-policy-autopilot-policy-generation/src/extraction/python/resource_direct_calls_extractor.rs (+200 -177)

📄 Description

Issue #, if available:

Description of changes:

Moved ResourceDirectCallsExtractor::new(service_index) out of the loop
Instead of iterating over patterns, crated an ast-grep config with any

We should make sure this doesn't regress on our benchmark set before merging.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/awslabs/iam-policy-autopilot/pull/15 **Author:** [@mschlaipfer](https://github.com/mschlaipfer) **Created:** 11/21/2025 **Status:** ❌ Closed **Base:** `prerelease` ← **Head:** `optimize-python-extraction` --- ### 📝 Commits (1) - [`00aea26`](https://github.com/awslabs/iam-policy-autopilot/commit/00aea26e1ad2f46a12a417e3e6730d251db72398) Fix some low-hanging performance issues ### 📊 Changes **4 files changed** (+275 additions, -240 deletions) <details> <summary>View changed files</summary> 📝 `iam-policy-autopilot-policy-generation/src/extraction/extractor.rs` (+13 -0) 📝 `iam-policy-autopilot-policy-generation/src/extraction/python/boto3_resources_model.rs` (+20 -59) 📝 `iam-policy-autopilot-policy-generation/src/extraction/python/extractor.rs` (+42 -4) 📝 `iam-policy-autopilot-policy-generation/src/extraction/python/resource_direct_calls_extractor.rs` (+200 -177) </details> ### 📄 Description *Issue #, if available:* *Description of changes:* * Moved `ResourceDirectCallsExtractor::new(service_index)` out of the loop * Instead of iterating over patterns, crated an ast-grep config with `any` We should make sure this doesn't regress on our benchmark set before merging. By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>