Skip to content

feat(convo): add new fields to conversation api#902

Merged
sicoyle merged 4 commits into
dapr:mainfrom
sicoyle:feat-convo-api-new-fields
Feb 6, 2026
Merged

feat(convo): add new fields to conversation api#902
sicoyle merged 4 commits into
dapr:mainfrom
sicoyle:feat-convo-api-new-fields

Conversation

@sicoyle

@sicoyle sicoyle commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Description

Python SDK client updates for conversation API new fields for response format, prompt cache, usage metrics, and model return on conversation api. I ran the script to update the proto grpc code generator and added tests for the field additions.

dapr/dapr#9241

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@sicoyle
sicoyle requested review from a team as code owners February 3, 2026 15:43
Signed-off-by: Samantha Coyle <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>
@codecov

codecov Bot commented Feb 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.56098% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 86.58%. Comparing base (bffb749) to head (22d12c8).
⚠️ Report is 66 commits behind head on main.

Files with missing lines Patch % Lines
dapr/clients/grpc/client.py 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #902      +/-   ##
==========================================
- Coverage   86.63%   86.58%   -0.05%     
==========================================
  Files          84      103      +19     
  Lines        4473     6979    +2506     
==========================================
+ Hits         3875     6043    +2168     
- Misses        598      936     +338     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@CasperGN CasperGN left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor requests to move TODO's out of doc strings

Comment thread dapr/proto/runtime/v1/ai_pb2.pyi
Comment thread dapr/proto/runtime/v1/ai_pb2.pyi
Comment thread dapr/proto/runtime/v1/ai_pb2.pyi
@sicoyle
sicoyle merged commit 40b5734 into dapr:main Feb 6, 2026
13 checks passed
@sicoyle
sicoyle deleted the feat-convo-api-new-fields branch February 6, 2026 18:56
@nelson-parente nelson-parente modified the milestones: v1.16, v1.17 Feb 12, 2026
sicoyle added a commit to sicoyle/python-sdk that referenced this pull request Mar 5, 2026
* feat(convo): add new fields to conversation api

Signed-off-by: Samantha Coyle <[email protected]>

* fix: update proto/grpc code generator and add more tests

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* style: tox -e type fixes

Signed-off-by: Samantha Coyle <[email protected]>

---------

Signed-off-by: Samantha Coyle <[email protected]>
sicoyle added a commit to sicoyle/python-sdk that referenced this pull request Mar 5, 2026
* feat(convo): add new fields to conversation api

Signed-off-by: Samantha Coyle <[email protected]>

* fix: update proto/grpc code generator and add more tests

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* style: tox -e type fixes

Signed-off-by: Samantha Coyle <[email protected]>

---------

Signed-off-by: Samantha Coyle <[email protected]>
sicoyle added a commit that referenced this pull request Mar 6, 2026
* Workflow: remove sleeps from example (#894)

* Workflow: remove sleeps from example

Remove arbitrary sleeps from workflow examples

Signed-off-by: joshvanl <[email protected]>

* tox -e ruff

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>

* Workflow versioning (#893)

Signed-off-by: Albert Callarisa <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>

* fix: signal when dt reader stream is ready within wf client start call (#901)

* fix: signal when dt reader stream is ready within wf client start call

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* fix: enable configurability + lint fixes

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): add tests and fix bug

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): add more tests

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): add another test

Signed-off-by: Samantha Coyle <[email protected]>

* test: add even more tests for build to pass lol

Signed-off-by: Samantha Coyle <[email protected]>

---------

Signed-off-by: Samantha Coyle <[email protected]>

* feat(convo): add new fields to conversation api (#902)

* feat(convo): add new fields to conversation api

Signed-off-by: Samantha Coyle <[email protected]>

* fix: update proto/grpc code generator and add more tests

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* style: tox -e type fixes

Signed-off-by: Samantha Coyle <[email protected]>

---------

Signed-off-by: Samantha Coyle <[email protected]>

* add venv (#921)

Signed-off-by: Cassandra Coyle <[email protected]>
Co-authored-by: Sam <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>

* fix: Align Workflow Multi App Naming Convention (#932)

Signed-off-by: Matheus André <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>

* fix(bug): workflow client missing close function to pass to durable task (#889)

* chore(deps): bump dapr dep to 1.17.0.dev

Signed-off-by: Casper Nielsen <[email protected]>

* chore(deps): bump durabletask-dapr to 0.2.0a15

Signed-off-by: Casper Nielsen <[email protected]>

* feat: ensure taskhubgrpcclient expose .close() method for clients to pass down a close call to durabletask

Signed-off-by: Casper Nielsen <[email protected]>

---------

Signed-off-by: Casper Nielsen <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>

* fix: signal when dt reader stream is ready within wf client start call (#901)

* fix: signal when dt reader stream is ready within wf client start call

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* fix: enable configurability + lint fixes

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): add tests and fix bug

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): add more tests

Signed-off-by: Samantha Coyle <[email protected]>

* style: appease linter

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): add another test

Signed-off-by: Samantha Coyle <[email protected]>

* test: add even more tests for build to pass lol

Signed-off-by: Samantha Coyle <[email protected]>

---------

Signed-off-by: Samantha Coyle <[email protected]>

* style: rm prefix of == APP ==

Signed-off-by: Samantha Coyle <[email protected]>

* fix(build): grab latest 1.17 runtime

Signed-off-by: Samantha Coyle <[email protected]>

* fix: exclude prereleases

Signed-off-by: Samantha Coyle <[email protected]>

* fix: update readmes for release branch

Signed-off-by: Samantha Coyle <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>
Signed-off-by: Albert Callarisa <[email protected]>
Signed-off-by: Cassandra Coyle <[email protected]>
Signed-off-by: Matheus André <[email protected]>
Signed-off-by: Casper Nielsen <[email protected]>
Co-authored-by: Josh van Leeuwen <[email protected]>
Co-authored-by: Albert Callarisa <[email protected]>
Co-authored-by: Cassie Coyle <[email protected]>
Co-authored-by: Matheus André <[email protected]>
Co-authored-by: Casper Nielsen <[email protected]>
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.

3 participants