Refactor route_event method to use closure that embeds origin_id

[thep2p]

Summary

The route_event method in NetworkHub should be refactored to use a closure that embeds the origin_id, improving code organization and reducing parameter passing.

Current Implementation

In src/network/mock/hub.rs:46-48, the route_event method currently takes origin_id as a separate parameter:

// TODO: route_event should be a closure that embeds the origin_id.
/// Routes an event to the appropriate mock network based on the target node identifier.
pub fn route_event(&self, origin_id: Identifier, target_id: Identifier, event: Event) -> anyhow::Result<()> {

Proposed Solution

Refactor the method to use a closure-based approach where the origin_id is embedded within the routing logic, potentially improving:

• Code clarity and maintainability
• Functional programming patterns
• Reduced parameter passing overhead

Technical Details

• File: src/network/mock/hub.rs
• Line: 46
• Component: NetworkHub::route_event method
• Impact: Mock network infrastructure for testing

Acceptance Criteria

• Replace current route_event method signature with closure-based approach
• Ensure origin_id is properly embedded in the routing logic
• Maintain backward compatibility with existing test infrastructure
• Update any dependent code that calls this method
• Add tests to verify the new implementation works correctly

Labels

• enhancement
• refactoring
• mock-network