chore(node/service): remove process method from NodeActor trait

[theochap]

Description

This PR removes the process method from NodeActor trait to simplify the structure of the NodeActor

Progress to #2176

[clabby]

Good start. I think a nice follow-up would be to think about how these Inbound<X>Event enums actually come into play here. Right now they're really just creating an unnecessary (but minimal) memory overhead, for example in the EngineActor where we're Boxing big values to avoid the big enum.

Perhaps rather than a single process method, we just start breaking out individual actions into separate functions.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.2%. Comparing base (1d2d6d6) to head (4b306cc).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[theochap]

Good start. I think a nice follow-up would be to think about how these Inbound<X>Event enums actually come into play here. Right now they're really just creating an unnecessary (but minimal) memory overhead, for example in the EngineActor where we're Boxing big values to avoid the big enum.

Perhaps rather than a single process method, we just start breaking out individual actions into separate functions.

Totally agree, will do!