[Roger Barreto (rogerbarreto)]

Motivation and Context

Structured data mutation functions need a clear boundary between model-provided request data and Entity Framework persistence entities. Applications also need to control how update and delete targets are resolved before a mutation runs.

Description

• Makes the convenience plugin factory query-only by default and rejects mutation operations that lack explicit application configuration.
• Keeps the legacy direct-entity mutation overloads for compatibility, but makes them fail closed.
• Adds safe mutation overloads that use separate request types, explicit insert/update mapping, and application-scoped tracked-entity resolution for update and delete.
• Updates the structured data sample to demonstrate explicit safe function composition.
• Adds regression coverage for request mapping, scoped resolution, detached entities, and the fail-closed legacy surface.

Validation:

• dotnet build dotnet\src\Plugins\Plugins.StructuredData.EntityFramework\Plugins.StructuredData.EntityFramework.csproj -c Debug --tl:off --warnaserror --no-restore — passed for net10.0, net8.0, and net462 with 0 warnings and 0 errors.
• dotnet build dotnet\src\Plugins\Plugins.UnitTests\Plugins.UnitTests.csproj -c Debug -f net10.0 --tl:off --warnaserror --no-restore — passed with 0 warnings and 0 errors.
• dotnet test dotnet\src\Plugins\Plugins.UnitTests\Plugins.UnitTests.csproj -c Debug -f net10.0 --no-build --no-restore -v Normal --logger trx — 592 passed, 15 skipped, 0 failed.
• dotnet build dotnet\samples\Demos\StructuredDataPlugin\StructuredDataPlugin.csproj -c Debug --tl:off --warnaserror --no-restore — passed with 0 warnings and 0 errors.
• Scoped dotnet format --verify-no-changes — passed natively and with mcr.microsoft.com/dotnet/sdk:10.0 for all affected projects.

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
• All unit tests pass, and I have added new tests where possible
• I didn't break anyone 😄

[Copilot]

🟡 Changes recommended

Update and delete operations resolve products by non-unique names, so valid duplicate-name products cause exceptions and cannot be targeted individually without using their unique IDs.

1 open issue

• Use product ID to target updates and deletions · New

What changed in this PR

This PR hardens .NET structured-data mutations by separating model-provided requests from Entity Framework entities and requiring application-controlled mapping and target resolution.

Changes:

• Makes the convenience factory query-only and disables legacy direct-entity mutation overloads.
• Adds explicit request-based mutation APIs and updates the sample to compose them.
• Adds regression coverage for mapping, scoped resolution, tracked entities, and fail-closed behavior.

File	Description
dotnet/​src/​Plugins/​Plugins.UnitTests/​StructuredData/​StructuredDataServiceTests.cs	Adds mutation-boundary regression tests and supporting fixtures.
dotnet/​src/​Plugins/​Plugins.StructuredData.EntityFramework/​StructuredDataServiceExtensions.cs	Adds request-based mutation overloads and disables legacy overloads.
dotnet/​src/​Plugins/​Plugins.StructuredData.EntityFramework/​StructuredDataPluginFactory.cs	Restricts convenience composition to queries and rejects mutations.
dotnet/​src/​Plugins/​Plugins.StructuredData.EntityFramework/​StructuredDataOperation.cs	Changes default operations to selection only.
dotnet/​samples/​Demos/​StructuredDataPlugin/​Program.cs	Explicitly composes query and application-configured mutation functions.
dotnet/​samples/​Demos/​StructuredDataPlugin/​Product.cs	Adds separate create, update, and delete request types.

_{Review effort: Balanced}