Fix more linting errors

.trunk/trunk.yaml

@@ -34,6 +34,13 @@ lint:
     - [email protected]
     - [email protected]
     - [email protected]
+  ignore:
+    - linters: [ALL]
+      paths:
+        - .github/**/*
+        - .trunk/**/*
+        - mkdocs.yml
+        - .DS_Store
 actions:
   enabled:
     - trunk-announce

README.md

@@ -1,4 +1,4 @@
-## AI TA Backend for UIUC's Course Assistant Chatbot
+# AI TA Backend for UIUC's Course Assistant Chatbot
 
 A Flask application hosting endpoints for AI TA backend.
 
@@ -26,7 +26,7 @@ The docs are auto-built and deployed to [our docs website](https://uiuc-chatbot.
 
 ### Course metadata structure
 
-```
+```text
 'text': doc.page_content,
 'readable_filename': doc.metadata['readable_filename'],
 'course_name ': doc.metadata['course_name'],

ai_ta_backend/main.py

@@ -497,4 +497,4 @@ def export_convo_history():
 
 
 if __name__ == '__main__':
-  app.run(debug=True, port=int(os.getenv("PORT", default=8000)))
+  app.run(debug=True, port=int(os.getenv("PORT", default=8000)))  # nosec -- reasonable bandit error suppression

ai_ta_backend/vector_database.py

@@ -1027,7 +1027,7 @@ def get_context_stuffed_prompt(self, user_question: str, course_name: str, top_n
     )
 
     requests = []
-    for i, doc in enumerate(found_docs):
+    for doc in found_docs:
       print("doc", doc)
       dictionary = {
           "model": "gpt-3.5-turbo",

ai_ta_backend/web_scrape.py

@@ -4,6 +4,7 @@
 import shutil
 import time
 from tempfile import NamedTemporaryFile
+from typing import List, Optional
 from zipfile import ZipFile
 
 import boto3  # type: ignore
@@ -131,7 +132,8 @@ def base_requirements(self, url: str, base: str):
 
     return False
 
-  def find_urls(self, soup: BeautifulSoup, site: str, urls: list = []):
+  def find_urls(self, soup: BeautifulSoup, site: str, urls: Optional[List] = None):
+    urls = urls if urls is not None else []
     try:
       for i in soup.find_all("a"):  # type: ignore
         try:
@@ -597,9 +599,9 @@ def is_github_repo(url):
 
 
 def mit_course_download(url: str, course_name: str, local_dir: str):
-  '''
+  """
   Download an MIT course using its url
-  '''
+  """
   ingester = Ingest()
   if url.endswith("download"):
     pass