Skip to content

Conversation

@utarn
Copy link

@utarn utarn commented Jun 2, 2025

Related GitHub Issue

Related to #4260

Description

Add ModelHarbor as the provider in the select list

Test Procedure

Type of Change

  • 🐛 Bug Fix: Non-breaking change that fixes an issue.
  • New Feature: Non-breaking change that adds functionality.
  • 💥 Breaking Change: Fix or feature that would cause existing functionality to not work as expected.
  • ♻️ Refactor: Code change that neither fixes a bug nor adds a feature.
  • 💅 Style: Changes that do not affect the meaning of the code (white-space, formatting, etc.).
  • 📚 Documentation: Updates to documentation files.
  • ⚙️ Build/CI: Changes to the build process or CI configuration.
  • 🧹 Chore: Other changes that don't modify src or test files.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Code Quality:
    • My code adheres to the project's style guidelines.
    • There are no new linting errors or warnings (npm run lint).
    • All debug code (e.g., console.log) has been removed.
  • Testing:
    • New and/or updated tests have been added to cover my changes.
    • All tests pass locally (npm test).
    • The application builds successfully with my changes.
  • Branch Hygiene: My branch is up-to-date (rebased) with the main branch.
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Changeset: A changeset has been created using npm run changeset if this PR includes user-facing changes or dependency updates.
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Screenshot 2568-06-03 at 05 41 51

Documentation Updates

Does this PR necessitate updates to user-facing documentation?

  • No documentation updates are required.
  • Yes, documentation updates are required. (Please describe what needs to be updated or link to a PR in the docs repository).

Additional Notes

Get in Touch

Discord: utharn.b


Important

Integrates ModelHarbor as a new provider, updating API handling, UI components, and adding tests for the integration.

  • Behavior:
    • Adds ModelHarbor as a new provider option in ApiOptions.tsx and constants.ts.
    • Implements ModelHarborHandler in modelharbor.ts for API interactions.
    • Updates webviewMessageHandler.ts to handle ModelHarbor model requests.
  • API and Models:
    • Adds getModelHarborModels() in modelharbor.ts to fetch models from ModelHarbor API.
    • Updates modelCache.ts to cache ModelHarbor models.
    • Adds modelharbor.ts to handle ModelHarbor API responses and model transformations.
  • UI Components:
    • Adds ModelHarbor component in ModelHarbor.tsx for UI integration.
    • Updates ModelPicker.tsx and ApiOptions.tsx to support ModelHarbor model selection.
  • Testing:
    • Adds tests for ModelHarbor integration in modelharbor.test.ts and useSelectedModel.modelharbor.test.ts.
    • Updates validate.ts to include ModelHarbor in validation logic.

This description was created by Ellipsis for ced8518. You can customize this summary. It will automatically update as commits are pushed.

@utarn utarn requested review from cte and mrubens as code owners June 2, 2025 23:19
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. enhancement New feature or request labels Jun 2, 2025
@utarn utarn changed the title feat: add ModelHarbor provider integration and update related components #4260 feat: add ModelHarbor provider integration and update related components Jun 2, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jun 3, 2025
@hannesrudolph
Copy link
Collaborator

Sorry, the issue has not been approved.

@github-project-automation github-project-automation bot moved this from PR [Needs Prelim Review] to Done in Roo Code Roadmap Jun 3, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 3, 2025
@utarn
Copy link
Author

utarn commented Jun 6, 2025

Do I have to do something? Is it considered to be in RooCode now?

@utarn utarn deleted the modelharbor branch June 6, 2025 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request PR - Needs Preliminary Review size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants