Compact CLI output defaults#32
Open
andrei-hasna wants to merge 2 commits into
Open
Conversation
Preserve reviewed non-risky working tree changes before fleet repo redistribution. Risky local-only paths, if any, were left uncommitted.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--limit,--cursor/--offset,--verbose,show/inspect, and preserve script-oriented--json/CSV behavior unless a limit is explicitly requested.list_tasksoutput by default, add cursor continuation, and reject invalid cursors instead of silently replaying or ending pagination.Before / After
Before, a noisy task database could make
todos list --alldump every long row into the terminal with no paging/detail hint. A 55-task example printed all 55 long rows.After, the same shape of data starts with a compact page:
Adversarial Review Reconciliation
A read-only adversarial reviewer focused on CLI compatibility, output ergonomics, and token bloat found four issues; all are fixed here:
--offset, and--cursoris rejected with those modes.search --jsonkeeps uncapped machine-readable behavior unless--limitis explicit.Validation
bun run typecheckbun test src/mcp/mcp.test.ts --timeout 30000bun test src/cli/cli.test.ts --timeout 30000git diff --checkbun run buildtimeout 300s bun testwas attempted; it hit the 300s cap without reporting a failure. Focused CLI/MCP suites above pass and cover this change.Changed Files
src/cli/commands/task-commands.tssrc/cli/commands/project-commands.tssrc/cli/helpers.tssrc/db/task-crud.tssrc/mcp/token-utils.tssrc/mcp/tools/task-crud.tssrc/cli/cli.test.tssrc/mcp/mcp.test.tsREADME.md,AGENTS.md,src/lib/cli-reference.ts,src/lib/cli-manpage.tsLocal State Note
The branch was created after a safe
git fetch --prunewithmainaligned toorigin/main. The worktree already had unrelated uncommitted release/SDK/server/public-release changes and untracked SDK files; those were left uncommitted and are not part of this PR.