From 35c116906ed530f8055975e96f6c72ceed008f24 Mon Sep 17 00:00:00 2001
From: zxhlyh <jasonapring2015@outlook.com>
Date: Tue, 2 Sep 2025 17:30:00 +0800
Subject: [PATCH] fix node memory

---
 .../llm/components/memory-system/hooks.ts     |  4 +-
 .../memory-system/linear-memory.tsx           | 98 +++++++++----------
 2 files changed, 51 insertions(+), 51 deletions(-)

diff --git a/web/app/components/workflow/nodes/llm/components/memory-system/hooks.ts b/web/app/components/workflow/nodes/llm/components/memory-system/hooks.ts
index 9f1d1bed78..593280b531 100644
--- a/web/app/components/workflow/nodes/llm/components/memory-system/hooks.ts
+++ b/web/app/components/workflow/nodes/llm/components/memory-system/hooks.ts
@@ -14,7 +14,7 @@ export const useMemory = (
   id: string,
   data: LLMNodeType,
 ) => {
-  const { memory } = data
+  const { memory = {} as Memory } = data
   const initCollapsed = useMemo(() => {
     if (!memory?.enabled)
       return true
@@ -81,7 +81,7 @@ export const useMemory = (
         return 'block'
     }
     else {
-      if (memory?.window.enabled)
+      if (memory?.window?.enabled)
         return 'linear'
 
       return 'disabled'
diff --git a/web/app/components/workflow/nodes/llm/components/memory-system/linear-memory.tsx b/web/app/components/workflow/nodes/llm/components/memory-system/linear-memory.tsx
index f85bd1ad7e..6d1aaf4434 100644
--- a/web/app/components/workflow/nodes/llm/components/memory-system/linear-memory.tsx
+++ b/web/app/components/workflow/nodes/llm/components/memory-system/linear-memory.tsx
@@ -122,59 +122,59 @@ const LinearMemory = ({
 
   return (
     <>
-    <div className={cn('flex justify-between', className)}>
-      <div className='flex h-8 items-center space-x-2'>
-        <Switch
-          defaultValue={payload?.window?.enabled}
-          onChange={handleWindowEnabledChange}
-          size='md'
-          disabled={readonly}
-        />
-        <div className='system-xs-medium-uppercase text-text-tertiary'>{t(`${i18nPrefix}.windowSize`)}</div>
-      </div>
-      <div className='flex h-8 items-center space-x-2'>
-        <Slider
-          className='w-[144px]'
-          value={(payload.window?.size || WINDOW_SIZE_DEFAULT) as number}
-          min={WINDOW_SIZE_MIN}
-          max={WINDOW_SIZE_MAX}
-          step={1}
-          onChange={handleWindowSizeChange}
-          disabled={readonly || !payload.window?.enabled}
-        />
-        <Input
-          value={(payload.window?.size || WINDOW_SIZE_DEFAULT) as number}
-          wrapperClassName='w-12'
-          className='appearance-none pr-0'
-          type='number'
-          min={WINDOW_SIZE_MIN}
-          max={WINDOW_SIZE_MAX}
-          step={1}
-          onChange={e => handleWindowSizeChange(e.target.value)}
-          onBlur={handleBlur}
-          disabled={readonly || !payload.window?.enabled}
-        />
-      </div>
-    </div>
-    {canSetRoleName && (
-      <div className='mt-4'>
-        <div className='text-xs font-medium uppercase leading-6 text-text-tertiary'>{t(`${i18nPrefix}.conversationRoleName`)}</div>
-        <div className='mt-1 space-y-2'>
-          <RoleItem
-            readonly={!!readonly}
-            title={t(`${i18nPrefix}.user`)}
-            value={payload.role_prefix?.user || ''}
-            onChange={handleRolePrefixChange(MemoryRole.user)}
+      <div className={cn('flex justify-between', className)}>
+        <div className='flex h-8 items-center space-x-2'>
+          <Switch
+            defaultValue={payload?.window?.enabled}
+            onChange={handleWindowEnabledChange}
+            size='md'
+            disabled={readonly}
           />
-          <RoleItem
-            readonly={!!readonly}
-            title={t(`${i18nPrefix}.assistant`)}
-            value={payload.role_prefix?.assistant || ''}
-            onChange={handleRolePrefixChange(MemoryRole.assistant)}
+          <div className='system-xs-medium-uppercase text-text-tertiary'>{t(`${i18nPrefix}.windowSize`)}</div>
+        </div>
+        <div className='flex h-8 items-center space-x-2'>
+          <Slider
+            className='w-[144px]'
+            value={(payload.window?.size || WINDOW_SIZE_DEFAULT) as number}
+            min={WINDOW_SIZE_MIN}
+            max={WINDOW_SIZE_MAX}
+            step={1}
+            onChange={handleWindowSizeChange}
+            disabled={readonly || !payload.window?.enabled}
+          />
+          <Input
+            value={(payload.window?.size || WINDOW_SIZE_DEFAULT) as number}
+            wrapperClassName='w-12'
+            className='appearance-none pr-0'
+            type='number'
+            min={WINDOW_SIZE_MIN}
+            max={WINDOW_SIZE_MAX}
+            step={1}
+            onChange={e => handleWindowSizeChange(e.target.value)}
+            onBlur={handleBlur}
+            disabled={readonly || !payload.window?.enabled}
           />
         </div>
       </div>
-    )}
+      {canSetRoleName && (
+        <div className='mt-4'>
+          <div className='text-xs font-medium uppercase leading-6 text-text-tertiary'>{t(`${i18nPrefix}.conversationRoleName`)}</div>
+          <div className='mt-1 space-y-2'>
+            <RoleItem
+              readonly={!!readonly}
+              title={t(`${i18nPrefix}.user`)}
+              value={payload.role_prefix?.user || ''}
+              onChange={handleRolePrefixChange(MemoryRole.user)}
+            />
+            <RoleItem
+              readonly={!!readonly}
+              title={t(`${i18nPrefix}.assistant`)}
+              value={payload.role_prefix?.assistant || ''}
+              onChange={handleRolePrefixChange(MemoryRole.assistant)}
+            />
+          </div>
+        </div>
+      )}
     </>
   )
 }