fix(rpc): review fixes — correct error codes, remove dead code, deduplicate

- Use `?` for resolve_block_id in eth endpoints so hash-not-found
  returns -32001 instead of -32000
- Use `?` for cold storage errors so they route through #[from] impls
  with sanitized "server error" messages instead of leaking internals
- Remove unused EvmRevert variants from DebugError and SignetError
- Add EthError::task_panic() to replace 20+ inline copies
- Fix resolve_error_code to be const fn (clippy)
- Map decode_2718 errors to InvalidParams (-32602) instead of Internal
- Remove tracked docs/ files from git, widen .gitignore to docs/

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

Author: prestwich

.gitignore

@@ -2,4 +2,4 @@
 .DS_Store
 Cargo.lock
 .idea/
-docs/superpowers/
+docs/

crates/rpc/src/debug/error.rs

@@ -1,9 +1,6 @@
 //! Error types for the debug namespace.
 
-use alloy::{
-    eips::BlockId,
-    primitives::{B256, Bytes},
-};
+use alloy::{eips::BlockId, primitives::B256};
 use std::borrow::Cow;
 
 /// Errors that can occur in the `debug` namespace.
@@ -24,12 +21,6 @@ pub enum DebugError {
     /// Unsupported tracer type.
     #[error("unsupported: {0}")]
     Unsupported(&'static str),
-    /// EVM execution reverted with output data.
-    #[error("execution reverted")]
-    EvmRevert {
-        /// ABI-encoded revert reason bytes.
-        output: Bytes,
-    },
     /// EVM execution halted.
     #[error("execution halted: {reason}")]
     EvmHalt {
@@ -45,15 +36,14 @@ pub enum DebugError {
 }
 
 impl ajj::IntoErrorPayload for DebugError {
-    type ErrData = Bytes;
+    type ErrData = ();
 
     fn error_code(&self) -> i64 {
         match self {
             Self::Cold(_) | Self::Hot(_) | Self::EvmHalt { .. } => -32000,
             Self::Resolve(r) => crate::eth::error::resolve_error_code(r),
             Self::InvalidTracerConfig => -32602,
             Self::Unsupported(_) => -32601,
-            Self::EvmRevert { .. } => 3,
             Self::BlockNotFound(_) | Self::TransactionNotFound(_) => -32001,
         }
     }
@@ -64,17 +54,13 @@ impl ajj::IntoErrorPayload for DebugError {
             Self::Resolve(r) => crate::eth::error::resolve_error_message(r),
             Self::InvalidTracerConfig => "invalid tracer config".into(),
             Self::Unsupported(msg) => format!("unsupported: {msg}").into(),
-            Self::EvmRevert { .. } => "execution reverted".into(),
             Self::EvmHalt { reason } => format!("execution halted: {reason}").into(),
             Self::BlockNotFound(id) => format!("block not found: {id}").into(),
             Self::TransactionNotFound(h) => format!("transaction not found: {h}").into(),
         }
     }
 
     fn error_data(self) -> Option<Self::ErrData> {
-        match self {
-            Self::EvmRevert { output } => Some(output),
-            _ => None,
-        }
+        None
     }
 }