teddy-cli 0.1.17.dev1329__tar.gz → 0.1.17.dev1330__tar.gz

{teddy_cli-0.1.17.dev1329 → teddy_cli-0.1.17.dev1330}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: teddy-cli
-Version: 0.1.17.dev1329
+Version: 0.1.17.dev1330
 Summary: The local-first, test-driven agentic engineering harness. Because you should own the means of development.
 Author: Raphael Atteritano
 License: AGPL-3.0-only

{teddy_cli-0.1.17.dev1329 → teddy_cli-0.1.17.dev1330}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "teddy-cli"
-version = "0.1.17.dev1329"
+version = "0.1.17.dev1330"
 description = "The local-first, test-driven agentic engineering harness. Because you should own the means of development."
 readme = "README.md"
 keywords = [

{teddy_cli-0.1.17.dev1329 → teddy_cli-0.1.17.dev1330}/src/teddy_executor/adapters/outbound/local_file_system_adapter.py

@@ -260,13 +260,16 @@ class LocalFileSystemAdapter(IFileSystemManager):
     def create_file(self, path: str, content: str, overwrite: bool = False) -> None:
         """
         Creates a new file with the given content.
+
+        Newline-deterministic: ``newline=""`` keeps LF line endings verbatim
+        on every OS, matching the write_file and edit_file contracts.
         """
         try:
             file_path = self._resolve_path(path)
             file_path.parent.mkdir(parents=True, exist_ok=True)
 
             mode = "w" if overwrite else "x"
-            with open(file_path, mode, encoding="utf-8") as f:
+            with open(file_path, mode, encoding="utf-8", newline="") as f:
                 f.write(content)
         except FileExistsError as e:
             # Raise a domain-specific exception to conform to the port's contract.
@@ -327,6 +330,11 @@ class LocalFileSystemAdapter(IFileSystemManager):
     ) -> list[float]:
         """
         Modifies an existing file by applying a list of find-and-replace blocks.
+
+        Newline-deterministic: ``newline=""`` on the write disables platform
+        newline translation, so the simulated content is emitted verbatim on
+        every OS. This prevents Windows from doubling CRLF sequences that a
+        clipboard plan carries into the REPLACE block.
         """
         file_path = self._resolve_path(path)
         content = file_path.read_text(encoding="utf-8")
@@ -344,5 +352,5 @@ class LocalFileSystemAdapter(IFileSystemManager):
             match_all=match_all,
         )
 
-        file_path.write_text(new_content, encoding="utf-8")
+        file_path.write_text(new_content, encoding="utf-8", newline="")
         return scores

{teddy_cli-0.1.17.dev1329 → teddy_cli-0.1.17.dev1330}/src/teddy_executor/core/services/markdown_plan_parser.py

@@ -60,32 +60,65 @@ class MarkdownPlanParser(IPlanParser):
             ),
         }
 
-    def parse(self, plan_content: str, plan_path: Optional[str] = None) -> Plan:
-        """
-        Parses the specified Markdown plan string into a structured Plan object.
+    def _normalize_plan_content(self, plan_content: str) -> str:
+        """Normalize raw plan text into canonical Markdown before parsing.
+
+        Converts Windows clipboard CRLF endings to LF, trims trailing
+        whitespace, strips any preamble before the first heading, and
+        normalizes the H1 title.
         """
-        from mistletoe.block_token import (
-            Document,
-        )
+        # Normalize Windows clipboard line endings to LF so fence content and
+        # extracted message sections never carry stray carriage returns.
+        normalized_content = plan_content.replace("\r\n", "\n")
 
         # Trim trailing whitespace to prevent mistletoe from
         # interpreting trailing indentation as an unexpected code block.
         # We keep leading whitespace for potential Markdown significance (though rare at top-level).
-        clean_content = plan_content.rstrip()
+        clean_content = normalized_content.rstrip()
 
         if not clean_content:
             raise InvalidPlanError("Plan content cannot be empty.")
 
-        # Strip preamble (text before the first # heading at start of a line)
-        # Use MULTILINE so ^ matches start of any line. Allow optional leading whitespace
-        # before # because Markdown permits up to 3 spaces before heading markers.
+        # Strip preamble (text before the first # heading at start of a line).
+        # MULTILINE lets ^ match the start of any line, and Markdown permits up
+        # to 3 leading spaces before a heading marker.
         h1_match = re.search(r"^[ \t]*#", clean_content, re.MULTILINE)
         if h1_match and h1_match.start() > 0:
             clean_content = clean_content[h1_match.start() :]
 
-        # Normalize H1 heading on the first line (e.g., #Title -> # Title)
-        # This runs after preamble stripping so it always targets the heading line
-        clean_content = normalize_headings(clean_content)
+        # Normalize the H1 heading (e.g. #Title -> # Title) after preamble
+        # stripping so it always targets the heading line.
+        return normalize_headings(clean_content)
+
+    @staticmethod
+    def _is_session_plan(plan_path: Optional[str]) -> bool:
+        """Return True when the plan path points inside a session directory."""
+        if not plan_path:
+            return False
+        normalized_path = plan_path.replace("\\", "/").lower()
+        return ".teddy/sessions/" in normalized_path
+
+    def _write_back_session_plan(self, plan_path: str, clean_content: str) -> None:
+        """Persist normalized content to a session plan file when it changed."""
+        from pathlib import Path
+
+        path_obj = Path(plan_path)
+        try:
+            current_disk = path_obj.read_text(encoding="utf-8")
+        except Exception:
+            current_disk = None
+        if current_disk is not None and current_disk.rstrip() != clean_content:
+            path_obj.write_text(clean_content, encoding="utf-8", newline="")
+
+    def parse(self, plan_content: str, plan_path: Optional[str] = None) -> Plan:
+        """
+        Parses the specified Markdown plan string into a structured Plan object.
+        """
+        from mistletoe.block_token import (
+            Document,
+        )
+
+        clean_content = self._normalize_plan_content(plan_content)
 
         processed_content = self._preprocessor.process(clean_content)
         doc = Document(processed_content)
@@ -106,10 +139,7 @@ class MarkdownPlanParser(IPlanParser):
                 stream, clean_content, section_heading, doc
             )
 
-            is_session = False
-            if plan_path:
-                normalized_path = plan_path.replace("\\", "/").lower()
-                is_session = ".teddy/sessions/" in normalized_path
+            is_session = self._is_session_plan(plan_path)
 
             plan = Plan(
                 title=title,
@@ -122,17 +152,8 @@ class MarkdownPlanParser(IPlanParser):
                 raw_content=clean_content,
             )
 
-            # Write corrected content back to source file if it came from a session file path
-            if plan_path and is_session:
-                from pathlib import Path
-
-                path_obj = Path(plan_path)
-                try:
-                    current_disk = path_obj.read_text(encoding="utf-8")
-                except Exception:
-                    current_disk = None
-                if current_disk is not None and current_disk.rstrip() != clean_content:
-                    path_obj.write_text(clean_content, encoding="utf-8", newline="")
+            if is_session and plan_path:
+                self._write_back_session_plan(plan_path, clean_content)
 
             return plan
         except InvalidPlanError as e: