refactor(cli): standardize string quotes and improve code formatting across CLI components

Author: marcuxyz

mvc_flask/cli.py

@@ -31,17 +31,14 @@ def generate():
 
 
 @generate.command()
-@click.argument('name')
+@click.argument("name")
 @click.option(
-    '--path', '-p',
+    "--path",
+    "-p",
     default=CLIConfig.DEFAULT_CONTROLLERS_PATH,
-    help='Path where to create the controller'
-)
-@click.option(
-    '--force', '-f',
-    is_flag=True,
-    help='Overwrite existing controller file'
+    help="Path where to create the controller",
 )
+@click.option("--force", "-f", is_flag=True, help="Overwrite existing controller file")
 @with_appcontext
 def controller(name: str, path: str, force: bool) -> None:
     """Generate a new controller.
@@ -64,53 +61,44 @@ def controller(name: str, path: str, force: bool) -> None:
         click.echo(
             click.style(
                 f"✓ Controller created successfully at {controller_file}",
-                fg=CLIConfig.SUCCESS_COLOR
+                fg=CLIConfig.SUCCESS_COLOR,
             )
         )
 
         # Provide helpful next steps
-        controller_name = name if name.endswith('_controller') else f"{name}_controller"
-        click.echo(
-            click.style(
-                "\nNext steps:",
-                fg=CLIConfig.INFO_COLOR,
-                bold=True
-            )
-        )
+        controller_name = name if name.endswith("_controller") else f"{name}_controller"
+        click.echo(click.style("\nNext steps:", fg=CLIConfig.INFO_COLOR, bold=True))
         click.echo(
             click.style(
                 "1. Add routes for your controller in your routes.py file",
-                fg=CLIConfig.INFO_COLOR
+                fg=CLIConfig.INFO_COLOR,
             )
         )
         click.echo(
             click.style(
                 f"2. Create templates in views/{name}/ directory",
-                fg=CLIConfig.INFO_COLOR
+                fg=CLIConfig.INFO_COLOR,
             )
         )
         click.echo(
             click.style(
                 f"3. Implement your business logic in {controller_name}.py",
-                fg=CLIConfig.INFO_COLOR
+                fg=CLIConfig.INFO_COLOR,
             )
         )
 
     except (ControllerGenerationError, InvalidControllerNameError) as e:
         logger.error(f"Controller generation failed: {e}")
-        click.echo(
-            click.style(f"✗ Error: {e}", fg=CLIConfig.ERROR_COLOR),
-            err=True
-        )
+        click.echo(click.style(f"✗ Error: {e}", fg=CLIConfig.ERROR_COLOR), err=True)
         raise click.Abort() from e
     except Exception as e:
         logger.exception(f"Unexpected error during controller generation: {e}")
         click.echo(
             click.style(
                 f"✗ Unexpected error: {e}. Please check the logs for more details.",
-                fg=CLIConfig.ERROR_COLOR
+                fg=CLIConfig.ERROR_COLOR,
             ),
-            err=True
+            err=True,
         )
         raise click.Abort() from e
 

mvc_flask/core/config.py

@@ -14,22 +14,22 @@ class CLIConfig:
     DEFAULT_MODELS_PATH = "app/models"
 
     # Template configuration
-    TEMPLATES_DIR = Path(__file__).parent.parent / 'templates'
+    TEMPLATES_DIR = Path(__file__).parent.parent / "templates"
 
     # Controller template settings
-    CONTROLLER_TEMPLATE = 'base_controller.jinja2'
+    CONTROLLER_TEMPLATE = "base_controller.jinja2"
 
     # File encoding
-    FILE_ENCODING = 'utf-8'
+    FILE_ENCODING = "utf-8"
 
     # CLI styling
-    SUCCESS_COLOR = 'green'
-    ERROR_COLOR = 'red'
-    WARNING_COLOR = 'yellow'
-    INFO_COLOR = 'blue'
+    SUCCESS_COLOR = "green"
+    ERROR_COLOR = "red"
+    WARNING_COLOR = "yellow"
+    INFO_COLOR = "blue"
 
     # Environment variables for overriding defaults
-    ENV_PREFIX = 'FLASK_MVC_'
+    ENV_PREFIX = "FLASK_MVC_"
 
     @classmethod
     def get_controllers_path(cls) -> str:
@@ -39,8 +39,7 @@ def get_controllers_path(cls) -> str:
             Controllers directory path
         """
         return os.getenv(
-            f'{cls.ENV_PREFIX}CONTROLLERS_PATH',
-            cls.DEFAULT_CONTROLLERS_PATH
+            f"{cls.ENV_PREFIX}CONTROLLERS_PATH", cls.DEFAULT_CONTROLLERS_PATH
         )
 
     @classmethod
@@ -50,7 +49,7 @@ def get_templates_dir(cls) -> Path:
         Returns:
             Templates directory path
         """
-        custom_dir = os.getenv(f'{cls.ENV_PREFIX}TEMPLATES_DIR')
+        custom_dir = os.getenv(f"{cls.ENV_PREFIX}TEMPLATES_DIR")
         if custom_dir:
             return Path(custom_dir)
         return cls.TEMPLATES_DIR
@@ -62,7 +61,4 @@ def get_file_encoding(cls) -> str:
         Returns:
             File encoding string
         """
-        return os.getenv(
-            f'{cls.ENV_PREFIX}FILE_ENCODING',
-            cls.FILE_ENCODING
-        )
+        return os.getenv(f"{cls.ENV_PREFIX}FILE_ENCODING", cls.FILE_ENCODING)

mvc_flask/core/exceptions.py

@@ -3,19 +3,23 @@
 
 class MVCFlaskError(Exception):
     """Base exception for MVC Flask CLI errors."""
+
     pass
 
 
 class ControllerGenerationError(MVCFlaskError):
     """Exception raised when controller generation fails."""
+
     pass
 
 
 class TemplateNotFoundError(MVCFlaskError):
     """Exception raised when template file is not found."""
+
     pass
 
 
 class InvalidControllerNameError(MVCFlaskError):
     """Exception raised when controller name is invalid."""
+
     pass

mvc_flask/core/file_handler.py

@@ -47,7 +47,7 @@ def write_file(file_path: Path, content: str) -> None:
             ControllerGenerationError: If file writing fails
         """
         try:
-            with open(file_path, 'w', encoding='utf-8') as f:
+            with open(file_path, "w", encoding="utf-8") as f:
                 f.write(content)
         except Exception as e:
             raise ControllerGenerationError(

mvc_flask/core/generators.py

@@ -26,7 +26,9 @@ def __init__(self, templates_dir: Optional[Path] = None):
         self.name_utils = NameUtils()
         self.config = CLIConfig()
 
-    def generate(self, name: str, output_path: Optional[str] = None, force: bool = False) -> Path:
+    def generate(
+        self, name: str, output_path: Optional[str] = None, force: bool = False
+    ) -> Path:
         """Generate a new controller file.
 
         Args:
@@ -68,8 +70,7 @@ def generate(self, name: str, output_path: Optional[str] = None, force: bool = F
 
             # Render template
             content = self.template_renderer.render(
-                self.config.CONTROLLER_TEMPLATE,
-                {'class_name': class_name}
+                self.config.CONTROLLER_TEMPLATE, {"class_name": class_name}
             )
 
             # Write file

mvc_flask/core/name_utils.py

@@ -21,7 +21,7 @@ def validate_controller_name(name: str) -> None:
         if not name:
             raise InvalidControllerNameError("Controller name cannot be empty")
 
-        if not re.match(r'^[a-zA-Z]\w*$', name):
+        if not re.match(r"^[a-zA-Z]\w*$", name):
             raise InvalidControllerNameError(
                 "Controller name must start with a letter and contain only "
                 "letters, numbers, and underscores"
@@ -39,7 +39,7 @@ def normalize_controller_name(name: str) -> str:
         """
         NameUtils.validate_controller_name(name)
 
-        if not name.endswith('_controller'):
+        if not name.endswith("_controller"):
             return f"{name}_controller"
         return name
 
@@ -54,10 +54,10 @@ def generate_class_name(controller_name: str) -> str:
             Properly formatted class name
         """
         # Remove _controller suffix if present
-        base_name = controller_name.replace('_controller', '')
+        base_name = controller_name.replace("_controller", "")
 
         # Split by underscore and capitalize each word
-        words = base_name.split('_')
-        class_name = ''.join(word.capitalize() for word in words)
+        words = base_name.split("_")
+        class_name = "".join(word.capitalize() for word in words)
 
         return f"{class_name}Controller"

mvc_flask/core/template_renderer.py

@@ -18,9 +18,7 @@ def __init__(self, templates_dir: Path):
         """
         self.templates_dir = templates_dir
         self._env = Environment(
-            loader=FileSystemLoader(templates_dir),
-            trim_blocks=True,
-            lstrip_blocks=True
+            loader=FileSystemLoader(templates_dir), trim_blocks=True, lstrip_blocks=True
         )
 
     def render(self, template_name: str, context: Dict[str, Any]) -> str: