[MehranMazhar]

Requested by Mehran · project thread

📋 Description

Before: main.rs declared mod node itself, so the whole module tree compiled twice (once in the library, once in the binary). That second copy is why 13 #[allow(dead_code)] attributes were needed. The repo also held unused functions, two unused crates, copy-pasted RPC handlers and a few stray files.

After: the binary uses clutch_node::node::… from the library, the tree compiles once, and the build has zero warnings with no allow(dead_code) anywhere. 145 lines added, 305 removed.

What changed:

• Removed unused code: Database::get_keys_by_cf_name, StateUpdate::from_legacy / from_legacy_vec, the async-trait crate and the direct tungstenite 0.24 crate. The node already uses tokio_tungstenite::tungstenite. The lockfile only loses entries.
• websocket.rs: send_transaction and send_raw_transaction share one submit path. The five list_* methods share one list_response, and the three trip lists share one TripFilter. Error texts and response shapes are unchanged.
• Transaction: sign, sign_personal and sign_tron share one sign_bytes. The signed bytes and verification are unchanged.
• balance_effect.rs: the per-account index entries now serialize the records that were already built, instead of building the same struct a second time. The stored bytes are identical.
• blockchain.rs:
  • Two doc comments had drifted onto the wrong function: validate_authorities and drop_intra_block_conflicts. They are back on the right ones.
  • Fixed a broken line continuation that put 14 spaces inside a panic message.
  • Developer mode now writes its JSON dump through one helper.
• Sync handlers: dropped parameters they never used.
• Deleted files:
  • sweep.timestamp, a cargo-sweep marker. It is now in .gitignore.
  • setup-protection.ps1, which only printed manual steps.
  • The root pull_request_template.md, a duplicate of .github/PULL_REQUEST_TEMPLATE.md.
  • examples/README.md, which listed four example files that never existed.

No RPC method, response field, wire or RLP format, signature scheme, state key or stored value changed, so this needs no chain reset. block_reward/reward_recipient in get_block_by_index are left in on purpose: the explorer may still read them.

🎯 Type of Change

• 🔧 Refactoring

🧪 Testing

• Unit tests pass (cargo test -- --test-threads=1: all suites green, the same command CI runs)
• cargo build --all-targets with zero warnings

✅ Checklist

• I have performed a self-review of my code
• My changes generate no new warnings
• New and existing unit tests pass locally with my changes

🤖 Generated with Claude Code

https://claude.ai/code/session_01VPqyiX6mcp9E33S59NwbKo

---

Generated by Claude Code