[GH-ISSUE #124] Remove fallback snake_case to operation name conversion from `Operation::from_call` #107

New issue

Open

opened 2026-03-07 19:42:06 +03:00 by kerem · 0 comments

kerem commented

2026-03-07 19:42:06 +03:00

Owner

Originally created by @mschlaipfer on GitHub (Jan 27, 2026).
Original GitHub issue: https://github.com/awslabs/iam-policy-autopilot/issues/124

https://github.com/awslabs/iam-policy-autopilot/pull/91 introduces an allow override because removing it is non-trivial.

When trying to remove it we run into issues where we do not find the operation in boto3_method_to_operation because of wrong case conversion (see https://github.com/awslabs/iam-policy-autopilot/issues/123), which leads to the test_enrichment_engine_comprehensive test failing. This means we currently rely on this fallback.

The above test for me also failed for other reasons like iotthingsgraph not being in the service reference, or the elastictranscoder service not having SDK name mappings in the service reference (elastictranscoder has been deprecated and I had an old botocore model locally when running the test, meaning we also need to update our botocore model). We need to resolve such issues along the way of removing the fallback.

Originally created by @mschlaipfer on GitHub (Jan 27, 2026). Original GitHub issue: https://github.com/awslabs/iam-policy-autopilot/issues/124 https://github.com/awslabs/iam-policy-autopilot/pull/91 introduces an `allow` override because removing it is non-trivial. When trying to remove it we run into issues where we do not find the operation in `boto3_method_to_operation` because of wrong case conversion (see https://github.com/awslabs/iam-policy-autopilot/issues/123), which leads to the `test_enrichment_engine_comprehensive` test failing. This means we currently rely on this fallback. The above test for me also failed for other reasons like `iotthingsgraph` not being in the service reference, or the `elastictranscoder` service not having SDK name mappings in the service reference (`elastictranscoder` has been deprecated and I had an old botocore model locally when running the test, meaning we also need to update our botocore model). We need to resolve such issues along the way of removing the fallback.