refactor: bridge QuoteMetadata read/write

[micaelae]

Explanation

References

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed
• I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

---

Note

High Risk
Declared breaking API and behavior changes to fee/metadata shapes affect swap/bridge quote display, sorting, balance checks, and analytics; clients must upgrade optional access patterns.

Overview
BREAKING: QuoteMetadata and nested TokenAmountValues are now DeepPartial, and fee helpers stop coercing missing inputs to "0" or null—undefined fields stay undefined. Consumers must use optional chaining (as in bridge-status-controller) and may see different UI when gas or rate data is incomplete.

Quote metadata logic moves out of selectors.ts into utils/quote-metadata.ts (calcQuoteMetadata, mergeQuoteMetadata). selectBridgeQuotesWithMetadata calls those instead of inlining calculations; QuoteMetadata / TokenAmountValues are exported from the new module rather than types.ts. Sorting and metrics paths use safer access (e.g. quote?.cost?.valueInCurrency).

bridge-status-controller tests and utils adopt mergeQuoteMetadata to attach metadata to quote fixtures and align history/metrics/snaps with optional metadata fields.

^{Reviewed by Cursor Bugbot for commit 9afe7b6. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 9afe7b6. Configure here.}

[cursor]

Missing trade gas crashes fee

High Severity

When tradeGasLimit is missing or zero, calcTotalGasFee leaves totalGasLimitInDec undefined, then optional chaining on times yields undefined and the following .plus(l1GasFeesInDecGWei) runs on undefined, which throws. Approval-only gas and L1 fees are also skipped instead of being summed like before.

 

^{Reviewed by Cursor Bugbot for commit 9afe7b6. Configure here.}