chore: sync formating of wallet/rpc code with Bitcoin Core

[knst]

Issue being fixed or feature implemented

Doing backports for any RPC-related is struggle, because at some points formatting of files got different; due to missing changes from bitcoin#14726, bitcoin#25029, bitcoin#17809 and multiple other occasions.

What was done?

Manually sync formatting for wallet/rpc with Bitcoin Core v25.99 (commit bbbf89a)

See also: #7129
These changes are split to 2 PR to reduce amount of conflicts and rebases in case of any backport are done due to huge diff size.

Hint: use 'ignore whitespace' feature to review this PR.

How Has This Been Tested?

On top of this PR finally succeed to backport bitcoin#24118 for which I had lots of troubles to resolve conflicts.

Breaking Changes

N/A

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated relevant unit/integration/functional/e2e tests
• I have made corresponding changes to the documentation
• I have assigned this pull request to a milestone

[github-actions]

⚠️ Potential Merge Conflicts Detected

This PR has potential conflicts with the following open PRs:

• backport: auto 20251204 #7048 - backport: auto 20251204
• backport: bitcoin#24836, #25379, #25773, #25957, #25986, #26158, #26294, #26349, #26388, #26404, #26424, #26625, #26920, #27350 #7085 - backport: bitcoin#24836, #25379, #25773, #25957, #25986, #26158, #26294, #26349, #26388, #26404, #26424, #26625, #26920, #27350

Please coordinate with the authors of these PRs to avoid merge conflicts.

[coderabbitai]

Walkthrough

This PR restructures RPC help definitions and schemas across the wallet subsystem (addresses, backup, coins, encrypt, spend, transactions, and wallet RPC handlers). The changes reorganize RPCHelpMan declarations to provide more explicit argument definitions, expand result schemas with detailed field specifications, and reformat help text for consistency. No functional logic or runtime behavior is modified—the core RPC implementations remain unchanged.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• dashpay/dash#6858: Directly overlaps with modifications to src/wallet/rpc/addresses.cpp, specifically RPCHelpMan definitions for getaddressinfo and listaddressgroupings with related formatting and schema updates.

Suggested reviewers

• PastaPastaPasta
• UdjinM6
• kwvg

🚥 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 primary change: formatting synchronization of wallet/rpc code with Bitcoin Core upstream. It is concise and directly related to the changeset.
Description check	✅ Passed	The description clearly explains the purpose (sync formatting with Bitcoin Core), motivation (simplify backports), what was done (applied formatting changes), and testing approach (successful backport). It is directly related to the changeset.

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

✨ Finishing touches

• 📝 Generate docstrings

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

[UdjinM6]

utACK d46466f