Skip to content

Yet More CritterWatch Support#2353

Merged
jeremydmiller merged 2 commits intomainfrom
cw
Mar 25, 2026
Merged

Yet More CritterWatch Support#2353
jeremydmiller merged 2 commits intomainfrom
cw

Conversation

@jeremydmiller
Copy link
Copy Markdown
Member

Summary

  • Post RecordEffectiveTime to CritterWatch accumulation pipelineMessageSucceeded() and MessageFailed() in WolverineRuntime.Tracking.cs now post RecordEffectiveTime to the metrics accumulator, enabling the wolverine-effective-time metric to flow through the CritterWatch observer pipeline to the UI
  • Add IEventSubscriptionAgent interface — New interface in Wolverine.Runtime.Agents that both Wolverine.Marten and Wolverine.Polecat EventSubscriptionAgent classes implement. Allows CritterWatch to issue Rebuild commands without coupling to a specific event store provider

Test plan

  • CritterWatch effective_time_metrics integration test verifies non-zero effective time data flows end-to-end
  • CritterWatch projection_commands integration test verifies Pause/Restart/Rebuild relay through IEventSubscriptionAgent
  • Both Wolverine.Marten and Wolverine.Polecat build successfully with the interface change

🤖 Generated with Claude Code

jeremydmiller and others added 2 commits March 24, 2026 17:22
MessageSucceeded() and MessageFailed() now post RecordEffectiveTime to
the metrics accumulator when running in CritterWatch or Hybrid metrics
mode. Previously effective time was only recorded to
System.Diagnostics.Metrics histograms but not to the CritterWatch
pipeline, causing zero effective-time data in the CritterWatch UI.

Follows the same pattern already used by Sent(), Received(), and
MovedToErrorQueue().

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…mands

Extract IEventSubscriptionAgent from IAgent to provide a provider-agnostic
interface for projection rebuild operations. Both Wolverine.Marten and
Wolverine.Polecat EventSubscriptionAgent classes now implement this
interface, allowing CritterWatch to issue Rebuild commands without
coupling to a specific event store provider.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jeremydmiller jeremydmiller merged commit 5836098 into main Mar 25, 2026
19 of 20 checks passed
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