Repository navigation
Write a skill on a screen of its own, and come back to lists where you left them - #906
Open
guidovizoso wants to merge 4 commits into
Open
guidovizoso wants to merge 4 commits into
guidovizoso wants to merge 4 commits into
Conversation
As a full-width band it covered the top of every scrolled page for a control that sits beside the centred column. It now floats over the scroller's corner, only as wide as its buttons, and the column keeps the same starting position through its own padding.
New skill and each skill's page (/capabilities/skills/new and /capabilities/skills/$slug) replace the dialogs. The form is split into Details, Instructions, Tools and Bots sections; tools are picked one app at a time, in a step whose list scrolls inside the window; Bots are switches, picked before creating or saved as switched on an existing skill; and Delete is a section of the skill's page, behind a confirmation. The list shows each skill as a card that opens it, with its command and a chip for the Bots it is on and the tools it needs. The old ?new and ?edit addresses redirect to the new screens.
Scroll restoration was never turned on, so every list started over at the top after a detail screen. It is now on, keyed by address rather than history entry, because the way back is usually the back button, a link that makes a new entry. PageShell's scroller is named so it is restored too, and a page opened for the first time still starts at the top.
The demonstration recorder that used the single-column form is gone from main, so the form and the tools picker are only ever a screen of their own; the inline layout and its framed and fit options had no caller left.
guidovizoso
requested review from
MikeRyanDev,
davidmckayv,
dustywusty,
mxmzb,
samjulien and
tylerslaton
as code owners
October 30, 2026 15:24
Open
9 tasks done
This branch has not been deployed
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
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.
What this changes
Skills get screens of their own, laid out a section at a time. New skill (
/capabilities/skills/new) and each skill's page (/capabilities/skills/$slug) replace the dialogs. The?newand?edit=addresses redirect to them.The list: each skill is a card that opens it when clicked. It shows the command as a pill and a chip such as "2 Bots · 5 tools", with small Bot avatars. The "Put on a Bot" button is gone.
Back button: it floats in the page's top-left corner, only as wide as itself, instead of a full-width band that covered the top of a scrolled page.
Lists keep their scroll position. TanStack Router's scroll restoration was never turned on, so every list started at the top after a detail screen. It is now on, keyed by address rather than history entry. The "<" button opens the list as a new history entry, so keying by entry would never find a position to restore. PageShell's scroller carries
data-scroll-restoration-id, so pages under_authed/_app(which scroll inside the page, not the window) are restored too. A page opened for the first time still starts at the top.Cleanup:
mainremoved the demonstration recorder, the only user of the skill form's single-column layout, so that layout and theframed/fitoptions are gone.Where it runs
sessionStorage, through TanStack Router.POST/DELETE /api/plugins/skillsandPOST/DELETE /api/plugins/grants.Boundary and audit
Changelog
Unreleased.Proof
bunx tsc --noEmitinapp/passes.bunx biome lint --error-on-warnings app/srcpasses.biome checkon the changed files passes; the 4 import-order errors it reports elsewhere are already onmain.bun test app/tests: 2128 pass and 1 fails. The failure isserve.test.ts› "a stalled handshake times out…", which this branch doesn't touch./sections-testwith a Bot switched on created the skill already on that Bot./admin/pluginsscrolled to 400 came back to 400 after opening Composio, which opened at 0./capabilities/skillsscrolled to 150 came back to 150, both with the back button and with browser Back.