Skip to content

Remove CLI code from core repository#13704

Merged
silv-io merged 7 commits intomainfrom
flc-263-remove-cli-from-community-repository
Feb 11, 2026
Merged

Remove CLI code from core repository#13704
silv-io merged 7 commits intomainfrom
flc-263-remove-cli-from-community-repository

Conversation

@silv-io
Copy link
Member

@silv-io silv-io commented Feb 5, 2026 •
edited
Loading

Motivation

The CLI code is being extracted from the main repositories into a standalone package (localstack-cli-standalone). This allows the CLI to be maintained and released independently, simplifying the codebase and release process.

Changes

  • Remove CLI code from localstack-core (moved to localstack/localstack-cli-standalone)
  • Remove CLI tests and CI workflow (now lives in the standalone repo)
  • Install localstack-cli-standalone into the Docker image venv

Tests

The testing pipeline has moved to the standalone repo and we're also conducting manual tests of the separate pipeline

Related

FLC-263

@silv-io silv-io added semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Feb 5, 2026
@github-actions
Copy link

github-actions bot commented Feb 5, 2026 •
edited
Loading

Test Results - Preflight, Unit

23 099 tests   - 41   21 237 ✅  - 39   6m 19s ⏱️ -3s
     1 suites ± 0    1 862 💤  -  2 
     1 files   ± 0        0 ❌ ± 0 

Results for commit 921f714. ± Comparison against base commit 17e97cc.

This pull request removes 41 tests. 
tests.unit.cli.test_cli ‑ test_completion[bash]
tests.unit.cli.test_cli ‑ test_completion[fish]
tests.unit.cli.test_cli ‑ test_completion[zsh]
tests.unit.cli.test_cli ‑ test_completion_unknown_shell
tests.unit.cli.test_cli ‑ test_container_not_runnin_error[ssh]
tests.unit.cli.test_cli ‑ test_container_not_runnin_error[stop]
tests.unit.cli.test_cli ‑ test_create_with_plugins
tests.unit.cli.test_cli ‑ test_disable_publish_analytics_event_on_command_invocation
tests.unit.cli.test_cli ‑ test_do_not_publish_analytics_event_on_invalid_command_invocation[config show --format invalid]
tests.unit.cli.test_cli ‑ test_do_not_publish_analytics_event_on_invalid_command_invocation[invalid]
…

♻️ This comment has been updated with latest results.

@silv-io silv-io marked this pull request as ready for review February 10, 2026 10:18
@github-actions
Copy link

github-actions bot commented Feb 10, 2026 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 4s ⏱️ +5s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 921f714. ± Comparison against base commit 17e97cc.

♻️ This comment has been updated with latest results.

@dfangl dfangl removed their request for review February 10, 2026 10:58
@github-actions
Copy link

github-actions bot commented Feb 10, 2026 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 39m 12s ⏱️
5 608 tests 5 100 ✅ 508 💤 0 ❌
5 614 runs  5 100 ✅ 514 💤 0 ❌

Results for commit 921f714.

♻️ This comment has been updated with latest results.

@silv-io silv-io force-pushed the flc-263-remove-cli-from-community-repository branch from 2995807 to 628825d Compare February 11, 2026 14:26
@silv-io silv-io force-pushed the flc-263-remove-cli-from-community-repository branch from 628825d to 775b58c Compare February 11, 2026 14:29
alexrashed
alexrashed reviewed Feb 11, 2026
View reviewed changes
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

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

Finally, the CLI is being extracted! This will resolve so many devx issues and friction we have seen when doing releases! 💯 🚀
I think there's only one old thing left in the Dockerfile which needs to be updated, afterwards we should be good to go! 💯

@alexrashed
Copy link
Member

alexrashed commented Feb 11, 2026 •
edited
Loading

FYI: I think there were some (hopefully all) failures with the Community pipeline earlier. A rebase might fix some of them, as explained by @bentsku here.

@silv-io
Copy link
Member Author

silv-io commented Feb 11, 2026

FYI: I think there were some (hopefully all) failures with the Community pipeline earlier. A rebase might fix some of them, as explained by @bentsku here.

I've just recently rebased so those are in already.

For merging this I'll wait until the Pro PR is merged so that the community against pro will be green here as well.

alexrashed
alexrashed approved these changes Feb 11, 2026
View reviewed changes
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

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

Thanks for jumping on the comments! The community tests unfortunately failed because of the different branch names (the missing companion branch detection).
But if both PRs are 💚 otherwise, I think we have enough trust to merge this PR! :shipit:

@alexrashed alexrashed force-pushed the flc-263-remove-cli-from-community-repository branch from 7680a1d to 921f714 Compare February 11, 2026 15:11
@github-actions
Copy link

github-actions bot commented Feb 11, 2026

S3 Image Test Results (AMD64 / ARM64)

    2 files      2 suites   8m 40s ⏱️
  559 tests   507 ✅  52 💤 0 ❌
1 118 runs  1 014 ✅ 104 💤 0 ❌

Results for commit 921f714.

@silv-io silv-io merged commit 41e8292 into main Feb 11, 2026
38 of 41 checks passed
@silv-io silv-io deleted the flc-263-remove-cli-from-community-repository branch February 11, 2026 17:09
@tiurin tiurin mentioned this pull request Feb 11, 2026
Open
3 tasks
@alexrashed alexrashed mentioned this pull request Feb 12, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexrashed alexrashed alexrashed approved these changes

@thrau thrau Awaiting requested review from thrau

@k-a-il k-a-il Awaiting requested review from k-a-il k-a-il is a code owner

@bentsku bentsku Awaiting requested review from bentsku bentsku is a code owner

@dominikschubert dominikschubert Awaiting requested review from dominikschubert dominikschubert is a code owner

Assignees

No one assigned

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@silv-io @alexrashed