refactor: stabilize token service and release checks

Author: antiantidev

.github/workflows/release.yml

@@ -31,6 +31,9 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
+      - name: Validate tag matches package version
+        run: node -e "const fs=require('fs'); const pkg=JSON.parse(fs.readFileSync('package.json','utf8')); const tag=process.env.GITHUB_REF_NAME; const expected='v'+pkg.version; if(tag!==expected){console.error('Tag/version mismatch. tag='+tag+', package.json='+pkg.version); process.exit(1)}"
+
       - name: Build and Publish
         run: pnpm run release
         env:

services/tokenService.js

@@ -5,22 +5,30 @@ class TokenService {
   constructor(options = {}) {
     this.fsImpl = options.fsImpl || fs;
     this.env = options.env || process.env;
+    this.platform = options.platform || process.platform;
   }
 
   loadLocalToken() {
+    if (this.platform !== "win32") {
+      return { token: null, error: "Unsupported platform. Local token extraction is only available on Windows." };
+    }
+
     const appdata = this.env.APPDATA;
     const localappdata = this.env.LOCALAPPDATA;
-    
-    if (!appdata || !localappdata) {
-      return { token: null, error: "Windows Environment Variables (APPDATA/LOCALAPPDATA) not found." };
+
+    const dirs = [];
+    if (appdata) {
+      dirs.push(path.join(appdata, "slobs-client", "Local Storage", "leveldb"));
+    }
+    if (localappdata) {
+      dirs.push(path.join(localappdata, "Google", "Chrome", "User Data", "Default", "Local Storage", "leveldb"));
+      dirs.push(path.join(localappdata, "BraveSoftware", "Brave-Browser", "User Data", "Default", "Local Storage", "leveldb"));
+      dirs.push(path.join(localappdata, "Microsoft", "Edge", "User Data", "Default", "Local Storage", "leveldb"));
     }
 
-    const dirs = [
-      path.join(appdata, "slobs-client", "Local Storage", "leveldb"),
-      path.join(localappdata, "Google", "Chrome", "User Data", "Default", "Local Storage", "leveldb"),
-      path.join(localappdata, "BraveSoftware", "Brave-Browser", "User Data", "Default", "Local Storage", "leveldb"),
-      path.join(localappdata, "Microsoft", "Edge", "User Data", "Default", "Local Storage", "leveldb")
-    ];
+    if (dirs.length === 0) {
+      return { token: null, error: "Windows Environment Variables (APPDATA/LOCALAPPDATA) not found." };
+    }
 
     const tokenRegex = /"apiToken":"([a-f0-9]+)"/gi;
 
@@ -36,6 +44,7 @@ class TokenService {
       for (const file of files) {
         try {
           const raw = this.fsImpl.readFileSync(file, "utf8").replace(/\x00/g, "");
+          tokenRegex.lastIndex = 0;
           let match;
           let last = null;
           while ((match = tokenRegex.exec(raw)) !== null) {
@@ -57,4 +66,4 @@ class TokenService {
   }
 }
 
-module.exports = { TokenService };
+module.exports = { TokenService };

src/App.jsx

@@ -14,6 +14,7 @@ import LiveSetup from './pages/LiveSetup'
 import TokenVault from './pages/TokenVault'
 import Pulse from './pages/Pulse'
 import Settings from './pages/Settings'
+import { useDriverReadiness } from './hooks/useDriverReadiness'
 
 const CHROME_DOWNLOAD_URL = 'https://www.google.com/chrome/'
 
@@ -123,26 +124,15 @@ const App = () => {
     }
   }, [showModal, t])
 
-  const bootstrapDriver = async () => {
-    setIsLoading(true)
-    setLoadingMessage(t('driver.preparing'))
-    const res = await window.api.bootstrapDriver()
-    setIsLoading(false)
-
-    if (res.ok) {
-      setIsDriverMissing(false)
-      return true
-    }
-
-    if (res.code === 'CHROME_NOT_FOUND') {
-      await showChromeMissingModal()
-      setIsDriverMissing(true)
-      return false
-    }
-
-    pushToast(res.error || t('common.error'), 'error')
-    return false
-  }
+  const ensureDriverReady = useDriverReadiness({
+    t,
+    showModal,
+    showChromeMissingModal,
+    pushToast,
+    setIsLoading,
+    setLoadingMessage,
+    setIsDriverMissing
+  })
 
   const closeModal = useCallback((value) => {
     setModal(prev => { if (prev.resolve) prev.resolve({ value }); return { ...prev, show: false, resolve: null } })
@@ -216,26 +206,9 @@ const App = () => {
 
   const loadWebToken = async (existingAccountId = null) => {
     pushStatus('Web token capture started', 'info')
-    const driverExists = await window.api.checkDriverExists()
-
-    if (!driverExists) {
-      const choice = await showModal(
-        t('driver.missing_title'),
-        t('driver.missing_desc'),
-        [
-          { label: t('common.cancel'), value: 'cancel', primary: false },
-          { label: t('driver.download_now'), value: 'download', primary: true }
-        ]
-      )
-      if (choice.value === 'download') {
-        const bootstrapped = await bootstrapDriver()
-        if (!bootstrapped) {
-          return
-        }
-      } else {
-        setIsDriverMissing(true)
-        return
-      }
+    const driverStatus = await ensureDriverReady()
+    if (!driverStatus.ok) {
+      return
     }
 
     setIsWebLoading(true)
@@ -334,31 +307,11 @@ const App = () => {
         i18n.changeLanguage('en')
       }
       setLoadProgress(60); setLoadingMessage('Verifying system dependencies...')
-      const driverExists = await window.api.checkDriverExists()
-      if (!driverExists) {
-        setIsDriverMissing(true)
-        const choice = await showModal(
-          t('driver.missing_title'),
-          t('driver.missing_desc'),
-          [
-            { label: t('common.cancel'), value: 'cancel', primary: false },
-            { label: t('driver.download_now'), value: 'download', primary: true }
-          ]
-        )
-
-        if (choice.value === 'download') {
-          const bootstrapped = await bootstrapDriver()
-          if (!bootstrapped) {
-            return
-          }
-          setLoadProgress(85)
-        } else {
-          return
-        }
-      } else {
-        setIsDriverMissing(false)
-        setLoadProgress(85)
+      const driverStatus = await ensureDriverReady()
+      if (!driverStatus.ok) {
+        return
       }
+      setLoadProgress(85)
       if (data?.settings?.autoRefresh && (token || data?.token)) {
         setLoadingMessage('Synchronizing account status...')
         await refreshAccountInfo(token || data.token, activeAccountId || data.activeAccountId); 

src/hooks/useDriverReadiness.js

@@ -0,0 +1,60 @@
+import { useCallback } from 'react'
+
+export const useDriverReadiness = ({
+  t,
+  showModal,
+  showChromeMissingModal,
+  pushToast,
+  setIsLoading,
+  setLoadingMessage,
+  setIsDriverMissing
+}) => {
+  return useCallback(async () => {
+    const driverExists = await window.api.checkDriverExists()
+    if (driverExists) {
+      setIsDriverMissing(false)
+      return { ok: true, alreadyExists: true }
+    }
+
+    const choice = await showModal(
+      t('driver.missing_title'),
+      t('driver.missing_desc'),
+      [
+        { label: t('common.cancel'), value: 'cancel', primary: false },
+        { label: t('driver.download_now'), value: 'download', primary: true }
+      ]
+    )
+
+    if (choice.value !== 'download') {
+      setIsDriverMissing(true)
+      return { ok: false, canceled: true }
+    }
+
+    setIsLoading(true)
+    setLoadingMessage(t('driver.preparing'))
+    const res = await window.api.bootstrapDriver()
+    setIsLoading(false)
+
+    if (res.ok) {
+      setIsDriverMissing(false)
+      return { ok: true, alreadyExists: false }
+    }
+
+    if (res.code === 'CHROME_NOT_FOUND') {
+      await showChromeMissingModal()
+      setIsDriverMissing(true)
+      return { ok: false, chromeMissing: true }
+    }
+
+    pushToast(res.error || t('common.error'), 'error')
+    return { ok: false, failed: true }
+  }, [
+    pushToast,
+    setIsDriverMissing,
+    setIsLoading,
+    setLoadingMessage,
+    showChromeMissingModal,
+    showModal,
+    t
+  ])
+}