Skip to content

feat: Steering support for CLI channel#241

Open
frostming wants to merge 4 commits into
mainfrom
feat/no-steering-buffer
Open

feat: Steering support for CLI channel#241
frostming wants to merge 4 commits into
mainfrom
feat/no-steering-buffer

Conversation

@frostming

Copy link
Copy Markdown
Collaborator

Signed-off-by: Frost Ming me@frostming.com

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 17, 2026

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 01a2c07 Commit Preview URL

Branch Preview URL
Jun 18 2026, 12:51 AM

…ramework and channel manager

Signed-off-by: Frost Ming <me@frostming.com>
…es for steering logic

Signed-off-by: Frost Ming <me@frostming.com>
…ents

Signed-off-by: Frost Ming <me@frostming.com>
… for steering handling

Signed-off-by: Frost Ming <me@frostming.com>
@frostming frostming force-pushed the feat/no-steering-buffer branch from e7756ad to 01a2c07 Compare June 18, 2026 00:48
@frostming frostming changed the title refactor: remove SteeringBuffer and related steering logic from the framework and channel manager feat: Steering support for CLI channel Jun 18, 2026
@frostming

frostming commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator Author

@Gezi-lzq I made some changes to the steering control. I removed the steering buffer and expose a new hook handle_steering so the plugin is free to define whatever data structure to store and feed the steering messages.
The downside is that ChannelManager is no longer aware of the steering queue and can't do anything about it.

This PR also implements a simple steering support for CLI channel, it's single-threaded. Run bub chat to try it.

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