Update for TR#1

Author: yukaty

tdd-ai-agents/.claude/settings.json

@@ -1,15 +1,5 @@
 {
-  "hooks": {
-    "PreToolUse": [
-      {
-        "matcher": "Edit|Write",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "jq -e '.tool_input.file_path | contains(\"/tests/\")' >/dev/null && echo 'tests/ is read-only during implementation' >&2 && exit 2 || exit 0"
-          }
-        ]
-      }
-    ]
+  "permissions": {
+    "deny": ["Edit(tests/**)"]
   }
 }

tdd-ai-agents/tests/test_core.py

@@ -10,20 +10,41 @@ def test_smaller_release_sorts_first():
     assert Version("1.2.0") < Version("1.3.0")
 
 
+def test_versions_with_different_lengths_compare():
+    assert Version("1.2") < Version("1.2.1")
+
+
+def test_greater_than_works():
+    assert Version("2.0.0") > Version("1.9.9")
+
+
+def test_unequal_versions_are_not_equal():
+    assert Version("1.0.0") != Version("1.0.1")
+
+
 def test_double_digit_segments_sort_numerically():
     assert Version("3.9") < Version("3.10")
 
 
+def test_sorting_a_list_of_versions():
+    versions = [Version("1.10.0"), Version("1.2.0"), Version("1.9.0")]
+    assert sorted(versions) == [
+        Version("1.2.0"),
+        Version("1.9.0"),
+        Version("1.10.0"),
+    ]
+
+
 @pytest.mark.parametrize(
     "lower, higher",
     [
-        ("1.0.dev1", "1.0"),  # Dev before final
-        ("1.0a1", "1.0"),  # Pre-release before final
-        ("1.0", "1.0.post1"),  # Post-release after final
-        ("1.0.dev1", "1.0a1"),  # Dev before pre-release
-        ("1.0a1", "1.0b1"),  # Alpha before beta
-        ("1.0b1", "1.0rc1"),  # Beta before release candidate
-        ("1.0.dev1", "1.0.post1"),  # Dev before post, spanning the release
+        ("1.0.dev1", "1.0"),
+        ("1.0a1", "1.0"),
+        ("1.0", "1.0.post1"),
+        ("1.0.dev1", "1.0a1"),
+        ("1.0a1", "1.0b1"),
+        ("1.0b1", "1.0rc1"),
+        ("1.0.dev1", "1.0.post1"),
     ],
 )
 def test_phase_ordering(lower, higher):
@@ -33,14 +54,17 @@ def test_phase_ordering(lower, higher):
 @pytest.mark.parametrize(
     "text",
     [
-        "",  # Empty string
-        "   ",  # Whitespace only
-        "1.x.0",  # Non-numeric segment
-        "banana",  # Not a version at all
-        "1..0",  # Empty segment
-        "v1.0",  # Leading junk
+        "",
+        "   ",
+        "1.x.0",
+        "banana",
+        "1..0",
     ],
 )
 def test_invalid_input_raises(text):
     with pytest.raises(ValueError):
         Version(text)
+
+
+def test_comparing_against_other_types_returns_notimplemented():
+    assert Version("1.0").__eq__("1.0") is NotImplemented

tdd-ai-agents/version_check/core.py

@@ -4,8 +4,8 @@
 _VERSION = re.compile(
     r"^\s*(?P<release>\d+(?:\.\d+)*)"
     r"(?:(?P<pre_label>a|b|rc)(?P<pre_num>\d+))?"
-    r"(?:\.dev(?P<dev>\d+))?"
-    r"(?:\.post(?P<post>\d+))?\s*$"
+    r"(?:\.post(?P<post>\d+))?"
+    r"(?:\.dev(?P<dev>\d+))?\s*$"
 )
 _PRE_ORDER = {"a": 0, "b": 1, "rc": 2}
 
@@ -23,11 +23,11 @@ def __init__(self, text):
             self._pre = (_PRE_ORDER[match["pre_label"]], int(match["pre_num"]))
         else:
             self._pre = None
-        self._dev = int(match["dev"]) if match["dev"] else None
         self._post = int(match["post"]) if match["post"] else None
+        self._dev = int(match["dev"]) if match["dev"] else None
 
     def _key(self):
-        if self._dev is not None and self._pre is None and self._post is None:
+        if self._dev is not None and self._pre is None:
             phase = (0, self._dev)
         elif self._pre is not None:
             phase = (1, self._pre)