refactor: rename Builder to Factory in coderunner_builder

- Rename ICodeRunnerBuilder to ICodeRunnerFactory
- Rename TVimCodeRunnerBuilder to TVimCodeRunnerFactory
- Rename build() method to create()
- Rename all _build_* private methods to _create_*
- Factory pattern better reflects single-call object creation
- All 87 tests passing

Author: ZaharChernenko

README.md

@@ -162,6 +162,7 @@ TInterpolatorCommandBuilder ..|> ICommandBuilder : implements
 TInterpolatorCommandBuilder "1" o--> "1" IProjectInfoExtractor : aggregates
 TInterpolatorCommandBuilder "1" o--> "1" IFileInfoExtractor : aggregates
 IProjectInfoExtractor "1" o--> "1" IFileInfoExtractor : aggregates
+TBaseFileInfoExtractor ..|> IFileInfoExtractor : implements
 
 
 class TCodeRunner {
@@ -312,7 +313,6 @@ class IProjectInfoExtractor {
 	+ get_all_files_filter_by_file_type(file_types: set[str]) Iterable[str]
 }
 
-
 class IFileInfoExtractor {
 	<<interface>>
 	Declares all commands that are directly connected to the file.

autoload/coderunner.vim

@@ -63,10 +63,10 @@ def coderunner_on_exit():
 
 sys.path.insert(0, os.path.dirname(vim.eval("s:script_folder_path")))
 try:
-    from python_coderunner import TCodeRunner, TVimCodeRunnerBuilder
+    from python_coderunner import TCodeRunner, TVimCodeRunnerFactory
 
 
-    coderunner: TCodeRunner = TVimCodeRunnerBuilder().build()
+    coderunner: TCodeRunner = TVimCodeRunnerFactory().create()
 except Exception as error:
     vim.command("redraw | echohl ErrorMsg")
     for line in traceback.format_exc().splitlines():

python_coderunner/__init__.py

@@ -1,2 +1,2 @@
 from .src.coderunner import TCodeRunner
-from .src.coderunner_builder import TVimCodeRunnerBuilder
+from .src.coderunner_builder import TVimCodeRunnerFactory

python_coderunner/pyproject.toml

@@ -94,7 +94,7 @@ class-const-naming-style = "UPPER_CASE"
 class-rgx="^(((I|T|E|Supports)[A-Z][a-zA-Z0-9]*)|[A-Z][a-zA-Z0-9]*Error)$"
 const-naming-style= "any"
 function-naming-style = "snake_case"
-typevar="^T[A-Z][a-zA-Z0-9]*$"
+typevar-rgx="^T[A-Z][a-zA-Z0-9]*$"
 variable-naming-style = "snake_case"
 module-naming-style = "snake_case"
 inlinevar-naming-style = "any"

python_coderunner/src/coderunner_builder/__init__.py

@@ -1,2 +1,2 @@
-from .interface import ICodeRunnerBuilder
-from .vim_coderunner_builder import TVimCodeRunnerBuilder
+from .interface import ICodeRunnerFactory
+from .vim_coderunner_builder import TVimCodeRunnerFactory

python_coderunner/src/coderunner_builder/interface.py

@@ -4,7 +4,7 @@
 from ..coderunner import TCodeRunner
 
 
-class ICodeRunnerBuilder(ABC):
+class ICodeRunnerFactory(ABC):
     @abstractmethod
-    def build(self) -> Optional[TCodeRunner]:
+    def create(self) -> Optional[TCodeRunner]:
         raise NotImplementedError

python_coderunner/src/coderunner_builder/vim_coderunner_builder.py

@@ -24,11 +24,11 @@
 from ..file_info_extractor import TVimFileInfoExtractor
 from ..message_printer import TVimMessagePrinter
 from ..project_info_extractor import TVimProjectInfoExtractor
-from .interface import ICodeRunnerBuilder
+from .interface import ICodeRunnerFactory
 
 
-class TVimCodeRunnerBuilder(ICodeRunnerBuilder):
-    def build(self) -> Optional[TCodeRunner]:
+class TVimCodeRunnerFactory(ICodeRunnerFactory):
+    def create(self) -> Optional[TCodeRunner]:
         config_manager: TVimConfigManager = TVimConfigManager(TVimConfigGetter(), TBasicConfigValidator())
         message_printer: TVimMessagePrinter = TVimMessagePrinter()
 
@@ -42,7 +42,7 @@ def build(self) -> Optional[TCodeRunner]:
             )
 
             command_dispatcher_strategy_selector: TBasicCommandDispatcherStrategySelector = (
-                self._build_command_dispatcher_strategy_selector(
+                self._create_command_dispatcher_strategy_selector(
                     config_manager, file_info_extractor, project_info_extractor
                 )
             )
@@ -61,7 +61,7 @@ def build(self) -> Optional[TCodeRunner]:
 
         return None
 
-    def _build_command_dispatcher_strategy_selector(
+    def _create_command_dispatcher_strategy_selector(
         self,
         config_manager: TVimConfigManager,
         file_info_extractor: TVimFileInfoExtractor,
@@ -71,17 +71,17 @@ def _build_command_dispatcher_strategy_selector(
             file_info_extractor
         )
         file_ext_command_builders_dispatcher: TFileExtCommandBuildersDispatcher = (
-            self._build_file_ext_command_builders_dispatcher(
+            self._create_file_ext_command_builders_dispatcher(
                 config_manager, file_info_extractor, project_info_extractor
             )
         )
         file_type_command_builders_dispatcher: TFileTypeCommandBuildersDispatcher = (
-            self._build_file_type_command_builders_dispatcher(
+            self._create_file_type_command_builders_dispatcher(
                 config_manager, file_info_extractor, project_info_extractor
             )
         )
-        glob_command_builders_dispatcher: TGlobCommandBuildersDispatcher = self._build_glob_command_builders_dispatcher(
-            config_manager, file_info_extractor, project_info_extractor
+        glob_command_builders_dispatcher: TGlobCommandBuildersDispatcher = (
+            self._create_glob_command_builders_dispatcher(config_manager, file_info_extractor, project_info_extractor)
         )
 
         return TBasicCommandDispatcherStrategySelector(
@@ -92,7 +92,7 @@ def _build_command_dispatcher_strategy_selector(
             config_manager=config_manager,
         )
 
-    def _build_file_ext_command_builders_dispatcher(
+    def _create_file_ext_command_builders_dispatcher(
         self,
         config_manager: TVimConfigManager,
         file_info_extractor: TVimFileInfoExtractor,
@@ -106,7 +106,7 @@ def _build_file_ext_command_builders_dispatcher(
             file_info_extractor,
         )
 
-    def _build_file_type_command_builders_dispatcher(
+    def _create_file_type_command_builders_dispatcher(
         self,
         config_manager: TVimConfigManager,
         file_info_extractor: TVimFileInfoExtractor,
@@ -120,7 +120,7 @@ def _build_file_type_command_builders_dispatcher(
             file_info_extractor,
         )
 
-    def _build_glob_command_builders_dispatcher(
+    def _create_glob_command_builders_dispatcher(
         self,
         config_manager: TVimConfigManager,
         file_info_extractor: TVimFileInfoExtractor,

python_coderunner/src/file_info_extractor/__init__.py

@@ -1,9 +1,3 @@
 from .base import TBaseFileInfoExtractor
 from .interface import IFileInfoExtractor
 from .vim_file_info_extractor import TVimFileInfoExtractor
-
-__all__ = [
-    "IFileInfoExtractor",
-    "TBaseFileInfoExtractor",
-    "TVimFileInfoExtractor",
-]