Skip to content

refactor: Import upstream in as dependency and rework LocalStack extensions into modules #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 4 commits into
base: localstack
Choose a base branch
from

Conversation

gregfurman
Copy link

Changes

  • Uses the replace directive to import in the upstream aws/aws-lambda-runtime-interface as a Golang package/dependency as opposed to relying on the fork relationship.
    • In go.mod: replace go.amzn.com => github.com/aws/aws-lambda-runtime-interface-emulator v0.0.0-20250423173140-3a0772eae98d (v0.0.0-20250423173140-3a0772eae98d is a SHA pointing to the latest tagged version).
  • Reworked LocalStack extension code into modules instead of all being in the main package.
  • All extension code now resides in internal/ so as to restrict it being imported outside of this repo.

TODO

  • Simplify CustomInteropServer implementation.
  • Improve usage of dependency injection throughout codebase.
  • Create structs/models for many of these modules.
  • Add tests.
  • Update documentation to remove references to the upstream where appropriate.

@gregfurman gregfurman self-assigned this May 20, 2025
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.

1 participant