Skip to content

fix(wallet): report unavailable balances and RPC failures - #138

Merged
brendanjryan merged 1 commit into
tempoxyz:mainfrom
mablr:fix/mpp-wallet-readiness
Sep 9, 2026
Merged

fix(wallet): report unavailable balances and RPC failures#138
brendanjryan merged 1 commit into
tempoxyz:mainfrom
mablr:fix/mpp-wallet-readiness

Conversation

@mablr

@mablr mablr commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Report unavailable balances as null instead of zero and include RPC diagnostics in wallet and key output. whoami now reports ready: false when its balance query fails while retaining wallet/key details and locally recorded session reserves.

Update schemas and documentation to distinguish RPC failures from successful zero-balance queries.

Validation: 177 tests passed, including actual CLI coverage, plus typechecks, lint, formatting and changelog validation.

AI-assisted

@mattsse mattsse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@brendanjryan
brendanjryan force-pushed the fix/mpp-wallet-readiness branch from 7efd640 to d3c0120 Compare September 9, 2026 17:20
Report failed balance queries as null instead of zero, include RPC diagnostics
in whoami and key output, and mark readiness false when the balance cannot be
queried. Preserve configured wallet/key details and locally recorded session
reserves; a successful zero-balance query remains distinct from an RPC failure.

Update output schemas, CLI metadata and documentation to describe nullable
balances and the diagnostic fields.
@brendanjryan
brendanjryan force-pushed the fix/mpp-wallet-readiness branch from d3c0120 to 4827707 Compare September 9, 2026 17:24
@brendanjryan
brendanjryan merged commit f5655ed into tempoxyz:main Sep 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants