@compilr-dev/factory 0.1.1 → 0.1.3

package/dist/model/tools.js

@@ -112,7 +112,18 @@ function createAppModelGetTool(config) {
 function createAppModelUpdateTool(config) {
     return defineTool({
         name: 'app_model_update',
-        description: `Apply a semantic operation to the Application Model. Operations: addEntity, updateEntity, removeEntity, renameEntity, reorderEntities, addField, updateField, removeField, renameField, addRelationship, removeRelationship, updateIdentity, updateLayout, updateFeatures, updateTheme, updateTechStack. If no model exists, one is created automatically with the first addEntity.`,
+        description: `Apply a semantic operation to the Application Model. If no model exists, one is created automatically.
+
+EXAMPLES:
+- updateIdentity: { "op": "updateIdentity", "updates": { "name": "My App", "description": "A task manager" } }
+- updateTechStack: { "op": "updateTechStack", "updates": { "toolkit": "react-node" } }
+- addEntity: { "op": "addEntity", "entity": { "name": "Task", "pluralName": "Tasks", "icon": "📋", "fields": [{ "name": "title", "label": "Title", "type": "string", "required": true }], "views": ["list", "detail", "card"], "relationships": [] } }
+- addField: { "op": "addField", "entity": "Task", "field": { "name": "dueDate", "label": "Due Date", "type": "date", "required": false } }
+- addRelationship: { "op": "addRelationship", "entity": "Task", "relationship": { "type": "belongsTo", "target": "Project" } }
+- updateFeatures: { "op": "updateFeatures", "updates": { "dashboard": true, "darkMode": true } }
+- updateTheme: { "op": "updateTheme", "updates": { "primaryColor": "#1976D2" } }
+- updateLayout: { "op": "updateLayout", "updates": { "shell": "sidebar-header" } }
+- removeEntity: { "op": "removeEntity", "entity": "Task", "force": true }`,
         inputSchema: {
             type: 'object',
             properties: {
@@ -143,23 +154,31 @@ function createAppModelUpdateTool(config) {
                     description: 'Expected current revision for optimistic locking. Optional.',
                 },
                 entity: {
-                    description: 'Entity name (string) or full entity object (for addEntity).',
+                    oneOf: [{ type: 'string' }, { type: 'object', additionalProperties: true }],
+                    description: 'Entity name (string) for most ops, or full entity object (for addEntity). Entity object requires: name (PascalCase), pluralName, icon, fields array, views array, relationships array.',
                 },
                 field: {
-                    description: 'Field name (string) or full field object (for addField).',
+                    oneOf: [{ type: 'string' }, { type: 'object', additionalProperties: true }],
+                    description: 'Field name (string) for updateField/removeField/renameField, or full field object (for addField). Field object requires: name (camelCase), label, type (string|number|boolean|date|enum), required.',
                 },
                 relationship: {
-                    type: 'object',
-                    description: 'Relationship object for addRelationship.',
-                    properties: {
-                        type: { type: 'string', enum: ['belongsTo', 'hasMany'] },
-                        target: { type: 'string' },
-                        fieldName: { type: 'string' },
-                    },
+                    oneOf: [
+                        {
+                            type: 'object',
+                            additionalProperties: true,
+                            properties: {
+                                type: { type: 'string', enum: ['belongsTo', 'hasMany'] },
+                                target: { type: 'string' },
+                                fieldName: { type: 'string' },
+                            },
+                        },
+                        { type: 'string' },
+                    ],
+                    description: 'Relationship object for addRelationship. Requires: type ("belongsTo" or "hasMany"), target (entity name). Optional: fieldName.',
                 },
                 updates: {
-                    type: 'object',
-                    description: 'Partial updates for update operations.',
+                    oneOf: [{ type: 'object', additionalProperties: true }, { type: 'string' }],
+                    description: 'Partial updates object for updateIdentity, updateLayout, updateFeatures, updateTheme, updateTechStack, updateEntity.',
                 },
                 newName: {
                     type: 'string',
@@ -222,28 +241,53 @@ function createAppModelUpdateTool(config) {
         },
     });
 }
+/**
+ * Coerce a value to an object. LLMs sometimes send stringified JSON for nested objects.
+ * Returns the parsed object, or null if coercion fails.
+ */
+function coerceToObject(value) {
+    if (typeof value === 'object' && value !== null && !Array.isArray(value)) {
+        return value;
+    }
+    if (typeof value === 'string') {
+        try {
+            const parsed = JSON.parse(value);
+            if (typeof parsed === 'object' && parsed !== null && !Array.isArray(parsed)) {
+                return parsed;
+            }
+        }
+        catch {
+            // Not valid JSON
+        }
+    }
+    return null;
+}
 function buildOperation(input) {
     switch (input.op) {
         case 'addEntity': {
-            if (!input.entity || typeof input.entity !== 'object') {
-                throw new Error('addEntity requires entity object');
+            const entity = coerceToObject(input.entity);
+            if (!entity) {
+                throw new Error(`addEntity requires "entity" as an object with name, pluralName, icon, fields, views, relationships. ` +
+                    `Got: ${input.entity === undefined ? 'missing' : typeof input.entity}. ` +
+                    `Example: { "op": "addEntity", "entity": { "name": "Employee", "pluralName": "Employees", "icon": "👤", "fields": [{ "name": "name", "label": "Name", "type": "string", "required": true }], "views": ["list", "detail", "card"], "relationships": [] } }`);
             }
-            const entity = input.entity;
-            return { op: 'addEntity', entity };
+            return { op: 'addEntity', entity: entity };
         }
-        case 'updateEntity':
+        case 'updateEntity': {
             if (typeof input.entity !== 'string')
-                throw new Error('updateEntity requires entity name (string)');
-            if (!input.updates)
-                throw new Error('updateEntity requires updates object');
-            return { op: 'updateEntity', entity: input.entity, updates: input.updates };
+                throw new Error(`updateEntity requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateEntity requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}`);
+            return { op: 'updateEntity', entity: input.entity, updates };
+        }
         case 'removeEntity':
             if (typeof input.entity !== 'string')
-                throw new Error('removeEntity requires entity name (string)');
+                throw new Error(`removeEntity requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
             return { op: 'removeEntity', entity: input.entity, force: input.force };
         case 'renameEntity':
             if (typeof input.entity !== 'string')
-                throw new Error('renameEntity requires entity name (string)');
+                throw new Error(`renameEntity requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
             if (!input.newName)
                 throw new Error('renameEntity requires newName');
             return { op: 'renameEntity', entity: input.entity, newName: input.newName };
@@ -253,36 +297,39 @@ function buildOperation(input) {
             return { op: 'reorderEntities', order: input.order };
         case 'addField': {
             if (typeof input.entity !== 'string')
-                throw new Error('addField requires entity name (string)');
-            if (!input.field || typeof input.field !== 'object')
-                throw new Error('addField requires field object');
-            const field = input.field;
-            return { op: 'addField', entity: input.entity, field };
-        }
-        case 'updateField':
-            if (typeof input.entity !== 'string')
-                throw new Error('updateField requires entity name (string)');
-            if (typeof input.field !== 'string')
-                throw new Error('updateField requires field name (string)');
-            if (!input.updates)
-                throw new Error('updateField requires updates object');
+                throw new Error(`addField requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
+            const field = coerceToObject(input.field);
+            if (!field)
+                throw new Error(`addField requires "field" as an object with name, label, type, required. ` +
+                    `Got: ${input.field === undefined ? 'missing' : typeof input.field}. ` +
+                    `Example: { "op": "addField", "entity": "Employee", "field": { "name": "email", "label": "Email", "type": "string", "required": true } }`);
             return {
-                op: 'updateField',
+                op: 'addField',
                 entity: input.entity,
-                field: input.field,
-                updates: input.updates,
+                field: field,
             };
+        }
+        case 'updateField': {
+            if (typeof input.entity !== 'string')
+                throw new Error(`updateField requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
+            if (typeof input.field !== 'string')
+                throw new Error(`updateField requires "field" as a string (field name). Got: ${input.field === undefined ? 'missing' : typeof input.field}`);
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateField requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}`);
+            return { op: 'updateField', entity: input.entity, field: input.field, updates };
+        }
         case 'removeField':
             if (typeof input.entity !== 'string')
-                throw new Error('removeField requires entity name (string)');
+                throw new Error(`removeField requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
             if (typeof input.field !== 'string')
-                throw new Error('removeField requires field name (string)');
+                throw new Error(`removeField requires "field" as a string (field name). Got: ${input.field === undefined ? 'missing' : typeof input.field}`);
             return { op: 'removeField', entity: input.entity, field: input.field };
         case 'renameField':
             if (typeof input.entity !== 'string')
-                throw new Error('renameField requires entity name (string)');
+                throw new Error(`renameField requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
             if (typeof input.field !== 'string')
-                throw new Error('renameField requires field name (string)');
+                throw new Error(`renameField requires "field" as a string (field name). Got: ${input.field === undefined ? 'missing' : typeof input.field}`);
             if (!input.newName)
                 throw new Error('renameField requires newName');
             return {
@@ -293,38 +340,59 @@ function buildOperation(input) {
             };
         case 'addRelationship': {
             if (typeof input.entity !== 'string')
-                throw new Error('addRelationship requires entity name (string)');
-            if (!input.relationship)
-                throw new Error('addRelationship requires relationship object');
-            const relationship = input.relationship;
-            return { op: 'addRelationship', entity: input.entity, relationship };
+                throw new Error(`addRelationship requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
+            const relationship = coerceToObject(input.relationship);
+            if (!relationship)
+                throw new Error(`addRelationship requires "relationship" object with type and target. ` +
+                    `Got: ${input.relationship === undefined ? 'missing' : typeof input.relationship}. ` +
+                    `Example: { "op": "addRelationship", "entity": "Task", "relationship": { "type": "belongsTo", "target": "Project" } }`);
+            return {
+                op: 'addRelationship',
+                entity: input.entity,
+                relationship: relationship,
+            };
         }
         case 'removeRelationship':
             if (typeof input.entity !== 'string')
-                throw new Error('removeRelationship requires entity name (string)');
+                throw new Error(`removeRelationship requires "entity" as a string (entity name). Got: ${input.entity === undefined ? 'missing' : typeof input.entity}`);
             if (!input.target)
                 throw new Error('removeRelationship requires target');
             return { op: 'removeRelationship', entity: input.entity, target: input.target };
-        case 'updateIdentity':
-            if (!input.updates)
-                throw new Error('updateIdentity requires updates object');
-            return { op: 'updateIdentity', updates: input.updates };
-        case 'updateLayout':
-            if (!input.updates)
-                throw new Error('updateLayout requires updates object');
-            return { op: 'updateLayout', updates: input.updates };
-        case 'updateFeatures':
-            if (!input.updates)
-                throw new Error('updateFeatures requires updates object');
-            return { op: 'updateFeatures', updates: input.updates };
-        case 'updateTheme':
-            if (!input.updates)
-                throw new Error('updateTheme requires updates object');
-            return { op: 'updateTheme', updates: input.updates };
-        case 'updateTechStack':
-            if (!input.updates)
-                throw new Error('updateTechStack requires updates object');
-            return { op: 'updateTechStack', updates: input.updates };
+        case 'updateIdentity': {
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateIdentity requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}. ` +
+                    `Example: { "op": "updateIdentity", "updates": { "name": "My App", "description": "An app" } }`);
+            return { op: 'updateIdentity', updates };
+        }
+        case 'updateLayout': {
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateLayout requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}. ` +
+                    `Example: { "op": "updateLayout", "updates": { "shell": "sidebar-header" } }`);
+            return { op: 'updateLayout', updates };
+        }
+        case 'updateFeatures': {
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateFeatures requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}. ` +
+                    `Example: { "op": "updateFeatures", "updates": { "dashboard": true } }`);
+            return { op: 'updateFeatures', updates };
+        }
+        case 'updateTheme': {
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateTheme requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}. ` +
+                    `Example: { "op": "updateTheme", "updates": { "primaryColor": "#1976D2" } }`);
+            return { op: 'updateTheme', updates };
+        }
+        case 'updateTechStack': {
+            const updates = coerceToObject(input.updates);
+            if (!updates)
+                throw new Error(`updateTechStack requires "updates" object. Got: ${input.updates === undefined ? 'missing' : typeof input.updates}. ` +
+                    `Example: { "op": "updateTechStack", "updates": { "toolkit": "react-node" } }`);
+            return { op: 'updateTechStack', updates };
+        }
         default:
             throw new Error(`Unknown operation: ${input.op}`);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compilr-dev/factory",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "AI-driven application scaffolder for the compilr-dev ecosystem",
   "type": "module",
   "main": "dist/index.js",