@cpm-lhj/dsh-feishu 0.1.0 → 0.1.1

package/README.md

@@ -1,8 +1,38 @@
 # @cpm-lhj/dsh-feishu
 
-飞书长连接插件。白名单里的人发消息后，常驻的 dsh 在固定目录里执行；进度和批准显示在触发消息下的卡片上，最终回答回复在那条消息下面。
+飞书长连接插件。白名单里的人发消息后，常驻的 dsh 在固定目录里执行。每一轮在当次触发消息下发一张卡片，过程中原地更新；这一轮结束后把结果写进这张卡片，并 @ 触发的人说「完成了，请看上方结果」。
 
-适配 `@larksuiteoapi/node-sdk` 1.76。对照当前安装包，`im.message.receive_v1`、`card.action.trigger`、`client.im.message.reply`（含 `reply_in_thread`）、`client.im.message.patch` 和 `GET /open-apis/bot/v3/info` 的字段一致。话题 id 没有用一条真实群消息核对。键规则按设计实现：私聊是 `p2p:<chat_id>`；群里有 `thread_id` 时是 `group:<chat_id>:thread:<thread_id>`；否则是 `group:<chat_id>:root:<root_id 或 message_id>`。线上字段若不同，只改 `src/transport.ts` 和 `src/keys.ts`。
+## 配置
+
+App ID、App Secret、工作目录和可以发起会话的人，都在 Web 的插件页里填。入口在插件包的组件行里。
+
+用装了并且启用了本插件的那个 profile 打开 Web：
+
+1. 左侧边栏点**插件**。
+2. 在已安装列表里打开 **@cpm-lhj/dsh-feishu**。
+3. 在**包含的组件**里点 **feishu**。点这一行的标题，标题右侧有箭头。旁边的开关只负责启用。
+
+| 字段 | 填写 |
+| --- | --- |
+| 工作目录 | 已经存在的绝对路径。所有会话都在这个目录里执行。 |
+| 可以发起会话的人 | 每行一个飞书 open_id。留空则没有人可以发起。 |
+| App ID | 飞书应用的 App ID。 |
+| App Secret | 飞书应用的 App Secret。 |
+
+点**保存**。App ID 和 App Secret 写入凭证库，配置里只留下 `FEISHU_APP_ID` 和 `FEISHU_APP_SECRET` 这两个名字。凭证框留空表示保留已经保存的值。保存后插件按新配置重新连接。
+
+页面写着「插件还没在运行，这个页面暂时不能保存」时，先把 `feishu` 这一行的开关打开，等它变成运行中再保存。
+
+只有这个 profile 的 Web 里有这张表。换一个没装本插件的 Web，或者没打开 Web 的常驻进程，都到不了这里。
+
+关掉 `tool-working-directory` 要写在该 profile 自己的 `cordis.patch.yml`。后一层按整行替换，不是深合并：
+
+```yaml
+- id: tool-working-directory
+  disabled: true
+```
+
+不改 `sandbox-policy` 的 `workspaceRoot`。
 
 ## 安装
 
