fix(isActive): Do not match when path is not at the beginning (#90)

Author: techniq

.changeset/brave-ideas-scream.md

@@ -0,0 +1,5 @@
+---
+'@layerstack/utils': patch
+---
+
+fix(isActive): Do not match when path is not at the beginning

packages/utils/src/lib/routing.test.ts

@@ -26,4 +26,10 @@ describe('isActive()', () => {
     const path = '/foo';
     expect(isActive(currentUrl, path)).false;
   });
+
+  it('path should not match if not at beginning', () => {
+    const currentUrl = new URL('http://localhost/foo/bar');
+    const path = '/bar';
+    expect(isActive(currentUrl, path)).false;
+  });
 });

packages/utils/src/lib/routing.ts

@@ -35,7 +35,7 @@ export function isActive(currentUrl: URL, path: string) {
     // home must be direct match (otherwise matches all)
     return currentUrl.pathname === path;
   } else {
-    // Matches full path next character is `/`
-    return currentUrl.pathname.match(path + '($|\\/)') != null;
+    // Matches path at start of pathname, followed by end or slash
+    return currentUrl.pathname.match('^' + path + '($|\\/)') != null;
   }
 }