lintro 0.11.0__tar.gz → 0.13.0__tar.gz

{lintro-0.11.0/lintro.egg-info → lintro-0.13.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lintro
-Version: 0.11.0
+Version: 0.13.0
 Summary: A unified CLI tool for code formatting, linting, and quality assurance
 Author-email: TurboCoder13 <turbocoder13@gmail.com>
 License: MIT License

{lintro-0.11.0 → lintro-0.13.0}/assets/images/coverage-badge.svg

@@ -15,7 +15,7 @@
     <g fill="#fff" text-anchor="middle" font-family="DejaVu Sans,Verdana,Geneva,sans-serif" font-size="110">
         <text x="325" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="530">coverage</text>
         <text x="325" y="140" transform="scale(.1)" textLength="530">coverage</text>
-        <text x="800" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="260">83.9%</text>
-        <text x="800" y="140" transform="scale(.1)" textLength="260">83.9%</text>
+        <text x="800" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="260">84.2%</text>
+        <text x="800" y="140" transform="scale(.1)" textLength="260">84.2%</text>
     </g>
 </svg>

{lintro-0.11.0 → lintro-0.13.0}/docs/tool-analysis/ruff-analysis.md

@@ -151,7 +151,7 @@ lintro check --tool-options "ruff:format_check=True"
 ruff check src/
 
 # With specific rules
-ruff check --select E501,W503 src/
+ruff check --select E501,W503,B006 src/
 
 # Auto-fixing
 ruff check --fix src/
@@ -176,7 +176,7 @@ ruff_tool.fix()
 
 # With specific options
 ruff_tool.set_options(
-    select=["E501", "W503"],
+    select=["E501", "W503", "B006"],
     line_length=88,
     unsafe_fixes=True
 )

{lintro-0.11.0 → lintro-0.13.0}/lintro/__init__.py

@@ -1,3 +1,3 @@
 """Lintro - A unified CLI core for code formatting, linting, and quality assurance."""
 
-__version__ = "0.11.0"
+__version__ = "0.13.0"

{lintro-0.11.0 → lintro-0.13.0}/lintro/cli_utils/commands/format.py

@@ -140,7 +140,7 @@ def format_code_legacy(
         None: This function does not return a value.
 
     Raises:
-        Exception: If format fails for any reason.
+        RuntimeError: If format fails for any reason.
     """
     args: list[str] = []
     if paths:
@@ -163,5 +163,5 @@ def format_code_legacy(
     runner = CliRunner()
     result = runner.invoke(format_code, args)
     if result.exit_code != DEFAULT_EXIT_CODE:
-        raise Exception(f"Format failed: {result.output}")
+        raise RuntimeError(f"Format failed: {result.output}")
     return None

{lintro-0.11.0 → lintro-0.13.0}/lintro/formatters/tools/ruff_formatter.py

@@ -94,9 +94,11 @@ def format_ruff_issues(
     non_fixable_issues: list[RuffIssue] = []
 
     for issue in issues:
-        if isinstance(issue, RuffFormatIssue):
-            fixable_issues.append(issue)
-        elif isinstance(issue, RuffIssue) and issue.fixable:
+        if (
+            isinstance(issue, RuffFormatIssue)
+            or isinstance(issue, RuffIssue)
+            and issue.fixable
+        ):
             fixable_issues.append(issue)
         elif isinstance(issue, RuffIssue):
             non_fixable_issues.append(issue)
@@ -105,7 +107,11 @@ def format_ruff_issues(
     if format == "json":
         columns = descriptor.get_columns()
         rows = descriptor.get_rows(issues)
-        return formatter.format(columns=columns, rows=rows, tool_name="ruff")
+        return formatter.format(
+            columns=columns,
+            rows=rows,
+            tool_name="ruff",
+        )
 
     sections: list[str] = []
 

{lintro-0.11.0 → lintro-0.13.0}/lintro/parsers/prettier/prettier_parser.py

@@ -36,7 +36,7 @@ def parse_prettier_output(output: str) -> list[PrettierIssue]:
 
     lines = normalized_output.splitlines()
 
-    for i, line in enumerate(lines):
+    for _i, line in enumerate(lines):
         line = line.strip()
         if not line:
             continue

{lintro-0.11.0 → lintro-0.13.0}/lintro/tools/__init__.py

@@ -21,7 +21,7 @@ tool_manager = ToolManager()
 AVAILABLE_TOOLS = {tool_enum: tool_enum.value for tool_enum in ToolEnum}
 
 
-for tool_enum, tool_class in AVAILABLE_TOOLS.items():
+for _tool_enum, tool_class in AVAILABLE_TOOLS.items():
     tool_manager.register_tool(tool_class)
 
 # Consolidated exports

{lintro-0.11.0 → lintro-0.13.0}/lintro/tools/implementations/tool_ruff.py

@@ -127,6 +127,7 @@ class RuffTool(BaseTool):
     def __post_init__(self) -> None:
         """Initialize the tool with default configuration."""
         super().__post_init__()
+
         # Load ruff configuration from pyproject.toml
         ruff_config = _load_ruff_config()
 
@@ -151,6 +152,13 @@ class RuffTool(BaseTool):
         if "unsafe_fixes" in ruff_config:
             self.options["unsafe_fixes"] = ruff_config["unsafe_fixes"]
 
+        # Allow environment variable override for unsafe fixes
+        # Useful for development and CI environments
+        # This must come after config loading to override config values
+        env_unsafe_fixes = os.environ.get("RUFF_UNSAFE_FIXES", "").lower()
+        if env_unsafe_fixes in ("true", "1", "yes", "on"):
+            self.options["unsafe_fixes"] = True
+
     def set_options(
         self,
         select: list[str] | None = None,
@@ -187,14 +195,26 @@ class RuffTool(BaseTool):
         Raises:
             ValueError: If an option value is invalid.
         """
-        if select is not None and not isinstance(select, list):
-            raise ValueError("select must be a list of rule codes")
-        if ignore is not None and not isinstance(ignore, list):
-            raise ValueError("ignore must be a list of rule codes")
-        if extend_select is not None and not isinstance(extend_select, list):
-            raise ValueError("extend_select must be a list of rule codes")
-        if extend_ignore is not None and not isinstance(extend_ignore, list):
-            raise ValueError("extend_ignore must be a list of rule codes")
+        if select is not None:
+            if isinstance(select, str):
+                select = [select]
+            elif not isinstance(select, list):
+                raise ValueError("select must be a string or list of rule codes")
+        if ignore is not None:
+            if isinstance(ignore, str):
+                ignore = [ignore]
+            elif not isinstance(ignore, list):
+                raise ValueError("ignore must be a string or list of rule codes")
+        if extend_select is not None:
+            if isinstance(extend_select, str):
+                extend_select = [extend_select]
+            elif not isinstance(extend_select, list):
+                raise ValueError("extend_select must be a string or list of rule codes")
+        if extend_ignore is not None:
+            if isinstance(extend_ignore, str):
+                extend_ignore = [extend_ignore]
+            elif not isinstance(extend_ignore, list):
+                raise ValueError("extend_ignore must be a string or list of rule codes")
         if line_length is not None:
             if not isinstance(line_length, int):
                 raise ValueError("line_length must be an integer")
@@ -512,9 +532,9 @@ class RuffTool(BaseTool):
         # Optionally run ruff check --fix (lint fixes)
         remaining_issues = []
         remaining_count = 0
+        success: bool = True  # Default to True when lint_fix is disabled
         if self.options.get("lint_fix", True):
             cmd: list[str] = self._build_check_command(files=python_files, fix=True)
-            success: bool
             output: str
             success, output = self._run_subprocess(cmd=cmd, timeout=timeout)
             remaining_issues = parse_ruff_output(output=output)
@@ -621,10 +641,7 @@ class RuffTool(BaseTool):
 
         # Success should be based on whether there are remaining issues after fixing
         # If there are no initial issues, success should be True
-        if total_initial_count == 0:
-            overall_success = True
-        else:
-            overall_success = remaining_count == 0
+        overall_success = True if total_initial_count == 0 else remaining_count == 0
 
         return ToolResult(
             name=self.name,

{lintro-0.11.0 → lintro-0.13.0}/lintro/utils/tool_executor.py

@@ -65,7 +65,7 @@ def _get_tools_to_run(
             available_names: list[str] = [e.name.lower() for e in ToolEnum]
             raise ValueError(
                 f"Unknown tool '{name.lower()}'. Available tools: {available_names}",
-            )
+            ) from None
 
     return tools_to_run
 
@@ -231,7 +231,7 @@ def run_lint_tools_simple(
         post_cfg_early = load_post_checks_config()
         post_enabled_early = bool(post_cfg_early.get("enabled", False))
         post_tools_early: set[str] = (
-            set(t.lower() for t in (post_cfg_early.get("tools", []) or []))
+            {t.lower() for t in (post_cfg_early.get("tools", []) or [])}
             if post_enabled_early
             else set()
         )
@@ -392,10 +392,7 @@ def run_lint_tools_simple(
                 if not json_output_mode:
                     # Use raw output if raw_output is true, otherwise use
                     # formatted output
-                    if raw_output:
-                        display_output = output
-                    else:
-                        display_output = formatted_output
+                    display_output = output if raw_output else formatted_output
                     logger.print_tool_result(
                         tool_name=tool_name,
                         output=display_output,

{lintro-0.11.0 → lintro-0.13.0}/lintro/utils/tool_utils.py

@@ -104,7 +104,7 @@ def parse_tool_list(tools_str: str | None) -> list[str]:
         try:
             result.append(ToolEnum[t.upper()])
         except KeyError:
-            raise ValueError(f"Unknown core: {t}")
+            raise ValueError(f"Unknown core: {t}") from None
     return result
 
 
@@ -200,10 +200,7 @@ def get_table_columns(
         columns = expected_columns
     else:
         # Fallback: use all unique keys from the first issue
-        if issues:
-            columns = list(issues[0].keys())
-        else:
-            columns = []
+        columns = list(issues[0].keys()) if issues else []
 
     # Convert issues to rows
     rows: list[list[str]] = []
@@ -434,13 +431,11 @@ def walk_files_with_excludes(
                             matches_pattern = True
                             break
 
-                    if matches_pattern:
-                        # Check if file should be excluded
-                        if not should_exclude_path(
-                            path=rel_path,
-                            exclude_patterns=exclude_patterns,
-                        ):
-                            all_files.append(file_path)
+                    if matches_pattern and not should_exclude_path(
+                        path=rel_path,
+                        exclude_patterns=exclude_patterns,
+                    ):
+                        all_files.append(file_path)
 
     return sorted(all_files)
 

{lintro-0.11.0 → lintro-0.13.0/lintro.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lintro
-Version: 0.11.0
+Version: 0.13.0
 Summary: A unified CLI tool for code formatting, linting, and quality assurance
 Author-email: TurboCoder13 <turbocoder13@gmail.com>
 License: MIT License

{lintro-0.11.0 → lintro-0.13.0}/lintro.egg-info/SOURCES.txt

@@ -127,8 +127,11 @@ test_samples/darglint_violations.py
 test_samples/prettier_violations.js
 test_samples/ruff_annotations_violations.py
 test_samples/ruff_black_e501_wrappable.py
+test_samples/ruff_bugbear_violations.py
+test_samples/ruff_c4_comprehensions_violations.py
 test_samples/ruff_clean.py
 test_samples/ruff_naming_violations.py
+test_samples/ruff_sim_simplify_violations.py
 test_samples/ruff_violations.py
 test_samples/yaml_violations.yml
 tests/__init__.py
@@ -147,8 +150,10 @@ tests/integration/test_bandit_integration.py
 tests/integration/test_darglint_integration.py
 tests/integration/test_hadolint_integration.py
 tests/integration/test_prettier_integration.py
+tests/integration/test_ruff_additional_coverage.py
 tests/integration/test_ruff_annotations.py
 tests/integration/test_ruff_black_policy.py
+tests/integration/test_ruff_bugbear.py
 tests/integration/test_ruff_integration.py
 tests/integration/test_ruff_naming.py
 tests/integration/test_yamllint_integration.py

{lintro-0.11.0 → lintro-0.13.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "lintro"
-version = "0.11.0"
+version = "0.13.0"
 description = "A unified CLI tool for code formatting, linting, and quality assurance"
 keywords = [ "linting", "formatting", "code-quality", "cli", "python", "javascript", "yaml", "docker",]
 classifiers = [ "Development Status :: 4 - Beta", "Intended Audience :: Developers", "Operating System :: OS Independent", "Programming Language :: Python :: 3", "Programming Language :: Python :: 3.13", "Topic :: Software Development :: Quality Assurance", "Topic :: Software Development :: Libraries :: Python Modules", "Topic :: Utilities",]
@@ -65,19 +65,21 @@ indent-style = "space"
 skip-magic-trailing-comma = false
 
 [tool.ruff.lint]
-select = [ "E", "F", "W", "I", "COM", "N", "D", "UP", "ANN",]
+select = [ "E", "F", "W", "I", "COM", "N", "D", "UP", "ANN", "B", "C4", "SIM",]
 ignore = [ "ANN",]
 
-[tool.ruff.pydocstyle]
-convention = "google"
-
 [tool.lintro.post_checks]
 enabled = true
 tools = [ "black",]
 enforce_failure = true
 
+[tool.lintro.ruff]
+
 [tool.pytest.ini_options]
 testpaths = [ "tests",]
 python_files = "test_*.py"
 python_classes = "Test*"
 python_functions = "test_*"
+
+[tool.ruff.lint.pydocstyle]
+convention = "google"

lintro-0.13.0/test_samples/ruff_bugbear_violations.py

@@ -0,0 +1,243 @@
+"""Test file with flake8-bugbear (B) rule violations for testing Ruff integration."""
+
+import os
+from typing import Any
+
+
+def mutable_default_argument(
+    items: list = None,
+):  # B006: Do not use mutable data structures for argument defaults
+    """Function with mutable default argument."""
+    if items is None:
+        items = []
+    items.append("test")
+    return items
+
+
+def dictionary_comprehension_with_unused_loop_variable():
+    """Dictionary comprehension with unused loop variable."""
+    data = {"a": 1, "b": 2, "c": 3}
+    # B007: Loop control variable not used within loop body
+    return {k: v for k, v in data.items() if v > 1}
+
+
+def assert_without_message():
+    """Function using assert without message."""
+    x = 5
+    # B011: Do not use assert False since Python -O removes these calls
+    raise AssertionError("This should not happen")
+
+
+def exception_handling_without_exception():
+    """Function with bare except clause."""
+    try:
+        risky_operation()
+    except:  # B001: Do not use bare except
+        pass
+
+
+def unused_variable_in_comprehension():
+    """List comprehension with unused variable."""
+    numbers = [1, 2, 3, 4, 5]
+    # B023: Function definition does not bind loop variable
+    return [x for x in numbers if x > 2]
+
+
+def risky_operation():
+    """Placeholder for risky operation."""
+    raise ValueError("Simulated error")
+
+
+def main():
+    """Main function to demonstrate flake8-bugbear violations."""
+
+    # B008: Do not perform function calls in argument defaults
+    def delayed_call(value=os.getcwd()):
+        return value
+
+    # B009: Do not call getattr with a constant attribute name
+    obj = {"attr": "value"}
+    result = getattr(obj, "attr", None)
+
+    # B010: Do not call setattr with a constant attribute name
+    obj.new_attr = "new_value"
+
+    # B012: Do not use break/continue/return inside finally
+    try:
+        return "success"
+    finally:
+        # This would trigger B012 if we had break/continue/return here
+        pass
+
+    # B013: A length-one tuple literal is redundant
+    single_item = (1,)  # Should be just 1
+
+    # B014: Convert namedtuple to dataclass
+    from collections import namedtuple
+
+    Point = namedtuple("Point", ["x", "y"])  # B014
+
+    # B015: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B015
+
+    # B016: Cannot raise a non-exception class
+    # This would be: raise "string"  # B016
+
+    # B017: assertRaises(Exception) should be considered more specific
+    # This is more relevant in test code
+
+    # B018: Found useless expression
+    # This would be: 1 + 1  # B018 (useless expression)
+
+    # B019: Use functools.lru_cache instead of functools.cache
+    import functools
+
+    @functools.cache  # B019
+    def cached_function(x):
+        return x * 2
+
+    # B020: Loop variable overrides iterable it iterates
+    items = [1, 2, 3]
+    for items in items:  # B020
+        print(items)
+
+    # B021: f-string used as docstring
+    def f_string_docstring():
+        f"""This is an f-string docstring."""  # B021
+        pass
+
+    # B022: No arguments passed to contextlib.suppress
+    import contextlib
+
+    with contextlib.suppress():  # B022
+        pass
+
+    # B024: BaseException is too broad, prefer Exception
+    try:
+        risky_operation()
+    except BaseException:  # B024
+        pass
+
+    # B025: Missing required keyword-only arguments
+    def required_kwargs(*, required_arg):
+        return required_arg
+
+    # B026: Star-arg unpacking after a keyword argument
+    def star_arg_after_kwarg(**kwargs):
+        return kwargs
+
+    # B027: Empty method in an abstract base class
+    from abc import ABC, abstractmethod
+
+    class AbstractClass(ABC):
+        @abstractmethod
+        def empty_method(self):  # B027
+            pass
+
+    # B028: No explicit stacklevel in warnings.warn
+    import warnings
+
+    warnings.warn("This is a warning", stacklevel=2)  # B028
+
+    # B029: Except handler does not have access to the exception
+    try:
+        risky_operation()
+    except ValueError:
+        # B029: This would trigger if we didn't bind the exception
+        pass
+
+    # B030: Except handler does not have access to the exception
+    try:
+        risky_operation()
+    except ValueError as e:
+        # This is correct - we bind the exception
+        print(f"Error: {e}")
+
+    # B031: Except handler does not have access to the exception
+    try:
+        risky_operation()
+    except ValueError:
+        # B031: This would trigger if we didn't bind the exception
+        pass
+
+    # B032: Possible hardcoded password
+    password = "secret123"  # B032
+
+    # B033: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B033
+
+    # B034: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B034
+
+    # B035: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B035
+
+    # B036: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B036
+
+    # B037: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B037
+
+    # B038: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B038
+
+    # B039: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B039
+
+    # B040: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B040
+
+    # B041: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B041
+
+    # B042: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B042
+
+    # B043: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B043
+
+    # B044: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B044
+
+    # B045: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B045
+
+    # B046: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B046
+
+    # B047: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B047
+
+    # B048: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B048
+
+    # B049: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B049
+
+    # B050: Do not use assert in a loop
+    for i in range(3):
+        assert i >= 0  # B050
+
+    return "Completed flake8-bugbear violations demonstration"
+
+
+if __name__ == "__main__":
+    main()