dsh-plugin-subagent-bridge 0.14.1 → 0.15.0

package/README.md

@@ -56,6 +56,24 @@ dsh_task({ prompt: "…", cwd: "/repo/clawer" })
 - 返回结果里带上 `workspace: { id, title, path }`，方便确认落在哪个项目；
 - 项目操作失败不影响派发本身（分组是外观，不因此让任务失败）。
 
+## 没人值守：让它自己拿主意（`assumeDefaults`）
+
+worker 卡在 `ask_user_question` 上时，需要有人在界面上点一下；但调用方是另一个 agent 时，那个人往往不在。
+
+所以桥在派发时会**自动追加一条常驻指令**：
+
+```
+（如果缺少必要信息或需要在多个方案里选择，直接采用最合理的默认做法继续做完，
+  不要把选择权抛回给用户、不要调用 ask_user_question 等回答；
+  在结果里用一行说明你假设了什么。）
+```
+
+- 默认开启，配置项 `assumeDefaults: false` 可关掉；
+- `dsh_task`、`dsh_spawn`、`dsh_followup` 三条路径都会加；
+- prompt 里本来就提到 `ask_user_question` 时不追加，避免和调用方的明确意图打架。
+
+效果：worker 不会停下来等人，而是选一个合理默认做法做完，并在结果里交代假设，调用方据此复核。
+
 ## 回答 worker 的问题（`dsh_answer`）
 
 worker 需要用户拍板时会调用 `ask_user_question`，然后**卡在那里等**。如果人不在，任务就停住了。

package/index.js

@@ -75,6 +75,7 @@ const DEFAULTS = {
   codexBin: 'codex',
   codexSandbox: 'workspace-write',
   codexTimeoutMs: 300000,
+  assumeDefaults: true,
   claudeBin: 'claude',
   claudeModel: '',
   claudeTimeoutMs: 300000,
@@ -309,6 +310,28 @@ const TOOLS = [
   },
 ]
 
+/**
+ * Standing instruction appended to a delegated task.
+ *
+ * A worker that stops to ask a question needs somebody at the UI; when the caller is
+ * another agent, nobody is. The caller would rather have a reasonable assumption and a
+ * finished result, so the worker is told to make one and report it.
+ */
+const ASSUMPTION_POLICY =
+  '如果缺少必要信息或需要在多个方案里选择，直接采用最合理的默认做法继续做完，不要把选择权抛回给用户、不要调用 ask_user_question 等回答；在结果里用一行说明你假设了什么。'
+
+/**
+ * Attach the standing instruction when the deployment wants it.
+ * @param settings - resolved settings.
+ * @param message - caller-supplied text.
+ * @returns the text to deliver.
+ */
+function withAssumptionPolicy(settings, message) {
+  if (settings.assumeDefaults !== true) return message
+  if (message.includes('ask_user_question')) return message
+  return `${message}\n\n（${ASSUMPTION_POLICY}）`
+}
+
 /**
  * Take over the questions a worker asks, so the calling agent can answer them.
  *
@@ -965,7 +988,7 @@ export function createApi(ctx, state, settings) {
       const before = Number(sessions()?.get?.(sessionId)?.seq ?? 0)
       const requestId = randomUUID()
       const accepted = await service.prompt(
-        { sessionId, requestId, mode: mode === 'steer' ? 'steer' : 'queue', content: [{ type: 'text', text: message }] },
+        { sessionId, requestId, mode: mode === 'steer' ? 'steer' : 'queue', content: [{ type: 'text', text: withAssumptionPolicy(settings, message) }] },
         AbortSignal.timeout(30000),
       )
       state.dispatched.add(sessionId)
@@ -1000,7 +1023,7 @@ export function createApi(ctx, state, settings) {
       const before = Number(sessions()?.get?.(target.sessionId)?.seq ?? 0)
       const requestId = randomUUID()
       const accepted = await service.prompt(
-        { sessionId: target.sessionId, requestId, mode: mode === 'steer' ? 'steer' : 'queue', content: [{ type: 'text', text: message }] },
+        { sessionId: target.sessionId, requestId, mode: mode === 'steer' ? 'steer' : 'queue', content: [{ type: 'text', text: withAssumptionPolicy(settings, message) }] },
         AbortSignal.timeout(30000),
       )
       state.baseline.set(target.sessionId, before)
@@ -1079,7 +1102,7 @@ export function createApi(ctx, state, settings) {
       const finished = waitForTurnEnd(ctx, targetId, waitAfter, seconds * 1000)
       const requestId = randomUUID()
       const accepted = await service.prompt(
-        { sessionId: targetId, requestId, mode: deliveryMode, content: [{ type: 'text', text: prompt }] },
+        { sessionId: targetId, requestId, mode: deliveryMode, content: [{ type: 'text', text: withAssumptionPolicy(settings, prompt) }] },
         AbortSignal.timeout(30000),
       )
       state.dispatched.add(targetId)
@@ -1141,6 +1164,7 @@ export function createApi(ctx, state, settings) {
         let settled = false
         for (const target of targets) {
           const resolved = resolveWorker(state, target)
+          if (resolved !== undefined) attachQuestionAnswerer(state, ctx.get('agents')?.get?.(resolved.sessionId))
           const sessionId = resolved === undefined ? target : resolved.sessionId
           const value = resolved === undefined ? 'not_found' : workerStatus(ctx, sessionId, state, settings)
           if (resolved !== undefined) await rememberOutcome(settings, state, sessionId, value)
@@ -2308,6 +2332,10 @@ function resolveSettings(config) {
     }
     if (key === 'codexBin' && typeof value === 'string' && value !== '') merged.codexBin = value
     if (key === 'codexSandbox' && typeof value === 'string' && value !== '') merged.codexSandbox = value
+    if (key === 'assumeDefaults') {
+      merged.assumeDefaults = value === true || value === 'true'
+      continue
+    }
     if (key === 'claudeBin' && typeof value === 'string' && value !== '') merged.claudeBin = value
     if (key === 'claudeModel' && typeof value === 'string') merged.claudeModel = value
     if (key === 'workersFile' && typeof value === 'string' && value !== '') merged.workersFile = expandHome(value)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-subagent-bridge",
-  "version": "0.14.1",
+  "version": "0.15.0",
   "type": "module",
   "description": "Subagent bridge: lets Codex delegate a task to DeepSeek Harness the way it would spawn a subagent (one call, waits for the answer), plus a codex_dispatch tool for the reverse direction.",
   "keywords": [