Skip to content

tidy: align Grpc/Polecat to 6.8 line and fix Swagger title#35

Merged
erikshafer merged 1 commit into
mainfrom
tidy/grpc-polecat-6.8-align-and-swagger
Jun 25, 2026
Merged

tidy: align Grpc/Polecat to 6.8 line and fix Swagger title#35
erikshafer merged 1 commit into
mainfrom
tidy/grpc-polecat-6.8-align-and-swagger

Conversation

@erikshafer

Copy link
Copy Markdown
Owner

Summary

Step 0 foundation tidy ahead of the transport-activation roadmap (Telemetry-first). Three surgical, no-behavior-change items. Full rationale: docs/planning/2026-06-25-state-of-the-repo-transport-and-critterwatch.md §1.4 + §6 Phase 0.

1. Align WolverineFx.Grpc + .Polecat to the 6.8 line

Directory.Packages.props: both bumped 5.38.0 → 6.8.0, joining the rest of the Wolverine suite (core/Http/Marten/ASB/Kafka/SignalR/RuntimeCompilation already at 6.8.0). Removes a cross-major package island sitting directly on the gRPC critical path. Both versions verified on nuget.org. Inert at compile time — no .cs file references either package yet, so restore touches neither.

2. Fix the dev Swagger title

src/CritterCab.Dispatch/Program.cs: "CritterBids API""CritterCab Dispatch API" (was naming the wrong sibling project in the public showcase's Swagger UI).

3. Register (not fix) at-threshold skill debt

docs/skills/DEBT.md: two rows from retro 005 — the marker-interface union return type and the event-triggered automation handler shape — grouped under wolverine-handlers (or a possible new wolverine-marten-automation skill). Registering stops the debt evaporating between sessions; the fix is a future tidy: skills PR. The bundling-rule gap was deliberately not registered — neither retro 005 nor the handoff names a target skill, and the file's convention requires one.

Out of scope (deliberately)

  • Full-suite refresh to 6.14 — a separate later tidy.
  • Fixing the registered skill debt — a separate tidy: skills session.

Verification

  • dotnet restore — clean, no NU1605/NU1608.
  • dotnet build CritterCab.slnx0 warnings, 0 errors.
  • dotnet test11/11 passing (1 smoke + 3×5.1 + 4×5.2 + 3×5.3), Docker-backed Testcontainers.PostgreSql.

Foundation tidy ahead of transport-activation work. Three surgical changes,
no behavior change:

- Directory.Packages.props: bump WolverineFx.Grpc and WolverineFx.Polecat
  5.38.0 -> 6.8.0, joining the rest of the Wolverine suite already pinned at
  6.8.0. Removes a cross-major package island sitting on the gRPC critical
  path. Both versions verified on nuget.org. Inert at compile time — no .cs
  references either package yet — and restore/build/test stay green.
- src/CritterCab.Dispatch/Program.cs: rename the dev Swagger UI title from the
  wrong sibling-project name "CritterBids API" to "CritterCab Dispatch API".
- docs/skills/DEBT.md: register (not fix) two at-threshold skill-debt rows
  surfaced by retro 005 — the marker-interface union return type and the
  event-triggered automation handler shape — grouped under wolverine-handlers
  (or a possible new wolverine-marten-automation skill). The bundling-rule gap
  was deliberately not registered: no source names a target skill.

Deliberately NOT in scope: full-suite refresh to 6.14, fixing the registered
skill debt. Those are separate tidy sessions.

Verification: dotnet restore clean (no NU1605/NU1608); dotnet build 0/0;
dotnet test 11/11 passing (Docker-backed Testcontainers.PostgreSql).
@erikshafer erikshafer self-assigned this Jun 25, 2026
@erikshafer erikshafer merged commit 2cd3387 into main Jun 25, 2026
1 check passed
@erikshafer erikshafer deleted the tidy/grpc-polecat-6.8-align-and-swagger branch June 25, 2026 23:48
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.

1 participant