@seedprotocol/sdk 0.4.3 → 0.4.4

package/dist/cjs/{ModelProperty-MkN5Rmx7.js → ModelProperty-BRKRREYP.js}

@@ -1,7 +1,7 @@
 'use strict';
 
 var xstate = require('xstate');
-var main_cjs = require('./index-BmIVfqGN.js');
+var main_cjs = require('./index-C3j3m34a.js');
 require('pluralize');
 var drizzleOrm = require('drizzle-orm');
 var debug = require('debug');
@@ -52,9 +52,9 @@ const getSchemaFilePath = (name, version, schemaFileId) => {
  * @throws Error if schema not found or missing schemaFileId
  */
 async function getSchemaFileId(schemaName) {
-    const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
+    const { BaseDb } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.BaseDb$1; });
     const db = BaseDb.getAppDb();
-    const { schemas } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.SchemaSchema; });
+    const { schemas } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.SchemaSchema; });
     const { eq, desc } = await import('drizzle-orm');
     const dbSchema = await db
         .select()
@@ -67,6 +67,40 @@ async function getSchemaFileId(schemaName) {
     }
     return dbSchema[0].schemaFileId;
 }
+/**
+ * Write the full schema to a new version file (e.g. when new models were added).
+ * Used when _editedProperties contains 'schema:models' and there are no property-level updates.
+ * @param schemaName - Schema name
+ * @param schema - Full schema object (e.g. from _buildModelsFromInstances)
+ * @returns The file path of the new schema version
+ */
+async function writeFullSchemaNewVersion(schemaName, schema) {
+    const latestVersion = await main_cjs.getLatestSchemaVersion(schemaName);
+    const newVersion = latestVersion + 1;
+    const schemaWithNewVersion = {
+        ...schema,
+        version: newVersion,
+        metadata: {
+            ...schema.metadata,
+            updatedAt: new Date().toISOString(),
+        },
+        migrations: [
+            ...(schema.migrations || []),
+            {
+                version: newVersion,
+                timestamp: new Date().toISOString(),
+                description: 'New schema version (e.g. new models added)',
+                changes: [{ type: 'full_schema_write' }],
+            },
+        ],
+    };
+    const newFilePath = getSchemaFilePath(schemaName, newVersion, schema.id ?? (await getSchemaFileId(schemaName)));
+    const newContent = JSON.stringify(schemaWithNewVersion, null, 2);
+    await main_cjs.BaseFileManager.saveFile(newFilePath, newContent);
+    await main_cjs.BaseFileManager.waitForFileWithContent(newFilePath);
+    logger$4(`Created new schema version ${newVersion} for ${schemaName} at ${newFilePath}`);
+    return newFilePath;
+}
 /**
  * Get model name from modelId
  * @param modelId - The model ID to look up
@@ -290,11 +324,9 @@ async function loadSchemaWithRenames(schemaFilePath, modelRenames) {
             },
         ])),
     };
-    // Use dynamic import to break circular dependency
-    const { createModelsFromJson } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.json; });
     // Generate schema ID if missing
     if (!schemaFile.id) {
-        const { generateId } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.index; });
+        const { generateId } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.index; });
         schemaFile.id = generateId();
         logger$4('Generated schema ID for schema:', schemaFile.id);
     }
@@ -302,7 +334,7 @@ async function loadSchemaWithRenames(schemaFilePath, modelRenames) {
     // This ensures Model instances are created with correct IDs
     const modelFileIds = new Map();
     const propertyFileIds = new Map();
-    const { generateId } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.index; });
+    const { generateId } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.index; });
     for (const [modelName, model] of Object.entries(schemaFile.models)) {
         // Generate model ID if missing
         if (!model.id) {
@@ -324,7 +356,7 @@ async function loadSchemaWithRenames(schemaFilePath, modelRenames) {
         }
     }
     // Convert JSON models to Model classes, passing modelFileIds and propertyFileIds so Model instances use correct IDs
-    const modelDefinitions = await createModelsFromJson(importData, modelFileIds, propertyFileIds);
+    const modelDefinitions = await main_cjs.createModelsFromJson(importData, modelFileIds, propertyFileIds);
     // Convert schema file metadata to schema input for database
     const schemaInput = {
         name: schemaName,
@@ -337,7 +369,7 @@ async function loadSchemaWithRenames(schemaFilePath, modelRenames) {
         updatedAt: new Date(schemaFile.metadata.updatedAt).getTime(),
     };
     // Use dynamic import to break circular dependency
-    const { addSchemaToDb, addModelsToDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.db; });
+    const { addSchemaToDb, addModelsToDb } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
     // Add schema to database with schemaFileId
     const schemaRecord = await addSchemaToDb(schemaInput, schemaFile.id);
     // Add models to database with model renames handled and schemaFileIds
@@ -362,11 +394,11 @@ const logger$3 = debug('seedSdk:modelProperty:actors:saveToSchema');
  */
 async function getSchemaNameFromModel(modelName) {
     // Get the latest schema files and find which one contains this model
-    const { listLatestSchemaFiles } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema$1; });
+    const { listLatestSchemaFiles } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.schema$1; });
     const latestSchemas = await listLatestSchemaFiles();
     for (const schema of latestSchemas) {
         try {
-            const { BaseFileManager } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseFileManager$1; });
+            const { BaseFileManager } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.BaseFileManager$1; });
             const content = await BaseFileManager.readFileAsString(schema.filePath);
             const schemaFile = JSON.parse(content);
             if (schemaFile.models && schemaFile.models[modelName]) {
@@ -383,7 +415,7 @@ async function getSchemaNameFromModel(modelName) {
 const saveToSchema = xstate.fromCallback(({ sendBack, input: { context } }) => {
     const _saveToSchema = async () => {
         // Use dynamic import to break circular dependency
-        const { SchemaValidationService } = await Promise.resolve().then(function () { return require('./SchemaValidationService-BgIzc3-r.js'); });
+        const { SchemaValidationService } = await Promise.resolve().then(function () { return require('./SchemaValidationService-DUtQUerg.js'); });
         const validationService = new SchemaValidationService();
         // Validate property structure before saving
         const validationResult = validationService.validatePropertyStructure(context);
@@ -411,8 +443,8 @@ const saveToSchema = xstate.fromCallback(({ sendBack, input: { context } }) => {
         await updateModelProperties(schemaName, [propertyUpdate]);
         // Clear isEdited flag in database after saving to schema file
         try {
-            const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
-            const { properties: propertiesTable, models: modelsTable } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema; });
+            const { BaseDb } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.BaseDb$1; });
+            const { properties: propertiesTable, models: modelsTable } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.schema; });
             const { eq, and } = await import('drizzle-orm');
             const db = BaseDb.getAppDb();
             if (db && context.modelName && context.name) {
@@ -465,45 +497,86 @@ var saveToSchema$1 = /*#__PURE__*/Object.freeze({
 const logger$2 = debug('seedSdk:modelProperty:actors:compareAndMarkDraft');
 const compareAndMarkDraft = xstate.fromCallback(({ sendBack, input: { context } }) => {
     const _compareAndMarkDraft = async () => {
+        // Fill modelName/dataType from _originalValues when missing, then from DB by schemaFileId
+        let fullContext = {
+            ...context,
+            modelName: context.modelName ?? context._originalValues?.modelName,
+            dataType: context.dataType ?? context._originalValues?.dataType,
+        };
+        const schemaFileIdForResolve = fullContext._propertyFileId || (typeof fullContext.id === 'string' ? fullContext.id : undefined);
+        if (schemaFileIdForResolve && (fullContext.modelName === undefined || fullContext.dataType === undefined)) {
+            try {
+                const { getPropertyModelNameAndDataType, getModelNameByModelId } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
+                let fromDb;
+                for (let attempt = 0; attempt < 6; attempt++) {
+                    fromDb = await getPropertyModelNameAndDataType(schemaFileIdForResolve);
+                    if (fromDb)
+                        break;
+                    if (attempt < 5)
+                        await new Promise((r) => setTimeout(r, 40));
+                }
+                if (fromDb) {
+                    fullContext = {
+                        ...fullContext,
+                        modelName: fullContext.modelName ?? fromDb.modelName,
+                        dataType: fullContext.dataType ?? fromDb.dataType,
+                    };
+                }
+                if (fullContext.modelName === undefined && schemaFileIdForResolve) {
+                    const mod = await Promise.resolve().then(function () { return ModelProperty$1; });
+                    const ModelProperty = mod?.ModelProperty ?? mod?.default;
+                    const pendingModelId = ModelProperty?.getPendingModelId?.(schemaFileIdForResolve);
+                    if (pendingModelId != null) {
+                        const modelName = await getModelNameByModelId(pendingModelId);
+                        if (modelName) {
+                            fullContext = { ...fullContext, modelName };
+                        }
+                    }
+                }
+            }
+            catch {
+                // ignore
+            }
+        }
+        const schemaFileId = fullContext._propertyFileId || (typeof fullContext.id === 'string' ? fullContext.id : undefined);
         // If _originalValues is not set, we still need to save to database if the property exists
         // This handles the case where the name is changed before _originalValues is initialized
-        if (!context._originalValues) {
+        if (!fullContext._originalValues) {
             logger$2('No original values to compare against');
-            logger$2(`[compareAndMarkDraft] Context: modelName=${context.modelName}, name=${context.name}, id=${context.id}, _propertyFileId=${context._propertyFileId}`);
+            logger$2(`[compareAndMarkDraft] Context: modelName=${fullContext.modelName}, name=${fullContext.name}, id=${fullContext.id}, _propertyFileId=${fullContext._propertyFileId}`);
             // If we have a name and modelName, try to save to database anyway
             // This ensures name changes are persisted even if _originalValues isn't initialized yet
             // We need either schemaFileId (id or _propertyFileId) to find the property in the database
-            const schemaFileId = context._propertyFileId || (typeof context.id === 'string' ? context.id : undefined);
-            if (context.modelName && context.name && schemaFileId) {
+            if (fullContext.modelName && fullContext.name && schemaFileId) {
                 logger$2(`[compareAndMarkDraft] _originalValues not set, but saving to database anyway for property ${context.modelName}:${context.name} (schemaFileId: ${schemaFileId})`);
                 try {
-                    const { savePropertyToDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.db; });
+                    const { savePropertyToDb } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
                     // Ensure _propertyFileId is set for savePropertyToDb to find the property
                     const contextWithFileId = {
-                        ...context,
+                        ...fullContext,
                         _propertyFileId: schemaFileId,
                     };
                     await savePropertyToDb(contextWithFileId);
-                    logger$2(`[compareAndMarkDraft] Successfully saved property ${context.modelName}:${context.name} to database (no _originalValues)`);
+                    logger$2(`[compareAndMarkDraft] Successfully saved property ${fullContext.modelName}:${fullContext.name} to database (no _originalValues)`);
                 }
                 catch (error) {
                     logger$2(`[compareAndMarkDraft] Error saving property to database (no _originalValues): ${error}`);
                     // Don't throw - this is a best-effort save, but log the error for debugging
-                    console.error(`[compareAndMarkDraft] Failed to save property ${context.modelName}:${context.name}:`, error);
+                    console.error(`[compareAndMarkDraft] Failed to save property ${fullContext.modelName}:${fullContext.name}:`, error);
                 }
             }
             else {
-                logger$2(`[compareAndMarkDraft] Cannot save property ${context.modelName}:${context.name} - missing required fields (schemaFileId: ${schemaFileId})`);
+                logger$2(`[compareAndMarkDraft] Cannot save property ${fullContext.modelName}:${fullContext.name} - missing required fields (schemaFileId: ${schemaFileId})`);
             }
             return;
         }
-        logger$2(`[compareAndMarkDraft] Comparing: context.name=${context.name}, _originalValues.name=${context._originalValues?.name}`);
+        logger$2(`[compareAndMarkDraft] Comparing: context.name=${fullContext.name}, _originalValues.name=${fullContext._originalValues?.name}`);
         // Compare current values with original
         // Only compare property fields, not internal fields
         const propertyFields = ['name', 'dataType', 'ref', 'refModelName', 'refModelId', 'refValueType', 'storageType', 'localStorageDir', 'filenameSuffix', 'modelName', 'modelId'];
         const hasChanges = propertyFields.some(key => {
-            const currentValue = context[key];
-            const originalValue = context._originalValues?.[key];
+            const currentValue = fullContext[key];
+            const originalValue = fullContext._originalValues?.[key];
             // Handle name changes specifically
             if (key === 'name') {
                 const nameChanged = currentValue !== originalValue;
@@ -514,8 +587,8 @@ const compareAndMarkDraft = xstate.fromCallback(({ sendBack, input: { context }
             }
             // Handle ref fields - compare by name
             if (key === 'ref' || key === 'refModelName') {
-                const currentRef = context.refModelName || context.ref;
-                const originalRef = context._originalValues?.refModelName || context._originalValues?.ref;
+                const currentRef = fullContext.refModelName || fullContext.ref;
+                const originalRef = fullContext._originalValues?.refModelName || fullContext._originalValues?.ref;
                 // Both undefined/null means no ref, so they're the same
                 if (!currentRef && !originalRef)
                     return false;
@@ -533,53 +606,55 @@ const compareAndMarkDraft = xstate.fromCallback(({ sendBack, input: { context }
             return currentValue !== originalValue;
         });
         if (hasChanges) {
-            logger$2(`Property ${context.modelName}:${context.name} has changes, marking as edited`);
-            logger$2(`[compareAndMarkDraft] Context when saving: id=${context.id}, _propertyFileId=${context._propertyFileId}, name=${context.name}, _originalValues.name=${context._originalValues?.name}`);
+            logger$2(`Property ${fullContext.modelName}:${fullContext.name} has changes, marking as edited`);
+            logger$2(`[compareAndMarkDraft] Context when saving: id=${fullContext.id}, _propertyFileId=${fullContext._propertyFileId}, name=${fullContext.name}, _originalValues.name=${fullContext._originalValues?.name}`);
             // Use dynamic import to break circular dependency
-            const { savePropertyToDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.db; });
+            const { savePropertyToDb } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
             // Save to database (but not JSON file) - always save to DB when there are changes
             try {
-                await savePropertyToDb(context);
-                logger$2(`[compareAndMarkDraft] Successfully saved property ${context.modelName}:${context.name} to database`);
+                await savePropertyToDb(fullContext);
+                logger$2(`[compareAndMarkDraft] Successfully saved property ${fullContext.modelName}:${fullContext.name} to database`);
             }
             catch (error) {
                 logger$2(`[compareAndMarkDraft] Error saving property to database: ${error}`);
                 throw error;
             }
             // Mark schema as draft if schema name is available
-            if (context._schemaName) {
+            if (fullContext._schemaName) {
                 // Get the Schema instance and mark it as draft
-                const { Schema } = await Promise.resolve().then(function () { return require('./Schema-B5cr_JVK.js'); });
-                const schema = Schema.create(context._schemaName);
+                const { Schema } = await Promise.resolve().then(function () { return require('./Schema-CV9hMg8m.js'); });
+                const schema = Schema.create(fullContext._schemaName, {
+                    waitForReady: false,
+                });
                 // Send event to Schema machine to mark as draft
                 schema.getService().send({
                     type: 'markAsDraft',
-                    propertyKey: `${context.modelName}:${context.name}`,
+                    propertyKey: `${fullContext.modelName}:${fullContext.name}`,
                 });
             }
         }
         else {
             // No changes - clear edited flag in database and context
-            logger$2(`Property ${context.modelName}:${context.name} has no changes`);
+            logger$2(`Property ${fullContext.modelName}:${fullContext.name} has no changes`);
             // Clear isEdited flag in database
             try {
-                const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
-                const { properties: propertiesTable, models: modelsTable } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema; });
+                const { BaseDb } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.BaseDb$1; });
+                const { properties: propertiesTable, models: modelsTable } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.schema; });
                 const { eq, and } = await import('drizzle-orm');
                 const db = BaseDb.getAppDb();
-                if (db && context.modelName && context.name) {
+                if (db && fullContext.modelName && fullContext.name) {
                     // Find model by name
                     const modelRecords = await db
                         .select({ id: modelsTable.id })
                         .from(modelsTable)
-                        .where(eq(modelsTable.name, context.modelName))
+                        .where(eq(modelsTable.name, fullContext.modelName))
                         .limit(1);
                     if (modelRecords.length > 0) {
                         // Find property by name and modelId
                         const propertyRecords = await db
                             .select({ id: propertiesTable.id })
                             .from(propertiesTable)
-                            .where(and(eq(propertiesTable.name, context.name), eq(propertiesTable.modelId, modelRecords[0].id)))
+                            .where(and(eq(propertiesTable.name, fullContext.name), eq(propertiesTable.modelId, modelRecords[0].id)))
                             .limit(1);
                         if (propertyRecords.length > 0) {
                             // Clear isEdited flag in database
@@ -587,7 +662,7 @@ const compareAndMarkDraft = xstate.fromCallback(({ sendBack, input: { context }
                                 .update(propertiesTable)
                                 .set({ isEdited: false })
                                 .where(eq(propertiesTable.id, propertyRecords[0].id));
-                            logger$2(`Cleared isEdited flag in database for property ${context.modelName}:${context.name}`);
+                            logger$2(`Cleared isEdited flag in database for property ${fullContext.modelName}:${fullContext.name}`);
                         }
                     }
                 }
@@ -614,20 +689,82 @@ const compareAndMarkDraft = xstate.fromCallback(({ sendBack, input: { context }
 const logger$1 = debug('seedSdk:modelProperty:actors:validateProperty');
 const validateProperty = xstate.fromCallback(({ sendBack, input: { context } }) => {
     const _validateProperty = async () => {
-        console.log('[validateProperty] Starting validation for property:', context.name);
+        // Use full context for validation: fill modelName/dataType from _originalValues when missing, then
+        // from DB by schemaFileId (context.id) so just-created renames don't fail structure validation.
+        let fullContext = {
+            ...context,
+            modelName: context.modelName ?? context._originalValues?.modelName,
+            dataType: context.dataType ?? context._originalValues?.dataType,
+        };
+        const schemaFileId = typeof context.id === 'string' ? context.id : undefined;
+        if (schemaFileId && (fullContext.modelName === undefined || fullContext.dataType === undefined)) {
+            // Brief wait so trackPendingWrite from ModelProperty.create() has time to run (it's in setTimeout(0))
+            await new Promise((r) => setTimeout(r, 60));
+            try {
+                const { getPropertyModelNameAndDataType, getModelNameByModelId } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
+                // Try pending write first (property row may not exist yet)
+                if (fullContext.modelName === undefined && schemaFileId) {
+                    const mod = await Promise.resolve().then(function () { return ModelProperty$1; });
+                    const ModelProperty = mod?.ModelProperty ?? mod?.default;
+                    const pendingModelId = ModelProperty?.getPendingModelId?.(schemaFileId);
+                    if (pendingModelId != null) {
+                        const modelName = await getModelNameByModelId(pendingModelId);
+                        if (modelName) {
+                            fullContext = { ...fullContext, modelName };
+                        }
+                    }
+                }
+                // Then DB property lookup with retry (catches row after initial write)
+                let fromDb;
+                for (let attempt = 0; attempt < 6; attempt++) {
+                    fromDb = await getPropertyModelNameAndDataType(schemaFileId);
+                    if (fromDb)
+                        break;
+                    if (attempt < 5)
+                        await new Promise((r) => setTimeout(r, 40));
+                }
+                if (fromDb) {
+                    fullContext = {
+                        ...fullContext,
+                        modelName: fullContext.modelName ?? fromDb.modelName,
+                        dataType: fullContext.dataType ?? fromDb.dataType,
+                    };
+                }
+                // If still no modelName, try pending write again (may have been set during retries)
+                if (fullContext.modelName === undefined && schemaFileId) {
+                    const mod = await Promise.resolve().then(function () { return ModelProperty$1; });
+                    const ModelProperty = mod?.ModelProperty ?? mod?.default;
+                    const pendingModelId = ModelProperty?.getPendingModelId?.(schemaFileId);
+                    if (pendingModelId != null) {
+                        const modelName = await getModelNameByModelId(pendingModelId);
+                        if (modelName) {
+                            fullContext = { ...fullContext, modelName };
+                        }
+                    }
+                }
+            }
+            catch {
+                // ignore
+            }
+        }
+        // Last resort for structure validation: TProperty requires dataType; allow name+id to pass if we have schemaFileId
+        if (fullContext.dataType === undefined && (fullContext.modelName !== undefined || schemaFileId)) {
+            fullContext = { ...fullContext, dataType: (fullContext.dataType ?? context._originalValues?.dataType) || 'Text' };
+        }
+        console.log('[validateProperty] Starting validation for property:', fullContext.name);
         console.log('[validateProperty] Context:', {
-            name: context.name,
-            modelName: context.modelName,
-            _schemaName: context._schemaName,
-            _originalValues: context._originalValues
+            name: fullContext.name,
+            modelName: fullContext.modelName,
+            _schemaName: fullContext._schemaName,
+            _originalValues: fullContext._originalValues
         });
         // Use dynamic imports to break circular dependencies
-        const { SchemaValidationService } = await Promise.resolve().then(function () { return require('./SchemaValidationService-BgIzc3-r.js'); });
+        const { SchemaValidationService } = await Promise.resolve().then(function () { return require('./SchemaValidationService-DUtQUerg.js'); });
         const validationService = new SchemaValidationService();
-        const { Schema } = await Promise.resolve().then(function () { return require('./Schema-B5cr_JVK.js'); });
+        const { Schema } = await Promise.resolve().then(function () { return require('./Schema-CV9hMg8m.js'); });
         // Validate property structure
         console.log('[validateProperty] Validating property structure...');
-        const structureResult = validationService.validatePropertyStructure(context);
+        const structureResult = validationService.validatePropertyStructure(fullContext);
         console.log('[validateProperty] Structure validation result:', structureResult.isValid);
         if (!structureResult.isValid) {
             console.log('[validateProperty] Structure validation failed:', structureResult.errors);
@@ -636,11 +773,13 @@ const validateProperty = xstate.fromCallback(({ sendBack, input: { context } })
         }
         console.log('[validateProperty] Structure validation passed');
         // If we have schema name and model name, validate against schema
-        console.log('[validateProperty] Checking schema validation:', { hasSchemaName: !!context._schemaName, hasModelName: !!context.modelName });
-        if (context._schemaName && context.modelName) {
+        console.log('[validateProperty] Checking schema validation:', { hasSchemaName: !!fullContext._schemaName, hasModelName: !!fullContext.modelName });
+        if (fullContext._schemaName && fullContext.modelName) {
             try {
-                console.log('[validateProperty] Creating schema instance:', context._schemaName);
-                const schema = Schema.create(context._schemaName);
+                console.log('[validateProperty] Creating schema instance:', fullContext._schemaName);
+                const schema = Schema.create(fullContext._schemaName, {
+                    waitForReady: false,
+                });
                 const schemaSnapshot = schema.getService().getSnapshot();
                 const schemaStatus = schemaSnapshot.value;
                 console.log('[validateProperty] Schema status:', schemaStatus);
@@ -653,28 +792,28 @@ const validateProperty = xstate.fromCallback(({ sendBack, input: { context } })
                     if (schemaContext.models && Object.keys(schemaContext.models).length > 0) {
                         // If property name has changed, validate against the original name (from schema file)
                         // This handles the case where a property is renamed - the schema file still has the old name
-                        const propertyNameToValidate = context._originalValues?.name && context._originalValues.name !== context.name
-                            ? context._originalValues.name
-                            : context.name || '';
+                        const propertyNameToValidate = fullContext._originalValues?.name && fullContext._originalValues.name !== fullContext.name
+                            ? fullContext._originalValues.name
+                            : fullContext.name || '';
                         console.log('[validateProperty] Validating against schema:', {
-                            modelName: context.modelName,
+                            modelName: fullContext.modelName,
                             propertyNameToValidate,
-                            originalName: context._originalValues?.name,
-                            currentName: context.name
+                            originalName: fullContext._originalValues?.name,
+                            currentName: fullContext.name
                         });
-                        const schemaResult = validationService.validateProperty(schemaContext, context.modelName, propertyNameToValidate, context);
+                        const schemaResult = validationService.validateProperty(schemaContext, fullContext.modelName, propertyNameToValidate, fullContext);
                         console.log('[validateProperty] Schema validation result:', { isValid: schemaResult.isValid, errors: schemaResult.errors.length, errorCodes: schemaResult.errors.map(e => e.code) });
                         if (!schemaResult.isValid) {
                             // If property was renamed, some validation errors are expected (like property_not_found, missing_type)
                             // Only fail if it's a critical error that's not related to the rename
-                            const isRenamed = context._originalValues?.name && context._originalValues.name !== context.name;
+                            const isRenamed = fullContext._originalValues?.name && fullContext._originalValues.name !== fullContext.name;
                             const criticalErrors = schemaResult.errors.filter(err => {
                                 // Allow property_not_found and missing_type errors when property is renamed
                                 if (isRenamed && (err.code === 'property_not_found' || err.code === 'missing_type')) {
                                     return false; // Not critical
                                 }
                                 // For non-renamed properties, only allow property_not_found if we're validating with the same name
-                                if (err.code === 'property_not_found' && propertyNameToValidate === context.name) {
+                                if (err.code === 'property_not_found' && propertyNameToValidate === fullContext.name) {
                                     return false; // Not critical
                                 }
                                 return true; // Critical error
@@ -808,6 +947,15 @@ const modelPropertyMachine = xstate.setup({
                     }
                     newContext[key] = value;
                 }
+                // Preserve modelName and dataType from context or _originalValues so validation and savePropertyToDb
+                // never run with missing required/needed fields (e.g. when a just-created property is renamed before
+                // the full context is available from the creator).
+                if (newContext.modelName === undefined && context._originalValues?.modelName !== undefined) {
+                    newContext.modelName = context._originalValues.modelName;
+                }
+                if (newContext.dataType === undefined && context._originalValues?.dataType !== undefined) {
+                    newContext.dataType = context._originalValues.dataType;
+                }
                 // Compare with original values and set _isEdited flag (only for non-internal updates)
                 if (!onlyInternalFields && context._originalValues) {
                     const hasChanges = Object.keys(event).some((key) => {
@@ -877,6 +1025,23 @@ const modelPropertyMachine = xstate.setup({
                 _schemaName: event.schemaName,
             })),
         },
+        destroyStarted: {
+            actions: xstate.assign({ _destroyInProgress: true, _destroyError: null }),
+        },
+        destroyDone: {
+            actions: xstate.assign({ _destroyInProgress: false }),
+        },
+        destroyError: {
+            actions: xstate.assign(({ event }) => ({
+                _destroyInProgress: false,
+                _destroyError: event.error instanceof Error
+                    ? { message: event.error.message, name: event.error.name }
+                    : { message: String(event.error) },
+            })),
+        },
+        clearDestroyError: {
+            actions: xstate.assign({ _destroyError: null }),
+        },
     },
     states: {
         idle: {
@@ -1002,6 +1167,32 @@ const modelPropertyMachine = xstate.setup({
     },
 });
 
+/**
+ * Returns property rows for a model identified by modelFileId (model's schemaFileId).
+ * Resolves modelFileId -> modelId via models table, then selects all properties where modelId = ?.
+ * Each row includes schemaFileId for use with ModelProperty.createById(schemaFileId).
+ */
+const getModelPropertiesData = async (modelFileId) => {
+    const appDb = main_cjs.BaseDb.getAppDb();
+    if (!appDb || !modelFileId) {
+        return [];
+    }
+    const modelRows = await appDb
+        .select({ id: main_cjs.models.id })
+        .from(main_cjs.models)
+        .where(drizzleOrm.eq(main_cjs.models.schemaFileId, modelFileId))
+        .limit(1);
+    if (modelRows.length === 0) {
+        return [];
+    }
+    const modelId = modelRows[0].id;
+    const rows = await appDb
+        .select()
+        .from(main_cjs.properties)
+        .where(drizzleOrm.eq(main_cjs.properties.modelId, modelId));
+    return rows;
+};
+
 const logger = debug('seedSdk:modelProperty:ModelProperty');
 // Define the property keys from TProperty
 const TPropertyKeys = [
@@ -1168,27 +1359,25 @@ class ModelProperty {
             return fallbackIsEdited;
         }
         try {
-            const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
-            const { properties: propertiesTable, models: modelsTable } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema; });
             const { eq, and } = await import('drizzle-orm');
-            const db = BaseDb.getAppDb();
+            const db = main_cjs.BaseDb.getAppDb();
             if (!db) {
                 return fallbackIsEdited;
             }
             // Find model by name
             const modelRecords = await db
-                .select({ id: modelsTable.id })
-                .from(modelsTable)
-                .where(eq(modelsTable.name, property.modelName))
+                .select({ id: main_cjs.models.id })
+                .from(main_cjs.models)
+                .where(eq(main_cjs.models.name, property.modelName))
                 .limit(1);
             if (modelRecords.length === 0) {
                 return fallbackIsEdited;
             }
             // Find property by name and modelId
             const propertyRecords = await db
-                .select({ isEdited: propertiesTable.isEdited })
-                .from(propertiesTable)
-                .where(and(eq(propertiesTable.name, property.name), eq(propertiesTable.modelId, modelRecords[0].id)))
+                .select({ isEdited: main_cjs.properties.isEdited })
+                .from(main_cjs.properties)
+                .where(and(eq(main_cjs.properties.name, property.name), eq(main_cjs.properties.modelId, modelRecords[0].id)))
                 .limit(1);
             if (propertyRecords.length > 0) {
                 return propertyRecords[0].isEdited ?? false;
@@ -1209,18 +1398,15 @@ class ModelProperty {
             return undefined;
         }
         try {
-            const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
-            const seedSchema = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema; });
-            const modelsTable = seedSchema.models;
             const { eq } = await import('drizzle-orm');
-            const db = BaseDb.getAppDb();
+            const db = main_cjs.BaseDb.getAppDb();
             if (!db) {
                 return undefined;
             }
             const refModelRecords = await db
                 .select()
-                .from(modelsTable)
-                .where(eq(modelsTable.name, refModelName))
+                .from(main_cjs.models)
+                .where(eq(main_cjs.models.name, refModelName))
                 .limit(1);
             if (refModelRecords.length > 0 && refModelRecords[0].id) {
                 return refModelRecords[0].id;
@@ -1241,8 +1427,8 @@ class ModelProperty {
             return undefined;
         }
         try {
-            const { Model } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.Model$1; });
-            const { modelPropertiesToObject } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.model; });
+            const { Model } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.Model$1; });
+            const { modelPropertiesToObject } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.model; });
             const model = await Model.getByNameAsync(property.modelName);
             if (!model || !model.properties || model.properties.length === 0) {
                 return undefined;
@@ -1291,22 +1477,17 @@ class ModelProperty {
             // Try to get schema name from database first (more reliable)
             if (this.modelId) {
                 try {
-                    const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
-                    const seedSchema = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema; });
-                    const modelsTable = seedSchema.models;
-                    const modelSchemas = seedSchema.modelSchemas;
-                    const schemas = seedSchema.schemas;
                     const { eq } = await import('drizzle-orm');
-                    const db = BaseDb.getAppDb();
+                    const db = main_cjs.BaseDb.getAppDb();
                     if (db) {
                         const modelSchemaRecords = await db
                             .select({
-                            schemaName: schemas.name,
+                            schemaName: main_cjs.schemas.name,
                         })
-                            .from(modelSchemas)
-                            .innerJoin(schemas, eq(modelSchemas.schemaId, schemas.id))
-                            .innerJoin(modelsTable, eq(modelSchemas.modelId, modelsTable.id))
-                            .where(eq(modelsTable.id, this.modelId))
+                            .from(main_cjs.modelSchemas)
+                            .innerJoin(main_cjs.schemas, eq(main_cjs.modelSchemas.schemaId, main_cjs.schemas.id))
+                            .innerJoin(main_cjs.models, eq(main_cjs.modelSchemas.modelId, main_cjs.models.id))
+                            .where(eq(main_cjs.models.id, this.modelId))
                             .limit(1);
                         if (modelSchemaRecords.length > 0) {
                             schemaName = modelSchemaRecords[0].schemaName;
@@ -1344,10 +1525,12 @@ class ModelProperty {
             schemaName,
         });
     }
-    static create(property) {
+    static create(property, options) {
         if (!property) {
             throw new Error('Property is required');
         }
+        const waitForReady = options?.waitForReady !== false;
+        const readyTimeout = options?.readyTimeout ?? 5000;
         // Debug: Log what's being passed to create()
         console.log(`[ModelProperty.create] Input property data:`, JSON.stringify({
             name: property.name,
@@ -1421,7 +1604,9 @@ class ModelProperty {
                 instance,
                 refCount: refCount + 1,
             });
-            return instance;
+            if (!waitForReady)
+                return instance;
+            return main_cjs.waitForEntityIdle(instance, { timeout: readyTimeout }).then(() => instance);
         }
         // Debug: Log what's being passed to the constructor
         console.log(`[ModelProperty.create] propertyWithId before constructor:`, JSON.stringify({
@@ -1496,7 +1681,7 @@ class ModelProperty {
                         else if (typeof propertyWithId.modelId === 'string') {
                             // modelId is a string (modelFileId), need to convert to database ID
                             try {
-                                const { getModelIdByFileId } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.db; });
+                                const { getModelIdByFileId } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
                                 resolvedModelId = await getModelIdByFileId(propertyWithId.modelId);
                                 logger(`Converted modelFileId "${propertyWithId.modelId}" to database modelId: ${resolvedModelId}`);
                             }
@@ -1509,7 +1694,7 @@ class ModelProperty {
                     // If we still don't have a modelId, try to resolve it from modelName
                     if (!resolvedModelId && propertyWithId.modelName) {
                         try {
-                            const { getModelId } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.db; });
+                            const { getModelId } = await Promise.resolve().then(function () { return require('./index-C3j3m34a.js'); }).then(function (n) { return n.db; });
                             // Get schemaName from context if available
                             const schemaName = snapshot.context._schemaName;
                             resolvedModelId = await getModelId(propertyWithId.modelName, schemaName);
@@ -1583,7 +1768,9 @@ class ModelProperty {
             // Start checking after a short delay to allow state machine to initialize
             setTimeout(checkAndSend, 0);
         }
-        return proxiedInstance;
+        if (!waitForReady)
+            return proxiedInstance;
+        return main_cjs.waitForEntityIdle(proxiedInstance, { timeout: readyTimeout }).then(() => proxiedInstance);
     }
     /**
      * Get ModelProperty instance by propertyFileId from static cache
@@ -1616,10 +1803,8 @@ class ModelProperty {
             return cachedInstance;
         }
         // Query database to get property data from ID
-        const { BaseDb } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.BaseDb$1; });
-        const { properties: propertiesTable, models: modelsTable } = await Promise.resolve().then(function () { return require('./index-BmIVfqGN.js'); }).then(function (n) { return n.schema; });
         const { eq } = await import('drizzle-orm');
-        const db = BaseDb.getAppDb();
+        const db = main_cjs.BaseDb.getAppDb();
         console.log('db', !!db);
         if (!db) {
             console.log('db not found');
@@ -1627,13 +1812,13 @@ class ModelProperty {
         }
         const testRecords = await db
             .select()
-            .from(propertiesTable)
+            .from(main_cjs.properties)
             .limit(100);
         console.log('testRecords', testRecords);
         const propertyRecords = await db
             .select()
-            .from(propertiesTable)
-            .where(eq(propertiesTable.schemaFileId, propertyFileId))
+            .from(main_cjs.properties)
+            .where(eq(main_cjs.properties.schemaFileId, propertyFileId))
             .limit(1);
         console.log('propertyRecords', propertyRecords);
         if (propertyRecords.length === 0) {
@@ -1642,9 +1827,9 @@ class ModelProperty {
         const propertyRecord = propertyRecords[0];
         // Get model name
         const modelRecords = await db
-            .select({ name: modelsTable.name })
-            .from(modelsTable)
-            .where(eq(modelsTable.id, propertyRecord.modelId))
+            .select({ name: main_cjs.models.name })
+            .from(main_cjs.models)
+            .where(eq(main_cjs.models.id, propertyRecord.modelId))
             .limit(1);
         if (modelRecords.length === 0) {
             return undefined;
@@ -1667,17 +1852,17 @@ class ModelProperty {
         // Get ref model name if applicable
         if (propertyRecord.refModelId) {
             const refModelRecords = await db
-                .select({ name: modelsTable.name })
-                .from(modelsTable)
-                .where(eq(modelsTable.id, propertyRecord.refModelId))
+                .select({ name: main_cjs.models.name })
+                .from(main_cjs.models)
+                .where(eq(main_cjs.models.id, propertyRecord.refModelId))
                 .limit(1);
             if (refModelRecords.length > 0) {
                 propertyData.refModelName = refModelRecords[0].name;
                 propertyData.ref = refModelRecords[0].name;
             }
         }
-        // Create ModelProperty instance
-        const instance = this.create(propertyData);
+        // Create ModelProperty instance (sync for createById so we can send updateContext)
+        const instance = this.create(propertyData, { waitForReady: false });
         // Set isEdited from database after creation
         if (isEditedFromDb) {
             instance._service.send({
@@ -1705,6 +1890,32 @@ class ModelProperty {
             readyTimeout,
         });
     }
+    /**
+     * Get all ModelProperty instances for a model.
+     * Loads property rows from DB for the given modelFileId, creates instances via createById, optionally waits for idle.
+     */
+    static async all(modelFileId, options) {
+        const { waitForReady = false, readyTimeout = 5000 } = options ?? {};
+        if (!modelFileId) {
+            return [];
+        }
+        const rows = await getModelPropertiesData(modelFileId);
+        const instances = [];
+        for (const row of rows) {
+            if (row.schemaFileId) {
+                const instance = await this.createById(row.schemaFileId);
+                if (instance) {
+                    instances.push(instance);
+                }
+            }
+        }
+        if (waitForReady && instances.length > 0) {
+            await Promise.all(instances.map((p) => main_cjs.waitForEntityIdle(p, {
+                timeout: readyTimeout,
+            })));
+        }
+        return instances;
+    }
     /**
      * Track a pending write for a property
      */
@@ -1743,6 +1954,13 @@ class ModelProperty {
             .filter(([_, write]) => write.modelId === modelId && write.status !== 'error')
             .map(([propertyFileId]) => propertyFileId);
     }
+    /**
+     * Get modelId for a property that has a pending write (row may not be in DB yet).
+     * Used to resolve modelName when validating a just-created property rename.
+     */
+    static getPendingModelId(propertyFileId) {
+        return this.pendingWrites.get(propertyFileId)?.modelId;
+    }
     getService() {
         return this._service;
     }
@@ -1849,6 +2067,61 @@ class ModelProperty {
             // Service might already be stopped
         }
     }
+    /**
+     * Destroy the model property: remove from caches, delete from database, update Schema context, stop service.
+     */
+    async destroy() {
+        const context = this._getSnapshotContext();
+        const cacheKey = context.modelName && context.name
+            ? `${context.modelName}:${context.name}`
+            : (context.id ?? '');
+        if (!cacheKey)
+            return;
+        main_cjs.forceRemoveFromCaches(this, {
+            getCacheKeys: () => [cacheKey],
+            caches: [ModelProperty.instanceCache],
+        });
+        await main_cjs.runDestroyLifecycle(this, {
+            getService: (instance) => instance._service,
+            doDestroy: async () => {
+                const db = main_cjs.BaseDb.getAppDb();
+                const schemaName = context._schemaName;
+                const modelName = context.modelName;
+                const propertyName = context.name;
+                if (!modelName || !propertyName)
+                    return;
+                if (db && schemaName) {
+                    const propertyRecords = await db
+                        .select({ propertyId: main_cjs.properties.id })
+                        .from(main_cjs.properties)
+                        .innerJoin(main_cjs.models, drizzleOrm.eq(main_cjs.properties.modelId, main_cjs.models.id))
+                        .innerJoin(main_cjs.modelSchemas, drizzleOrm.eq(main_cjs.models.id, main_cjs.modelSchemas.modelId))
+                        .innerJoin(main_cjs.schemas, drizzleOrm.eq(main_cjs.modelSchemas.schemaId, main_cjs.schemas.id))
+                        .where(drizzleOrm.and(drizzleOrm.eq(main_cjs.schemas.name, schemaName), drizzleOrm.eq(main_cjs.models.name, modelName), drizzleOrm.eq(main_cjs.properties.name, propertyName)))
+                        .limit(1);
+                    if (propertyRecords.length > 0 && propertyRecords[0].propertyId != null) {
+                        await db
+                            .delete(main_cjs.properties)
+                            .where(drizzleOrm.eq(main_cjs.properties.id, propertyRecords[0].propertyId));
+                    }
+                }
+                if (schemaName) {
+                    const { Schema } = await Promise.resolve().then(function () { return require('./Schema-CV9hMg8m.js'); });
+                    const schema = Schema.create(schemaName, { waitForReady: false });
+                    const snapshot = schema.getService().getSnapshot();
+                    const schemaContext = snapshot.context;
+                    if (schemaContext.models?.[modelName]?.properties?.[propertyName]) {
+                        const updatedModels = { ...schemaContext.models };
+                        const updatedProperties = { ...updatedModels[modelName].properties };
+                        delete updatedProperties[propertyName];
+                        updatedModels[modelName] = { ...updatedModels[modelName], properties: updatedProperties };
+                        schema.getService().send({ type: 'updateContext', models: updatedModels });
+                        schema.getService().send({ type: 'markAsDraft', propertyKey: `property:${modelName}:${propertyName}` });
+                    }
+                }
+            },
+        });
+    }
 }
 ModelProperty.instanceCache = new Map();
 // Pending writes tracking
@@ -1863,4 +2136,5 @@ exports.ModelProperty = ModelProperty;
 exports.ModelProperty$1 = ModelProperty$1;
 exports.convertPropertyToSchemaUpdate = convertPropertyToSchemaUpdate;
 exports.updateModelProperties = updateModelProperties;
-//# sourceMappingURL=ModelProperty-MkN5Rmx7.js.map
+exports.writeFullSchemaNewVersion = writeFullSchemaNewVersion;
+//# sourceMappingURL=ModelProperty-BRKRREYP.js.map