Skip to content

Align default Docker container name with lstk (localstack-aws)#26

Open
jhubbardsf wants to merge 1 commit intolocalstack:mainfrom
jhubbardsf:refactor/default-container-name-localstack-aws
Open

Align default Docker container name with lstk (localstack-aws)#26
jhubbardsf wants to merge 1 commit intolocalstack:mainfrom
jhubbardsf:refactor/default-container-name-localstack-aws

Conversation

@jhubbardsf
Copy link
Copy Markdown

Summary

This PR updates the MCP default Docker container name from localstack-main to localstack-aws to match lstk defaults.

Why

lstk initializes LocalStack with localstack-aws, while MCP previously looked for localstack-main by default. Aligning these defaults removes onboarding confusion and makes first-run behavior consistent.

Changes

  • Updated default container name in Docker client/runtime code to localstack-aws
  • Updated related tests to expect localstack-aws
  • Updated documentation to reflect the new default

Linked issue

Closes #25

Test plan

  • Run unit tests
  • Verify MCP connects successfully when only localstack-aws exists
  • Verify docs/readme examples are accurate

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.

Inconsistent container name between lstk (localstack-aws) and MCP (localstack-main)

1 participant