ENH: Accept dev prerelease versions (#26)

* accept dev prerelease versions

e.g. `1.2.3.dev0`

still doesn't accept valid pep440 versions with multiple fields like

1.2.3a1.dev0
1.2.3.post1.dev2

* fix: support dev as a dev suffix

---------

Co-authored-by: Angus Hollands <[email protected]>

Author: minrk

README.md

@@ -74,6 +74,7 @@ accepts the common version parts:
 - `minor`
 - `patch`
 - `pre-release`
+- `dev-release`
 
 e.g. `1.2.3-rc0`.
 

hatch_nodejs_version/version_source.py

@@ -7,22 +7,44 @@
 
 from hatchling.version.source.plugin.interface import VersionSourceInterface
 
+PRE_PATTERN = r"""
+        (?P<{prefix}pre_l>(a|b|c|rc|alpha|beta|pre|preview))
+        [-.]?
+        (?P<{prefix}pre_n>[0-9]+)?
+
+"""
+DEV_PATTERN = r"""
+        dev
+        [-.]?
+        (?P<{prefix}dev_n>[0-9]+)?
+"""
+
 # The Python-aware NodeJS version regex
 # This is very similar to `packaging.version.VERSION_PATTERN`, with a few changes:
 # - Don't accept underscores
 # - Only support four-component release, prerelease, and build segments
 # - Require - to indicate prerelease
-NODE_VERSION_PATTERN = r"""
+NODE_VERSION_PATTERN = rf"""
     (?P<major>[0-9]+)                             # major
     \.
     (?P<minor>[0-9]+)                             # minor
     \.
     (?P<patch>[0-9]+)                             # patch
-    (?P<pre>                                      # pre-release
+    (?:
+      (?P<pre_only>                               # pre-release    
         -
-        (?P<pre_l>(a|b|c|rc|alpha|beta|pre|preview))
-        [-\.]?
-        (?P<pre_n>[0-9]+)?
+        {PRE_PATTERN.format(prefix="pre_only_")}
+      ) | 
+      (?P<dev_only>                               # dev
+        -
+        {DEV_PATTERN.format(prefix="dev_only_")}
+      ) |
+      (?P<pre_dev>                                # pre-release and dev    
+        -
+        {PRE_PATTERN.format(prefix="pre_dev_")}
+        \.
+        {DEV_PATTERN.format(prefix="pre_dev_")}
+      )
     )?
     (?:
        \+
@@ -40,7 +62,7 @@
 
 # The NodeJS-aware Python version regex
 # This is very similar to `packaging.version.VERSION_PATTERN`, with a few changes:
-# - Only support four-component release, prerelease, and build segments
+# - Only support five-component release, prerelease, dev, and build segments
 PYTHON_VERSION_PATTERN = r"""
    v?
    (?:
@@ -50,12 +72,17 @@
        \.
        (?P<patch>[0-9]+)                           # patch
        (?P<pre>                                    # pre-release
-           [-_\.]?
+           [-_.]?
            (?P<pre_l>(alpha|beta|preview|a|b|c|rc|pre))
-           [-_\.]?
+           [-_.]?
            (?P<pre_n>[0-9]+)?
        )?
-       (?:
+       (?P<dev>
+           [-_.]?
+           dev
+           (?P<dev_n>[0-9]+)
+       )?
+       (?:                                         # local version number
            \+
            (?P<local>
                 [0-9A-Za-z][0-9A-Za-z-_]*          # non-hyphen/dash leading identifier
@@ -105,16 +132,31 @@ def node_version_to_python(version: str) -> str:
         if match is None:
             raise ValueError(f"Version {version!r} did not match regex")
 
-        parts = ["{major}.{minor}.{patch}".format_map(match)]
+        groups = {}
+        for key, value in match.groupdict().items():
+            key = re.sub("pre_dev_|pre_only_|dev_only_", "", key)
+            if key not in groups:
+                groups[key] = value
+            elif value is not None:
+                groups[key] = value
 
-        if match["pre"]:
-            if match["pre_n"] is None:
-                parts.append("{pre_l}".format_map(match))
+        del match
+        parts = ["{major}.{minor}.{patch}".format_map(groups)]
+
+        if groups["pre_only"] or groups["pre_dev"]:
+            if groups["pre_n"] is None:
+                parts.append("{pre_l}".format_map(groups))
             else:
-                parts.append("{pre_l}{pre_n}".format_map(match))
+                parts.append("{pre_l}{pre_n}".format_map(groups))
 
-        if match["build"]:
-            parts.append("+{build}".format_map(match))
+        if groups["dev_only"] or groups["pre_dev"]:
+            if groups["dev_n"] is None:
+                parts.append("dev0")
+            else:
+                parts.append("dev{dev_n}".format_map(groups))
+
+        if groups["build"]:
+            parts.append("+{build}".format_map(groups))
 
         return "".join(parts)
 
@@ -137,6 +179,12 @@ def python_version_to_node(version: str) -> str:
             else:
                 parts.append("-{pre_l}{pre_n}".format_map(match))
 
+        if match["dev"]:
+            if match["pre"]:
+                parts.append(".dev{dev_n}".format_map(match))
+            else:
+                parts.append("-dev{dev_n}".format_map(match))
+
         if match["local"]:
             parts.append("+{local}".format_map(match))
         return "".join(parts)

tests/test_version_config.py

@@ -14,6 +14,8 @@
     ("1.4.5-b0", "1.4.5b0"),
     ("1.4.5-c1", "1.4.5c1"),
     ("1.4.5-rc0", "1.4.5rc0"),
+    ("1.4.5-dev0", "1.4.5dev0"),
+    ("1.4.5-a1.dev0", "1.4.5a1dev0"),
     ("1.4.5-alpha0", "1.4.5alpha0"),
     ("1.4.5-beta0", "1.4.5beta0"),
     ("1.4.5-pre9", "1.4.5pre9"),