Skip to content

docs: fix README to match current CLI and TUI state#25

Merged
njbrake merged 1 commit intomainfrom
fix/issue-22
Apr 2, 2026
Merged

docs: fix README to match current CLI and TUI state#25
njbrake merged 1 commit intomainfrom
fix/issue-22

Conversation

@njbrake
Copy link
Copy Markdown
Contributor

@njbrake njbrake commented Apr 2, 2026

Description

The README was out of date after the max-turns removal (#24) and TUI-first refactor. This PR brings it in sync with the actual codebase:

  • Removes references to --tui and --max-turns flags (both no longer exist)
  • Documents --headless mode and explains TUI-default vs headless dispatch
  • Expands TUI Controls with action bar shortcuts, trade builder, discard, help overlay
  • Updates the architecture tree with new files (headless.rs, lib.rs, screens.rs, menu.rs, tui_human.rs)
  • Replaces emdashes with double dashes per project coding style
  • Fixes Save/Resume section (no longer references max turns)
  • Updates Features list to reflect human-first TUI design

Fixes #22

PR Type

  • New Feature
  • Bug Fix
  • Refactor
  • Documentation
  • Infrastructure / CI

Checklist

  • I understand the code I am submitting
  • New and existing tests pass
  • Documentation was updated where necessary

AI Usage

  • No AI was used
  • AI was used for drafting/refactoring
  • This is fully AI-generated

AI Model/Tool used: Claude Opus 4.6 via Claude Code

Any Additional AI Details you'd like to share: AI cross-referenced the actual CLI struct in headless.rs, TUI input handling in ui/mod.rs, and file structure to identify all README inaccuracies.

NOTE:
When responding to reviewer questions, please respond yourself rather than copy/pasting reviewer comments into an AI and pasting back its answer. We want to discuss with you, not your AI :)

  • I am an AI Agent filling out this form (check box if true)

The README was out of date after the max-turns removal and TUI-first
refactor. Removes references to --tui and --max-turns flags, documents
--headless mode, expands TUI controls to cover action bar / trade builder /
discard keybindings, updates the architecture tree with new files
(headless.rs, lib.rs, screens.rs, menu.rs, tui_human.rs), and replaces
emdashes with double dashes per project style.

Fixes #22

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@njbrake njbrake merged commit 69172bf into main Apr 2, 2026
5 checks passed
@njbrake njbrake deleted the fix/issue-22 branch April 2, 2026 14:22
njbrake added a commit that referenced this pull request Apr 6, 2026
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.

Fix the Readme

1 participant