fix resend_message_list_ enqueue

[MrGuin]

Summary by CodeRabbit

• Bug Fixes
  • Improved message re-enqueue behavior during reconnection so remaining messages are reliably processed even when entries are missing or non-contiguous.
• Style
  • Minor logging formatting adjusted (no change to emitted message or behavior).

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Replaced bulk re-enqueue with per-element move-and-enqueue loops in reconnection paths of cc_stream_sender.cpp, and added <utility> include; also a minor log-message formatting change in cc_shard.cpp. No public API changes.

Changes

Cohort / File(s)	Summary
Message re-enqueue refactor tx_service/src/remote/cc_stream_sender.cpp	Replaced try_enqueue_bulk bulk re-enqueue with a per-element loop that moves non-null messages and enqueues them individually in reconnection paths; added #include <utility> for std::move.
Log message formatting tx_service/src/cc/cc_shard.cpp	Minor formatting change splitting a read-lock recovery skip log line across two source lines; no semantic or control-flow change.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Review per-element enqueue semantics vs previous bulk behavior (ordering, atomicity).
• Check for potential race conditions or message loss during reconnect handling.
• Validate correct use of std::move and ownership after moves.

Possibly related PRs

• fix ConnectStreams: break loop if resend message fails #272 — Modifies reconnection resend/enqueue logic and introduces need_reconnect handling that interacts with this PR's re-enqueue changes.

Suggested reviewers

• liunyl

Poem

🐰 I hop through queues with nimble paws,

Moving each message without a clause.
No bulky shove, just one-by-one,
Reconnects fixed, and the stream's well-run.
🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is empty; no description was provided by the author despite the repository template requiring multiple sections including tests, documentation, issue references, and test suite confirmation.	Add a complete pull request description following the repository template, including test confirmation, documentation updates, issue reference, and test suite results.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix resend_message_list_ enqueue' is concise and directly related to the main changes in the pull request—specifically fixing the enqueue behavior in cc_stream_sender.cpp.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix_resend_list

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a1f2f6 and 1b54445.

📒 Files selected for processing (1)

• tx_service/src/cc/cc_shard.cpp (1 hunks)

✅ Files skipped from review due to trivial changes (1)

• tx_service/src/cc/cc_shard.cpp

---

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.}