@stonyx/orm 0.2.1-alpha.12 → 0.2.1-alpha.13

package/.claude/index.md

@@ -3,7 +3,6 @@
 ## Detailed Guides
 
 - [Usage Patterns](usage-patterns.md) — Model definitions, serializers, transforms, CRUD, DB schema, persistence, access control, REST API, and include parameters
-- [Views](views.md) — Read-only computed views with aggregate helpers, in-memory resolver, and MySQL VIEW auto-generation
 - [Middleware Hooks System](hooks.md) — Before/after hooks for CRUD operations, halting, context object, change detection, and testing
 - [Code Style Rules](code-style-rules.md) — Strict prettier/eslint rules to apply across all Stonyx projects
 
@@ -22,7 +21,6 @@
 5. **REST API Generation**: Auto-generated RESTful endpoints with access control
 6. **Data Transformation**: Custom type conversion and formatting
 7. **Middleware Hooks**: Before/after hooks for all CRUD operations with halting capability
-8. **Views**: Read-only computed projections over model data with aggregate helpers (count, avg, sum, min, max)
 
 ---
 
@@ -40,9 +38,6 @@
 8. **Include Logic** (inline in [src/orm-request.js](src/orm-request.js)) - Parses include query params, traverses relationships, collects and deduplicates included records
 9. **Hooks** ([src/hooks.js](src/hooks.js)) - Middleware-based hook registry for CRUD lifecycle
 10. **MySQL Driver** ([src/mysql/mysql-db.js](src/mysql/mysql-db.js)) - MySQL persistence, migrations, schema introspection. Loads records in topological order. `_rowToRawData()` converts TINYINT(1) → boolean, remaps FK columns, strips timestamps.
-11. **View** ([src/view.js](src/view.js)) - Read-only base class for computed views (does NOT extend Model)
-12. **Aggregates** ([src/aggregates.js](src/aggregates.js)) - AggregateProperty class + helper functions (count, avg, sum, min, max)
-13. **ViewResolver** ([src/view-resolver.js](src/view-resolver.js)) - In-memory view resolver (iterates source model, computes aggregates + resolve map)
 
 ### Project Structure
 
@@ -71,9 +66,6 @@ stonyx-orm/
 │   ├── commands.js               # CLI commands (db:migrate-*, etc.)
 │   ├── utils.js                  # Pluralize wrapper for dasherized names
 │   ├── plural-registry.js        # Plural name registry (populated at init, supports Model.pluralName overrides)
-│   ├── view.js                   # View base class (read-only, source, resolve, memory)
-│   ├── aggregates.js             # AggregateProperty + count/avg/sum/min/max helpers
-│   ├── view-resolver.js          # In-memory view resolver
 │   ├── exports/
 │   │   └── db.js                 # Convenience re-export of DB instance
 │   └── mysql/
@@ -94,7 +86,6 @@ stonyx-orm/
 │       ├── serializers/          # Example serializers
 │       ├── transforms/           # Custom transforms
 │       ├── access/               # Access control
-│       ├── views/                # Example views
 │       ├── db-schema.js          # DB schema
 │       └── payload.js            # Test data
 └── package.json
