From 7feacc63714fd4f446c1548147dd813a57f1f9e6 Mon Sep 17 00:00:00 2001
From: jackwener <jakevingoo@gmail.com>
Date: Thu, 14 May 2026 18:48:46 +0800
Subject: [PATCH] fix(daemon): drop redundant `ok` from extract payload
 (collides with Response.ok)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Response 用 #[serde(flatten)] 把 q_* 返回的 Value 拼到 `{ok, error, ...data}`
里，q_extract 里再塞一个 `"ok": true` 就会在 wire 上写出两个同名 key，CLI
端 `serde_json::from_str::<Response>` 直接报「duplicate field `ok`」，对外
表现是「extract 失败 / 解析 daemon 响应失败」，但 daemon 实际已经把图解出来
了。其他 q_* 都没塞 ok（biz_articles / sessions / history 等），保持一致。
---
 src/daemon/query.rs | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/daemon/query.rs b/src/daemon/query.rs
index 5a5d1b9..6bd46b2 100644
--- a/src/daemon/query.rs
+++ b/src/daemon/query.rs
@@ -3549,9 +3549,11 @@ pub async fn q_extract(
         std::fs::write(&output_path2, &decoded.data)
             .with_context(|| format!("写出文件失败：{}", output_path2.display()))?;
 
+        // 注意：不要在这里塞 `ok: true`。dispatch 会用 Response::ok(v) 包一层，
+        // Response 的 `data: Value` 字段是 #[serde(flatten)] 写出的，本 payload
+        // 的 `ok` 会和 Response 自带的 `ok` 在线上拼成两个同名 key，CLI 反序列化时
+        // serde_json 直接报 "duplicate field"，业务请求看上去像 daemon 解析失败。
         Ok(json!({
-            "ok": true,
-            "attachment_id": attachment_id_str(&id_for_task)?,
             "kind": id_for_task.kind.as_str(),
             "md5": resolved.md5,
             "dat_path": resolved.dat_path.display().to_string(),
@@ -3593,10 +3595,6 @@ fn parse_attachment_kinds(
     Ok(out)
 }
 
-fn attachment_id_str(id: &crate::attachment::AttachmentId) -> Result<String> {
-    id.encode()
-}
-
 #[cfg(test)]
 mod biz_tests {
     use super::*;