Skip to content

feat(lease): expose managed fencing authority - #9

Merged
smiggleworth merged 1 commit into
mainfrom
issue-14-lease-authority
Aug 10, 2026
Merged

feat(lease): expose managed fencing authority#9
smiggleworth merged 1 commit into
mainfrom
issue-14-lease-authority

Conversation

@smiggleworth

Copy link
Copy Markdown
Contributor

Summary

  • add an immutable LeaseAuthority snapshot to additive managed-lease callback overloads while preserving existing callbacks and legacy ILeaseClient implementations
  • expose the final ACQUIRE fencing token, including queued grants, while keeping renewal credential rotation private
  • cancel and settle managed callbacks immediately on observed disconnect or renewal loss with deterministic error aggregation
  • document the admission-epoch contract and prove package/public-surface and broker handoff behavior

TDD evidence

  • red: authority tests failed to compile with CS0246/CS1593 before the API existed
  • red: lifecycle regressions failed 3/6 for synchronous pre-await cancellation and synchronous disconnect registration disposal
  • green: focused managed lease and compatibility tests 16/16

Validation

  • dotnet format Fitz.sln --verify-no-changes --no-restore --severity warn
  • dotnet build Fitz.sln -c Release --no-restore (0 warnings/errors)
  • non-integration tests 237/237
  • complete reconnect/integration suite 267/267
  • conformance matrix: TCP/WebSocket x anonymous/JWT, 5/5 tests and 17/17 scenarios per leg
  • packed all three packages and built a consumer from a fresh isolated NuGet cache
  • independent review: no findings after lifecycle and compatibility fixes

Copilot AI lite review requested due to automatic review settings August 10, 2026 23:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@smiggleworth
smiggleworth merged commit 86425a6 into main Aug 10, 2026
12 checks passed
@smiggleworth
smiggleworth deleted the issue-14-lease-authority branch August 10, 2026 23:47
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.

2 participants