fix: slug string conflict with scope

yarnpkg · Feb 2, 2024 · 2028879 · 2028879
1 parent 943a9f0
commit 2028879
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/packages/yarnpkg-core/sources/structUtils.ts b/packages/yarnpkg-core/sources/structUtils.ts
@@ -665,7 +665,7 @@ export function stringifyLocator(locator: Locator) {
  */
 export function slugifyIdent(ident: Ident) {
   if (ident.scope !== null) {
-    return `@${ident.scope}-${ident.name}`;
+    return `@${ident.scope}_${ident.name}`;
   } else {
     return ident.name;
   }

diff --git a/packages/yarnpkg-core/tests/structUtils.test.ts b/packages/yarnpkg-core/tests/structUtils.test.ts
@@ -0,0 +1,13 @@
+import * as structUtils from '../sources/structUtils';
+
+describe(`structUtils`, () => {
+  describe(`slugifyIdent`, () => {
+    it(`should return a unique slug string with scope`, () => {
+      const slugA = structUtils.slugifyIdent({scope: `myscope`, name: `user-email`});
+      expect(slugA).toEqual(`@myscope_user-email`);
+      const slugB = structUtils.slugifyIdent({scope: `myscope-user`, name: `email`});
+      expect(slugB).toEqual(`@myscope-user_email`);
+      expect(slugA).not.toEqual(slugB);
+    });
+  });
+});