stacklok / toolhive-studio / 25329799380

66%

main: 71%

Pull #2185

github

fix(skills): address Copilot review feedback on install-dialog prefill

Three follow-ups to PR #2185 catching edge cases reported by the
Copilot reviewer:

1. `getBuildInstallDefaults` no longer treats a bare OCI reference like
   `ghcr.io/org/my-skill` (no `:tag`, no `@digest`) as a version when
   `build.name` is also present. Only version-only strings without a `/`
   may now be promoted to the Version field.
2. The "Install now" toast after a successful build now parses the
   user-supplied build tag before falling back to it as the version, so
   pasting a full OCI reference into the build form's Tag field never
   sends `version: "ghcr.io/org/skill:v3.4.5"` to the install API.
3. The dialog-install-skill blur splitter now keys off
   `dirtyFields.version` (subscribed via `useFormState`) instead of
   raw emptiness. A version that came from `defaultVersion` is still
   considered pristine, so pasting a new `name:tag` into the Name field
   correctly replaces the prefill instead of leaving a mismatched
   name/version pair behind.

Co-authored-by: Cursor <cursoragent@cursor.com>

4327 of 7116 branches covered (60.81%)

31 of 32 new or added lines in 8 files covered. (96.88%)

6593 of 9918 relevant lines covered (66.48%)

123.72 hits per line