@npeercy/skills 0.1.6 → 0.1.7

package/bin/skills.js

@@ -9,7 +9,7 @@ import {
   cmdEdit, cmdValidate, cmdPublish, cmdImport, cmdDoctor,
   cmdShare, cmdUnshare, cmdVisibility,
 } from '../lib/skills.js';
-import { loadConfig, saveConfig } from '../lib/config.js';
+import { runStartupChecks } from '../lib/startup.js';
 
 const USAGE = `skill-sharer — CLI skill manager for coding agents
 
@@ -51,49 +51,8 @@ const PKG = JSON.parse(
   readFileSync(join(dirname(fileURLToPath(import.meta.url)), '..', 'package.json'), 'utf8')
 );
 
-function cmpSemver(a, b) {
-  const pa = String(a || '').split('.').map(n => parseInt(n, 10) || 0);
-  const pb = String(b || '').split('.').map(n => parseInt(n, 10) || 0);
-  const len = Math.max(pa.length, pb.length);
-  for (let i = 0; i < len; i++) {
-    const da = pa[i] || 0;
-    const db = pb[i] || 0;
-    if (da > db) return 1;
-    if (da < db) return -1;
-  }
-  return 0;
-}
-
-async function maybeWarnOutdated() {
-  if (process.env.SKILLS_NO_UPDATE_CHECK === '1') return;
-
-  const cfg = loadConfig();
-  const now = Date.now();
-  const oneDayMs = 24 * 60 * 60 * 1000;
-  const lastChecked = cfg.updateCheck?.lastChecked ? Date.parse(cfg.updateCheck.lastChecked) : 0;
-  let latest = cfg.updateCheck?.latestVersion || '';
-
-  if (!lastChecked || now - lastChecked > oneDayMs || !latest) {
-    try {
-      const res = await fetch('https://registry.npmjs.org/@npeercy%2fskills/latest');
-      if (res.ok) {
-        const data = await res.json();
-        latest = data.version || latest;
-        cfg.updateCheck = {
-          lastChecked: new Date(now).toISOString(),
-          latestVersion: latest,
-        };
-        saveConfig(cfg);
-      }
-    } catch {
-      // Silent fail; command should continue even offline.
-    }
-  }
-
-  if (latest && cmpSemver(latest, PKG.version) > 0) {
-    console.log(`⚠ Update available: @npeercy/skills ${PKG.version} → ${latest}`);
-    console.log('  Run: npm install -g @npeercy/skills\n');
-  }
+function hasJsonFlag(args) {
+  return args.includes('--json');
 }
 
 async function main() {
@@ -101,20 +60,24 @@ async function main() {
 
   // `skills` by itself -> list all local skills (managed + unmanaged + bundled)
   if (args.length === 0) {
-    await maybeWarnOutdated();
+    await runStartupChecks({ currentVersion: PKG.version, command: 'list', jsonMode: false });
     await cmdList({ verify: false, outdated: false, json: false });
     return;
   }
 
   if (args[0] === '--help' || args[0] === '-h') {
+    await runStartupChecks({ currentVersion: PKG.version, command: 'help', jsonMode: false });
     console.log(USAGE);
     return;
   }
 
   const cmd = args[0];
   const rest = args.slice(1);
-
-  await maybeWarnOutdated();
+  await runStartupChecks({
+    currentVersion: PKG.version,
+    command: cmd,
+    jsonMode: cmd === 'list' && hasJsonFlag(rest),
+  });
 
   try {
     switch (cmd) {

package/lib/skills.js

@@ -255,6 +255,26 @@ function bundledBuiltins() {
   return out;
 }
 
+export function getLocalDiagnostics() {
+  const st = loadState();
+  const agents = detectAgents();
+  const bundled = bundledBuiltins();
+  const unmanaged = discoverUnmanaged(agents, st);
+  const conflicts = discoverAgentNameConflicts(agents);
+
+  return {
+    state: st,
+    agents,
+    managedCount: Object.keys(st.installed).length,
+    bundledBuiltinsCount: bundled.length,
+    hasBuiltinsInstalled: Object.keys(st.installed).some(id => id.startsWith('__builtin__/local/')),
+    unmanagedCount: unmanaged.size,
+    conflictCount: conflicts.length,
+    unmanaged,
+    conflicts,
+  };
+}
+
 async function installBuiltins(agents) {
   const st = loadState();
 

package/lib/startup.js

@@ -0,0 +1,86 @@
+import { loadConfig, saveConfig } from './config.js';
+import { getLocalDiagnostics } from './skills.js';
+
+function cmpSemver(a, b) {
+  const pa = String(a || '').split('.').map(n => parseInt(n, 10) || 0);
+  const pb = String(b || '').split('.').map(n => parseInt(n, 10) || 0);
+  const len = Math.max(pa.length, pb.length);
+  for (let i = 0; i < len; i++) {
+    const da = pa[i] || 0;
+    const db = pb[i] || 0;
+    if (da > db) return 1;
+    if (da < db) return -1;
+  }
+  return 0;
+}
+
+async function latestPublishedVersion() {
+  const controller = new AbortController();
+  const timeout = setTimeout(() => controller.abort(), 2500);
+  try {
+    const res = await fetch('https://registry.npmjs.org/@npeercy%2fskills/latest', {
+      signal: controller.signal,
+    });
+    if (!res.ok) return '';
+    const data = await res.json();
+    return data.version || '';
+  } catch {
+    return '';
+  } finally {
+    clearTimeout(timeout);
+  }
+}
+
+async function runUpdateCheck(currentVersion, { silent = false } = {}) {
+  if (process.env.SKILLS_NO_UPDATE_CHECK === '1') return;
+
+  const cfg = loadConfig();
+  const latestLive = await latestPublishedVersion();
+  const latest = latestLive || cfg.updateCheck?.latestVersion || '';
+
+  if (latestLive) {
+    cfg.updateCheck = {
+      lastChecked: new Date().toISOString(),
+      latestVersion: latestLive,
+    };
+    saveConfig(cfg);
+  }
+
+  if (!silent && latest && cmpSemver(latest, currentVersion) > 0) {
+    console.log(`⚠ Update available: @npeercy/skills ${currentVersion} → ${latest}`);
+    console.log('  Run: npm install -g @npeercy/skills\n');
+  }
+}
+
+function printSetupWarnings(command, startupInfo, { jsonMode = false } = {}) {
+  if (jsonMode) return;
+
+  // Avoid noisy preflight banner for direct setup/auth commands.
+  if (['init', 'login'].includes(command)) return;
+
+  const warnings = [];
+  if (!startupInfo.hasBuiltinsInstalled && startupInfo.bundledBuiltinsCount > 0) {
+    warnings.push('Built-in skills are not installed. Run: skills init --no-import');
+  }
+  if (startupInfo.unmanagedCount > 0) {
+    warnings.push(`Found ${startupInfo.unmanagedCount} unmanaged local skill(s). Run: skills import`);
+  }
+  if (startupInfo.conflictCount > 0) {
+    warnings.push(`Found ${startupInfo.conflictCount} duplicate skill-name conflict(s). Run: skills doctor`);
+  }
+
+  if (warnings.length > 0) {
+    console.log('⚠ Startup checks:');
+    for (const w of warnings) {
+      console.log(`  - ${w}`);
+    }
+    console.log('');
+  }
+}
+
+export async function runStartupChecks({ currentVersion, command, jsonMode = false }) {
+  await runUpdateCheck(currentVersion, { silent: jsonMode });
+  const startupInfo = getLocalDiagnostics();
+  printSetupWarnings(command, startupInfo, { jsonMode });
+  return startupInfo;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@npeercy/skills",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "CLI-first skill marketplace for coding agents",
   "type": "module",
   "bin": {