Update get_stack_output_role_arn method to return ArmoRoleArn

[kooomix]

PR Type

Bug fix

---

Description

• Updated get_stack_output_role_arn to return ArmoRoleArn.

• Fixed potential issue with incorrect stack output key.

---

Changes walkthrough 📝

	Relevant files
Bug fix	aws.py Corrected stack output key in `get_stack_output_role_arn` infrastructure/aws.py Updated get_stack_output_role_arn to return ArmoRoleArn. Replaced incorrect output key RoleArn with ArmoRoleArn. +1/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[codiumai-pr-agent-free]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Breaking Change The change from 'RoleArn' to 'ArmoRoleArn' may break existing code that relies on the old output key. Verify that all dependent code is updated accordingly. return self.get_stack_output("ArmoRoleArn")

[codiumai-pr-agent-free]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Possible issue	Add missing error handling Add error handling to catch and log specific cases where 'ArmoRoleArn' output is missing from the stack outputs infrastructure/aws.py [82-83] def get_stack_output_role_arn(self): - return self.get_stack_output("ArmoRoleArn") + try: + return self.get_stack_output("ArmoRoleArn") + except KeyError as e: + Logger.logger.error("Stack output 'ArmoRoleArn' not found in stack outputs") + raise Apply this suggestion Suggestion importance[1-10]: 7 Why: The suggestion adds important error handling to explicitly catch and log cases where the required 'ArmoRoleArn' output is missing from stack outputs, improving debugging and error visibility.	7

[github-actions]

Failed to generate code suggestions for PR