Some package audit fix (#2497)

* Some package audit fix * Message > MessageElement for Slack * Fix some Rust extra warnings * nit
dust-tt · Nov 13, 2023 · ab717d9 · ab717d9
1 parent 4a61aec
commit ab717d9
Show file tree

Hide file tree

Showing 9 changed files with 43 additions and 58 deletions.
diff --git a/connectors/package-lock.json b/connectors/package-lock.json
diff --git a/connectors/src/connectors/slack/bot.ts b/connectors/src/connectors/slack/bot.ts
@@ -1,5 +1,5 @@
 import { WebClient } from "@slack/web-api";
-import { Message } from "@slack/web-api/dist/response/ConversationsHistoryResponse";
+import { MessageElement } from "@slack/web-api/dist/response/ConversationsHistoryResponse";
 import { ConversationsRepliesResponse } from "@slack/web-api/dist/response/ConversationsRepliesResponse";
 import levenshtein from "fast-levenshtein";
 
@@ -620,7 +620,7 @@ async function makeContentFragment(
   startingAtTs: string | null,
   connector: Connector
 ) {
-  let allMessages: Message[] = [];
+  let allMessages: MessageElement[] = [];
 
   let next_cursor = undefined;
 

diff --git a/connectors/src/connectors/slack/temporal/activities.ts b/connectors/src/connectors/slack/temporal/activities.ts
@@ -6,7 +6,7 @@ import {
 } from "@slack/web-api";
 import {
   ConversationsHistoryResponse,
-  Message,
+  MessageElement,
 } from "@slack/web-api/dist/response/ConversationsHistoryResponse";
 import {
   Channel,
@@ -361,7 +361,7 @@ export async function syncNonThreaded(
   const dataSourceConfig = dataSourceConfigFromConnector(connector);
   const client = await getSlackClient(connectorId);
   const nextCursor: string | undefined = undefined;
-  const messages: Message[] = [];
+  const messages: MessageElement[] = [];
 
   const startTsSec = Math.round(startTsMs / 1000);
   const endTsSec = Math.round(endTsMs / 1000);
@@ -523,7 +523,7 @@ export async function syncThread(
   const dataSourceConfig = dataSourceConfigFromConnector(connector);
   const slackClient = await getSlackClient(connectorId);
 
-  let allMessages: Message[] = [];
+  let allMessages: MessageElement[] = [];
 
   let next_cursor = undefined;
 
@@ -645,7 +645,7 @@ async function processMessageForMentions(
 
 export async function formatMessagesForUpsert(
   channelId: string,
-  messages: Message[],
+  messages: MessageElement[],
   connectorId: ModelId,
   slackClient: WebClient
 ) {

diff --git a/core/src/providers/ai21.rs b/core/src/providers/ai21.rs
@@ -210,7 +210,7 @@ impl LLM for AI21LLM {
     ) -> Result<LLMGeneration> {
         let r = self
             .generate(
-                prompt.clone(),
+                prompt,
                 1, // num_results
                 match max_tokens {
                     Some(f) => f,

diff --git a/core/src/providers/azure_openai.rs b/core/src/providers/azure_openai.rs
@@ -312,7 +312,7 @@ impl LLM for AzureOpenAILLM {
                     self.api_key.clone().unwrap(),
                     None,
                     None,
-                    prompt.clone(),
+                    prompt,
                     max_tokens,
                     temperature,
                     n,
@@ -351,7 +351,7 @@ impl LLM for AzureOpenAILLM {
                     self.api_key.clone().unwrap(),
                     None,
                     None,
-                    prompt.clone(),
+                    prompt,
                     max_tokens,
                     temperature,
                     n,

diff --git a/core/src/providers/cohere.rs b/core/src/providers/cohere.rs
@@ -328,7 +328,7 @@ impl LLM for CohereLLM {
 
         let r = self
             .generate(
-                prompt.clone(),
+                prompt,
                 max_tokens,
                 temperature,
                 n,

diff --git a/core/src/providers/openai.rs b/core/src/providers/openai.rs
@@ -1221,7 +1221,7 @@ impl LLM for OpenAILLM {
                         None => None,
                     },
                     Some(self.id.clone()),
-                    prompt.clone(),
+                    prompt,
                     max_tokens,
                     temperature,
                     n,
@@ -1266,7 +1266,7 @@ impl LLM for OpenAILLM {
                         None => None,
                     },
                     Some(self.id.clone()),
-                    prompt.clone(),
+                    prompt,
                     max_tokens,
                     temperature,
                     n,

diff --git a/front/package-lock.json b/front/package-lock.json
diff --git a/front/pages/w/[wId]/builder/data-sources/managed.tsx b/front/pages/w/[wId]/builder/data-sources/managed.tsx
@@ -384,7 +384,7 @@ export default function DataSourcesView({
                             ds.connectorProvider as ConnectorProvider
                           ] ||
                           !isAdmin;
-                        const onclick = async () => {
+                        const onClick = async () => {
                           let isDataSourceAllowedInPlan: boolean;
 
                           switch (ds.connectorProvider) {
@@ -439,7 +439,7 @@ export default function DataSourcesView({
                                   variant="primary"
                                   icon={CloudArrowLeftRightIcon}
                                   disabled={disabled}
-                                  onClick={onclick}
+                                  onClick={onClick}
                                   label={label}
                                 />
                               )}
@@ -503,7 +503,7 @@ export default function DataSourcesView({
                                             variant="secondary"
                                             icon={CloudArrowLeftRightIcon}
                                             disabled={disabled}
-                                            onClick={onclick}
+                                            onClick={onClick}
                                             label="Acknowledge and Connect"
                                           />
                                         </DropdownMenu.Button>