Skip to content

refactor: extract Route53 and IAM TUI submodels#184

Merged
nathanhuh merged 1 commit intomainfrom
refactor/tui-route53-iam-submodels
Apr 27, 2026
Merged

refactor: extract Route53 and IAM TUI submodels#184
nathanhuh merged 1 commit intomainfrom
refactor/tui-route53-iam-submodels

Conversation

@nathanhuh
Copy link
Copy Markdown
Contributor

@nathanhuh nathanhuh commented Apr 24, 2026

Summary

Refactors the TUI root model by moving Route53 and IAM feature state, message handling, key handling, filtering, commands, and views behind dedicated feature submodels. Also removes the remaining CloudWatch Logs root-model compatibility wrappers so tests exercise the submodel directly.

This is a focused slice of the broader root model split; additional feature submodels remain for #107.

Related Issues

Refs #107

Validation

  • make test
  • make build

Checklist

  • Scope is focused
  • Branch name follows docs/branch-naming-harness.md
  • Documentation harness reviewed (docs/documentation-harness.md)
  • README updated if user-facing behavior changed
  • Relevant docs/ pages updated if architecture, auth, config, or workflow changed
  • Tests/validation included
  • Breaking changes documented

- move Route53 state, handlers, filters, commands, and views behind the feature submodel contract

- move IAM user/key/rotation flows behind the feature submodel contract

- remove CloudWatch Logs root-model compatibility wrappers

Refs #107
Copy link
Copy Markdown
Contributor

@amazon-q-developer amazon-q-developer Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This refactoring successfully extracts Route53 and IAM functionality into separate submodels, following the established pattern in the codebase. The changes improve code organization and maintainability by encapsulating related state and behavior into dedicated models. All functionality remains intact, and the implementation correctly adheres to the featureSubmodel interface.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@nathanhuh nathanhuh merged commit 7c629bc into main Apr 27, 2026
2 checks passed
@nathanhuh nathanhuh deleted the refactor/tui-route53-iam-submodels branch April 27, 2026 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants