fix: Improve clarity of download error messages

The messages are cute, but some people might not understand what "your file is in another castle" means, or what the awkward phrase "log into browser to download one" means. ;)

Author: Arcitec

comfy_cli/file_utils.py

@@ -37,11 +37,11 @@ def parse_json(input_data):
         if msg_json is not None:
             if "message" in msg_json:
                 return f"Unauthorized download ({status_code}).\n{msg_json['message']}\nor you can set civitai api token using `comfy model download --set-civitai-api-token <token>`"
-        return f"Unauthorized download ({status_code}), you might need to manually log into browser to download one"
+        return f"Unauthorized download ({status_code}), you might need to manually log into a browser to download this"
     elif status_code == 403:
-        return f"Forbidden url ({status_code}), you might need to manually log into browser to download one"
+        return f"Forbidden url ({status_code}), you might need to manually log into a browser to download this"
     elif status_code == 404:
-        return "Sorry, your file is in another castle (404)"
+        return "File not found on server (404)"
     return f"Unknown error occurred (status code: {status_code})"
 
 

tests/test_file_utils.py

@@ -25,7 +25,7 @@ def test_guess_status_code_reason_401_with_json():
 def test_guess_status_code_reason_401_without_json():
     result = guess_status_code_reason(401, "not json")
     assert "Unauthorized download (401)" in result
-    assert "manually log into browser" in result
+    assert "manually log into a browser" in result
 
 
 def test_guess_status_code_reason_403():
@@ -35,7 +35,7 @@ def test_guess_status_code_reason_403():
 
 def test_guess_status_code_reason_404():
     result = guess_status_code_reason(404, "")
-    assert "another castle (404)" in result
+    assert "not found on server (404)" in result
 
 
 def test_guess_status_code_reason_unknown():