fix: escape package name (see #127) (#128)

This change fixes [#127](#127) by simply escaping (i.e. replacing) all non-alphanumeric characters with `-` (including `-` itself). While the `escapePath()` is overly strict (e.g. does not allow a valid `_` character) it does not need relaxing for now.

Lifting limitations is always backwards-compatible; introducing them is not.

Co-authored-by: Mariusz Pasinski <[email protected]>

Author: mani3xis

packages/host/src/node/path-utils.ts

@@ -148,9 +148,10 @@ export function escapePath(modulePath: string) {
  */
 export function getLibraryName(modulePath: string, naming: NamingStrategy) {
   const { packageName, relativePath } = determineModuleContext(modulePath);
+  const escapedPackageName = escapePath(packageName);
   return naming.stripPathSuffix
-    ? packageName
-    : `${packageName}--${escapePath(relativePath)}`;
+    ? escapedPackageName
+    : `${escapedPackageName}--${escapePath(relativePath)}`;
 }
 
 export function prettyPath(p: string) {