Sourcery Starbot ⭐ refactored Amm1rr/Free-Chatbot-API #3

sourcery-ai-bot · 2023-08-15T03:06:39Z

Thanks for starring sourcery-ai/sourcery ✨ 🌟 ✨

Here's your pull request refactoring your most popular Python repo.

If you want Sourcery to refactor all your Python repos and incoming pull requests install our bot.

Review changes via command line

To manually merge these changes, make sure you're on the master branch, then run:

git fetch https://github.com/sourcery-ai-bot/Free-Chatbot-API master
git merge --ff-only FETCH_HEAD
git reset HEAD^

sourcery-ai-bot · 2023-08-15T03:06:40Z

examples/client_dev.py

-url = "http://localhost:8000/" + ai
+url = f"http://localhost:8000/{ai}"


Lines 19-31 refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

Replace if statement with if expression (assign-if-exp)

Simplify boolean if expression (boolean-if-exp-identity)

Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)

sourcery-ai-bot · 2023-08-15T03:06:40Z

examples/client_dev.py

-    headers={
-        "Content-Type": "application/json",
-        "Authorization": f"Bearer ",
-    },
+    headers={"Content-Type": "application/json", "Authorization": "Bearer "},


Lines 54-131 refactored with the following changes:

Replace f-string with no interpolated values with string [×2] (remove-redundant-fstring)

Remove unreachable code (remove-unreachable-code)

This removes the following comments ( why? ):

# print(f"{data}\n\n") # Hide API key from Rich traceback.

sourcery-ai-bot · 2023-08-15T03:06:40Z

examples/example_bard.py

-    if arg1.upper() == "TRUE":
-        stream = True
-    else:
-        stream = False
-
+    stream = arg1.upper() == "TRUE"


Lines 19-23 refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

Simplify boolean if expression (boolean-if-exp-identity)

Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)

sourcery-ai-bot · 2023-08-15T03:06:40Z

examples/example_chatgpt1.py

-    if arg1.upper() == "TRUE":
-        stream = True
-    else:
-        stream = False
-
+    stream = arg1.upper() == "TRUE"


Lines 18-65 refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

Replace f-string with no interpolated values with string (remove-redundant-fstring)

Simplify boolean if expression (boolean-if-exp-identity)

Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)

sourcery-ai-bot · 2023-08-15T03:06:40Z

examples/example_chatgpt2.py

-    if arg1.upper() == "TRUE":
-        stream = True
-    else:
-        stream = False
+    stream = arg1.upper() == "TRUE"


Lines 10-13 refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

Simplify boolean if expression (boolean-if-exp-identity)

Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)

sourcery-ai-bot · 2023-08-15T03:06:42Z

src/claude.py

-        if response.status_code == 200:
-            return response.json()
-        else:
-            return False
+        return response.json() if response.status_code == 200 else False


Function Client.upload_attachment refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

sourcery-ai-bot · 2023-08-15T03:06:42Z

src/claude.py

-        if response.status_code == 200:
-            return True
-        else:
-            return False
+        return response.status_code == 200


Function Client.rename_chat refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

Simplify boolean if expression (boolean-if-exp-identity)

Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)

sourcery-ai-bot · 2023-08-15T03:06:42Z

src/main.py

-            # answer = {f"answer": "You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."}["answer"]
-            answer = f"You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."
-            # print(answer)
-            return answer
-
+            return f"You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."


Function ask_gpt refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

Hoist repeated code outside conditional statement (hoist-statement-from-if)

This removes the following comments ( why? ):

# print(answer) # answer = {f"answer": "You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."}["answer"]

sourcery-ai-bot · 2023-08-15T03:06:42Z

src/main.py

-                    f"Error": f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
+                    "Error": f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
                }

                print(response_error)
                return response_error
-                # raise ValueError(
-                #     f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
-                # )
+                            # raise ValueError(
+                            #     f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
+                            # )


Function ask_gptClaude refactored with the following changes:

Replace f-string with no interpolated values with string (remove-redundant-fstring)

