@lobehub/chat 1.107.5 → 1.107.6

package/CHANGELOG.md

@@ -2,6 +2,31 @@
 
 # Changelog
 
+### [Version 1.107.6](https://github.com/lobehub/lobe-chat/compare/v1.107.5...v1.107.6)
+
+<sup>Released on **2025-08-05**</sup>
+
+#### 🐛 Bug Fixes
+
+- **misc**: Break line for Gemini Artifacts.
+
+<br/>
+
+<details>
+<summary><kbd>Improvements and Fixes</kbd></summary>
+
+#### What's fixed
+
+- **misc**: Break line for Gemini Artifacts, closes [#8627](https://github.com/lobehub/lobe-chat/issues/8627) ([65609dd](https://github.com/lobehub/lobe-chat/commit/65609dd))
+
+</details>
+
+<div align="right">
+
+[![](https://img.shields.io/badge/-BACK_TO_TOP-151515?style=flat-square)](#readme-top)
+
+</div>
+
 ### [Version 1.107.5](https://github.com/lobehub/lobe-chat/compare/v1.107.4...v1.107.5)
 
 <sup>Released on **2025-08-04**</sup>

package/changelog/v1.json

@@ -1,4 +1,13 @@
 [
+  {
+    "children": {
+      "fixes": [
+        "Break line for Gemini Artifacts."
+      ]
+    },
+    "date": "2025-08-05",
+    "version": "1.107.6"
+  },
   {
     "children": {
       "improvements": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lobehub/chat",
-  "version": "1.107.5",
+  "version": "1.107.6",
   "description": "Lobe Chat - an open-source, high-performance chatbot framework that supports speech synthesis, multimodal, and extensible Function Call plugin system. Supports one-click free deployment of your private ChatGPT/LLM web application.",
   "keywords": [
     "framework",

package/src/features/Conversation/Messages/Assistant/Tool/Inspector/Debug.tsx

@@ -3,7 +3,7 @@ import { Tabs } from 'antd';
 import { memo } from 'react';
 import { useTranslation } from 'react-i18next';
 
-import PluginResult from './PluginResultJSON';
+import PluginResult from './PluginResult';
 import PluginState from './PluginState';
 
 interface DebugProps {

package/src/features/Conversation/Messages/Assistant/Tool/Inspector/{PluginResultJSON.tsx → PluginResult.tsx}

@@ -12,17 +12,22 @@ export interface FunctionMessageProps {
 const PluginResult = memo<FunctionMessageProps>(({ toolCallId, variant }) => {
   const toolMessage = useChatStore(chatSelectors.getMessageByToolCallId(toolCallId));
 
-  const data = useMemo(() => {
+  const { data, language } = useMemo(() => {
     try {
-      return JSON.stringify(JSON.parse(toolMessage?.content || ''), null, 2);
+      const parsed = JSON.parse(toolMessage?.content || '');
+      // Special case: if the parsed result is a string, it means the original content was a stringified string
+      if (typeof parsed === 'string') {
+        return { data: parsed, language: 'plaintext' }; // Return the parsed string directly, do not re-serialize
+      }
+      return { data: JSON.stringify(parsed, null, 2), language: 'json' };
     } catch {
-      return toolMessage?.content || '';
+      return { data: toolMessage?.content || '', language: 'plaintext' };
     }
   }, [toolMessage?.content]);
 
   return (
     <Highlighter
-      language={'json'}
+      language={language}
       style={{ maxHeight: 200, overflow: 'scroll', width: '100%' }}
       variant={variant}
     >

package/src/features/Conversation/Messages/Assistant/Tool/Render/CustomRender.tsx

@@ -6,7 +6,7 @@ import { memo, useCallback, useEffect, useState } from 'react';
 import { useTranslation } from 'react-i18next';
 import { Flexbox } from 'react-layout-kit';
 
-import PluginResult from '@/features/Conversation/Messages/Assistant/Tool/Inspector/PluginResultJSON';
+import PluginResult from '@/features/Conversation/Messages/Assistant/Tool/Inspector/PluginResult';
 import PluginRender from '@/features/PluginsUI/Render';
 import { useChatStore } from '@/store/chat';
 import { chatSelectors } from '@/store/chat/selectors';

package/src/features/Conversation/components/ChatItem/utils.test.ts

@@ -160,6 +160,20 @@ describe('processWithArtifact', () => {
 
     expect(output).toEqual(`<lobeThinking>这是一个思考过程。</lobeThinking>
 
+<lobeArtifact identifier="test" type="image/svg+xml" title="测试"><svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">  <rect width="100" height="100" fill="blue"/></svg></lobeArtifact>`);
+  });
+
+  it('should handle Gemini case with no line break between lobeThinking and lobeArtifact tags', () => {
+    const input = `<lobeThinking>这是一个思考过程。</lobeThinking><lobeArtifact identifier="test" type="image/svg+xml" title="测试">
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+  <rect width="100" height="100" fill="blue"/>
+</svg>
+</lobeArtifact>`;
+
+    const output = processWithArtifact(input);
+
+    expect(output).toEqual(`<lobeThinking>这是一个思考过程。</lobeThinking>
+
 <lobeArtifact identifier="test" type="image/svg+xml" title="测试"><svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">  <rect width="100" height="100" fill="blue"/></svg></lobeArtifact>`);
   });
 
@@ -431,4 +445,56 @@ This HTML document includes the temperature converter with the requested feature
 
 This HTML document includes the temperature converter with the requested features: the logic is wrapped in an IIFE, and event listeners are attached in JavaScript.`);
   });
+
+  describe('idempotency tests', () => {
+    it('should not add extra blank lines when running processWithArtifact multiple times', () => {
+      const input = `<lobeThinking>这是一个思考过程。</lobeThinking><lobeArtifact identifier="test" type="image/svg+xml" title="测试">
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+  <rect width="100" height="100" fill="blue"/>
+</svg>
+</lobeArtifact>`;
+
+      // First run
+      const firstRun = processWithArtifact(input);
+
+      // Second run - should produce the same result
+      const secondRun = processWithArtifact(firstRun);
+
+      // Third run - should still produce the same result
+      const thirdRun = processWithArtifact(secondRun);
+
+      // All runs should produce the same output
+      expect(firstRun).toEqual(secondRun);
+      expect(secondRun).toEqual(thirdRun);
+
+      // Verify the output has exactly two newlines between tags
+      expect(firstRun).toContain('</lobeThinking>\n\n<lobeArtifact');
+      expect(firstRun.match(/(<\/lobeThinking>)\n\n(<lobeArtifact)/)).toBeTruthy();
+    });
+
+    it('should handle already processed content with proper spacing', () => {
+      const alreadyProcessed = `<lobeThinking>这是一个思考过程。</lobeThinking>
+
+<lobeArtifact identifier="test" type="image/svg+xml" title="测试"><svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">  <rect width="100" height="100" fill="blue"/></svg></lobeArtifact>`;
+
+      const result = processWithArtifact(alreadyProcessed);
+
+      // Should remain unchanged
+      expect(result).toEqual(alreadyProcessed);
+    });
+
+    it('should not convert spaces between tags into extra blank lines', () => {
+      const inputWithSpaces = `<lobeThinking>这是一个思考过程。</lobeThinking> <lobeArtifact identifier="test" type="image/svg+xml" title="测试">
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+  <rect width="100" height="100" fill="blue"/>
+</svg>
+</lobeArtifact>`;
+
+      const output = processWithArtifact(inputWithSpaces);
+
+      // Should still have the space and not convert it to newlines
+      expect(output).toContain('</lobeThinking> <lobeArtifact');
+      expect(output).not.toContain('</lobeThinking>\n\n<lobeArtifact');
+    });
+  });
 });

package/src/features/Conversation/components/ChatItem/utils.ts

@@ -22,7 +22,8 @@ export const processWithArtifact = (input: string = '') => {
   }
 
   // Add empty line between lobeThinking and lobeArtifact if they are adjacent
-  output = output.replace(/(<\/lobeThinking>)\r?\n(<lobeArtifact)/, '$1\n\n$2');
+  // Support both cases: with line break (e.g. from other models) and without (e.g. from Gemini)
+  output = output.replace(/(<\/lobeThinking>)(?:\r?\n)?(<lobeArtifact)/, '$1\n\n$2');
 
   // Remove fenced code block between lobeArtifact and HTML content
   output = output.replace(

package/src/store/image/slices/createImage/action.test.ts

@@ -95,6 +95,9 @@ describe('CreateImageAction', () => {
 
       // Verify refresh was called
       expect(mockRefreshGenerationBatches).toHaveBeenCalled();
+
+      // Verify prompt is cleared after successful image creation
+      expect(result.current.parameters?.prompt).toBe('');
     });
 
     it('should create new topic when no active topic exists', async () => {
@@ -134,6 +137,9 @@ describe('CreateImageAction', () => {
         imageNum: 4,
         params: { prompt: 'test prompt', width: 1024, height: 1024 },
       });
+
+      // Verify prompt is cleared after successful image creation
+      expect(result.current.parameters?.prompt).toBe('');
     });
 
     it('should throw error when parameters is not initialized', async () => {
@@ -193,6 +199,9 @@ describe('CreateImageAction', () => {
 
       // The service should have been called before the error
       expect(mockImageService.createImage).toHaveBeenCalled();
+
+      // Verify prompt is NOT cleared when error occurs
+      expect(result.current.parameters?.prompt).toBe('test prompt');
     });
 
     it('should handle service error with new topic', async () => {
@@ -223,6 +232,37 @@ describe('CreateImageAction', () => {
       // Verify topic was created before the error
       expect(mockCreateGenerationTopic).toHaveBeenCalled();
       expect(mockSwitchGenerationTopic).toHaveBeenCalled();
+
+      // Verify prompt is NOT cleared when error occurs
+      expect(result.current.parameters?.prompt).toBe('test prompt');
+    });
+
+    it('should clear prompt input after successful image creation', async () => {
+      const mockRefreshGenerationBatches = vi.fn().mockResolvedValue(undefined);
+      const { result } = renderHook(() => useImageStore());
+
+      // Set initial prompt value
+      act(() => {
+        useImageStore.setState({
+          parameters: { prompt: 'detailed landscape artwork', width: 1024, height: 1024 },
+          refreshGenerationBatches: mockRefreshGenerationBatches,
+        });
+      });
+
+      // Verify initial prompt is set
+      expect(result.current.parameters?.prompt).toBe('detailed landscape artwork');
+
+      // Create image
+      await act(async () => {
+        await result.current.createImage();
+      });
+
+      // Verify prompt is cleared
+      expect(result.current.parameters?.prompt).toBe('');
+
+      // Verify other parameters remain unchanged
+      expect(result.current.parameters?.width).toBe(1024);
+      expect(result.current.parameters?.height).toBe(1024);
     });
   });
 

package/src/store/image/slices/createImage/action.ts

@@ -85,8 +85,17 @@ export const createCreateImageSlice: StateCreator<
       if (!isNewTopic) {
         await get().refreshGenerationBatches();
       }
+
+      // 7. Clear the prompt input after successful image creation
+      set(
+        (state) => ({
+          parameters: { ...state.parameters, prompt: '' },
+        }),
+        false,
+        'createImage/clearPrompt',
+      );
     } finally {
-      // 7. Reset all creating states
+      // 8. Reset all creating states
       if (isNewTopic) {
         set(
           { isCreating: false, isCreatingWithNewTopic: false },

package/src/tools/artifacts/systemRole.ts

@@ -27,7 +27,7 @@ The assistant can create and reference artifacts during conversations. Artifacts
 <artifact_instructions>
   When collaborating with the user on creating content that falls into compatible categories, the assistant should follow these steps:
 
-  1. Immediately before invoking an artifact, think for one sentence in <lobeThinking> tags about how it evaluates against the criteria for a good and bad artifact. Consider if the content would work just fine without an artifact. If it's artifact-worthy, in another sentence determine if it's a new artifact or an update to an existing one (most common). For updates, reuse the prior identifier.
+  1. Immediately before invoking an artifact, think for one sentence in <lobeThinking> tags about how it evaluates against the criteria for a good and bad artifact. Consider if the content would work just fine without an artifact. If it's artifact-worthy, in another sentence determine if it's a new artifact or an update to an existing one (most common). For updates, reuse the prior identifier. IMPORTANT: Always ensure there is a line break between the closing </lobeThinking> tag and the opening <lobeArtifact> tag.
   2. Wrap the content in opening and closing \`<lobeArtifact>\` tags.
   3. Assign an identifier to the \`identifier\` attribute of the opening \`<lobeArtifact>\` tag. For updates, reuse the prior identifier. For new artifacts, the identifier should be descriptive and relevant to the content, using kebab-case (e.g., "example-code-snippet"). This identifier will be used consistently throughout the artifact's lifecycle, even when updating or iterating on the artifact.
   4. Include a \`title\` attribute in the \`<lobeArtifact>\` tag to provide a brief title or description of the content.