Skip to content

Remove unused bot imports; Python 0.2.229 - #96

Merged
luke-lombardi merged 3 commits into
masterfrom
codex/fix-cli-imports
Oct 16, 2026
Merged

luke-lombardi merged 3 commits into
masterfrom
codex/fix-cli-imports

Conversation

@luke-lombardi

@luke-lombardi luke-lombardi commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Drop unused Bot exports, use uv for tool upgrades, and verify installed wheels on Linux/macOS.

Installer: slai-labs/beam_frontend#828.

@luke-lombardi

Copy link
Copy Markdown
Contributor Author

@jackf-beamcloud ready for approval here and beam-cloud/k8s-platform#162. CI and staging verification passed.

@jackf-beamcloud jackf-beamcloud left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Lazy exports look right: 8/8 tests pass against beta9 0.1.286, every name in __all__ resolves, and beam vm list --help works with pydantic/fastapi blocked. Non-blocking: from beam import integrations now hits __getattr__ and returns beta9.integrations instead of the beam/integrations.py submodule (same MCPServer exports, so harmless, but a special case like Client's would keep it exact).

@jackf-beamcloud

Copy link
Copy Markdown

Already approved this one and beam-cloud/k8s-platform#162 at their current heads (b30e852, c1808fd), no new commits since. Merging is yours.

@luke-lombardi
luke-lombardi marked this pull request as draft October 16, 2026 17:49
@luke-lombardi luke-lombardi changed the title Fix CLI imports; Python 0.2.229 Fix CLI installation; Python 0.2.229 Oct 16, 2026
@luke-lombardi

Copy link
Copy Markdown
Contributor Author

@jackf-beamcloud the import rewrite is removed. Please re-review this plus beam-cloud/beta9#2020 and slai-labs/beam_frontend#828 for the dependency and installer fixes.

@luke-lombardi luke-lombardi changed the title Fix CLI installation; Python 0.2.229 Verify CLI installations; Python 0.2.229 Oct 16, 2026
@luke-lombardi
luke-lombardi marked this pull request as ready for review October 16, 2026 17:55

@jackf-beamcloud jackf-beamcloud left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Import rewrite is gone; what's left is the uv-aware upgrade hint (uv-receipt.toml in sys.prefix means uv tool upgrade beam-client, else pip via sys.executable) and a CI step that installs the built wheel in a clean venv on Linux and macOS and runs pip check, beam --version and beam vm list --help. All four checks green. beta9#2020 is closed per your note, so nothing to re-review there. Merging is yours.

@luke-lombardi luke-lombardi changed the title Verify CLI installations; Python 0.2.229 Remove unused bot imports; Python 0.2.229 Oct 16, 2026
@luke-lombardi
luke-lombardi merged commit 589f8c7 into master Oct 16, 2026
4 checks passed
@luke-lombardi
luke-lombardi deleted the codex/fix-cli-imports branch October 16, 2026 18:01

@jackf-beamcloud jackf-beamcloud left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Drops the Bot/BotContext/BotEventType/BotLocation and experimental re-exports so CLI startup no longer pulls in beta9.abstractions.experimental, with a CI assert on the installed wheel that it stays out of sys.modules; all four checks green. Non-blocking: from beam import Bot and beam.experimental now raise on upgrade, so anyone with an old bot app breaks without a deprecation path; worth a changelog line if that's intended. Merging is yours.

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