Deduplicate wallet and sandbox modules

[dotunv]

Description

The wallet/ and sandbox/ modules share nearly identical logic:

• generateKeypair
• fundFromFriendbot
• Payment sending functions

Both are used by different frontend pages but implement the same core operations.

Directories:

• apps/api/src/modules/wallet/
• apps/api/src/modules/sandbox/

Proposed Approach

Option A: Merge into a single wallet module and have the Sandbox page call wallet endpoints.

Option B: Keep separate modules but extract shared logic into a common service.

Benefits

• Reduced code duplication
• Easier maintenance
• Single source of truth for wallet operations

Acceptance Criteria

• No duplicated logic between modules
• Both frontend pages (Sandbox and Wallet) continue to work
• All existing functionality is preserved
• Tests cover the deduplicated code

[Okorie2000-code]

Hi Maintainer, I'd love to work on this issue. I understand the requirements and will submit a clean, well-tested PR within 24hrs. Please assign it to me

[Menjay7]

Hi! I came across this issue and would love to work on it. I have experience with similar tasks and I'm confident I can investigate, implement a clean solution, and thoroughly test it before submitting a PR. If the issue is still open, I would appreciate it if you could assign it to me. Thank you!

[jaynomyaro]

Hi! I’m a full-stack developer with a strong blockchain background. I’d love to handle this issue and can deliver a clean, reliable solution. Kindly assign it to me.

[Babigdk]

Hi @dotunv, I'd like to work on this issue.

I have experience refactoring NestJS/TypeScript applications to eliminate duplication while preserving existing behavior. My approach will be to identify the shared wallet operations, extract them into a common service to maintain a single source of truth, and keep both the Wallet and Sandbox modules working without breaking their public APIs. I'll also add/update tests to cover the shared implementation and verify there are no regressions before submitting a clean PR.

[rudrasatani13]

I'd like to work on this. I checked the scope and can take it on — both modules delegate to one shared Stellar service and the copy that wipes the raw secret is dropped.

Please assign it to me.

[rupesh-kumar-sah]

Hey team,

I can tackle Deduplicate wallet and sandbox modules. Proposed implementation roadmap:

• Analysis: Inspect current behavior and edge cases in Savitura/Savitools
• Core Implementation: Deliver a clean, modular solution for Deduplicate wallet and sandbox modules strictly following Savitura/Savitools conventions.
• Quality Assurance: Cover changes with automated tests ensuring CI passes smoothly.

Happy to pick this up today—please assign to me!