python · mattwang44 · Sep 5, 2025 · Sep 5, 2025 · Sep 5, 2025 · Sep 7, 2025
@@ -921,6 +921,10 @@ def format_usage(self):
     def __call__(self, parser, namespace, values, option_string=None):
         raise NotImplementedError('.__call__() not defined')
 
+    def _check_compatibility(self, container):
+        """Check whether the action is compatible with the container."""
+        pass
+
 
 class BooleanOptionalAction(Action):
     def __init__(self,
@@ -959,6 +963,10 @@ def __call__(self, parser, namespace, values, option_string=None):
     def format_usage(self):
         return ' | '.join(self.option_strings)
 
+    def _check_compatibility(self, container):
+        if '-' not in container.prefix_chars:
+            raise ValueError("BooleanOptionalAction requires '-' in prefix_chars.")
+
 
 class _StoreAction(Action):
 
@@ -1541,6 +1549,9 @@ def add_argument(self, *args, **kwargs):
         if not action.option_strings and action.nargs == 0:
             raise ValueError(f'action {action_name!r} is not valid for positional arguments')
 
+        # raise an error if the action is not compatible with the parser
+        action._check_compatibility(self)
+
         # raise an error if the action type is not callable
         type_func = self._registry_get('type', action.type, action.type)
         if not callable(type_func):

@@ -805,6 +805,11 @@ def test_invalid_name(self):
         self.assertEqual(str(cm.exception),
                          "invalid option name '--no-foo' for BooleanOptionalAction")
 
+    def test_prefix_chars_incompatibility(self):
+        parser = argparse.ArgumentParser(prefix_chars='+')
+        with self.assertRaisesRegex(ValueError, "BooleanOptionalAction requires '-' in prefix_chars."):
+            parser.add_argument('++foo', action=argparse.BooleanOptionalAction)
+
 class TestBooleanOptionalActionRequired(ParserTestCase):
     """Tests BooleanOptionalAction required"""
 

diff --git a/Misc/NEWS.d/next/Library/2025-09-05-15-42-16.gh-issue-138525.ngZ0qT.rst b/Misc/NEWS.d/next/Library/2025-09-05-15-42-16.gh-issue-138525.ngZ0qT.rst
@@ -0,0 +1,2 @@
+Raise error when :class:`argparse.BooleanOptionalAction` is used with
+:class:`argparse.ArgumentParser` instance whose ``prefix_chars`` has no dash char ``'-'``.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Raise error when :class:`argparse.BooleanOptionalAction` is used with
		:class:`argparse.ArgumentParser` instance whose ``prefix_chars`` has no dash char ``'-'``.