fix(rpc): address review feedback on debug endpoints

- Document partial-move pattern in get_raw_receipts
- Consolidate duplicate -32602 error code arms in DebugError
- Fix resolve_header error mapping: use Resolve(e) instead of
  BlockNotFound(id) to preserve the distinction between "block not
  found" (-32001) and "storage layer failed" (-32000). The
  resolve_error_message helper already sanitizes DB errors to the
  opaque "server error" string, so no internals are leaked.

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

Author: prestwich

crates/rpc/src/debug/endpoints.rs

@@ -325,7 +325,7 @@ where
 
         let sealed = ctx.resolve_header(BlockId::Number(block_num.into())).map_err(|e| {
             tracing::warn!(error = %e, block_num, "header resolution failed");
-            DebugError::BlockNotFound(id)
+            DebugError::Resolve(e)
         })?;
 
         let Some(sealed) = sealed else {
@@ -387,6 +387,9 @@ where
         let encoded = receipts
             .into_iter()
             .map(|cr| {
+                // Compute bloom before moving logs out. `status` and
+                // `cumulative_gas_used` are Copy, so they remain
+                // accessible after the partial move of `logs`.
                 let logs_bloom = cr.receipt.bloom();
                 let logs: Vec<Log> = cr.receipt.logs.into_iter().map(|l| l.inner).collect();
                 let receipt = Receipt {
@@ -438,7 +441,7 @@ where
 
         let sealed = ctx.resolve_header(BlockId::Number(block_num.into())).map_err(|e| {
             tracing::warn!(error = %e, block_num, "header resolution failed");
-            DebugError::BlockNotFound(id)
+            DebugError::Resolve(e)
         })?;
 
         let Some(sealed) = sealed else {

crates/rpc/src/debug/error.rs

@@ -48,10 +48,9 @@ impl ajj::IntoErrorPayload for DebugError {
         match self {
             Self::Cold(_) | Self::Hot(_) | Self::EvmHalt { .. } => -32000,
             Self::Resolve(r) => crate::eth::error::resolve_error_code(r),
-            Self::InvalidTracerConfig => -32602,
+            Self::InvalidTracerConfig | Self::RlpDecode(_) => -32602,
             Self::Unsupported(_) => -32601,
             Self::BlockNotFound(_) | Self::TransactionNotFound(_) => -32001,
-            Self::RlpDecode(_) => -32602,
             Self::SenderRecovery => -32000,
         }
     }