Skip to content

✨ feat: tapesctl session list now defaults to a table view - #74

Merged
jpmcb merged 1 commit into
mainfrom
session-table
Aug 24, 2026
Merged

✨ feat: tapesctl session list now defaults to a table view#74
jpmcb merged 1 commit into
mainfrom
session-table

Conversation

@jpmcb

@jpmcb jpmcb commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Adds comfy-table for tapesctl session list to actually look nice - with optional --json flag

Closes PCC-1264

@jpmcb
jpmcb marked this pull request as ready for review August 21, 2026 15:43
@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

PCC-1264

@greptile-apps

greptile-apps Bot commented Aug 21, 2026

Copy link
Copy Markdown

Greptile Summary

The PR changes tapesctl sessions list to render a human-readable table by default while preserving the original response through --json.

  • Adds table projection, formatting, sanitization, truncation, timestamp formatting, cursor output, and focused tests.
  • Adds the CLI flag and documents the updated output contract.
  • Adds comfy-table and its locked transitive dependencies.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
crates/tapesctl/src/api/table.rs Adds the session-table renderer and now neutralizes both terminal control characters and the full Unicode Bidi_Control set, resolving the prior findings.
crates/tapesctl/src/api/mod.rs Selects raw JSON or table output for session listings according to the new flag.
crates/tapesctl/src/cli.rs Adds the documented --json switch to the session-list arguments.
Cargo.toml Adds the workspace-level comfy-table dependency used by the new renderer.
Cargo.lock Locks the table-rendering dependency graph; the reported paste package is unchanged from the base revision.

Reviews (3): Last reviewed commit: "✨ feat: tapesctl session list now defaul..." | Re-trigger Greptile

Comment thread crates/tapesctl/src/api/table.rs
Comment thread crates/tapesctl/src/api/table.rs
Signed-off-by: John McBride <john@papercompute.com>
@jpmcb
jpmcb merged commit ea33b1d into main Aug 24, 2026
18 checks passed
@jpmcb
jpmcb deleted the session-table branch August 24, 2026 11:36
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