check existing entries for schemaSerialization

Author: NiloCK

packages/studio-ui/src/utils/courseConfigRegistration.ts

@@ -42,7 +42,7 @@ export interface ProcessedDataShape {
 }
 
 /**
- * Check if a data shape is already registered in the course config
+ * Check if a data shape is already registered in the course config with valid schema
  */
 export function isDataShapeRegistered(
   dataShape: ProcessedDataShape,
@@ -53,7 +53,10 @@ export function isDataShapeRegistered(
     course: dataShape.course,
   });
 
-  return courseConfig.dataShapes.some((ds) => ds.name === namespacedName);
+  const existingDataShape = courseConfig.dataShapes.find((ds) => ds.name === namespacedName);
+
+  // Must exist AND have serialized schema to be considered "registered"
+  return existingDataShape !== undefined && existingDataShape.serializedZodSchema !== undefined;
 }
 
 /**
@@ -122,7 +125,7 @@ export function registerDataShape(
 ): boolean {
   if (isDataShapeRegistered(dataShape, courseConfig)) {
     console.log(
-      `   ℹ️  DataShape '${dataShape.name}' from '${dataShape.course}' already registered`
+      `   ℹ️  DataShape '${dataShape.name}' from '${dataShape.course}' already registered with schema`
     );
     return false;
   }
@@ -141,6 +144,13 @@ export function registerDataShape(
     serializedZodSchema = undefined;
   }
 
+  // Check if entry exists but without schema (legacy entry) - remove it
+  const existingIndex = courseConfig.dataShapes.findIndex((ds) => ds.name === namespacedName);
+  if (existingIndex !== -1) {
+    console.log(`   🔧 Replacing legacy DataShape entry: ${namespacedName}`);
+    courseConfig.dataShapes.splice(existingIndex, 1);
+  }
+
   courseConfig.dataShapes.push({
     name: namespacedName,
     questionTypes: [],