@test-lab-ai/cli 0.2.12 → 0.2.13

package/bin/testlab.mjs

@@ -23,7 +23,7 @@ import { apiFetch } from "../lib/api.mjs"
 import { loadImportFile, runImport } from "../lib/import.mjs"
 import { browserLogin } from "../lib/login.mjs"
 import { EXAMPLES_TEXT } from "../lib/examples.mjs"
-import { TESTLAB_SKILLS, AGENTS, detectAgents, installSkill, installedSkillLocations } from "../lib/skills.mjs"
+import { TESTLAB_SKILLS, AGENTS, detectAgents, installSkill, installedSkillLocations, installedSkillTargets } from "../lib/skills.mjs"
 import { checkForUpdate, currentVersion, previousRunVersion } from "../lib/update-check.mjs"
 
 const log = (...a) => console.log(...a)
@@ -462,44 +462,73 @@ function cmdSkillsList() {
   log(`\nInstall with: testlab skills install [name] [--agent ${AGENTS.join("|")}|all]`)
 }
 
-async function cmdSkillsUpdate() {
-  let refreshed = 0
+// Install EVERY current skill at every location where any test-lab skill is
+// already present — refreshing existing copies AND adding skills shipped since
+// the user last installed, so an update delivers the latest skill SET (not just
+// fresher copies of the old set). `installedSkillTargets()` is the union of
+// (agent, scope) homes; a brand-new skill has no install locations of its own
+// yet, which is exactly why the old per-skill loop never picked it up. Returns
+// one result per (skill, location), with `isNew` flagging a first-time install.
+async function refreshInstalledSkills() {
+  const had = new Set()
   for (const name of TESTLAB_SKILLS) {
-    for (const loc of installedSkillLocations(name)) {
+    for (const loc of installedSkillLocations(name)) had.add(`${name}:${loc.agent}:${loc.global}`)
+  }
+  const results = []
+  for (const { agent, global } of installedSkillTargets()) {
+    for (const name of TESTLAB_SKILLS) {
+      const isNew = !had.has(`${name}:${agent}:${global}`)
       try {
-        const res = await installSkill(name, loc.agent, { global: loc.global })
-        refreshed++
-        log(`✓ ${name} → ${loc.agent}${loc.global ? " (global)" : ""}: ${res.dest}`)
+        const res = await installSkill(name, agent, { global })
+        results.push({ name, agent, global, isNew, ok: true, dest: res.dest })
       } catch (e) {
-        log(`  ✗ ${name} (${loc.agent}): ${e.message}`)
+        results.push({ name, agent, global, isNew, ok: false, error: e.message })
       }
     }
   }
-  if (refreshed === 0) log("No installed skills found here. Run `testlab skills install` first.")
-  else log(`\nRefreshed ${refreshed} location(s). Restart your agent to load the changes.`)
+  return results
+}
+
+async function cmdSkillsUpdate() {
+  const results = await refreshInstalledSkills()
+  if (results.length === 0) {
+    log("No installed skills found here. Run `testlab skills install` first.")
+    return
+  }
+  for (const r of results) {
+    const where = `${r.agent}${r.global ? " (global)" : ""}`
+    if (!r.ok) log(`  ✗ ${r.name} (${where}): ${r.error}`)
+    else log(`${r.isNew ? "+ installed" : "✓ refreshed"} ${r.name} → ${where}: ${r.dest}`)
+  }
+  const ok = results.filter((r) => r.ok)
+  const added = ok.filter((r) => r.isNew).length
+  const bits = []
+  if (ok.length - added) bits.push(`${ok.length - added} refreshed`)
+  if (added) bits.push(`${added} newly installed`)
+  log(`\n${bits.join(", ") || "Nothing to update"}. Restart your agent to load the changes.`)
 }
 
-// After a CLI upgrade, refresh any already-installed skills in place — runs once
-// per version bump (best effort). Skipped in CI / when NO_UPDATE_NOTIFIER is set.
+// After a CLI upgrade, refresh installed skills in place AND add any newly
+// shipped ones — runs once per version bump (best effort). Skipped in CI / when
+// NO_UPDATE_NOTIFIER is set.
 async function maybeRefreshSkillsOnUpgrade() {
   if (process.env.CI || process.env.NO_UPDATE_NOTIFIER) return
   const ver = currentVersion()
   const prev = previousRunVersion(ver)
   if (!prev || prev === ver) return
-  let refreshed = 0
-  for (const name of TESTLAB_SKILLS) {
-    for (const loc of installedSkillLocations(name)) {
-      try {
-        await installSkill(name, loc.agent, { global: loc.global })
-        refreshed++
-      } catch {
-        /* best effort — never break the actual command */
-      }
-    }
-  }
-  if (refreshed > 0) {
-    process.stderr.write(`↻ CLI upgraded ${prev} → ${ver}; refreshed ${refreshed} installed skill location(s).\n`)
+  let ok
+  try {
+    ok = (await refreshInstalledSkills()).filter((r) => r.ok)
+  } catch {
+    return // best effort — never break the actual command
   }
+  if (ok.length === 0) return
+  const added = ok.filter((r) => r.isNew).length
+  const refreshed = ok.length - added
+  const bits = []
+  if (refreshed) bits.push(`refreshed ${refreshed}`)
+  if (added) bits.push(`installed ${added} new`)
+  process.stderr.write(`↻ CLI upgraded ${prev} → ${ver}; ${bits.join(", ")} skill location(s).\n`)
 }
 
 async function main() {

package/lib/skills.mjs

@@ -70,6 +70,23 @@ export function installedSkillLocations(name) {
   ].filter((c) => fs.existsSync(c.path))
 }
 
+/**
+ * The distinct (agent, scope) locations where AT LEAST ONE test-lab skill is
+ * currently installed. `skills update` and the on-upgrade refresh install EVERY
+ * current skill at each of these, so a skill added in a later CLI release lands
+ * wherever the user already keeps test-lab skills — not just fresher copies of
+ * the skills that happened to be installed before.
+ */
+export function installedSkillTargets() {
+  const seen = new Map()
+  for (const name of TESTLAB_SKILLS) {
+    for (const loc of installedSkillLocations(name)) {
+      seen.set(`${loc.agent}:${loc.global}`, { agent: loc.agent, global: loc.global })
+    }
+  }
+  return [...seen.values()]
+}
+
 /** Resolve the install target (base dir + format) for one agent. */
 export function agentTarget(agent, { global } = {}) {
   const home = os.homedir()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@test-lab-ai/cli",
-  "version": "0.2.12",
+  "version": "0.2.13",
   "description": "Import existing test plans into test-lab.ai from the command line (or an AI agent).",
   "type": "module",
   "bin": {