nexscope 1.0.0 → 1.0.2

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "nexscope",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "NexScope CLI tool",
   "main": "src/index.js",
   "bin": {
-    "nexscope": "bin/nexscope.js"
+    "nexscope": "./bin/nexscope.js"
   },
   "scripts": {
     "test": "node bin/nexscope.js --help"
@@ -16,5 +16,8 @@
   "license": "MIT",
   "engines": {
     "node": ">=14.0.0"
+  },
+  "dependencies": {
+    "listr2": "^5.0.8"
   }
 }

package/src/commands/install.js

@@ -2,6 +2,7 @@
 
 const { spawn } = require('child_process');
 const https = require('https');
+const { Listr } = require('listr2');
 
 const SKILLS_REPO = 'nexscope-ai/eCommerce-Skills';
 const REPO_API = 'https://api.github.com/repos/nexscope-ai/eCommerce-Skills/contents';
@@ -47,37 +48,40 @@ async function fetchSkillList() {
   return JSON.parse(data).filter((i) => i.type === 'dir').map((i) => i.name);
 }
 
-// Parse directory-related flags to pass through to npx skills
 function parseDirFlags(args) {
   const flags = [];
-
-  if (args.includes('--global') || args.includes('-g')) {
-    flags.push('-g');
-  }
-
+  if (args.includes('--global') || args.includes('-g')) flags.push('-g');
   const agentIndex = args.findIndex((a) => a === '--agent' || a === '-a');
   if (agentIndex !== -1 && args[agentIndex + 1]) {
     flags.push('--agent', args[agentIndex + 1]);
   }
-
   return flags;
 }
 
-function installOne(skillName, dirFlags) {
-  return new Promise((resolve) => {
-    const skillsArgs = ['skills', 'add', SKILLS_REPO, '--skill', skillName, '-y', ...dirFlags];
-    const proc = spawn('npx', skillsArgs, { shell: true });
-
+function spawnAsync(cmd, args) {
+  return new Promise((resolve, reject) => {
+    const proc = spawn(cmd, args, { shell: true });
     let output = '';
     proc.stdout.on('data', (d) => (output += d.toString()));
     proc.stderr.on('data', (d) => (output += d.toString()));
-
     proc.on('close', (code) => {
-      resolve({ skillName, success: code === 0, output });
+      if (code === 0) resolve(output);
+      else reject(new Error(output.trim()));
     });
   });
 }
 
+async function getInstalledSkills(dirFlags) {
+  try {
+    const output = await spawnAsync('npx', ['skills', 'ls', '--json', ...dirFlags]);
+    const data = JSON.parse(output);
+    // Each entry may be a string or object with a name field
+    return data.map((s) => (typeof s === 'string' ? s : s.name || s.skill || '')).filter(Boolean);
+  } catch {
+    return [];
+  }
+}
+
 function runSkillsSync(skillsArgs) {
   return new Promise((resolve) => {
     const proc = spawn('npx', ['skills', ...skillsArgs], { shell: true, stdio: 'inherit' });
@@ -85,6 +89,10 @@ function runSkillsSync(skillsArgs) {
   });
 }
 
+function runSkillsSilent(skillsArgs) {
+  return spawnAsync('npx', ['skills', ...skillsArgs]);
+}
+
 async function run(args) {
   if (args.includes('--help') || args.includes('-h')) {
     printHelp();
@@ -106,60 +114,50 @@ async function run(args) {
       console.error("Run 'nexscope install --help' for usage.");
       process.exit(1);
     }
-    // Single skill: pass through output directly
-    const code = await runSkillsSync(['add', SKILLS_REPO, '--skill', skillName, '-y', ...dirFlags]);
+
+    const installed = await getInstalledSkills(dirFlags);
+
+    const tasks = new Listr([{
+      title: skillName,
+      skip: () => installed.includes(skillName) ? 'Already installed' : false,
+      task: () => runSkillsSilent(['add', SKILLS_REPO, '--skill', skillName, '-y', ...dirFlags]),
+    }], { exitOnError: false });
+
+    try {
+      await tasks.run();
+    } catch {
+      // error already shown by listr2
+    }
+
     printThankYou();
-    process.exit(code || 0);
   } else {
-    // All skills: fetch list then install concurrently
     process.stdout.write('Fetching skill list... ');
-    const skills = await fetchSkillList();
+    const [skills, installed] = await Promise.all([
+      fetchSkillList(),
+      getInstalledSkills(dirFlags),
+    ]);
     console.log(`${skills.length} skills found.\n`);
 
-    console.log(`Installing all skills concurrently...\n`);
-
-    // Track per-skill status
-    const status = {};
-    skills.forEach((s) => (status[s] = 'pending'));
-
-    function printStatus() {
-      process.stdout.write('\x1B[2J\x1B[0f'); // clear screen
-      console.log(`Installing ${skills.length} skills concurrently...\n`);
-      for (const s of skills) {
-        const icon =
-          status[s] === 'pending' ? '⏳' :
-          status[s] === 'running' ? '⚙ ' :
-          status[s] === 'done'    ? '✓ ' : '✗ ';
-        console.log(`  ${icon} ${s}`);
+    const tasks = new Listr(
+      skills.map((skillName) => ({
+        title: skillName,
+        skip: () => installed.includes(skillName) ? 'Already installed' : false,
+        task: () => spawnAsync('npx', ['skills', 'add', SKILLS_REPO, '--skill', skillName, '-y', ...dirFlags]),
+      })),
+      {
+        concurrent: true,
+        exitOnError: false,
+        rendererOptions: { collapseErrors: false },
       }
-    }
-
-    printStatus();
-
-    const promises = skills.map((skillName) => {
-      status[skillName] = 'running';
-      printStatus();
-      return installOne(skillName, dirFlags).then((result) => {
-        status[skillName] = result.success ? 'done' : 'failed';
-        printStatus();
-        return result;
-      });
-    });
-
-    const results = await Promise.all(promises);
-
-    const succeeded = results.filter((r) => r.success);
-    const failed = results.filter((r) => !r.success);
-
-    console.log(`\n${succeeded.length}/${skills.length} skills installed successfully.`);
+    );
 
-    if (failed.length > 0) {
-      console.log('\nFailed skills:');
-      failed.forEach((r) => console.log(`  ✗ ${r.skillName}`));
+    try {
+      await tasks.run();
+    } catch {
+      // listr2 throws if any task fails; results are already displayed
     }
 
     printThankYou();
-    process.exit(failed.length > 0 ? 1 : 0);
   }
 }