Skip to content

feat: support Cortecs#238

Merged
andreynering merged 10 commits intocharmbracelet:mainfrom
harmw:cortecs
Apr 1, 2026
Merged

feat: support Cortecs#238
andreynering merged 10 commits intocharmbracelet:mainfrom
harmw:cortecs

Conversation

@harmw
Copy link
Copy Markdown
Contributor

@harmw harmw commented Mar 31, 2026

  • I have read CONTRIBUTING.md.
  • I have created a discussion that was approved by a maintainer (for new features).

This implements Cortecs, the European LLM router.

@harmw harmw requested a review from a team as a code owner March 31, 2026 14:57
@harmw harmw requested review from andreynering and raphamorim and removed request for a team March 31, 2026 14:57
Copy link
Copy Markdown
Member

@andreynering andreynering left a comment

Choose a reason for hiding this comment

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

Thank you @harmw!

I looks good overall. I added a few suggestions below.

I'd also like to ask you to properly test it, since you have a key. To do that, just run this server with go run . and run Crush with env CATWALK_URL=http://localhost:8080 crush.

@harmw
Copy link
Copy Markdown
Contributor Author

harmw commented Apr 1, 2026

✅ tested it (using kimi-k2.5)

Found a bug as well 🤔 looks like the last used model is not refreshed (which makes sense, b/c how would crush know it needs to refresh), as I had changed the MaxTokens value.

Is this a familiar observation? Should there maybe be some sort of refresh providers option?

nvm: crush update-providers

I should maybe consider writing some dev docs on how to manage/test providers here 😛

Copy link
Copy Markdown
Member

@andreynering andreynering left a comment

Choose a reason for hiding this comment

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

Thank you @harmw!

I just pushed two commits with cosmetic changes.

@andreynering andreynering merged commit 014d131 into charmbracelet:main Apr 1, 2026
8 checks passed
puffitos pushed a commit to puffitos/catwalk that referenced this pull request Apr 1, 2026
Co-authored-by: Andrey Nering <andreynering@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