From a6eea4bf767b1aabec7685bcfe54996fa408376f Mon Sep 17 00:00:00 2001
From: Aitor Algorta <aitor@airy.co>
Date: Mon, 8 Jan 2024 16:20:20 +0100
Subject: [PATCH] some fixes

---
 .../Connectors/ConnectorWrapper/index.tsx     | 16 ++-------
 .../TopicDescription/TopicDescription.tsx     | 13 ++++++-
 .../src/pages/Topics/index.module.scss        |  2 +-
 .../src/selectors/connectors.ts               | 19 ----------
 lib/typescript/model/Config.ts                | 35 -------------------
 5 files changed, 15 insertions(+), 70 deletions(-)

diff --git a/frontend/control-center/src/pages/Connectors/ConnectorWrapper/index.tsx b/frontend/control-center/src/pages/Connectors/ConnectorWrapper/index.tsx
index 36952a5c19..98bb8d4141 100644
--- a/frontend/control-center/src/pages/Connectors/ConnectorWrapper/index.tsx
+++ b/frontend/control-center/src/pages/Connectors/ConnectorWrapper/index.tsx
@@ -183,12 +183,7 @@ const ConnectorWrapper = (props: ConnectorWrapperProps) => {
                 <div className={styles.componentTitle}>
                   <h1 className={styles.headlineText}>{connectorInfo && connectorInfo?.displayName}</h1>
                   <ConfigStatusButton
-                    // REMOVE THIS. TEMPORARY FIX FOR LLM CONNECTOR
-                    componentStatus={
-                      connectorInfo?.name === 'llm-connector'
-                        ? getComponentStatus(isHealthy, isInstalled, isConfigured, isEnabled)
-                        : getComponentStatus(true, true, true, true)
-                    }
+                    componentStatus={getComponentStatus(isHealthy, isInstalled, isConfigured, isEnabled)}
                     customStyle={styles.configStatusButton}
                   />
                 </div>
@@ -209,14 +204,7 @@ const ConnectorWrapper = (props: ConnectorWrapperProps) => {
                   </div>
                   {isConfigured && (
                     <SmartButton
-                      // REMOVE THIS. TEMPORARY FIX FOR LLM CONNECTOR
-                      title={
-                        connectorInfo?.name !== 'llm-connector'
-                          ? t('disableComponent')
-                          : isEnabled
-                          ? t('disableComponent')
-                          : t('enableComponent')
-                      }
+                      title={isEnabled ? t('disableComponent') : t('enableComponent')}
                       height={40}
                       width={132}
                       pending={isPending}
diff --git a/frontend/control-center/src/pages/Topics/TopicItem/TopicDescription/TopicDescription.tsx b/frontend/control-center/src/pages/Topics/TopicItem/TopicDescription/TopicDescription.tsx
index 26d4d42d4a..1a7d8f8dc9 100644
--- a/frontend/control-center/src/pages/Topics/TopicItem/TopicDescription/TopicDescription.tsx
+++ b/frontend/control-center/src/pages/Topics/TopicItem/TopicDescription/TopicDescription.tsx
@@ -40,7 +40,18 @@ const TopicDescription = (props: TopicDescriptionProps) => {
       } else {
         wrapperSection.current.style.height = `${basicHeight}px`;
       }
+      if ((wrapperSection.current.style.height.replace('px', '') as number) > 700) {
+        wrapperSection.current.style.height = '640px';
+      }
+    }
+  };
+
+  const recalculateCodeHeight = (code: string) => {
+    const codeHeight = calculateHeightOfCodeString(code);
+    if (codeHeight > 478) {
+      return 478;
     }
+    return codeHeight;
   };
 
   const calculateRetentionTime = (_jsonCode): string => {
@@ -87,7 +98,7 @@ const TopicDescription = (props: TopicDescriptionProps) => {
           </div>
           {expanded && (
             <MonacoEditor
-              height={calculateHeightOfCodeString(code)}
+              height={recalculateCodeHeight(code)}
               language="yaml"
               value={code}
               options={{
diff --git a/frontend/control-center/src/pages/Topics/index.module.scss b/frontend/control-center/src/pages/Topics/index.module.scss
index 222fb30a20..51071723d7 100644
--- a/frontend/control-center/src/pages/Topics/index.module.scss
+++ b/frontend/control-center/src/pages/Topics/index.module.scss
@@ -54,7 +54,7 @@
   }
 
   h2:first-of-type {
-    width: 90%;
+    width: 100%;
     padding-left: 70px;
     margin-right: 70px;
   }
diff --git a/frontend/control-center/src/selectors/connectors.ts b/frontend/control-center/src/selectors/connectors.ts
index e6f1f78c09..6abc6511f6 100644
--- a/frontend/control-center/src/selectors/connectors.ts
+++ b/frontend/control-center/src/selectors/connectors.ts
@@ -54,16 +54,6 @@ export const getMergedConnectors = createSelector(
         connectedChannels: connectedChannels,
       };
 
-      //REMOVE THIS. TESTING ONLY
-      if (name === 'llm-controller') {
-        structuredCatalog = {
-          ...structuredCatalog,
-          isConfigured: true,
-          isHealthy: true,
-          isEnabled: true,
-        };
-      }
-
       if (isApp) {
         structuredCatalog = {
           ...structuredCatalog,
@@ -88,15 +78,6 @@ export const getMergedConnectors = createSelector(
         isConfigured: isConfigured,
       };
 
-      //REMOVE THIS. TESTING ONLY
-      if (connector[0] === 'llm-controller') {
-        structuredCatalog = {
-          name: connector[0],
-          configureValues: connector[1],
-          isConfigured: true,
-        };
-      }
-
       connectorsList[structuredConnector.name] = structuredConnector;
     });
 
diff --git a/lib/typescript/model/Config.ts b/lib/typescript/model/Config.ts
index a360d200c1..6631e1441f 100644
--- a/lib/typescript/model/Config.ts
+++ b/lib/typescript/model/Config.ts
@@ -26,26 +26,6 @@ export enum Language {
 
 export const getComponents = (config: Config) => {
   const {services} = config;
-
-  // REMOVE THIS. TESTING ONLY
-  services['faiss'] = {
-    enabled: true,
-    healthy: true,
-    component: 'faiss',
-  };
-
-  services['faiss-connector'] = {
-    enabled: true,
-    healthy: true,
-    component: 'faiss-connector',
-  };
-
-  services['openai-connector'] = {
-    enabled: true,
-    healthy: true,
-    component: 'openai-connector',
-  };
-
   const servicesSorted = Object.fromEntries(
     Object.entries(services).sort((a, b) => a[1].component.localeCompare(b[1].component))
   );
@@ -53,21 +33,6 @@ export const getComponents = (config: Config) => {
   return Object.keys(servicesSorted).reduce((agg, key) => {
     const {healthy, enabled, component} = services[key];
 
-    // REMOVE THIS. TESTING ONLY
-    if (key === 'frontend-copilot') {
-      return {
-        ...agg,
-        [component]: {
-          enabled,
-          // A component is only healthy if all its services are healthy
-          healthy: true,
-          services: agg[component]
-            ? [...agg[component].services, {name: key, healthy: true}]
-            : [{name: key, healthy: true}],
-        },
-      };
-    }
-
     return {
       ...agg,
       [component]: {