Skip to content

fix(ui): open Plugins on this PC and keep the project menu at the right - #606

Merged
klNuno merged 8 commits into
mainfrom
boite/bots-detect-project-menu
Oct 19, 2026
Merged

klNuno merged 8 commits into
mainfrom
boite/bots-detect-project-menu

Conversation

@klNuno

@klNuno klNuno commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Note

Clanker Claude Opus 5.5 is responding on behalf of meetsu

Settings > Plugins now opens on this PC, where a desktop app like Bots runs, instead of the machine of the open conversation; and the sidebar's ... menu sits at the same place on every project.

Plugins page with its machine menu

what changed

  • The Plugins page reads the plugins of this PC when the window has one, else the window's machine (PluginsPage.svelte).
  • With several owned machines, a menu beside the heading names the machine shown and switches to another.
  • Each machine gets a freshly mounted list (PluginsMachine.svelte), so a late answer from the previous core never lands on the new one.
  • The project menu ends the sidebar header after the counter pills and keeps its slot while hidden (Sidebar.svelte), as before feat(ui): lighter sidebar project headers #592.
  • docs/plugins.md says which machine the page shows.

to question

The menu slot is reserved again, so a project name has 28 px less at rest. Overlaying the end of the header (#592) put the menu at a different x on each project, depending on its counters.

verified

  • bun x vitest run src/components/PluginsPage.test.ts: 12 pass; the new machine case fails on main, and its late-Stop step fails on 896eb41c.
  • bun test tests/e2e/plugins.test.ts: 5 pass.
  • GPT-6.1 Sol (high) found the late-answer leak on 896eb41c, fixed in 17ad49e5.
  • bun test tests/e2e/project-views.test.ts: 5 pass; the new alignment step fails on main (two different right edges).
  • bun run check: exit 0.
  • The PC's core already lists Bots 0.1.17 as installed: the Plugins page was showing the server, which has no plugins.
  • not verified: the Windows app, a touch screen.

captures

Every project menu forced visible: one right edge (269 px) on all four headers.

Sidebar with aligned project menus

@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 35 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: beboite/boite/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4532003a-e62c-43ec-abcf-18d202b3182d
📥 Commits

Reviewing files that changed from the base of the PR and between 18aa29c and 925aa97.

📒 Files selected for processing: 8
  • docs/plugins.md
  • packages/ui/src/components/PluginsMachine.svelte
  • packages/ui/src/components/PluginsPage.svelte
  • packages/ui/src/components/PluginsPage.test.ts
  • packages/ui/src/components/Sidebar.svelte
  • packages/ui/src/lib/strings/en/plugins.ts
  • packages/ui/src/lib/strings/fr/plugins.ts
  • tests/e2e/project-views.test.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@github-actions github-actions Bot added documentation Improvements or additions to documentation ui packages/ui labels Oct 18, 2026
@github-actions github-actions Bot added the ci Workflows and CI scripts label Oct 18, 2026
@klNuno

klNuno commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

Note

Clanker Claude Opus 5.5 is responding on behalf of meetsu

#614 split strings.en.ts and strings.fr.ts into one file per part. When git merge origin/main stops on them, run bun scripts/migrate/split-catalogues.ts: it replays this branch's catalogue edits onto main's parts and stages them. A part both sides changed on the same lines keeps conflict markers.

…ject-menu

# Conflicts:
#	packages/ui/src/lib/strings.en.ts
#	packages/ui/src/lib/strings.fr.ts
#	scripts/ci/budgets.json
@klNuno
klNuno merged commit 0ea8235 into main Oct 19, 2026
27 checks passed
@klNuno
klNuno deleted the boite/bots-detect-project-menu branch October 19, 2026 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Workflows and CI scripts documentation Improvements or additions to documentation ui packages/ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant