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

.pre-commit-config.yaml

@@ -18,14 +18,6 @@ repos:
         stages: [pre-commit]
         verbose: true
 
-      - id: isort
-        name: isort
-        entry: bash -c "cd python_coderunner && isort ."
-        language: system
-        types: [python]
-        stages: [pre-commit]
-        verbose: true
-
       - id: mypy
         name: mypy
         entry: bash -c "cd python_coderunner && mypy ."

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

@@ -3,13 +3,12 @@ name = "vim-code-runner"
 authors = [
   {name = "Zahar Chernenko", email = "zaharchernenko35@gmail.com"},
 ]
-version = "1.0.0"
+version = "1.0.1"
 requires-python = ">=3.10"
 
 [dependency-groups]
 dev = [
     "commitlint>=1.3.0",
-    "isort>=4.3.21",
     "mypy>=0.910",
     "pre-commit>=1.21.0",
     "pylint>=2.6.2",
@@ -19,7 +18,6 @@ dev = [
     "ruff>=0.12.0",
 ]
 pre-commit = [
-    "isort>=4.3.21",
     "mypy>=0.910",
     "pre-commit>=1.21.0",
     "pylint>=2.6.2",
@@ -35,9 +33,6 @@ test = [
 pythonpath = ["."]  # to run without python -m, otherwise there will be no src in the path
 testpaths = ["tests"]
 
-[tool.isort]
-profile = "black"
-
 [tool.mypy]
 check_untyped_defs = true
 explicit_package_bases = true  # without this, mypy complains about duplicate packages.
@@ -52,6 +47,10 @@ disable_error_code = [
 line-length = 120
 [tool.ruff.format]
 quote-style = "double"
+[tool.ruff.lint]
+extend-select = ["I", "F401"]
+extend-unsafe-fixes = ["F401"]
+ignore = ["W292"]
 [tool.ruff.lint.per-file-ignores]
 "__init__.py" = [
     "F401", # ignore unused imports
@@ -91,10 +90,11 @@ argument-naming-style = "snake_case"
 attr-naming-style = "snake_case"
 class-attribute-naming-style = "any"
 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)$"
+class-rgx="^(((I|T|E)[A-Z][a-zA-Z0-9]*)|([A-Z][a-zA-Z0-9]*(Error|Protocol)))$"
 const-naming-style= "any"
 function-naming-style = "snake_case"
-typevar="^T[A-Z][a-zA-Z0-9]*$"
+typealias-rgx="^(((I|T|E)[A-Z][a-zA-Z0-9]*)|([A-Z][a-zA-Z0-9]*(Error|Protocol)))$"
+typevar-rgx="^[A-Z][a-zA-Z0-9]*Type$"
 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",
-]