@notebook-intelligence/notebook-intelligence 2.4.1 → 2.5.0

package/src/chat-sidebar.tsx

@@ -46,24 +46,15 @@ import {
   VscEyeClosed,
   VscTriangleRight,
   VscTriangleDown,
-  VscWarning,
   VscSettingsGear,
   VscPassFilled,
   VscTools,
   VscTrash
 } from 'react-icons/vsc';
 
-import { MdOutlineCheckBoxOutlineBlank, MdCheckBox } from 'react-icons/md';
-
 import { extractLLMGeneratedCode, isDarkTheme } from './utils';
-import * as path from 'path';
-
-const OPENAI_COMPATIBLE_CHAT_MODEL_ID = 'openai-compatible-chat-model';
-const LITELLM_COMPATIBLE_CHAT_MODEL_ID = 'litellm-compatible-chat-model';
-const OPENAI_COMPATIBLE_INLINE_COMPLETION_MODEL_ID =
-  'openai-compatible-inline-completion-model';
-const LITELLM_COMPATIBLE_INLINE_COMPLETION_MODEL_ID =
-  'litellm-compatible-inline-completion-model';
+import { CheckBoxItem } from './components/checkbox';
+import { mcpServerSettingsToEnabledState } from './components/mcp-util';
 
 export enum RunChatCompletionType {
   Chat,
@@ -264,30 +255,6 @@ export class GitHubCopilotLoginDialogBody extends ReactWidget {
   private _onLoggedIn: () => void;
 }
 
-export class ConfigurationDialogBody extends ReactWidget {
-  constructor(options: {
-    onSave: () => void;
-    onEditMCPConfigClicked: () => void;
-  }) {
-    super();
-
-    this._onEditMCPConfigClicked = options.onEditMCPConfigClicked;
-    this._onSave = options.onSave;
-  }
-
-  render(): JSX.Element {
-    return (
-      <ConfigurationDialogBodyComponent
-        onEditMCPConfigClicked={this._onEditMCPConfigClicked}
-        onSave={this._onSave}
-      />
-    );
-  }
-
-  private _onEditMCPConfigClicked: () => void;
-  private _onSave: () => void;
-}
-
 interface IChatMessageContent {
   id: string;
   type: ResponseStreamDataType;
@@ -684,30 +651,6 @@ async function submitCompletionRequest(
   }
 }
 
-function CheckBoxItem(props: any) {
-  const indent = props.indent || 0;
-
-  return (
-    <div
-      className={`checkbox-item checkbox-item-indent-${indent} ${props.header ? 'checkbox-item-header' : ''}`}
-      title={props.title}
-      onClick={event => props.onClick(event)}
-    >
-      <div className="checkbox-item-toggle">
-        {props.checked ? (
-          <MdCheckBox className="checkbox-icon" />
-        ) : (
-          <MdOutlineCheckBoxOutlineBlank className="checkbox-icon" />
-        )}
-        {props.label}
-      </div>
-      {props.title && (
-        <div className="checkbox-item-description">{props.title}</div>
-      )}
-    </div>
-  );
-}
-
 function SidebarComponent(props: any) {
   const [chatMessages, setChatMessages] = useState<IChatMessage[]>([]);
   const [prompt, setPrompt] = useState<string>('');
@@ -743,7 +686,9 @@ function SidebarComponent(props: any) {
   const [selectedToolCount, setSelectedToolCount] = useState(0);
   const [notebookExecuteToolSelected, setNotebookExecuteToolSelected] =
     useState(false);
-  const [toolConfig, setToolConfig] = useState({
+
+  const [renderCount, setRenderCount] = useState(1);
+  const toolConfigRef = useRef({
     builtinToolsets: [
       { id: BuiltinToolsetType.NotebookEdit, name: 'Notebook edit' },
       { id: BuiltinToolsetType.NotebookExecute, name: 'Notebook execute' }
@@ -751,6 +696,16 @@ function SidebarComponent(props: any) {
     mcpServers: [],
     extensions: []
   });
+  const mcpServerSettingsRef = useRef(NBIAPI.config.mcpServerSettings);
+  const [mcpServerEnabledState, setMCPServerEnabledState] = useState(
+    new Map<string, Set<string>>(
+      mcpServerSettingsToEnabledState(
+        toolConfigRef.current.mcpServers,
+        mcpServerSettingsRef.current
+      )
+    )
+  );
+
   const [showModeTools, setShowModeTools] = useState(false);
   const toolSelectionsInitial: any = {
     builtinToolsets: [BuiltinToolsetType.NotebookEdit],
@@ -762,25 +717,37 @@ function SidebarComponent(props: any) {
     mcpServers: {},
     extensions: {}
   };
-  const [toolSelections, setToolSelections] = useState(toolSelectionsInitial);
+  const [toolSelections, setToolSelections] = useState(
+    structuredClone(toolSelectionsInitial)
+  );
   const [hasExtensionTools, setHasExtensionTools] = useState(false);
   const [lastScrollTime, setLastScrollTime] = useState(0);
   const [scrollPending, setScrollPending] = useState(false);
 
-  NBIAPI.configChanged.connect(() => {
-    setToolConfig(NBIAPI.config.toolConfig);
-  });
+  useEffect(() => {
+    NBIAPI.configChanged.connect(() => {
+      toolConfigRef.current = NBIAPI.config.toolConfig;
+      mcpServerSettingsRef.current = NBIAPI.config.mcpServerSettings;
+      const newMcpServerEnabledState = mcpServerSettingsToEnabledState(
+        toolConfigRef.current.mcpServers,
+        mcpServerSettingsRef.current
+      );
+      setMCPServerEnabledState(newMcpServerEnabledState);
+      setToolSelections(structuredClone(toolSelectionsInitial));
+      setRenderCount(renderCount => renderCount + 1);
+    });
+  }, []);
 
   useEffect(() => {
     let hasTools = false;
-    for (const extension of toolConfig.extensions) {
+    for (const extension of toolConfigRef.current.extensions) {
       if (extension.toolsets.length > 0) {
         hasTools = true;
         break;
       }
     }
     setHasExtensionTools(hasTools);
-  }, [toolConfig]);
+  }, [toolConfigRef.current]);
 
   useEffect(() => {
     const builtinToolSelCount = toolSelections.builtinToolsets.length;
@@ -862,7 +829,9 @@ function SidebarComponent(props: any) {
       return false;
     }
 
-    const mcpServer = toolConfig.mcpServers.find(server => server.id === id);
+    const mcpServer = toolConfigRef.current.mcpServers.find(
+      server => server.id === id
+    );
 
     const selectedServerTools: string[] = toolSelections.mcpServers[id];
 
@@ -881,10 +850,16 @@ function SidebarComponent(props: any) {
       delete newConfig.mcpServers[id];
       setToolSelections(newConfig);
     } else {
-      const mcpServer = toolConfig.mcpServers.find(server => server.id === id);
+      const mcpServer = toolConfigRef.current.mcpServers.find(
+        server => server.id === id
+      );
       const newConfig = { ...toolSelections };
       newConfig.mcpServers[id] = structuredClone(
-        mcpServer.tools.map((tool: any) => tool.name)
+        mcpServer.tools
+          .filter((tool: any) =>
+            mcpServerEnabledState.get(mcpServer.id).has(tool.name)
+          )
+          .map((tool: any) => tool.name)
       );
       setToolSelections(newConfig);
     }
@@ -931,7 +906,7 @@ function SidebarComponent(props: any) {
       return false;
     }
 
-    const extension = toolConfig.extensions.find(
+    const extension = toolConfigRef.current.extensions.find(
       extension => extension.id === extensionId
     );
 
@@ -956,7 +931,9 @@ function SidebarComponent(props: any) {
       return false;
     }
 
-    const extension = toolConfig.extensions.find(ext => ext.id === extensionId);
+    const extension = toolConfigRef.current.extensions.find(
+      ext => ext.id === extensionId
+    );
     const extensionToolset = extension.toolsets.find(
       (toolset: any) => toolset.id === toolsetId
     );
@@ -988,7 +965,7 @@ function SidebarComponent(props: any) {
       setToolSelections(newConfig);
     } else {
       const newConfig = { ...toolSelections };
-      const extension = toolConfig.extensions.find(
+      const extension = toolConfigRef.current.extensions.find(
         ext => ext.id === extensionId
       );
       if (extensionId in newConfig.extensions) {
@@ -1030,7 +1007,7 @@ function SidebarComponent(props: any) {
       }
       setToolSelections(newConfig);
     } else {
-      const extension = toolConfig.extensions.find(
+      const extension = toolConfigRef.current.extensions.find(
         ext => ext.id === extensionId
       );
       const extensionToolset = extension.toolsets.find(
@@ -1194,7 +1171,16 @@ function SidebarComponent(props: any) {
 
   const handleChatToolsButtonClick = async () => {
     if (!showModeTools) {
-      NBIAPI.fetchCapabilities();
+      NBIAPI.fetchCapabilities().then(() => {
+        toolConfigRef.current = NBIAPI.config.toolConfig;
+        mcpServerSettingsRef.current = NBIAPI.config.mcpServerSettings;
+        const newMcpServerEnabledState = mcpServerSettingsToEnabledState(
+          toolConfigRef.current.mcpServers,
+          mcpServerSettingsRef.current
+        );
+        setMCPServerEnabledState(newMcpServerEnabledState);
+        setRenderCount(renderCount => renderCount + 1);
+      });
     }
     setShowModeTools(!showModeTools);
   };
@@ -1729,10 +1715,12 @@ function SidebarComponent(props: any) {
   const [ghLoginRequired, setGHLoginRequired] = useState(getGHLoginRequired());
   const [chatEnabled, setChatEnabled] = useState(getChatEnabled());
 
-  NBIAPI.configChanged.connect(() => {
-    setGHLoginRequired(getGHLoginRequired());
-    setChatEnabled(getChatEnabled());
-  });
+  useEffect(() => {
+    NBIAPI.configChanged.connect(() => {
+      setGHLoginRequired(getGHLoginRequired());
+      setChatEnabled(getChatEnabled());
+    });
+  }, []);
 
   useEffect(() => {
     setGHLoginRequired(getGHLoginRequired());
@@ -1876,7 +1864,7 @@ function SidebarComponent(props: any) {
                     if (event.target.value === 'ask') {
                       setToolSelections(toolSelectionsEmpty);
                     } else if (event.target.value === 'agent') {
-                      setToolSelections(toolSelectionsInitial);
+                      setToolSelections(structuredClone(toolSelectionsInitial));
                     }
                     setShowModeTools(false);
                     setChatMode(event.target.value);
@@ -1980,7 +1968,7 @@ function SidebarComponent(props: any) {
               <div className="mode-tools-popover-tool-list">
                 <div className="mode-tools-group-header">Built-in</div>
                 <div className="mode-tools-group mode-tools-group-built-in">
-                  {toolConfig.builtinToolsets.map((toolset: any) => (
+                  {toolConfigRef.current.builtinToolsets.map((toolset: any) => (
                     <CheckBoxItem
                       key={toolset.id}
                       label={toolset.name}
@@ -1995,87 +1983,111 @@ function SidebarComponent(props: any) {
                     />
                   ))}
                 </div>
-                {toolConfig.mcpServers.length > 0 && (
-                  <div className="mode-tools-group-header">MCP Servers</div>
-                )}
-                {toolConfig.mcpServers.map((mcpServer, index: number) => (
-                  <div className="mode-tools-group">
-                    <CheckBoxItem
-                      label={mcpServer.id}
-                      header={true}
-                      checked={getMCPServerState(mcpServer.id)}
-                      onClick={() => onMCPServerClicked(mcpServer.id)}
-                    />
-                    {mcpServer.tools.map((tool: any, index: number) => (
-                      <CheckBoxItem
-                        label={tool.name}
-                        title={tool.description}
-                        indent={1}
-                        checked={getMCPServerToolState(mcpServer.id, tool.name)}
-                        onClick={() =>
-                          setMCPServerToolState(
-                            mcpServer.id,
-                            tool.name,
-                            !getMCPServerToolState(mcpServer.id, tool.name)
+                {renderCount > 0 &&
+                  mcpServerEnabledState.size > 0 &&
+                  toolConfigRef.current.mcpServers.length > 0 && (
+                    <div className="mode-tools-group-header">MCP Servers</div>
+                  )}
+                {renderCount > 0 &&
+                  toolConfigRef.current.mcpServers
+                    .filter(mcpServer =>
+                      mcpServerEnabledState.has(mcpServer.id)
+                    )
+                    .map((mcpServer, index: number) => (
+                      <div className="mode-tools-group">
+                        <CheckBoxItem
+                          label={mcpServer.id}
+                          header={true}
+                          checked={getMCPServerState(mcpServer.id)}
+                          onClick={() => onMCPServerClicked(mcpServer.id)}
+                        />
+                        {mcpServer.tools
+                          .filter((tool: any) =>
+                            mcpServerEnabledState
+                              .get(mcpServer.id)
+                              .has(tool.name)
                           )
-                        }
-                      />
+                          .map((tool: any, index: number) => (
+                            <CheckBoxItem
+                              label={tool.name}
+                              title={tool.description}
+                              indent={1}
+                              checked={getMCPServerToolState(
+                                mcpServer.id,
+                                tool.name
+                              )}
+                              onClick={() =>
+                                setMCPServerToolState(
+                                  mcpServer.id,
+                                  tool.name,
+                                  !getMCPServerToolState(
+                                    mcpServer.id,
+                                    tool.name
+                                  )
+                                )
+                              }
+                            />
+                          ))}
+                      </div>
                     ))}
-                  </div>
-                ))}
                 {hasExtensionTools && (
                   <div className="mode-tools-group-header">Extension tools</div>
                 )}
-                {toolConfig.extensions.map((extension, index: number) => (
-                  <div className="mode-tools-group">
-                    <CheckBoxItem
-                      label={`${extension.name} (${extension.id})`}
-                      header={true}
-                      checked={getExtensionState(extension.id)}
-                      onClick={() => onExtensionClicked(extension.id)}
-                    />
-                    {extension.toolsets.map((toolset: any, index: number) => (
-                      <>
-                        <CheckBoxItem
-                          label={`${toolset.name} (${toolset.id})`}
-                          title={toolset.description}
-                          indent={1}
-                          checked={getExtensionToolsetState(
-                            extension.id,
-                            toolset.id
-                          )}
-                          onClick={() =>
-                            onExtensionToolsetClicked(extension.id, toolset.id)
-                          }
-                        />
-                        {toolset.tools.map((tool: any, index: number) => (
+                {toolConfigRef.current.extensions.map(
+                  (extension, index: number) => (
+                    <div className="mode-tools-group">
+                      <CheckBoxItem
+                        label={`${extension.name} (${extension.id})`}
+                        header={true}
+                        checked={getExtensionState(extension.id)}
+                        onClick={() => onExtensionClicked(extension.id)}
+                      />
+                      {extension.toolsets.map((toolset: any, index: number) => (
+                        <>
                           <CheckBoxItem
-                            label={tool.name}
-                            title={tool.description}
-                            indent={2}
-                            checked={getExtensionToolsetToolState(
+                            label={`${toolset.name} (${toolset.id})`}
+                            title={toolset.description}
+                            indent={1}
+                            checked={getExtensionToolsetState(
                               extension.id,
-                              toolset.id,
-                              tool.name
+                              toolset.id
                             )}
                             onClick={() =>
-                              setExtensionToolsetToolState(
+                              onExtensionToolsetClicked(
+                                extension.id,
+                                toolset.id
+                              )
+                            }
+                          />
+                          {toolset.tools.map((tool: any, index: number) => (
+                            <CheckBoxItem
+                              label={tool.name}
+                              title={tool.description}
+                              indent={2}
+                              checked={getExtensionToolsetToolState(
                                 extension.id,
                                 toolset.id,
-                                tool.name,
-                                !getExtensionToolsetToolState(
+                                tool.name
+                              )}
+                              onClick={() =>
+                                setExtensionToolsetToolState(
                                   extension.id,
                                   toolset.id,
-                                  tool.name
+                                  tool.name,
+                                  !getExtensionToolsetToolState(
+                                    extension.id,
+                                    toolset.id,
+                                    tool.name
+                                  )
                                 )
-                              )
-                            }
-                          />
-                        ))}
-                      </>
-                    ))}
-                  </div>
-                ))}
+                              }
+                            />
+                          ))}
+                        </>
+                      ))}
+                    </div>
+                  )
+                )}
               </div>
             </div>
           )}
@@ -2497,527 +2509,3 @@ function GitHubCopilotLoginDialogBodyComponent(props: any) {
     </div>
   );
 }
-
-function ConfigurationDialogBodyComponent(props: any) {
-  const nbiConfig = NBIAPI.config;
-  const llmProviders = nbiConfig.llmProviders;
-  const [chatModels, setChatModels] = useState([]);
-  const [inlineCompletionModels, setInlineCompletionModels] = useState([]);
-  const [mcpServerNames, setMcpServerNames] = useState(
-    nbiConfig.toolConfig.mcpServers?.map((server: any) => server.id) || []
-  );
-
-  const handleSaveClick = async () => {
-    const config: any = {
-      default_chat_mode: defaultChatMode,
-      chat_model: {
-        provider: chatModelProvider,
-        model: chatModel,
-        properties: chatModelProperties
-      },
-      inline_completion_model: {
-        provider: inlineCompletionModelProvider,
-        model: inlineCompletionModel,
-        properties: inlineCompletionModelProperties
-      }
-    };
-
-    if (
-      chatModelProvider === 'github-copilot' ||
-      inlineCompletionModelProvider === 'github-copilot'
-    ) {
-      config.store_github_access_token = storeGitHubAccessToken;
-    }
-
-    await NBIAPI.setConfig(config);
-
-    props.onSave();
-  };
-
-  const handleRefreshOllamaModelListClick = async () => {
-    await NBIAPI.updateOllamaModelList();
-    updateModelOptionsForProvider(chatModelProvider, 'chat');
-  };
-
-  const [chatModelProvider, setChatModelProvider] = useState(
-    nbiConfig.chatModel.provider || 'none'
-  );
-  const [inlineCompletionModelProvider, setInlineCompletionModelProvider] =
-    useState(nbiConfig.inlineCompletionModel.provider || 'none');
-  const [defaultChatMode, setDefaultChatMode] = useState<string>(
-    nbiConfig.defaultChatMode
-  );
-  const [chatModel, setChatModel] = useState<string>(nbiConfig.chatModel.model);
-  const [chatModelProperties, setChatModelProperties] = useState<any[]>([]);
-  const [inlineCompletionModelProperties, setInlineCompletionModelProperties] =
-    useState<any[]>([]);
-  const [inlineCompletionModel, setInlineCompletionModel] = useState(
-    nbiConfig.inlineCompletionModel.model
-  );
-  const [storeGitHubAccessToken, setStoreGitHubAccessToken] = useState(
-    nbiConfig.storeGitHubAccessToken
-  );
-
-  const updateModelOptionsForProvider = (
-    providerId: string,
-    modelType: 'chat' | 'inline-completion'
-  ) => {
-    if (modelType === 'chat') {
-      setChatModelProvider(providerId);
-    } else {
-      setInlineCompletionModelProvider(providerId);
-    }
-    const models =
-      modelType === 'chat'
-        ? nbiConfig.chatModels
-        : nbiConfig.inlineCompletionModels;
-    const selectedModelId =
-      modelType === 'chat'
-        ? nbiConfig.chatModel.model
-        : nbiConfig.inlineCompletionModel.model;
-
-    const providerModels = models.filter(
-      (model: any) => model.provider === providerId
-    );
-    if (modelType === 'chat') {
-      setChatModels(providerModels);
-    } else {
-      setInlineCompletionModels(providerModels);
-    }
-    let selectedModel = providerModels.find(
-      (model: any) => model.id === selectedModelId
-    );
-    if (!selectedModel) {
-      selectedModel = providerModels?.[0];
-    }
-    if (selectedModel) {
-      if (modelType === 'chat') {
-        setChatModel(selectedModel.id);
-        setChatModelProperties(selectedModel.properties);
-      } else {
-        setInlineCompletionModel(selectedModel.id);
-        setInlineCompletionModelProperties(selectedModel.properties);
-      }
-    } else {
-      if (modelType === 'chat') {
-        setChatModelProperties([]);
-      } else {
-        setInlineCompletionModelProperties([]);
-      }
-    }
-  };
-
-  const onModelPropertyChange = (
-    modelType: 'chat' | 'inline-completion',
-    propertyId: string,
-    value: string
-  ) => {
-    const modelProperties =
-      modelType === 'chat'
-        ? chatModelProperties
-        : inlineCompletionModelProperties;
-    const updatedProperties = modelProperties.map((property: any) => {
-      if (property.id === propertyId) {
-        return { ...property, value };
-      }
-      return property;
-    });
-    if (modelType === 'chat') {
-      setChatModelProperties(updatedProperties);
-    } else {
-      setInlineCompletionModelProperties(updatedProperties);
-    }
-  };
-
-  const handleReloadMCPServersClick = async () => {
-    const data = await NBIAPI.reloadMCPServerList();
-    setMcpServerNames(data.mcpServers?.map((server: any) => server.id) || []);
-  };
-
-  useEffect(() => {
-    updateModelOptionsForProvider(chatModelProvider, 'chat');
-    updateModelOptionsForProvider(
-      inlineCompletionModelProvider,
-      'inline-completion'
-    );
-  }, []);
-
-  return (
-    <div className="config-dialog">
-      <div className="config-dialog-body">
-        <div className="model-config-section">
-          <div className="model-config-section-header">Default chat mode</div>
-          <div className="model-config-section-body">
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                <div>
-                  <select
-                    className="jp-mod-styled"
-                    value={defaultChatMode}
-                    onChange={event => setDefaultChatMode(event.target.value)}
-                  >
-                    <option value="ask">Ask</option>
-                    <option value="agent">Agent</option>
-                  </select>
-                </div>
-              </div>
-              <div className="model-config-section-column"> </div>
-            </div>
-          </div>
-        </div>
-
-        <div className="model-config-section">
-          <div className="model-config-section-header">Chat model</div>
-          <div className="model-config-section-body">
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                <div>Provider</div>
-                <div>
-                  <select
-                    className="jp-mod-styled"
-                    onChange={event =>
-                      updateModelOptionsForProvider(event.target.value, 'chat')
-                    }
-                  >
-                    {llmProviders.map((provider: any, index: number) => (
-                      <option
-                        key={index}
-                        value={provider.id}
-                        selected={provider.id === chatModelProvider}
-                      >
-                        {provider.name}
-                      </option>
-                    ))}
-                    <option
-                      key={-1}
-                      value="none"
-                      selected={
-                        chatModelProvider === 'none' ||
-                        !llmProviders.find(
-                          provider => provider.id === chatModelProvider
-                        )
-                      }
-                    >
-                      None
-                    </option>
-                  </select>
-                </div>
-              </div>
-              {!['openai-compatible', 'litellm-compatible', 'none'].includes(
-                chatModelProvider
-              ) &&
-                chatModels.length > 0 && (
-                  <div className="model-config-section-column">
-                    <div>Model</div>
-                    {![
-                      OPENAI_COMPATIBLE_CHAT_MODEL_ID,
-                      LITELLM_COMPATIBLE_CHAT_MODEL_ID
-                    ].includes(chatModel) &&
-                      chatModels.length > 0 && (
-                        <div>
-                          <select
-                            className="jp-mod-styled"
-                            onChange={event => setChatModel(event.target.value)}
-                          >
-                            {chatModels.map((model: any, index: number) => (
-                              <option
-                                key={index}
-                                value={model.id}
-                                selected={model.id === chatModel}
-                              >
-                                {model.name}
-                              </option>
-                            ))}
-                          </select>
-                        </div>
-                      )}
-                  </div>
-                )}
-            </div>
-
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                {chatModelProvider === 'ollama' && chatModels.length === 0 && (
-                  <div className="ollama-warning-message">
-                    No Ollama models found! Make sure{' '}
-                    <a href="https://ollama.com/" target="_blank">
-                      Ollama
-                    </a>{' '}
-                    is running and models are downloaded to your computer.{' '}
-                    <a
-                      href="javascript:void(0)"
-                      onClick={handleRefreshOllamaModelListClick}
-                    >
-                      Try again
-                    </a>{' '}
-                    once ready.
-                  </div>
-                )}
-              </div>
-            </div>
-
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                {chatModelProperties.map((property: any, index: number) => (
-                  <div className="form-field-row" key={index}>
-                    <div className="form-field-description">
-                      {property.name} {property.optional ? '(optional)' : ''}
-                    </div>
-                    <input
-                      name="chat-model-id-input"
-                      placeholder={property.description}
-                      className="jp-mod-styled"
-                      spellCheck={false}
-                      value={property.value}
-                      onChange={event =>
-                        onModelPropertyChange(
-                          'chat',
-                          property.id,
-                          event.target.value
-                        )
-                      }
-                    />
-                  </div>
-                ))}
-              </div>
-            </div>
-          </div>
-        </div>
-
-        <div className="model-config-section">
-          <div className="model-config-section-header">Auto-complete model</div>
-          <div className="model-config-section-body">
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                <div>Provider</div>
-                <div>
-                  <select
-                    className="jp-mod-styled"
-                    onChange={event =>
-                      updateModelOptionsForProvider(
-                        event.target.value,
-                        'inline-completion'
-                      )
-                    }
-                  >
-                    {llmProviders.map((provider: any, index: number) => (
-                      <option
-                        key={index}
-                        value={provider.id}
-                        selected={provider.id === inlineCompletionModelProvider}
-                      >
-                        {provider.name}
-                      </option>
-                    ))}
-                    <option
-                      key={-1}
-                      value="none"
-                      selected={
-                        inlineCompletionModelProvider === 'none' ||
-                        !llmProviders.find(
-                          provider =>
-                            provider.id === inlineCompletionModelProvider
-                        )
-                      }
-                    >
-                      None
-                    </option>
-                  </select>
-                </div>
-              </div>
-              {!['openai-compatible', 'litellm-compatible', 'none'].includes(
-                inlineCompletionModelProvider
-              ) && (
-                <div className="model-config-section-column">
-                  <div>Model</div>
-                  {![
-                    OPENAI_COMPATIBLE_INLINE_COMPLETION_MODEL_ID,
-                    LITELLM_COMPATIBLE_INLINE_COMPLETION_MODEL_ID
-                  ].includes(inlineCompletionModel) && (
-                    <div>
-                      <select
-                        className="jp-mod-styled"
-                        onChange={event =>
-                          setInlineCompletionModel(event.target.value)
-                        }
-                      >
-                        {inlineCompletionModels.map(
-                          (model: any, index: number) => (
-                            <option
-                              key={index}
-                              value={model.id}
-                              selected={model.id === inlineCompletionModel}
-                            >
-                              {model.name}
-                            </option>
-                          )
-                        )}
-                      </select>
-                    </div>
-                  )}
-                </div>
-              )}
-            </div>
-
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                {inlineCompletionModelProperties.map(
-                  (property: any, index: number) => (
-                    <div className="form-field-row" key={index}>
-                      <div className="form-field-description">
-                        {property.name} {property.optional ? '(optional)' : ''}
-                      </div>
-                      <input
-                        name="inline-completion-model-id-input"
-                        placeholder={property.description}
-                        className="jp-mod-styled"
-                        spellCheck={false}
-                        value={property.value}
-                        onChange={event =>
-                          onModelPropertyChange(
-                            'inline-completion',
-                            property.id,
-                            event.target.value
-                          )
-                        }
-                      />
-                    </div>
-                  )
-                )}
-              </div>
-            </div>
-          </div>
-        </div>
-
-        {(chatModelProvider === 'github-copilot' ||
-          inlineCompletionModelProvider === 'github-copilot') && (
-          <div className="model-config-section">
-            <div className="model-config-section-header access-token-config-header">
-              GitHub Copilot login{' '}
-              <a
-                href="https://github.com/notebook-intelligence/notebook-intelligence/blob/main/README.md#remembering-github-copilot-login"
-                target="_blank"
-              >
-                {' '}
-                <VscWarning
-                  className="access-token-warning"
-                  title="Click to learn more about security implications"
-                />
-              </a>
-            </div>
-            <div className="model-config-section-body">
-              <div className="model-config-section-row">
-                <div className="model-config-section-column">
-                  <label>
-                    <input
-                      type="checkbox"
-                      checked={storeGitHubAccessToken}
-                      onChange={event => {
-                        setStoreGitHubAccessToken(event.target.checked);
-                      }}
-                    />
-                    Remember my GitHub Copilot access token
-                  </label>
-                </div>
-              </div>
-            </div>
-          </div>
-        )}
-
-        <div className="model-config-section">
-          <div className="model-config-section-header">
-            MCP Servers ({mcpServerNames.length}) [
-            <a href="javascript:void(0)" onClick={props.onEditMCPConfigClicked}>
-              edit
-            </a>
-            ]
-          </div>
-          <div className="model-config-section-body">
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                {mcpServerNames.length === 0 && (
-                  <div>
-                    No MCP servers found. Add MCP servers in the configuration
-                    file.
-                  </div>
-                )}
-                {mcpServerNames.length > 0 && (
-                  <div>{mcpServerNames.sort().join(', ')}</div>
-                )}
-              </div>
-              <div
-                className="model-config-section-column"
-                style={{ flexGrow: 'initial' }}
-              >
-                <button
-                  className="jp-Dialog-button jp-mod-reject jp-mod-styled"
-                  onClick={handleReloadMCPServersClick}
-                >
-                  <div className="jp-Dialog-buttonLabel">Reload</div>
-                </button>
-              </div>
-            </div>
-          </div>
-        </div>
-
-        <div className="model-config-section">
-          <div className="model-config-section-header">Config file path</div>
-          <div className="model-config-section-body">
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                <span
-                  className="user-code-span"
-                  onClick={() => {
-                    navigator.clipboard.writeText(
-                      path.join(NBIAPI.config.userConfigDir, 'config.json')
-                    );
-                    return true;
-                  }}
-                >
-                  {path.join(NBIAPI.config.userConfigDir, 'config.json')}{' '}
-                  <span
-                    className="copy-icon"
-                    dangerouslySetInnerHTML={{ __html: copySvgstr }}
-                  ></span>
-                </span>
-              </div>
-            </div>
-          </div>
-          <div className="model-config-section-header">
-            MCP config file path
-          </div>
-          <div className="model-config-section-body">
-            <div className="model-config-section-row">
-              <div className="model-config-section-column">
-                <span
-                  className="user-code-span"
-                  onClick={() => {
-                    navigator.clipboard.writeText(
-                      path.join(NBIAPI.config.userConfigDir, 'mcp.json')
-                    );
-                    return true;
-                  }}
-                >
-                  {path.join(NBIAPI.config.userConfigDir, 'mcp.json')}{' '}
-                  <span
-                    className="copy-icon"
-                    dangerouslySetInnerHTML={{ __html: copySvgstr }}
-                  ></span>
-                </span>
-              </div>
-            </div>
-          </div>
-        </div>
-      </div>
-
-      <div className="config-dialog-footer">
-        <button
-          className="jp-Dialog-button jp-mod-accept jp-mod-styled"
-          onClick={handleSaveClick}
-        >
-          <div className="jp-Dialog-buttonLabel">Save</div>
-        </button>
-      </div>
-    </div>
-  );
-}