gowalk-cicd 1.0.28 → 1.0.29

package/action/.daemux-version

@@ -1 +1 @@
-1.0.28
+1.0.29

package/action/scripts/mmv_decide_create.py

@@ -139,7 +139,21 @@ def _stale_editable_id(versions: list[dict], target: str) -> str | None:
         vid = v.get("id")
         if not vid:
             continue
-        if semver_tuple(version_string) >= target_t:
+        # STRICTLY greater, not >=. The skip above is a STRING compare while
+        # this is a TUPLE compare, so a version that is the SAME but written
+        # differently — "1.0" against a target of "1.0.0" — slipped past the
+        # skip and then tripped this guard, refusing the whole deploy.
+        #
+        # That is precisely what App Store Connect hands a brand-new app: it
+        # auto-creates version "1.0" when the record is made, while a Flutter
+        # project ships MARKETING_VERSION "1.0.0". Every first release hit it.
+        #
+        # Renaming an editable to an EQUAL version downgrades nothing — it
+        # normalises the string — so it falls through to the PATCH-rename
+        # candidates below. The guard's real target is unaffected: an editable
+        # genuinely AHEAD of the project (draft 1.2 vs target 1.1) still
+        # refuses, because renaming that one WOULD lose an in-progress draft.
+        if semver_tuple(version_string) > target_t:
             _refuse_downgrade(target, version_string, vid, state)
         # In-review rows (WAITING_FOR_REVIEW / IN_REVIEW) are editable
         # but not safe to rename — leave them alone and POST a fresh row.

package/action/scripts/test_mmv_decide_create.py

@@ -117,3 +117,36 @@ class StaleEditableSelectionTests(unittest.TestCase):
 
 if __name__ == "__main__":
     unittest.main()
+
+
+class EqualVersionWrittenDifferently(unittest.TestCase):
+    """ASC auto-creates "1.0" with a new app record; Flutter ships "1.0.0".
+
+    The skip above the guard is a STRING compare while the guard itself is a
+    TUPLE compare, so this pair slipped past the skip and then tripped the
+    guard — refusing the very FIRST deploy of every newly created app.
+    "1.0" and "1.0.0" are the same version: renaming the editable normalises
+    the string and downgrades nothing.
+    """
+
+    def test_equal_version_is_patch_renamed_not_refused(self):
+        editable = _v("1.0", "PREPARE_FOR_SUBMISSION", vid="ed-10")
+        result, stderr, calls = _run_decide_for_version(
+            "1.0.0", versions=[editable], ground_truth=None,
+        )
+        self.assertEqual(result["decision"], "CREATE")
+        self.assertEqual(result["versionString"], "1.0.0")
+        self.assertEqual(calls[0].kwargs.get("stale_editable_id"), "ed-10")
+
+    def test_a_genuinely_higher_draft_still_refuses(self):
+        """The guard's real purpose must survive: renaming a draft AHEAD of
+        the project would lose in-progress work."""
+        editable = _v("1.2.0", "PREPARE_FOR_SUBMISSION", vid="ed-120")
+        stderr = _assert_decide_exits(
+            self, "1.1.0", versions=[editable], ground_truth=None,
+        )
+        self.assertIn("downgrade", stderr.lower())
+
+
+if __name__ == "__main__":
+    unittest.main()

package/android-action/.daemux-version

@@ -1 +1 @@
-1.0.28
+1.0.29

package/backend-action/.daemux-version

@@ -1 +1 @@
-1.0.28
+1.0.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gowalk-cicd",
-  "version": "1.0.28",
+  "version": "1.0.29",
   "description": "Zero-config GitHub Actions delivery for iOS TestFlight and Android Google Play.",
   "type": "module",
   "bin": {