lintro 0.9.0__tar.gz → 0.12.0__tar.gz

{lintro-0.9.0/lintro.egg-info → lintro-0.12.0}/PKG-INFO

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

{lintro-0.9.0 → lintro-0.12.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
 )
@@ -209,8 +209,10 @@ Lintro preserves all Ruff rule categories:
 | **Pyflakes**              | F      | Logical errors and undefined names |
 | **pycodestyle**           | E, W   | PEP 8 style violations             |
 | **isort**                 | I      | Import sorting issues              |
+| **pep8-naming**           | N      | Naming conventions (PEP 8)         |
 | **pydocstyle**            | D      | Docstring style violations         |
 | **pyupgrade**             | UP     | Python upgrade suggestions         |
+| **flake8-annotations**    | ANN    | Type annotation requirements       |
 | **flake8-bugbear**        | B      | Bug detection and complexity       |
 | **flake8-comprehensions** | C4     | Comprehension improvements         |
 | **flake8-simplify**       | SIM    | Code simplification suggestions    |

{lintro-0.9.0 → lintro-0.12.0}/lintro/__init__.py

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

{lintro-0.9.0 → lintro-0.12.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.9.0 → lintro-0.12.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.9.0 → lintro-0.12.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.9.0 → lintro-0.12.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
 

{lintro-0.9.0 → lintro-0.12.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
 
 

{lintro-0.9.0 → lintro-0.12.0/lintro.egg-info}/PKG-INFO

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

{lintro-0.9.0 → lintro-0.12.0}/lintro.egg-info/SOURCES.txt

@@ -125,8 +125,11 @@ test_samples/actionlint_violations.yml
 test_samples/bandit_violations.py
 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_clean.py
+test_samples/ruff_naming_violations.py
 test_samples/ruff_violations.py
 test_samples/yaml_violations.yml
 tests/__init__.py
@@ -145,8 +148,11 @@ 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_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
 tests/scripts/__init__.py
 tests/scripts/test_ci_post_pr_comment.py

{lintro-0.9.0 → lintro-0.12.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "lintro"
-version = "0.9.0"
+version = "0.12.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,11 +65,8 @@ indent-style = "space"
 skip-magic-trailing-comma = false
 
 [tool.ruff.lint]
-select = [ "E", "F", "W", "I", "COM", "D", "UP",]
-ignore = []
-
-[tool.ruff.pydocstyle]
-convention = "google"
+select = [ "E", "F", "W", "I", "COM", "N", "D", "UP", "ANN", "B",]
+ignore = [ "ANN",]
 
 [tool.lintro.post_checks]
 enabled = true
@@ -81,3 +78,6 @@ testpaths = [ "tests",]
 python_files = "test_*.py"
 python_classes = "Test*"
 python_functions = "test_*"
+
+[tool.ruff.lint.pydocstyle]
+convention = "google"

lintro-0.12.0/test_samples/ruff_annotations_violations.py

@@ -0,0 +1,53 @@
+"""Sample file with flake8-annotations (ANN) violations for testing Ruff.
+
+Intentionally violates several ANN-rules, such as:
+- ANN101: missing type annotation for self
+- ANN102: missing type annotation for cls
+- ANN201: missing return type annotation for public function
+- ANN204: missing return type annotation for special method
+- ANN205: missing return type annotation for static method
+- ANN003: missing type annotation for **kwargs
+"""
+
+from typing import Any
+
+
+class SampleClass:
+    """Sample class with annotation violations."""
+
+    def __init__(self, value: int):  # ANN101: missing type for self
+        self.value = value
+
+    @classmethod
+    def from_string(cls, s: str):  # ANN102: missing type for cls
+        return cls(int(s))
+
+    @staticmethod
+    def helper_func(x: int):  # ANN205: missing return type
+        return x * 2
+
+    def __str__(self):  # ANN204: missing return type for special method
+        return f"SampleClass({self.value})"
+
+    def public_method(self, data: list[str]):  # ANN201: missing return type
+        return len(data)
+
+    def kwargs_method(self, **kwargs):  # ANN003: missing type for **kwargs
+        return kwargs
+
+
+def public_function(a: int, b: str):  # ANN201: missing return type
+    """Public function without return type annotation."""
+    return f"{a}: {b}"
+
+
+def kwargs_function(**kwargs):  # ANN003: missing type for **kwargs
+    """Function with kwargs without type annotation."""
+    return kwargs
+
+
+def nested_function():
+    """Function that returns a nested function."""
+    def inner(x: int):  # ANN201: missing return type
+        return x + 1
+    return inner

lintro-0.12.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()

lintro-0.12.0/test_samples/ruff_naming_violations.py

@@ -0,0 +1,26 @@
+"""Sample file with pep8-naming (N) violations for testing Ruff.
+
+Intentionally violates several N-rules, such as:
+- N802: function name should be lowercase
+- N803: argument name should be lowercase
+- N806: variable in function should be lowercase
+- N815: mixedCase variable in class scope
+"""
+
+
+def BadFunctionName(BadArg: int) -> int:  # N802, N803
+    BadLocal = BadArg + 1  # N806
+    return BadLocal
+
+
+class SampleClass:
+    camelCaseAttr = 1  # N815
+
+    def GoodMethod(self, BadParam: int) -> int:  # N803
+        BadLocalVar = BadParam * 2  # N806
+        return BadLocalVar
+
+
+CONSTANT_ok = 1  # not enforcing caps here; focus on N8xx basics
+
+

{lintro-0.9.0 → lintro-0.12.0}/tests/cli/test_cli.py

@@ -9,7 +9,7 @@ from assertpy import assert_that
 from lintro.cli import cli
 
 
-def test_cli_help():
+def test_cli_help() -> None:
     """Test that CLI shows help."""
     from click.testing import CliRunner
 
@@ -19,7 +19,7 @@ def test_cli_help():
     assert_that(result.output).contains("Lintro")
 
 
-def test_cli_version():
+def test_cli_version() -> None:
     """Test that CLI shows version."""
     from click.testing import CliRunner
 
@@ -29,7 +29,7 @@ def test_cli_version():
     assert_that(result.output.lower()).contains("version")
 
 
-def test_cli_commands_registered():
+def test_cli_commands_registered() -> None:
     """Test that all commands are registered."""
     from click.testing import CliRunner
 
@@ -42,7 +42,7 @@ def test_cli_commands_registered():
     assert_that(result.exit_code).is_equal_to(0)
 
 
-def test_main_function():
+def test_main_function() -> None:
     """Test the main function."""
     from click.testing import CliRunner
 
@@ -52,7 +52,7 @@ def test_main_function():
     assert_that(result.output).contains("Lintro")
 
 
-def test_cli_command_aliases():
+def test_cli_command_aliases() -> None:
     """Test that command aliases work."""
     from click.testing import CliRunner
 
@@ -65,7 +65,7 @@ def test_cli_command_aliases():
     assert_that(result.exit_code).is_equal_to(0)
 
 
-def test_cli_with_no_args():
+def test_cli_with_no_args() -> None:
     """Test CLI with no arguments."""
     from click.testing import CliRunner
 
@@ -75,7 +75,7 @@ def test_cli_with_no_args():
     assert_that(result.output).is_equal_to("")
 
 
-def test_main_module_execution():
+def test_main_module_execution() -> None:
     """Test that __main__.py can be executed directly."""
     with patch.object(sys, "argv", ["lintro", "--help"]):
         import lintro.__main__
@@ -83,7 +83,7 @@ def test_main_module_execution():
         assert_that(lintro.__main__).is_not_none()
 
 
-def test_main_module_as_script():
+def test_main_module_as_script() -> None:
     """Test that __main__.py works when run as a script."""
     result = subprocess.run(
         [sys.executable, "-m", "lintro", "--help"],

{lintro-0.9.0 → lintro-0.12.0}/tests/conftest.py

@@ -32,7 +32,7 @@ def _ensure_test_docker_images_built() -> None:
     return
 
 
-def pytest_collection_modifyitems(config, items):
+def pytest_collection_modifyitems(config, items) -> None:
     """Optionally skip docker tests locally unless explicitly enabled.
 
     If `LINTRO_RUN_DOCKER_TESTS` is not set to "1", mark tests under