skills-atlas-cli 0.15.0 → 0.16.1

package/bin/skills.js

@@ -90,7 +90,10 @@ async function main() {
   if (sub !== 'update' && sub !== 'gap-analyze' && sub !== 'setup' && !process.env.SKILLS_ATLAS_SUBCALL) {
     try { require('../src/data').maybeBackgroundRefresh(); } catch { /* ignore */ }
   }
-  if (sub !== 'telemetry' && !process.env.SKILLS_ATLAS_SUBCALL) telemetry.emit('cli_cmd', { target: sub });
+  if (sub !== 'telemetry' && !process.env.SKILLS_ATLAS_SUBCALL) {
+    telemetry.emit('cli_cmd', { target: sub });
+    try { require('../src/localskills').reportCreated(); } catch { /* ignore */ }
+  }
   try {
     await cmd(rest);
   } catch (e) {

package/bin/telemetry-send.js

@@ -1,20 +1,39 @@
 #!/usr/bin/env node
 'use strict';
-// Detached sender. Reads {endpoint, client, events} from the temp file in argv[2], POSTs once
-// with a short timeout, then deletes the file. Fully fail-silent — telemetry must never matter.
+// Detached sender. Reads {endpoint, client, events} from the temp file in argv[2], POSTs once,
+// then deletes the file. Fully fail-silent — telemetry must never matter.
+// In a proxied environment Node's built-in fetch (undici) IGNORES *_PROXY env, so the POST would
+// silently fail; there we fall back to curl, which honors the proxy. Non-proxied → plain fetch.
 const fs = require('fs');
 const file = process.argv[2];
 const cleanup = () => { try { fs.unlinkSync(file); } catch { /* ignore */ } };
 if (!file) process.exit(0);
 let p; try { p = JSON.parse(fs.readFileSync(file, 'utf8')); } catch { cleanup(); process.exit(0); }
 if (!p || !p.endpoint || !Array.isArray(p.events) || !p.events.length) { cleanup(); process.exit(0); }
+
+const body = JSON.stringify({ client: p.client, events: p.events });
+const proxy = process.env.HTTPS_PROXY || process.env.https_proxy || process.env.HTTP_PROXY ||
+              process.env.http_proxy || process.env.ALL_PROXY || process.env.all_proxy;
+
+if (proxy) {
+  // curl picks up *_PROXY from the environment automatically; body via stdin avoids arg limits.
+  try {
+    require('child_process').spawnSync('curl', [
+      '-s', '--max-time', '4', '-X', 'POST', p.endpoint,
+      '-H', 'Content-Type: application/json', '--data-binary', '@-',
+    ], { input: body, stdio: ['pipe', 'ignore', 'ignore'] });
+  } catch { /* curl missing → drop; telemetry must never matter */ }
+  cleanup();
+  process.exit(0);
+}
+
 try {
   const ac = new AbortController();
   const timer = setTimeout(() => ac.abort(), 2000);
   fetch(p.endpoint, {
     method: 'POST',
     headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({ client: p.client, events: p.events }),
+    body,
     signal: ac.signal,
   }).catch(() => {}).finally(() => { clearTimeout(timer); cleanup(); });
 } catch { cleanup(); }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skills-atlas-cli",
-  "version": "0.15.0",
+  "version": "0.16.1",
   "description": "Search, install and learn AI agent skills from the terminal — powered by the Skills Atlas catalog.",
   "bin": {
     "skills-atlas": "bin/skills.js",

package/src/commands/suggest.js

@@ -183,7 +183,9 @@ module.exports = async function suggest() {
     try {
       const names = candidates.map(c => c.skill);
       require('../apstate').recordSuggested(names);
-      require('../telemetry').emit('ap_suggest', { target: names[0] || '', detail: String(names.length) });
+      // Derived signals only — NEVER the prompt text: how many matched + a coarse length bucket.
+      const plen = Math.min(2000, Math.round(prompt.length / 50) * 50);
+      require('../telemetry').emit('ap_suggest', { target: names[0] || '', detail: 'matched=' + names.length + ' plen=' + plen });
     } catch { /* never break the hook */ }
     writeState(file, state);
   } catch {