fix(playground): address follow-up json patch review

Author: yy-wow

sites/playground/web/src/components/genui-template/template-chat-utils/json-patch-format.ts

@@ -3,6 +3,7 @@ import type { JsonPatchOp } from 'jsondiffpatch/formatters/jsonpatch-apply';
 import { t } from '../../../i18n';
 import {
   findComponentPath,
+  getComponentItem,
   getPositionRelativePath,
   mergePath,
   resolveJsonPointerAppendSentinel,
@@ -82,6 +83,32 @@ function finalizeAbsolutePath(templeSchema: any, item: IFormattedJsonPatchOperat
   }
 }
 
+function regenerateCopiedNodeIds(templeSchema: any, item: IFormattedJsonPatchOperation): boolean {
+  if (item.op !== 'copy' || typeof item.path !== 'string') {
+    return true;
+  }
+
+  const copiedNode = getComponentItem(templeSchema, item.path).node;
+  if (!copiedNode) {
+    return false;
+  }
+
+  const resetIds = (node: any) => {
+    if (!node || typeof node !== 'object') {
+      return;
+    }
+
+    delete node.id;
+    if (Array.isArray(node.children)) {
+      node.children.forEach(resetIds);
+    }
+  };
+
+  resetIds(copiedNode);
+  generateIdForComponents(copiedNode);
+  return true;
+}
+
 export const formatJsonPatch = (
   currentSchema: any,
   value: any[],
@@ -116,6 +143,9 @@ export const formatJsonPatch = (
     }
 
     jsonPatchFormatter.patch(templeSchema, [toStandardPatchOp(item)]);
+    if (!regenerateCopiedNodeIds(templeSchema, item)) {
+      item.idToPath = null;
+    }
 
     return item;
   });

sites/playground/web/src/components/genui-template/template-chat-utils/json-patch-validate.ts

@@ -23,6 +23,22 @@ function validateOperation(operation: any): boolean {
     return false;
   }
 
+  if (
+    operation.op === 'replace' &&
+    operation.path === undefined &&
+    (
+      !operation.value ||
+      typeof operation.value !== 'object' ||
+      Array.isArray(operation.value) ||
+      typeof operation.value.componentName !== 'string' ||
+      operation.value.componentName.length === 0 ||
+      typeof operation.value.id !== 'string' ||
+      operation.value.id.length === 0
+    )
+  ) {
+    return false;
+  }
+
   return true;
 }
 

sites/playground/web/src/components/genui-template/template-chat-utils/schema-path.ts

@@ -166,7 +166,8 @@ export function getPositionRelativePath(
     }
     return `../${anchorIndex + 1}`;
   } else if (position === 'inside') {
-    const children = schema ? getComponentItem(schema, componentPath).node?.children : [];
+    const anchorNode = componentPath === '/' ? schema : getComponentItem(schema, componentPath).node;
+    const children = anchorNode?.children;
     if (!Array.isArray(children)) {
       return undefined;
     }