feat: update ConsumersService to check for queue prefetch changes in …

[mmalkhatib]

…consumer configuration

Summary by CodeRabbit

• Chores

  • Updated assembly version to 8.0.0.0 across core libraries.
  • Updated primitive types dependency to version 8.1.3.

• Bug Fixes

  • Improved consumer refresh detection to properly identify queue prefetch setting changes and apply updates accordingly.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 136eaf41-a98d-4516-abd3-bf10db6d6108

📥 Commits

Reviewing files that changed from the base of the PR and between 20104c1 and 76ec750.

📒 Files selected for processing (3)

• SW.Bus.RabbitMqExtensions/SW.Bus.RabbitMqExtensions.csproj
• SW.Bus/ConsumersService.cs
• SW.Bus/SW.Bus.csproj

---

📝 Walkthrough

Walkthrough

The PR updates assembly version metadata in two project files and adjusts consumer restart logic in ConsumersService to trigger on QueuePrefetch changes instead of ConsumerArgs dictionary equality. It also bumps the SimplyWorks.PrimitiveTypes NuGet dependency from 8.1.2 to 8.1.3.

Changes

Cohort / File(s)	Summary
Assembly Version Updates SW.Bus.RabbitMqExtensions/SW.Bus.RabbitMqExtensions.csproj, SW.Bus/SW.Bus.csproj	Added explicit <AssemblyVersion>8.0.0.0</AssemblyVersion> metadata to both project files.
NuGet Dependency Update SW.Bus.RabbitMqExtensions/SW.Bus.RabbitMqExtensions.csproj	Updated SimplyWorks.PrimitiveTypes from 8.1.2 to 8.1.3.
Consumer Restart Logic SW.Bus/ConsumersService.cs	Modified RefreshConsumers() to trigger consumer restart on QueuePrefetch changes (prefetchChanged) instead of ConsumerArgs dictionary equality (argsChanged). The DictionariesEqual() function is now unused.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: improve error logging and streamline consumer prefetch handling… #31: Modifies consumer-reconfiguration logic for QueuePrefetch/BasicQos updates and restart conditions
• Enhance consumer management with extended options and statistics #27: Adjusts RefreshConsumers() change-detection logic and leaves DictionariesEqual unused
• Muhannad/dynamic consumer props #21: Updates consumer-change detection to consider QueuePrefetch instead of consumer-args equality

Suggested reviewers

• samerzughul

Poem

🐰 Assembly versions align with grace,
PrefetchQueues now set the pace,
Dependencies refresh anew,
Consumer restarts, oh so true!
A rabbit's hop through RabbitMQ. 🐇

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch muhannad/fix-prefetch-and-assembly

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}