dsh-site-bridge 0.6.4 → 0.7.2

package/README.md

@@ -197,6 +197,14 @@ instead of `<all_urls>` and does not inject a content script into every page. Th
 step and Chrome's debugging banner. In exchange the permission is granted per tab, for the duration
 of a session, and requests are read where they are made — no credential is copied out of the browser.
 
+### Consent
+
+Attaching a tab is a decision for the person by default. The agent may list tabs freely, but
+`site_tabs {action:"claim"}` on a tab the user has not attached is refused unless the call carries
+`confirmed: true` — and the refusal names the tab, so the agent knows what to ask about. Closing a tab
+the user attached needs the same confirmation. This is a rule inside the plugin, not an OS permission:
+it makes the agent stop and ask instead of quietly widening what it can reach.
+
 ## Limitations
 
 Cross-origin iframes, closed shadow roots and body size caps are analysed in

package/README.ru.md

@@ -196,6 +196,15 @@ site_inject {js:"..."}          → выполнить ваш скрипт в с
 Chrome. Взамен право выдаётся на вкладку и на время сессии, а запросы читаются там, где они сделаны:
 учётные данные не покидают браузер.
 
+### Согласие
+
+Подключение вкладки по умолчанию — решение человека. Список вкладок агент получает свободно, но
+`site_tabs {action:"claim"}` для вкладки, которую пользователь не подключал, отклоняется, пока в вызове
+нет `confirmed: true` — причём в отказе названа сама вкладка, чтобы было понятно, о чём спрашивать.
+Закрытие вкладки, подключённой пользователем, требует такого же подтверждения. Это правило внутри
+плагина, а не разрешение ОС: оно заставляет агента остановиться и спросить, вместо того чтобы тихо
+расширять свою область доступа.
+
 ## Ограничения
 
 Кросс-доменные iframe, закрытые shadow-корни и лимиты тела разобраны в

package/README.zh-CN.md

@@ -162,6 +162,13 @@ site_inject {js:"..."}          → 在页面里执行你的脚本以验证
 content script。代价是配对步骤与 Chrome 的调试提示条。换来的是：权限按标签页、按会话授予，请求在
 产生的地方被读取，凭据不离开浏览器。
 
+### 用户同意
+
+默认情况下，连接标签页是人的决定。智能体可以自由列出标签页，但对用户未连接的标签页调用
+`site_tabs {action:"claim"}` 会被拒绝，除非调用带有 `confirmed: true`——拒绝信息会指出具体标签页，便于
+智能体询问。关闭用户自己连接的标签页同样需要确认。这是插件内部的规则，而不是操作系统权限：它让智能体
+停下来询问，而不是悄悄扩大可达范围。
+
 ## 限制
 
 跨域 iframe、封闭的 shadow root 与响应体大小上限在 [docs/limitations.md](docs/limitations.md) 中逐条分析 —— 包含代码位置、消除方案与预估成本。

package/extension/_locales/en/messages.json

@@ -149,5 +149,8 @@
   },
   "disconnectTab": {
     "message": "Disconnect this tab"
+  },
+  "notClaimable": {
+    "message": "This page cannot be attached: Chrome does not allow service pages or other extensions' pages"
   }
 }

package/extension/_locales/ru/messages.json

@@ -149,5 +149,8 @@
   },
   "disconnectTab": {
     "message": "Отключить эту вкладку"
+  },
+  "notClaimable": {
+    "message": "Эту страницу нельзя подключить: Chrome не даёт доступ к служебным страницам и страницам других расширений"
   }
 }

package/extension/_locales/zh_CN/messages.json

@@ -149,5 +149,8 @@
   },
   "disconnectTab": {
     "message": "断开此标签页"
+  },
+  "notClaimable": {
+    "message": "无法连接此页面：Chrome 不允许访问服务页面或其他扩展的页面"
   }
 }

package/extension/manifest.json

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

package/extension/popup.js

