codeplain 0.3.4__tar.gz → 0.3.5__tar.gz

codeplain-0.3.5/.claude/settings.local.json

@@ -0,0 +1,10 @@
+{
+  "permissions": {
+    "allow": [
+      "Read(//Users/nero/DevelCodeplain/**)",
+      "Read(//Users/nero/DevelCodeplain/plain2code_rest_api/**)",
+      "Bash(/private/tmp/claude-501/-Users-nero-DevelCodeplain-codeplain/8382c3ca-47e9-4519-afd2-160102a63874/scratchpad/tvenv/bin/python /Users/nero/DevelCodeplain/codeplain/plain2code.py hello_world_python.plain --headless)",
+      "Bash(echo \"EXIT=$?\")"
+    ]
+  }
+}

{codeplain-0.3.4 → codeplain-0.3.5}/CLAUDE.md

@@ -179,7 +179,7 @@ black . --check && isort . --check-only && flake8 . && mypy . --check-untyped-de
 - `plain2code_logger.py` - Custom logging with elapsed time timestamps
 - `plain2code_console.py` - Rich console wrapper with custom styles
 - `plain2code_state.py` - Runtime state (render ID, counters, timing)
-- `standard_template_library/` - Built-in code templates
+- `standard_template_library/` - Built-in `.plain` templates (git subtree from `plainlang-examples`)
 - `examples/` - Sample `.plain` projects
 
 ### Testing Architecture
@@ -223,6 +223,26 @@ Two logging modes:
 ### Running Plain2Code from Another Directory
 The tool can be run from any directory by providing an absolute or relative path to the `.plain` file. All paths (build folder, log file, templates) are resolved relative to the `.plain` file's directory, not the current working directory, unless explicitly overridden via CLI arguments.
 
+### Standard Template Library
+
+`standard_template_library/` is a git subtree sourced from the `plainlang-examples` repository. It is **not** updated automatically — you must pull changes manually when the templates change upstream:
+
+```bash
+git subtree pull --prefix=standard_template_library git@github.com:Codeplain-ai/plainlang-examples.git subtree/standard-template-library --squash
+```
+
+The subtree does not track a branch pointer. The branch (`main` above) is just an argument passed at pull time — git uses the `git-subtree-split` hash embedded in the commit history to determine what's new since the last sync.
+
+**Important:** You must pull from the `subtree/standard-template-library` branch, not `main`. This is a split branch containing only the `standard_template_library/` subdirectory contents at root level. Pulling from `main` would bring in the entire `plainlang-examples` repo.
+
+If the split branch is outdated, regenerate it first in `plainlang-examples`:
+
+```bash
+# In plainlang-examples/
+git subtree split --prefix=standard_template_library -b subtree/standard-template-library
+git push origin subtree/standard-template-library
+```
+
 ### Windows Support
 Windows users must use WSL (Windows Subsystem for Linux). The codebase has some platform-specific script handling (`.ps1` for Windows, `.sh` for Unix).
 

