feat: add --initialPermissions flag to reduce execution time #212
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR implements the
--initialPermissionsflag (closes #91) which allows users to specify permissions upfront to reduce MPF execution time. This also provides a workaround for the Terraform remote backend issue (#172).Changes
New Feature:
--initialPermissionsflag--initialPermissions "perm1,perm2,perm3"--initialPermissions @path/to/file.jsonImplementation
parseInitialPermissions()- Parses comma-separated or @file.json formatappendUserInitialPermissions()- Helper to reduce code duplication across commandsIterationCountfield added toMPFResultto track discovery iterationsTesting
TestParseInitialPermissionsunit tests for parsing logicTestTerraformACIWithInitialPermissionse2e test that verifies 0 iterations when all permissions are provided upfrontDocumentation
Use Cases
Example Usage
Related Issues