-
Notifications
You must be signed in to change notification settings - Fork 5k
Add thread message delete operation #50665
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request introduces a delete operation for thread messages in the Persistent Agents SDK, along with the necessary tests and documentation updates.
- Updated the commit hash in the configuration file.
- Added synchronous and asynchronous delete methods in ThreadMessages.cs with corresponding tests.
- Updated sample documentation and assets to reflect the new feature.
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
sdk/ai/Azure.AI.Agents.Persistent/tsp-location.yaml | Updated commit reference. |
sdk/ai/Azure.AI.Agents.Persistent/tests/PersistentAgentsTests.cs | Added tests to verify the delete message operation. |
sdk/ai/Azure.AI.Agents.Persistent/src/Custom/ThreadMessages.cs | Added DeleteMessage and DeleteMessageAsync methods. |
sdk/ai/Azure.AI.Agents.Persistent/samples/Sample20_PersistentAgents_FileSearch_Steaming.md | Updated sample title to include streaming. |
sdk/ai/Azure.AI.Agents.Persistent/samples/README.md | Added a link for the new sample. |
sdk/ai/Azure.AI.Agents.Persistent/assets.json | Updated asset tag revision. |
sdk/ai/Azure.AI.Agents.Persistent/CHANGELOG.md | Documented the addition of the delete operation. |
Comments suppressed due to low confidence (2)
sdk/ai/Azure.AI.Agents.Persistent/src/Custom/ThreadMessages.cs:376
- The diagnostic scope name 'PersistentAgentsClient.DeleteThread' does not match the delete operation for messages. Consider updating it to 'PersistentAgentsClient.DeleteMessage' for clarity and consistency.
using DiagnosticScope scope = ClientDiagnostics.CreateScope("PersistentAgentsClient.DeleteThread");
sdk/ai/Azure.AI.Agents.Persistent/src/Custom/ThreadMessages.cs:402
- Similarly, in the asynchronous method, the diagnostic scope name should reflect the delete operation for messages by using 'PersistentAgentsClient.DeleteMessage' to avoid confusion.
using DiagnosticScope scope = ClientDiagnostics.CreateScope("PersistentAgentsClient.DeleteThread");
d5a407a
to
4208b77
Compare
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
4542cec
to
0b7bae6
Compare
0b7bae6
to
c5e2f6a
Compare
#50607
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.