fix: collapse nested if statements in signal adapter routing

[l33t0]

Summary

Fixes clippy collapsible_if lint introduced by the Signal adapter merge (#347). Two nested if blocks in send_message_to_another_channel.rs are collapsed into single conditional expressions using let chains.

This is currently breaking CI on all PRs branched from main.

Test plan

• cargo clippy --all-targets -- -D warnings passes clean
• All lib tests pass

---

Note

AI Summary: Collapses two nested if blocks into single conditions using let chains. First block (lines 153-160) combines the adapter check with optional current adapter assignment. Second block (lines 191-207) combines the implicit signal shorthand check with target parsing, reducing nesting and improving readability. Changes are purely structural with no logic modification.

_{Written by Tembo for commit 0faf9c8. This will update automatically on new commits.}

[coderabbitai]

Walkthrough

Refactored conditional logic in Signal message routing to streamline explicit and implicit Signal shorthand handling. Combined nested if structures into single conditionals using if-let binding patterns. Maintains existing behavior for broadcasting, error handling, and logging while reducing code complexity.

Changes

Cohort / File(s)	Summary
Signal Handling Refactoring src/tools/send_message_to_another_channel.rs	Consolidated conditional logic for explicit Signal prefix checking and implicit Signal shorthand parsing. Replaced nested if structures with single conditionals using && let Some(target) = ... binding pattern, eliminating inner branches while preserving functional behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: collapsing nested if statements in signal adapter routing code.
Description check	✅ Passed	The pull request description clearly explains the purpose (fixing clippy lints), the specific changes (collapsing nested if blocks using let chains), and includes test plan validation.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[jamiepine]

Closing — this change is included verbatim in #401 (same hunks in send_message_to_another_channel.rs), so it'll land there. Thanks!