fix: geminiBaseUrl is not worked

[aixgeek]

Related GitHub Issue

Closes: #

Roo Code Task Context (Optional)

Description

Test Procedure

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.
• Testing: New and/or updated tests have been added to cover my changes (if applicable).
• Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
• Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Additional Notes

Get in Touch

---

Important

Adds support for custom googleGeminiBaseUrl in GeminiHandler by updating countTokens method to use httpOptions.

• Behavior:
  • Adds support for custom googleGeminiBaseUrl in GeminiHandler class.
  • Updates countTokens method to use httpOptions with baseUrl if googleGeminiBaseUrl is provided.
• Functions:
  • Modifies countTokens in gemini.ts to include config with httpOptions for custom base URL.
• Misc:
  • No changes to other methods or files.

^{This description was created by for 275e4c8. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Review Summary

I've reviewed the changes in this PR. Here are the issues that need to be addressed:

• Add test coverage for the countTokens method with custom base URL support

Follow Along on Roo Code Cloud

[roomote]

This change adds support for custom base URL in countTokens, but there are no tests covering this method. The test file (gemini.spec.ts) has comprehensive tests for createMessage and completePrompt, but countTokens is not tested at all. Consider adding a test that verifies the httpOptions.baseUrl is properly passed when googleGeminiBaseUrl is set, similar to the existing test at line 122-124 in gemini.spec.ts for completePrompt.

[daniel-lxs]

I'm not clear on what issue this change is fixing, so closing for now. Feel free to reopen or clarify if there's additional context.