Skip to content

feat(channels): configure reasoning from Slack model picker - #3690

Draft
overlord-letta wants to merge 3 commits into
mainfrom
overlord/slack-model-reasoning
Draft

feat(channels): configure reasoning from Slack model picker#3690
overlord-letta wants to merge 3 commits into
mainfrom
overlord/slack-model-reasoning

Conversation

@overlord-letta

@overlord-letta overlord-letta commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a reasoning selector beside Slack's existing /model picker when the current model supports configurable reasoning.
  • Route picker and text selections through the shared channel command and existing update_model protocol.
  • Read the current model from the listener before each change, then preserve its provider alias and context-window variant.
  • Carry backend-reported effort limits for OpenAI-compatible proxy models into the Slack selector and action path.

Before and after

Before, Slack /model could show and switch the routed conversation's model. It could not change reasoning.

After, the same response includes a reasoning selector for supported models. @agent /model reasoning <level> provides a text fallback. Unsupported models do not show the selector, and invalid levels return usage guidance.

Risk and limits

The change reuses the model update resolver. It does not add provider payload handling. Model switching and non-Slack adapters keep their existing rendering.

A live Slack workspace smoke was not run. The Slack Block Kit payload, action acknowledgement, thread routing, shared command routing, model update resolution, provider aliases, context-window variants, and unsupported models are covered by tests.

Validation

  • bun run check
  • bun test src/channels/model-reasoning-command.test.ts
  • bun test src/channels/model-reasoning-options.test.ts
  • bun test src/channels/commands.test.ts
  • bun test src/channels/registry-command-routing.test.ts
  • bun test src/channels/slack/adapter.test.ts
  • bun test src/channels/gateway-core.test.ts
  • bun test src/channels/routed-runtime-registration.test.ts
  • bun test src/channels/registry-lifecycle.test.ts
  • bun test src/cli/app/use-reasoning-cycle.test.ts
  • bun test src/websocket/listen-model-update.test.ts
  • bun test src/websocket/listener/protocol-inbound.test.ts
  • bun test src/websocket/listener/commands/model-toolset.test.ts
  • bun test src/websocket/listener/commands/model-catalog.test.ts

— Overlord (agent-c2adbf5c-8419-4211-8cd8-3740db164974)

lettamate and others added 3 commits August 5, 2026 11:39
Letta Code (agent-c2adbf5c-8419-4211-8cd8-3740db164974)

Let Slack users choose supported reasoning levels without changing the selected model, provider alias, or context-window variant. Reuse the existing model update protocol for picker and text-command selections.

Co-authored-by: Cameron Pfiffer <cameron@pfiffer.org>

👾 Generated with [Letta Code](https://letta.com)

Co-Authored-By: Letta Code <noreply@letta.com>
Letta Code (agent-c2adbf5c-8419-4211-8cd8-3740db164974)

Resolve Slack reasoning actions against one fresh listener response that contains both the current model and its supported tiers. This prevents stale channel state from switching the model and keeps proxy Default selections executable.

Co-authored-by: Cameron Pfiffer <cameron@pfiffer.org>

👾 Generated with [Letta Code](https://letta.com)

Co-Authored-By: Letta Code <noreply@letta.com>
Letta Code (agent-c2adbf5c-8419-4211-8cd8-3740db164974)

Reconcile the reasoning picker with the new gateway-owned MessageChannel lifecycle and retain both sets of validation.

Co-authored-by: Cameron Pfiffer <cameron@pfiffer.org>

👾 Generated with [Letta Code](https://letta.com)

Co-Authored-By: Letta Code <noreply@letta.com>
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