@jayyuen66/dsh-ocr-review 0.1.4 → 0.1.6

package/package.json

@@ -1,9 +1,12 @@
 {
   "name": "@jayyuen66/dsh-ocr-review",
-  "version": "0.1.4",
-  "description": "dsh 工作区代码审查插件：把 open-code-review（ocr CLI）封装为 5 个模型工具 + 设置卡片（从 settings.yaml 选 provider/model 应用到 OCR 配置）。",
+  "version": "0.1.6",
+  "description": "dsh 工作区代码审查插件：把 open-code-review（ocr CLI）封装为 5 个模型工具 + 设置卡片（从宿主设置服务的 provider 清单选 provider/model 应用到 OCR 配置）。",
   "keywords": [
+    "AI code review",
+    "DeepSeek Harness",
     "code-review",
+    "deepseek-harness",
     "dsh",
     "dsh-plugin",
     "ocr",
@@ -44,71 +47,64 @@
     "registry": "https://registry.npmjs.org"
   },
   "dependencies": {
-    "@jayyuen66/dsh-plugin-shared": "^0.1.4",
+    "@jayyuen66/dsh-plugin-shared": "^0.1.6",
     "js-yaml": "^5.4.3"
   },
   "devDependencies": {
-    "@deepseek-ai/cordis": "4.0.5-alpha.1",
-    "@deepseek-ai/dsh-agent": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-api-remotes": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-atomic-write": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-locale": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-store": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-ui-layout": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-ui-plugin-manager": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-ui-renderer": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-ui-settings": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-client-ui-slots": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-credentials": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-home-paths": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-host-webserver": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-jobs": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-jobs-local": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-output-retention": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-package-manifest": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-ptc-runtime": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-sandbox": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-scope": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-session": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-session-projection": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-settings": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-shell": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-subprocess": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-system-prompt": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-tools": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-user-approval": "0.2.1-alpha.2",
-    "@deepseek-ai/schemastery": "3.18.5-alpha.1",
+    "@deepseek-ai/cordis": "4.0.5",
+    "@deepseek-ai/dsh-agent": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-api-remotes": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-atomic-write": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-locale": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-store": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-ui-layout": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-ui-plugin-manager": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-ui-renderer": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-ui-settings": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-client-ui-slots": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-credentials": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-home-paths": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-host-webserver": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-jobs": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-jobs-local": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-output-retention": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-package-manifest": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-ptc-runtime": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-sandbox": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-scope": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-session": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-session-projection": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-settings": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-shell": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-subprocess": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-system-prompt": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-tools": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-typert-protocol": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-user-approval": "0.2.1-rc.2",
+    "@deepseek-ai/schemastery": "3.18.5",
     "@types/node": "26.6.5",
     "@types/react": "19.3.0",
-    "@vitest/coverage-v8": "5.0.3",
+    "@vitest/coverage-v8": "5.0.5",
     "eslint-plugin-sonarjs": "4.2.2",
-    "oxfmt": "0.72.0",
-    "oxlint": "1.87.0",
+    "oxfmt": "0.73.0",
+    "oxlint": "1.88.0",
     "oxlint-tsgolint": "7.0.2004",
     "react": "^19.3.0",
-    "rolldown": "1.2.13",
+    "rolldown": "1.2.14",
     "typescript": "7.0.2",
     "vite": "8.3.4",
-    "vitest": "5.0.3"
+    "vitest": "5.0.5"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh": "^0.2.1-alpha.2",
-    "@deepseek-ai/dsh-atomic-write": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-credentials": "0.2.1-alpha.2",
-    "@deepseek-ai/dsh-home-paths": "0.2.1-alpha.2",
-    "@deepseek-ai/schemastery": "3.18.5-alpha.1"
-  },
-  "peerDependenciesMeta": {
-    "@deepseek-ai/dsh": {
-      "optional": true
-    }
-  },
-  "optionalDependencies": {
-    "@alibaba-group/open-code-review": "^1.12.13"
+    "@deepseek-ai/dsh-atomic-write": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-credentials": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-home-paths": "0.2.1-rc.2",
+    "@deepseek-ai/dsh-typert-protocol": "0.2.1-rc.2",
+    "@deepseek-ai/schemastery": "3.18.5"
   },
   "icon": "icon.svg",
   "engines": {
-    "dsh": ">=0.2.1-alpha.2",
+    "dsh": "^0.2.1-rc.2",
     "node": ">=22.12.0"
   },
   "dsh": {
@@ -121,6 +117,7 @@
       "immediately": true
     }
   },
