@lightdash/common 0.1458.0 → 0.1459.0

package/dist/utils/slug.test.js

@@ -0,0 +1,42 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const slugs_1 = require("./slugs");
+describe('Slug', () => {
+    test('should generate space slugs', async () => {
+        expect((0, slugs_1.generateSlug)('my space name')).toEqual('my-space-name');
+        expect((0, slugs_1.generateSlug)('my space name')).toEqual('my-space-name');
+        expect((0, slugs_1.generateSlug)('a_SPACE_NAME.with!special?chars/and\\slashes')).toEqual('a-space-name-with-special-chars-and-slashes');
+        expect((0, slugs_1.generateSlug)('!trim_special_chars!')).toEqual('trim-special-chars');
+    });
+    test('should generate chart slugs', async () => {
+        expect((0, slugs_1.generateSlug)('a CHART_NAME.with!special?chars/and\\slashes')).toEqual('a-chart-name-with-special-chars-and-slashes');
+    });
+    test('should generate dashboard slugs', async () => {
+        expect((0, slugs_1.generateSlug)('my dashboard name')).toEqual('my-dashboard-name');
+        expect((0, slugs_1.generateSlug)('a DASHBOARD_NAME.with!special?chars/and\\slashes')).toEqual('a-dashboard-name-with-special-chars-and-slashes');
+    });
+    test('should handle emojis and special characters properly', () => {
+        // Test multiple emojis and special characters
+        expect((0, slugs_1.generateSlug)('🎉 Party 🎊 Time! 🎈 (Special Event)')).toEqual('party-time-special-event');
+        // Test emoji at the end
+        expect((0, slugs_1.generateSlug)('My Dashboard 🚀')).toEqual('my-dashboard');
+        // Ensure backwards compatibility with existing test cases
+        expect((0, slugs_1.generateSlug)('my dashboard name')).toEqual('my-dashboard-name');
+        expect((0, slugs_1.generateSlug)('!special.chars!')).toEqual('special-chars');
+        // Test Unicode emojis
+        expect((0, slugs_1.generateSlug)('📊 Analytics Dashboard 👨‍💻')).toEqual('analytics-dashboard');
+        // Handle japanese characters
+        expect((0, slugs_1.generateSlug)('Lightdashライトダッシュ')).toEqual('lightdash');
+        // Return short slug (but not empty) if all characters are special
+        expect((0, slugs_1.generateSlug)('LDライトダッシュ')).toEqual('ld');
+        expect((0, slugs_1.generateSlug)('!"·$%&0!"·$%')).toEqual('0');
+        expect((0, slugs_1.generateSlug)('x!!!!')).toEqual('x');
+    });
+    test('if all slug are special characters, return a 5 char random string', () => {
+        expect((0, slugs_1.generateSlug)('!!!!').length).toEqual(5);
+        // Test multiple emojis andspecial characters
+        expect((0, slugs_1.generateSlug)('🎉🎊').length).toEqual(5);
+        // Handle japanese characters
+        expect((0, slugs_1.generateSlug)('ライトダッシュ').length).toEqual(5);
+    });
+});

package/dist/utils/slugs.js

@@ -3,8 +3,19 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.generateSlug = void 0;
 const sanitizeSlug = (slug) => slug
     .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-') // Replace all non-alphanumeric characters with hyphens
-    .replace(/-+/g, '-') // Replace multiple hyphens with a single hyphen
-    .replace(/^-+|-+$/g, ''); // Trim leading and trailing hyphens
-const generateSlug = (name) => `${sanitizeSlug(name)}`;
+    // Then replace remaining non-alphanumeric characters with hyphens
+    .replace(/[^a-z0-9]+/g, '-')
+    // Replace multiple hyphens with a single hyphen
+    .replace(/-+/g, '-')
+    // Trim leading and trailing hyphens
+    .replace(/^-+|-+$/g, '');
+const generateSlug = (name) => {
+    const sanitizedSlug = sanitizeSlug(name);
+    if (sanitizedSlug.length === 0) {
+        // Return a random 5 character string
+        // Base-36 uses all digits (0-9) and all letters (a-z)
+        return Math.random().toString(36).substring(2, 7);
+    }
+    return sanitizedSlug;
+};
 exports.generateSlug = generateSlug;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightdash/common",
-  "version": "0.1458.0",
+  "version": "0.1459.0",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "files": [

package/dist/slug.test.js

@@ -1,18 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-const slugs_1 = require("./utils/slugs");
-describe('Slug', () => {
-    test('should generate space slugs', async () => {
-        expect((0, slugs_1.generateSlug)('my space name')).toEqual('my-space-name');
-        expect((0, slugs_1.generateSlug)('my space name')).toEqual('my-space-name');
-        expect((0, slugs_1.generateSlug)('a_SPACE_NAME.with!special?chars/and\\slashes')).toEqual('a-space-name-with-special-chars-and-slashes');
-        expect((0, slugs_1.generateSlug)('!trim_special_chars!')).toEqual('trim-special-chars');
-    });
-    test('should generate chart slugs', async () => {
-        expect((0, slugs_1.generateSlug)('a CHART_NAME.with!special?chars/and\\slashes')).toEqual('a-chart-name-with-special-chars-and-slashes');
-    });
-    test('should generate dashboard slugs', async () => {
-        expect((0, slugs_1.generateSlug)('my dashboard name')).toEqual('my-dashboard-name');
-        expect((0, slugs_1.generateSlug)('a DASHBOARD_NAME.with!special?chars/and\\slashes')).toEqual('a-dashboard-name-with-special-chars-and-slashes');
-    });
-});