Skip to content

chore: refactor (chat)completions endpoints to use shared params struct#3761

Merged
ehhuang merged 1 commit intomainfrom
pr3761
Oct 10, 2025
Merged

chore: refactor (chat)completions endpoints to use shared params struct#3761
ehhuang merged 1 commit intomainfrom
pr3761

Conversation

@ehhuang
Copy link
Contributor

@ehhuang ehhuang commented Oct 9, 2025

What does this PR do?

Converts openai(_chat)_completions params to pydantic BaseModel to reduce code duplication across all providers.

Test Plan

CI


Stack created with Sapling. Best reviewed with ReviewStack.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 9, 2025
@ehhuang ehhuang force-pushed the pr3761 branch 10 times, most recently from af63e22 to 68b645c Compare October 10, 2025 16:51
@ehhuang ehhuang changed the title test feat: allows passing through extra_body parameters to inference providers Oct 10, 2025
@ehhuang ehhuang marked this pull request as ready for review October 10, 2025 16:56
@ehhuang ehhuang changed the title feat: allows passing through extra_body parameters to inference providers [WIP - do not review] feat: allows passing through extra_body parameters to inference providers Oct 10, 2025
@ehhuang ehhuang changed the title [WIP - do not review] feat: allows passing through extra_body parameters to inference providers [WIP - do not review] chore: refactor (chat)completions endpoints to use shared params struct Oct 10, 2025
@ehhuang ehhuang force-pushed the pr3761 branch 3 times, most recently from f5b5a2d to 16adbdc Compare October 10, 2025 19:23
@ehhuang ehhuang changed the title [WIP - do not review] chore: refactor (chat)completions endpoints to use shared params struct chore: refactor (chat)completions endpoints to use shared params struct Oct 10, 2025
Copy link
Contributor

@ashwinb ashwinb left a comment

Choose a reason for hiding this comment

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

this is nice

# What does this PR do?


## Test Plan
# What does this PR do?


## Test Plan
@ehhuang ehhuang merged commit 80d58ab into main Oct 10, 2025
34 of 43 checks passed
@ehhuang ehhuang deleted the pr3761 branch October 10, 2025 22:46
franciscojavierarceo pushed a commit to franciscojavierarceo/llama-stack that referenced this pull request Oct 11, 2025
…ct (llamastack#3761)

# What does this PR do?

Converts openai(_chat)_completions params to pydantic BaseModel to
reduce code duplication across all providers.

## Test Plan
CI









---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with
[ReviewStack](https://reviewstack.dev/llamastack/llama-stack/pull/3761).
* llamastack#3777
* __->__ llamastack#3761
jwm4 pushed a commit to jwm4/llama-stack that referenced this pull request Oct 13, 2025
…ct (llamastack#3761)

# What does this PR do?

Converts openai(_chat)_completions params to pydantic BaseModel to
reduce code duplication across all providers.

## Test Plan
CI









---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with
[ReviewStack](https://reviewstack.dev/llamastack/llama-stack/pull/3761).
* llamastack#3777
* __->__ llamastack#3761
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants