@dsh-workbench/core 0.1.11 → 0.1.12

package/dist/host/index.js

@@ -1474,7 +1474,7 @@ var panelsUnit = {
 };
 //#endregion
 //#region src/modules/panels/model/approval.ts
-var READ$1 = { kind: "read" };
+var READ = { kind: "read" };
 var READ_ONLY_ACTIONS = /* @__PURE__ */ new Set(["list", ""]);
 function actionOf$1(call) {
 	const raw = call.arguments;
@@ -1502,7 +1502,7 @@ var panelsApproval = (call) => {
 	const readAction = action !== void 0 && action !== "" && READ_ONLY_ACTIONS.has(action);
 	const what = whatOf(id);
 	if (call.name === "workbench_modules") {
-		if (readAction) return READ$1;
+		if (readAction) return READ;
 		if (action === "update-dependency") return askOf(`workbench_modules update-dependency ${id ?? ""} upgrades a shared dependency on this machine`, {
 			en: "it upgrades that shared dependency on this machine. Allow?",
 			zh: `会升级共用依赖 ${what}。允许吗？`
@@ -1521,7 +1521,7 @@ var panelsApproval = (call) => {
 		});
 	}
 	if (call.name !== "workbench_panels") return void 0;
-	if (readAction) return READ$1;
+	if (readAction) return READ;
 	if (action === "hide") return askOf("workbench_panels hide removes a module from the panel", {
 		en: "it takes that module off this panel (its agent tools go offline here too). Allow?",
 		zh: `${what}会从这个项目的面板上收起来（这个项目里它的 agent 工具也一起下线）。允许吗？`
@@ -3999,54 +3999,6 @@ function createUnitRegistry() {
 	};
 }
 //#endregion
-//#region src/shared/api/host/approval-legacy.ts
-var READ = { kind: "read" };
-function confirmed(call) {
-	const raw = call.arguments;
-	if (typeof raw !== "object" || raw === null) return void 0;
-	const value = raw.confirm;
-	return typeof value === "boolean" ? value : void 0;
-}
-var legacyApproval = (call) => {
-	const name = call.name ?? "";
-	if (name === "workbench_cmdb_release") {
-		if (confirmed(call) !== true) return READ;
-		return {
-			kind: "ask",
-			reason: "confirm=true releases a version to a real environment",
-			question: {
-				en: "it deploys on the platform for real (non-production). Allow?",
-				zh: "会在平台上真的部署一次（非生产环境）。允许吗？"
-			},
-			fallback: {
-				en: "The agent wants to deploy a version for real (confirm: true) — it deploys on the platform (non-production). Allow?",
-				zh: "agent 想在非生产环境上**真的部署一个版本**（confirm: true）—— 会在平台上部署。允许吗？"
-			}
-		};
-	}
-	if (name !== "workbench_coding_issue_comment" && name !== "workbench_coding_issue_transfer") return void 0;
-	if (confirmed(call) !== true) return READ;
-	const comment = name === "workbench_coding_issue_comment";
-	return {
-		kind: "ask",
-		reason: comment ? "confirm=true posts a comment on the platform" : "confirm=true moves an issue to another status on the platform",
-		question: comment ? {
-			en: "it posts that comment on the platform for real. Allow?",
-			zh: "会在平台上真的发表这条评论。允许吗？"
-		} : {
-			en: "it changes that issue status on the platform for real. Allow?",
-			zh: "会在平台上真的改这条缺陷的状态。允许吗？"
-		},
-		fallback: comment ? {
-			en: "The agent wants to post a comment on the platform for real (confirm: true) — it writes on the platform. Allow?",
-			zh: "agent 想**真的给一条缺陷加评论**（confirm: true）—— 会写平台数据。允许吗？"
-		} : {
-			en: "The agent wants to move an issue to another status for real (confirm: true) — it writes on the platform. Allow?",
-			zh: "agent 想**真的把一条缺陷流转到别的状态**（confirm: true）—— 会写平台数据。允许吗？"
-		}
-	};
-};
-//#endregion
 //#region src/shared/api/host/tool-scope.ts
 function agentOf(value) {
 	return typeof value === "object" && value !== null ? value : void 0;
@@ -4389,13 +4341,15 @@ function apply(ctx) {
 			const owner = tool === void 0 ? void 0 : mounted.get(tool.unit)?.result;
 			const detailOf = owner === void 0 ? void 0 : (call) => owner.approvalDetail?.(call);
 			const workspace = sessionWorkspace(exec) ?? void 0;
-			return await approvalFor({
+			const call = {
 				...name === void 0 ? {} : { name },
 				arguments: exec.arguments,
 				...workspace === void 0 ? {} : { workspace }
-			}, {
+			};
+			const policy = owner?.needsApproval;
+			return await approvalFor(call, {
 				detailOf,
-				policy: owner?.needsApproval ?? legacyApproval,
+				policy,
 				writes: tool?.writes ?? false
 			}) ?? decision;
 		});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dsh-workbench/core",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "个人工作台：Agent 工作台外壳（左栏 + 常驻对话 + 自研右侧数据面板）。",
   "type": "module",
   "icon": "./icon.svg",
@@ -86,7 +86,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@dsh-workbench/shared": "0.1.11"
+    "@dsh-workbench/shared": "0.1.12"
   },
   "scripts": {
     "build": "node scripts/build.mjs && node scripts/build-types.mjs",

package/skills/workbench-module-diy/DATASOURCE.md

@@ -0,0 +1,54 @@
+# 数据从哪来：干净机器上先用 opencli 探，再把它固化成一条读
+
+用户往往只说"我要看发布情况""我想看缺陷看板" —— 他不知道那套系统叫什么、接口在哪、要什么参数。
+干净机器上你也没有现成的 skill 可抄。所以这一步是**探查**，不是查手册。
+
+## 先判：这是 DIY 域还是工程域
+
+| 情况 | 判 |
+|---|---|
+| 他已经在浏览器里登录过那套系统 | **DIY 域** —— 用 opencli 复用他的登录去探 |
+| 要**新凭据**（新账号 / 密钥 / token） | **工程域** —— 停下来，说清要谁去配什么 |
+| 要起新服务 / 新进程 / 加新依赖 | **工程域** |
+| 数据在本地文件 / 一条已有命令 / 一个不要凭据的 GET 上 | **DIY 域** —— 直接造 |
+
+判的是「**要不要新凭据**」，不是「要不要接新平台」。复用已有登录的探查，正是把一大类需求留在 DIY 域的原因。
+
+## 四步
+
+1. **问出他在看什么（用他的词）** —— "你平时是在哪个网页 / 哪个软件里看这个的？" 记下他说的入口。
+   **不要**问"你们用 argocd 还是 jenkins" —— 他多半不知道，也答不上来。
+2. **探** —— 用 `opencli`。**它随工作台预装**：隔离运行时在 `$DSH_HOME/workbench/opencli/`，
+   启动体检自己装，**不碰**用户本机的 `~/.opencli`（工作台已经把它挂成命令，`bin/opencli-isolated` 是那层壳）。
+   目标是找到**承载数据的那一处**：列表页本身，或它背后的那一个 XHR。记下：命令 / URL、需要的参数、凭据从哪来。
+   - 数据通常来自 **opencli 的某个插件**（`opencli <plugin> …`）—— 先看这台机器上哪些插件能用，
+     **别自己拼接口**。
+   - 隔离运行时没装好时它会自己说"opencli 的隔离运行时还没装好" —— 把体检那屏的原话转给用户，
+     别硬编 URL 猜（那正是"看起来能用、其实读不到"的来源）。
+3. **试一条最小读** —— 先证明"能拿到"，再写模块：
+   - 拿得到结构化输出 → 记下字段形状
+   - 要登录但已过期 → 记下"未登录"长什么样
+   - 超时 / 被墙 → 记下这两种失败的形状
+   这一步的产出是**一条可重复的命令或请求**，不是"我探过了"。
+4. **固化进模块** —— 把它写进模块的**宿主半侧**（一条命令，或只读路由里的一个 HTTP GET）。
+   之后卡片只依赖**这一条**；渲染怎么改都不动它。
+
+`opencli` 是**侦察兵，不是日常通道**：探明之后，日常读必须是一条固定的、便宜的、能解释的调用。
+
+## 固化时一并写掉三态
+
+拿不到数据是常态（第一次读、登录过期、站点改版、接口超时）。卡片**必须**画得出来：
+
+| 情况 | 卡片说什么 |
+|---|---|
+| 没登录 / 凭据过期 | "去 XX 登录一下"（可点，指到他平时那个入口） |
+| 超时 / 网络不通 | "这次没读到，稍后重试" + 重试 |
+| 站点改版、字段没了 | "读到的形状和预期不一样"（别显示空白，也别显示 0） |
+
+空数据下抛异常的卡片 = 用户眼里的"卡片一直不出来"。这条 `check:cards` 会拦（见 [REVIEW.md](REVIEW.md)）。
+
+## 你自己的门禁会查
+
+- 数据源写在**宿主半侧**，卡片只消费它的返回；
+- 只读路由出错时回 `{ ok: false, error: { code, message, hint? } }`（HTTP 仍是 200，外壳原样转发）；
+- `?wbMock=1` 下三档夹具都要能出（正常 / 空 / 出错）—— 这是你自己验收"三态"的快捷方式。

package/skills/workbench-module-diy/DESIGN.md

@@ -13,12 +13,13 @@
 | 卡片要能自己配（钉一个环境、选一个仓库） | **DIY 域** | 配置清单（[PANEL.md](PANEL.md) §5） |
 | 要 agent 替他做一两件**只读**的事 | **DIY 域** | 能力清单（[TOOLS.md](TOOLS.md)） |
 | 要 agent 动真东西（提交、发布、改配置） | **DIY 域**，但必须写明点头那一步 | 预览 + 确认两拍（[TOOLS.md](TOOLS.md) §5–§7） |
-| 要接一个新平台的账号 / 凭据 / 新命令 | **工程域** | 停，说清为什么，回工作台仓库走工程流程 |
+| 他已经在浏览器里登录过那套系统，只是**没人告诉他数据在哪** | **DIY 域** | 用 opencli 复用他的登录去探（[DATASOURCE.md](DATASOURCE.md)） |
+| 要**新凭据**（新账号 / 密钥 / token） | **工程域** | 停，说清要谁去配什么，回工作台仓库走工程流程 |
 | 一条流：多步、要等很久、中途要人、失败要能接着来 | **工程域** | 同上 |
 | 要新建服务 / 新进程 / 新依赖 | **工程域** | 同上 |
 | 要改工作台外壳，或改一份**已经发布**的模块 | **工程域** | 同上 |
 
-判域产出的是**他能听懂的一句话**（"这件事要接一个新平台的账号，得按工程流程走"），
+判域产出的是**他能听懂的一句话**（"这件事得先有人给你配一个 XX 的账号，我按工程流程走"），
 以及"接下来会发生什么、还要多久"。不是"你这个需求超纲了"。
 
 ## 1 两条分工（贯穿全程）

package/skills/workbench-module-diy/SKILL.md

@@ -7,18 +7,20 @@ description: 在工作台里凭需求造一个模块（面板上的一块内容
 
 交付物是**面板上能点开的一块内容**，不是一份要他读的技术方案。
 
-## 六步
+## 七步
 
 1. **判域** —— 先读 [DESIGN.md](DESIGN.md) §0。不是所有需求都该在这里做；判完给他**一句人话 + 下一步**。
 2. **问三张表** —— 见 [STEPS.md](STEPS.md)。只问他看得见、能决定的事；**实现一律不问**。
    这一步的产出是 ASCII 画面 + 三张表，不是文字。
-3. **从模板改** —— [BUILD.md](BUILD.md) §1–§2：在工坊里 `dsh-wb-module init` 一个骨架，
+3. **先定数据从哪来** —— [DATASOURCE.md](DATASOURCE.md)：干净机器上没有现成 skill 可抄，
+   用 `opencli` 复用他的登录去探，探明后**固化成一条读**。顺手把三态写掉。
+4. **从模板改** —— [BUILD.md](BUILD.md) §1–§2：在工坊里 `dsh-wb-module init` 一个骨架，
    只改数据源与渲染。**不从零写。**
-4. **自己过门禁** —— `dsh-wb-module build` + `check`；红了自己修（[REVIEW.md](REVIEW.md) 有对照），
+5. **自己过门禁** —— `dsh-wb-module build` + `check`；红了自己修（[REVIEW.md](REVIEW.md) 有对照），
    不抛给他、也不贴报错给他看。
-5. **让他看一眼** —— 先对着 ASCII 图确认，再**装上看看**（装是宿主侧那一跳，它会顺手在面板里打开它）。
+6. **让他看一眼** —— 先对着 ASCII 图确认，再**装上看看**（装是宿主侧那一跳，它会顺手在面板里打开它）。
    **这一跳一辈子只走一次**：之后只重建、不重装。
-6. **验收** —— 逐条对着三张表走：「打开哪 → 看什么 → 点什么 → 应该看到什么」。
+7. **验收** —— 逐条对着三张表走：「打开哪 → 看什么 → 点什么 → 应该看到什么」。
    他说哪不对就回对话改，改完让他看**同一处**；不给他看 diff、日志、代码。
 
 ## 每个问题都带推荐答案
@@ -33,6 +35,7 @@ description: 在工作台里凭需求造一个模块（面板上的一块内容
 | 判这个需求该不该在这里做 | [DESIGN.md](DESIGN.md) §0 |
 | 两条分工、三张表、怎么用 ASCII 跟他说话 | [DESIGN.md](DESIGN.md) |
 | 步骤 1 / 2 / 3 各自问什么、什么时候算做完 | [STEPS.md](STEPS.md) |
+| **数据从哪来**（干净机器上怎么找到那一条读） | [DATASOURCE.md](DATASOURCE.md) |
 | 面板上该有什么、状态矩阵怎么写、界面习惯 | [PANEL.md](PANEL.md) |
 | 能力怎么划、传什么拿回什么、哪几件要点头 | [TOOLS.md](TOOLS.md) |
 | agent 动完面板哪一格变、他正在干的时候看哪儿 | [LINKAGE.md](LINKAGE.md) |

package/dist/types/shared/api/host/__tests__/approval-legacy.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/types/shared/api/host/approval-legacy.d.ts

@@ -1,2 +0,0 @@
-import type { ApprovalPolicy } from '@dsh-workbench/shared/host';
-export declare const legacyApproval: ApprovalPolicy;