+  "ocrCliVersion": "1.13.0",
   "scripts": {
     "build:client": "node build-client.mjs",
     "build": "node build-host.mjs",

package/scripts/get-cred.mjs

@@ -3,7 +3,7 @@
 // get-cred.mjs —— open-code-review 的 api_key_cmd 入口（本包唯一还在读宿主凭据
 // 文件的地方，且**只读不问服务**）。
 //
-// 为什么这个脚本必须存在（官方通道改造后仍然保留）：
+// 为什么这个脚本必须存在：
 //   - dsh 侧的 provider 清单与 key 状态已经改走官方通道（ctx.settings.describe() /
 //     ctx.credentials），但 OCR 是**另一个进程树**里的外部 CLI：它只会按
 //     ~/.opencodereview/config.json 里的 api_key_cmd 起子进程取 key，够不到 cordis
@@ -24,15 +24,18 @@
 //   本脚本的 homedir() 兜底随之读 X/.dsh 就跑偏了；部署若以非 env 方式指定 dsh home，
 //   env 档同样不可靠。
 //
-// 取值的层序镜像官方 credentials-local 的优先级里**够得到**的那两层：
-//   1. 继承来的进程环境。注意：dsh 起 shell 会经 scrubbedParentEnv 剥掉名字命中
+// 取值的层序必须与宿主**同源**：官方 credentials-local 的 LocalCredentialProvider
+// 按「托管文件 > 继承环境 > .env」取值，本脚本只要排得不一样，同一 ref 在两处都有值
+// 时 OCR 拿到的 key 就和宿主卡片认下的有效 key 分叉——两边都"成功"，打的是不同账号。
+// 照抄官方优先级里本脚本**够得到**的那三层：
+//   1. `$DSH_HOME/.credentials.yaml` 的 `refs.<REF>`（version: 1 托管文档，格式来源
+//      packages/credentials/credentials-local 的 parseCredentialsDocument）；
+//   2. 继承来的进程环境。注意：dsh 起 shell 会经 scrubbedParentEnv 剥掉名字命中
 //      KEY/PASSWORD/SECRET/TOKEN 或 DSH_ 前缀的变量——凭据 ref 几乎全中招，所以对
 //      经 dsh shell 起的 OCR，这一层对凭据 ref 实际常为空；它只在 ref 名不被净化
 //      模式命中、或调用链不经 dsh shell（手工在终端跑 ocr）时才有值。宿主侧的
 //      select/migrate 已按同一事实加闸（keyReachableAtOcrRuntime）：key 只存在于
 //      环境层/项目 .env 的配置会在写盘前被拒绝，而不是等 OCR 401。
-//   2. `$DSH_HOME/.credentials.yaml` 的 `refs.<REF>`（version: 1 托管文档，
-//      格式来源 packages/credentials/credentials-local/src/index.ts:169-216）；
 //   3. `$DSH_HOME/.env`（用户级回退层）。
 //   **项目 .env 刻意不读**：本脚本的 cwd 是 OCR 当时所在的仓库，跟 dsh 的启动工作区
 //   不是一回事，去读它等于把「被审查仓库」里的键当成用户的凭据。
@@ -107,12 +110,6 @@ function isNotFound(error) {
   return isRecord(error) && error.code === "ENOENT";
 }
 
-/**
- * 托管 YAML 文档里的 refs.<REF>（形状不合一律回 undefined，不抛）。
- * @param {string} credPath
- * @param {string} refName
- * @returns {string | undefined}
- */
 /**
  * YAML 解析错误的**脱敏**描述：只留 reason + 行列定位，不带出错位置前后若干行的原文。
  *
@@ -154,6 +151,13 @@ function yamlErrorDetail(error) {
   return "";
 }
 
+/**
+ * 托管 YAML 文档里的 refs.<REF>（形状不合一律回 undefined，不抛）。
+ * 解析失败走 `yamlErrorDetail`，绝不把邻域原文带进 stderr。
+ * @param {string} credPath
+ * @param {string} refName
+ * @returns {string | undefined}
+ */
 function fromCredentialsDoc(credPath, refName) {
   let text;
   try {
@@ -207,6 +211,16 @@ function unquotedValue(raw) {
   return trimmed;
 }
 
+/**
+ * 继承来的进程环境层：去首尾空白，空白按「这一层没有值」处理。
+ * @param {string | undefined} raw
+ * @returns {string | undefined}
+ */
+function fromInheritedEnv(raw) {
+  const value = typeof raw === "string" ? raw.trim() : "";
+  return value === "" ? undefined : value;
+}
+
 /**
  * $DSH_HOME/.env 的 KEY=VALUE（去掉可选引号；文件不存在按「该层没有值」处理）。
  * @param {string} envPath
@@ -243,11 +257,10 @@ if (!REF_PATTERN.test(refName)) {
   process.exit(1);
 }
 
-const inherited = process.env[refName];
 const home = dataDirFrom(process.argv[3]);
 const found =
-  (typeof inherited === "string" && inherited.trim().length > 0 ? inherited.trim() : undefined) ??
   fromCredentialsDoc(path.join(home, CREDENTIALS_FILENAME), refName) ??
+  fromInheritedEnv(process.env[refName]) ??
   fromUserDotEnv(path.join(home, ".env"), refName);
 
 if (found === undefined) {
@@ -255,7 +268,7 @@ if (found === undefined) {
   // 各层的坏消息），否则用户只能看到「llm 请求 401」这种隔了一层的症状。
   const notes = problems.length === 0 ? "" : `；${problems.join("；")}`;
   process.stderr.write(
-    `[get-cred] 取不到 ${refName}：查过进程环境、${path.join(home, CREDENTIALS_FILENAME)} 与 ${path.join(home, ".env")}${notes}\n`,
+    `[get-cred] 取不到 ${refName}：查过 ${path.join(home, CREDENTIALS_FILENAME)}、进程环境与 ${path.join(home, ".env")}${notes}\n`,
   );
   process.exit(1);
 } else {