Revert some changes

Author: pamelafox

.vscode/settings.json

@@ -36,6 +36,5 @@
         "htmlcov": true,
         ".mypy_cache": true,
         ".coverage": true
-    },
-    "python.REPL.enableREPLSmartSend": false
+    }
 }

src/backend/fastapi_app/__init__.py

@@ -58,17 +58,11 @@ def create_app(testing: bool = False):
     else:
         if not testing:
             load_dotenv(override=True)
-        logging.basicConfig(level=logging.DEBUG)
-
-        # Enable detailed HTTP traffic logging
-        logging.getLogger("httpx").setLevel(logging.DEBUG)
-        logging.getLogger("azure.core.pipeline.policies.http_logging_policy").setLevel(logging.DEBUG)
-        logging.getLogger("azure.identity").setLevel(logging.DEBUG)
-        logging.getLogger("urllib3").setLevel(logging.DEBUG)
+        logging.basicConfig(level=logging.INFO)
 
-        # Configure httpx logging to show full bodies
-        os.environ["HTTPX_LOG_LEVEL"] = "DEBUG"
-        os.environ["HTTPCORE_LOG_LEVEL"] = "DEBUG"
+    # Turn off particularly noisy INFO level logs from Azure Core SDK:
+    logging.getLogger("azure.core.pipeline.policies.http_logging_policy").setLevel(logging.WARNING)
+    logging.getLogger("azure.identity").setLevel(logging.WARNING)
 
     if os.getenv("APPLICATIONINSIGHTS_CONNECTION_STRING"):
         logger.info("Configuring Azure Monitor")

src/backend/fastapi_app/prompts/query.txt

@@ -1,2 +1,5 @@
 Your job is to find search results based off the user's question and past messages.
+You have access to only these tools:
+1. **search_database**: This tool allows you to search a table for items based on a query.
+  You can pass in a search query and optional filters.
 Once you get the search results, you're done.

src/backend/fastapi_app/rag_advanced.py

@@ -82,7 +82,7 @@ async def search_database(
         Search PostgreSQL database for relevant products based on user query
 
         Args:
-            search_query: Query string to use for full text search, e.g. 'red shoes'
+            search_query: English query string to use for full text search, e.g. 'red shoes'.
             price_filter: Filter search results based on price of the product
             brand_filter: Filter search results based on brand of the product
 
@@ -109,7 +109,7 @@ async def search_database(
     async def prepare_context(self) -> tuple[list[ItemPublic], list[ThoughtStep]]:
         few_shots = ModelMessagesTypeAdapter.validate_json(self.query_fewshots)
         user_query = f"Find search results for user query: {self.chat_params.original_user_query}"
-        results = await self.search_agent.run(  # type: ignore[call-overload]
+        results = await self.search_agent.run(
             user_query,
             message_history=few_shots + self.chat_params.past_messages,
             deps=self.chat_params,

src/backend/fastapi_app/routes/api_routes.py

@@ -1,4 +1,3 @@
-import http.client
 import json
 import logging
 from collections.abc import AsyncGenerator
@@ -27,8 +26,6 @@
 router = fastapi.APIRouter()
 
 
-http.client.HTTPConnection.debuglevel = 1
-
 ERROR_FILTER = {"error": "Your message contains content that was flagged by the content filter."}
 
 

src/frontend/src/components/Answer/Answer.tsx

@@ -34,6 +34,7 @@ export const Answer = ({
     const parsedAnswer = useMemo(() => parseAnswerToHtml(messageContent, isStreaming, onCitationClicked), [answer]);
 
     const sanitizedAnswerHtml = DOMPurify.sanitize(parsedAnswer.answerHtml);
+
     return (
         <Stack className={`${styles.answerContainer} ${isSelected && styles.selected}`} verticalAlign="space-between">
             <Stack.Item>

tests/test_postgres_searcher.py

@@ -1,6 +1,6 @@
 import pytest
 
-from fastapi_app.api_models import BrandFilter, ItemPublic, PriceFilter
+from fastapi_app.api_models import Filter, ItemPublic
 from tests.data import test_data
 
 
@@ -12,7 +12,7 @@ def test_postgres_build_filter_clause_without_filters(postgres_searcher):
 def test_postgres_build_filter_clause_with_filters(postgres_searcher):
     assert postgres_searcher.build_filter_clause(
         [
-            BrandFilter(comparison_operator="=", value="AirStrider"),
+            Filter(column="brand", comparison_operator="=", value="AirStrider"),
         ]
     ) == (
         "WHERE brand = 'AirStrider'",
@@ -23,11 +23,11 @@ def test_postgres_build_filter_clause_with_filters(postgres_searcher):
 def test_postgres_build_filter_clause_with_filters_numeric(postgres_searcher):
     assert postgres_searcher.build_filter_clause(
         [
-            PriceFilter(comparison_operator="<", value=30),
+            Filter(column="price", comparison_operator="<", value=30),
         ]
     ) == (
-        "WHERE price < 30.0",
-        "AND price < 30.0",
+        "WHERE price < 30",
+        "AND price < 30",
     )