Skip to content

refactor(host): keep mimir-host tokens and pending aligned on narrow terminals and with wide device names, through Loom's table (MMR-596) - #517

Merged
dbtlr merged 2 commits into
mainfrom
chore/mmr-596
Oct 24, 2026
Merged

dbtlr merged 2 commits into
mainfrom
chore/mmr-596

Conversation

@dbtlr

@dbtlr dbtlr commented Oct 24, 2026

Copy link
Copy Markdown
Owner

mimir-host tokens and pending padded their columns by character count, so a device named 研究室のマシン pushed its whole row out of line. On a narrow terminal, rows ran off the edge. tokens also printed a REVOKED column that is always - unless you pass --all.

Both commands now lay out through Loom's table() pack view. Columns align by terminal cell and wrap to the terminal's width. When even the narrowest unbroken text won't fit, each row prints as a LABEL value block. tokens now leads with the device and shows last use as an age (19m ago, never) and pairing as a date. REVOKED appears only under --all. JSON output is unchanged.

DEVICE          USER   LAST USED  PAIRED
asgard-laptop   owner  <1m ago    2026-10-23
café-desk 🦊    owner  never      2026-10-23
研究室のマシン  owner  never      2026-10-23

In the Loom CLI, text views now receive Loom's view context. The artifact feed uses it to pad its free-text titles and tags by terminal cell. Task, search, and seed rows keep their headerless layout. They pad only ASCII ids and fixed words, so their output is unchanged and parity needs no new difference entries.

Loom's dim is a theme token, and neither app installs a theme, so table headers print unstyled. The pending hint keeps its existing raw-SGR dim.

Verified with bun run verify, bun run test:host, and a pseudo-terminal smoke at 100 and 30 columns with wide-character device names. test:ui flaked on unrelated console focus tests, tracked as MMR-597.

Closes MMR-596.

Made by Claude Opus 5.5 in Claude Code.

dbtlr added 2 commits October 23, 2026 23:37
…le, and align the artifact feed by terminal cell (MMR-596)

mimir-host tokens and pending render through @loomcli/plugins/table:
columns align by terminal cell, wrap to the terminal, and fall back to
one record block per row when even the narrowest words do not fit.
tokens leads with the device, shows last use as an age and pairing as a
date, and adds REVOKED only under --all.

The Loom CLI's text views now receive the view context, so the artifact
feed pads its free-text title and tags with Loom's cell measure. Task,
search, and seed rows pad ASCII-only columns and keep their layout.
… table rows carry no trailing spaces (MMR-596)

Loom's table pads every cell up to the last column off a terminal, so a
blank REVOKED cell left active rows ending in spaces. The host commands
also compose their pack views with the exported jsonViews rather than a
placeholder records callback.
@kody-ai

kody-ai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug ✅
Performance ✅
Security ✅
Business Logic ✅

Access your configuration settings here.

​

@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Next included review available in 5 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 84 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: b4332906-f506-40c4-8624-c78ef52dbec7
Commits

Reviewing files that changed from the base of the PR and between d9568b4 and 0d71ece.

Files selected for processing: 13
  • .changes/mmr-596.md
  • packages/cli/src/commands/artifacts.ts
  • packages/cli/src/render/feeds.test.ts
  • packages/cli/src/render/feeds.ts
  • packages/cli/src/render/node.ts
  • packages/cli/src/render/presentation.ts
  • packages/cli/src/render/views.ts
  • packages/host/src/admin/commands/pairing.test.ts
  • packages/host/src/admin/commands/pairing.ts
  • packages/host/src/admin/commands/tokens.test.ts
  • packages/host/src/admin/commands/tokens.ts
  • packages/host/src/admin/render.ts
  • packages/host/src/admin/testing.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@dbtlr
dbtlr merged commit 696cff4 into main Oct 24, 2026
10 checks passed
@dbtlr
dbtlr deleted the chore/mmr-596 branch October 24, 2026 09:32
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.

1 participant