@drunkcoding/dknet-implementation-skills 14.1.1 → 14.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@drunkcoding/dknet-implementation-skills",
-  "version": "14.1.1",
+  "version": "14.2.1",
   "description": "Agent skills (Claude Code plugin + Agent Skills standard) that teach coding agents how to build features on solutions generated from DKNet.Minimal.Template. Install with `npx skills add baoduy/DKNet.Templates`.",
   "keywords": [
     "dknet",

package/plugin/.claude-plugin/plugin.json

@@ -3,7 +3,7 @@
   "name": "dknet-minimal",
   "displayName": "DKNet Minimal Template",
   "description": "Skills and subagents that teach an agent how to build features on the DKNet.Minimal.Template (.NET 10, vertical-slice DDD/CQRS): endpoints (hand-mapped, generated CRUD, generic helpers), actions and queries, FluentValidation + Mapster DTOs, domain entities and static seeding, SlimMessageBus internal and Azure Service Bus events, auth/ownership and every platform flag — plus slash workflows that scaffold a slice end-to-end.",
-  "version": "14.1.1",
+  "version": "14.2.1",
   "author": {
     "name": "baoduy",
     "email": "baoduy2412@gmail.com",

package/plugin/skills/dknet-crud/SKILL.md

@@ -321,6 +321,7 @@ of the five the generator genuinely cannot express (`dknet-feature-lifecycle` §
    `Result.Ok()`.
 4. Wire each into `<Feature>V1Endpoint.cs` with a literal `Map*` call (`dknet-endpoint`) so
    DataAnnotations validation, if any, is actually enforced.
+5. Keep `ApiEndpoints/<YourApp>.Client` in step by adding the matching client methods (`I{Plural}Client`, contracts under `Contracts/`) in the same change.
 
 **mode=auto** (mirror `AutomatedSample/Product`):
 
@@ -332,6 +333,7 @@ of the five the generator genuinely cannot express (`dknet-feature-lifecycle` §
 4. For an operation that must write more than one aggregate, or must reject a repeat call as a
    domain failure rather than a `200`, write a `...Command` (not `...Request`) and drop the
    generated route for that member with `CrudMapOptions.Exclude("MethodName")` in the endpoint.
+5. Keep `ApiEndpoints/<YourApp>.Client` in step by adding the matching client methods for every route the endpoint maps, in the same change.
 
 ## Validation checklist
 

package/plugin/skills/dknet-endpoint/SKILL.md

@@ -404,6 +404,7 @@ skill.
    through `IMessageBus`.
 4. Add `.RequiredIdempotentKey()` to the create route if a duplicate submit must not create two rows.
 5. Add `.WithDescription(...)`/`.Produces<T>(...)` to every route.
+6. Keep `ApiEndpoints/<YourApp>.Client` in step by adding the matching client method (one Refit method per route, on `I{Plural}Client`) in the same change — `<YourApp>.App.Tests/Client/ClientEndpointParityTests.cs` fails on any endpoint without one.
 
 ### `mode=auto`
 
@@ -416,6 +417,7 @@ skill.
    `o.Configure(...)` only for a route whose scope its HTTP method cannot decide.
 4. For anything the generator can't express, hand-map it below the composite call, dropping the
    generated route it replaces via `o.Exclude(...)` when one exists.
+5. Keep `ApiEndpoints/<YourApp>.Client` in step by adding the matching client method for every generated and hand-mapped route (on `I{Plural}Client`) in the same change — the parity test fails on any drift.
 
 ## Verification
 

package/plugin/skills/dknet-feature/SKILL.md

@@ -102,6 +102,8 @@ the package version supports it, dropping to `o.Configure(...)`/`.RequireAuthori
 behind the `FeatureOptions.RequireAuthorization` guard — only for a route whose HTTP method cannot
 decide its scope.
 
+Client, either mode: keep `ApiEndpoints/<YourApp>.Client` in step by adding the matching client methods — one Refit method per route on a new `I{Plural}Client`, its contracts under `Contracts/` — in the same change; `<YourApp>.App.Tests/Client/ClientEndpointParityTests.cs` must stay green.
+
 Build green.
 
 ### Phase 5 — Unit/integration tests

package/plugin/skills/dknet-feature-remove/SKILL.md

@@ -62,10 +62,11 @@ intermediate build failure points at real coupling rather than at the ordering.
    generated `*.feature.cs`, plus `Steps/`).
 3. Tests — `<YourApp>.App.Tests/Unit/<Feature>/` and `<YourApp>.App.Tests/Integration/<Feature>/`.
    Also grep `<YourApp>.App.Tests/Architecture/` — a convention test may assert on this feature by name.
-4. Api — `<YourApp>.Api/ApiEndpoints/<Feature>/`.
-5. AppServices — `<YourApp>.AppServices/<Feature>/`.
-6. Infra — `<YourApp>.Infra/Features/<Feature>/`.
-7. Domains — `<YourApp>.Domains/Features/<Feature>/`.
+4. Client — keep `<YourApp>.Client` in step by removing the matching client methods (the feature's `I{Plural}Client` and the `Contracts/` only it uses) in the same change. A removed sample also takes its named tests in `<YourApp>.App.Tests/Client/` with it — the drift ATs in `Client/ClientEndpointParityTests.cs` name `ManualSample` and `AutomatedSample` routes.
+5. Api — `<YourApp>.Api/ApiEndpoints/<Feature>/`.
+6. AppServices — `<YourApp>.AppServices/<Feature>/`.
+7. Infra — `<YourApp>.Infra/Features/<Feature>/`.
+8. Domains — `<YourApp>.Domains/Features/<Feature>/`.
 
 ## Phase 2 — Out-of-folder touchpoints
 

package/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "dknet-minimal",
   "description": "Agent skills and agents for building vertical-slice DDD/CQRS features on solutions generated from DKNet.Minimal.Template (.NET 10, EF Core, .NET Aspire, SlimMessageBus, FluentValidation, Mapster, Reqnroll BDD).",
-  "version": "14.1.1",
+  "version": "14.2.1",
   "author": {
     "name": "baoduy"
   },