ensure proper error message if we have slashes in problem set branch name (#313)

ngc92 · web-flow · commit 30237c72fef4 · 2025-07-17T14:53:21.000-07:00
diff --git a/src/discord-cluster-manager/cogs/admin_cog.py b/src/discord-cluster-manager/cogs/admin_cog.py
@@ -526,6 +526,9 @@ async def update_problems(
             )
             return
 
+        if "/" in branch:
+            raise KernelBotError(f"branch names with slashes (`{branch}`) are not supported.")
+
         repository_name = repository_name or env.PROBLEMS_REPO
         url = f"https://github.com/{repository_name}/archive/{branch}.zip"
         folder_name = repository_name.split("/")[-1] + "-" + branch

Original file line number	Diff line number	Diff line change
`@@ -526,6 +526,9 @@ async def update_problems(`
`526`	`526`	`)`
`527`	`527`	`return`
`528`	`528`
	`529`	`+ if "/" in branch:`
	`530`	+ raise KernelBotError(f"branch names with slashes (`{branch}`) are not supported.")
	`531`	`+`
`529`	`532`	`repository_name = repository_name or env.PROBLEMS_REPO`
`530`	`533`	`url = f"https://github.com/{repository_name}/archive/{branch}.zip"`
`531`	`534`	`folder_name = repository_name.split("/")[-1] + "-" + branch`