Updates

Use a separate directory for each file. Allows for 'download.zip'. Also allows saving filename to work, since I need to get it from *somewhere*.
Add errors for missing inputs.
Also give a stupid excuse when there's an error. Mostly we blame the user.

app.py

@@ -5,27 +5,47 @@ import os
 from huggingface_hub import whoami
 from huggingface_hub import HfApi
 from huggingface_hub import login
+import random
+import time
 
 api=HfApi()
 REPO_TYPES = ["model", "dataset", "space"]
 
-def duplicate(source_url, dst_repo, token, file_name, dest,repo_type):
+def duplicate(source_url, dst_repo, token, new_name, dst_repo_path, repo_type):
     try:
         _ = whoami(token)
         # ^ this will throw if token is invalid
 
+        # make sure the user fills out the other required paths.
+        if not dst_repo_path[len(dst_repo_path)-1] == '/':
+                raise Exception("Your destination path *must* end with a /")
+        if not source_url:
+                raise Exception("You haven't chosen a file to download!")
+        if not dst_repo:
+                raise Exception("You haven't chosen a repo to download to")
         login(token=token)
 
-        subprocess.check_call([r"mkdir","-p","downloads"])
-        os.chdir('downloads')
-        subprocess.check_call([r"aria2c","-x16","--split=16",source_url])
+        # keep things separate, partly in case people download different files with same name (`download.zip`). Especially, it also allows saving filename to work
+        dir="/home/user/apps/downloads/"+int(time.time())+random.getrandbits(8)+"/"
+        subprocess.check_call([r"mkdir","-p",dir])
+        subprocess.check_call([r"aria2c","-x16","--split=16",source_url,"--dir="+dir])
+        files=os.listdir(dir)
+
+        if new_name:
+                dst_repo_path=dst_repo_path+new_name
+        else:
+                dst_repo_path=dst_repo_path+files[0]
 
         api.upload_file(
-                path_or_fileobj=file_name,
-                path_in_repo=dest,
+                path_or_fileobj=dir+files[0],
+                path_in_repo=dst_repo_path,
                 repo_id=dst_repo,
                 repo_type=repo_type
         )
+
+        # now clean up
+        os.remove(filetoup)
+        os.rmdir(dir)
         
         return (
             f'Find your repo <a href=\'{repo_url}\' target="_blank" style="text-decoration:underline">here</a>',
@@ -33,12 +53,20 @@ def duplicate(source_url, dst_repo, token, file_name, dest,repo_type):
         )
 
     except Exception as e:
+        blames=["grandma","my boss","your boss","God","you","you. It's *all* your fault.","the pope"]
+        blameweights=(1,1,1,1,4,2,1)
+        excuses=["I blame it all on "+random.choices(blames,weights=blameweights)[0],"It's my fault, sorry.","I did it on purpose.","That file doesn't want to be downloaded.","You nincompoop!"]
+        excusesweights=(12,1,1,2,3)
+        excuse=random.choices(excuses,weights=excusesweights)[0]
         return (
             f"""
         ### Error 😢😢😢
         
         {e}
-        """,
+        
+        
+        
+        """ + excuse,
             None,
         )
 
@@ -49,8 +77,8 @@ interface = gr.Interface(
         gr.Textbox(placeholder="Source URL (e.g. civitai.com/api/download/models/4324322534)"),
         gr.Textbox(placeholder="Destination repository (e.g. osanseviero/dst)"),
         gr.Textbox(placeholder="Write access token", type="password"),
-        gr.Textbox(placeholder="Post-download name of your file, cuz I'm lazy (e.g. stupidmodel.safetensors)"),
-        gr.Textbox(placeholder="Destination for your file (e.g. /models/Stable-diffusion/stupidmodel.safetensors)"),
+        gr.Textbox(placeholder="Post-download name of your file, if you want it changed (e.g. stupidmodel.safetensors)"),
+        gr.Textbox(placeholder="Destination for your file within your repo. Don't include the filename, end path with a / (e.g. /models/Stable-diffusion/)"),
         gr.Dropdown(choices=REPO_TYPES, value="model"),
     ],
     outputs=[