fix: Replace type pipe with Optional

Author: kvankova

code_embedder/code_embedding.py

@@ -1,3 +1,5 @@
+from typing import Optional
+
 from loguru import logger
 
 from code_embedder.script_content_reader import ScriptContentReaderInterface
@@ -9,7 +11,7 @@ class CodeEmbedder:
     def __init__(
         self,
         readme_paths: list[str],
-        changed_files: list[str] | None,
+        changed_files: Optional[list[str]],
         script_metadata_extractor: ScriptMetadataExtractorInterface,
         script_content_reader: ScriptContentReaderInterface,
     ) -> None:
@@ -56,7 +58,7 @@ def _read_readme(self, readme_path: str) -> list[str]:
 
     def _extract_scripts(
         self, readme_content: list[str], readme_path: str
-    ) -> list[ScriptMetadata] | None:
+    ) -> Optional[list[ScriptMetadata]]:
         scripts = self._script_metadata_extractor.extract(readme_content=readme_content)
         if not scripts:
             logger.debug(f"No script paths found in README in path {readme_path}. Skipping.")

code_embedder/script_content_reader.py

@@ -1,6 +1,6 @@
 import ast
 import re
-from typing import Protocol
+from typing import Optional, Protocol
 
 from code_embedder.script_metadata import ScriptMetadata
 
@@ -83,7 +83,9 @@ def _extract_part(self, script: ScriptMetadata) -> str:
 
         return "\n".join(lines[start:end])
 
-    def _extract_object_part(self, script: ScriptMetadata) -> tuple[int | None, int | None]:
+    def _extract_object_part(
+        self, script: ScriptMetadata
+    ) -> tuple[Optional[int], Optional[int]]:
         tree = ast.parse(script.content)
 
         for node in ast.walk(tree):
@@ -100,8 +102,8 @@ def _extract_object_part(self, script: ScriptMetadata) -> tuple[int | None, int
         return None, None
 
     def _extract_section_part(
-        self, lines: list[str], section: str | None = None
-    ) -> tuple[int | None, int | None]:
+        self, lines: list[str], section: Optional[str] = None
+    ) -> tuple[Optional[int], Optional[int]]:
         if not section:
             return None, None
 

code_embedder/script_metadata.py

@@ -1,5 +1,5 @@
 from dataclasses import dataclass
-from typing import Literal
+from typing import Literal, Optional
 
 
 @dataclass
@@ -8,5 +8,5 @@ class ScriptMetadata:
     readme_end: int
     path: str
     extraction_type: Literal["section", "object", "full"] = "full"
-    extraction_part: str | None = None
+    extraction_part: Optional[str] = None
     content: str = ""

pyproject.toml

@@ -27,5 +27,5 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.ruff]
 line-length = 95
-lint.select = ["E", "F", "I", "BLE", "UP", "FA"]
-target-version = "py311"
+lint.select = ["E", "F", "I", "BLE", "UP"]
+target-version = "py39"

tests/test_script_content_reader.py

@@ -1,4 +1,4 @@
-from typing import Literal
+from typing import Literal, Optional
 
 import pytest
 
@@ -9,7 +9,7 @@
 def create_script_metadata(
     path: str,
     extraction_type: Literal["full", "section", "object"] = "full",
-    extraction_part: str | None = None,
+    extraction_part: Optional[str] = None,
     content: str = "",
 ) -> ScriptMetadata:
     return ScriptMetadata(

tests/test_script_metadata_extractor.py

@@ -1,4 +1,4 @@
-from typing import Literal
+from typing import Literal, Optional
 
 import pytest
 
@@ -11,7 +11,7 @@ def create_script_metadata(
     readme_end: int,
     path: str,
     extraction_type: Literal["section", "object", "full"] = "full",
-    extraction_part: str | None = None,
+    extraction_part: Optional[str] = None,
     content: str = "",
 ) -> ScriptMetadata:
     return ScriptMetadata(