@mawaru/sdk 0.24.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/_schemas/graph.test.ts

@@ -160,17 +160,25 @@ describe("aiConfigSchema", () => {
 })
 
 describe("humanConfigSchema", () => {
-  it("既定値を補う（fixed・any・units 空＝tenant の誰でも）", () => {
-    expect(humanConfigSchema.parse({ assignment: {} })).toEqual({
-      assignment: { strategy: "fixed", approval_mode: "any", units: [] },
+  it("strategy / approval_mode の既定値を補う（fixed・any）", () => {
+    const u1 = uuid()
+    expect(humanConfigSchema.parse({ assignment: { units: [[u1]] } })).toEqual({
+      assignment: { strategy: "fixed", approval_mode: "any", units: [[u1]] },
     })
   })
 
+  it("担当なし（units 空・省略）を拒否する（「誰でも」は無い）", () => {
+    expect(() => humanConfigSchema.parse({ assignment: {} })).toThrow()
+    expect(() =>
+      humanConfigSchema.parse({ assignment: { units: [] } }),
+    ).toThrow(/担当/)
+  })
+
   // 承認ビューは不透明なキー（program_dir / ai_dir と同じ）。実在するか・そのテナントが
   // 選べるかは mawaru 側のカタログ（非公開）と保存 API が見るので、ここは形だけ
   it("承認ビューを受け付ける", () => {
     const parsed = humanConfigSchema.parse({
-      assignment: {},
+      assignment: { units: [[uuid()]] },
       view: "acme/invoice",
     })
     expect(parsed.view).toBe("acme/invoice")
@@ -178,7 +186,7 @@ describe("humanConfigSchema", () => {
 
   it("空のビューキーは拒否する", () => {
     expect(() =>
-      humanConfigSchema.parse({ assignment: {}, view: "" }),
+      humanConfigSchema.parse({ assignment: { units: [[uuid()]] }, view: "" }),
     ).toThrow()
   })
 
@@ -513,6 +521,8 @@ const humanNode = () => ({
     port("out", "reject"),
   ]),
   kind: "human" as const,
+  // 担当は必須（「担当なし＝誰でも」は無い）
+  human: { assignment: { units: [[uuid()]] } },
 })
 const aiNode = () => ({
   ...makeNode([
@@ -538,6 +548,13 @@ const outByKey = (node: TestNode, key: string): GraphPort => {
 }
 
 describe("saveGraphBodySchema（kind 別ポート構成）", () => {
+  it("human の担当なし（human 設定の省略）を拒否する", () => {
+    const { human: _human, ...noAssign } = humanNode()
+    expect(() =>
+      saveGraphBodySchema.parse({ nodes: [noAssign], connections: [] }),
+    ).toThrow(/担当/)
+  })
+
   it("human の旧 ok/ng 構成を拒否し、in/approve/reject を受け付ける", () => {
     expect(() =>
       saveGraphBodySchema.parse({ nodes: [humanNode()], connections: [] }),

package/_schemas/graph.ts

@@ -135,7 +135,8 @@ export type AssignMember = z.infer<typeof assignMemberSchema>
 
 // Human ノードの kind 別設定（node_humans ＋ node_human_assignees に対応）。
 // 担当 ＝ ユニット（1人以上のメンバー）の順序付きリスト ＋ 承認モード ＋ 選出戦略。
-// units[i] はメンバーの配列（1人＝個人・複数＝グループ）。units 空 = tenant の誰でも。
+// units[i] はメンバーの配列（1人＝個人・複数＝グループ）。**担当は必ず明示する**
+// （「担当なし＝tenant の誰でも」は無い。Human担当の必須化（誰でもの廃止）.md）。
 // approval_mode はノード単位（any=誰か1人 / all=全員。差し戻しは1人で成立＝veto）。
 // メンバーが tenant の membership であることの検証は保存 API 側で行う
 // （docs/tasks/backlog/Human担当者の高度化（複数人承認とラウンドロビン）.md）
@@ -148,7 +149,9 @@ export const humanConfigSchema = z.object({
     .object({
       strategy: z.enum(["fixed", "round_robin"]).default("fixed"),
       approval_mode: z.enum(["any", "all"]).default("any"),
-      units: z.array(z.array(assignMemberSchema).min(1)).default([]),
+      units: z
+        .array(z.array(assignMemberSchema).min(1))
+        .min(1, "担当を1人以上指定してください"),
     })
     .superRefine((assignment, ctx) => {
       if (assignment.strategy === "fixed" && assignment.units.length > 1) {
@@ -403,6 +406,16 @@ export const saveGraphBodySchema = z
         hookDirs.add(hook.hook_dir)
       })
 
+      // human は担当を必ず明示する（「担当なし＝tenant の誰でも」は無い。
+      // Human担当の必須化（誰でもの廃止）.md）
+      if (node.kind === "human" && node.human === undefined) {
+        ctx.addIssue({
+          code: "custom",
+          path: ["nodes", i, "human"],
+          message: "human ノードには担当（assignment.units）が必要です",
+        })
+      }
+
       // component は参照先が無いと成立しない（他 kind の設定はデフォルトで補えるが
       // component_loop_id は補えない）
       if (node.kind === "component" && node.component === undefined) {

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/dist/_schemas/graph.d.ts

@@ -58,7 +58,7 @@ export declare const humanConfigSchema: z.ZodObject<{
             any: "any";
             all: "all";
         }>>;
-        units: z.ZodDefault<z.ZodArray<z.ZodArray<z.ZodUnion<readonly [z.ZodUUID, z.ZodLiteral<"run_starter">]>>>>;
+        units: z.ZodArray<z.ZodArray<z.ZodUnion<readonly [z.ZodUUID, z.ZodLiteral<"run_starter">]>>>;
     }, z.core.$strip>;
 }, z.core.$strip>;
 export type HumanConfig = z.infer<typeof humanConfigSchema>;
@@ -199,7 +199,7 @@ export declare const graphNodeSchema: z.ZodObject<{
                 any: "any";
                 all: "all";
             }>>;
-            units: z.ZodDefault<z.ZodArray<z.ZodArray<z.ZodUnion<readonly [z.ZodUUID, z.ZodLiteral<"run_starter">]>>>>;
+            units: z.ZodArray<z.ZodArray<z.ZodUnion<readonly [z.ZodUUID, z.ZodLiteral<"run_starter">]>>>;
         }, z.core.$strip>;
     }, z.core.$strip>>;
     component: z.ZodOptional<z.ZodObject<{
@@ -298,7 +298,7 @@ export declare const saveGraphBodySchema: z.ZodObject<{
                     any: "any";
                     all: "all";
                 }>>;
-                units: z.ZodDefault<z.ZodArray<z.ZodArray<z.ZodUnion<readonly [z.ZodUUID, z.ZodLiteral<"run_starter">]>>>>;
+                units: z.ZodArray<z.ZodArray<z.ZodUnion<readonly [z.ZodUUID, z.ZodLiteral<"run_starter">]>>>;
             }, z.core.$strip>;
         }, z.core.$strip>>;
         component: z.ZodOptional<z.ZodObject<{

package/dist/_schemas/graph.js

@@ -96,7 +96,8 @@ export const RUN_STARTER = "run_starter";
 export const assignMemberSchema = z.union([z.uuid(), z.literal(RUN_STARTER)]);
 // Human ノードの kind 別設定（node_humans ＋ node_human_assignees に対応）。
 // 担当 ＝ ユニット（1人以上のメンバー）の順序付きリスト ＋ 承認モード ＋ 選出戦略。
-// units[i] はメンバーの配列（1人＝個人・複数＝グループ）。units 空 = tenant の誰でも。
+// units[i] はメンバーの配列（1人＝個人・複数＝グループ）。**担当は必ず明示する**
+// （「担当なし＝tenant の誰でも」は無い。Human担当の必須化（誰でもの廃止）.md）。
 // approval_mode はノード単位（any=誰か1人 / all=全員。差し戻しは1人で成立＝veto）。
 // メンバーが tenant の membership であることの検証は保存 API 側で行う
 // （docs/tasks/backlog/Human担当者の高度化（複数人承認とラウンドロビン）.md）
@@ -109,7 +110,9 @@ export const humanConfigSchema = z.object({
         .object({
         strategy: z.enum(["fixed", "round_robin"]).default("fixed"),
         approval_mode: z.enum(["any", "all"]).default("any"),
-        units: z.array(z.array(assignMemberSchema).min(1)).default([]),
+        units: z
+            .array(z.array(assignMemberSchema).min(1))
+            .min(1, "担当を1人以上指定してください"),
     })
         .superRefine((assignment, ctx) => {
         if (assignment.strategy === "fixed" && assignment.units.length > 1) {
@@ -335,6 +338,15 @@ export const saveGraphBodySchema = z
             }
             hookDirs.add(hook.hook_dir);
         });
+        // human は担当を必ず明示する（「担当なし＝tenant の誰でも」は無い。
+        // Human担当の必須化（誰でもの廃止）.md）
+        if (node.kind === "human" && node.human === undefined) {
+            ctx.addIssue({
+                code: "custom",
+                path: ["nodes", i, "human"],
+                message: "human ノードには担当（assignment.units）が必要です",
+            });
+        }
         // component は参照先が無いと成立しない（他 kind の設定はデフォルトで補えるが
         // component_loop_id は補えない）
         if (node.kind === "component" && node.component === undefined) {

package/package.json

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