fix: Remove spaces from IAM policy sids

[jasonwashburn]

Summary

This PR addresses an issue discovered with spaces in SIDs, which was identified after the terraform AWS provider update to version 5.82.0 introduced additional validation. The update caused errors due to invalid SIDs containing spaces, as SIDs must not contain spaces according to [AWS IAM policy documentation]. The additional validation added in 5.82.0 has since been reverted (it caused additional issues, see: hashicorp/terraform-provider-aws#40639) and 5.82.1 was released without the validation. However it brought to light that spaces are not allowed in IAM Policy Document SIDs. Ref: (https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_sid.html).

Changes Made

• Removed spaces from SIDs to comply with AWS formatting rules for IAM policy documents.

References

• AWS IAM Policy Documentation: SIDs

[jasonwashburn]

Seems that tflint workflow is failing because terraform-docs isn't configured to ignore lockfiles. When it's not configured to ignore lockfiles and there's no file checked in, it will cause a version update in the output based on whatever the CI runner has installed at runtime rather than just documenting the actual requirement like >= 4.62.0. See excerpt from https://terraform-docs.io/user-guide/configuration/settings/#examples

Added --lockfile=false to args to get CI passing as I assume this is not the desired behavior?