fix: keep slash in scoped package names for cache key

Author: Huxpro

package.json

@@ -44,6 +44,7 @@
     "@rslib/core": "0.20.1",
     "@rslint/core": "^0.3.3",
     "@rstest/core": "0.9.5",
+    "@types/cross-spawn": "^6.0.6",
     "@types/fs-extra": "^11.0.4",
     "@types/minimist": "^1.2.5",
     "@types/node": "24.12.0",
@@ -65,5 +66,8 @@
   "publishConfig": {
     "access": "public",
     "registry": "https://registry.npmjs.org/"
+  },
+  "dependencies": {
+    "cross-spawn": "^7.0.6"
   }
 }

pnpm-lock.yaml

@@ -8,7 +8,11 @@ const NPM_TEMPLATE_PREFIX = 'npm:';
  * Sanitize package name and version to create a valid cache key
  */
 export const sanitizeCacheKey = (packageName: string, version: string) => {
-  const normalized = packageName.replace(/[\\/]/g, '_');
+  // Keep the slash for scoped packages (e.g., @scope/package)
+  // but replace other slashes that would be invalid in file paths
+  const normalized = packageName.startsWith('@')
+    ? packageName
+    : packageName.replace(/[\\/]/g, '_');
   const versionLabel = version || 'latest';
   return `${normalized}@${versionLabel}`;
 };

src/template-manager.ts

@@ -2,7 +2,11 @@ import { expect, test } from '@rstest/core';
 import {
   checkCancel,
   create,
+  isNpmTemplate,
   multiselect,
+  resolveCustomTemplate,
+  resolveNpmTemplate,
+  sanitizeCacheKey,
   select,
   text,
 } from '../dist/index.js';
@@ -14,3 +18,37 @@ test('should export public APIs', () => {
   expect(typeof select).toBe('function');
   expect(typeof text).toBe('function');
 });
+
+test('should export npm template utilities', () => {
+  expect(typeof isNpmTemplate).toBe('function');
+  expect(typeof resolveCustomTemplate).toBe('function');
+  expect(typeof resolveNpmTemplate).toBe('function');
+  expect(typeof sanitizeCacheKey).toBe('function');
+});
+
+test('should detect npm templates correctly', () => {
+  // npm: prefix
+  expect(isNpmTemplate('npm:my-package')).toBe(true);
+  expect(isNpmTemplate('npm:@scope/package')).toBe(true);
+
+  // Scoped packages
+  expect(isNpmTemplate('@scope/package')).toBe(true);
+
+  // Pure package names
+  expect(isNpmTemplate('my-package')).toBe(true);
+  expect(isNpmTemplate('my-package-name')).toBe(true);
+
+  // Not npm templates
+  expect(isNpmTemplate('./local-path')).toBe(false);
+  expect(isNpmTemplate('../relative-path')).toBe(false);
+  expect(isNpmTemplate('github:user/repo')).toBe(false);
+  expect(isNpmTemplate('https://example.com')).toBe(false);
+  expect(isNpmTemplate('/absolute/path')).toBe(false);
+});
+
+test('should sanitize cache keys correctly', () => {
+  expect(sanitizeCacheKey('my-package', '1.0.0')).toBe('my-package@1.0.0');
+  expect(sanitizeCacheKey('@scope/package', 'latest')).toBe('@scope/package@latest');
+  expect(sanitizeCacheKey('my-package', '')).toBe('my-package@latest');
+  expect(sanitizeCacheKey('my/package', '1.0.0')).toBe('my_package@1.0.0');
+});