@@ -22,20 +52,11 @@ dsh plugin --profile <name> add <本包>
       name: '@cpm-lhj/dsh-feishu'
 ```
 
-装好之后，用带 Web 界面的同一个 profile 打开**插件**，进入 `@cpm-lhj/dsh-feishu`，点 `feishu` 这一行的**配置**。页面上填写工作目录、每行一个 open_id、App ID 和 App Secret，然后保存。App ID 和 App Secret 写入凭证库，配置里只留下 `FEISHU_APP_ID` 和 `FEISHU_APP_SECRET` 这两个名字。留空的凭证不会覆盖已经保存的值。目录必须是已存在的绝对路径；open_id 留空时没有人可以触发。保存后插件按新配置重新连接。
-
-这个配置页只出现在加载了 Web 客户端的 profile 里。没打开 Web 的常驻进程上看不到它。
-
-关掉 `tool-working-directory` 仍写在该 profile 自己的 `cordis.patch.yml`。后一层按整行替换，不是深合并：
-
-```yaml
-- id: tool-working-directory
-  disabled: true
-```
+装好之后按上一节打开配置页。
 
-不改 `sandbox-policy` 的 `workspaceRoot`。
+常驻 profile 只叠 base 和本插件。不要用 headless 当常驻进程，也不要在同一个 profile 上挂 Web 批准界面。同一套飞书应用和 domain 只跑一个进程；两个进程会各自处理消息。要改配置，用装了本插件的那个带 Web 的 profile 打开插件页。
 
-常驻 profile 只叠 base 和本插件。不要用 headless 当常驻进程，也不要在同一个 profile 上挂 Web 批准界面。同一套飞书应用和 domain 只跑一个进程；两个进程会各自处理消息。
+飞书会话使用该 profile 的默认 agent preset。Web 的默认是 `standard`，因此有 shell 和读文件。已经记下 preset 的会话在恢复时沿用那一个。没有 preset 注册表的 profile 继续用进程上已有的工具。
 
 `@deepseek-ai/cordis`、`@deepseek-ai/schemastery` 和 `@deepseek-ai/dsh-llm` 是 peer，由正在运行的 dsh 提供。不要再装一份。本包直接依赖 `@larksuiteoapi/node-sdk` 和 `zod`。
 
@@ -56,6 +77,10 @@ dsh plugin --profile <name> add <本包>
 
 本包声明 `engines.dsh` 上限 `<0.3.0`。把 dsh 升到 0.3.x 之后，已安装的 profile 在启动时会静默跳过这个 bundle：结果记在 `skippedBundles`，终端不打印。机器人不再响应，仍处于执行中的卡片也不会被恢复。升级前先确认本包的兼容范围。安装和更新路径会直接拒绝范围外的 dsh；已经装好的机器才是静默跳过。
 
+## 协议字段
+
+适配 `@larksuiteoapi/node-sdk` 1.76。对照当前安装包，`im.message.receive_v1`、`card.action.trigger`、`client.im.message.reply`（含 `reply_in_thread`）、`client.im.message.patch` 和 `GET /open-apis/bot/v3/info` 的字段一致。话题 id 没有用一条真实群消息核对。键规则按设计实现：私聊是 `p2p:<chat_id>`；群里有 `thread_id` 时是 `group:<chat_id>:thread:<thread_id>`；否则是 `group:<chat_id>:root:<root_id 或 message_id>`。线上字段若不同，只改 `src/transport.ts` 和 `src/keys.ts`。
+
 ## 固定文案
 
 这些句子不做成配置：
@@ -65,6 +90,7 @@ dsh plugin --profile <name> add <本包>
 | 不在名单 | 你不在可触发名单里，这条消息不会执行。 |
 | 这条线还在执行 | 这条会话还在执行，新消息不会执行。 |
 | 没有正文 | 请写下要处理的问题。 |
+| 这一轮结束 | @触发的人，然后「完成了，请看上方结果」。 |
 | 卡片 | 执行中、已停止、同意、拒绝、停止、成功、失败 |
 
 ## 开发

package/lib/card.js

@@ -3,13 +3,16 @@ import { COPY } from './copy.js'
 
 /**
  * Render one progress card. Button values carry the action and id only.
- * Settled cards keep the rows and omit the running and stopped status.
+ * Settled cards keep the rows and the completion text, and omit the running and stopped status.
  * @param doc - Card content derived from the line and the open approval, if any.
  * @returns Feishu interactive card JSON.
  */
 export function renderCard(doc              )           {
   const elements                            = []
   for (const row of doc.rows) elements.push(textElement(rowText(row)))
+  if (doc.status === 'settled' && doc.answer !== undefined && doc.answer !== '') {
+    elements.push(textElement(doc.answer))
+  }
   if (doc.approval !== undefined) {
     if (doc.approval.reason !== '') elements.push(textElement(doc.approval.reason))
     elements.push(actions([

package/lib/copy.js

@@ -11,4 +11,5 @@ export const COPY = {
   denied: '你不在可触发名单里，这条消息不会执行。',
   busy: '这条会话还在执行，新消息不会执行。',
   empty: '请写下要处理的问题。',
+  done: '完成了，请看上方结果',
 }         

package/lib/plugin.js

@@ -2,7 +2,7 @@
 import { createUserMessage } from '@deepseek-ai/dsh-llm'
 import { feishuDomainSpec } from './domain.js'
 import { createRuntime } from './runtime.js'
-                                                                      
+                                                                                         
 import { LineStore,                  } from './store.js'
 import { connectionReady,                      } from './startup.js'
 import { createLarkTransport } from './transport.js'
@@ -143,6 +143,7 @@ async function startConnected(ctx         , ready                 )
       transport,
       agents: ctx.agents,
       modelSelection: () => ctx.agentDefaultModel.currentSelection(),
+      presets: () => presetSource(ctx.get('agentPresets', false)),
       createUserMessage,
       log: ctx.logger('feishu'),
     })
@@ -162,4 +163,21 @@ async function startConnected(ctx         , ready                 )
   }
 }
 
+/**
+ * Keep a preset registry when this profile has one. A profile without the
+ * service leaves tools on the host, and this returns undefined.
+ * @param value - Whatever `ctx.get` returned for `agentPresets`.
+ * @returns The registry, or undefined when the value cannot mount a preset.
+ */
+function presetSource(value         )                                {
+  return isPresetSource(value) ? value : undefined
+}
+
+function isPresetSource(value         )                             {
+  return typeof value === 'object' && value !== null
+    && 'resolve' in value && typeof value.resolve === 'function'
+    && 'mount' in value && typeof value.mount === 'function'
+    && 'presetForSession' in value && typeof value.presetForSession === 'function'
+}
+
                             

package/lib/runtime-types.js

@@ -34,6 +34,29 @@
                                                                                                                            
                                     
 
+/** Session identity the preset registry reads while an agent is being composed. */
+                                
+            
+ 
+
+/** Agent handed to setup. Only the session identity is read. */
+                              
+                        
+ 
+
+/**
+ * Optional preset registry. Absent on a profile that keeps tools on the host.
+ * @param id - Explicit preset, or the registry default when omitted.
+ */
+                                    
+                                               
+                                                      
+                                                         
+ 
+
+/** Compose one agent: mount its preset, then bind Feishu listeners. */
+                                                                                          
+
 /** Scoped context of one agent. Listeners see only that agent. */
                              
      
@@ -55,16 +78,16 @@
 /** Options for a new session whose cwd is the configured directory. */
                               
                    
-                       
+                                             
                           
-                                       
+                  
  
 
 /** Options for resuming a persisted session. */
                               
                          
                           
-                                       
+                  
  
 
 /** Model route copied from the host default selection. */

package/lib/runtime.js

@@ -6,12 +6,14 @@ import { objectLabelFromArguments } from './object-label.js'
                                            
              
                
+                    
              
                   
                   
             
                  
                 
+              
                
                            
                                                                                          
@@ -24,6 +26,8 @@ import { objectLabelFromArguments } from './object-label.js'
                             
                       
                                       
+                                                                           
+                                               
                                                                                                             
                                        
  
@@ -52,6 +56,8 @@ export function createRuntime(deps             )
   const assistantText = new Map                ()
   const approvals = new ApprovalBook()
   const patches = new Map                       ()
+  const senders = new Map                ()
+  const answers = new Map                ()
   let closed = false
   let timer                                            
 
@@ -81,6 +87,7 @@ export function createRuntime(deps             )
         return
       }
       if (admitted.decision.kind !== 'start' || admitted.line === undefined) return
+      senders.set(admitted.line.sessionId, event.senderOpenId)
       await startTurn(admitted.line, event.text)
     } catch (error) {
       deps.log.error(`feishu message ${event.eventId} failed: ${errorText(error)}`)
@@ -94,7 +101,7 @@ export function createRuntime(deps             )
       if (!claimed) return
       if (!deps.allowOpenIds.includes(event.operatorOpenId)) return
       const line = await deps.store.findByAlias(event.messageId)
-      if (line === undefined) return
+      if (line === undefined || line.cardMessageId !== event.messageId) return
       if (event.kind === 'stop') {
         await stop(line)
         return
@@ -223,7 +230,7 @@ export function createRuntime(deps             )
     const existing = agents.get(line.sessionId)
     if (existing !== undefined) return existing
     const route = routeOf(deps.modelSelection())
-    const setup = (agentCtx            ) => bind(agentCtx, line.sessionId)
+    const setup = (agentCtx            , agent             ) => attach(agentCtx, agent, line.sessionId)
     try {
       const handle = await deps.agents.resume({
         resumeSessionId: line.sessionId,
@@ -234,17 +241,39 @@ export function createRuntime(deps             )
       return handle.agent
     } catch (error) {
       if (!isMissingSession(error, line.sessionId)) throw error
+      const agentPreset = await defaultPreset()
       const handle = await deps.agents.create({
         sessionId: line.sessionId,
-        meta: { cwd: deps.directory },
+        meta: { cwd: deps.directory, ...agentPreset === undefined ? {} : { agentPreset } },
         agentOptions: route,
-        setup,
+        setup: (agentCtx, agent) => attach(agentCtx, agent, line.sessionId, agentPreset),
       })
       agents.set(line.sessionId, handle.agent)
       return handle.agent
     }
   }
 
+  async function defaultPreset()                              {
+    const presets = deps.presets?.()
+    if (presets === undefined) return undefined
+    return (await presets.resolve()).id
+  }
+
+  /**
+   * Mount the preset for this agent, then bind Feishu listeners.
+   * A newly created session uses the id written into its header. A resumed
+   * session uses the id its log recorded, or the current default when it has none.
+   */
+  async function attach(agentCtx            , agent             , sessionId        , fresh         )                {
+    const presets = deps.presets?.()
+    if (presets !== undefined) {
+      const recorded = presets.presetForSession(agent.session)
+      const id = fresh ?? (recorded === null || recorded === '' ? undefined : recorded) ?? (await presets.resolve()).id
+      await presets.mount(agentCtx, id)
+    }
+    bind(agentCtx, sessionId)
+  }
+
   function bind(agentCtx            , sessionId        )       {
     agentCtx.on('session/event', (session, event) => {
       if (session.id !== sessionId) return
@@ -349,17 +378,20 @@ export function createRuntime(deps             )
       const stopped = line.cancelRequested || ended.kind === 'aborted'
       const trigger = line.triggerMessageId
       const text = assistantText.get(sessionId) ?? ''
+      const sender = senders.get(sessionId) ?? ''
       assistantText.delete(sessionId)
       calls.delete(sessionId)
       approvals.cancelSession(sessionId)
+      if (stopped || text === '') answers.delete(line.canonicalKey)
+      else answers.set(line.canonicalKey, text)
       line.cardStatus = stopped ? 'stopped' : 'settled'
       line.phase = 'idle'
       line.turn = null
       line.triggerMessageId = null
       line.cancelRequested = false
       await deps.store.save(line)
-      const reply = !stopped && trigger !== null && text !== ''
-        ? { messageId: trigger, text, inThread: line.canonicalKey.startsWith('group:') }
+      const reply = !stopped && trigger !== null
+        ? { messageId: trigger, text: doneNotice(sender), inThread: line.canonicalKey.startsWith('group:') }
         : undefined
       return { canonicalKey: line.canonicalKey, show: true, reply }
     }
@@ -382,11 +414,13 @@ export function createRuntime(deps             )
   async function paintNow(seed            , create         )                {
     const line = await deps.store.findByAlias(seed.canonicalKey) ?? seed
     const view = line.phase === 'running' ? approvals.view(line.sessionId) : undefined
+    const answer = line.cardStatus === 'settled' ? answers.get(line.canonicalKey) : undefined
     const card = renderCard({
       status: line.cardStatus,
       rows: line.rows,
       showStop: line.phase === 'running' && line.cardStatus === 'running',
       ...view === undefined ? {} : { approval: view },
+      ...answer === undefined || answer === '' ? {} : { answer },
     })
     if (line.cardMessageId === null) {
       if (!create || line.triggerMessageId === null) return
@@ -429,6 +463,11 @@ export function createRuntime(deps             )
   }
 }
 
+function doneNotice(openId        )         {
+  if (!/^ou_[0-9A-Za-z]+$/.test(openId)) return COPY.done
+  return `<at user_id="${openId}">你</at> ${COPY.done}`
+}
+
 function routeOf(selection                )                                                                {
   return {
     provider: selection.provider,

package/lib/store.js

@@ -82,6 +82,7 @@ export class LineStore {
 
   /**
    * Decide one message and, when it starts a turn, mark the line running before returning.
+   * A started turn clears the card message id so the driver posts a new card under this trigger.
    * @param message - Normalized inbound message.
    * @param allowOpenIds - Trigger allow list.
    * @returns The decision and the line when one exists or was created.
@@ -113,6 +114,7 @@ export class LineStore {
         ...addAliases(base, ids),
         phase: 'running',
         cardStatus: 'running',
+        cardMessageId: null,
         turn: null,
         triggerMessageId: message.messageId,
         cancelRequested: false,

package/lib/types.js

@@ -64,6 +64,8 @@
                           
                                                
                    
+                                                                                
+                 
  
 
 /** Feishu JSON object sent as an interactive card. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cpm-lhj/dsh-feishu",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "private": false,
   "type": "module",
   "description": "Feishu long-connection plugin that runs a resident dsh session from allowlisted messages.",