Skip to content

Feat/desktop multi kb picker#266

Open
claytonlin1110 wants to merge 2 commits into
vouchdev:testfrom
claytonlin1110:feat/desktop-multi-kb-picker
Open

Feat/desktop multi kb picker#266
claytonlin1110 wants to merge 2 commits into
vouchdev:testfrom
claytonlin1110:feat/desktop-multi-kb-picker

Conversation

@claytonlin1110

Copy link
Copy Markdown
Contributor

What changed

Adds a Tauri native shell under desktop/app/ with File → Open KB…, File → Recent KBs (last five), and File → New KB…. Switching KBs restarts the vouch review-ui sidecar against the new project root; recent opens persist in ~/.config/vouch-desktop/state.json. Shared Python helpers live in src/vouch/desktop/ with vouch desktop * CLI commands. Review-ui /healthz and templates now expose kb_label; vouch init --json supports structured init output for the shell.

Closes #207

Why

Real users keep a .vouch/ per project; the CLI walks up from cwd to find it, but a desktop app needs an explicit picker so you can switch projects without env-var dance or restarting the app (#207).

What might break

No breaking changes to existing .vouch/ directories or kb.* protocol.

  • Additive: /healthz gains kb_label; review-ui masthead/title show the active project name instead of the generic "review console" string.
  • Additive: vouch init --json, vouch desktop … subcommands.
  • No file moves, on-disk schema changes, or review-gate behaviour changes.

VEP

Not required — new desktop packaging layer and additive CLI/web fields only; no object-model or kb.* surface change.

Tests

  • make check passes locally (lint + mypy + pytest)
  • New / changed behaviour has a test (tests/test_desktop.py, tests/test_desktop_ports.py, tests/test_desktop_sidecar.py, updated tests/test_web.py)
  • CHANGELOG.md updated under ## [Unreleased]

@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 4587fc20-7259-427a-a4b7-ef7d6d8cc778

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@claytonlin1110 claytonlin1110 changed the base branch from main to test June 23, 2026 02:53
@claytonlin1110

Copy link
Copy Markdown
Contributor Author

@plind-junior Please review

@claytonlin1110 claytonlin1110 force-pushed the feat/desktop-multi-kb-picker branch from 2b79fe5 to 55f92bc Compare June 29, 2026 19:20
@github-actions github-actions Bot added docs documentation, specs, examples, and repo guidance ci github actions and automation cli command line interface review-ui browser review ui packaging packaging, build metadata, and make targets tests tests and fixtures size: XL 1000 or more changed non-doc lines labels Jun 29, 2026
@claytonlin1110

Copy link
Copy Markdown
Contributor Author

@plind-junior Please review

@claytonlin1110 claytonlin1110 force-pushed the feat/desktop-multi-kb-picker branch from 55f92bc to 8d31ac1 Compare June 30, 2026 06:58
@plind-junior

Copy link
Copy Markdown
Collaborator

Vouch is hardly reshaped and all the previous issues and PRs are unnecessary. I will create the issue tickets

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci github actions and automation cli command line interface docs documentation, specs, examples, and repo guidance packaging packaging, build metadata, and make targets review-ui browser review ui size: XL 1000 or more changed non-doc lines tests tests and fixtures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(desktop): multi-KB picker — switch between projects

2 participants