@impulselab/directory 1.0.6 → 1.0.7

package/index.js

@@ -133,6 +133,60 @@ function shouldUseHttps(baseUrl) {
   return true;
 }
 
+function toKebabCase(input) {
+  return String(input)
+    .normalize('NFKD')                    // strip accents
+    .replace(/[\u0300-\u036f]/g, '')
+    .replace(/['"]/g, '')
+    .replace(/[^a-zA-Z0-9]+/g, '-')       // collapse non-alnum to dashes
+    .replace(/^-+|-+$/g, '')
+    .toLowerCase();
+}
+
+function stripTrailingIdSegment(slug) {
+  const normalized = toKebabCase(slug || '');
+  const segments = normalized.split('-');
+  if (segments.length <= 1) return normalized;
+  const last = segments[segments.length - 1];
+  if (/^[a-z0-9]{5,}$/i.test(last)) {
+    segments.pop();
+  }
+  return segments.join('-');
+}
+
+function sanitizeIdentifierForFilename(identifier) {
+  return String(identifier || '')
+    .normalize('NFKD')
+    .replace(/[\u0300-\u036f]/g, '')      // remove accents
+    .replace(/[\u0000-\u001f\u007f-\u009f]/g, '') // remove control chars
+    .replace(/[<>:"/\\|?*]/g, '')         // remove filesystem unsafe chars
+    .replace(/\.\./g, '')                 // remove parent directory traversal
+    .replace(/[/\\]+/g, '-')              // replace all path separators with dashes
+    .replace(/^\.+/g, '')                 // remove leading dots
+    .replace(/\.+$/g, '')                 // remove trailing dots
+    .replace(/^-+|-+$/g, '')              // trim edge dashes
+    .replace(/-+/g, '-')                  // collapse multiple dashes
+    .toLowerCase()
+    || 'unknown';                         // fallback for empty results
+}
+
+function generateUniqueFilename(directory, baseName, extension) {
+  const baseFilename = `${baseName}${extension}`;
+  
+  if (!checkFileExists(directory, baseFilename)) {
+    return baseFilename;
+  }
+  
+  let counter = 1;
+  let candidate = `${baseName}-${counter}${extension}`;
+  while (checkFileExists(directory, candidate)) {
+    counter += 1;
+    candidate = `${baseName}-${counter}${extension}`;
+  }
+  
+  return candidate;
+}
+
 function downloadCommand(baseUrl, fullPath) {
   return new Promise((resolve, reject) => {
     const useHttps = shouldUseHttps(baseUrl);
@@ -290,6 +344,58 @@ function downloadStackWithHttp(baseUrl, stackId) {
   });
 }
 
+function downloadRawWithHeaders(baseUrl, fullPath) {
+  return new Promise((resolve, reject) => {
+    const useHttps = shouldUseHttps(baseUrl);
+    const protocol = useHttps ? https : http;
+    const scheme = useHttps ? 'https' : 'http';
+    const url = `${scheme}://${baseUrl}${RAW_PATH}${fullPath}`;
+
+    const handleOk = (res) => {
+      let data = '';
+      res.on('data', (chunk) => { data += chunk; });
+      res.on('end', () => resolve({ content: data, headers: res.headers }));
+    };
+
+    const handleHttpsFailover = (reason) => {
+      if (useHttps && canFallbackToHttp(baseUrl)) {
+        console.log(`HTTPS failed (${reason}), trying HTTP...`);
+        downloadRawWithHeadersHttp(baseUrl, fullPath).then(resolve).catch(reject);
+      } else {
+        reject(new Error(typeof reason === 'string' ? reason : reason.message));
+      }
+    };
+
+    protocol
+      .get(url, (res) => {
+        if (res.statusCode === 200) return handleOk(res);
+        if (res.statusCode === 404) return reject(new Error(`Command '${fullPath}' not found (404)`));
+        return handleHttpsFailover(`${res.statusCode}`);
+      })
+      .on('error', handleHttpsFailover);
+  });
+}
+
+function downloadRawWithHeadersHttp(baseUrl, fullPath) {
+  return new Promise((resolve, reject) => {
+    const url = `http://${baseUrl}${RAW_PATH}${fullPath}`;
+    console.log(`Trying HTTP: ${url}`);
+    http
+      .get(url, (res) => {
+        if (res.statusCode === 200) {
+          let data = '';
+          res.on('data', (chunk) => { data += chunk; });
+          res.on('end', () => resolve({ content: data, headers: res.headers }));
+        } else if (res.statusCode === 404) {
+          reject(new Error(`Command '${fullPath}' not found (404)`));
+        } else {
+          reject(new Error(`HTTP ${res.statusCode}: ${res.statusMessage}`));
+        }
+      })
+      .on('error', reject);
+  });
+}
+
 function findClosestFolder(folderName) {
   let currentDir = process.cwd();
 
@@ -505,13 +611,19 @@ async function main() {
 
       ensureDirectoryExists(directory);
 
-      const commandName = identifier.split('/').pop();
-      const simpleFilename = `${commandName}${targetConfig.extension}`;
-      const fallbackFilename = `${identifier.replace('/', '-')}${targetConfig.extension}`;
-      const fileExists = checkFileExists(directory, simpleFilename);
-      const filename = fileExists ? fallbackFilename : simpleFilename;
+      const commandNameFromPath = identifier.split('/').pop();
+      // First download the content to capture headers (and title)
+      const { content, headers } = await downloadRawWithHeaders(baseUrl, identifier);
+      const titleHeader = headers['x-prompt-title'];
+      const title = Array.isArray(titleHeader) ? titleHeader[0] : titleHeader;
 
-      const content = await downloadCommand(baseUrl, identifier);
+      const baseNameCandidate = title
+        ? toKebabCase(title)
+        : stripTrailingIdSegment(commandNameFromPath);
+      const finalBaseName = sanitizeIdentifierForFilename(baseNameCandidate);
+
+      // Overwrite if exists; do not append author or random suffix
+      const filename = `${finalBaseName}${targetConfig.extension}`;
 
       saveCommand({
         directory,
@@ -547,7 +659,9 @@ async function main() {
     for (let index = 0; index < prompts.length; index += 1) {
       const prompt = prompts[index];
       const commandPath = prompt.path || prompt.slug || prompt.id || `prompt-${index + 1}`;
-      const baseName = prompt.slug || `prompt-${index + 1}`;
+      const baseName = prompt.title
+        ? toKebabCase(prompt.title)
+        : stripTrailingIdSegment(prompt.slug || `prompt-${index + 1}`);
 
       const resolvedTarget = target || prompt.target || DEFAULT_TARGET;
       const targetConfig = TARGET_CONFIG[resolvedTarget];
@@ -560,30 +674,9 @@ async function main() {
       const directory = targetConfig.resolveDirectory();
       ensureDirectoryExists(directory);
 
-      const simpleFilename = `${baseName}${targetConfig.extension}`;
-      const fallbackBase = stack.slug
-        ? `${stack.slug}-${baseName}`
-        : `${stack.id || identifier}-${baseName}`;
-      const fallbackFilename = `${fallbackBase}${targetConfig.extension}`;
-
-      const simpleExists = checkFileExists(directory, simpleFilename);
-      const fallbackExists = checkFileExists(directory, fallbackFilename);
-
-      let filename = simpleFilename;
-      if (simpleExists) {
-        filename = fallbackFilename;
-      }
-      if ((simpleExists && fallbackExists) || checkFileExists(directory, filename)) {
-        const baseForUniq = fallbackBase;
-        const ext = targetConfig.extension;
-        let counter = 1;
-        let candidate = `${baseForUniq}-${counter}${ext}`;
-        while (checkFileExists(directory, candidate)) {
-          counter += 1;
-          candidate = `${baseForUniq}-${counter}${ext}`;
-        }
-        filename = candidate;
-      }
+      // Sanitize and overwrite if exists (no author or random suffix)
+      const sanitizedBaseName = sanitizeIdentifierForFilename(baseName);
+      const filename = `${sanitizedBaseName}${targetConfig.extension}`;
 
       if (!prompt.markdown) {
         console.warn(`⚠️  Skipping prompt '${commandPath}' (no content available)`);

package/package.json

@@ -1,16 +1,11 @@
 {
   "name": "@impulselab/directory",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "description": "Download and install Claude commands from Impulse Directory",
   "main": "index.js",
   "bin": {
     "directory": "./index.js"
   },
-  "scripts": {
-    "test": "node index.js --help",
-    "predeploy": "chmod +x index.js",
-    "deploy": "pnpm publish --access public --no-git-checks"
-  },
   "keywords": [
     "claude",
     "commands",
@@ -25,5 +20,10 @@
   },
   "publishConfig": {
     "access": "public"
+  },
+  "scripts": {
+    "test": "node index.js --help",
+    "predeploy": "chmod +x index.js",
+    "deploy": "pnpm publish --access public --no-git-checks"
   }
-}
+}