skillfish 1.0.1 → 1.0.3

package/README.md

@@ -3,7 +3,7 @@
 Install AI agent skills from GitHub with a single command.
 
 ```bash
-npx skillfish owner/repo
+npx skillfish add owner/repo
 ```
 
 ## Overview
@@ -159,7 +159,7 @@ This CLI collects anonymous, aggregate install counts to understand skill popula
 To opt out, set `DO_NOT_TRACK=1` in your environment:
 
 ```bash
-DO_NOT_TRACK=1 npx skillfish owner/repo
+DO_NOT_TRACK=1 npx skillfish add owner/repo
 ```
 
 Telemetry is also automatically disabled in CI environments (`CI=true`).

package/dist/commands/add.js

@@ -151,6 +151,7 @@ Examples:
     // Install each selected skill
     let totalInstalled = 0;
     let totalSkipped = 0;
+    const telemetryPromises = [];
     // SECURITY: Ask for confirmation before installation (unless --yes is used)
     // Single confirmation for all selected skills
     if (!trustSource && !jsonMode && isInputTTY()) {
@@ -221,14 +222,18 @@ Examples:
         }
         totalInstalled += result.installed.length;
         totalSkipped += result.skipped.length;
-        // Track successful installs (fire-and-forget telemetry)
+        // Track successful installs (telemetry with timeout)
         if (result.installed.length > 0) {
             // Construct github value to match skills.github column format: owner/repo/path/to/skill
             const skillDir = skillPath.replace(/\/?SKILL\.md$/i, '').replace(/^\.?\/?/, '');
             const github = skillDir ? `${owner}/${repo}/${skillDir}` : `${owner}/${repo}`;
-            trackInstall(github);
+            telemetryPromises.push(trackInstall(github));
         }
     }
+    // Wait for telemetry to complete (with timeout built into trackInstall)
+    if (telemetryPromises.length > 0) {
+        await Promise.all(telemetryPromises);
+    }
     // Summary
     if (jsonMode) {
         outputJsonAndExit(EXIT_CODES.SUCCESS);

package/dist/telemetry.d.ts

@@ -1,10 +1,8 @@
 /**
- * Track a skill install. Fire-and-forget - never blocks or throws.
- *
- * NOTE: Due to Node.js event loop behavior, this request may not complete
- * if the CLI process exits immediately after calling. This is acceptable
- * for directional metrics (like npm download counts).
+ * Track a skill install. Returns a promise that resolves when the request
+ * completes (or times out). Never rejects.
  *
  * @param github Full GitHub path (e.g., owner/repo/path/to/skill)
+ * @returns Promise that resolves when telemetry is sent (or times out)
  */
-export declare function trackInstall(github: string): void;
+export declare function trackInstall(github: string): Promise<void>;

package/dist/telemetry.js

@@ -1,27 +1,34 @@
 const TELEMETRY_URL = 'https://mcpmarket.com/api/telemetry';
+/** Timeout for telemetry requests (ms) */
+const TELEMETRY_TIMEOUT = 2000;
 /**
- * Track a skill install. Fire-and-forget - never blocks or throws.
- *
- * NOTE: Due to Node.js event loop behavior, this request may not complete
- * if the CLI process exits immediately after calling. This is acceptable
- * for directional metrics (like npm download counts).
+ * Track a skill install. Returns a promise that resolves when the request
+ * completes (or times out). Never rejects.
  *
  * @param github Full GitHub path (e.g., owner/repo/path/to/skill)
+ * @returns Promise that resolves when telemetry is sent (or times out)
  */
 export function trackInstall(github) {
     try {
-        if (process.env.DO_NOT_TRACK === '1' || process.env.CI === 'true')
-            return;
-        if (!github || github.length > 500)
-            return;
-        // POST with JSON body - fire and forget
-        fetch(TELEMETRY_URL, {
+        if (process.env.DO_NOT_TRACK === '1' || process.env.CI === 'true') {
+            return Promise.resolve();
+        }
+        if (!github || github.length > 500) {
+            return Promise.resolve();
+        }
+        // Race between the fetch and a timeout to ensure we don't block the CLI
+        const fetchPromise = fetch(TELEMETRY_URL, {
             method: 'POST',
             headers: { 'Content-Type': 'application/json' },
             body: JSON.stringify({ github }),
-        }).catch(() => { });
+        }).then(() => { }).catch(() => { });
+        const timeoutPromise = new Promise((resolve) => {
+            setTimeout(resolve, TELEMETRY_TIMEOUT);
+        });
+        return Promise.race([fetchPromise, timeoutPromise]);
     }
     catch {
         // Telemetry should never throw
+        return Promise.resolve();
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillfish",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Install AI agent skills from GitHub with a single command",
   "type": "module",
   "bin": {