Skip to content

Split Plugins and Skills into separate workspaces - #2902

Closed
brsbl wants to merge 23 commits into
mainfrom
bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv
Closed

Split Plugins and Skills into separate workspaces#2902
brsbl wants to merge 23 commits into
mainfrom
bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv

Conversation

@brsbl

@brsbl brsbl commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Human comments

What was wrong

  • Plugins and Skills shared an Extensions entry, workspace sidebar, and top navigation instead of having separate destinations.
  • Plugin and Automation icons differed between the sidebar and + menu.

What changed

  • Replace Extensions with sibling Plugins and Skills entries, each with its own workspace navigation: Browse plugins / Installed plugins and Browse skills / My skills.
  • Make those four subpage rows text-only; preserve their labels, routes, and active states.
  • Use /plugins and /skills as canonical routes. Redirect legacy /extensions/... and /tools/... paths while preserving path parameters, query strings, and hashes.
  • Redirect /settings/plugins to /plugins?view=installed. Keep individual /settings/plugins/:pluginId configuration pages unchanged.
  • Preserve existing page functionality, shipped SDK identifiers, and saved sidebar order/visibility preferences.
  • Match icons across the main sidebar and + menu: Plug02 for Plugins, the original Zap for Skills, and Repeat for Automations. No hover icon swapping.

How you verified

  • Remote CI passed on f5a77a2ce: app, server, integration, package, type/lint, and Linux/macOS packaging checks. All 11 resource-sidebar tests pass, including icon-free rows, destination links, workspace ownership, and active states. Existing redirect and sidebar-preference coverage also passes.
  • Chrome for Testing 151.0.7922.71, isolated branch web app on f5a77a2ce: exercised all four subpage links, active highlighting, return navigation, My skills after reload, and sidebar/menu transitions. Confirmed text-only subpage rows and unchanged main sidebar/+ menu icons. No prompts were submitted or interaction errors observed.
  • Verified light-theme desktop web at 1440×900 and phone-responsive web at 390×844, both DPR 1. Phone screenshots show the open sidebar; this is responsive web coverage, not native iOS verification.
  • Before screenshots use exact merge base 647337060; all after screenshots use exact PR head f5a77a2ce. Matched fixtures, corresponding legacy/canonical pages, light theme, and software rendering. Main navigation uses an empty project/thread fixture and Codex 5.6-Sol Medium; desktop captures show the open + menu, phone capture shows the open sidebar. Plugins captures use the Kanban example.
View Before — 1440×900 After — web/desktop 1440×900 After — mobile 390×844
Main navigation Before: Extensions and old menu icons After: separate destinations and matching menu icons After: phone main sidebar
Browse plugins Before: shared Extensions sidebar After: text-only Plugins subpages After: phone Plugins sidebar
My skills Before: shared Extensions sidebar After: text-only Skills subpages After: phone Skills sidebar

BB-Thread-ID: thr_czdxmmyenv

AGENT GENERATED

@brsbl
brsbl force-pushed the bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv branch from 086af2a to f7d28d2 Compare September 3, 2026 00:43
@brsbl
brsbl force-pushed the bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv branch from f7d28d2 to c748e62 Compare September 3, 2026 04:27
@brsbl
brsbl force-pushed the bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv branch from f0c898b to 3003993 Compare September 9, 2026 22:22
@brsbl
brsbl force-pushed the bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv branch from 36bb541 to cbf974e Compare September 10, 2026 02:10
@brsbl

brsbl commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by #3384, a standalone, single-commit PR against main. The replacement preserves the same feature patch (stable patch ID verified), passes remote CI, and includes refreshed before/after desktop and phone screenshots. Closing this PR preserves its discussion and stack history; nothing is being merged into main.

@brsbl brsbl closed this Sep 10, 2026
@brsbl
brsbl deleted the bb/split-extensions-into-sidebar-rows-thr_czdxmmyenv branch September 10, 2026 19:15
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