Skip to content

fix(output): render vault tokens on separate lines - #101

Open
chalabi2 wants to merge 1 commit into
mainfrom
fix/vault-token-lines
Open

chalabi2 wants to merge 1 commit into
mainfrom
fix/vault-token-lines

Conversation

@chalabi2

@chalabi2 chalabi2 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Render one token per line in vault-state pretty output, with amounts right-aligned across the section.
  • Keep labels on the first token line and indent continuation lines under the value column, matching the layout requested in bug(q/bme/vault-state): each token should be on a new line when in pretty mode #99.
  • Preserve token order, zero balances, readable units, and comma separators through the shared CLI/monitor renderer. JSON/YAML and compact coin output elsewhere are unchanged.
  • Update the specification, design, changelog, and vault-output golden fixture.

Closes #99

Verification

  • The regression test reproduces the issue's exact amounts and checks all seven token lines, right alignment, labels, zero values, and unchanged compact formatting. It failed before the fix and passes afterward.
  • GOWORK=off make akt
  • GOWORK=off go test ./...
  • GOWORK=off golangci-lint run --timeout=10m — 0 issues.

The local suite includes offline E2E coverage. Opt-in localnet and live Console tests were not enabled.

Make vault token balances easier to compare with one coin per line and
a right-aligned amount column in the shared CLI and monitor renderer.
Preserve coin units, zero values, and compact formatting elsewhere.

Closes #99

Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
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.

bug(q/bme/vault-state): each token should be on a new line when in pretty mode

1 participant