chore: Adds local linter to Makefile

Author: agustingroh

Makefile

@@ -50,12 +50,17 @@ publish_test:  ## Publish the Python package to TestPyPI
 	@echo "Publishing package to TestPyPI..."
 	twine upload --repository testpypi dist/*
 
-linter: ## Run ruff linter with docker
-	docker run --rm -v $(PWD):/src -w /src ghcr.io/astral-sh/ruff:0.14.2 check $$(git diff --name-only --diff-filter=ACMR | grep '\.py$$' | xargs)
+linter-docker: ## Run ruff linter with docker
+	@./tools/linter.sh --docker
 
-fix-linter: ## Run ruff linter with docker
-	docker run --rm -v $(PWD):/src -w /src ghcr.io/astral-sh/ruff:0.14.2 check $$(git diff --name-only --diff-filter=ACMR | grep '\.py$$' | xargs) --fix
+linter-docker-fix: ## Run ruff linter with docker and auto-fix
+	@./tools/linter.sh --docker --fix
 
+linter: ## Run ruff linter locally
+	@./tools/linter.sh
+
+linter-fix: ## Run ruff linter locally with auto-fix
+	@./tools/linter.sh --fix
 
 publish:  ## Publish Python package to PyPI
 	@echo "Publishing package to PyPI..."

src/scanoss/gitlabqualityreport.py

@@ -141,7 +141,7 @@ def _write_output(self, data: list[CodeQuality], output_file: str = None) -> boo
                 file.close()
                 self.print_debug(f"Wrote output to file: {output_file}")
             else:
-                self.print_debug(f"Wrote output to 'stdout'")
+                self.print_debug("Wrote output to 'stdout'")
             return True
         except Exception as e:
             self.print_stderr(f'Error writing output: {str(e)}')
@@ -184,7 +184,7 @@ def _produce_from_str(self, json_str: str, output_file: str = None) -> bool:
         self.print_debug(f"Parsing JSON string of length: {len(json_str)}")
         try:
             data = json.loads(json_str)
-            self.print_debug(f"Successfully parsed JSON data")
+            self.print_debug("Successfully parsed JSON data")
             self.print_trace(f"Parsed data structure: {type(data)}")
         except Exception as e:
             self.print_stderr(f'ERROR: Problem parsing input JSON: {e}')

src/scanoss/inspection/summary/match_summary.py

@@ -216,7 +216,10 @@ def _get_matches_summary(self) -> ComponentMatchSummary:
                     gitlab_matches_summary.files.append(summary_item)
                     self.print_trace(f"Added file match for {file_name}")
 
-        self.print_debug(f"Match summary complete: {len(gitlab_matches_summary.files)} file matches, {len(gitlab_matches_summary.snippet)} snippet matches")
+        self.print_debug(
+            f"Match summary complete: {len(gitlab_matches_summary.files)} file matches, "
+            f"{len(gitlab_matches_summary.snippet)} snippet matches"
+        )
 
         return gitlab_matches_summary
 
@@ -237,7 +240,10 @@ def _markdown(self, gitlab_matches_summary: ComponentMatchSummary) -> str:
             self.print_debug("No matches to format - returning empty string")
             return ""
 
-        self.print_trace(f"Formatting {len(gitlab_matches_summary.files)} file matches and {len(gitlab_matches_summary.snippet)} snippet matches")
+        self.print_trace(
+            f"Formatting {len(gitlab_matches_summary.files)} file matches and "
+            f"{len(gitlab_matches_summary.snippet)} snippet matches"
+        )
 
         # Define table headers
         file_match_headers = ['File', 'License', 'Similarity', 'PURL', 'Version']
@@ -304,7 +310,10 @@ def run(self):
         4. Outputs to file or stdout
         """
         self.print_debug("Starting match summary generation process")
-        self.print_trace(f"Configuration - Results path: {self.scanoss_results_path}, Output: {self.output}, Line range prefix: {self.line_range_prefix}")
+        self.print_trace(
+            f"Configuration - Results path: {self.scanoss_results_path}, Output: {self.output}, "
+            f"Line range prefix: {self.line_range_prefix}"
+        )
 
         # Load and process scan results into categorized matches
         self.print_trace("Loading and processing scan results")

tools/linter.sh

@@ -0,0 +1,50 @@
+#!/usr/bin/env bash
+# SPDX-License-Identifier: MIT
+#
+# Lint Python files changed since merge base with origin/main
+# Usage: lint_changed_files.sh [--fix] [--docker]
+
+set -e
+
+# Parse arguments
+FIX_FLAG=""
+USE_DOCKER=false
+
+while [[ $# -gt 0 ]]; do
+    case $1 in
+        --fix)
+            FIX_FLAG="--fix"
+            shift
+            ;;
+        --docker)
+            USE_DOCKER=true
+            shift
+            ;;
+        *)
+            echo "Unknown option: $1"
+            echo "Usage: $0 [--fix] [--docker]"
+            exit 1
+            ;;
+    esac
+done
+
+# Find merge base with origin/main
+merge_base=$(git merge-base origin/main HEAD)
+
+# Get all changed Python files since merge base
+files=$(git diff --name-only "$merge_base" HEAD | grep '\.py$' || true)
+
+# Check if there are any Python files changed
+if [ -z "$files" ]; then
+    echo "No Python files changed"
+    exit 0
+fi
+
+# Run linter
+if [ "$USE_DOCKER" = true ]; then
+    # Run with Docker
+    docker run --rm -v "$(pwd)":/src -w /src ghcr.io/astral-sh/ruff:0.14.2 check $files $FIX_FLAG
+else
+    # Run locally
+    python3 -m ruff check $files $FIX_FLAG
+fi