stacklok / toolhive-studio / 25374819099

68%

Build Type

push

github

Committed by web-flow

Commit Message

feat(deep-links): skill registry detail and one-click install (#2187)

* feat(deep-links): add skill registry detail and install intents

Adds two new v1 deep-link intents that mirror the existing MCP server ones, with skills addressed by namespace + skillName:

- open-registry-skill-detail?namespace=<ns>&skillName=<n>

- open-registry-skill-install?namespace=<ns>&skillName=<n>[&version=<v>]

Both params are validated via the existing safeIdentifier regex. The optional version param on -install reuses the same regex (tag-style only — OCI digests sha256:… intentionally rejected because they contain a colon). Parser tests cover valid params, dotted namespaces, missing params, special chars, the digest limitation, and that extra query params are stripped.

Refs #2186

* feat(skills): auto-open install dialog from deep links with version prefill

Wires the skill detail route to honour ?install=true and ?version=<v>, mirroring the MCP registry detail page pattern.

- validateSearch on /skills/$namespace/$skillName extracts install (boolean) and version (string).

- SkillDetailPage now accepts initialInstall and initialVersion props, opens the dialog on mount when initialInstall is true, and listens for the toolhive:open-install-skill-modal custom event so same-page deep-link visits also reopen the dialog with a possibly different version.

- The install dialog is keyed by namespace/name#openCount so it remounts on every (re)open. Required because DialogInstallSkill reads defaultReference/defaultVersion only inside useForm's defaultValues, which would otherwise cache the first deep-link's values.

- Manual Install button resets any deep-link version override and bumps the key, so clicks after a deep link still show the metadata-derived version.

- New skill-detail-page.test.tsx covers the closed-by-default state, mount-time install prefill, version override on mount, same-page reopen via the custom event, and the mismatched-skill ignore case.

Refs #2186

*... (continued)

Coverage Stats

4350 of 7132 branches covered (60.99%)

11 of 20 new or added lines in 3 files covered. (55.0%)

3 existing lines in 3 files now uncovered.

6613 of 9934 relevant lines covered (66.57%)

123.75 hits per line