{codeplain-0.3.4 → codeplain-0.3.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeplain
-Version: 0.3.4
+Version: 0.3.5
 Summary: Transform plain language specifications into working code
 License-File: LICENSE
 Classifier: Environment :: Console
@@ -12,20 +12,20 @@ Requires-Dist: gitpython==3.1.50
 Requires-Dist: mistletoe==1.3.0
 Requires-Dist: networkx==3.6.1
 Requires-Dist: python-frontmatter==1.3.0
-Requires-Dist: python-liquid2==0.3.0
+Requires-Dist: python-liquid2==0.4.0
 Requires-Dist: pyyaml==6.0.3
-Requires-Dist: requests==2.34.0
+Requires-Dist: requests==2.34.2
 Requires-Dist: rich==15.0.0
-Requires-Dist: sentry-sdk==2.62.0
+Requires-Dist: sentry-sdk==2.63.0
 Requires-Dist: textual>=7.5.0
-Requires-Dist: tiktoken==0.12.0
+Requires-Dist: tiktoken==0.13.0
 Requires-Dist: transitions==0.9.3
 Provides-Extra: dev
 Requires-Dist: black==26.3.1; extra == 'dev'
 Requires-Dist: flake8==7.3.0; extra == 'dev'
 Requires-Dist: isort==5.13.2; extra == 'dev'
-Requires-Dist: mypy==1.11.2; extra == 'dev'
-Requires-Dist: pytest==8.3.5; extra == 'dev'
+Requires-Dist: mypy==2.1.0; extra == 'dev'
+Requires-Dist: pytest==9.1.1; extra == 'dev'
 Description-Content-Type: text/markdown
 
 # Codeplain plain2code renderer

codeplain-0.3.5/_version.py

@@ -0,0 +1 @@
+__version__ = "0.3.5"

{codeplain-0.3.4 → codeplain-0.3.5}/file_utils.py

@@ -8,8 +8,9 @@ from liquid2.exceptions import UndefinedError
 
 import plain_spec
 from plain2code_console import console
-from plain2code_exceptions import UnsupportedResourceType
+from plain2code_exceptions import UnsupportedBase64Content, UnsupportedResourceType
 from plain2code_nodes import Plain2CodeIncludeTag, Plain2CodeLoaderMixin
+from plain2code_utils import find_large_base64_blob
 from plain_modules import CODEPLAIN_MEMORY_SUBFOLDER, CODEPLAIN_METADATA_FOLDER
 
 BINARY_FILE_EXTENSIONS = [".pyc"]
@@ -157,7 +158,7 @@ def store_response_files(target_folder, response_files, existing_files):
 
         os.makedirs(os.path.dirname(full_file_name), exist_ok=True)
 
-        with open(full_file_name, "w") as f:
+        with open(full_file_name, "w", encoding="utf-8") as f:
             f.write(response_files[file_name])
 
         if file_name not in existing_files:
@@ -206,6 +207,16 @@ def load_linked_resources(template_dirs: list[str], resources_list, module_name:
                 Please ensure that the resource exists in one of these locations, or specify the correct --template-dir if using custom templates.
                 """)
 
+        blob = find_large_base64_blob(content)
+        if blob is not None:
+            raise UnsupportedBase64Content(
+                f"Referenced resource '{file_name}' in module '{module_name}' contains a large "
+                f"base64-encoded blob ({len(blob)} characters), such as an embedded image. Inline "
+                "base64 data is not supported. Remove the data from the resource. "
+                "If the data should be used by the end software, "
+                "save the data to a separate file and include the file path in the specification without it being a reference file."
+            )
+
         linked_resources[file_name] = content
 
     return linked_resources

{codeplain-0.3.4 → codeplain-0.3.5}/plain2code.py

@@ -1,4 +1,3 @@
-import importlib.resources
 import logging
 import logging.config
 import os
@@ -37,6 +36,7 @@ from plain2code_exceptions import (
     PlainSyntaxError,
     RenderCancelledError,
     RenderingCreditBalanceTooLow,
+    UnsupportedBase64Content,
     UnsupportedResourceType,
 )
 from plain2code_logger import (
@@ -54,7 +54,7 @@ from system_config import system_config
 from tui.plain2code_tui import Plain2CodeTUI
 from tui.plain_module_render_choice_tui import PlainModuleRenderChoiceTUI
 
-DEFAULT_TEMPLATE_DIRS = importlib.resources.files("standard_template_library")
+DEFAULT_TEMPLATE_DIRS = "standard_template_library"
 RENDER_THREAD_SHUTDOWN_TIMEOUT = 0.7
 
 # Exceptions that represent expected, user-facing error conditions. They are
@@ -74,6 +74,7 @@ EXPECTED_EXCEPTIONS = (
     NetworkConnectionError,
     ModuleDoesNotExistError,
     UnsupportedResourceType,
+    UnsupportedBase64Content,
     GitNotInstalledError,
     SystemExit,
 )
@@ -139,8 +140,8 @@ def setup_logging(
     return logging.getLevelName(configured_log_level)
 
 
-def _check_connection(codeplainAPI: codeplain_api.CodeplainAPI):
-    """Check API connectivity and validate API key and client version."""
+def _check_connection(codeplainAPI: codeplain_api.CodeplainAPI) -> Optional[str]:
+    """Check API connectivity, validate API key and client version, and return the user email."""
     response = codeplainAPI.connection_check(system_config.client_version)
 
     if not response.get("api_key_valid", False):
@@ -157,33 +158,58 @@ def _check_connection(codeplainAPI: codeplain_api.CodeplainAPI):
             "Please update using: uv tool upgrade codeplain"
         )
 
+    return response.get("user_email")
 
-def render(args, run_state: RunState, event_bus: EventBus, default_log_level: str = "INFO"):  # noqa: C901
-    template_dirs = file_utils.get_template_directories(args.filename, args.template_dir, DEFAULT_TEMPLATE_DIRS)
 
+def warn_if_acceptance_tests_without_conformance_script(plain_module, args) -> None:
+    """Warn when any loaded module (including required modules) defines acceptance tests
+    but no conformance tests script is configured.
+
+    Acceptance tests are treated as conformance tests, so a conformance tests script is required
+    to actually run them. Without it, the acceptance tests cannot be executed.
+    """
+    if args.conformance_tests_script:
+        return
+
+    module_names_with_acceptance_tests = [
+        module.module_name
+        for module in plain_module.all_required_modules + [plain_module]
+        if plain_spec.has_acceptance_tests(module.plain_source)
+    ]
+    if not module_names_with_acceptance_tests:
+        return
+
+    module_names = ", ".join(module_names_with_acceptance_tests)
+    console.warning(
+        f"Acceptance tests were found ({module_names}) but no conformance tests script is configured. "
+        "Acceptance tests are treated as conformance tests and require a conformance tests script "
+        "(--conformance-tests-script or 'conformance_tests_script' in config) to be executed."
+    )
+
+
+def render(  # noqa: C901
+    plain_module: plain_modules.PlainModule,
+    args,
+    run_state: RunState,
+    event_bus: EventBus,
+    default_log_level: str = "INFO",
+):
     # Compute render range from either --render-range or --render-from
     render_range = None
     if args.render_range or args.render_from:
-        # Parse the plain file to get the plain_source for FRID extraction
-        _, plain_source, _ = plain_file.plain_file_parser(args.filename, template_dirs)
-        render_range = plain_spec.compute_render_range(args, plain_source)
+        render_range = plain_spec.compute_render_range(args, plain_module.plain_source)
 
     codeplainAPI = codeplain_api.CodeplainAPI(args.api_key, console)
     assert args.api is not None and args.api != "", "API URL is required"
     codeplainAPI.api_url = args.api
 
-    _check_connection(codeplainAPI)
+    run_state.user_email = _check_connection(codeplainAPI)
 
     stop_event = threading.Event()
     enter_pause_event = threading.Event()
     signal.signal(signal.SIGTERM, lambda _signum, _frame: stop_event.set())
 
-    plain_module = plain_modules.PlainModule(
-        args.filename,
-        args.build_folder,
-        args.conformance_tests_folder,
-        template_dirs,
-    )
+    warn_if_acceptance_tests_without_conformance_script(plain_module, args)
 
     render_choice = None
     if render_range is None:
@@ -308,28 +334,38 @@ def main():  # noqa: C901
             console.error(f"Error fetching status: {str(e)}")
         return
 
-    # Handle early-exit flags before heavy initialization
-    if args.dry_run or args.full_plain:
-        template_dirs = file_utils.get_template_directories(args.filename, args.template_dir, DEFAULT_TEMPLATE_DIRS)
+    template_dirs = file_utils.get_template_directories(args.filename, args.template_dir, DEFAULT_TEMPLATE_DIRS)
 
+    # Handle full plain early-exit (raw text dump; does not require a parsed module).
+    if args.full_plain:
         try:
-            if args.full_plain:
-                module_name = Path(args.filename).stem
-                plain_source = plain_file.read_module_plain_source(module_name, template_dirs)
-                [full_plain_source, _] = file_utils.get_loaded_templates(template_dirs, plain_source)
-                console.info("Full plain text:\n")
-                console.info(full_plain_source)
-                return
-
-            if args.dry_run:
-                console.info("Printing dry run output...\n")
-                _, plain_source_tree, _ = plain_file.plain_file_parser(args.filename, template_dirs)
-                render_range = plain_spec.compute_render_range(args, plain_source_tree)
-                print_dry_run_output(plain_source_tree, render_range)
-                return
+            module_name = Path(args.filename).stem
+            plain_source = plain_file.read_module_plain_source(module_name, template_dirs)
+            [full_plain_source, _] = file_utils.get_loaded_templates(template_dirs, plain_source)
+            console.info("Full plain text:\n")
+            console.info(full_plain_source)
         except Exception as e:
             console.error(f"Error: {str(e)}")
-            return
+        return
+
+    # Parse the plain file (and its required modules) once; reused by dry-run and rendering.
+    try:
+        plain_module = plain_modules.PlainModule(
+            args.filename,
+            args.build_folder,
+            args.conformance_tests_folder,
+            template_dirs,
+        )
+    except Exception as e:
+        console.error(f"Error: {str(e)}")
+        return
+
+    if args.dry_run:
+        console.info("Printing dry run output...\n")
+        render_range = plain_spec.compute_render_range(args, plain_module.plain_source)
+        print_dry_run_output(plain_module.plain_source, render_range)
+        warn_if_acceptance_tests_without_conformance_script(plain_module, args)
+        return
 
     event_bus = EventBus()
 
@@ -357,7 +393,7 @@ def main():  # noqa: C901
             raise MissingAPIKey(
                 "Your API key is required. Please set the CODEPLAIN_API_KEY environment variable or provide it with the --api-key argument.\n"
             )
-        render(args, run_state, event_bus, default_log_level)
+        render(plain_module, args, run_state, event_bus, default_log_level)
     except BaseException as e:
         if isinstance(e, KeyboardInterrupt):
             error_message = "Keyboard interrupt"

{codeplain-0.3.4 → codeplain-0.3.5}/plain2code_exceptions.py

@@ -29,6 +29,11 @@ class PlainSyntaxError(Exception):
     pass
 
 
+class UnsupportedBase64Content(Exception):
+
+    pass
+
+
 class InternalClientError(Exception):
     pass
 

{codeplain-0.3.4 → codeplain-0.3.5}/plain2code_state.py

@@ -27,6 +27,7 @@ class RunState:
         self.current_module: Optional[str] = None
         self.current_frid: Optional[str] = None
         self.current_render_state: Optional[str] = None
+        self.user_email: Optional[str] = None
 
     def increment_call_count(self):
         self.call_count += 1

{codeplain-0.3.4 → codeplain-0.3.5}/plain2code_telemetry.py

@@ -98,6 +98,8 @@ def capture_crash(exc_info, run_state: Optional[RunState], args) -> bool:
                 scope.set_tag("render_state", run_state.current_render_state)
                 scope.set_tag("current_module", run_state.current_module)
                 scope.set_tag("current_frid", run_state.current_frid)
+                if run_state.user_email:
+                    scope.set_user({"email": run_state.user_email})
             scope.set_tag("headless", bool(getattr(args, "headless", False)))
             scope.set_tag("unittests_script_provided", bool(getattr(args, "unittests_script", None)))
             scope.set_tag("conformance_tests_script_provided", bool(getattr(args, "conformance_tests_script", None)))

{codeplain-0.3.4 → codeplain-0.3.5}/plain2code_utils.py

@@ -1,3 +1,21 @@
+import re
+from typing import Optional
+
+#
+MAX_BASE64_BLOB_LENGTH = 8192
+
+# Matches a long contiguous base64 / base64url run, optionally preceded by a data: URI header.
+_BASE64_BLOB_PATTERN = re.compile(
+    r"(?:data:[\w.+-]+/[\w.+-]+;base64,)?[A-Za-z0-9+/_-]{%d,}={0,2}" % MAX_BASE64_BLOB_LENGTH
+)
+
+
+def find_large_base64_blob(text: str) -> Optional[str]:
+    """Return the first contiguous base64 blob at or above the threshold, or None."""
+    match = _BASE64_BLOB_PATTERN.search(text)
+    return match.group(0) if match else None
+
+
 def format_duration_hms(total_seconds: int) -> str:
     """Format a duration in seconds as hours, minutes, and seconds (e.g. ``1h 2m 3.45s``, ``45.67s``)."""
     if total_seconds < 0:

{codeplain-0.3.4 → codeplain-0.3.5}/plain_file.py

@@ -17,8 +17,9 @@ from mistletoe.utils import traverse
 import concept_utils
 import file_utils
 import plain_spec
-from plain2code_exceptions import ModuleDoesNotExistError, PlainSyntaxError
+from plain2code_exceptions import ModuleDoesNotExistError, PlainSyntaxError, UnsupportedBase64Content
 from plain2code_nodes import Plain2CodeIncludeTag, Plain2CodeLoaderMixin
+from plain2code_utils import find_large_base64_blob
 
 RESOURCE_MARKER = "[resource]"
 
@@ -203,6 +204,26 @@ def _is_acceptance_test_heading(token) -> tuple[bool, str | None]:
     return False, None
 
 
+def _find_acceptance_test_heading(token):
+    """
+    Recursively search a token tree for an `acceptance tests` heading.
+
+    Returns the heading token if one is found anywhere in the subtree, otherwise None.
+    Used to detect acceptance tests nested under sections other than functional specs.
+    """
+    is_acceptance_test_heading, _ = _is_acceptance_test_heading(token)
+    if is_acceptance_test_heading:
+        return token
+
+    if getattr(token, "children", None):
+        for child in token.children:
+            found = _find_acceptance_test_heading(child)
+            if found is not None:
+                return found
+
+    return None
+
+
 def _process_single_acceptance_test_requirement(functional_requirement: mistletoe.block_token.ListItem):
     """
     Process a single functionality to extract acceptance tests.
@@ -473,6 +494,11 @@ def parse_plain_source(  # noqa: C901
 
             specification_heading = token.children[0].children[0].children[0].content
 
+            if specification_heading == plain_spec.ACCEPTANCE_TEST_HEADING:
+                raise PlainSyntaxError(
+                    f"Plain syntax error: Syntax error at line {token.line_number}: {plain_spec.ACCEPTANCE_TEST_HEADING} heading should be nested under specific functional spec."
+                )
+
             if specification_heading not in plain_spec.ALLOWED_SPECIFICATION_HEADINGS:
                 raise PlainSyntaxError(
                     f"Plain syntax error: Syntax error at line {token.line_number}: Invalid specification heading (`{specification_heading}`). Allowed headings: {', '.join(plain_spec.ALLOWED_SPECIFICATION_HEADINGS)}"
@@ -511,6 +537,22 @@ def parse_plain_source(  # noqa: C901
                 f"Plain syntax error: Syntax error at line {token.line_number}: Invalid source structure  (`{token_text}`)"
             )
 
+    # Acceptance tests are only allowed nested under functional specs. Reject them when
+    # they are nested under any other section (e.g. definitions, implementation reqs, test reqs).
+    for non_functional_heading in (
+        plain_spec.DEFINITIONS,
+        plain_spec.NON_FUNCTIONAL_REQUIREMENTS,
+        plain_spec.TEST_REQUIREMENTS,
+    ):
+        section = plain_source.get(non_functional_heading)
+        if section is None:
+            continue
+        nested_acceptance_test_heading = _find_acceptance_test_heading(section)
+        if nested_acceptance_test_heading is not None:
+            raise PlainSyntaxError(
+                f"Plain syntax error: Syntax error at line {nested_acceptance_test_heading.line_number}: {plain_spec.ACCEPTANCE_TEST_HEADING} heading should be nested under specific functional spec."
+            )
+
     if plain_source[plain_spec.DEFINITIONS] is not None:
         with PlainRenderer() as renderer:
             for token in plain_source[plain_spec.DEFINITIONS].children:
@@ -536,6 +578,16 @@ def read_module_plain_source(module_name: str, template_dirs: list[str]) -> str:
     plain_source_text = file_utils.open_from(template_dirs, module_name + PLAIN_SOURCE_FILE_EXTENSION)
     if plain_source_text is None:
         raise ModuleDoesNotExistError(f"Module does not exist ({module_name}).")
+
+    blob = find_large_base64_blob(plain_source_text)
+    if blob is not None:
+        raise UnsupportedBase64Content(
+            f"Module '{module_name}' contains a base64-encoded blob ({len(blob)} characters) "
+            "inlined in the specification. This is not supported."
+            "Remove the base64 data from the .plain file or if necessary,"
+            "include the binary file path in the specification."
+        )
+
     return plain_source_text
 
 

{codeplain-0.3.4 → codeplain-0.3.5}/plain_spec.py

@@ -25,6 +25,15 @@ ALLOWED_SPECIFICATION_HEADINGS = [
 ALLOWED_IMPORT_SPECIFICATION_HEADINGS = [DEFINITIONS, NON_FUNCTIONAL_REQUIREMENTS, TEST_REQUIREMENTS]
 
 
+def has_acceptance_tests(plain_source_tree) -> bool:
+    """Return True if any functional requirement in the plain source defines acceptance tests."""
+    functional_requirements = plain_source_tree.get(FUNCTIONAL_REQUIREMENTS) or []
+    for functional_requirement in functional_requirements:
+        if functional_requirement.get(ACCEPTANCE_TESTS):
+            return True
+    return False
+
+
 def collect_specification_linked_resources(specification, specification_heading, linked_resources_list):
     linked_resources = []
     if "linked_resources" in specification:

{codeplain-0.3.4 → codeplain-0.3.5}/pyproject.toml

@@ -15,10 +15,10 @@ classifiers = [
     "Topic :: Software Development :: Code Generators",
 ]
 dependencies = [
-    "python-liquid2==0.3.0",
+    "python-liquid2==0.4.0",
     "mistletoe==1.3.0",
-    "requests==2.34.0",
-    "tiktoken==0.12.0",
+    "requests==2.34.2",
+    "tiktoken==0.13.0",
     "PyYAML==6.0.3",
     "gitpython==3.1.50",
     "transitions==0.9.3",
@@ -26,16 +26,16 @@ dependencies = [
     "rich==15.0.0",
     "python-frontmatter==1.3.0",
     "networkx==3.6.1",
-    "sentry-sdk==2.62.0",
+    "sentry-sdk==2.63.0",
 ]
 
 [project.optional-dependencies]
 dev = [
-    "pytest==8.3.5",
+    "pytest==9.1.1",
     "flake8==7.3.0",
     "black==26.3.1",
     "isort==5.13.2",
-    "mypy==1.11.2",
+    "mypy==2.1.0",
 ]
 
 [project.scripts]

{codeplain-0.3.4 → codeplain-0.3.5}/render_machine/actions/fix_conformance_test.py

@@ -9,12 +9,17 @@ from plain2code_exceptions import InternalClientError
 from render_machine.actions.base_action import BaseAction
 from render_machine.implementation_code_helpers import ImplementationCodeHelpers
 from render_machine.render_context import RenderContext
-from render_machine.render_types import TestExecutionPhase
+from render_machine.render_types import RenderError, TestExecutionPhase
+
+MAX_CONFORMANCE_TEST_FIX_ATTEMPTS = 20
+MAX_CONFORMANCE_TEST_RERENDER_ATTEMPTS = 1
 
 
 class FixConformanceTest(BaseAction):
     IMPLEMENTATION_CODE_NOT_UPDATED = "implementation_code_not_updated"
     IMPLEMENTATION_CODE_UPDATED = "implementation_code_updated"
+    LIMIT_EXCEEDED_OUTCOME = "conformance_test_fix_limit_exceeded"
+    REGENERATE_CONFORMANCE_TESTS_OUTCOME = "regenerate_conformance_tests"
 
     ISSUE_REASON_CODE_CONFORMANCE_TESTS = 0
     ISSUE_REASON_CODE_IMPLEMENTATION_CODE = 1
@@ -22,6 +27,23 @@ class FixConformanceTest(BaseAction):
     ISSUE_REASON_CODE_CONFLICTING_ACCEPTANCE_TESTS = 3
 
     def execute(self, render_context: RenderContext, previous_action_payload: Any | None):
+        ctx = render_context.conformance_tests_running_context
+        ctx.fix_attempts += 1
+
+        if ctx.fix_attempts >= MAX_CONFORMANCE_TEST_FIX_ATTEMPTS:
+            if ctx.conformance_tests_render_attempts >= MAX_CONFORMANCE_TEST_RERENDER_ATTEMPTS:
+                error_msg = f"The renderer was unable to produce an implementation that passes conformance tests for functionality '{render_context.frid_context.frid}' after many attempts. Please review and rewrite the specification. (Render ID: {render_context.run_state.render_id})"
+                render_context.last_error_message = error_msg
+                return (
+                    self.LIMIT_EXCEEDED_OUTCOME,
+                    RenderError.encode(message=error_msg).to_payload(),
+                )
+            else:
+                ctx.regenerating_conformance_tests = True
+                return self.REGENERATE_CONFORMANCE_TESTS_OUTCOME, None
+
+        console.info(f"Running conformance tests attempt {ctx.fix_attempts + 1}.")
+
         console.info(
             f"Fixing conformance test for functionality {render_context.conformance_tests_running_context.current_testing_frid} in module {render_context.conformance_tests_running_context.current_testing_module_name}."
         )

{codeplain-0.3.4 → codeplain-0.3.5}/render_machine/actions/prepare_testing_environment.py

@@ -12,6 +12,12 @@ class PrepareTestingEnvironment(BaseAction):
     FAILED_OUTCOME = "testing_environment_preparation_failed"
 
     def execute(self, render_context: RenderContext, _previous_action_payload: Any | None):
+        if (
+            render_context.prepare_environment_script is None
+            or not render_context.conformance_tests_running_context.should_prepare_testing_environment
+        ):
+            return self.SUCCESSFUL_OUTCOME, None
+
         console.info(
             f"Running testing environment preparation script {render_context.prepare_environment_script} for build folder {render_context.build_folder}."
         )

{codeplain-0.3.4 → codeplain-0.3.5}/render_machine/actions/refactor_code.py

@@ -5,13 +5,31 @@ from plain2code_console import console
 from render_machine.actions.base_action import BaseAction
 from render_machine.implementation_code_helpers import ImplementationCodeHelpers
 from render_machine.render_context import RenderContext
+from render_machine.render_types import RenderError
+
+MAX_REFACTORING_ITERATIONS = 5
 
 
 class RefactorCode(BaseAction):
     SUCCESSFUL_OUTCOME = "refactoring_successful"
     NO_FILES_REFACTORED_OUTCOME = "no_files_refactored"
+    ITERATION_LIMIT_EXCEEDED_OUTCOME = "refactoring_iteration_limit_exceeded"
 
     def execute(self, render_context: RenderContext, _previous_action_payload: Any | None):
+        if render_context.frid_context.refactoring_iteration == 0:
+            console.info("Refactoring the generated code...")
+
+        render_context.frid_context.refactoring_iteration += 1
+
+        if render_context.frid_context.refactoring_iteration >= MAX_REFACTORING_ITERATIONS:
+            error_message = "Refactoring iterations limit of {MAX_REFACTORING_ITERATIONS} reached for functionality {render_context.frid_context.frid}."
+            render_context.last_error_message = error_message
+
+            return (
+                self.ITERATION_LIMIT_EXCEEDED_OUTCOME,
+                RenderError.encode(message=error_message).to_payload(),
+            )
+
         existing_files, existing_files_content = ImplementationCodeHelpers.fetch_existing_files(
             render_context.build_folder
         )

{codeplain-0.3.4 → codeplain-0.3.5}/render_machine/actions/render_functional_requirement.py

@@ -16,8 +16,22 @@ MAX_CODE_GENERATION_RETRIES = 2
 class RenderFunctionalRequirement(BaseAction):
     SUCCESSFUL_OUTCOME = "code_and_unit_tests_generated"
     FUNCTIONAL_REQUIREMENT_TOO_COMPLEX_OUTCOME = "functional_requirement_too_complex"
+    ITERATION_LIMIT_EXCEEDED_OUTCOME = "frid_iteration_limit_exceeded"
 
     def execute(self, render_context: RenderContext, _previous_action_payload: Any | None):
+        if render_context.frid_context.functional_requirement_render_attempts >= MAX_CODE_GENERATION_RETRIES:
+            error_msg = f"Unittests could not be fixed after rendering the functionality {render_context.frid_context.frid} for the {MAX_CODE_GENERATION_RETRIES} times."
+            render_context.last_error_message = error_msg
+            return self.ITERATION_LIMIT_EXCEEDED_OUTCOME, None
+
+        render_context.frid_context.functional_requirement_render_attempts += 1
+
+        if render_context.frid_context.functional_requirement_render_attempts > 1:
+            console.info(
+                f"Unittests could not be fixed after rendering the functionality. "
+                f"Restarting rendering functionality {render_context.frid_context.frid} from scratch."
+            )
+
         render_utils.revert_changes_for_frid(render_context)
         existing_files, existing_files_content = ImplementationCodeHelpers.fetch_existing_files(
             render_context.build_folder
@@ -48,7 +62,7 @@ class RenderFunctionalRequirement(BaseAction):
                 render_context.run_state,
             )
         except FunctionalRequirementTooComplex as e:
-            error_message = f"The functionality:\n{render_context.frid_context.functional_requirement_text}\n is too complex to be implemented. Please break down the functionality into smaller parts ({str(e)})."
+            error_message = f"The functionality:\n{render_context.frid_context.functional_requirement_text}\n is too complex to be implemented. Please break down the functionality into smaller parts."
             if e.proposed_breakdown:
                 error_message += "\nProposed breakdown:"
                 for _, part in e.proposed_breakdown["functional_requirements"].items():