refactor: use registryVersion field instead of parsing attempt messages

Add registryVersion to ResolveResult so shipped skills fallback
doesn't need to regex-parse human readable log messages.

Author: harlan-zw

src/commands/sync-parallel.ts

@@ -401,7 +401,7 @@ async function syncBaseSkill(
   const localDeps = await readLocalDependencies(cwd).catch(() => [])
   const localVersion = localDeps.find(d => d.name === packageName)?.version
 
-  const { package: resolvedPkg, attempts } = await resolvePackageDocsWithAttempts(requestedTag ? packageSpec : packageName, {
+  const { package: resolvedPkg, attempts, registryVersion } = await resolvePackageDocsWithAttempts(requestedTag ? packageSpec : packageName, {
     version: localVersion,
     cwd,
     onProgress: step => update(packageName, 'resolving', RESOLVE_STEP_LABELS[step]),
@@ -415,7 +415,7 @@ async function syncBaseSkill(
 
   if (!resolved) {
     // Even without docs, the package may ship its own skills (skills-npm convention)
-    const shippedVersion = localVersion || attempts.find(a => a.source === 'npm' && a.status === 'success')?.message?.match(/@(.+)$/)?.[1] || 'latest'
+    const shippedVersion = localVersion || registryVersion || 'latest'
     const earlyShipped = handleShippedSkills(packageName, shippedVersion, cwd, config.agent, config.global)
     if (earlyShipped) {
       const shared = !config.global && getSharedSkillsDir(cwd)

src/commands/sync.ts

@@ -291,7 +291,7 @@ async function syncSinglePackage(packageSpec: string, config: SyncConfig): Promi
 
   if (!resolved) {
     // Even without docs, the package may ship its own skills (skills-npm convention)
-    const shippedVersion = localVersion || resolveResult.attempts.find(a => a.source === 'npm' && a.status === 'success')?.message?.match(/@(.+)$/)?.[1] || 'latest'
+    const shippedVersion = localVersion || resolveResult.registryVersion || 'latest'
     const earlyShipped = handleShippedSkills(packageName, shippedVersion, cwd, config.agent, config.global)
     if (earlyShipped) {
       const shared = !config.global && getSharedSkillsDir(cwd)

src/sources/npm.ts

@@ -375,10 +375,10 @@ export async function resolvePackageDocsWithAttempts(packageName: string, option
 
   // Must have at least one source
   if (!result.docsUrl && !result.llmsUrl && !result.readmeUrl && !result.gitDocsUrl) {
-    return { package: null, attempts }
+    return { package: null, attempts, registryVersion: pkg.version }
   }
 
-  return { package: result, attempts }
+  return { package: result, attempts, registryVersion: pkg.version }
 }
 
 /**

src/sources/types.ts

@@ -74,4 +74,6 @@ export interface ResolveAttempt {
 export interface ResolveResult {
   package: ResolvedPackage | null
   attempts: ResolveAttempt[]
+  /** npm registry version, available even when doc resolution fails */
+  registryVersion?: string
 }