feat: allow hyphens in custom command names and aliases

Author: paduszyk

docs/README.md

@@ -124,15 +124,15 @@ Example:
 
 ```python
 MANAGEMENT_COMMANDS_PATHS = {
-    "mycommand": "mysite.commands.MyCommand",
+    "my-command": "mysite.commands.MyCommand",
 }
 ```
 
-You can now run the custom command `mycommand` implemented in the `MyCommand` class
+You can now run the custom command `my-command` implemented in the `MyCommand` class
 from the `mysite.commands` module:
 
 ```console
-python manage.py mycommand
+python manage.py my-command
 ```
 
 > In Django, the class representing a command must be named `Command`. The plugin
@@ -141,8 +141,8 @@ python manage.py mycommand
 
 **Important Notes:**
 
-- All keys and values must be valid Python identifiers and absolute dotted paths,
-  respectively.
+- All keys and values must be valid Python identifiers (with hyphens allowed) and
+  absolute dotted paths, respectively.
 - Paths must point to command classes, not modules.
 - Commands must subclass `django.core.management.base.BaseCommand`.
 - This setting takes precedence over others when discovering commands.
@@ -214,26 +214,26 @@ Example:
 
 ```python
 MANAGEMENT_COMMANDS_ALIASES = {
-    "fullcheck": [
+    "full-check": [
         "check --fail-level ERROR --deploy",
         "makemigrations --check --dry-run --no-input",
         "migrate --no-input",
     ],
 }
 ```
 
-You can now execute all the commands aliased by `fullcheck` with a single command:
+You can now execute all the commands aliased by `full-check` with a single command:
 
 ```console
-python manage.py fullcheck
+python manage.py full-check
 ```
 
 Aliases can refer to commands defined in the `MANAGEMENT_COMMANDS_PATHS` setting
 or other aliases.
 
 **Important Notes:**
 
-- Keys must be valid Python identifiers.
+- Keys must be valid Python identifiers (with hyphens allowed).
 - Values should be command expressions with parsable arguments and options.
 - Circular references within aliases are not allowed, as they lead to infinite recursion.
 

src/management_commands/conf.py

@@ -8,7 +8,7 @@
 
 
 def _is_identifier(s: str) -> bool:
-    return s.isidentifier() and not iskeyword(s)
+    return s.replace("-", "_").isidentifier() and not iskeyword(s)
 
 
 def _is_dotted_path(s: str, /, *, min_parts: int = 0) -> bool:
@@ -53,7 +53,7 @@ def configure_paths(self, setting_value: dict[str, str]) -> dict[str, str]:
             if not _is_identifier(key):
                 msg = (
                     f"invalid key {key!r} in PATHS; "
-                    f"keys must be valid Python identifiers"
+                    f"keys must be valid Python identifiers (with hyphens allowed)"
                 )
 
                 raise self.improperly_configured(msg, "paths.key")
@@ -103,7 +103,7 @@ def configure_aliases(
             if not _is_identifier(key):
                 msg = (
                     f"invalid key {key!r} in ALIASES; "
-                    f"keys must be valid Python identifiers"
+                    f"keys must be valid Python identifiers (with hyphens allowed)"
                 )
 
                 raise self.improperly_configured(msg, "aliases.key")