@@ -112,8 +103,7 @@ config.orm = {
     model: './models',
     serializer: './serializers',
     transform: './transforms',
-    access: './access',
-    view: './views'
+    access: './access'
   },
   db: {
     autosave: 'false',
@@ -239,10 +229,9 @@ The ORM supports two storage modes, configured via `db.mode`:
 **Import the ORM:**
 ```javascript
 import {
-  Orm, Model, View, Serializer, attr, hasMany, belongsTo,
+  Orm, Model, Serializer, attr, hasMany, belongsTo,
   createRecord, updateRecord, store,
-  beforeHook, afterHook, clearHook, clearAllHooks,
-  count, avg, sum, min, max
+  beforeHook, afterHook, clearHook, clearAllHooks
 } from '@stonyx/orm';
 ```
 

package/.claude/usage-patterns.md

@@ -232,69 +232,3 @@ GET /scenes/e001-s001?include=slides.dialogue.character
 - `traverseIncludePath()` - Recursively traverses relationship paths
 - `collectIncludedRecords()` - Orchestrates traversal and deduplication
 - All implemented in [src/orm-request.js](src/orm-request.js)
-
-## 10. Views (Read-Only Computed Data)
-
-Views are read-only projections that compute derived data from existing models. They work in both JSON mode (in-memory) and MySQL mode (auto-generated SQL VIEWs). See the full guide at [views.md](views.md).
-
-### Defining a View
-
-```javascript
-// views/owner-stats.js
-import { View, attr, belongsTo, count, avg } from '@stonyx/orm';
-
-export default class OwnerStatsView extends View {
-  static source = 'owner';  // Required: model whose records produce view records
-
-  animalCount = count('pets');     // COUNT of hasMany relationship
-  averageAge = avg('pets', 'age'); // AVG of a field on related records
-  owner = belongsTo('owner');      // Link back to source record
-}
-```
-
-### Aggregate Helpers
-
-| Helper | Example | JS Behavior | MySQL |
-|--------|---------|-------------|-------|
-| `count(rel)` | `count('pets')` | `records.length` | `COUNT(table.id)` |
-| `avg(rel, field)` | `avg('pets', 'age')` | Average of values | `AVG(table.field)` |
-| `sum(rel, field)` | `sum('pets', 'age')` | Sum of values | `SUM(table.field)` |
-| `min(rel, field)` | `min('pets', 'age')` | Minimum value | `MIN(table.field)` |
-| `max(rel, field)` | `max('pets', 'age')` | Maximum value | `MAX(table.field)` |
-
-### Resolve Map (Escape Hatch)
-
-For fields that can't be expressed as aggregates:
-
-```javascript
-export default class OwnerStatsView extends View {
-  static source = 'owner';
-  static resolve = {
-    gender: 'gender',              // String path from source data
-    score: (owner) => owner.__data.age * 10,  // Function
-  };
-
-  gender = attr('string');  // Must also define as attr()
-  score = attr('number');
-  animalCount = count('pets');
-}
-```
-
-### Querying Views
-
-```javascript
-const stats = await store.findAll('owner-stats');
-const stat = await store.find('owner-stats', ownerId);
-```
-
-### Read-Only Enforcement
-
-```javascript
-createRecord('owner-stats', data);   // Throws: Cannot create records for read-only view
-updateRecord(viewRecord, data);       // Throws: Cannot update records for read-only view
-store.remove('owner-stats', id);      // Throws: Cannot remove records from read-only view
-```
-
-### REST API
-
-Only GET endpoints are mounted for views — no POST, PATCH, or DELETE.

package/config/environment.js

@@ -4,7 +4,6 @@ const {
   ORM_REST_ROUTE,
   ORM_SERIALIZER_PATH,
   ORM_TRANSFORM_PATH,
-  ORM_VIEW_PATH,
   ORM_USE_REST_SERVER,
   DB_AUTO_SAVE,
   DB_FILE,
@@ -37,8 +36,7 @@ export default {
     access: ORM_ACCESS_PATH ?? './access', // Optional for restServer access hooks
     model: ORM_MODEL_PATH ?? './models',
     serializer: ORM_SERIALIZER_PATH ?? './serializers',
-    transform: ORM_TRANSFORM_PATH ?? './transforms',
-    view: ORM_VIEW_PATH ?? './views'
+    transform: ORM_TRANSFORM_PATH ?? './transforms'
   },
   mysql: MYSQL_HOST ? {
     host: MYSQL_HOST ?? 'localhost',

package/package.json

@@ -4,7 +4,7 @@
     "stonyx-async",
     "stonyx-module"
   ],
-  "version": "0.2.1-alpha.12",
+  "version": "0.2.1-alpha.13",
   "description": "",
   "main": "src/main.js",
   "type": "module",
@@ -33,9 +33,9 @@
   },
   "homepage": "https://github.com/abofs/stonyx-orm#readme",
   "dependencies": {
-    "stonyx": "0.2.3-beta.5",
-    "@stonyx/events": "0.1.1-beta.6",
-    "@stonyx/cron": "0.2.1-beta.10"
+    "stonyx": "0.2.3-beta.6",
+    "@stonyx/events": "0.1.1-beta.7",
+    "@stonyx/cron": "0.2.1-beta.12"
   },
   "peerDependencies": {
     "mysql2": "^3.0.0"
@@ -46,8 +46,8 @@
     }
   },
   "devDependencies": {
-    "@stonyx/rest-server": "0.2.1-beta.14",
-    "@stonyx/utils": "0.2.3-beta.4",
+    "@stonyx/rest-server": "0.2.1-beta.16",
+    "@stonyx/utils": "0.2.3-beta.5",
     "qunit": "^2.24.1",
     "sinon": "^21.0.0"
   },

package/src/index.js

@@ -15,20 +15,17 @@
  */
 
 import Model from './model.js';
-import View from './view.js';
 import Serializer from './serializer.js';
 
 import attr from './attr.js';
 import belongsTo from './belongs-to.js';
 import hasMany from './has-many.js';
 import { createRecord, updateRecord } from './manage-record.js';
-import { count, avg, sum, min, max } from './aggregates.js';
 
 export { default } from './main.js';
 export { store, relationships } from './main.js';
-export { Model, View, Serializer }; // base classes
+export { Model, Serializer }; // base classes
 export { attr, belongsTo, hasMany, createRecord, updateRecord }; // helpers
-export { count, avg, sum, min, max }; // aggregate helpers
 export { beforeHook, afterHook, clearHook, clearAllHooks } from './hooks.js'; // middleware hooks
 
 // Store API:

package/src/main.js

@@ -37,7 +37,6 @@ export default class Orm {
   
   models = {};
   serializers = {};
-  views = {};
   transforms = { ...baseTransforms };
   warnings = new Set();
 
@@ -80,28 +79,14 @@ export default class Orm {
     // Wait for imports before db & rest server setup
     await Promise.all(promises);
 
-    // Discover views from paths.view (separate from model/serializer/transform)
-    if (paths.view) {
-      await forEachFileImport(paths.view, (exported, { name }) => {
-        const alias = `${kebabCaseToPascalCase(name)}View`;
-        Orm.store.set(name, new Map());
-        registerPluralName(name, exported);
-        this.views[alias] = exported;
-      }, { ignoreAccessFailure: true, rawName: true, recursive: true, recursiveNaming: true });
-    }
-
     // Setup event names for hooks after models are loaded
     const eventNames = [];
     const operations = ['list', 'get', 'create', 'update', 'delete'];
-    const viewOperations = ['list', 'get'];
     const timings = ['before', 'after'];
 
     for (const modelName of Orm.store.data.keys()) {
-      const isView = this.isView(modelName);
-      const ops = isView ? viewOperations : operations;
-
       for (const timing of timings) {
-        for (const operation of ops) {
+        for (const operation of operations) {
           eventNames.push(`${timing}:${operation}:${modelName}`);
         }
       }
@@ -156,27 +141,13 @@ export default class Orm {
 
   getRecordClasses(modelName) {
     const modelClassPrefix = kebabCaseToPascalCase(modelName);
-
-    // Check views first, then models
-    const viewClass = this.views[`${modelClassPrefix}View`];
-    if (viewClass) {
-      return {
-        modelClass: viewClass,
-        serializerClass: this.serializers[`${modelClassPrefix}Serializer`] || Serializer
-      };
-    }
-
+  
     return {
       modelClass: this.models[`${modelClassPrefix}Model`],
       serializerClass: this.serializers[`${modelClassPrefix}Serializer`] || Serializer
     };
   }
 
-  isView(modelName) {
-    const modelClassPrefix = kebabCaseToPascalCase(modelName);
-    return !!this.views[`${modelClassPrefix}View`];
-  }
-
   // Queue warnings to avoid the same error from being logged in the same iteration
   warn(message) {
     this.warnings.add(message);

package/src/manage-record.js

@@ -14,11 +14,6 @@ export function createRecord(modelName, rawData={}, userOptions={}) {
 
   if (!initialized && !options.isDbRecord) throw new Error('ORM is not ready');
 
-  // Guard: read-only views cannot have records created directly
-  if (orm?.isView?.(modelName) && !options.isDbRecord) {
-    throw new Error(`Cannot create records for read-only view '${modelName}'`);
-  }
-
   const modelStore = store.get(modelName);
   const globalRelationships = relationships.get('global');
   const pendingRelationships = relationships.get('pending');
@@ -88,12 +83,6 @@ export function createRecord(modelName, rawData={}, userOptions={}) {
 export function updateRecord(record, rawData, userOptions={}) {
   if (!rawData) throw new Error('rawData must be passed in to updateRecord call');
 
-  // Guard: read-only views cannot be updated
-  const modelName = record?.__model?.__name;
-  if (modelName && Orm.instance?.isView?.(modelName)) {
-    throw new Error(`Cannot update records for read-only view '${modelName}'`);
-  }
-
   const options = { ...defaultOptions, ...userOptions, update:true };
 
   record.serialize(rawData, options);

package/src/model-property.js

@@ -22,8 +22,8 @@ export default class ModelProperty {
       return this._value = newValue;
     }
 
-    if (newValue === undefined || newValue === null) return;
+    if (newValue === undefined) return;
 
-    this._value = Orm.instance.transforms[this.type](newValue);
+    this._value = newValue === null ? null : Orm.instance.transforms[this.type](newValue);
   }
 }

package/src/mysql/migration-generator.js

@@ -1,4 +1,4 @@
-import { introspectModels, introspectViews, buildTableDDL, buildViewDDL, schemasToSnapshot, viewSchemasToSnapshot, getTopologicalOrder } from './schema-introspector.js';
+import { introspectModels, buildTableDDL, schemasToSnapshot, getTopologicalOrder } from './schema-introspector.js';
 import { readFile, createFile, createDirectory, fileExists } from '@stonyx/utils/file';
 import path from 'path';
 import config from 'stonyx/config';
@@ -16,18 +16,9 @@ export async function generateMigration(description = 'migration') {
   const previousSnapshot = await loadLatestSnapshot(migrationsPath);
   const diff = diffSnapshots(previousSnapshot, currentSnapshot);
 
-  // Don't return early — check view changes too before deciding
   if (!diff.hasChanges) {
-    // Check if there are view changes before returning null
-    const viewSchemasPrelim = introspectViews();
-    const currentViewSnapshotPrelim = viewSchemasToSnapshot(viewSchemasPrelim);
-    const previousViewSnapshotPrelim = extractViewsFromSnapshot(previousSnapshot);
-    const viewDiffPrelim = diffViewSnapshots(previousViewSnapshotPrelim, currentViewSnapshotPrelim);
-
-    if (!viewDiffPrelim.hasChanges) {
-      log.db('No schema changes detected.');
-      return null;
-    }
+    log.db('No schema changes detected.');
+    return null;
   }
 
   const upStatements = [];
@@ -94,71 +85,17 @@ export async function generateMigration(description = 'migration') {
     downStatements.push(`ALTER TABLE \`${table}\` ADD FOREIGN KEY (\`${column}\`) REFERENCES \`${references.references}\`(\`${references.column}\`) ON DELETE SET NULL;`);
   }
 
-  // View migrations — views are created AFTER tables (dependency order)
-  const viewSchemas = introspectViews();
-  const currentViewSnapshot = viewSchemasToSnapshot(viewSchemas);
-  const previousViewSnapshot = extractViewsFromSnapshot(previousSnapshot);
-  const viewDiff = diffViewSnapshots(previousViewSnapshot, currentViewSnapshot);
-
-  if (viewDiff.hasChanges) {
-    upStatements.push('');
-    upStatements.push('-- Views');
-    downStatements.push('');
-    downStatements.push('-- Views');
-
-    // Added views
-    for (const name of viewDiff.addedViews) {
-      try {
-        const ddl = buildViewDDL(name, viewSchemas[name], schemas);
-        upStatements.push(ddl + ';');
-        downStatements.unshift(`DROP VIEW IF EXISTS \`${viewSchemas[name].viewName}\`;`);
-      } catch (error) {
-        upStatements.push(`-- WARNING: Could not generate DDL for view '${name}': ${error.message}`);
-      }
-    }
-
-    // Removed views
-    for (const name of viewDiff.removedViews) {
-      upStatements.push(`-- WARNING: View '${name}' was removed. Uncomment to drop view:`);
-      upStatements.push(`-- DROP VIEW IF EXISTS \`${previousViewSnapshot[name].viewName}\`;`);
-      downStatements.push(`-- Recreate view for removed view '${name}' manually if needed`);
-    }
-
-    // Changed views (source or aggregates changed)
-    for (const name of viewDiff.changedViews) {
-      try {
-        const ddl = buildViewDDL(name, viewSchemas[name], schemas);
-        upStatements.push(ddl + ';');
-      } catch (error) {
-        upStatements.push(`-- WARNING: Could not generate DDL for changed view '${name}': ${error.message}`);
-      }
-    }
-  }
-
-  const combinedHasChanges = diff.hasChanges || viewDiff.hasChanges;
-
-  if (!combinedHasChanges) {
-    log.db('No schema changes detected.');
-    return null;
-  }
-
-  // Merge view snapshot into the main snapshot
-  const combinedSnapshot = { ...currentSnapshot };
-  for (const [name, viewSnap] of Object.entries(currentViewSnapshot)) {
-    combinedSnapshot[name] = viewSnap;
-  }
-
   const sanitizedDescription = description.replace(/\s+/g, '_').replace(/[^a-zA-Z0-9_]/g, '');
   const timestamp = Math.floor(Date.now() / 1000);
   const filename = `${timestamp}_${sanitizedDescription}.sql`;
   const content = `-- UP\n${upStatements.join('\n')}\n\n-- DOWN\n${downStatements.join('\n')}\n`;
 
   await createFile(path.join(migrationsPath, filename), content);
-  await createFile(path.join(migrationsPath, '.snapshot.json'), JSON.stringify(combinedSnapshot, null, 2));
+  await createFile(path.join(migrationsPath, '.snapshot.json'), JSON.stringify(currentSnapshot, null, 2));
 
   log.db(`Migration generated: ${filename}`);
 
-  return { filename, content, snapshot: combinedSnapshot };
+  return { filename, content, snapshot: currentSnapshot };
 }
 
 export async function loadLatestSnapshot(migrationsPath) {
@@ -249,38 +186,3 @@ export function detectSchemaDrift(schemas, snapshot) {
   const current = schemasToSnapshot(schemas);
   return diffSnapshots(snapshot, current);
 }
-
-export function extractViewsFromSnapshot(snapshot) {
-  const views = {};
-  for (const [name, entry] of Object.entries(snapshot)) {
-    if (entry.isView) views[name] = entry;
-  }
-  return views;
-}
-
-export function diffViewSnapshots(previous, current) {
-  const addedViews = [];
-  const removedViews = [];
-  const changedViews = [];
-
-  for (const name of Object.keys(current)) {
-    if (!previous[name]) {
-      addedViews.push(name);
-    } else if (
-      current[name].viewQuery !== previous[name].viewQuery ||
-      current[name].source !== previous[name].source
-    ) {
-      changedViews.push(name);
-    }
-  }
-
-  for (const name of Object.keys(previous)) {
-    if (!current[name]) {
-      removedViews.push(name);
-    }
-  }
-
-  const hasChanges = addedViews.length > 0 || removedViews.length > 0 || changedViews.length > 0;
-
-  return { hasChanges, addedViews, removedViews, changedViews };
-}

package/src/mysql/mysql-db.js

@@ -1,6 +1,6 @@
 import { getPool, closePool } from './connection.js';
 import { ensureMigrationsTable, getAppliedMigrations, getMigrationFiles, applyMigration, parseMigrationFile } from './migration-runner.js';
-import { introspectModels, introspectViews, getTopologicalOrder, schemasToSnapshot } from './schema-introspector.js';
+import { introspectModels, getTopologicalOrder, schemasToSnapshot } from './schema-introspector.js';
 import { loadLatestSnapshot, detectSchemaDrift } from './migration-generator.js';
 import { buildInsert, buildUpdate, buildDelete, buildSelect } from './query-builder.js';
 import { createRecord, store } from '@stonyx/orm';
@@ -14,7 +14,7 @@ import path from 'path';
 const defaultDeps = {
   getPool, closePool, ensureMigrationsTable, getAppliedMigrations,
   getMigrationFiles, applyMigration, parseMigrationFile,
-  introspectModels, introspectViews, getTopologicalOrder, schemasToSnapshot,
+  introspectModels, getTopologicalOrder, schemasToSnapshot,
   loadLatestSnapshot, detectSchemaDrift,
   buildInsert, buildUpdate, buildDelete, buildSelect,
   createRecord, store, confirm, readFile, getPluralName, config, log, path
@@ -148,35 +148,6 @@ export default class MysqlDB {
         throw error;
       }
     }
-
-    // Load views with memory: true
-    const viewSchemas = this.deps.introspectViews();
-
-    for (const [viewName, viewSchema] of Object.entries(viewSchemas)) {
-      const { modelClass: viewClass } = Orm.instance.getRecordClasses(viewName);
-      if (viewClass?.memory !== true) {
-        this.deps.log.db(`Skipping memory load for view '${viewName}' (memory: false)`);
-        continue;
-      }
-
-      const schema = { table: viewSchema.viewName, columns: viewSchema.columns || {}, foreignKeys: viewSchema.foreignKeys || {} };
-      const { sql, values } = this.deps.buildSelect(schema.table);
-
-      try {
-        const [rows] = await this.pool.execute(sql, values);
-
-        for (const row of rows) {
-          const rawData = this._rowToRawData(row, schema);
-          this.deps.createRecord(viewName, rawData, { isDbRecord: true, serialize: false, transform: false });
-        }
-      } catch (error) {
-        if (error.code === 'ER_NO_SUCH_TABLE') {
-          this.deps.log.db(`View '${viewSchema.viewName}' does not exist yet. Skipping load for '${viewName}'.`);
-          continue;
-        }
-        throw error;
-      }
-    }
   }
 
   /**
@@ -195,16 +166,7 @@ export default class MysqlDB {
    */
   async findRecord(modelName, id) {
     const schemas = this.deps.introspectModels();
-    let schema = schemas[modelName];
-
-    // Check views if not found in models
-    if (!schema) {
-      const viewSchemas = this.deps.introspectViews();
-      const viewSchema = viewSchemas[modelName];
-      if (viewSchema) {
-        schema = { table: viewSchema.viewName, columns: viewSchema.columns || {}, foreignKeys: viewSchema.foreignKeys || {} };
-      }
-    }
+    const schema = schemas[modelName];
 
     if (!schema) return undefined;
 
@@ -237,16 +199,7 @@ export default class MysqlDB {
    */
   async findAll(modelName, conditions) {
     const schemas = this.deps.introspectModels();
-    let schema = schemas[modelName];
-
-    // Check views if not found in models
-    if (!schema) {
-      const viewSchemas = this.deps.introspectViews();
-      const viewSchema = viewSchemas[modelName];
-      if (viewSchema) {
-        schema = { table: viewSchema.viewName, columns: viewSchema.columns || {}, foreignKeys: viewSchema.foreignKeys || {} };
-      }
-    }
+    const schema = schemas[modelName];
 
     if (!schema) return [];
 
@@ -324,10 +277,6 @@ export default class MysqlDB {
   }
 
   async persist(operation, modelName, context, response) {
-    // Views are read-only — no-op for all write operations
-    const Orm = (await import('@stonyx/orm')).default;
-    if (Orm.instance?.isView?.(modelName)) return;
-
     switch (operation) {
       case 'create':
         return this._persistCreate(modelName, context, response);