@@ -138,10 +138,17 @@ function render(status) {
 
   // Nothing to claim once a tab is bound: the action would be a no-op.
   const selfAttached = selfTabId != null && targets.some((item) => item.id === selfTabId)
+  // Service pages (chrome://, other extensions, the Web Store) cannot be driven;
+  // do not offer a button that Chrome will refuse.
+  const claimable = selfTabUrl == null || /^(https?|file):/.test(selfTabUrl)
   claim.hidden = targets.length > 0
-  claim.disabled = !connected || targets.length > 0
+  claim.disabled = !connected || !claimable || targets.length > 0
+  claim.title = claimable ? '' : t('notClaimable')
   claim.textContent = t('connectTab')
-  $('claim-more').hidden = targets.length === 0 || selfAttached
+  const claimMore = $('claim-more')
+  claimMore.hidden = targets.length === 0 || selfAttached
+  claimMore.disabled = !connected || !claimable
+  claimMore.title = claimable ? t('attachAnother') : t('notClaimable')
 
   const pick = $('pick')
   pick.classList.toggle('active', status.picking === true)
@@ -170,12 +177,14 @@ function render(status) {
 // Which tab this popup is open on: attaching that tab again is a no-op, so the
 // attach button is only offered when the current tab is not attached yet.
 let selfTabId = null
+let selfTabUrl = null
 
 async function refresh() {
   try {
     if (chrome.tabs?.query) {
-      const [tab] = await chrome.tabs.query({ active: true, currentWindow: true })
+      const tab = await currentBrowserTab()
       selfTabId = tab?.id ?? null
+      selfTabUrl = tab?.url ?? null
     }
     render(await send({ type: 'status' }))
   } catch {
@@ -183,6 +192,31 @@ async function refresh() {
   }
 }
 
+/**
+ * The browser tab this popup belongs to. `currentWindow` from a popup can point
+ * at the popup's own extension page, and Chrome refuses to read other
+ * extensions' pages, so prefer a real http(s)/file tab from the focused window.
+ */
+async function currentBrowserTab() {
+  const scheme = (tab) => (/^(https?|file):/.test(String(tab?.url ?? '')) ? 'web' : 'other')
+  const scopes = [
+    { active: true, lastFocusedWindow: true, windowType: 'normal' },
+    { active: true, windowType: 'normal' },
+    { active: true, lastFocusedWindow: true },
+    { active: true, currentWindow: true }
+  ]
+  for (const scope of scopes) {
+    try {
+      const tabs = await chrome.tabs.query(scope)
+      if (!Array.isArray(tabs) || tabs.length === 0) continue
+      return tabs.find((tab) => scheme(tab) === 'web') ?? tabs[0]
+    } catch {
+      // try the next scope
+    }
+  }
+  return null
+}
+
 /* Autosave: a complete pair is stored as soon as it appears, no button. */
 let saveTimer = null
 
@@ -262,7 +296,7 @@ $('import-file').addEventListener('change', async (event) => {
 async function attachCurrentTab() {
   let tabId = null
   try {
-    const [tab] = await chrome.tabs.query({ active: true, currentWindow: true })
+    const tab = await currentBrowserTab()
     tabId = tab?.id ?? null
   } catch {
     tabId = null

package/extension/sw.js

@@ -165,7 +165,7 @@ function handleMessage(message) {
 
 async function handleMessageInner(message) {
   if (message?.type === 'claim') {
-    const target = await claimTab(message.tabId)
+    const target = await claimTab(message.tabId, 'user')
     return { ok: true, target }
   }
   if (message?.type === 'use') {
@@ -247,7 +247,7 @@ async function detach(tabId) {
   await chrome.debugger.detach({ tabId: session.tabId }).catch(() => {})
 }
 
-async function attach(tabId) {
+async function attach(tabId, attachedBy = 'agent') {
   const existing = state.sessions.get(tabId)
   if (existing) {
     state.activeTabId = tabId
@@ -265,7 +265,8 @@ async function attach(tabId) {
   const tab = await chrome.tabs.get(tabId)
   const session = {
     tabId,
-    target: { id: tab.id, url: tab.url, title: tab.title },
+    attachedBy,
+    target: { id: tab.id, url: tab.url, title: tab.title, attachedBy },
     recording: false,
     entries: [],
     picking: false,
@@ -278,26 +279,39 @@ async function attach(tabId) {
   return session.target
 }
 
-async function claimTab(tabId) {
+async function claimTab(tabId, attachedBy = 'agent') {
   // 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 })
+    const candidates = await chrome.tabs
+      .query({ active: true, lastFocusedWindow: true, windowType: 'normal' })
+      .catch(() => [])
+    const active = candidates.find((tab) => /^(https?|file):/.test(tab.url ?? '')) ?? candidates[0]
     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)
+  let tab = null
+  try {
+    tab = await chrome.tabs.get(id)
+  } catch (error) {
+    // Chrome refuses to read pages of other extensions and its own service
+    // pages; say that in words a person can act on.
+    throw new Error(`эту вкладку нельзя подключить: ${String(error?.message ?? error)}`)
+  }
   if (!/^https?:|^file:/.test(tab.url ?? '')) {
-    throw new Error(`cannot drive ${tab.url || 'this tab'}: only http(s) and file:// pages are supported`)
+    const scheme = String(tab.url ?? '').split(':')[0] || 'неизвестная схема'
+    throw new Error(
+      `эту страницу нельзя подключить (${scheme}:): Chrome не даёт доступ к служебным страницам и страницам других расширений. Откройте обычный сайт http(s) или file://`
+    )
   }
-  return attach(id)
+  return attach(id, attachedBy)
 }
 
 chrome.tabs.onUpdated.addListener((tabId, info, tab) => {
   const session = state.sessions.get(tabId)
   if (!session) return
-  const next = { id: tab.id, url: tab.url, title: tab.title }
+  const next = { id: tab.id, url: tab.url, title: tab.title, attachedBy: session.attachedBy }
   if (next.title === session.target.title && next.url === session.target.url) return
   session.target = next
   void evt('target', { tab: next, tabId })
@@ -1530,7 +1544,7 @@ async function dispatch(op, args) {
     }
     case 'tab.open': {
       const tab = await chrome.tabs.create({ url: String(args.url ?? 'about:blank'), active: args.active !== false })
-      await attach(tab.id)
+      await attach(tab.id, 'agent')
       // A fresh tab has no title yet: wait for the load so the caller gets a
       // target it can show, instead of an empty string.
       const deadline = Date.now() + Math.min(Number(args.waitMs ?? 10000), 30000)
@@ -1542,7 +1556,7 @@ async function dispatch(op, args) {
       }
       const session = state.sessions.get(tab.id)
       if (session) {
-        session.target = { id: info.id, url: info.url, title: info.title }
+        session.target = { id: info.id, url: info.url, title: info.title, attachedBy: session.attachedBy }
         await evt('target', { tab: session.target, tabId: tab.id })
       }
       return { target: session?.target ?? { id: tab.id, url: tab.url, title: tab.title } }

package/lib/bridge.js

@@ -320,7 +320,9 @@ export class Bridge {
       const tabId = Number(event.tabId ?? tab?.id)
       if (single) {
         const id = Number(tab.id)
-        this.targets.set(id, { id, url: String(tab.url ?? ''), title: String(tab.title ?? ''), at: Date.now() })
+        // Keep whatever else the extension says about the tab (attachedBy, for
+        // example): the consent rule depends on it.
+        this.targets.set(id, { ...tab, id, url: String(tab.url ?? ''), title: String(tab.title ?? ''), at: Date.now() })
         if (this.activeTabId == null) this.activeTabId = id
       } else if (Number.isFinite(tabId)) {
         this.targets.delete(tabId)

package/lib/index.js

@@ -259,15 +259,25 @@ export function apply(ctx, config) {
       'site_tabs',
       'Вкладки браузера и выбор рабочей вкладки. Можно держать подключёнными несколько вкладок: ' +
       'claim добавляет ещё одну, use переключает активную, release отключает, open открывает новую и сразу ' +
-      'подключает её, focus выводит вкладку на передний план. Вкладка должна быть http(s) или file://.',
+      'подключает её, focus выводит вкладку на передний план. Вкладка должна быть http(s) или file://.\n' +
+      'ПРАВИЛО: подключать вкладку, которую пользователь не подключал сам, можно только с его явного ' +
+      'разрешения. Без confirmed:true вызов claim отклоняется — сначала спросите пользователя.',
       {
         action: { type: 'string', description: 'list | claim | use | release | open | close | focus' },
-        tabId: { type: 'number', description: 'id вкладки для claim / use / release / focus (из action=list)' },
-        url: { type: 'string', description: 'адрес для action=open (по умолчанию about:blank)' }
+        tabId: { type: 'number', description: 'id вкладки для claim / use / release / focus / close (из action=list)' },
+        url: { type: 'string', description: 'адрес для action=open (по умолчанию about:blank)' },
+        confirmed: {
+          type: 'boolean',
+          description:
+            'Только после явного разрешения пользователя: подтверждает подключение вкладки, ' +
+            'которую он не подключал, или закрытие вкладки, которую он подключил сам'
+        }
       },
       async (args) => {
         const instance = await ensureBridge()
         const action = String(args.action ?? 'list')
+        const confirmed = args.confirmed === true
+        const attached = (id) => (instance.state().targets ?? []).find((item) => item.id === id) ?? null
         if (action === 'open') {
           const result = await instance.enqueue(
             'tab.open',
@@ -280,6 +290,13 @@ export function apply(ctx, config) {
         if (action === 'close') {
           const tabId = Number(args.tabId ?? instance.state().activeTabId)
           if (!Number.isFinite(tabId)) throw new Error('нечего закрывать: передайте tabId')
+          const known = attached(tabId)
+          if (known?.attachedBy === 'user' && !confirmed) {
+            throw new Error(
+              `Вкладку [${tabId}] ${known.title || known.url || ''} подключили вы, а не агент. ` +
+              'Закрытие такой вкладки требует явного подтверждения: спросите пользователя и повторите с confirmed:true.'
+            )
+          }
           await instance.enqueue('tab.close', { tabId }, { timeout: 20000 })
           return ok(`Вкладка ${tabId} закрыта.`, { tabId })
         }
@@ -292,6 +309,16 @@ export function apply(ctx, config) {
         if (action === 'claim' || action === 'use' || action === 'release') {
           const tabId = Number(args.tabId)
           if (!Number.isFinite(tabId)) throw new Error(`tabId is required for action=${action}`)
+          if (action === 'claim' && !attached(tabId) && !confirmed) {
+            const list = await instance.enqueue('tabs.list', {}, { timeout: 20000 })
+            const known = (list?.tabs ?? []).find((item) => item.id === tabId)
+            const label = known ? `${known.title || '(без названия)'} — ${known.url}` : `вкладка ${tabId}`
+            throw new Error(
+              `Вкладка не подключена: ${label}. Пользователь её не подключал, а правило плагина требует его ` +
+              'явного разрешения на доступ. Спросите пользователя (назовите эту вкладку) и только после ' +
+              'согласия повторите вызов с confirmed:true.'
+            )
+          }
           const op = action === 'claim' ? 'tab.claim' : action === 'use' ? 'tab.use' : 'tab.release'
           const result = await instance.enqueue(op, { tabId }, { timeout: 20000 })
           if (action === 'release') return ok(`Вкладка ${tabId} отключена.`, { tabId })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-site-bridge",
-  "version": "0.6.4",
+  "version": "0.7.2",
   "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",