Skip to content

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

nick863
Copy link
Member

@nick863 nick863 commented Jun 17, 2025

#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.

@Copilot Copilot AI review requested due to automatic review settings June 17, 2025 18:54
Copy link
Contributor

@Copilot Copilot AI left a 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");

Copy link

github-actions bot commented Jun 17, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.AI.Agents.Persistent

@nick863 nick863 force-pushed the nirovins/add_message_delete branch 2 times, most recently from 4542cec to 0b7bae6 Compare June 18, 2025 01:01
@nick863 nick863 force-pushed the nirovins/add_message_delete branch from 0b7bae6 to c5e2f6a Compare June 23, 2025 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants