Skip to content

refactor: OpenRouter model integration and settings#75

Merged
adi-wan-askui merged 4 commits intomainfrom
refactor/openrouter-improvements
Jun 24, 2025
Merged

refactor: OpenRouter model integration and settings#75
adi-wan-askui merged 4 commits intomainfrom
refactor/openrouter-improvements

Conversation

@onur-askui
Copy link
Contributor

  • Renamed OpenRouterGetModel -> OpenRouterModel.
  • Introduced structured output for OpenRouter.
  • Updated the README example to reflect the new model usage.
  • Renamed handler.py to model.py for OpenRouter and Claude.
  • Refactored propmpts for Openrouter and Claude.
  • Introduced ChatCompletionsCreateSettings for managing chat completion settings.
  • Moved settings to to OpenRouterSettings.
  • Added tests for OpenRouter.

- Renamed `OpenRouterGetModel` -> `OpenRouterModel`.
- Introduced structured output for OpenRouter.
- Updated the README example to reflect the new model usage.
- Renamed `handler.py` to `model.py` for OpenRouter and Claude.
- Refactored propmpts for Openrouter and Claude.
- Introduced `ChatCompletionsCreateSettings` for managing chat completion settings.
- Moved settings to to `OpenRouterSettings`.
- Added tests for OpenRouter.
@onur-askui onur-askui force-pushed the refactor/openrouter-improvements branch from 6f4ea12 to 5884799 Compare June 13, 2025 09:32
@onur-askui onur-askui marked this pull request as ready for review June 16, 2025 08:03
@adi-wan-askui adi-wan-askui self-requested a review June 16, 2025 11:41
Copy link
Contributor

@adi-wan-askui adi-wan-askui left a comment

Choose a reason for hiding this comment

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

Solid work :)

adi-wan-askui

This comment was marked as duplicate.

@adi-wan-askui adi-wan-askui merged commit 1254b98 into main Jun 24, 2025
1 check passed
@adi-wan-askui adi-wan-askui deleted the refactor/openrouter-improvements branch June 24, 2025 05:15
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