teddy-cli 0.1.8__tar.gz → 0.1.10__tar.gz

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: teddy-cli
-Version: 0.1.8
+Version: 0.1.10
 Summary: A local-first, file-based AI coding workflow that applies the UNIX philosophy to AI collaboration.
 Author: Raphael Atteritano
 License: AGPL-3.0-only

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "teddy-cli"
-version = "0.1.8"
+version = "0.1.10"
 description = "A local-first, file-based AI coding workflow that applies the UNIX philosophy to AI collaboration."
 authors = [{ name = "Raphael Atteritano" }]
 license = { text = "AGPL-3.0-only" }

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/src/teddy_executor/__main__.py

@@ -288,7 +288,7 @@ def update(
     elif experimental:
         typer.echo(f"A new experimental version {latest} is available.")
         typer.echo(
-            "To upgrade, run: uv tool install teddy-cli --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ --index-strategy unsafe-best-match"
+            "To upgrade, run: uv tool install teddy-cli --pre --force --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ --index-strategy unsafe-best-match"
         )
         typer.echo("To apply prompt updates, run: teddy init prompts")
     else:

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/src/teddy_executor/adapters/outbound/shell_adapter.py

@@ -108,7 +108,10 @@ class ShellAdapter(IShellExecutor):
             "shell": use_shell,
             "stdout": subprocess.PIPE,
             "stderr": subprocess.PIPE,
-            "stdin": subprocess.DEVNULL,
+            # Use PIPE instead of DEVNULL to provide a valid file descriptor for
+            # Python 3.14.2's C-level initialization. The pipe is closed
+            # immediately after Popen so children see EOF on reads.
+            "stdin": subprocess.PIPE,
             "text": True,
             "cwd": cwd,
             "env": env,
@@ -279,6 +282,14 @@ class ShellAdapter(IShellExecutor):
             kwargs = self._prepare_subprocess_kwargs(use_shell, cwd, env)
             process = self._popen(command_args, **kwargs)  # nosec
 
+            # Note: stdin is subprocess.PIPE (not DEVNULL) to provide a valid file
+            # descriptor for Python 3.14.2's C-level initialization. We do NOT close
+            # the stdin pipe here — communicate() internally flushes stdin before
+            # reading output, so closing stdin prematurely causes:
+            #   ValueError: I/O operation on closed file.
+            # communicate() handles the pipe EOF naturally when the subprocess finishes.
+            # No input data is written to the pipe, so children see empty stdin.
+
             try:
                 # Type cast is required because Mypy cannot infer str types when
                 # text=True is passed via **kwargs.

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/src/teddy_executor/core/services/execution_orchestrator.py

@@ -106,7 +106,10 @@ class ExecutionOrchestrator(IRunPlanUseCase):
             action_log = self._action_executor.handle_failed_action(action, str(e))
             captured_message = ""
 
-        if captured_message:
+        # Only propagate captured_message to user_request for non-MESSAGE actions.
+        # MESSAGE action replies belong in the action log (action_log.details),
+        # not in the User Request section of the report.
+        if captured_message and action.type.upper() != "MESSAGE":
             plan.metadata["user_request"] = captured_message
 
         should_halt = (

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/src/teddy_executor/core/services/session_orchestrator.py

@@ -83,12 +83,17 @@ def _print_header_bar(plan: Any, is_session: bool) -> None:
 
 
 def _print_user_message(
-    message: Optional[str], is_session: bool, plan: Optional[Any] = None
+    message: Optional[str],
+    is_session: bool,
+    plan: Optional[Any] = None,
+    action_logs: Optional[list] = None,
 ) -> None:
     """Print the user message after all actions execute.
 
-    If message is not provided (None) and plan is set, falls back to
-    plan.metadata.get("user_request").
+    Priority order:
+    1. Direct `message` parameter (from TUI `m` input).
+    2. `plan.metadata["user_request"]` (for non-MESSAGE action replies).
+    3. MESSAGE action replies from `action_logs` (for MESSAGE action responses).
 
     Only prints when is_session=True and message is non-empty.
     Output::
@@ -99,14 +104,30 @@ def _print_user_message(
     """
     if not is_session:
         return
-    # Resolve message from plan metadata if not provided
+
+    # Priority 1: Direct message parameter
+    # Priority 2: plan.metadata["user_request"] (non-MESSAGE actions, TUI 'm' input)
     if not message or not message.strip():
         if plan:
             meta_msg = getattr(plan, "metadata", {}).get("user_request") or ""
             if meta_msg.strip():
                 message = meta_msg.strip()
-        if not message or not message.strip():
-            return
+
+    # Priority 3: Fallback to MESSAGE action replies from action_logs
+    # When Bug 16 prevents MESSAGE replies from being stored in
+    # plan.metadata["user_request"], we still want to print them to
+    # terminal by reading directly from the action logs.
+    if (not message or not message.strip()) and action_logs:
+        for log in action_logs:
+            log_type = getattr(log, "action_type", "") or ""
+            log_details = getattr(log, "details", "") or ""
+            if log_type.upper() == "MESSAGE" and log_details.strip():
+                message = log_details.strip()
+                break  # Use the last MESSAGE action's reply
+
+    if not message or not message.strip():
+        return
+
     typer.secho("")
     typer.secho("User Message:")
     typer.secho(message.strip())
@@ -312,9 +333,15 @@ class SessionOrchestrator(IRunPlanUseCase):
             # Print user message after all actions executed (only for session mode)
             # Uses plan.metadata["user_request"] which is populated by _dispatch_single_action
             # when the user provides a reply during execution.
+            # Also extracts MESSAGE action replies from action_logs as a fallback
+            # (see Bug 17: Bug 16's guard prevents MESSAGE replies from being stored
+            # in plan.metadata["user_request"]).
             if is_session:
                 user_reply = plan.metadata.get("user_request", "") if plan else ""
-                _print_user_message(user_reply, is_session, plan=plan)
+                action_logs = report.action_logs if report else []
+                _print_user_message(
+                    user_reply, is_session, plan=plan, action_logs=action_logs
+                )
 
             # 4. Turn Transition
             if is_session and plan_path:

{teddy_cli-0.1.8 → teddy_cli-0.1.10}/src/teddy_executor/core/services/templates/execution_report.md.j2

@@ -268,7 +268,7 @@
 {% endfor %}
 {% endfor %}
 
-_**Hint:** NOW you MUST use READ on the most promising results)_
+_**Hint:** NOW you MUST use READ on the most promising results_
 {% endif %}
 
 {% if action_type == 'READ' and log.details and log.details is mapping and log.details.get('content') and params.get('lines') %}