fix(backend): resolve clippy lints

[kyscott18]

Replacement for #101, which GitHub marked as merged into the #100 branch before the changes landed on main.\n\nStacked on #106 to keep this PR focused on the backend clippy fixes.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
monode	Ready	Preview, Comment	Jun 11, 2026 3:28pm

[greptile-apps]

Greptile Summary

This PR resolves a set of Clippy lints across the backend Rust codebase with no functional changes — all fixes are mechanical rewrites that leave runtime behavior identical.

• client.rs: Removes redundant as u64 casts on u64 - u64 subtractions; timestamp_ns and the stored start values are already u64, making the cast a no-op.
• server.rs: Boxes EventData inside EventDataOrMetrics::Event to shrink the enum's stack size, removes an explicit return, and converts an if let … { true } else { false } to matches!; all match sites are updated.
• event_listener.rs: Renames from_str to from_name to avoid shadowing the standard FromStr::from_str interface, and switches % 100 == 0 to is_multiple_of(100).
• event_filter.rs: Accepts &[T] instead of &Vec<T> in ArrayPrefixFilter::matches, and drops a redundant double-borrow on the is_native_transfer call.
• serializable_event.rs: Removes an unnecessary .clone() on a Copy (u64) field.

Confidence Score: 5/5

All changes are mechanical Clippy fixes with no logic changes; safe to merge.

Every change in this PR is a direct, non-functional Clippy correction: removing redundant casts and clones on Copy types, tightening a slice parameter, dropping a double-borrow, boxing a large enum variant, replacing an explicit return, and using the matches! macro. No branching logic, data transformations, or public interfaces are altered. The rename from from_str to from_name is the only potentially broad change, and the single internal call site in event_to_data is correctly updated; the Rust compiler would catch any missed callers.

No files require special attention.

Important Files Changed

Filename	Overview
backend/src/bin/client.rs	Removes two redundant as u64 casts on u64–u64 timestamp subtractions; semantics unchanged.
backend/src/lib/event_filter.rs	Switches &Vec<T> to &[T] in ArrayPrefixFilter::matches and removes a redundant double-borrow on is_native_transfer.
backend/src/lib/event_listener.rs	Renames from_str → from_name (all call sites updated) and uses is_multiple_of(100) in the debug log gate.
backend/src/lib/serializable_event.rs	Drops an unnecessary .clone() on a u64 Copy field.
backend/src/lib/server.rs	Boxes EventData in EventDataOrMetrics::Event to reduce enum stack footprint; removes explicit return and converts if let boolean pattern to matches!.

Sequence Diagram

sequenceDiagram
    participant Ring as Event Ring
    participant Listener as run_event_listener
    participant Forwarder as run_event_forwarder_task
    participant Broadcast as broadcast::Sender<EventDataOrMetrics>
    participant Client as client_write_task (per WS client)

    Ring->>Listener: raw ExecEvent
    Listener->>Listener: EventName::from_name(name)?
    Listener->>Forwarder: EventData (mpsc)
    Forwarder->>Forwarder: enrich txn_hash, update TPSTracker
    Forwarder->>Broadcast: "EventDataOrMetrics::Event(Box<EventData>)"
    alt BlockEnd
        Forwarder->>Broadcast: EventDataOrMetrics::TopAccesses(...)
    end
    alt BlockStart
        Forwarder->>Broadcast: EventDataOrMetrics::TPS(...)
    end
    Broadcast->>Client: EventDataOrMetrics (cloned)
    Client->>Client: "process_event -> SerializableEventData::from(&*event_data)"
    Client->>Client: filter.matches_event(...)
    Client-->>WS: ServerMessage::Events / TopAccesses / TPS (JSON)

Loading

_{Reviews (1): Last reviewed commit: "fix(backend): remove redundant u64 casts..." | Re-trigger Greptile}

[kyscott18]

Superseded by #109. The first replacement branch included the CI commits as its base; #109 is rebuilt from main with only the clippy fixes so the stack can merge cleanly.