.NET: Fix case-sensitive property mismatch in CosmosChatHistoryProvider queries

[Copilot]

Motivation and Context

SQL queries in CosmosChatHistoryProvider used c.Type (uppercase) while the document model defines [JsonProperty("type")] (lowercase). Cosmos DB's case-sensitive queries failed to match any documents, causing GetMessageCountAsync() and ClearMessagesAsync() to always return 0.

Description

Changed queries:

• GetMessageCountAsync() - line 555: c.Type → c.type
• ClearMessagesAsync() - line 585: c.Type → c.type

Added tests:

• GetMessageCountAsync_WithMessages_ShouldReturnCorrectCountAsync
• GetMessageCountAsync_WithNoMessages_ShouldReturnZeroAsync
• ClearMessagesAsync_WithMessages_ShouldDeleteAndReturnCountAsync
• ClearMessagesAsync_WithNoMessages_ShouldReturnZeroAsync

Test updates:

• Fixed IDE0090 style violations by using target-typed new expressions (new(...) instead of new ChatMessage(...))
• Updated tests to work with current API: constructor now uses state factory pattern, methods require AgentSession parameter

Tests follow existing patterns (skip when Cosmos DB Emulator unavailable).

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the Contribution Guidelines
• All unit tests pass, and I have added new tests where possible
• Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>.NET: [Bug]: CosmosChatMessageStore methods GetMessageCountAsync and ClearMessagesAsync always return 0 due to case-sensitivity in query</issue_title>
<issue_description>### Description

What happened?
When using CosmosChatMessageStore, calls to GetMessageCountAsync() and ClearMessagesAsync() always return 0, even when there are messages present in the container that can be retrieved via other methods.

What did you expect to happen?
GetMessageCountAsync() should return the actual number of messages for the given conversation, and ClearMessagesAsync() should delete the messages and return the count of removed items.

Steps to reproduce the issue:

1. Initialize CosmosChatMessageStore with a valid connection string and container.
2. Add messages to the store (or verify they exist).
3. Call GetMessageCountAsync().
4. Observe that it returns 0 regardless of the actual data.

Code Sample

var cosmosProvider = new CosmosChatMessageStore(
    "AccountEndpoint=xxxx;AccountKey=xxxx",
    "my-databaseid",
    "my-container",
    "my-conversationId");

// Retrieval works fine
var messages = await cosmosProvider.InvokingAsync(new ChatMessageStore.InvokingContext([]));

// Bug: Always returns 0
var count = await cosmosProvider.GetMessageCountAsync();

// Bug: No messages are deleted and returns 0
var removedMessagesCount1 = await cosmosProvider.ClearMessagesAsync();

Error Messages / Stack Traces

No exception is thrown; the methods silently fail to match any documents due to an incorrect property name in the underlying Cosmos DB SQL query.

Package Versions

Microsoft.Agents.AI.CosmosNoSql: 1.0.0-preview.260121.1

.NET Version

NET 10

Additional Context

The issue appears to be caused by a case-sensitivity mismatch in the query string.
In CosmosChatMessageStore.cs, the query uses c.Type = @type instead of c.type = @type (lowercase 't').

• Current NuGet source: Line 555 in CosmosChatMessageStore.cs
• Main branch (Migrated to CosmosChatHistoryProvider): The bug seems to persist in the new implementation here: Line 555 in CosmosChatHistoryProvider.cs
  </issue_description>

<agent_instructions>Analyse this error and add your findings to the issue</agent_instructions>

Comments on the Issue (you are @copilot in this section)

• Fixes .NET: [Bug]: CosmosChatMessageStore methods GetMessageCountAsync and ClearMessagesAsync always return 0 due to case-sensitivity in query #3460

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[westey-m]

@copilot, the build is failing with style issues, please fix.

/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs(897,13): error IDE0090: 'new' expression can be simplified [/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/Microsoft.Agents.AI.CosmosNoSql.UnitTests.csproj]
/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs(898,13): error IDE0090: 'new' expression can be simplified [/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/Microsoft.Agents.AI.CosmosNoSql.UnitTests.csproj]
/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs(899,13): error IDE0090: 'new' expression can be simplified [/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/Microsoft.Agents.AI.CosmosNoSql.UnitTests.csproj]
/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs(897,13): error IDE0090: 'new' expression can be simplified [/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/Microsoft.Agents.AI.CosmosNoSql.UnitTests.csproj]
/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs(898,13): error IDE0090: 'new' expression can be simplified [/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/Microsoft.Agents.AI.CosmosNoSql.UnitTests.csproj]
/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs(899,13): error IDE0090: 'new' expression can be simplified [/app/dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/Microsoft.Agents.AI.CosmosNoSql.UnitTests.csproj]

[Copilot]

@copilot, the build is failing with style issues, please fix.

...

Fixed in 02a65ac. Applied target-typed new expressions and updated tests to work with the new API that was introduced after the merge.

[Copilot]

Pull request overview

Fixes a Cosmos DB SQL case-sensitivity bug in .NET where queries used c.Type even though the stored JSON field is type, causing GetMessageCountAsync() and ClearMessagesAsync() to return 0 incorrectly.

Changes:

• Update CosmosChatHistoryProvider SQL queries to use c.type (matching [JsonProperty("type")]).
• Add Cosmos emulator-backed unit tests covering message counting and clearing scenarios.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
dotnet/src/Microsoft.Agents.AI.CosmosNoSql/CosmosChatHistoryProvider.cs	Fixes case-sensitive Cosmos SQL queries for count and delete operations by querying c.type.
dotnet/tests/Microsoft.Agents.AI.CosmosNoSql.UnitTests/CosmosChatHistoryProviderTests.cs	Adds new emulator-skippable tests validating correct counts and deletes for empty/non-empty conversations.