Moved logic specific to the main command line from Cmd.read_input() to Cmd._read_command_line().

No longer setting Cmd.session.input to a pipe since that causes warnings from prompt-toolkit.
For non-interactive sessions (stdin is a pipe), we use stdin.readline().

Author: kmvanbrunt

cmd2/__init__.py

@@ -52,7 +52,6 @@
 from .string_utils import stylize
 from .styles import Cmd2Style
 from .utils import (
-    CompletionMode,
     CustomCompletionSettings,
     Settable,
     categorize,
@@ -103,7 +102,6 @@
     "Cmd2Style",
     # Utilities
     'categorize',
-    'CompletionMode',
     'CustomCompletionSettings',
     'Settable',
     'set_default_str_sort_key',

cmd2/cmd2.py

@@ -4,6 +4,7 @@
 # nothing here should be considered part of the public API of this module
 
 INFINITY = float('inf')
+EOF = 'eof'
 
 # Used for command parsing, output redirection, completion, and word breaks. Do not change.
 QUOTES = ['"', "'"]

cmd2/constants.py

@@ -11,6 +11,7 @@
 )
 
 from prompt_toolkit import print_formatted_text
+from prompt_toolkit.application import get_app
 from prompt_toolkit.completion import (
     Completer,
     Completion,
@@ -95,7 +96,7 @@ def get_completions(self, document: Document, _complete_event: object) -> Iterab
         # and returning early, we trigger a new completion cycle where the quote
         # is already present, allowing for proper common prefix calculation.
         if completions._add_opening_quote and search_text_length > 0:
-            buffer = self.cmd_app.session.app.current_buffer
+            buffer = get_app().current_buffer
 
             buffer.cursor_left(search_text_length)
             buffer.insert_text(completions._quote_char)

cmd2/pt_utils.py

@@ -17,7 +17,6 @@
     MutableSequence,
 )
 from difflib import SequenceMatcher
-from enum import Enum
 from typing import (
     TYPE_CHECKING,
     Any,
@@ -733,24 +732,6 @@ def get_defining_class(meth: Callable[..., Any]) -> type[Any] | None:
     return cast(type, getattr(meth, '__objclass__', None))  # handle special descriptor objects
 
 
-class CompletionMode(Enum):
-    """Enum for what type of completion to perform in cmd2.Cmd.read_input()."""
-
-    # Completion will be disabled during read_input() call
-    # Use of custom up-arrow history supported
-    NONE = 1
-
-    # read_input() will complete cmd2 commands and their arguments
-    # cmd2's command line history will be used for up arrow if history is not provided.
-    # Otherwise use of custom up-arrow history supported.
-    COMMANDS = 2
-
-    # read_input() will complete based on one of its following parameters:
-    #     choices, choices_provider, completer, parser
-    # Use of custom up-arrow history supported
-    CUSTOM = 3
-
-
 class CustomCompletionSettings:
     """Used by cmd2.Cmd.complete() to complete strings other than command arguments."""
 

cmd2/utils.py

@@ -32,13 +32,6 @@ def do_basic_with_history(self, _) -> None:
         else:
             self.custom_history.append(input_str)
 
-    @cmd2.with_category(EXAMPLE_COMMANDS)
-    def do_commands(self, _) -> None:
-        """Call read_input the same way cmd2 prompt does to read commands."""
-        self.poutput("Tab completing and up-arrow history configured for commands")
-        with contextlib.suppress(EOFError):
-            self.read_input("> ", completion_mode=cmd2.CompletionMode.COMMANDS)
-
     @cmd2.with_category(EXAMPLE_COMMANDS)
     def do_custom_choices(self, _) -> None:
         """Call read_input to use custom history and choices."""
@@ -47,17 +40,16 @@ def do_custom_choices(self, _) -> None:
             input_str = self.read_input(
                 "> ",
                 history=self.custom_history,
-                completion_mode=cmd2.CompletionMode.CUSTOM,
                 choices=['choice_1', 'choice_2', 'choice_3'],
             )
         except EOFError:
             pass
         else:
             self.custom_history.append(input_str)
 
-    def choices_provider(self) -> list[str]:
+    def choices_provider(self) -> cmd2.Choices:
         """Example choices provider function."""
-        return ["from_provider_1", "from_provider_2", "from_provider_3"]
+        return cmd2.Choices.from_values(["from_provider_1", "from_provider_2", "from_provider_3"])
 
     @cmd2.with_category(EXAMPLE_COMMANDS)
     def do_custom_choices_provider(self, _) -> None:
@@ -67,7 +59,6 @@ def do_custom_choices_provider(self, _) -> None:
             input_str = self.read_input(
                 "> ",
                 history=self.custom_history,
-                completion_mode=cmd2.CompletionMode.CUSTOM,
                 choices_provider=ReadInputApp.choices_provider,
             )
         except EOFError:
@@ -80,9 +71,7 @@ def do_custom_completer(self, _) -> None:
         """Call read_input to use custom history and completer function."""
         self.poutput("Tab completing paths and using custom history")
         try:
-            input_str = self.read_input(
-                "> ", history=self.custom_history, completion_mode=cmd2.CompletionMode.CUSTOM, completer=cmd2.Cmd.path_complete
-            )
+            input_str = self.read_input("> ", history=self.custom_history, completer=cmd2.Cmd.path_complete)
             self.custom_history.append(input_str)
         except EOFError:
             pass
@@ -99,9 +88,7 @@ def do_custom_parser(self, _) -> None:
         self.poutput(parser.format_usage())
 
         try:
-            input_str = self.read_input(
-                "> ", history=self.custom_history, completion_mode=cmd2.CompletionMode.CUSTOM, parser=parser
-            )
+            input_str = self.read_input("> ", history=self.custom_history, parser=parser)
         except EOFError:
             pass
         else: