Revert "feature: add not filter tool options to shopify"

This reverts commit addba72.

Author: cdxker

clients/trieve-shopify-extension/app/components/settings/LLMSettings.tsx

@@ -24,7 +24,6 @@ import {
   PriceToolCallOptions,
   RelevanceToolCallOptions,
   SearchToolCallOptions,
-  NotFilterToolCallOptions,
 } from "trieve-ts-sdk";
 
 export const defaultRelevanceToolCallOptions: RelevanceToolCallOptions = {
@@ -55,28 +54,20 @@ export const defaultSearchToolCallOptions: SearchToolCallOptions = {
     "Call this tool anytime it seems like we need to skip the search step. This tool tells our system that the user is asking about what they were previously shown.",
 };
 
-export const defaultNotFilterToolCallOptions: NotFilterToolCallOptions = {
-  userMessageTextPrefix: "Here is the user query:",
-  toolDescription:
-    "Set to true if the query is not interested in the products they were shown previously or would like to see something different. Ensure that this is only set to true when the user wants to see something different from the previously returned results or is not interested in those previously returned results.",
-};
-
 interface LLMSettingsProps {
   shopDataset: Dataset;
   existingPdpPrompt: string;
   existingRelevanceToolCallOptions: RelevanceToolCallOptions | null;
   existingPriceToolCallOptions: PriceToolCallOptions | null;
   existingSearchToolCallOptions: SearchToolCallOptions | null;
-  existingNotFilterToolCallOptions: NotFilterToolCallOptions | null;
 }
 
 export function LLMSettings({
   shopDataset,
   existingPdpPrompt,
   existingRelevanceToolCallOptions,
   existingPriceToolCallOptions,
-  existingSearchToolCallOptions,
-  existingNotFilterToolCallOptions,
+  existingSearchToolCallOptions
 }: LLMSettingsProps) {
   const shopify = useAppBridge();
   const submit = useSubmit();
@@ -101,10 +92,6 @@ export function LLMSettings({
     existingSearchToolCallOptions ?? defaultSearchToolCallOptions,
   );
 
-  const [notFilterToolCallOptions, setNotFilterToolCallOptions] = useState(
-    existingNotFilterToolCallOptions ?? defaultNotFilterToolCallOptions,
-  );
-
   const onLLMSettingsSave = async () => {
     submit(
       {
@@ -127,7 +114,6 @@ export function LLMSettings({
         relevance_tool_call_options: JSON.stringify(relevanceToolCallOptions),
         price_tool_call_options: JSON.stringify(priceToolCallOptions),
         search_tool_call_options: JSON.stringify(searchToolCallOptions),
-        not_filter_tool_call_options: JSON.stringify(notFilterToolCallOptions),
         dataset_id: shopDataset.id,
         type: "tool_call_options",
       },
@@ -532,48 +518,6 @@ export function LLMSettings({
                 <Button onClick={saveToolCallOptions}>Save</Button>
               </InlineStack>
             </Card>
-            <Card roundedAbove="sm">
-              <BlockStack gap="400">
-                <FormLayout>
-                  <Text as="h1" variant="headingMd">
-                    Not Filter Tool Configuration
-                  </Text>
-                  <BlockStack gap="400">
-                    <TextField
-                      label="Tool Description"
-                      helpText="The description of the tool"
-                      value={notFilterToolCallOptions.toolDescription ?? ""}
-                      onChange={(e) =>
-                        setNotFilterToolCallOptions({
-                          ...notFilterToolCallOptions,
-                          toolDescription: e,
-                        })
-                      }
-                      multiline={3}
-                      autoComplete="off"
-                    />
-                    <TextField
-                      label="User Message Text Prefix"
-                      helpText="The prefix to use before showing the the users message"
-                      value={
-                        notFilterToolCallOptions.userMessageTextPrefix ?? ""
-                      }
-                      onChange={(e) =>
-                        setNotFilterToolCallOptions({
-                          ...notFilterToolCallOptions,
-                          userMessageTextPrefix: e,
-                        })
-                      }
-                      multiline={3}
-                      autoComplete="off"
-                    />
-                  </BlockStack>
-                </FormLayout>
-              </BlockStack>
-              <InlineStack align="end" gap="200">
-                <Button onClick={saveToolCallOptions}>Save</Button>
-              </InlineStack>
-            </Card>
           </BlockStack>
         </InlineGrid>
       </BlockStack>

clients/trieve-shopify-extension/app/routes/app._dashboard.settings.tsx

@@ -17,7 +17,6 @@ import {
   PriceToolCallOptions,
   type Dataset,
   SearchToolCallOptions,
-  NotFilterToolCallOptions,
 } from "trieve-ts-sdk";
 import { createWebPixel, isWebPixelInstalled } from "app/queries/webPixel";
 import { getAppMetafields, setAppMetafields } from "app/queries/metafield";
@@ -42,7 +41,6 @@ export const loader = async ({
   relevanceToolCallOptions: RelevanceToolCallOptions | null;
   searchToolCallOptions: SearchToolCallOptions | null;
   priceToolCallOptions: PriceToolCallOptions | null;
-  notFilterToolCallOptions: NotFilterToolCallOptions | null;
 }> => {
   const { session } = await authenticate.admin(request);
   const key = await validateTrieveAuth(request);
@@ -97,11 +95,6 @@ export const loader = async ({
     fetcher,
     "price_tool_call_options",
   );
-  const notFilterToolCallOptions =
-    await getAppMetafields<NotFilterToolCallOptions>(
-      fetcher,
-      "not_filter_tool_call_options",
-    );
 
   return {
     crawlSettings: crawlSettings?.crawlSettings,
@@ -112,7 +105,6 @@ export const loader = async ({
     relevanceToolCallOptions,
     searchToolCallOptions,
     priceToolCallOptions,
-    notFilterToolCallOptions,
   };
 };
 
@@ -221,9 +213,6 @@ export const action = async ({ request }: ActionFunctionArgs) => {
       );
       const priceToolCallOptions = formData.get("price_tool_call_options");
       const searchToolCallOptions = formData.get("search_tool_call_options");
-      const notFilterToolCallOptions = formData.get(
-        "not_filter_tool_call_options",
-      );
       await setAppMetafields(fetcher, [
         {
           key: "relevance_tool_call_options",
@@ -240,11 +229,6 @@ export const action = async ({ request }: ActionFunctionArgs) => {
           value: searchToolCallOptions as string,
           type: "json",
         },
-        {
-          key: "not_filter_tool_call_options",
-          value: notFilterToolCallOptions as string,
-          type: "json",
-        },
       ]);
       return { success: true };
     }
@@ -299,7 +283,6 @@ export default function Dataset() {
     relevanceToolCallOptions,
     searchToolCallOptions,
     priceToolCallOptions,
-    notFilterToolCallOptions,
   } = useLoaderData<typeof loader>();
   const [selectedTab, setSelectedTab] = useState(0);
 
@@ -365,7 +348,6 @@ export default function Dataset() {
         existingRelevanceToolCallOptions={relevanceToolCallOptions}
         existingSearchToolCallOptions={searchToolCallOptions}
         existingPriceToolCallOptions={priceToolCallOptions}
-        existingNotFilterToolCallOptions={notFilterToolCallOptions}
       />
     ),
     "preset-questions": <PresetQuestions initialQuestions={presetQuestions} />,