dsh-site-bridge 0.6.4 → 0.7.0

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/manifest.json

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

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,7 +279,7 @@ 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)
@@ -291,13 +292,13 @@ async function claimTab(tabId) {
   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(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 +1531,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 +1543,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.0",
   "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",