dsh-adb 0.1.2 → 0.1.4

package/README.md

@@ -64,16 +64,51 @@ npm pack --dry-run     # 校验发布包内容（lib/ + cordis.patch.yml）
 
 注意：本机若设了 `NODE_ENV=production`，npm 会跳过 devDependencies，安装时用 `npm install --include=dev`。
 
-## 真机/台架冒烟（2026-08-14，SA_DIREWOLF_IVI 台架）
+## 真机/台架冒烟（2026-08-14，SA_DIREWOLF_IVI 台架，Android 13）
 
-在 headless profile 中端到端验证（`dsh --profile bench "任务"`）：
+headless profile 端到端验证（`dsh --profile bench "任务"`），**7 个工具全部实测通过**：
 
-- `adb_devices` → `f20c9b04 device msmnile_gvmq_for_arm64` ✅
-- `adb_perf_snapshot` meminfo(com.android.systemui) → PSS 133995 KB / RSS 283832 KB / Java Heap 12328 KB / Native Heap 26692 KB ✅
-- `adb_logcat` level=E tail=5 → 真实设备错误日志结构化返回 ✅
-- 配置覆盖（`adbPath` / `defaultSerial`）✅ · 错误码（`ADB_NOT_FOUND`）✅
+| 工具 | 实测结果 |
+| --- | --- |
+| `adb_devices` | ✅ 返回 `f20c9b04 device msmnile_gvmq_for_arm64` |
+| `adb_connect` / `adb_disconnect` | ✅ 不可达端点正确返回 CONNECT_FAILED；disconnect 正常 |
+| `adb_logcat`（前向） | ✅ level=E tail=5 返回真实错误日志 |
+| `adb_logcat`（后台采集） | ✅ job_output 读回 43065 行/7MB 流，job_kill 正常 |
+| `adb_install` | ✅ pull 真机 apk → `install -r` → "Performing Streamed Install / Success" |
+| `adb_file` | ✅ ls / push / pull 往返字节一致 |
+| `adb_perf_snapshot` | ✅ meminfo（SystemUI PSS 131MB）+ gfxinfo（232 帧/60.78% 卡顿）解析正确；battery 空（台架无电池服务） |
+
+配置覆盖（`adbPath`/`defaultSerial`）✅ · 错误码（`ADB_NOT_FOUND`/`CONNECT_FAILED`）✅
+
+> 冒烟发现并修复三个已发布 bug：
+> - **v0.1.0 → v0.1.1**：`ctx.tools.register` 未声明 `inject: ['tools']` 导致加载失败（Cordis Guard 拒绝未声明依赖）
+> - **v0.1.1 → v0.1.2**：后台 logcat 的 `readOutput` 返回 `{added,text}` 对象，违反 jobs 契约（须返回字符串）导致 `job_output` 报 `value.text must be a string`
+> - **v0.1.2 → v0.1.3**：`DEVICE_NOT_FOUND` 分类只匹配带 `error:` 前缀的 adb 输出，真实输出 `adb.exe: device 'X' not found` 落入通用 `ADB_EXIT_1`
 
-> v0.1.0 → v0.1.1 修复：冒烟发现 `ctx.tools.register` 未声明 `inject: ['tools']` 导致加载失败（Cordis Guard 拒绝未声明依赖），已修复并重新发布。
+## 测试覆盖率矩阵（v0.1.3）
+
+| 功能 | 单元测试 | 端到端实测（台架） |
+| --- | --- | --- |
+| `adb_devices` | ✅ 解析器 | ✅ |
+| `adb_connect`（失败路径） | ✅ 分类器 | ✅ CONNECT_FAILED |
+| `adb_connect`（成功路径） | - | ⚠️ 环境受限（台架无线不可达），需车内/可路由网络 |
+| `adb_disconnect` | - | ✅ |
+| `adb_logcat` 前向（level/tail） | ✅ 解析器+过滤 | ✅ |
+| `adb_logcat` tag/keyword 过滤 | ✅ 过滤助手 | ✅（tag+keyword 均正向命中） |
+| `adb_logcat` since/until | ⚠️ 仅逻辑（时间串字典序） | 未单独 E2E（同过滤链） |
+| `adb_logcat` 后台采集 | - | ✅ job_output 读回 43065 行/7MB，job_kill 正常 |
+| `adb_install` 成功 | - | ✅ install -r Success |
+| `adb_install` 失败（LOCAL_FILE_NOT_FOUND/INSTALL_FAILED） | ✅ 分类器 | ✅ 伪 apk → INSTALL_PARSE_FAILED_NOT_APK |
+| `adb_file` ls / ls -lR | - | ✅ |
+| `adb_file` push/pull 往返 | - | ✅ 字节一致 |
+| `adb_file` rm | - | ✅ 删除生效 |
+| `adb_perf_snapshot` meminfo | ✅ 解析器 | ✅ SystemUI PSS/RSS/Heap |
+| `adb_perf_snapshot` gfxinfo | ✅ 解析器 | ✅ 帧率/卡顿/百分位 |
+| `adb_perf_snapshot` battery | ✅ 解析器 | ⚠️ 台架无电池服务返回空（行为正确） |
+| 配置 adbPath/defaultSerial | - | ✅ 覆盖生效 |
+| 错误码 ADB_NOT_FOUND/DEVICE_NOT_FOUND/NO_DEVICES/CONNECT_FAILED/INSTALL_FAILED | ✅ 分类器 | ✅ 实测/确定性验证 |
+
+**原则：提交即测。** 未能在本台架覆盖的路径（正向无线连接、battery 实数据）已明确标注，不视为「已测通过」。
 
 ## License
 

package/lib/adb.js

@@ -117,7 +117,7 @@ export async function runAdb(ctx, cfg, argv, options = {}) {
 export function classifyFailure(result) {
     const stderr = result.stderr;
     const stdout = result.stdout;
-    if (/error: device .* not found/i.test(stderr)) {
+    if (/(?:error:\s*)?device ['"]?[^'\r\n]*['"]? not found/i.test(stderr)) {
         return new AdbError('DEVICE_NOT_FOUND', excerpt(stderr));
     }
     if (/no devices\/emulators found/i.test(stderr)) {

package/lib/tools/perf.js

@@ -24,7 +24,11 @@ export function registerPerfTool(ctx, cfg) {
             const metrics = args.metrics ?? ['meminfo', 'gfxinfo', 'battery'];
             const result = { package: args.package, metrics: [...metrics] };
             for (const metric of metrics) {
-                const output = await runAdb(ctx, cfg, ['shell', 'dumpsys', metric, args.package], {
+                // dumpsys battery is device-global and takes no package argument.
+                const argv = metric === 'battery'
+                    ? ['shell', 'dumpsys', 'battery']
+                    : ['shell', 'dumpsys', metric, args.package];
+                const output = await runAdb(ctx, cfg, argv, {
                     signal: exec.signal,
                     serial: args.serial,
                     maxBytes: 4 * 1024 * 1024,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-adb",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "ADB device & bench operations for DeepSeek Harness: device discovery, structured logcat, apk install, file pull/push, performance snapshots",
   "type": "module",
   "main": "lib/index.js",