codeplain 0.3.12.dev24__py3-none-any.whl → 0.3.12.dev25__py3-none-any.whl

{codeplain-0.3.12.dev24.dist-info → codeplain-0.3.12.dev25.dist-info}/METADATA

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

{codeplain-0.3.12.dev24.dist-info → codeplain-0.3.12.dev25.dist-info}/RECORD

@@ -4,7 +4,7 @@ concept_utils.py,sha256=TpyiQ4DfMtO231j-lPj65i0suRTPr8t9j3QNf9KiM4A,8080
 conformance_fix_journal.py,sha256=7wL3UOc3BWaUo5MCmiFbb3kIwOQ6iEpdQSyEmedo8Vk,9488
 diff_utils.py,sha256=AjiQlqo5pRos_8hVXZo5yBurl5BzSrTMGrQv4dCtRCg,1198
 event_bus.py,sha256=g5XLwRfjrvTIcB6urjREuCJzZj5TA93hy8lKXLGXuCE,650
-file_utils.py,sha256=knp_cUjYlhr8pq13jRSscTHcygdmk9YFJfkzB0gmg0g,12822
+file_utils.py,sha256=IshKvld9LJLwXSB3lxZ1HQyVHa93I626BsTScBu95ss,12118
 git_preflight.py,sha256=vB2A8GyYz_iWr8jOE4POFLBHr3qihmedWuidspGdsrw,1509
 git_utils.py,sha256=gCUA6_9zg19f6k_0aVcRnIVlAJSBD0Vt6takx-0pKME,16547
 memory_management.py,sha256=CwIOQqF7Xo1sNfvP69zQRK_CpjinQD9dbolhRPl6lTk,2446
@@ -77,7 +77,7 @@ tests/test_conformance_fix_journal.py,sha256=olTeAuxXgFwQ8LSQyt0FqtBy9_XY_gK-Ei_
 tests/test_console_log_styles.py,sha256=sxlhynMAP-kmqhBn1-pKvZXdCnlKSyCGJqwYKrh6sQ0,6468
 tests/test_distill_conformance_test_memory.py,sha256=Tftw9l-gLVwIU6QttY03h2VedvuFxpIsCmZIzvWfxMA,5481
 tests/test_dry_run_link_validation.py,sha256=ivvYXQPqUkQ8glbFyUrdCninCxF2mg-rAsDcfjcB7MM,2140
-tests/test_file_utils.py,sha256=69dmr6CJkJ8sru9FSwu-a1Mh62A-RIVYUN9lxYWGq4Q,6066
+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
@@ -117,8 +117,8 @@ 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=jitm2WHiKj-NVFL-hULxWzckPreRbUTl9OIZLN31-ek,6997
-codeplain-0.3.12.dev24.dist-info/METADATA,sha256=iu8oMhAyTgF52TtIJMRiY1AkSWjNQm7ZnHaj4AgGKhk,7257
-codeplain-0.3.12.dev24.dist-info/WHEEL,sha256=W3fkpkm7-wf9vBI5Z-7s0eWkeM-spu78I8Neb98DeEg,87
-codeplain-0.3.12.dev24.dist-info/entry_points.txt,sha256=oDZkBqu9WhtZApb_K6ia8-fn9aojwmAsgnKELceX5T4,46
-codeplain-0.3.12.dev24.dist-info/licenses/LICENSE,sha256=pCeKgQ1mXE5OmNUuKOfOZh1T1vqkeCUxghE7N8qgnzc,11345
-codeplain-0.3.12.dev24.dist-info/RECORD,,
+codeplain-0.3.12.dev25.dist-info/METADATA,sha256=vjSTEqWFnNjJGUdJ2eL4R3SXDquCx69-dG_Oi6wU55s,7257
+codeplain-0.3.12.dev25.dist-info/WHEEL,sha256=W3fkpkm7-wf9vBI5Z-7s0eWkeM-spu78I8Neb98DeEg,87
+codeplain-0.3.12.dev25.dist-info/entry_points.txt,sha256=oDZkBqu9WhtZApb_K6ia8-fn9aojwmAsgnKELceX5T4,46
+codeplain-0.3.12.dev25.dist-info/licenses/LICENSE,sha256=pCeKgQ1mXE5OmNUuKOfOZh1T1vqkeCUxghE7N8qgnzc,11345
+codeplain-0.3.12.dev25.dist-info/RECORD,,

file_utils.py

@@ -142,20 +142,7 @@ def get_existing_files_content(build_folder, existing_files):
     return existing_files_content
 
 
