@pdtf/schemas 3.5.1-7 → 3.5.1-8

package/index.js

@@ -104,7 +104,7 @@ const transactionSchemas = {
 };
 
 // Custom array merge function for oneOf arrays
-const arrayMerge = (target, source) => {
+const arrayMerge = (target, source, options) => {
   // For string arrays (enum, required, etc.)
   if (
     target.length > 0 &&
@@ -114,16 +114,23 @@ const arrayMerge = (target, source) => {
     source[0] &&
     typeof source[0] === "string"
   ) {
-    // For required arrays, combine them (remove duplicates)
-    // Check if this looks like a required array (contains field names, not enum values)
-    const isRequiredArray = target.some(
+    // Detect if this is a required array by checking the parent key
+    // Required arrays are property names (camelCase, no spaces)
+    // Enum arrays are values (can contain spaces, capitals, etc.)
+    const isRequiredArray = target.every(
       (item) =>
         typeof item === "string" &&
         item.length > 0 &&
-        !item.includes("Yes") &&
-        !item.includes("No") &&
-        !item.includes("Not known") &&
-        !item.includes("To be connected")
+        // Required field names are typically camelCase without spaces
+        !item.includes(" ") &&
+        // First character is usually lowercase for field names
+        (item[0] === item[0].toLowerCase() || item.startsWith("is") || item.startsWith("has"))
+    ) && source.every(
+      (item) =>
+        typeof item === "string" &&
+        item.length > 0 &&
+        !item.includes(" ") &&
+        (item[0] === item[0].toLowerCase() || item.startsWith("is") || item.startsWith("has"))
     );
 
     if (isRequiredArray) {
@@ -137,7 +144,7 @@ const arrayMerge = (target, source) => {
       return combined;
     }
 
-    // For other string arrays (like enum), use source if it exists, otherwise use target
+    // For other string arrays (like enum), replace with source
     return source.length > 0 ? source : target;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pdtf/schemas",
-  "version": "3.5.1-7",
+  "version": "3.5.1-8",
   "description": "Property Data Trust Framework Schemas and Utilities",
   "main": "index.js",
   "directories": {

package/src/tests/v3/transactionSchema.test.js

@@ -591,3 +591,25 @@ test("waterAndDrainage state with mainsFoulDrainage No missing offMainsDrainageS
   );
   expect(relevantError).toBeDefined();
 });
+
+test("TA7 overlay organisesBuildingInsurance enum has exactly three items", () => {
+  const schema = getTransactionSchema(schemaId, ["ta7ed3"]);
+
+  // Navigate to the organisesBuildingInsurance property in the merged schema
+  const organisesBuildingInsurance =
+    schema.properties?.propertyPack?.properties?.ownership?.properties
+      ?.ownershipsToBeTransferred?.items?.oneOf?.[2]?.properties
+      ?.leaseholdInformation?.properties?.contactDetails?.properties
+      ?.serviceContactAssignments?.properties?.organisesBuildingInsurance;
+
+  expect(organisesBuildingInsurance).toBeDefined();
+  expect(organisesBuildingInsurance.enum).toBeDefined();
+
+  // The TA7 overlay defines exactly 3 enum values, which should replace the base schema's 6 values
+  expect(organisesBuildingInsurance.enum).toHaveLength(3);
+  expect(organisesBuildingInsurance.enum).toEqual([
+    "the Lessees",
+    "Management Company",
+    "Landlord",
+  ]);
+});