@lvce-editor/chat-view 2.8.0 → 2.9.0

package/dist/chatViewWorkerMain.js

@@ -2649,7 +2649,8 @@ const deleteSession = async (state, id) => {
 };
 
 const handleClickOpenApiApiKeySettings = async state => {
-  await openExternal(state.openApiApiKeysSettingsUrl);
+  // Open the built-in settings editor so the user can inspect or edit their OpenAI API key.
+  await invoke('Main.openUri', 'app://settings.json');
   return state;
 };
 
@@ -3053,8 +3054,6 @@ const getMockOpenRouterAssistantText = async (messages, modelId, openRouterApiBa
   }
 };
 
-const OnFileSystem = 'onFileSystem';
-
 const isPathTraversalAttempt = path => {
   if (!path) {
     return false;
@@ -3071,6 +3070,7 @@ const isPathTraversalAttempt = path => {
   const segments = path.split(/[\\/]/);
   return segments.includes('..');
 };
+
 const normalizeRelativePath = path => {
   const segments = path.split(/[\\/]/).filter(segment => segment && segment !== '.');
   if (segments.length === 0) {
@@ -3078,6 +3078,87 @@ const normalizeRelativePath = path => {
   }
   return segments.join('/');
 };
+
+const executeListFilesTool = async (args, _options) => {
+  const folderPath = typeof args.path === 'string' && args.path ? args.path : '.';
+  if (isPathTraversalAttempt(folderPath)) {
+    return JSON.stringify({
+      error: 'Access denied: path must be relative and stay within the open workspace folder.'
+    });
+  }
+  const normalizedPath = normalizeRelativePath(folderPath);
+  try {
+    const entries = await invoke('FileSystem.readDirWithFileTypes', normalizedPath);
+    return JSON.stringify({
+      entries,
+      path: normalizedPath
+    });
+  } catch (error) {
+    return JSON.stringify({
+      error: String(error),
+      path: normalizedPath
+    });
+  }
+};
+
+const executeReadFileTool = async (args, _options) => {
+  const filePath = typeof args.path === 'string' ? args.path : '';
+  if (!filePath || isPathTraversalAttempt(filePath)) {
+    return JSON.stringify({
+      error: 'Access denied: path must be relative and stay within the open workspace folder.'
+    });
+  }
+  const normalizedPath = normalizeRelativePath(filePath);
+  try {
+    const content = await readFile(normalizedPath);
+    return JSON.stringify({
+      content,
+      path: normalizedPath
+    });
+  } catch (error) {
+    return JSON.stringify({
+      error: String(error),
+      path: normalizedPath
+    });
+  }
+};
+
+const OnFileSystem = 'onFileSystem';
+
+const executeFileSystemCommand = async (method, params, options) => {
+  return executeProvider({
+    assetDir: options.assetDir,
+    event: OnFileSystem,
+    method,
+    noProviderFoundMessage: 'No file system provider found',
+    params,
+    platform: options.platform
+  });
+};
+
+const executeWriteFileTool = async (args, options) => {
+  const filePath = typeof args.path === 'string' ? args.path : '';
+  const content = typeof args.content === 'string' ? args.content : '';
+  if (!filePath || isPathTraversalAttempt(filePath)) {
+    return JSON.stringify({
+      error: 'Access denied: path must be relative and stay within the open workspace folder.'
+    });
+  }
+  const normalizedPath = normalizeRelativePath(filePath);
+  try {
+    await executeFileSystemCommand(FileSystemWriteFile, ['file', normalizedPath, content], options);
+    return JSON.stringify({
+      ok: true,
+      path: normalizedPath
+    });
+  } catch (error) {
+    return JSON.stringify({
+      error: String(error),
+      path: normalizedPath
+    });
+  }
+};
+
 const parseToolArguments = rawArguments => {
   if (typeof rawArguments !== 'string') {
     return {};
@@ -3092,16 +3173,23 @@ const parseToolArguments = rawArguments => {
     return {};
   }
 };
-const executeFileSystemCommand = async (method, params, options) => {
-  return executeProvider({
-    assetDir: options.assetDir,
-    event: OnFileSystem,
-    method,
-    noProviderFoundMessage: 'No file system provider found',
-    params,
-    platform: options.platform
+
+const executeChatTool = async (name, rawArguments, options) => {
+  const args = parseToolArguments(rawArguments);
+  if (name === 'read_file') {
+    return executeReadFileTool(args);
+  }
+  if (name === 'write_file') {
+    return executeWriteFileTool(args, options);
+  }
+  if (name === 'list_files') {
+    return executeListFilesTool(args);
+  }
+  return JSON.stringify({
+    error: `Unknown tool: ${name}`
   });
 };
+
 const getBasicChatTools = () => {
   return [{
     function: {
@@ -3159,76 +3247,6 @@ const getBasicChatTools = () => {
     type: 'function'
   }];
 };
-const executeChatTool = async (name, rawArguments, options) => {
-  const args = parseToolArguments(rawArguments);
-  if (name === 'read_file') {
-    const filePath = typeof args.path === 'string' ? args.path : '';
-    if (!filePath || isPathTraversalAttempt(filePath)) {
-      return JSON.stringify({
-        error: 'Access denied: path must be relative and stay within the open workspace folder.'
-      });
-    }
-    const normalizedPath = normalizeRelativePath(filePath);
-    try {
-      const content = await readFile(normalizedPath);
-      return JSON.stringify({
-        content,
-        path: normalizedPath
-      });
-    } catch (error) {
-      return JSON.stringify({
-        error: String(error),
-        path: normalizedPath
-      });
-    }
-  }
-  if (name === 'write_file') {
-    const filePath = typeof args.path === 'string' ? args.path : '';
-    const content = typeof args.content === 'string' ? args.content : '';
-    if (!filePath || isPathTraversalAttempt(filePath)) {
-      return JSON.stringify({
-        error: 'Access denied: path must be relative and stay within the open workspace folder.'
-      });
-    }
-    const normalizedPath = normalizeRelativePath(filePath);
-    try {
-      await executeFileSystemCommand(FileSystemWriteFile, ['file', normalizedPath, content], options);
-      return JSON.stringify({
-        ok: true,
-        path: normalizedPath
-      });
-    } catch (error) {
-      return JSON.stringify({
-        error: String(error),
-        path: normalizedPath
-      });
-    }
-  }
-  if (name === 'list_files') {
-    const folderPath = typeof args.path === 'string' && args.path ? args.path : '.';
-    if (isPathTraversalAttempt(folderPath)) {
-      return JSON.stringify({
-        error: 'Access denied: path must be relative and stay within the open workspace folder.'
-      });
-    }
-    const normalizedPath = normalizeRelativePath(folderPath);
-    try {
-      const entries = await invoke('FileSystem.readDirWithFileTypes', normalizedPath);
-      return JSON.stringify({
-        entries,
-        path: normalizedPath
-      });
-    } catch (error) {
-      return JSON.stringify({
-        error: String(error),
-        path: normalizedPath
-      });
-    }
-  }
-  return JSON.stringify({
-    error: `Unknown tool: ${name}`
-  });
-};
 
 const getClientRequestIdHeader = () => {
   return {
@@ -4049,6 +4067,12 @@ const getOpenApiErrorMessage = errorResult => {
         const errorMessage = errorResult.errorMessage?.trim();
         const hasErrorCode = typeof errorResult.errorCode === 'string' && errorResult.errorCode.length > 0;
         const hasErrorType = typeof errorResult.errorType === 'string' && errorResult.errorType.length > 0;
+
+        // Provide a concise, user-friendly message when OpenAI reports an invalid API key.
+        if (errorResult.errorCode === 'invalid_api_key') {
+          const status = typeof errorResult.statusCode === 'number' ? errorResult.statusCode : 401;
+          return `OpenAI request failed (status ${status}): Invalid API key. Please verify your OpenAI API key in Chat settings.`;
+        }
         if (errorResult.statusCode === 429) {
           let prefix = 'OpenAI rate limit exceeded (429)';
           if (hasErrorCode) {
@@ -5970,6 +5994,31 @@ const getChatHeaderDomDetailMode = selectedSessionTitle => {
   }, text(selectedSessionTitle), ...getChatHeaderActionsDom()];
 };
 
+const getMessageNodeDom = node => {
+  if (node.type === 'text') {
+    return [{
+      childCount: 1,
+      className: Markdown,
+      type: P
+    }, text(node.text)];
+  }
+  return [{
+    childCount: node.items.length,
+    className: ChatOrderedList,
+    type: Ol
+  }, ...node.items.flatMap(item => {
+    return [{
+      childCount: 1,
+      className: ChatOrderedListItem,
+      type: Li
+    }, text(item.text)];
+  })];
+};
+
+const getMessageContentDom = nodes => {
+  return nodes.flatMap(getMessageNodeDom);
+};
+
 const getMissingApiKeyDom = ({
   getApiKeyText,
   inputName,
@@ -6272,28 +6321,6 @@ const parseMessageContent = rawMessage => {
     type: 'text'
   }] : nodes;
 };
-const getMessageContentDom = nodes => {
-  return nodes.flatMap(node => {
-    if (node.type === 'text') {
-      return [{
-        childCount: 1,
-        className: Markdown,
-        type: P
-      }, text(node.text)];
-    }
-    return [{
-      childCount: node.items.length,
-      className: ChatOrderedList,
-      type: Ol
-    }, ...node.items.flatMap(item => {
-      return [{
-        childCount: 1,
-        className: ChatOrderedListItem,
-        type: Li
-      }, text(item.text)];
-    })];
-  });
-};
 
 const getChatMessageDom = (message, openRouterApiKeyInput, openApiApiKeyInput = '', openRouterApiKeyState = 'idle') => {
   const roleClassName = message.role === 'user' ? MessageUser : MessageAssistant;
@@ -6630,30 +6657,22 @@ const saveState = state => {
   const {
     chatListScrollTop,
     composerValue,
-    height,
     messagesScrollTop,
     nextMessageId,
     renamingSessionId,
     selectedModelId,
     selectedSessionId,
-    viewMode,
-    width,
-    x,
-    y
+    viewMode
   } = state;
   return {
     chatListScrollTop,
     composerValue,
-    height,
     messagesScrollTop,
     nextMessageId,
     renamingSessionId,
     selectedModelId,
     selectedSessionId,
-    viewMode,
-    width,
-    x,
-    y
+    viewMode
   };
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/chat-view",
-  "version": "2.8.0",
+  "version": "2.9.0",
   "description": "Chat View Worker",
   "repository": {
     "type": "git",