Remove unnecessary else after guard condition (remove-unnecessary-else)

sourcery-ai-bot · 2023-08-15T03:06:42Z

src/main.py

-                    f"Error": f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
+                    "Error": f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
                }

                print(response_error)
                return response_error
-                # raise ValueError(
-                #     f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
-                # )
+                            # raise ValueError(
+                            #     f"You should set 'COOKIE' in '{CONFIG_FILE_NAME}' file for the Bard or send it as an argument."
+                            # )


Function ask_claude refactored with the following changes:

Replace f-string with no interpolated values with string (remove-redundant-fstring)

Inline variable that is immediately returned (inline-immediately-returned-variable)

This removes the following comments ( why? ):

# print(response)

sourcery-ai-bot · 2023-08-15T03:06:45Z

src/main.py

-            # answer = {f"answer": "You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."}["answer"]
-            answer = f"You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."
-            # print(answer)
-            return answer
-
+            return f"You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."


Function ask_chatgpt refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

This removes the following comments ( why? ):

# print(answer) # answer = {f"answer": "You should set ACCESS_TOKEN in {CONFIG_FILE_NAME} file or send it as an argument."}["answer"]

sourcery-ai-bot · 2023-08-15T03:06:45Z

src/main.py

-    for i in range(10):
+    for _ in range(10):


Function fake_data_streamer_OLD refactored with the following changes:

Replace unused for index with underscore (for-index-underscore)

sourcery-ai-bot · 2023-08-15T03:06:45Z

src/main.py

-    for i in range(10):
+    for _ in range(10):


Function fake_data_streamer refactored with the following changes:

Replace unused for index with underscore (for-index-underscore)

sourcery-ai-bot · 2023-08-15T03:06:45Z

src/main.py

-    if not session_id:
-        return False
-    if session_id.lower() == "none":
-        return False
-
-    return True
+    return False if not session_id else session_id.lower() != "none"


Function IsSession refactored with the following changes:

Lift code into else after jump in control flow [×2] (reintroduce-else)

Replace if statement with if expression [×2] (assign-if-exp)

Simplify boolean if expression (boolean-if-exp-identity)

Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)

sourcery-ai-bot · 2023-08-15T03:06:45Z

src/main.py

-    filtered_cookies = [
-        cookie for cookie in cookies if sessionName.lower() in cookie.name.lower()
-    ]
-
-    result = None
-    if filtered_cookies:
-        result = filtered_cookies[-1].value
-
-    return result
+    return (
+        filtered_cookies[-1].value
+        if (
+            filtered_cookies := [
+                cookie
+                for cookie in cookies
+                if sessionName.lower() in cookie.name.lower()
+            ]
+        )
+        else None
+    )


Function get_Cookie refactored with the following changes:

Move setting of default value for variable into else branch (introduce-default-else)

Use named expression to simplify assignment and conditional (use-named-expression)

Replace if statement with if expression (assign-if-exp)

Inline variable that is immediately returned (inline-immediately-returned-variable)

'Refactored by Sourcery'

e46b504

sourcery-ai-bot force-pushed the master branch from 930de57 to e46b504 Compare August 15, 2023 03:06

sourcery-ai-bot commented Aug 15, 2023

View reviewed changes

Amm1rr merged commit aa84a85 into Amm1rr:master Aug 15, 2023

Amm1rr pushed a commit that referenced this pull request Apr 23, 2024

Update directory path in README and reformat endpoint section.Issue: #3

b713e5c

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sourcery Starbot ⭐ refactored Amm1rr/Free-Chatbot-API #3

Sourcery Starbot ⭐ refactored Amm1rr/Free-Chatbot-API #3

sourcery-ai-bot commented Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

sourcery-ai-bot Aug 15, 2023

		url = "http://localhost:8000/" + ai
		url = f"http://localhost:8000/{ai}"

Sourcery Starbot ⭐ refactored Amm1rr/Free-Chatbot-API #3

Sourcery Starbot ⭐ refactored Amm1rr/Free-Chatbot-API #3

Conversation

sourcery-ai-bot commented Aug 15, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment