@pequity/squirrel 4.1.0 → 4.1.2

package/dist/cjs/link.js

@@ -14,12 +14,8 @@ const isValidUrl = (url) => {
     return false;
   }
 };
-const checkDomain = function(url) {
-  url = normalizeUrl(url);
-  return url.toLowerCase().replace(/([a-z])?:\/\//, "$1").split("/")[0];
-};
 const isExternalLink = function(url) {
   url = String(url);
-  return isValidUrl(url) && (url.indexOf(":") > -1 || url.indexOf("//") > -1) && checkDomain(location.href) !== checkDomain(url);
+  return isValidUrl(url) && (url.indexOf(":") > -1 || url.indexOf("//") > -1);
 };
 exports.isExternalLink = isExternalLink;

package/dist/es/link.js

@@ -12,13 +12,9 @@ const isValidUrl = (url) => {
     return false;
   }
 };
-const checkDomain = function(url) {
-  url = normalizeUrl(url);
-  return url.toLowerCase().replace(/([a-z])?:\/\//, "$1").split("/")[0];
-};
 const isExternalLink = function(url) {
   url = String(url);
-  return isValidUrl(url) && (url.indexOf(":") > -1 || url.indexOf("//") > -1) && checkDomain(location.href) !== checkDomain(url);
+  return isValidUrl(url) && (url.indexOf(":") > -1 || url.indexOf("//") > -1);
 };
 export {
   isExternalLink

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@pequity/squirrel",
   "description": "Squirrel component library",
-  "version": "4.1.0",
+  "version": "4.1.2",
   "packageManager": "pnpm@9.7.1",
   "type": "module",
   "scripts": {
@@ -52,8 +52,8 @@
   "devDependencies": {
     "@babel/core": "^7.25.2",
     "@babel/preset-env": "^7.25.4",
-    "@commitlint/cli": "^19.4.0",
-    "@commitlint/config-conventional": "^19.2.2",
+    "@commitlint/cli": "^19.4.1",
+    "@commitlint/config-conventional": "^19.4.1",
     "@pequity/eslint-config": "^0.0.13",
     "@playwright/test": "^1.46.1",
     "@popperjs/core": "2.11.8",
@@ -71,11 +71,11 @@
     "@storybook/theming": "^8.2.9",
     "@storybook/vue3": "^8.2.9",
     "@storybook/vue3-vite": "^8.2.9",
-    "@tanstack/vue-virtual": "3.10.4",
+    "@tanstack/vue-virtual": "3.10.5",
     "@types/jest": "^29.5.12",
     "@types/jsdom": "^21.1.7",
     "@types/lodash-es": "^4.17.12",
-    "@types/node": "^22.5.0",
+    "@types/node": "^22.5.1",
     "@vitejs/plugin-vue": "^5.1.2",
     "@vue/compiler-sfc": "3.4.38",
     "@vue/test-utils": "^2.4.6",

package/squirrel/utils/link.spec.js

@@ -1,12 +1,22 @@
 import { isExternalLink } from '@squirrel/utils/link';
 
 describe('isExternalLink', () => {
-  it.each(['https://www.example.com', 'http://www.example.com', '//www.example.com'])(
-    'should return true for external links (%s)',
-    (val) => {
-      expect(isExternalLink(val)).toBe(true);
-    }
-  );
+  beforeEach(() => {
+    delete window.location;
+
+    window.location = new URL('https://www.example.com');
+  });
+
+  it.each([
+    'https://www.example.com',
+    'http://www.example.com',
+    '//www.example.com',
+    'https://www.pequity.com',
+    'http://www.pequity.com',
+    '//www.pequity.com',
+  ])('should return true for external links (%s)', (val) => {
+    expect(isExternalLink(val)).toBe(true);
+  });
 
   it.each(['/home', 'home', '#home', '/home//1/', '/home:1/', '#home:', { name: 'home' }])(
     'should return false for internal links (%s)',
@@ -15,10 +25,14 @@ describe('isExternalLink', () => {
     }
   );
 
-  it.each(['ftp://www.example.com', 'mailto:test@example.com', 'tel:+1234567890', 'sms:+1234567890'])(
-    'should handle different protocols (%s)',
-    (val) => {
-      expect(isExternalLink(val)).toBe(true);
-    }
-  );
+  it.each([
+    'ftp://www.example.com',
+    'mailto:test@example.com',
+    'ftp://www.pequity.com',
+    'mailto:test@pequity.com',
+    'tel:+1234567890',
+    'sms:+1234567890',
+  ])('should handle different protocols (%s)', (val) => {
+    expect(isExternalLink(val)).toBe(true);
+  });
 });

package/squirrel/utils/link.ts

@@ -16,21 +16,8 @@ const isValidUrl = (url: string) => {
   }
 };
 
-const checkDomain = function (url: string) {
-  url = normalizeUrl(url);
-
-  return url
-    .toLowerCase()
-    .replace(/([a-z])?:\/\//, '$1')
-    .split('/')[0];
-};
-
 export const isExternalLink = function (url: string) {
   url = String(url);
 
-  return (
-    isValidUrl(url) &&
-    (url.indexOf(':') > -1 || url.indexOf('//') > -1) &&
-    checkDomain(location.href) !== checkDomain(url)
-  );
+  return isValidUrl(url) && (url.indexOf(':') > -1 || url.indexOf('//') > -1);
 };