Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 1, 2025

The enum list in aws_mq_broker_invalid_engine_type.go needed verification and documentation of supported engine types.

Changes

  • Added AWS API reference comments to both aws_mq_broker_invalid_engine_type.go and aws_mq_configuration_invalid_engine_type.go
  • Documented that Terraform uses mixed case (ActiveMQ, RabbitMQ) while AWS API uses uppercase (ACTIVEMQ, RABBITMQ)
  • Verified enum list is current against aws-sdk-ruby (Oct 31, 2025) and AWS documentation
// https://docs.aws.amazon.com/amazon-mq/latest/api-reference/broker-engine-types.html
// Terraform uses mixed case: ActiveMQ, RabbitMQ (AWS API uses uppercase: ACTIVEMQ, RABBITMQ)
enum: []string{
    "ActiveMQ",
    "RabbitMQ",
},

No new engine types have been added to AWS MQ - ActiveMQ and RabbitMQ remain the only supported broker engines.

Original prompt

Please update enum list in rules/aws_mq_broker_invalid_engine_type.go


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: wata727 <9624059+wata727@users.noreply.github.com>
Copilot AI changed the title [WIP] Update enum list in rules/aws_mq_broker_invalid_engine_type.go Add documentation to AWS MQ engine type validation rules Nov 1, 2025
Copilot AI requested a review from wata727 November 1, 2025 13:45
Copilot finished work on behalf of wata727 November 1, 2025 13:45
@wata727 wata727 marked this pull request as ready for review November 1, 2025 14:53
@wata727 wata727 enabled auto-merge (squash) November 1, 2025 14:53
@wata727 wata727 merged commit 092c988 into master Nov 1, 2025
21 checks passed
@wata727 wata727 deleted the copilot/update-enum-list-in-aws-mq-broker branch November 1, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants