fix: schema backwards compat for attributes_skipped key

specs/001-agentready-scorer/contracts/assessment-schema.json

@@ -10,11 +10,14 @@
     "overall_score",
     "certification_level",
     "attributes_assessed",
-    "attributes_skipped",
     "attributes_total",
     "findings",
     "duration_seconds"
   ],
+  "anyOf": [
+    {"required": ["attributes_skipped"]},
+    {"required": ["attributes_not_assessed"]}
+  ],
   "properties": {
     "schema_version": {
       "type": "string",
@@ -60,7 +63,14 @@
     "attributes_skipped": {
       "type": "integer",
       "minimum": 0,
-      "maximum": 25
+      "maximum": 25,
+      "description": "Canonical key name for skipped/not-assessed attributes"
+    },
+    "attributes_not_assessed": {
+      "type": "integer",
+      "minimum": 0,
+      "maximum": 25,
+      "description": "Deprecated alias for attributes_skipped (backwards compatibility)"
     },
     "attributes_total": {
       "type": "integer",

src/agentready/models/assessment.py

@@ -88,7 +88,7 @@ def to_dict(self) -> dict:
             "overall_score": self.overall_score,
             "certification_level": self.certification_level,
             "attributes_assessed": self.attributes_assessed,
-            "attributes_not_assessed": self.attributes_not_assessed,
+            "attributes_skipped": self.attributes_not_assessed,
             "attributes_total": self.attributes_total,
             "findings": [f.to_dict() for f in self.findings],
             "config": self.config.to_dict() if self.config else None,

src/agentready/services/learning_service.py

@@ -146,7 +146,7 @@ def extract_patterns_from_file(
             certification_level=assessment_data["certification_level"],
             attributes_assessed=assessment_data["attributes_assessed"],
             attributes_not_assessed=assessment_data.get(
-                "attributes_not_assessed", assessment_data.get("attributes_skipped", 0)
+                "attributes_skipped", assessment_data.get("attributes_not_assessed", 0)
             ),
             attributes_total=assessment_data["attributes_total"],
             findings=findings,

tests/fixtures/assessment_fixtures.py

@@ -68,7 +68,7 @@ def create_test_assessment_json(
         "overall_score": overall_score,
         "certification_level": certification_level,
         "attributes_assessed": num_findings,
-        "attributes_not_assessed": 0,
+        "attributes_skipped": 0,
         "attributes_total": num_findings,
         "findings": findings,
         "config": None,  # CRITICAL: Must be present in current schema

tests/unit/test_learning_service.py

@@ -62,7 +62,7 @@ def sample_assessment_file(temp_dir):
         "overall_score": 85.0,
         "certification_level": "Gold",
         "attributes_assessed": 2,
-        "attributes_not_assessed": 0,
+        "attributes_skipped": 0,
         "attributes_total": 2,
         "findings": [
             {
@@ -352,7 +352,7 @@ def test_extract_patterns_missing_assessment_keys(self, temp_dir):
 
     def test_extract_patterns_with_old_schema_key(self, temp_dir):
         """Test extract_patterns handles old schema key names."""
-        # Old schema used "attributes_skipped" instead of "attributes_not_assessed"
+        # Old reports used "attributes_not_assessed" instead of "attributes_skipped"
         old_schema_assessment = {
             "schema_version": "1.0.0",
             "timestamp": "2025-11-22T06:00:00",
@@ -428,7 +428,7 @@ def test_extract_patterns_empty_findings(self, mock_extractor, temp_dir):
             "overall_score": 0.0,
             "certification_level": "Needs Improvement",
             "attributes_assessed": 0,
-            "attributes_not_assessed": 1,
+            "attributes_skipped": 1,
             "attributes_total": 1,
             "findings": [
                 create_dummy_finding()