Skip to content

Conversation

@abhibongale
Copy link

This changes adds Optional metadata field to OpenAI compatible request and response object.

fixes: #3564

This changes adds Optional metadata field to OpenAI compatible request
and response object.

fixes: llamastack#3564
Signed-off-by: Abhishek Bongale <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 2025

✱ Stainless preview builds

This PR will update the llama-stack-client SDKs with the following commit message.

feat: Add metadata field to request and response

Edit this comment to update it. It will appear in the SDK's changelogs.

llama-stack-client-node studio · code · diff

Your SDK built successfully.
generate ❗build ✅lint ✅test ✅

npm install https://pkg.stainless.com/s/llama-stack-client-node/012a118ee248f3baa076145fd6670fafcab20750/dist.tar.gz
New diagnostics (18 note)
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
llama-stack-client-kotlin studio · code · diff

Your SDK built successfully.
generate ❗lint ✅test ❗

New diagnostics (18 note)
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
llama-stack-client-go studio · code · diff

Your SDK built successfully.
generate ❗lint ❗test ❗

go get github.com/stainless-sdks/llama-stack-client-go@885a14006a90a5237fcac50f397a17f24531f9ad
New diagnostics (18 note)
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
llama-stack-client-python studio · code · diff

Your SDK built successfully.
generate ❗build ⏳lint ⏳test ⏳

New diagnostics (18 note)
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.
💡 Schema/ArrayMissingItems: Schema has `type: array` but is missing an items definition
💡 Schema/ObjectHasNoProperties: This schema has neither `properties` nor `additionalProperties` so its type has been interpreted as `unknown`.

This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push.
Last updated: 2025-11-12 14:57:56 UTC

Copy link
Collaborator

@mattf mattf left a comment

Choose a reason for hiding this comment

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

@abhibongale which llm did you use for this?

the linked issue (#3564) correctly states the Responses API supports a metadata param.

this patch adds a metadata param to /v1/completions, /v1/chat/completions, and /v1/embeddings, of which only chat accepts a metadata param. it fails to add metadata to the /v1/responses api.

please use a prompt like -

add a metadata param to the /v1/responses api that accepts a dict[str, str]. the responses api is defined under agents in src/llama_stack/apis/agents/agents.py and implemented by src/llama_stack/providers/inline/agents/meta_reference/agents.py

@abhibongale
Copy link
Author

abhibongale commented Nov 12, 2025

@mattf Thank you — I’ll make the changes. I used Gemini and ChatGPT’s research features to explain and suggest code changes. I guess I’ll use documentation rather than relying solely on LLMs.

@abhibongale abhibongale marked this pull request as draft November 12, 2025 20:34
@mergify
Copy link

mergify bot commented Nov 18, 2025

This pull request has merge conflicts that must be resolved before it can be merged. @abhibongale please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify bot added the needs-rebase label Nov 18, 2025
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. needs-rebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Responses metadata

2 participants