Skip to content

feat(channel): support lifecycle channel startup control#218

Open
PsiACE wants to merge 2 commits into
mainfrom
lifecycle-channels
Open

feat(channel): support lifecycle channel startup control#218
PsiACE wants to merge 2 commits into
mainfrom
lifecycle-channels

Conversation

@PsiACE
Copy link
Copy Markdown
Collaborator

@PsiACE PsiACE commented May 24, 2026

Summary

  • add channel startup roles for interactive, listener, and lifecycle channels
  • start opt-in lifecycle channels alongside enabled channel runtimes
  • support !channel exclusions in enabled_channels / --enable-channel

PsiACE added 2 commits May 25, 2026 01:57
Signed-off-by: Chojan Shang <psiace@apache.org>
Signed-off-by: Chojan Shang <psiace@apache.org>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bub ea3e589 Commit Preview URL

Branch Preview URL
May 24 2026, 06:08 PM

@PsiACE
Copy link
Copy Markdown
Collaborator Author

PsiACE commented May 24, 2026

Previously, bub-mcp only registered remote tools after its lifecycle channel was started, which meant commands like bub chat did not naturally expose MCP tools. This change allows to mark bub-mcp an auto-start lifecycle channel so MCP tools are mounted by default in channel runtimes, while adding exclusions such as !mcp.lifecycle for debugging or for intentionally disabling MCP services.

@frostming
Copy link
Copy Markdown
Collaborator

frostming commented May 25, 2026

The property names start_with_channels and role are a bit cryptic, making it hard to understand what they are supposed to do. Alternatively, using channels to manage lifetime is purely a matter of convenience; it's not obvious from the class name that they are meant to handle this.

My suggestion is to subclass Channel as Lifecycle and Interface(names can be discussed), and make those properties hidden from users.

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