Skip to content

feat: NetEvolve.Pulse.AspNetCore — map commands and queries to Minimal API endpoints#156

Open
Copilot wants to merge 5 commits intomainfrom
copilot/add-minimal-api-route-integration
Open

feat: NetEvolve.Pulse.AspNetCore — map commands and queries to Minimal API endpoints#156
Copilot wants to merge 5 commits intomainfrom
copilot/add-minimal-api-route-integration

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 28, 2026

  • Replace string httpMethod with CommandHttpMethod enum in MapCommand overloads
  • Create CommandHttpMethod.cs with Post, Put, Patch, Delete values
  • Update unit and integration tests to use the enum
  • Update NetEvolve.Pulse.AspNetCore.csproj <Description> to mention CommandHttpMethod enum
  • Update README.md — Quick Start, Before/After snippets, per-method-type sections, new CommandHttpMethod reference table, Chaining and Grouping examples all reflect the enum

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Add minimal API integration for mediator commands and queries feat: NetEvolve.Pulse.AspNetCore — map commands and queries to Minimal API endpoints Mar 28, 2026
Copilot AI requested a review from samtrion March 28, 2026 07:43
@samtrion samtrion force-pushed the copilot/add-minimal-api-route-integration branch from 5ad7fba to 0b5ffed Compare March 28, 2026 08:11
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.74%. Comparing base (19680ce) to head (955707e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #156      +/-   ##
==========================================
+ Coverage   86.12%   86.74%   +0.61%     
==========================================
  Files          42       43       +1     
  Lines        1312     1350      +38     
  Branches      123      127       +4     
==========================================
+ Hits         1130     1171      +41     
+ Misses        145      142       -3     
  Partials       37       37              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@samtrion samtrion marked this pull request as ready for review March 28, 2026 08:28
@samtrion samtrion requested a review from a team as a code owner March 28, 2026 08:28
@samtrion samtrion requested a review from Spacemonkay March 28, 2026 08:28
@samtrion samtrion force-pushed the copilot/add-minimal-api-route-integration branch from 0b5ffed to 16acf47 Compare March 28, 2026 08:28
…mand overloads

Agent-Logs-Url: https://github.com/dailydevops/pulse/sessions/f0b45a84-51f2-443f-bdf6-64b2a87ae5fc

Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
<Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<TargetFrameworks>$(_ProjectTargetFrameworks)</TargetFrameworks>
<Description>ASP.NET Core Minimal API integration for the Pulse CQRS mediator. Provides IEndpointRouteBuilder extension methods that map mediator commands and queries directly to HTTP endpoints, eliminating boilerplate endpoint handlers that only forward to IMediator. Supports POST endpoints for commands (with typed response or 204 No Content for void commands) and GET endpoints for queries, with full CancellationToken propagation and OpenAPI compatibility.</Description>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update the whole documentation @copilot to the latest implementation

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in commit 4450a28. Both the <Description> in the .csproj and the README.md have been updated to reflect the CommandHttpMethod enum. Changes include: updated Quick Start and Before/After code samples, revised per-overload sections showing Post/Put/Patch/Delete examples, a new CommandHttpMethod Enum reference table directing readers to MapQuery for read-only operations, and updated Chaining/Grouping examples.

Copilot AI requested a review from samtrion March 28, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Minimal API integration — map commands and queries to HTTP endpoints (NetEvolve.Pulse.AspNetCore)

2 participants