Skip to content

Conversation

@Sma1lboy
Copy link

@Sma1lboy Sma1lboy commented Apr 1, 2025

Description

Added Tabby as API Provider. Tabby is a Self-hosted AI coding assistant. It also runs a Tabby Server for self-hosted LLM, also providing a lot of Multi-person collaboration features. At the same time, Tabby fully supports OpenAI restful API. So after the User sets up Tabby, they can share the auth token and endpoint to RooCode through Tabby client extension, so RooCode can directly request Tabby Server's /chat/completion interface

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Checklist:

Tried to use RooCode normally through implementation, tried various LMs provided by Tabby, reopened the configuration page multiple times to ensure persistent parameter storage is correct.
Reset RooCode to ensure welcome page Tabby runs correctly

  • My code follows the patterns of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

Additional context

Related Issues

TabbyML/tabby#3965

Reviewers

Demo

https://jam.dev/c/9a701b04-04c4-4bf7-8efe-4e5c47269328 (new version)

https://jam.dev/c/4e32b943-62e2-42fc-b305-ccc625c79b23

@changeset-bot
Copy link

changeset-bot bot commented Apr 1, 2025

⚠️ No Changeset found

Latest commit: 17b47c9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@hannesrudolph hannesrudolph moved this from New to PR [Pre Approval Review] in Roo Code Roadmap Apr 1, 2025
@Sma1lboy Sma1lboy marked this pull request as ready for review April 14, 2025 00:50
@Sma1lboy Sma1lboy requested review from cte and mrubens as code owners April 14, 2025 00:50
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Apr 14, 2025
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Apr 14, 2025

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]


Generated with ❤️ by ellipsis.dev

@dosubot dosubot bot added the enhancement New feature or request label Apr 14, 2025
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Apr 14, 2025

This pull request includes two distinct types of changes:

  1. Documentation Updates: The README files in multiple languages have been reformatted to improve the layout and readability of the contributors sections. These changes are consistent across the different language files and can be grouped together.

  2. API Provider Addition: The integration of the Tabby API provider involves significant code changes across multiple files, including the addition of new modules, handlers, and configurations.

To improve the review process and maintain clarity, I suggest splitting this pull request into two separate ones:

  • One pull request for the README updates across the different language files.
  • Another pull request for the Tabby API provider integration and related code changes.

This separation will help reviewers focus on the specific changes and ensure a more efficient review process. Thank you!

@Sma1lboy Sma1lboy force-pushed the feat-adding-tabbyml-as-thrid-api-provider branch from 7c9fd16 to 7f6fc47 Compare April 14, 2025 01:03
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. size:L This PR changes 100-499 lines, ignoring generated files. labels Apr 14, 2025
@Sma1lboy Sma1lboy force-pushed the feat-adding-tabbyml-as-thrid-api-provider branch from b05aaa8 to 3db1b9f Compare April 14, 2025 01:05
@dosubot dosubot bot removed the size:XL This PR changes 500-999 lines, ignoring generated files. label Apr 14, 2025
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Apr 14, 2025
@Sma1lboy Sma1lboy force-pushed the feat-adding-tabbyml-as-thrid-api-provider branch from 3db1b9f to 17b47c9 Compare April 14, 2025 01:06
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Apr 14, 2025
@hannesrudolph hannesrudolph moved this from New to PR [Pre Approval Review] in Roo Code Roadmap May 20, 2025
@hannesrudolph hannesrudolph moved this from PR [Needs Review] to TEMP in Roo Code Roadmap May 26, 2025
@daniel-lxs daniel-lxs moved this from TEMP to PR [Needs Review] in Roo Code Roadmap May 26, 2025
@hannesrudolph
Copy link
Collaborator

Thanks for submitting this PR.

We're closing it because, for adding new providers, we ask contributors to follow our issue-first approach. This allows us to discuss and approve proposed providers clearly before implementation.

We appreciate your understanding and would be happy to review this further if you create an issue outlining the provider details first. Thanks again!

@github-project-automation github-project-automation bot moved this from PR [Pre Approval Review] to Done in Roo Code Roadmap May 27, 2025
@github-project-automation github-project-automation bot moved this from Needs Preliminary Review to Done in Roo Code Roadmap May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants