From 989563c96741057f03ae40a6ed489e29c12efab2 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 12 Aug 2025 06:45:19 -0700
Subject: [PATCH 1/2] [LCHIB-641] if '%' is missing on Windows, suggest a
 likely cause of error

---
 launchable/utils/click.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/launchable/utils/click.py b/launchable/utils/click.py
index 1114cfafb..0828ef68a 100644
--- a/launchable/utils/click.py
+++ b/launchable/utils/click.py
@@ -33,10 +33,15 @@ def convert(self, value: str, param: Optional[click.core.Parameter], ctx: Option
                 x = float(value[:-1]) / 100
                 if 0 <= x <= 100:
                     return x
+            else:
+                missing_percent = True
         except ValueError:
             pass
 
-        self.fail("Expected percentage like 50% but got '{}'".format(value), param, ctx)
+        msg = "Expected percentage like 50% but got '{}'".format(value)
+        if missing_percent and sys.platform.startswith("win"):
+            msg += " ('%' is a special character in batch files, so please write '50%%' to pass in '50%')"
+        self.fail(msg, param, ctx)
 
 
 class DurationType(ParamType):

From 19ec11e92296a5f550c1a841910fadcf94f286f2 Mon Sep 17 00:00:00 2001
From: Naoto Ono <nono@cloudbees.com>
Date: Wed, 13 Aug 2025 08:21:24 +0900
Subject: [PATCH 2/2] Follow-up for #1102

---
 launchable/utils/click.py |  1 +
 tests/utils/test_click.py | 47 ++++++++++++++++++++++++++++++++++++++-
 2 files changed, 47 insertions(+), 1 deletion(-)

diff --git a/launchable/utils/click.py b/launchable/utils/click.py
index 0828ef68a..255a94123 100644
--- a/launchable/utils/click.py
+++ b/launchable/utils/click.py
@@ -29,6 +29,7 @@ class PercentageType(ParamType):
 
     def convert(self, value: str, param: Optional[click.core.Parameter], ctx: Optional[click.core.Context]):
         try:
+            missing_percent = False
             if value.endswith('%'):
                 x = float(value[:-1]) / 100
                 if 0 <= x <= 100:
diff --git a/tests/utils/test_click.py b/tests/utils/test_click.py
index b04029d64..9ee41600f 100644
--- a/tests/utils/test_click.py
+++ b/tests/utils/test_click.py
@@ -1,4 +1,5 @@
 import datetime
+import sys
 from datetime import timezone
 from typing import Sequence, Tuple
 from unittest import TestCase
@@ -7,7 +8,51 @@
 from click.testing import CliRunner
 from dateutil.tz import tzlocal
 
-from launchable.utils.click import DATETIME_WITH_TZ, KEY_VALUE, convert_to_seconds
+from launchable.utils.click import DATETIME_WITH_TZ, KEY_VALUE, PercentageType, convert_to_seconds
+
+
+class PercentageTypeTest(TestCase):
+    ERROR_MSG = "Expected percentage like 50% but got"
+    WINDOWS_ERROR_MSG = "please write '50%%' to pass in '50%'"
+
+    def test_invalid_value_windows(self):
+        pct = PercentageType()
+        orig_platform = sys.platform
+        sys.platform = "win32"
+        try:
+            with self.assertRaises(click.BadParameter) as cm:
+                pct.convert("50", None, None)
+            msg = str(cm.exception)
+            self.assertIn(self.ERROR_MSG + " '50'", msg)
+            self.assertIn(self.WINDOWS_ERROR_MSG, msg)
+        finally:
+            sys.platform = orig_platform
+
+    def test_invalid_value_non_windows(self):
+        pct = PercentageType()
+        orig_platform = sys.platform
+        sys.platform = "linux"
+        try:
+            with self.assertRaises(click.BadParameter) as cm:
+                pct.convert("50", None, None)
+            msg = str(cm.exception)
+            self.assertIn(self.ERROR_MSG + " '50'", msg)
+            self.assertNotIn(self.WINDOWS_ERROR_MSG, msg)
+        finally:
+            sys.platform = orig_platform
+
+    def test_invalid_float(self):
+        pct = PercentageType()
+        with self.assertRaises(click.BadParameter) as cm:
+            pct.convert("abc%", None, None)
+        msg = str(cm.exception)
+        self.assertIn(self.ERROR_MSG + " 'abc%'", msg)
+
+    def test_valid(self):
+        pct = PercentageType()
+        self.assertEqual(pct.convert("50%", None, None), 0.5)
+        self.assertEqual(pct.convert("0%", None, None), 0.0)
+        self.assertEqual(pct.convert("100%", None, None), 1.0)
 
 
 class DurationTypeTest(TestCase):