[stphnwlsh]

Summary

Updating or deleting a review no longer checks that the review exists before it writes. The write goes first, and if no rows changed the handler throws not found. That's one trip to the database instead of two, and it closes the gap where another request could slip in between the check and the write. Update now returns the changed review from Application.

The database gets a few sensible defaults in one migration (AddCollationsAndIndexes):

• queries don't track unless they need to
• author names and movie titles sort without caring about case
• the columns we sort and filter by have indexes
• new rows get version 7 GUIDs, so inserts land at the end of the key index instead of somewhere random

While I was in there, some tidy-ups:

• dropped the [ForeignKey] and [Required] attributes that repeated the fluent configuration
• Page<T>.Items is now an IReadOnlyList<T>
• the container owns MovieReviewsDatabase, so its connection gets closed
• virtual is on every repository method
• Version is now VersionInfo, so it stops tripping over System.Version, and its handler can be tested
• removed usings the implicit usings already cover

The shared layers are the same as LeanMeanCleanApps/CleanMinimalApi#67, with the name swapped.

In the MCP server, write_review uses the review the update hands back instead of reading it again, and delete_review just sends the command. Same results for the client, one less query.

The data-access-ef-core, database-migrations, errors-and-exceptions, paging, unit-tests and test-design skills now match the code.

Type of change

• Breaking change (the generated solution or template parameters change in a way existing users will notice)
• Refactoring or clean-up

Shared layers

• This touches the Application or Infrastructure layers, tests, pipeline or repository setup, so it needs to be carried across all five templates.

Checklist

• dotnet build and dotnet test pass locally with no warnings.
• I added or updated tests, and line coverage stays at or above 95%.
• If this changes how we do something, I updated the matching skill in .agents/skills.

[github-actions]

Build Pipeline

✅ Ready to merge. Every stage passed.

Stage	Result	Details
Build	🟢 Passed	Version 2026.11.03.148-pr-38 · Template package 208 KB
Test	🟢 Passed	235 passed · 0 failed · 0 skipped · Template installs, builds and passes its tests
Coverage	🟢 Passed	100% (772 of 772 lines) · Needs 95%
Static Analysis	🟢 Passed	No open CodeQL alerts
Dependency Review	🟢 Passed	0 vulnerable packages · 0 licence problems
Container Scan	🟢 Passed	Image starts and answers · No open Trivy alerts

_{Commit d74ed24 · Full run}