@orxataguy/tyr 1.0.42 → 1.0.43

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orxataguy/tyr",
-  "version": "1.0.42",
+  "version": "1.0.43",
   "type": "module",
   "bin": {
     "tyr": "./bin/tyr.js"

package/src/core/sys/modules.ts

@@ -41,8 +41,19 @@ function isValidHttpsUrl(value: string): boolean {
     }
 }
 
+// Allows namespaced command names like 'db:migrate', but not a leading/
+// trailing/doubled ':' (e.g. ':foo', 'foo:', 'a::b'), since those would
+// produce an empty segment.
 function isValidCommandName(name: string): boolean {
-    return /^[a-zA-Z0-9_-]+$/.test(name);
+    return /^[a-zA-Z0-9_-]+(:[a-zA-Z0-9_-]+)*$/.test(name);
+}
+
+// ':' is a valid character in a command name (namespacing, e.g. 'db:migrate')
+// but isn't a safe filename component on every platform (Windows reserves it).
+// Only used when deriving the on-disk file name — the real name with its
+// colons is still what's stored in map.yml, the lockfile, and the manifest.
+function toSafeFilename(commandName: string): string {
+    return commandName.replace(/:/g, '-');
 }
 
 function slugFromUrl(url: string): string {
@@ -183,14 +194,15 @@ export async function syncModules(context: TyrContext, options: { force?: boolea
                 const content = await web.get(fileUrl);
                 const fileContent = typeof content === 'string' ? content : JSON.stringify(content, null, 2);
 
-                const destPath = path.join(userRoot, 'commands', `${commandName}.tyr.ts`);
+                const fileName = toSafeFilename(commandName);
+                const destPath = path.join(userRoot, 'commands', `${fileName}.tyr.ts`);
                 await fs.write(destPath, fileContent);
 
                 if (managed && managed.module !== moduleName) {
                     logger.warn(`'${commandName}' was previously managed by module '${managed.module}'; now owned by '${moduleName}'.`);
                 }
 
-                mapConfig.commands[commandName] = `./commands/${commandName}.tyr.ts`;
+                mapConfig.commands[commandName] = `./commands/${fileName}.tyr.ts`;
                 lock.commands[commandName] = { module: moduleName, manifest: manifestUrl, source: fileUrl };
                 mapChanged = true;
                 lockChanged = true;