feat: add — interactive onboarding wizard#743
Merged
Merged
Conversation
Guides new users from zero to productive in one command: - Detects install (binary on PATH, existing store) - Asks which AI agent they use (Hermes/Claude/Cursor/Pi/OpenCode) - Picks integration mode (tool vs memory-provider) - Toggles features on/off (Aging, Auto-maintenance, Graph rerank, Salience/Recency boost, Server mode) - Writes uteke.toml config with selections - Runs uteke init with selected agent automatically - Prints a full feature showcase (all commands grouped by category) Non-interactive mode: uteke onboard --yes --agent hermes --namespace default Docs updated: getting-started.md, cli-reference.md, AGENT.md, CHANGELOG.md New dep: which crate for PATH detection
Add a prominent "🚀 Onboarding" section right after install, before "Why Uteke?". Includes an agent-readable callout so when a user drops the GitHub link to an AI agent (Hermes, Claude, etc.), the agent reads the README and knows to run `uteke onboard.
🔴 Must Fix: - AGENT.md: correct schema version v13 → v15 (room↔document junction, #689) - onboard.rs: remove misleading get_stats() placeholder, drop count claim - onboard.rs: write_config() now backs up existing uteke.toml to .bak and prompts for confirmation before overwriting (prevents credential loss) 🟡 Should Fix: - prompt_integration_mode(): print message for custom agents (no longer silent) - validate_agent(): now warns on unrecognized agents instead of being a no-op - Banner boxes: replace misaligned Unicode box drawing with consistent ASCII print_banner() helper (fixes off-by-1 alignment, ambiguous em-dash width) - --json flag: intentionally ignored in onboard to avoid mixing ASCII banners with parseable JSON output 🟢 Nice to Have: - Install URL: main → develop (matches repo default branch convention) - Add 7 unit tests for pure functions (get_toggle, validate_agent, print_banner, write_config TOML structure) All tests pass: cargo fmt --check ✅, cargo clippy -D warnings ✅, cargo test (50 passed) ✅ Co-authored-by: ajianaz <reviewer>
- Remove unused 'toggles' vec in test_write_config_generates_valid_toml - Replace useless format! with raw string literal - Remove stray closing parenthesis
🔍 Cora AI Code Review✅ No issues found. Code looks good! Review powered by cora-cli · BYOK · MIT |
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
2 participants
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
Recreation of #696 (fork deleted, could not push fixes). All review feedback addressed + clippy errors resolved.
Changes
uteke onboard--yesmodeFixes Applied (from #696 review)
);CI Notes
Branch rebased on latest develop (includes API versioning, dream config, endpoint rename).
Supersedes #696.