Skip to content

feat(commands): consolidate contextual command discovery - #191

Merged
BunsDev merged 3 commits into
mainfrom
fix/178-command-discovery
Oct 13, 2026
Merged

BunsDev merged 3 commits into
mainfrom
fix/178-command-discovery

Conversation

@BunsDev

@BunsDev BunsDev commented Oct 13, 2026

Copy link
Copy Markdown
Member

Summary

Command discovery now uses one canonical catalog for grouped commands, contextual completion and palette navigation, help, onboarding, and the generated command reference. Expired paths return their replacement before any TUI or command side effects.

Context

Changes

  • Compose built-in, configured, skill, and already-registered plugin commands with explicit collision precedence and aliases.
  • Rank local context and recent commands; progressively browse groups and execute selected leaves with one Enter.
  • Preserve arguments during completion, including edits inside grouped command paths; keep session resume as a picker until a selection is made.
  • Generate docs from the catalog and retain detailed handler guidance for argument-rich commands. Check catalog/reference drift and reviewed screen baselines.

Validation

Rust commands use rustup run stable (Rust 1.99) and CARGO_TARGET_DIR=/tmp/coven-code-pr182-target, run from src-rust/.

  • git diff --check
  • cargo fmt --all
  • cargo check --workspace
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace — 2128 passed, 7 ignored after integrating current main.
  • cargo test -p claurst-commands --features steer — 139 passed.
  • cargo build -p claurst; COVEN_BIN=/tmp/coven-code-pr182-target/debug/coven-code REQUIRE_TMUX=1 bash scripts/tui-tests/run.sh — 48 passed, including one-Enter palette execution and retired-path rejection.
  • Independent review: command dispatch, completion, TUI integration, generated registry, and all changed screen fixtures; findings addressed.
  • An initial post-link ACP smoke run timed out; direct startup completed in 0.08s and all four ACP smoke tests passed on recheck.
  • Not run: authenticated provider calls; discovery is verified with local state and offline terminal checks.

PR Readiness

  • Diff is limited to the intended files.
  • Generated reference and golden fixtures were regenerated by their commands and reviewed.
  • User-facing docs/help were updated.
  • No secrets, credentials, unrelated local data, or logs are included.
  • Remaining scope and verification limits are listed above.

Use a canonical catalog for grouped command discovery, contextual suggestions, help, onboarding, and generated reference docs. Retire expired command paths with explicit replacements and preserve existing command arguments through completion.

Closes #178

Coven-Harness: Codex / GPT-6
Integrate the verified stream-durability fixes before landing command discovery.

Closes #178

Coven-Harness: Codex / GPT-6
@vercel

vercel Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Oct 13, 2026 10:18am UTC

Request Review

Wait for the filtered frame instead of an entry already absent from contextual suggestions. Source the unauthenticated composer hint from the catalog and point setup guidance at the canonical account command.

Fixes #178

Coven-Harness: Codex / GPT-6
@BunsDev
BunsDev merged commit 998ddd1 into main Oct 13, 2026
4 checks passed
@BunsDev
BunsDev deleted the fix/178-command-discovery branch October 13, 2026 12:15

This branch was successfully deployed

1 active deployment
Preview — 41fd91ee Deployed Oct 13, 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.

Consolidate slash commands and add contextual discovery

1 participant