dsh-site-bridge 0.5.2 → 0.5.3

package/extension/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "default_locale": "en",
   "name": "__MSG_appName__",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "__MSG_appDesc__",
   "minimum_chrome_version": "116",
   "permissions": [

package/extension/popup.js

@@ -239,16 +239,33 @@ $('import-file').addEventListener('change', async (event) => {
   }
 })
 
-$('claim').addEventListener('click', async () => {
-  const [tab] = await chrome.tabs.query({ active: true, currentWindow: true })
-  if (!tab?.id) return
-  const result = await send({ type: 'claim', tabId: tab.id })
-  if (!result?.ok) {
-    setNote(t('claimFailed', [String(result?.error ?? t('claimFailedUnknown'))]), true)
+async function attachCurrentTab() {
+  let tabId = null
+  try {
+    const [tab] = await chrome.tabs.query({ active: true, currentWindow: true })
+    tabId = tab?.id ?? null
+  } catch {
+    tabId = null
+  }
+  if (tabId == null) {
+    setNote(t('claimFailed', [t('claimFailedUnknown')]), true)
+    return
+  }
+  try {
+    const result = await send({ type: 'claim', tabId })
+    if (!result?.ok) {
+      setNote(t('claimFailed', [String(result?.error ?? t('claimFailedUnknown'))]), true)
+      return
+    }
+    setNote('')
+  } catch (error) {
+    setNote(t('claimFailed', [String(error?.message ?? error)]), true)
     return
   }
   await refresh()
-})
+}
+
+$('claim').addEventListener('click', attachCurrentTab)
 
 $('capture').addEventListener('click', async () => {
   const status = await send({ type: 'status' })
@@ -277,11 +294,7 @@ $('pick').addEventListener('click', async () => {
   await refresh()
 })
 
-$('claim-more').addEventListener('click', async () => {
-  const result = await send({ type: 'claim' })
-  if (!result?.ok && result?.error) setNote(String(result.error), true)
-  await refresh()
-})
+$('claim-more').addEventListener('click', attachCurrentTab)
 
 $('tabs-select').addEventListener('change', async (event) => {
   const tabId = Number(event.target.value)

package/extension/sw.js

@@ -272,11 +272,19 @@ async function attach(tabId) {
 }
 
 async function claimTab(tabId) {
-  const tab = await chrome.tabs.get(tabId)
+  // Without an explicit id, claim whatever tab the user is looking at, so a
+  // caller that only knows "this tab" cannot silently pass NaN.
+  let id = Number(tabId)
+  if (!Number.isFinite(id)) {
+    const [active] = await chrome.tabs.query({ active: true, lastFocusedWindow: true })
+    id = active?.id
+  }
+  if (!Number.isFinite(id)) throw new Error('no tab to claim: pass tabId or open a page first')
+  const tab = await chrome.tabs.get(id)
   if (!/^https?:|^file:/.test(tab.url ?? '')) {
     throw new Error(`cannot drive ${tab.url || 'this tab'}: only http(s) and file:// pages are supported`)
   }
-  return attach(tabId)
+  return attach(id)
 }
 
 chrome.debugger.onDetach.addListener((source, reason) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-site-bridge",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "Chrome/Edge MV3 bridge for DeepSeek Harness: capture page markup (DOM, styles, selectors) and BFF traffic (xhr/fetch with headers and bodies) into a workspace snapshot bundle, drive the page with trusted CDP input, and verify injected scripts. Authorization never leaves the browser and is never written to disk.",
   "icon": "./icon.svg",
   "type": "module",