From 586a9e05a75dd071ddaa8006cdf889d8261e2ed3 Mon Sep 17 00:00:00 2001
From: chanx <1243304602@qq.com>
Date: Tue, 10 Feb 2026 16:12:59 +0800
Subject: [PATCH] Fix: Memory log style (#13090)

### What problem does this PR solve?

Fix: Memory log style

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
---
 .../dataset-overview/overview-table.tsx       |   4 +-
 web/src/pages/dataset/process-log-modal.tsx   |   8 +-
 .../memory/memory-message/message-table.tsx   | 139 ++++++------------
 3 files changed, 57 insertions(+), 94 deletions(-)

diff --git a/web/src/pages/dataset/dataset-overview/overview-table.tsx b/web/src/pages/dataset/dataset-overview/overview-table.tsx
index a3be9ebf48..3d7dfe3b0a 100644
--- a/web/src/pages/dataset/dataset-overview/overview-table.tsx
+++ b/web/src/pages/dataset/dataset-overview/overview-table.tsx
@@ -44,7 +44,7 @@ import { ArrowUpDown, ClipboardList, Eye, MonitorUp } from 'lucide-react';
 import { FC, useMemo, useState } from 'react';
 import { useParams } from 'react-router';
 import { RunningStatus } from '../dataset/constant';
-import ProcessLogModal from '../process-log-modal';
+import ProcessLogModal, { ILogInfo } from '../process-log-modal';
 import { LogTabs, ProcessingType, ProcessingTypeMap } from './dataset-common';
 import { DocumentLog, FileLogsTableProps, IFileLogItem } from './interface';
 
@@ -474,7 +474,7 @@ const FileLogsTable: FC<FileLogsTableProps> = ({
           title={active === LogTabs.FILE_LOGS ? t('fileLogs') : t('datasetLog')}
           visible={isModalVisible}
           onCancel={() => setIsModalVisible(false)}
-          logInfo={logInfo}
+          logInfo={logInfo as unknown as ILogInfo}
         />
       )}
     </div>
diff --git a/web/src/pages/dataset/process-log-modal.tsx b/web/src/pages/dataset/process-log-modal.tsx
index 3ecc911535..03282b4d35 100644
--- a/web/src/pages/dataset/process-log-modal.tsx
+++ b/web/src/pages/dataset/process-log-modal.tsx
@@ -9,6 +9,7 @@ import {
 import { RunningStatusMap } from '@/constants/knowledge';
 import { useTranslate } from '@/hooks/common-hooks';
 import React, { useMemo } from 'react';
+import { useTranslation } from 'react-i18next';
 import reactStringReplace from 'react-string-replace';
 import { RunningStatus } from './dataset/constant';
 export interface ILogInfo {
@@ -35,6 +36,7 @@ interface ProcessLogModalProps {
   onCancel: () => void;
   logInfo: ILogInfo;
   title: string;
+  translateKey?: string;
 }
 
 const InfoItem: React.FC<{
@@ -85,8 +87,10 @@ const ProcessLogModal: React.FC<ProcessLogModalProps> = ({
   onCancel,
   logInfo: initData,
   title,
+  translateKey,
 }) => {
-  const { t } = useTranslate('knowledgeDetails');
+  const { t } = useTranslate(translateKey || 'knowledgeDetails');
+  const { t: tc } = useTranslation();
   const blackKeyList = [''];
   const logInfo = useMemo(() => {
     return initData;
@@ -99,7 +103,7 @@ const ProcessLogModal: React.FC<ProcessLogModalProps> = ({
       onCancel={onCancel}
       footer={
         <div className="flex justify-end">
-          <Button onClick={onCancel}>{t('close')}</Button>
+          <Button onClick={onCancel}>{tc('common.close')}</Button>
         </div>
       }
       className="process-log-modal"
diff --git a/web/src/pages/memory/memory-message/message-table.tsx b/web/src/pages/memory/memory-message/message-table.tsx
index ecbf413ebd..2ee6e80b85 100644
--- a/web/src/pages/memory/memory-message/message-table.tsx
+++ b/web/src/pages/memory/memory-message/message-table.tsx
@@ -5,15 +5,6 @@ import {
 import { EmptyType } from '@/components/empty/constant';
 import Empty from '@/components/empty/empty';
 import { Button } from '@/components/ui/button';
-import {
-  Dialog,
-  DialogClose,
-  DialogContent,
-  DialogFooter,
-  DialogHeader,
-  DialogTitle,
-  DialogTrigger,
-} from '@/components/ui/dialog';
 import { Modal } from '@/components/ui/modal/modal';
 import { RAGFlowPagination } from '@/components/ui/ragflow-pagination';
 import { Switch } from '@/components/ui/switch';
@@ -25,9 +16,13 @@ import {
   TableHeader,
   TableRow,
 } from '@/components/ui/table';
+import { RunningStatus } from '@/constants/knowledge';
 import { Pagination } from '@/interfaces/common';
 import { cn } from '@/lib/utils';
-import { replaceText } from '@/pages/dataset/process-log-modal';
+import ProcessLogModal, {
+  ILogInfo,
+  replaceText,
+} from '@/pages/dataset/process-log-modal';
 import { MemoryOptions } from '@/pages/memories/constants';
 import {
   ColumnDef,
@@ -72,11 +67,11 @@ const columnHelper = createColumnHelper<IMessageInfo>();
 
 function getTaskStatus(progress: number) {
   if (progress >= 1) {
-    return 'success';
+    return RunningStatus.DONE;
   } else if (progress > 0 && progress < 1) {
-    return 'running';
+    return RunningStatus.RUNNING;
   } else {
-    return 'failed';
+    return RunningStatus.FAIL;
   }
 }
 
@@ -110,6 +105,21 @@ export function MemoryTable({
   const disabledRowFunc = (row: Row<IMessageInfo>) => {
     return row.original.forget_at !== 'None' && !!row.original.forget_at;
   };
+
+  const [isModalVisible, setIsModalVisible] = useState(false);
+  const [logInfo, setLogInfo] = useState<ILogInfo>();
+  const showLog = (row: Row<IMessageInfo>) => {
+    const task = row.original.task;
+    const logDetail = {
+      startTime: dayjs(task.create_time)
+        .locale(document.documentElement.lang)
+        .format('MM/DD/YYYY HH:mm:ss'),
+      status: getTaskStatus(task.progress),
+      details: task.progress_msg,
+    } as unknown as ILogInfo;
+    setLogInfo(logDetail);
+    setIsModalVisible(true);
+  };
   // Define columns for the memory table
   const columns: ColumnDef<IMessageInfo>[] = useMemo(
     () => [
@@ -233,83 +243,22 @@ export function MemoryTable({
           const taskStatus = getTaskStatus(task.progress);
 
           return (
-            <Dialog>
-              <DialogTrigger asChild>
-                <Button
-                  variant="transparent"
-                  size="icon"
-                  className="border-0 size-8"
-                >
-                  <div
-                    className={cn('size-1 rounded-full', {
-                      'bg-state-success': taskStatus === 'success',
-                      'bg-state-error': taskStatus === 'failed',
-                      'bg-state-warning': taskStatus === 'running',
-                    })}
-                  />
-                </Button>
-              </DialogTrigger>
-
-              <DialogContent>
-                <DialogHeader>
-                  <DialogTitle>{t('memory.taskLogDialog.title')}</DialogTitle>
-                </DialogHeader>
-
-                <dl className="space-y-4">
-                  <div className="grid grid-rows-2 grid-cols-2 grid-flow-col items-center gap-x-4 gap-y-1">
-                    <dt className="text-text-secondary">
-                      {t('memory.taskLogDialog.startTime')}
-                    </dt>
-                    <dd className="text-sm">
-                      {dayjs(task.create_time)
-                        .locale(document.documentElement.lang)
-                        .format('MM/DD/YYYY HH:mm:ss')}
-                    </dd>
-
-                    <dt className="text-text-secondary">
-                      {t('memory.taskLogDialog.status')}
-                    </dt>
-                    <dd className="text-sm">
-                      <div
-                        className={cn(
-                          'inline-flex items-center gap-1 text-xs rounded-full px-2 py-1',
-                          {
-                            'text-state-success bg-state-success-5':
-                              taskStatus === 'success',
-                            'text-state-error bg-state-error-5':
-                              taskStatus === 'failed',
-                            'text-state-warning bg-state-warning-5':
-                              taskStatus === 'running',
-                          },
-                        )}
-                      >
-                        <div className="size-1 rounded-full bg-current" />
-                        {t(`memory.taskLogDialog.${taskStatus}`)}
-                      </div>
-                    </dd>
-                  </div>
-
-                  <div className="space-y-1">
-                    <dt className="text-text-secondary">
-                      {t('memory.taskLogDialog.details')}
-                    </dt>
-                    <dd className="text-sm">
-                      <div className="bg-bg-card rounded-lg p-2 max-h-64 overflow-auto">
-                        <pre>
-                          <code>{task.progress_msg}</code>
-                        </pre>
-                      </div>
-                    </dd>
-                  </div>
-                </dl>
-
-                <DialogFooter>
-                  <DialogClose asChild>
-                    <Button variant="ghost">{t('common.close')}</Button>
-                  </DialogClose>
-                </DialogFooter>
-              </DialogContent>
-            </Dialog>
+            <Button
+              variant="transparent"
+              size="icon"
+              className="border-0 size-8"
+              onClick={() => {
+                showLog(row);
+              }}
+            >
+              <div
+                className={cn('size-1 rounded-full', {
+                  'bg-state-success': taskStatus === RunningStatus.DONE,
+                  'bg-state-error': taskStatus === RunningStatus.FAIL,
+                  'bg-state-warning': taskStatus === RunningStatus.RUNNING,
+                })}
+              />
+            </Button>
           );
         },
       }),
@@ -506,6 +455,16 @@ export function MemoryTable({
         </Modal>
       )}
 
+      {isModalVisible && (
+        <ProcessLogModal
+          title={t('memory.taskLogDialog.title')}
+          visible={isModalVisible}
+          onCancel={() => setIsModalVisible(false)}
+          translateKey="memory.taskLogDialog"
+          logInfo={logInfo as unknown as ILogInfo}
+        />
+      )}
+
       <div className="flex items-center justify-end  absolute bottom-3 right-3">
         <RAGFlowPagination
           {...pick(pagination, 'current', 'pageSize')}