temporarily patch custom defined llm clients on api

[miguelg719]

why

Serializing custom defined clients is not yet supported in the api

what changed

If a custom LLM client is defined in the constructor, we log a warning and disable api mode

test plan

[changeset-bot]

🦋 Changeset detected

Latest commit: cb4f80b

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

[greptile-apps]

Greptile Summary

This PR implements a temporary patch in lib/index.ts to handle custom LLM clients in API mode. The change adds a warning log message and automatically disables API mode when a custom LLM client is provided to prevent potential compatibility issues.

The change is relatively simple but important for system stability - it adds a safeguard to prevent custom LLM clients from being used in API mode, which could cause unexpected behavior. This is marked as a temporary solution, suggesting a more permanent fix will be developed later.

PR Description Notes:

• The PR description is empty and should be filled out with the required why/what/test plan sections

Confidence score: 4/5

1. Safe to merge - straightforward defensive programming change that prevents potential issues
2. High confidence due to the simplicity of the change and clear failure prevention
3. Files needing attention:
   • lib/index.ts - should verify that disabling API mode doesn't have unintended side effects

_{1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile}