Add helper for computing total portfolio value from a holdings array

[Chucks1093]

Summary

The wallet holdings response needs a total_portfolio_value field that sums key_balance × current_price across all entries. This calculation should live in a shared helper so it can be reused across the holdings endpoint and any future summary endpoints.

Scope

• Add a pure helper function that accepts an array of holding entries and returns the summed total value
• Use the helper in the holdings endpoint response serializer
• Add unit tests for the helper covering: single entry, multiple entries, all zero prices

Acceptance Criteria

• Helper returns correct sum for multiple holdings
• Helper returns zero when all prices are zero
• Holdings endpoint response includes correct total_portfolio_value field

---

Coordinate on Telegram

[YfengJ]

Hi maintainers, I can take this if it is still available.

If assigned, I will add a pure reusable total-portfolio-value helper, wire it into the holdings response serializer, and cover single holding, multiple holdings, and zero-price cases with focused unit tests.

I will wait for assignment or explicit maintainer confirmation before starting implementation or opening a PR.

[ekwe7]

I would love to work on this issue. I have experience building production ready applications and collaborating through GitHub workflows. I'll carefully analyze the current implementation, propose a reliable solution, and submit a well-tested PR. My focus is on clean code, maintainability, and timely delivery

[Unclebaffa]

Hi team,

I would love to take ownership of this issue. As an experienced web developer actively contributing to the GrantFox ecosystem, I can quickly implement this pure utility to ensure accurate and reusable portfolio calculations across the application.

Here is my straightforward implementation plan:

1. Helper Implementation

• Create a pure utility function (e.g., calculateTotalPortfolioValue) in the shared helpers directory.
• Use a functional approach (like .reduce()) to safely iterate over the holdings array, multiplying key_balance by current_price and returning the accumulated sum.

2. Endpoint Integration

• Update the holdings endpoint response serializer to invoke this new helper.
• Map the returned sum to the new total_portfolio_value field in the outgoing response payload.

3. Unit Testing

• Write focused unit tests validating the specific acceptance criteria: correct sums for a single entry, accurate aggregation for multiple entries, and strict zero returns when all prices (or balances) are zero.

Let me know if this plan aligns with your architecture. If so, please assign me and I will get a PR up shortly!

Best,
Unclebaffa

[Tobi-8]

Can i be assigned to this
I have the knowledge and experience to handle this issue

[grantfox-oss]

🦊 GrantFox — @Unclebaffa has been assigned to this issue as part of the Official Campaign campaign!

Next steps:

1. Open a Pull Request referencing this issue (e.g., Closes #430)
2. Your PR will be reviewed by the accesslayerorg maintainers

Good luck! Track your progress on GrantFox.

[YfengJ]

Thanks for the update. I see this has been assigned to @Unclebaffa via GrantFox, so I will step back and will not open a competing PR. Good luck with the implementation.

[grantfox-oss]

🎉 This issue has been marked as completed on GrantFox as part of the Official Campaign campaign!

@Unclebaffa's PR #434 was approved and merged by @Chucks1093.

🏆 @Unclebaffa: You earned 40 FoxPoints for this contribution! Your current tier: Builder (1,351 total points). Track your full progress on GrantFox.

👏 Great work, @Unclebaffa! Keep contributing to accesslayerorg.