Skip to content

Retire control and rooms TUI for headless surfaces - #280

Open
0thernet wants to merge 1 commit into
mainfrom
remove-tui-20261005
Open

0thernet wants to merge 1 commit into
mainfrom
remove-tui-20261005

Conversation

@0thernet

@0thernet 0thernet commented Oct 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Adopt hraness-control-kit v2.0.0 (desktop-foundation drops the TUI/dialog surface).
  • Remove vhalla tui and the standalone vhalla-rooms-tui crate; the local status/outputs text renderers keep the control goldens byte-identical (snapshot renderer now crops at the requested width like the removed ratatui surface).
  • Move the room form/signing layer (Form, labels, create_body/describe_body/archive_body, SigningSource) into vhalla-rooms-app so rooms submit/pending/status stay scriptable under the renamed experimental-rooms-replica feature.
  • rooms tui returns a clear retirement error pointing at the replica commands.
  • Update help, README, cli-parity, deploy/docs, CI feature lists, site copy, and the lockfile; the ratatui/crossterm/wezterm tree leaves Cargo.lock.

Test plan

  • cargo test -p vhalla-rooms-app sign::tests — 5 ported signing tests pass
  • cargo test -p vhalla-cli --test status --test rooms_submit --features experimental-rooms-replica — 17 pass incl. live-node e2e
  • Control snapshot goldens byte-identical; vhalla tui exits 2 unknown-command
  • cargo clippy --workspace --all-targets --all-features --locked -- -D warnings
  • cargo metadata --locked consistent

Generated with Devin

Adopt hraness-control-kit v2.0.0 (desktop-foundation drops the TUI
surface). Remove `vhalla tui` and the `vhalla-rooms-tui` crate; the
local status/outputs text renderers keep the existing goldens
byte-identical.

Move the room form/signing layer into vhalla-rooms-app so
`rooms submit/pending/status` stay scriptable under the renamed
`experimental-rooms-replica` feature. `rooms tui` returns a clear
retirement error.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@vercel

vercel Bot commented Oct 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
valhalla Ready Ready Preview Oct 5, 2026 7:52pm UTC

Request Review

@0thernet
0thernet enabled auto-merge (squash) October 5, 2026 19:52

This branch was successfully deployed

1 active deployment
Preview — daa2499a Deployed Oct 5, 2026 by vercel[bot]
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