codeplain 0.3.11.dev11__py3-none-any.whl → 0.3.11.dev13__py3-none-any.whl

{codeplain-0.3.11.dev11.dist-info → codeplain-0.3.11.dev13.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: codeplain
-Version: 0.3.11.dev11
+Version: 0.3.11.dev13
 Summary: Transform plain language specifications into working code
 License-File: LICENSE
 Classifier: Environment :: Console

{codeplain-0.3.11.dev11.dist-info → codeplain-0.3.11.dev13.dist-info}/RECORD

@@ -5,6 +5,7 @@ conformance_fix_journal.py,sha256=7wL3UOc3BWaUo5MCmiFbb3kIwOQ6iEpdQSyEmedo8Vk,94
 diff_utils.py,sha256=AjiQlqo5pRos_8hVXZo5yBurl5BzSrTMGrQv4dCtRCg,1198
 event_bus.py,sha256=g5XLwRfjrvTIcB6urjREuCJzZj5TA93hy8lKXLGXuCE,650
 file_utils.py,sha256=MNZ5li1ofIMOYB6FCrMa_UkX6nkGCgSZuaqzhqVc1Ok,11682
+git_preflight.py,sha256=vB2A8GyYz_iWr8jOE4POFLBHr3qihmedWuidspGdsrw,1509
 git_utils.py,sha256=gCUA6_9zg19f6k_0aVcRnIVlAJSBD0Vt6takx-0pKME,16547
 memory_management.py,sha256=CwIOQqF7Xo1sNfvP69zQRK_CpjinQD9dbolhRPl6lTk,2446
 metadata_utils.py,sha256=YO17o7IAma5TT1ZN0pTwrLVr_KxFQ8APv02KqKPpI5k,1641
@@ -23,7 +24,7 @@ plain2code_state.py,sha256=uePQlC29Pbhm1yXfdCFebONYdy6jo13gTTmXSwC11cQ,3168
 plain2code_telemetry.py,sha256=e4UCAoFklJMDWGQsL50i7nTLiOBy8gs86A0XO4WLgPo,4597
 plain2code_utils.py,sha256=unOxuqSQRpwG8YF6uoQhnZ3ZSTJRIUCPLdJ-1lRZR5M,1534
 plain_file.py,sha256=Kn8m_w6c04_RR-FBjAU05UcLOTDk4zcctpJXghYdZ8I,35757
-plain_modules.py,sha256=kUGR2mX908FEKOO8b2CJuk2rn9iRj8qyWGFMWTdfzoY,25977
+plain_modules.py,sha256=nXlmw0J2rKkuU1Y3A68LVtZ1Ffdc-D5iMjxaXSaSHTQ,25963
 plain_spec.py,sha256=7nb5lLJKHZ3uu9THkSXyJ5brIKkbGW0ooVfRclLdlzA,16062
 system_config.py,sha256=sA8wiPmBMtRsLwSQ40U5pVdLazAafbV-1Z19Yp0HiSQ,5314
 usage_summary.py,sha256=MA4WcVaAwz6IWCN4k_rgxVcwO4I157wN7BZlFi3h0R4,1282
@@ -79,6 +80,7 @@ tests/test_dry_run_link_validation.py,sha256=ivvYXQPqUkQ8glbFyUrdCninCxF2mg-rAsD
 tests/test_file_utils.py,sha256=R_ZIhrTtwlhHxgmbBtA8PFEU5-QB4Fi5skJ7PS2KzdU,3614
 tests/test_fix_unit_tests_conformance_context.py,sha256=IEsXK0xMLRuHmoQGBwrY0ENWVng2n-db_YzSO2qBfqM,9136
 tests/test_folder_path_resolution.py,sha256=K9v684xkPE0HOAsovg8uDixpZmaIk7ho-Wd3rTjKLF0,6560
+tests/test_git_preflight.py,sha256=6aQqInDzpk03pS-6UfkGQWjErrIXTEdHs5oeq3ZYzTw,3186
 tests/test_git_utils.py,sha256=h3rWhhs4XVRYqzVzdmj51siF_AYD-GkbZiefK2iGfWo,15614
 tests/test_headless_logging.py,sha256=soNESeotKQaU-SQyxtjpq35l-dLVsauZ_lJ-wk9i2mI,6419
 tests/test_imports.py,sha256=1ncbqQhtQW237kIjyV-Ufl7-R-PuduvoYFj2iS-DXW4,2608
@@ -99,22 +101,23 @@ tests/test_resolve_config_file.py,sha256=b87tDOvyzbsnE-QRWbZ97ou7NuyaAgskv-dbYBu
 tests/test_script_path_resolution.py,sha256=tiEqMevQRXr9YpfHN0xldOOQpLAUgnSqTzeZWkKHaUY,4156
 tests/test_system_config.py,sha256=gH5qs9q4lh5CljAdz28HUrJzuKqCvkZherFuv7enMak,2787
 tests/test_telemetry.py,sha256=LINvx1_Oz1OJ3ho2RcIcAA29IsNfUbkVVCbFaQcrBaA,8133
+tests/test_tui_components.py,sha256=a8MMZmJNU4EcT1GoOAGkb2gObK0h2AQvufbPCkTMkJk,3715
 tests/test_tui_usage.py,sha256=sfiAGTpRrT3MRr0-8DH3qjuECSxjODVgwzrK_K3iPXs,7959
 tests/test_usage_summary.py,sha256=Ps0kVzvmdZYeXrBR-cqkn3szTV5neYKs1PmmURngeIc,1079
 tests/e2e/conftest.py,sha256=5Dm67UTOE-M6hMb-mqZvwa6Ek_wmZgsR7fUv9ZEqCwY,4164
 tests/e2e/test_hello_world_python.py,sha256=2Cgs3-79dtQh5gVab257tFDHXTznefZeYqbTjlM4Nng,1632
 tests/e2e/test_hello_world_python_windows.py,sha256=ggynH_Wm4011M500zoPNNgvD-s4vZ3hvlorf1yARgQQ,2019
 tui/__init__.py,sha256=_fy7FowY0RkWdn4kC7XV2jgvQPCj-2gE8fNUb-Mny0A,60
-tui/components.py,sha256=XkC4Aq6VyTnFnBQs8PEHwVdAOczhm0ebKN92kiYMbq0,25791
+tui/components.py,sha256=eU0t2s5SFfTAjjCYpvh9c9eakpWFJwodBKkiFwz15DU,26200
 tui/models.py,sha256=KMjlWubmPD41GA-BjgiqkaV9v7tLJ0fuWKBDPmZj4JA,1493
 tui/plain2code_tui.py,sha256=8F7ZYRpmTiWRdTeQ63LPf2LGrreYjIYXvf_9XNSPyHs,17104
 tui/plain_module_render_choice_tui.py,sha256=XwBMMzyDat9FYqMkKdIV7kmIevddfwGA5Qk9kgy5sYY,12112
 tui/spinner.py,sha256=Ro6Gd9Przf-whuHqPRY6HwI0T57yJjyNPbhDbigZKZE,2471
 tui/state_handlers.py,sha256=zbUS_D9eU8qIwVpaxPEp9zL9bpg8gXsWNoUaqg9TLrQ,16664
 tui/styles.css,sha256=kiC7Og_2G1E7owoiZgV95-3_dSeQnsNj3qzaY04xkDg,6727
-tui/widget_helpers.py,sha256=ToO9MV2bF2Rp-XMNcjIYclnB_qmgO5sLw4UtQ2CyTMg,6652
-codeplain-0.3.11.dev11.dist-info/METADATA,sha256=7relBZ21L_WImUBplCaYHR9SrMI4HZDw--d4SR--2Hc,7257
-codeplain-0.3.11.dev11.dist-info/WHEEL,sha256=zOwg4jB6zX2kU910N-cMawjivD6tO8NEWvE12je1bVk,87
-codeplain-0.3.11.dev11.dist-info/entry_points.txt,sha256=oDZkBqu9WhtZApb_K6ia8-fn9aojwmAsgnKELceX5T4,46
-codeplain-0.3.11.dev11.dist-info/licenses/LICENSE,sha256=pCeKgQ1mXE5OmNUuKOfOZh1T1vqkeCUxghE7N8qgnzc,11345
-codeplain-0.3.11.dev11.dist-info/RECORD,,
+tui/widget_helpers.py,sha256=jitm2WHiKj-NVFL-hULxWzckPreRbUTl9OIZLN31-ek,6997
+codeplain-0.3.11.dev13.dist-info/METADATA,sha256=KVfLUeUZa4B9EB2D8iLyUz7RqGDLrDGSNrGCDwp1MIE,7257
+codeplain-0.3.11.dev13.dist-info/WHEEL,sha256=zOwg4jB6zX2kU910N-cMawjivD6tO8NEWvE12je1bVk,87
+codeplain-0.3.11.dev13.dist-info/entry_points.txt,sha256=oDZkBqu9WhtZApb_K6ia8-fn9aojwmAsgnKELceX5T4,46
+codeplain-0.3.11.dev13.dist-info/licenses/LICENSE,sha256=pCeKgQ1mXE5OmNUuKOfOZh1T1vqkeCUxghE7N8qgnzc,11345
+codeplain-0.3.11.dev13.dist-info/RECORD,,

git_preflight.py

@@ -0,0 +1,41 @@
+"""Verify a working git is available before GitPython is imported.
+
+GitPython probes for the git executable on import and raises from module scope
+if it is missing or broken, so the CLI would crash with a traceback before
+main() could explain it.
+"""
+
+import shutil
+import subprocess
+import sys
+
+from plain2code_console import console
+
+GIT_MISSING_MESSAGE = "git is not installed. Please install git and try again."
+GIT_BROKEN_MESSAGE = "git is installed but not working. Please repair your git installation and try again."
+MACOS_BROKEN_GIT_HINT = (
+    "This usually means the Command Line Tools are missing; install them with 'xcode-select --install'."
+)
+
+
+def git_runs(git_path):
+    """Whether `git version` succeeds. A git on PATH can still be unusable: on
+    macOS without the Command Line Tools /usr/bin/git is a stub that exits with
+    an xcrun error, and GIT_PYTHON_REFRESH=quiet does not suppress that."""
+    try:
+        return subprocess.run([git_path, "version"], capture_output=True, timeout=10).returncode == 0
+    except (OSError, subprocess.SubprocessError):
+        return False
+
+
+def require_git():
+    """Exit with a clean message unless a working git is available."""
+    git_path = shutil.which("git")
+    if git_path is None:
+        console.error(f"{GIT_MISSING_MESSAGE}\n")
+        sys.exit(1)
+
+    if not git_runs(git_path):
+        hint = f"\n{MACOS_BROKEN_GIT_HINT}" if sys.platform == "darwin" else ""
+        console.error(f"{GIT_BROKEN_MESSAGE}{hint}\n")
+        sys.exit(1)

plain_modules.py

@@ -6,18 +6,18 @@ import tempfile
 import zipfile
 from functools import cached_property
 
+from git_preflight import require_git
 from plain2code_exceptions import (
-    GitNotInstalledError,
     InvalidModuleArchiveError,
     MissingPreviousFunctionalitiesError,
     ModuleDoesNotExistError,
 )
 
-try:
-    from git import Repo
-    from git.exc import NoSuchPathError
-except ImportError:
-    raise GitNotInstalledError("git is not installed. Please install git and try again.")
+# Must run before GitPython is imported, which raises on a missing or broken git.
+require_git()
+
+from git import Repo
+from git.exc import NoSuchPathError
 
 import file_utils
 import git_utils

tests/test_git_preflight.py

@@ -0,0 +1,98 @@
+"""Regression tests for issue #133.
+
+When git is unusable the CLI must report a clean message and exit 1, never dump
+a traceback. The failure is at import time (GitPython raises from module scope
+before main() runs), so the CLI is run in a subprocess and asserted on what a
+user would actually see. sys.executable is launched by absolute path, so
+replacing PATH only controls which git, if any, can be found.
+"""
+
+import os
+import re
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+
+from git_preflight import GIT_BROKEN_MESSAGE, GIT_MISSING_MESSAGE
+
+REPO_ROOT = Path(__file__).resolve().parents[1]
+TRACEBACK_MARKER = "Traceback (most recent call last)"
+
+# The crash was in the import chain, so it hit every entry point equally.
+CLI_INVOCATIONS = [["--version"], ["--status"], ["does-not-exist.plain"]]
+
+
+def run_cli(args, path_dir=None):
+    """Run the CLI and return its exit code and output, stripped of styling and
+    with whitespace collapsed, so assertions survive console line wrapping."""
+    env = os.environ.copy()
+    # Drop GitPython overrides so the PATH search decides whether git is found.
+    env.pop("GIT_PYTHON_GIT_EXECUTABLE", None)
+    env.pop("GIT_PYTHON_REFRESH", None)
+    env.setdefault("CODEPLAIN_API_KEY", "dummy")
+    if path_dir is not None:
+        env["PATH"] = str(path_dir)
+
+    result = subprocess.run(
+        [sys.executable, "plain2code.py", *args],
+        cwd=REPO_ROOT,
+        env=env,
+        capture_output=True,
+        text=True,
+        timeout=120,
+    )
+    output = re.sub(r"\x1b\[[0-9;]*m", "", result.stdout + result.stderr)
+    return result.returncode, " ".join(output.split())
+
+
+@pytest.fixture
+def no_git_dir(tmp_path):
+    """A directory containing no git at all, used as the whole PATH."""
+    d = tmp_path / "no-git"
+    d.mkdir()
+    return d
+
+
+@pytest.fixture
+def broken_git_dir(tmp_path):
+    """A directory whose git resolves but always fails, used as the whole PATH.
+    This is the shape of macOS without the Command Line Tools."""
+    d = tmp_path / "broken-git"
+    d.mkdir()
+    if sys.platform == "win32":
+        (d / "git.bat").write_text("@echo off\nexit /b 127\n")
+    else:
+        git = d / "git"
+        git.write_text("#!/bin/sh\nexit 127\n")
+        git.chmod(0o755)
+    return d
+
+
+@pytest.mark.parametrize("args", CLI_INVOCATIONS)
+def test_missing_git_reports_cleanly(args, no_git_dir):
+    rc, output = run_cli(args, no_git_dir)
+
+    assert GIT_MISSING_MESSAGE in output, output
+    assert TRACEBACK_MARKER not in output, output
+    # GitPython's own advice about GIT_PYTHON_REFRESH must not reach the user.
+    assert "GIT_PYTHON_REFRESH" not in output, output
+    assert rc == 1, output
+
+
+@pytest.mark.parametrize("args", CLI_INVOCATIONS)
+def test_broken_git_reports_cleanly(args, broken_git_dir):
+    rc, output = run_cli(args, broken_git_dir)
+
+    assert GIT_BROKEN_MESSAGE in output, output
+    assert GIT_MISSING_MESSAGE not in output, output
+    assert TRACEBACK_MARKER not in output, output
+    assert rc == 1, output
+
+
+def test_cli_works_when_git_is_present():
+    rc, output = run_cli(["--version"])
+
+    assert rc == 0, output
+    assert "codeplain version" in output, output

tests/test_tui_components.py

@@ -0,0 +1,88 @@
+"""Regression tests: spec-derived text with square brackets must never be parsed as Textual markup."""
+
+import asyncio
+
+from textual.widgets import Static
+
+from event_bus import EventBus
+from plain2code_state import RunState
+from tui.components import ProgressItem, SubstateLine, TUIComponents
+from tui.models import Substate
+from tui.plain2code_tui import Plain2CodeTUI
+from tui.widget_helpers import display_error_message, display_success_message, update_progress_item_substates
+
+# Exact acceptance-test text that crashed the TUI with
+# "MarkupError: closing tag '[/#888888]' does not match any open tag":
+# the unterminated "[b" swallowed the timer's opening tag.
+UNTERMINATED_BRACKET_TEXT = (
+    "Rendering acceptance test: With a=1, b=2, c=3, d=4: ZRANK z c should reply 2, "
+    'ZREVRANK z c WITHSCORE should reply [1, "3"], ZRANK z zz should reply null, '
+    "ZCOUNT z (1 3 should reply 2, and ZLEXCOUNT z [b + should reply 3."
+)
+# Terminated brackets used to be silently eaten as a markup tag ("LPOP key ").
+TERMINATED_BRACKET_TEXT = "LPOP key [count] and RPOP key [count] reply one element"
+
+
+def _make_app(run_state: RunState, event_bus: EventBus) -> Plain2CodeTUI:
+    return Plain2CodeTUI(
+        event_bus=event_bus,
+        run_state=run_state,
+        on_ready=lambda: None,
+        render_id="test-render-id",
+        unittests_script=None,
+        conformance_tests_script="run_conformance_tests.sh",
+        prepare_environment_script=None,
+        state_machine_version="0.0.0",
+        css_path="styles.css",
+    )
+
+
+def test_substate_line_keeps_bracket_text_verbatim():
+    for text in (UNTERMINATED_BRACKET_TEXT, TERMINATED_BRACKET_TEXT):
+        line = SubstateLine(text, "    ", ProgressItem.PROCESSING)._format_line()
+        assert text in line.plain
+        assert line.plain.endswith("(0s)")
+        # Only the timer carries the muted style; the text itself is unstyled.
+        assert len(line.spans) == 1
+        assert line.plain[line.spans[0].start : line.spans[0].end] == "(0s)"
+
+
+def test_substate_with_brackets_renders_without_markup_error():
+    async def scenario():
+        event_bus = EventBus()
+        run_state = RunState(spec_filename="x.plain")
+        app = _make_app(run_state, event_bus)
+        async with app.run_test() as pilot:
+            widget_id = TUIComponents.FRID_PROGRESS_CONFORMANCE_TEST.value
+            update_progress_item_substates(
+                app, widget_id, [Substate(UNTERMINATED_BRACKET_TEXT), Substate(TERMINATED_BRACKET_TEXT)]
+            )
+            await pilot.pause()
+            await pilot.pause()
+
+            lines = app.query(".substate-line-text").results(Static)
+            rendered = [str(line.content) for line in lines]
+            assert any(UNTERMINATED_BRACKET_TEXT in text for text in rendered)
+            assert any(TERMINATED_BRACKET_TEXT in text for text in rendered)
+
+    asyncio.run(scenario())
+
+
+def test_status_messages_keep_brackets():
+    async def scenario():
+        event_bus = EventBus()
+        run_state = RunState(spec_filename="x.plain")
+        app = _make_app(run_state, event_bus)
+        async with app.run_test() as pilot:
+            status = app.query_one(f"#{TUIComponents.RENDER_STATUS_WIDGET.value}", Static)
+
+            display_success_message(app, "out/[weird] path")
+            await pilot.pause()
+            assert "rendering completed!" in str(status.content)
+            assert "generated code folder: out/[weird] path" in str(status.content)
+
+            display_error_message(app, "Error: [Errno 2] No such file: run_tests.sh [b")
+            await pilot.pause()
+            assert "[Errno 2] No such file: run_tests.sh [b" in str(status.content)
+
+    asyncio.run(scenario())

tui/components.py

@@ -3,6 +3,7 @@ from typing import Literal, Optional
 
 from rich.markup import escape
 from textual.containers import Horizontal, Vertical, VerticalScroll
+from textual.content import Content
 from textual.message import Message
 from textual.timer import Timer
 from textual.widgets import Button, Static
@@ -71,7 +72,7 @@ class CustomFooter(Horizontal):
         self._footer_text_widget = Static(footer_rendering_text, classes="custom-footer-text")
         yield self._footer_text_widget
         if self.render_id:
-            yield Static(f"render id: {self.render_id} ", classes="custom-footer-render-id")
+            yield Static(f"render id: {self.render_id} ", classes="custom-footer-render-id", markup=False)
 
     def update_footer_state(self, state: Literal["rendering", "pausing", "paused", "finished"]) -> None:
         self.remove_class("footer-state-default")
@@ -201,9 +202,12 @@ class SubstateLine(Horizontal):
     def _format_timer(self) -> str:
         return format_duration_hms(self._seconds_elapsed)
 
-    def _format_line(self) -> str:
+    def _format_line(self) -> Content:
+        # The substate text is arbitrary (acceptance-test text from the spec, module names, ...)
+        # and may contain square brackets, so it must never go through the markup parser.
+        # Build the line as Content and style only the timer part.
         timer = self._format_timer()
-        return f"{self.indent}  └ {self.text} [#888888]({timer})[/#888888]"
+        return Content.assemble(f"{self.indent}  └ {self.text} ", (f"({timer})", "#888888"))
 
     def _refresh_timer(self) -> None:
         try:
@@ -232,7 +236,7 @@ class ProgressItem(Vertical):
         # Main row with status and description
         with Horizontal(id=f"{self.id}-main-row", classes="progress-main-row"):
             yield Static(self._get_status_text(self.PENDING), classes=f"status {self.PENDING}")
-            yield Static(self.initial_text, classes="description")
+            yield Static(self.initial_text, classes="description", markup=False)
         # Substates container (full width, aligned to left)
         yield Vertical(id=f"{self.id}-substates", classes="substates-container")
 
@@ -367,8 +371,8 @@ class RenderingInfoBox(Vertical):
         self._refresh_content()
 
     def compose(self):
-        self.module_widget = Static(self.module_text, classes="rendering-info-row")
-        self.functionality_widget = Static(self.functionality_text, classes="rendering-info-row")
+        self.module_widget = Static(self.module_text, classes="rendering-info-row", markup=False)
+        self.functionality_widget = Static(self.functionality_text, classes="rendering-info-row", markup=False)
         yield Static("module status", classes="rendering-info-title")
         with Vertical(classes="rendering-info-box"):
             yield self.module_widget
@@ -430,9 +434,9 @@ class TestScriptsContainer(Vertical):
     def compose(self):
         yield Static("testing status", classes="test-scripts-title")
         with Vertical(classes="test-scripts-box"):
-            self.unit_widget = Static(self.unit_test_text, classes="test-script-row")
-            self.conformance_widget = Static(self.conformance_test_text, classes="test-script-row")
-            self.testing_widget = Static(self.testing_env_text, classes="test-script-row")
+            self.unit_widget = Static(self.unit_test_text, classes="test-script-row", markup=False)
+            self.conformance_widget = Static(self.conformance_test_text, classes="test-script-row", markup=False)
+            self.testing_widget = Static(self.testing_env_text, classes="test-script-row", markup=False)
             yield self.unit_widget
             yield self.conformance_widget
             yield self.testing_widget

tui/widget_helpers.py

@@ -2,6 +2,7 @@
 
 from datetime import datetime
 
+from textual.content import Content
 from textual.css.query import NoMatches
 from textual.widgets import Static
 
@@ -89,9 +90,14 @@ def display_success_message(tui, rendered_code_path: str):
         rendered_code_path: The path to the rendered code
     """
 
-    message = (
-        f"[#79FC96]✓ rendering completed![/#79FC96] [#888888](press enter to exit)[/#888888]\n"
-        f"[#888888]generated code folder: {rendered_code_path}[/#888888] "
+    # rendered_code_path is user-controlled and may contain square brackets, so build the
+    # message as Content instead of interpolating it into a markup string.
+    message = Content.assemble(
+        ("✓ rendering completed!", "#79FC96"),
+        " ",
+        ("(press enter to exit)", "#888888"),
+        "\n",
+        (f"generated code folder: {rendered_code_path} ", "#888888"),
     )
 
     widget: Static = tui.query_one(f"#{TUIComponents.RENDER_STATUS_WIDGET.value}", Static)
@@ -120,7 +126,8 @@ def transition_frid_progress(tui, from_status: str | None, to_status: str):
 def display_error_message(tui, error_message: str):
     widget: Static = tui.query_one(f"#{TUIComponents.RENDER_STATUS_WIDGET.value}", Static)
     widget.add_class("error")
-    widget.update(error_message)
+    # Error text may contain square brackets (paths, tracebacks); keep it out of the markup parser.
+    widget.update(Content(error_message))
 
 
 def display_usage_summary(tui, functionalities: int, render_time_seconds: float) -> None: