Inline isWindows check to getLocationAttr;

Remove `"@solid-primitives/platform"` dependency.

Author: thetarnav

.changeset/slick-glasses-dance.md

@@ -0,0 +1,5 @@
+---
+"@solid-devtools/debugger": patch
+---
+
+Inline `isWindows` check to `getLocationAttr` and remove `"@solid-primitives/platform"` dependency.

packages/debugger/package.json

@@ -54,7 +54,6 @@
         "@solid-primitives/bounds": "^0.1.1",
         "@solid-primitives/event-listener": "^2.4.1",
         "@solid-primitives/keyboard": "^1.3.1",
-        "@solid-primitives/platform": "^0.2.1",
         "@solid-primitives/rootless": "^1.5.1",
         "@solid-primitives/scheduled": "^1.5.1",
         "@solid-primitives/static-store": "^0.1.1",

packages/debugger/src/locator/index.test.ts

@@ -1,36 +1,18 @@
-let mockIsWindows = true
+import * as v from 'vitest'
+import {parseLocationString} from './locator.ts'
 
-import {beforeEach, describe, expect, test, vi} from 'vitest'
+v.describe('locator attribute pasting', () => {
 
-vi.mock('@solid-primitives/platform', () => ({
-    get isWindows() {
-        return mockIsWindows
-    },
-}))
-
-const fetchFunction = async () => (await import('./locator.ts')).parseLocationString
-
-describe('locator attribute pasting', () => {
-    beforeEach(() => {
-        vi.resetModules()
-    })
-
-    test('windows', async () => {
-        mockIsWindows = true
-        const getLocationFromAttribute = await fetchFunction()
-
-        expect(getLocationFromAttribute(`Users\\user\\Desktop\\test\\test.tsx:1:0`)).toEqual({
+    v.test('windows', () => {
+        v.expect(parseLocationString(`Users\\user\\Desktop\\test\\test.tsx:1:0`)).toEqual({
             file: 'Users\\user\\Desktop\\test\\test.tsx',
             line: 1,
             column: 0,
         })
     })
 
-    test('unix', async () => {
-        mockIsWindows = false
-        const getLocationFromAttribute = await fetchFunction()
-
-        expect(getLocationFromAttribute(`/home/username/project/src/App.tsx:10:5`)).toEqual({
+    v.test('unix', () => {
+        v.expect(parseLocationString(`/home/username/project/src/App.tsx:10:5`)).toEqual({
             file: '/home/username/project/src/App.tsx',
             line: 10,
             column: 5,

packages/debugger/src/locator/locator.ts

@@ -1,5 +1,4 @@
 import type {KbdKey} from '@solid-primitives/keyboard'
-import {isWindows} from '@solid-primitives/platform'
 import type {ToDyscriminatedUnion} from '@solid-devtools/shared/utils'
 import {type NodeID, type SourceLocation} from '../main/types.ts'
 
@@ -14,13 +13,13 @@ export type LocatorOptions = {
 }
 
 export type HighlightElementPayload = ToDyscriminatedUnion<{
-    node: {id: NodeID}
+    node:    {id: NodeID}
     element: {id: NodeID}
 }> | null
 
 // used by the transform
 export const WINDOW_PROJECTPATH_PROPERTY = '$sdt_projectPath'
-export const LOCATION_ATTRIBUTE_NAME = 'data-source-loc'
+export const LOCATION_ATTRIBUTE_NAME     = 'data-source-loc'
 
 export type LocationAttr = `${string}:${number}:${number}`
 
@@ -32,16 +31,15 @@ export type SourceCodeData = SourceLocation & {
 
 export type TargetURLFunction = (data: SourceCodeData) => string | void
 
-const LOC_ATTR_REGEX_WIN = /^((?:\\?[^\s][^/\\:\"\?\*<>\|]+)+):([0-9]+):([0-9]+)$/
-const LOC_ATTR_REGEX_UNIX =
-    /^((?:(?:\.\/|\.\.\/|\/)?(?:\.?\w+\/)*)(?:\.?\w+\.?\w+)):([0-9]+):([0-9]+)$/
-
-export const LOC_ATTR_REGEX = isWindows ? LOC_ATTR_REGEX_WIN : LOC_ATTR_REGEX_UNIX
+const LOC_ATTR_REGEX_WIN  = /^((?:\\?[^\s][^/\\:\"\?\*<>\|]+)+):([0-9]+):([0-9]+)$/
+const LOC_ATTR_REGEX_UNIX = /^((?:(?:\.\/|\.\.\/|\/)?(?:\.?\w+\/)*)(?:\.?\w+\.?\w+)):([0-9]+):([0-9]+)$/
 
 export function getLocationAttr(element: Element): LocationAttr | null {
     let attr = element.getAttribute(LOCATION_ATTRIBUTE_NAME)
-    if (!attr || !LOC_ATTR_REGEX.test(attr)) return null
-    return attr as LocationAttr
+    if (!attr) return null
+    let is_windows = /(win32|win64|windows|wince)/i.test(navigator.userAgent)
+    let regex = is_windows ? LOC_ATTR_REGEX_WIN : LOC_ATTR_REGEX_UNIX
+    return regex.test(attr) ? attr as LocationAttr : null
 }
 
 const targetIDEMap: Record<TargetIDE, (data: SourceCodeData) => string> = {