fixes

app.py

@@ -78,16 +78,26 @@ def test_validator_functionality():
         return False
     
     try:
-        # Test with a simple invalid RDF
-        test_rdf = '<?xml version="1.0"?><rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"><bf:Work/></rdf:RDF>'
+        # Test with minimally valid RDF/XML that matches SHACL targets but is missing required properties
+        # This ensures SHACL finds focus nodes (bf:Text Work) and reports violations
+        test_rdf = '''<?xml version="1.0"?>
+<rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
+         xmlns:bf="http://id.loc.gov/ontologies/bibframe/">
+  <bf:Work rdf:about="http://example.org/work/1">
+    <rdf:type rdf:resource="http://id.loc.gov/ontologies/bibframe/Text"/>
+    <!-- Intentionally missing title, language, content, adminMetadata to trigger SHACL violations -->
+  </bf:Work>
+</rdf:RDF>'''
         conforms, results = validate_rdf(test_rdf.encode('utf-8'), 'monograph')
         
-        # This should fail validation (missing namespace, missing properties)
+        # This should fail validation due to missing required properties
         if conforms:
             print("⚠️ WARNING: Validator returned 'conforms=True' for invalid RDF. Validator may not be working correctly!")
             return False
         else:
-            print(f"✅ Validator test passed. Got expected failure: {results[:100] if results else 'No results'}")
+            preview = (results or '').strip()
+            preview = preview[:200] + ('…' if len(preview) > 200 else '')
+            print(f"✅ Validator test passed. Got expected SHACL violations. Preview: {preview if preview else 'No results text returned'}")
             return True
             
     except Exception as e:
@@ -339,13 +349,14 @@ SAMPLE_VALID_RDF = '''<?xml version="1.0" encoding="UTF-8"?>
 </rdf:RDF>'''
 
 SAMPLE_INVALID_RDF = '''<?xml version="1.0" encoding="UTF-8"?>
-<rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#">
-    <!-- Missing namespace declarations -->
-    <!-- Missing required properties -->
-    <bf:Work rdf:about="http://example.org/work/1">
+<rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
+         xmlns:bf="http://id.loc.gov/ontologies/bibframe/">
+    <!-- Well-formed RDF/XML, but missing required properties to trigger SHACL violations -->
+    <bf:Work rdf:about="http://example.org/work/invalid-1">
+        <!-- Ensure target class is hit so SHACL runs -->
+        <rdf:type rdf:resource="http://id.loc.gov/ontologies/bibframe/Text"/>
+        <!-- Missing proper title structure, language, content, adminMetadata -->
         <bf:title>Incomplete Title</bf:title>
-        <!-- Missing rdf:type -->
-        <!-- Missing proper title structure -->
     </bf:Work>
 </rdf:RDF>'''
 
@@ -376,6 +387,25 @@ def validate_rdf_tool(rdf_content: str, template: str = "monograph") -> dict:
         }
     
     try:
+        # Fast syntax check before SHACL to give clearer errors on XML/prefix issues
+        try:
+            try:
+                import rdflib  # type: ignore
+            except ImportError:
+                rdflib = None  # type: ignore
+            if rdflib:
+                g = rdflib.Graph()  # type: ignore
+                # Parse as RDF/XML; raise on syntax errors like unbound prefixes
+                g.parse(data=rdf_content, format="application/rdf+xml")  # type: ignore
+            else:
+                logger.info("rdflib not installed; skipping pre-parse RDF/XML syntax check")
+        except Exception as parse_err:
+            logger.error(f"RDF/XML parse error before validation: {parse_err}")
+            return {
+                "error": f"RDF/XML parse error: {parse_err}",
+                "conforms": False
+            }
+        
         # Log what we're validating
         logger.info(f"Validating RDF with template '{template}', content length: {len(rdf_content)}")