fix: resolve ruff lint errors to pass CI

- Fix undefined 'repo' variable in align.py (use assessment.repository)
- Add noqa comments for intentional jsonschema import checks
- Fix unused variable warnings in test files

All linters now pass: black, isort, and ruff.

Author: jeremyeder

src/agentready/cli/align.py

@@ -118,7 +118,9 @@ def align(repository, dry_run, attributes, interactive):
         attribute_list = [a.strip() for a in attributes.split(",")]
 
     fixer_service = FixerService()
-    fix_plan = fixer_service.generate_fix_plan(assessment, repo, attribute_list)
+    fix_plan = fixer_service.generate_fix_plan(
+        assessment, assessment.repository, attribute_list
+    )
 
     if not fix_plan.fixes:
         click.echo("\n✅ No automatic fixes available.")

src/agentready/services/schema_validator.py

@@ -5,7 +5,7 @@
 from typing import Any
 
 try:
-    import jsonschema
+    import jsonschema  # noqa: F401
     from jsonschema import Draft7Validator, validators
 
     JSONSCHEMA_AVAILABLE = True

tests/integration/test_schema_commands.py

@@ -72,7 +72,7 @@ def sample_report(tmp_path):
 def test_validate_report_valid(runner, sample_report):
     """Test validate-report command with valid report."""
     try:
-        import jsonschema
+        import jsonschema  # noqa: F401
     except ImportError:
         pytest.skip("jsonschema not installed")
 
@@ -85,7 +85,7 @@ def test_validate_report_valid(runner, sample_report):
 def test_validate_report_nonexistent(runner, tmp_path):
     """Test validate-report command with nonexistent file."""
     try:
-        import jsonschema
+        import jsonschema  # noqa: F401
     except ImportError:
         pytest.skip("jsonschema not installed")
 
@@ -100,7 +100,7 @@ def test_validate_report_nonexistent(runner, tmp_path):
 def test_validate_report_invalid_json(runner, tmp_path):
     """Test validate-report command with invalid JSON."""
     try:
-        import jsonschema
+        import jsonschema  # noqa: F401
     except ImportError:
         pytest.skip("jsonschema not installed")
 
@@ -117,7 +117,7 @@ def test_validate_report_invalid_json(runner, tmp_path):
 def test_validate_report_no_strict(runner, sample_report):
     """Test validate-report with --no-strict option."""
     try:
-        import jsonschema
+        import jsonschema  # noqa: F401
     except ImportError:
         pytest.skip("jsonschema not installed")
 
@@ -174,7 +174,7 @@ def test_migrate_report_unsupported_version(runner, sample_report):
 def test_validate_then_migrate_workflow(runner, sample_report, tmp_path):
     """Test full workflow: validate, migrate, validate again."""
     try:
-        import jsonschema
+        import jsonschema  # noqa: F401
     except ImportError:
         pytest.skip("jsonschema not installed")
 

tests/unit/learners/test_llm_enricher.py

@@ -415,7 +415,7 @@ def test_call_claude_api_builds_prompt(
     """Test that _call_claude_api builds correct prompt."""
     enricher = LLMEnricher(mock_anthropic_client, cache_dir=tmp_path)
 
-    result = enricher._call_claude_api(
+    enricher._call_claude_api(
         basic_skill, sample_finding, sample_repository, "code samples"
     )
 

tests/unit/test_assessment_cache.py

@@ -20,7 +20,7 @@ def test_cache_directory_creation(self):
         """Test that cache directory is created if it doesn't exist."""
         with TemporaryDirectory() as tmpdir:
             cache_dir = Path(tmpdir) / "deep" / "nested" / "cache"
-            cache = AssessmentCache(cache_dir)
+            AssessmentCache(cache_dir)
             assert cache_dir.exists()
 
     def test_get_cache_stats_empty(self):

tests/unit/test_security_controls.py

@@ -105,7 +105,7 @@ def test_sql_injection_in_repo_url(self):
             commit_hash = "abc123"
 
             # Should not crash or execute SQL
-            count = cache.invalidate(malicious_url, commit_hash)
+            cache.invalidate(malicious_url, commit_hash)
 
             # Verify table still exists by querying
             stats = cache.get_stats()