@ifrankwang/openspec-agents 0.147.1 → 0.147.2

package/.dsh-plugin/opx-tools.mjs

@@ -8213,18 +8213,24 @@ async function agentSubmitExecute(params, ctx) {
       throw new Error(`worktree 未就绪，当前拒绝提交。
 请编排者先调用 opx_orch_set_worktree 确保 worktree 就绪后再提交。`);
     }
-    if (params.fixed_issue_ids)
-      params.fixed_issue_ids = params.fixed_issue_ids.map(normalizeIssueId);
-    if (params.exempt_issue_ids)
-      params.exempt_issue_ids = params.exempt_issue_ids.map(normalizeIssueId);
-    if (params.exempt_adjudications) {
-      for (const adj of params.exempt_adjudications)
-        adj.issue_id = normalizeIssueId(adj.issue_id);
-    }
-    if (params.recheck_adjudications) {
-      for (const adj of params.recheck_adjudications)
-        adj.issue_id = normalizeIssueId(adj.issue_id);
-    }
+    const normalized = { ...params };
+    if (normalized.fixed_issue_ids)
+      normalized.fixed_issue_ids = normalized.fixed_issue_ids.map(normalizeIssueId);
+    if (normalized.exempt_issue_ids)
+      normalized.exempt_issue_ids = normalized.exempt_issue_ids.map(normalizeIssueId);
+    if (normalized.exempt_adjudications) {
+      normalized.exempt_adjudications = normalized.exempt_adjudications.map((adj) => ({
+        ...adj,
+        issue_id: normalizeIssueId(adj.issue_id)
+      }));
+    }
+    if (normalized.recheck_adjudications) {
+      normalized.recheck_adjudications = normalized.recheck_adjudications.map((adj) => ({
+        ...adj,
+        issue_id: normalizeIssueId(adj.issue_id)
+      }));
+    }
+    params = normalized;
     const knownChildIds = new Set;
     for (const c of item.children) {
       knownChildIds.add(c.id);
@@ -8984,7 +8990,7 @@ function buildNativeToolExecutes(unattended, processCwd) {
   const wrap = (execute) => async (args, exec) => {
     exec.signal?.throwIfAborted?.();
     const ctx = resolveToolContext(exec, args, { processCwd });
-    const cleanArgs = { ...args };
+    const cleanArgs = structuredClone(args);
     delete cleanArgs[AGENT_ARG];
     if (unattended)
       await ensureDefaultUnattended(cleanArgs, ctx);

package/.mcp-server/cli.mjs

@@ -29810,18 +29810,24 @@ async function agentSubmitExecute(params, ctx) {
       throw new Error(`worktree 未就绪，当前拒绝提交。
 请编排者先调用 opx_orch_set_worktree 确保 worktree 就绪后再提交。`);
     }
-    if (params.fixed_issue_ids)
-      params.fixed_issue_ids = params.fixed_issue_ids.map(normalizeIssueId);
-    if (params.exempt_issue_ids)
-      params.exempt_issue_ids = params.exempt_issue_ids.map(normalizeIssueId);
-    if (params.exempt_adjudications) {
-      for (const adj of params.exempt_adjudications)
-        adj.issue_id = normalizeIssueId(adj.issue_id);
-    }
-    if (params.recheck_adjudications) {
-      for (const adj of params.recheck_adjudications)
-        adj.issue_id = normalizeIssueId(adj.issue_id);
+    const normalized = { ...params };
+    if (normalized.fixed_issue_ids)
+      normalized.fixed_issue_ids = normalized.fixed_issue_ids.map(normalizeIssueId);
+    if (normalized.exempt_issue_ids)
+      normalized.exempt_issue_ids = normalized.exempt_issue_ids.map(normalizeIssueId);
+    if (normalized.exempt_adjudications) {
+      normalized.exempt_adjudications = normalized.exempt_adjudications.map((adj) => ({
+        ...adj,
+        issue_id: normalizeIssueId(adj.issue_id)
+      }));
+    }
+    if (normalized.recheck_adjudications) {
+      normalized.recheck_adjudications = normalized.recheck_adjudications.map((adj) => ({
+        ...adj,
+        issue_id: normalizeIssueId(adj.issue_id)
+      }));
     }
+    params = normalized;
     const knownChildIds = new Set;
     for (const c of item.children) {
       knownChildIds.add(c.id);
@@ -30569,7 +30575,7 @@ async function ensureDefaultUnattended(args, ctx) {
     }
   } catch {}
 }
-var PKG_VERSION = "0.147.1";
+var PKG_VERSION = "0.147.2";
 function buildMcpServer(worktree, opts = {}) {
   const mcp = new McpServer({ name: "openspec-agents", version: PKG_VERSION });
   for (const [name, spec] of Object.entries(TOOL_SPECS)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ifrankwang/openspec-agents",
-  "version": "0.147.1",
+  "version": "0.147.2",
   "description": "OpenSpec change orchestration plugin for AI coding agents (Claude Code, Codex, ZCode, DeepSeek Harness, OpenCode)",
   "author": "ifrankwang",
   "license": "MIT",

package/src/adapters/deepseek-harness/native-tools.ts

@@ -132,7 +132,8 @@ export function buildNativeToolExecutes(
       exec.signal?.throwIfAborted?.()
       const ctx = resolveToolContext(exec, args, { processCwd })
       // 私有身份参数不进入内核参数（内核校验的是纯业务 schema）。
-      const cleanArgs: Record<string, unknown> = { ...args }
+      // DSH 会把入参整份深冻结后原样交给 execute，这里必须交出可写副本（浅拷贝只保护顶层，嵌套仍只读）。
+      const cleanArgs: Record<string, unknown> = structuredClone(args)
       delete cleanArgs[AGENT_ARG]
       if (unattended) await ensureDefaultUnattended(cleanArgs, ctx)
       const result = await execute(cleanArgs, ctx)

package/src/core/tools/submit.ts

@@ -619,15 +619,25 @@ export async function agentSubmitExecute(params: AgentSubmitParams, ctx: ToolCon
       )
     }
 
-    // 参数归一化：issue id 去 # 前缀（fixed/exempt/adjudication/recheck 均接受带 # 的 tg.issue 序号引用）
-    if (params.fixed_issue_ids) params.fixed_issue_ids = params.fixed_issue_ids.map(normalizeIssueId)
-    if (params.exempt_issue_ids) params.exempt_issue_ids = params.exempt_issue_ids.map(normalizeIssueId)
-    if (params.exempt_adjudications) {
-      for (const adj of params.exempt_adjudications) adj.issue_id = normalizeIssueId(adj.issue_id)
+    // 参数归一化：issue id 去 # 前缀（fixed/exempt/adjudication/recheck 均接受带 # 的 tg.issue 序号引用）。
+    // 归一化结果必须回写到 params 绑定：其后的读取点全部依赖归一化后的值。宿主（DSH）可能把入参整份
+    // 冻结为只读，因此只能在可写副本上归一化，不能动传入对象（含嵌套元素）。
+    const normalized: AgentSubmitParams = { ...params }
+    if (normalized.fixed_issue_ids) normalized.fixed_issue_ids = normalized.fixed_issue_ids.map(normalizeIssueId)
+    if (normalized.exempt_issue_ids) normalized.exempt_issue_ids = normalized.exempt_issue_ids.map(normalizeIssueId)
+    if (normalized.exempt_adjudications) {
+      normalized.exempt_adjudications = normalized.exempt_adjudications.map((adj) => ({
+        ...adj,
+        issue_id: normalizeIssueId(adj.issue_id),
+      }))
     }
-    if (params.recheck_adjudications) {
-      for (const adj of params.recheck_adjudications) adj.issue_id = normalizeIssueId(adj.issue_id)
+    if (normalized.recheck_adjudications) {
+      normalized.recheck_adjudications = normalized.recheck_adjudications.map((adj) => ({
+        ...adj,
+        issue_id: normalizeIssueId(adj.issue_id),
+      }))
     }
+    params = normalized
 
     // 无效 fixed/exempt id 入口显式拒绝（不静默吞掉修复/豁免声明）：未命中的 id 抛错并列出可用 id 清单。
     // 以 resolveChildByIssueId 做权威判定，与 submitForStep 的 childById 及 collectFixedExemptLayers 消费端解析语义一致。