Skip to content

docs: align maintained guidance with current CLI - #109

Merged
twaldin merged 2 commits into
mainfrom
ticket/twa-37
Sep 8, 2026
Merged

docs: align maintained guidance with current CLI#109
twaldin merged 2 commits into
mainfrom
ticket/twa-37

Conversation

@twaldin

@twaldin twaldin commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Changes

  • Correct CLI examples, keybinds, skill discovery, workflow run IDs/worktrees/retry limits, and storage paths against current source.
  • Repair instruction-file markers, repository-map and adapter-registration references; keep architecture decisions, research, personal operator constraints and third-party content.
  • Document isolated Bun checks and make the optional provider-backed skill probe use a private working directory.

Validation

  • bun install --frozen-lockfile, bunx --no-install tsc --noEmit, git diff --check passed.
  • Executed the documented isolated bun test recipe: 910 passed, 3 skipped, 0 failed. Skips: tool-gated TUI pilot and two pre-existing skipped tests.
  • Bun smoke checked README YAML against seeded presets, CLI help/list/status, keybinds, local links and AGENTS.md -> CLAUDE.md.
  • No application behavior changes, global installs, live fleet/controller changes, provider-backed flt runs or deployment.

Checked open PRs #95 and #108 first; none of their changed files are edited here. Per-adapter projection changes remain with #95. Independent Standards and Spec reviews passed at 96d2caf on Grok 4.6 (xhigh, model identity verified).

TWA-37

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8c70200f-7f7b-4ea8-9e56-72cf01a8c5f0

📥 Commits

Reviewing files that changed from the base of the PR and between 4fb1a75 and 96d2caf.

📒 Files selected for processing (6)
  • CLAUDE.md
  • CONTRIBUTING.md
  • README.md
  • docs/testing-tui.md
  • src/adapters/CLAUDE.md
  • templates/flt-skill/base.md

Included review availability: 1 review is currently available. Your included PR review attempts over the past 7 days set your current allowance at 8 reviews per hour.


📝 Walkthrough

Walkthrough

The pull request updates repository, adapter, README, contributor, TUI testing, and skill-template documentation. It documents current tmux startup, workflow runs, skill projection, adapter registration, artifact preservation, isolated testing, and CLI commands.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 96d2c

This updates documentation and testing guidance to match current CLI, workflow, adapter, and skill behavior. No application or deployment behavior changes are included, and the documented checks completed successfully.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: updating maintained documentation to match the current CLI.
Description check ✅ Passed The description directly covers the documentation updates, testing guidance, validation results, and scope of the changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ticket/twa-37

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

@twaldin
twaldin marked this pull request as ready for review September 8, 2026 04:51
@twaldin
twaldin merged commit 9874474 into main Sep 8, 2026
2 checks passed
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