Skip to content

fix(slack): resolve model aliases before LLM fallback - #1572

Draft
decofe wants to merge 1 commit into
mainfrom
centaur/fuzzy-message-overrides-1788195464
Draft

fix(slack): resolve model aliases before LLM fallback#1572
decofe wants to merge 1 commit into
mainfrom
centaur/fuzzy-message-overrides-1788195464

Conversation

@decofe

@decofe decofe commented Aug 31, 2026

Copy link
Copy Markdown
Member

Summary

  • resolve common natural-language harness, model, provider, and reasoning aliases before calling the LLM selector
  • allow one-character fuzzy matches for longer aliases inside explicit selection clauses
  • retain the LLM fallback for ambiguous requests and guard against incidental model-name mentions

Validation

  • pnpm --filter slackbotv2 test (252 passed, 1 skipped)
  • pnpm --filter slackbotv2 run check:types (blocked by three existing fetch.preconnect errors in untouched test/slack-user.test.ts)

Prompted by: @0xalpharush

Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.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