@mawaru/sdk 0.25.0 → 0.26.0

package/_schemas/duplicate-selection.test.ts

@@ -93,6 +93,14 @@ describe("duplicateSelection（選択ノードの複製）", () => {
     expect(new Set(newPortIds).size).toBe(newPortIds.length)
   })
 
+  it("node_key は引き継がない（同じ版に同じ node_key が2つあると保存 API が 422 になる）", () => {
+    const { nodes, connections, b } = chain()
+    const saved = nodes.map((n) => ({ ...n, node_key: uuid() }))
+    const result = dup(saved, connections, [b.id])
+    expect(result.nodes).toHaveLength(1)
+    expect(result.nodes[0]?.node_key).toBeUndefined()
+  })
+
   it("元のノード・接続を変更しない", () => {
     const { b, c, nodes, connections } = chain()
     const snapshot = structuredClone({ nodes, connections })

package/_schemas/duplicate-selection.ts

@@ -20,11 +20,13 @@ export const duplicateSelection = (params: {
   const selected = new Set(selectedIds)
 
   // start / end は回路の構造ノードなので複製しない（選択に混ざっていても落とす）。
-  // 並び順は元グラフのまま（選択の指定順に依存させない）
+  // 並び順は元グラフのまま（選択の指定順に依存させない）。
+  // node_key は既存行との突き合わせ用なので複製には引き継がない（同じ版に同じ key が
+  // 2つあると保存 API が 422 にする）
   const portIdMap = new Map<string, string>()
   const copied = nodes
     .filter((n) => selected.has(n.id) && n.kind !== "start" && n.kind !== "end")
-    .map((n) => ({
+    .map(({ node_key: _node_key, ...n }) => ({
       ...n,
       id: newId(),
       ports: n.ports.map((p) => {

package/_schemas/extract-component.test.ts

@@ -216,6 +216,28 @@ describe("buildExtractPlan（子グラフと親置換の生成）", () => {
     expect(internal).toBeDefined()
   })
 
+  it("子：node_key は持ち込まない（子 loop の draft に無い key で保存 API が 422 になる）", () => {
+    const { nodes, connections, b, c } = chain()
+    // 保存済みの親を読んだ状態（各ノードが node_key を持つ）から抽出する
+    const saved = nodes.map((n) => ({ ...n, node_key: uuid() }))
+    const plan = buildExtractPlan({
+      nodes: saved,
+      connections,
+      selectedIds: [b.id, c.id],
+      name: "PRレビュー",
+      componentLoopId: uuid(),
+      childStartOutPortId: uuid(),
+      childEnd: childEndNode(),
+      newId: uuid,
+    })
+    if (!plan.ok) throw new Error(plan.reason)
+    for (const n of plan.childNodes) expect(n.node_key).toBeUndefined()
+    expect(plan.groupNode.node_key).toBeUndefined()
+    // 親に残るノードの node_key はそのまま（既存行との突き合わせに使う）
+    const remaining = plan.parentNodes.filter((n) => n.kind !== "component")
+    for (const n of remaining) expect(n.node_key).toBeDefined()
+  })
+
   it("子：start.main → 入口、出口ポート → end.in が配線される（出口1つは seed の in を使う）", () => {
     const { plan, childStartOutPortId, childEnd } = build()
     const [nb, nc] = plan.childNodes

package/_schemas/extract-component.ts

@@ -134,12 +134,14 @@ export const buildExtractPlan = (params: {
 
   const selected = new Set(selectedIds)
   // ノード・ポートとも新 id を採番してコピー（kind 別設定・hooks は共有参照で問題ない
-  // ＝この後 JSON 化して保存 API に渡すだけで、エディタ状態には子側を持たない）
+  // ＝この後 JSON 化して保存 API に渡すだけで、エディタ状態には子側を持たない）。
+  // node_key は親の版の行との突き合わせ用なので子へは持ち込まない（子 loop の draft に
+  // 無い key を送ると保存 API が「この版に存在しないノード」で 422 にする）
   const nodeIdMap = new Map<string, string>()
   const portIdMap = new Map<string, string>()
   const childNodes = nodes
     .filter((n) => selected.has(n.id))
-    .map((n) => {
+    .map(({ node_key: _node_key, ...n }) => {
       const id = newId()
       nodeIdMap.set(n.id, id)
       return {

package/dist/_schemas/duplicate-selection.js

@@ -8,11 +8,13 @@ export const duplicateSelection = (params) => {
     const { nodes, connections, selectedIds, dx, dy, newId } = params;
     const selected = new Set(selectedIds);
     // start / end は回路の構造ノードなので複製しない（選択に混ざっていても落とす）。
-    // 並び順は元グラフのまま（選択の指定順に依存させない）
+    // 並び順は元グラフのまま（選択の指定順に依存させない）。
+    // node_key は既存行との突き合わせ用なので複製には引き継がない（同じ版に同じ key が
+    // 2つあると保存 API が 422 にする）
     const portIdMap = new Map();
     const copied = nodes
         .filter((n) => selected.has(n.id) && n.kind !== "start" && n.kind !== "end")
-        .map((n) => ({
+        .map(({ node_key: _node_key, ...n }) => ({
         ...n,
         id: newId(),
         ports: n.ports.map((p) => {

package/dist/_schemas/extract-component.js

@@ -61,12 +61,14 @@ export const buildExtractPlan = (params) => {
         return analysis;
     const selected = new Set(selectedIds);
     // ノード・ポートとも新 id を採番してコピー（kind 別設定・hooks は共有参照で問題ない
-    // ＝この後 JSON 化して保存 API に渡すだけで、エディタ状態には子側を持たない）
+    // ＝この後 JSON 化して保存 API に渡すだけで、エディタ状態には子側を持たない）。
+    // node_key は親の版の行との突き合わせ用なので子へは持ち込まない（子 loop の draft に
+    // 無い key を送ると保存 API が「この版に存在しないノード」で 422 にする）
     const nodeIdMap = new Map();
     const portIdMap = new Map();
     const childNodes = nodes
         .filter((n) => selected.has(n.id))
-        .map((n) => {
+        .map(({ node_key: _node_key, ...n }) => {
         const id = newId();
         nodeIdMap.set(n.id, id);
         return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mawaru/sdk",
-  "version": "0.25.0",
+  "version": "0.26.0",
   "description": "mawaru 実行 repo の開発 SDK。契約スキーマ（config.json 規約・ループ graph API）の正 ＋ typegen / validate CLI",
   "type": "module",
   "bin": {