-def ensure_final_newline(content: str) -> str:
-    """Ends non-empty text with a newline, as linters expect (e.g. RuboCop, ESLint eol-last, Ruff W292)."""
-    if content and not content.endswith("\n"):
-        return content + "\n"
-    return content
-
-
 def store_response_files(target_folder, response_files, existing_files):
-    """Writes the files returned by the API to target_folder. None content deletes the file.
-
-    Every written file ends with a newline - whole-file rewrites come back from the API without one. The
-    added newline is also written back into response_files, so callers that diff response_files against
-    the folder afterwards see exactly what is on disk.
-    """
     for file_name in response_files:
         full_file_name = os.path.join(target_folder, file_name)
 
@@ -169,8 +156,6 @@ def store_response_files(target_folder, response_files, existing_files):
 
             continue
 
-        response_files[file_name] = ensure_final_newline(response_files[file_name])
-
         os.makedirs(os.path.dirname(full_file_name), exist_ok=True)
 
         with open(full_file_name, "w", encoding="utf-8") as f:

tests/test_file_utils.py

@@ -3,8 +3,7 @@ import tempfile
 
 import pytest
 
-from diff_utils import get_code_diff
-from file_utils import get_existing_files_content, load_linked_resources, store_response_files
+from file_utils import load_linked_resources, store_response_files
 from plain2code_exceptions import UnsupportedBase64Content, UnsupportedResourceType
 from plain2code_utils import MAX_BASE64_BLOB_LENGTH
 
@@ -85,64 +84,9 @@ def test_load_linked_resources_real_base64_image_raises(template_dir):
 def test_store_response_files_writes_unicode_as_utf8(template_dir):
     # Content with a non-cp1252 character (📍 U+1F4CD) must be written as UTF-8
     # regardless of the platform's default text encoding (e.g. cp1252 on Windows).
-    content = "Location 📍 marker\n"
+    content = "Location 📍 marker"
     store_response_files(template_dir, {"notes.md": content}, [])
 
     file_path = os.path.join(template_dir, "notes.md")
     with open(file_path, "rb") as f:
         assert f.read().decode("utf-8") == content
-
-
-def _read_bytes(folder, file_name):
-    with open(os.path.join(folder, file_name), "rb") as f:
-        return f.read().decode("utf-8")
-
-
-@pytest.mark.parametrize(
-    "content, expected",
-    [
-        ("class Foo\nend", "class Foo\nend\n"),
-        ("class Foo\nend\n", "class Foo\nend\n"),
-        ("class Foo\nend\n\n", "class Foo\nend\n\n"),
-        ("class Foo\r\nend\r\n", "class Foo\r\nend\r\n"),
-        ("", ""),
-    ],
-)
-def test_store_response_files_ends_every_text_file_with_a_newline(template_dir, content, expected):
-    # Linters gate on a final newline (RuboCop Layout/TrailingEmptyLines, ESLint eol-last, Ruff W292), so a
-    # whole-file rewrite that comes back without one must not reach the disk that way.
-    store_response_files(template_dir, {"app/foo.rb": content}, [])
-
-    assert _read_bytes(template_dir, "app/foo.rb") == expected
-
-
-def test_store_response_files_hands_back_the_content_it_wrote(template_dir):
-    # Callers diff the response files against the build folder after storing them (fix journal, conformance
-    # fixes handed to the unit tests fixer), so the response files must match what is on disk.
-    response_files = {"app/foo.rb": "class Foo\nend", "app/bar.rb": "class Bar\nend\n"}
-
-    store_response_files(template_dir, response_files, [])
-
-    assert response_files == {"app/foo.rb": "class Foo\nend\n", "app/bar.rb": "class Bar\nend\n"}
-
-
-def test_rewriting_a_file_without_its_final_newline_does_not_change_it(template_dir):
-    existing_files = store_response_files(template_dir, {"app/foo.rb": "class Foo\nend\n"}, [])
-    existing_files_content = get_existing_files_content(template_dir, existing_files)
-
-    rewrite = {"app/foo.rb": "class Foo\nend"}
-    store_response_files(template_dir, rewrite, existing_files)
-
-    assert get_existing_files_content(template_dir, existing_files) == existing_files_content
-    assert get_code_diff(rewrite, existing_files_content) == {}
-
-
-def test_store_response_files_still_deletes_a_file_whose_content_is_none(template_dir):
-    existing_files = store_response_files(template_dir, {"app/foo.rb": "class Foo\nend\n"}, [])
-
-    response_files = {"app/foo.rb": None}
-    store_response_files(template_dir, response_files, existing_files)
-
-    assert not os.path.exists(os.path.join(template_dir, "app/foo.rb"))
-    assert existing_files == []
-    assert response_files == {"app/foo.rb": None}