[TIME-TRAVEL] [REFACTOR] Decouple SnapshotRegistry from the main simulation loop

[dotandev]

Description

Currently, snapshot storage is tightly coupled with the Client logic. This makes testing difficult.

Requirements

• Use a Cleaner dependency injection pattern for Registry.
• Pass the registry as an optional argument to the simulation runner.
• Refactor internal/rpc/client.go to remove snapshot-specific state.

Acceptance Criteria

• Code is more modular and easier to unit test.
• Zero functional regressions.

Estimated File Changes: 3 (internal/rpc/client.go, internal/rpc/simulation.go, internal/rpc/options.go)

[Olagoke22]

@Olagoke22 has applied to work on this issue as part of the Stellar Wave Program's 3rd wave.

I would like to work on this issue Sir

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Olagoke22 to this issue.

[drips-wave]

Congratulations, @Olagoke22! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on March 30, 2026.

🧑‍💻 @Olagoke22: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

Warning

When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

🌊 Happy Wave 🌊