fix(playground): update JSON patch application result handling

Refactor the JSON patch application logic by renaming the setJsonPatchApplyFailed function to setJsonPatchApplyResult, which now accepts a result parameter to indicate success or failure. This change enhances clarity in error handling and improves the management of JSON patch application states across components.

Author: yy-wow

sites/playground/web/src/components/genui-template/GenuiTemplateChat.vue

@@ -32,6 +32,7 @@ import {
   getLastUserMessage,
   isManualSchemaSaveMessage,
   resolveJsonPatchApplyFailed,
+  setJsonPatchApplyResult,
 } from './template-chat-utils';
 import { generateId } from '../../utils';
 import { useTemplateContext } from './composables';
@@ -304,8 +305,11 @@ const handleNotification = (event: INotificationPayload) => {
     || findLatestPendingSchemaCard(messages.value)?.cardId
     || '';
   const card = cardId ? findSchemaCardByCardId(messages.value, cardId) : null;
-  const applyFailed =
-    card?.type === 'json-patch' && resolveJsonPatchApplyFailed(card, messages.value);
+  let applyFailed = false;
+  if (card?.type === 'json-patch') {
+    applyFailed = resolveJsonPatchApplyFailed(card, messages.value);
+    setJsonPatchApplyResult(applyFailed ? 'failed' : 'success', messages.value, cardId);
+  }
   const preview = schema.currentPreviewSchema;
   if (preview && !applyFailed) {
     generateIdForComponents(preview);

sites/playground/web/src/components/genui-template/composables/use-template-stream-render.ts

@@ -4,7 +4,7 @@ import {
   validateJsonPatch,
   PARSE_PARTIAL_JSON_STATE,
   applyJsonPatchOperations,
-  setJsonPatchApplyFailed,
+  setJsonPatchApplyResult,
 } from '../template-chat-utils';
 import { clonePlainJson } from '../template-chat-utils/json-patch-format';
 import { stripSchemaFieldsWhileStreaming } from '../../../utils';
@@ -115,16 +115,15 @@ async function jsonPatchRenderer(props: {
     const targetSchema = applyJsonPatchOperations(patchBaseline, operations as never[]);
     if (!targetSchema) {
       if (isStreamComplete) {
-        setJsonPatchApplyFailed(messages.value, cardId, true);
+        setJsonPatchApplyResult('failed', messages.value, cardId);
       }
       return;
     }
 
-    setJsonPatchApplyFailed(messages.value, cardId, false);
     const strippedSchema = stripSchemaFieldsWhileStreaming(targetSchema, isStreamComplete);
     setCurrentPreviewSchema(strippedSchema, isStreamComplete);
   } catch (error) {
-    setJsonPatchApplyFailed(messages.value, props.cardId, true);
+    setJsonPatchApplyResult('failed', messages.value, props.cardId);
     console.error('jsonPatch error ===>', error);
   }
 }

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

@@ -125,14 +125,14 @@ export function backfillJsonPatchApplyFailedFlags(messages?: ChatMessage[]): boo
   return updated;
 }
 
-export function setJsonPatchApplyFailed(
+export function setJsonPatchApplyResult(
+  result: 'success' | 'failed',
   messages: ChatMessage[] | undefined,
   cardId: string,
-  applyFailed: boolean,
 ): void {
   const card = findSchemaCardByCardId(messages, cardId);
   if (card?.type === 'json-patch') {
-    card.applyFailed = applyFailed;
+    card.applyFailed = result === 'failed';
   }
 }