json-rest-api 1.0.23 → 1.0.24

package/plugins/core/bulk-operations-plugin.js

@@ -1,4 +1,4 @@
-import { RestApiValidationError, RestApiResourceError } from '../../lib/rest-api-errors.js'
+import { RestApiValidationError } from '../../lib/rest-api-errors.js'
 
 export const BulkOperationsPlugin = {
   name: 'bulk-operations',
@@ -353,41 +353,33 @@ export const BulkOperationsPlugin = {
       // Create route handlers
       const createBulkRouteHandler = (method) => {
         return async ({ context, body, query }) => {
-          try {
-            // Parse query params for atomic mode override
-            const atomic = query?.atomic !== undefined
-              ? query.atomic === 'true'
-              : defaultAtomic
-
-            let params
-            if (method === 'bulkPost') {
-              // For bulk create, expect array in data field (JSON:API style)
-              params = {
-                inputRecords: body?.data || body,
-                atomic
-              }
-            } else if (method === 'bulkPatch') {
-              // For bulk update, expect operations array
-              params = {
-                operations: body?.operations || body,
-                atomic
-              }
-            } else if (method === 'bulkDelete') {
-              // For bulk delete, expect IDs array
-              params = {
-                ids: body?.data || body?.ids || body,
-                atomic
-              }
+          // Parse query params for atomic mode override
+          const atomic = query?.atomic !== undefined
+            ? query.atomic === 'true'
+            : defaultAtomic
+
+          let params
+          if (method === 'bulkPost') {
+            // For bulk create, expect array in data field (JSON:API style)
+            params = {
+              inputRecords: body?.data || body,
+              atomic
+            }
+          } else if (method === 'bulkPatch') {
+            // For bulk update, expect operations array
+            params = {
+              operations: body?.operations || body,
+              atomic
+            }
+          } else if (method === 'bulkDelete') {
+            // For bulk delete, expect IDs array
+            params = {
+              ids: body?.data || body?.ids || body,
+              atomic
             }
-
-            // Call the scope method
-            const result = await api.scopes[scopeName][method](params)
-
-            return result
-          } catch (error) {
-            // Let transport plugin handle error mapping
-            throw error
           }
+
+          return await api.scopes[scopeName][method](params)
         }
       }
 

package/plugins/core/connectors/express-plugin.js

@@ -15,7 +15,6 @@
  */
 
 import { requirePackage } from 'hooked-api'
-import { parseJsonApiQuery } from '../lib/querying-writing/connectors-query-parser.js'
 import { createContext } from './lib/request-helpers.js'
 import { createEnhancedLogger } from '../../../lib/enhanced-logger.js'
 import {
@@ -337,11 +336,6 @@ export const ExpressPlugin = {
           router[method.toLowerCase()](path, ...beforeMiddleware, expressHandler)
 
           // Debug logging for route registration
-
-          // Check if the route was actually added
-          if (router.stack) {
-            const lastRoute = router.stack[router.stack.length - 1]
-          }
         }
       } catch (routeError) {
         log.error('[EXPRESS DEBUG] Error creating route:', {

package/plugins/core/connectors/lib/busboy-detector.js

@@ -101,14 +101,14 @@ export function createBusboyDetector (options = {}) {
           const chunks = []
           let size = 0
 
-          const filePromise = new Promise((fileResolve, fileReject) => {
+          const filePromise = new Promise((resolve, reject) => {
             file.on('data', (chunk) => {
               chunks.push(chunk)
               size += chunk.length
             })
 
             file.on('limit', () => {
-              fileReject(new Error(`File size limit exceeded for field '${fieldname}'`))
+              reject(new Error(`File size limit exceeded for field '${fieldname}'`))
             })
 
             file.on('end', () => {
@@ -119,10 +119,10 @@ export function createBusboyDetector (options = {}) {
                 size,
                 data: Buffer.concat(chunks)
               }
-              fileResolve()
+              resolve()
             })
 
-            file.on('error', fileReject)
+            file.on('error', reject)
           })
 
           filePromises.push(filePromise)

package/plugins/core/lib/anyapi/schema-utils.js

@@ -18,6 +18,7 @@ export const DEFAULT_CANONICAL_CONFIG = {
   scopeName: 'anyapi_records',
   tableName: 'any_records',
   type: 'anyapi_records',
+  logicalIdColumn: 'logical_id',
   tenantColumn: 'tenant_id',
   resourceColumn: 'resource',
   relationshipColumn: 'relationship',
@@ -34,20 +35,37 @@ const ensureTable = async (knex, tableName, tableBuilder) => {
   }
 }
 
-const ensureIndex = async (knex, tableName, columns, indexName) => {
+const ensureColumn = async (knex, tableName, columnName, columnBuilder) => {
+  const exists = await knex.schema.hasTable(tableName)
+  if (!exists) return
+
+  const hasColumn = await knex.schema.hasColumn(tableName, columnName)
+  if (!hasColumn) {
+    await knex.schema.table(tableName, columnBuilder)
+  }
+}
+
+const ensureUniqueIndex = async (knex, tableName, columns, indexName) => {
   const exists = await knex.schema.hasTable(tableName)
   if (!exists) return
   const hasIndex = await knex.schema.withSchema('main').hasIndex?.(tableName, indexName)
   if (!hasIndex) {
-    await knex.schema.table(tableName, (table) => {
-      table.index(columns, indexName)
-    })
+    try {
+      await knex.schema.table(tableName, (table) => {
+        table.unique(columns, indexName)
+      })
+    } catch (error) {
+      if (!String(error?.message || '').includes('already exists')) {
+        throw error
+      }
+    }
   }
 }
 
 export const ensureAnyApiSchema = async (knex) => {
   await ensureTable(knex, 'any_records', (table) => {
     table.increments('id').primary()
+    table.string('logical_id')
     table.string('tenant_id').notNullable()
     table.string('resource').notNullable()
 
@@ -77,9 +95,30 @@ export const ensureAnyApiSchema = async (knex) => {
     table.dateTime('deleted_at')
 
     table.index(['tenant_id', 'resource'])
+    table.unique(['tenant_id', 'resource', 'logical_id'], 'any_records_tenant_resource_logical_id_unique')
     table.index(['resource'])
   })
 
+  await ensureColumn(knex, 'any_records', 'logical_id', (table) => {
+    table.string('logical_id')
+  })
+  await ensureUniqueIndex(
+    knex,
+    'any_records',
+    ['tenant_id', 'resource', 'logical_id'],
+    'any_records_tenant_resource_logical_id_unique'
+  )
+
+  const rowsMissingLogicalId = await knex('any_records')
+    .select('id')
+    .whereNull('logical_id')
+
+  for (const row of rowsMissingLogicalId) {
+    await knex('any_records')
+      .where({ id: row.id })
+      .update({ logical_id: String(row.id) })
+  }
+
   await ensureTable(knex, 'any_links', (table) => {
     table.increments('id').primary()
     table.string('tenant_id').notNullable()

package/plugins/core/lib/anyapi/utils/descriptor-helpers.js

@@ -22,7 +22,10 @@ export const findSchemaFieldByAlias = (descriptor, alias) => {
 export const resolveFieldInfo = (descriptor, field) => {
   if (!descriptor) return null
   if (field === 'id') {
-    return { column: 'id', definition: { type: 'id' } }
+    return {
+      column: descriptor?.canonical?.logicalIdColumn || 'logical_id',
+      definition: { type: 'id' }
+    }
   }
 
   const directField = descriptor.fields?.[field]

package/plugins/core/lib/dbIntrospection.js

@@ -763,7 +763,7 @@ function deriveSqliteColumnNumbers (dataType, args = '') {
 async function introspectSqliteTableSnapshot (knex, { tableName, idColumn }) {
   const quotedTableName = quoteSqliteIdentifier(tableName)
   const tableRows = normalizeRows(await knex.raw(
-    `SELECT name, sql FROM sqlite_master WHERE type = 'table' AND name = ? LIMIT 1`,
+    'SELECT name, sql FROM sqlite_master WHERE type = \'table\' AND name = ? LIMIT 1',
     [tableName]
   ))
   const firstTableRow = Array.isArray(tableRows) ? tableRows[0] : null

package/plugins/core/lib/dbTablesOperations.js

@@ -103,13 +103,14 @@ function buildSetColumnType (values = []) {
   return `set(${normalized.map((value) => quoteJsString(value)).join(', ')})`
 }
 
-function createTableBuilderContext (schemaStructure = {}, idColumn = 'id') {
+function createTableBuilderContext (schemaStructure = {}, idColumn = 'id', storage = undefined) {
   return {
     schemaStructure,
     idProperty: idColumn,
     storageInfo: buildStorageInfo({
       schemaStructure,
-      idProperty: idColumn
+      idProperty: idColumn,
+      storage
     })
   }
 }
@@ -400,7 +401,8 @@ function buildImplicitIdDefinition (idColumn) {
 function resolveTableSchemaContext (schemaLike, options = {}) {
   const schemaStructure = schemaLike?.structure || schemaLike || {}
   const configuredIdColumn = options.idProperty || schemaStructure.id?.storage?.column || 'id'
-  const tableContext = createTableBuilderContext(schemaStructure, configuredIdColumn)
+  const storage = options.storage ?? schemaLike?.storage
+  const tableContext = createTableBuilderContext(schemaStructure, configuredIdColumn, storage)
 
   let resourceIdField = null
   for (const [fieldName, definition] of Object.entries(schemaStructure)) {
@@ -888,6 +890,7 @@ export async function createKnexTable (knex, schemaInfo, tableSchemaInstance, op
   const dialect = detectKnexDialect(knex)
   const tableSchemaContext = resolveTableSchemaContext(tableSchemaInstance, {
     idProperty: schemaInfo.idProperty,
+    storage: schemaInfo.storage,
     tableName
   })
 
@@ -918,6 +921,7 @@ export async function addKnexFields (knex, tableName, schema, options = {}) {
   assertNoTopLevelTableMetadata(schema, 'addKnexFields')
   const dialect = detectKnexDialect(knex)
   const tableSchemaContext = resolveTableSchemaContext(schema, {
+    storage: options.storage,
     ...options,
     tableName
   })
@@ -936,6 +940,7 @@ export async function alterKnexFields (knex, tableName, fields, options = {}) {
   assertNoTopLevelTableMetadata(fields, 'alterKnexFields')
   const dialect = detectKnexDialect(knex)
   const tableSchemaContext = resolveTableSchemaContext(fields, {
+    storage: options.storage,
     ...options,
     tableName
   })
@@ -961,6 +966,7 @@ export function generateKnexMigration (tableName, schema, options = {}) {
   const { autoIncrement = true, timestamps = false, dialect = '' } = options
   const tableSchemaContext = resolveTableSchemaContext(schema, {
     ...options,
+    storage: options.storage,
     tableName
   })
 
@@ -1028,6 +1034,7 @@ export function generateKnexMigrationDiff (tableName, currentSnapshot, schema, o
   const resolvedDialect = normalizeText(dialect || currentSnapshot?.dialect).toLowerCase()
   const tableSchemaContext = resolveTableSchemaContext(schema, {
     ...options,
+    storage: options.storage,
     tableName
   })
   const desiredColumns = buildDesiredColumnsMap(tableSchemaContext, { autoIncrement })

package/plugins/core/lib/querying/knex-cross-table-search.js

@@ -208,7 +208,6 @@ export const buildJoinChain = async (scopes, log, fromScopeName, targetPath, sea
     let foundRelationship = null
     let relationshipType = null
     let relationshipField = null
-    let relationshipDef = null
 
     if (currentRelationships) {
       for (const [relName, relDef] of Object.entries(currentRelationships)) {
@@ -222,7 +221,6 @@ export const buildJoinChain = async (scopes, log, fromScopeName, targetPath, sea
               foundRelationship = targetScope
               relationshipType = 'hasManyPolymorphic'
               relationshipField = { typeField, idField, via: relDef.via }
-              relationshipDef = relDef
               log.trace('[BUILD-JOIN] Found polymorphic hasMany relationship:', {
                 relName, targetScope, via: relDef.via, typeField, idField
               })
@@ -238,7 +236,6 @@ export const buildJoinChain = async (scopes, log, fromScopeName, targetPath, sea
               foreignKey: relDef.foreignKey,
               otherKey: relDef.otherKey
             }
-            relationshipDef = relDef
             log.trace('[BUILD-JOIN] Found many-to-many relationship:', {
               relName,
               targetScope,
@@ -256,7 +253,6 @@ export const buildJoinChain = async (scopes, log, fromScopeName, targetPath, sea
             log.error('[BUILD-JOIN] Missing foreignKey in hasMany relationship:', { relName, currentScope })
             throw new Error(`Missing foreignKey in hasMany relationship '${relName}' for scope '${currentScope}'`)
           }
-          relationshipDef = relDef
           log.trace('[BUILD-JOIN] Found hasMany relationship:', { relName, targetScope, foreignKey: relationshipField })
           break
         }
@@ -269,7 +265,6 @@ export const buildJoinChain = async (scopes, log, fromScopeName, targetPath, sea
           foundRelationship = targetScope
           relationshipType = 'belongsTo'
           relationshipField = fieldName
-          relationshipDef = fieldDef
           log.trace('[BUILD-JOIN] Found belongsTo relationship:', { fieldName, targetScope })
           break
         }
@@ -282,9 +277,7 @@ export const buildJoinChain = async (scopes, log, fromScopeName, targetPath, sea
       )
     }
 
-    const sourceSchema = scopes[currentScope].vars.schemaInfo.schemaInstance
     const sourceTableName = scopes[currentScope].vars.schemaInfo.tableName
-    const targetSchemaInstance = scopes[targetScope].vars.schemaInfo.schemaInstance
     const targetTableName = scopes[targetScope].vars.schemaInfo.tableName
 
     if (relationshipType === 'manyToMany') {

package/plugins/core/lib/querying/knex-json-api-transformers-querying.js

@@ -439,7 +439,6 @@ export const buildJsonApiResponse = async (scope, records, included = [], isSing
     return jsonApiRecord
   })
 
-  const urlPrefix = context.urlPrefix || scope.vars.transport?.mountPath || ''
   const normalizedData = isSingle ? processedRecords[0] : processedRecords
 
   if (normalizedData) {

package/plugins/core/lib/querying/knex-query-helpers.js

@@ -191,7 +191,7 @@ export const polymorphicFiltersHook = async (hookParams, dependencies) => {
   // Step 2: Build polymorphic JOINs
   log.trace('[POLYMORPHIC-SEARCH] Building JOINs for polymorphic searches')
 
-  for (const [filterKey, searchInfo] of polymorphicSearches) {
+  for (const [, searchInfo] of polymorphicSearches) {
     const { fieldDef, polymorphicField } = searchInfo
 
     // Get the relationship definition
@@ -485,7 +485,6 @@ export const crossTableFiltersHook = async (hookParams, dependencies) => {
   }
 
   const schemaInfo = scopes[scopeName].vars.schemaInfo
-  const tableName = schemaInfo.tableName
   const tableAlias = adapterUtils.defaultAliasForScope(scopeName)
   const aliasScopeMap = new Map()
   aliasScopeMap.set(tableAlias, scopeName)
@@ -730,7 +729,7 @@ export const crossTableFiltersHook = async (hookParams, dependencies) => {
           fieldDef.applyFilter.call(this, this, filterValue)
           break
 
-        default:
+        default: {
           const targetField = fieldDef.actualField || filterKey
           const columnRef = resolveFieldColumn(targetField)
           const operator = resolveSearchOperator(fieldDef)
@@ -738,6 +737,7 @@ export const crossTableFiltersHook = async (hookParams, dependencies) => {
           applyWhereForOperator({ builder: this, columnRef, operator, value: normalized, knex })
           break
         }
+      }
     }
   })
 }
@@ -891,8 +891,6 @@ export const basicFiltersHook = async (hookParams, dependencies) => {
   const scopeName = hookParams.context?.knexQuery?.scopeName
   const filters = hookParams.context?.knexQuery?.filters
   const query = hookParams.context?.knexQuery?.query
-  const db = hookParams.context?.knexQuery?.db || knex
-
   if (!scopeName || !filters || !scopes[scopeName]) {
     return
   }
@@ -993,7 +991,7 @@ export const basicFiltersHook = async (hookParams, dependencies) => {
           fieldDef.applyFilter.call(this, this, filterValue)
           break
 
-        default:
+        default: {
           // Standard filtering
           const actualField = fieldDef.actualField || filterKey
           // Always qualify field names
@@ -1005,6 +1003,7 @@ export const basicFiltersHook = async (hookParams, dependencies) => {
           applyWhereForOperator({ builder: this, columnRef: dbField, operator, value: normalized, knex })
           break
         }
+      }
     }
   })
 }

package/plugins/core/lib/querying/knex-relationship-includes.js

@@ -75,7 +75,6 @@
  */
 
 import { applyFieldSelectionToQuery, buildFieldSelection } from '../querying-writing/knex-field-helpers.js'
-import { getForeignKeyFields } from '../querying-writing/field-utils.js'
 import { toJsonApiRecord } from './knex-json-api-transformers-querying.js'
 import { buildWindowedIncludeQuery, applyStandardIncludeConfig, buildOrderByClause } from './knex-window-queries.js'
 import { unwrapQueryBuilderState } from './query-builder-utils.js'
@@ -491,8 +490,6 @@ export const loadBelongsTo = async (scope, deps) => {
     // Get target table info
     const targetTableName = scopes[targetScope].vars.schemaInfo.tableName
     const targetIdColumn = getIdColumnForScope(scopes, targetScope)
-    const targetSchema = scopes[targetScope].vars.schemaInfo.schemaInstance
-
     // Build field selection for sparse fieldsets
     const targetScopeObject = scopes[targetScope]
     const storageAdapter = targetScopeObject.vars.storageAdapter
@@ -548,9 +545,9 @@ export const loadBelongsTo = async (scope, deps) => {
 
     // Create lookup map
     const targetById = {}
-      targetRecords.forEach(record => {
-        targetById[getFieldValueForScope(scopes, targetScope, record, 'id')] = record
-      })
+    targetRecords.forEach(record => {
+      targetById[getFieldValueForScope(scopes, targetScope, record, 'id')] = record
+    })
 
     // Set relationships on original records
     const targetRecordsToProcess = []
@@ -695,7 +692,7 @@ export const loadHasMany = async (scope, deps) => {
       const otherKey = relDef.otherKey
 
       if (!foreignKey || !otherKey) {
-        throw new Error(`Missing foreignKey or otherKey in many-to-many relationship '${relName}' for scope '${scopeName}'`)
+        throw new Error(`Missing foreignKey or otherKey in many-to-many relationship '${includeName}' for scope '${scopeName}'`)
       }
 
       // For manyToMany, the target scope is the relationship name itself
@@ -737,12 +734,11 @@ export const loadHasMany = async (scope, deps) => {
       })
 
       // Step 3: Build field selection for sparse fieldsets
-      const targetSchema = scopes[targetScope].vars.schemaInfo.schemaInstance
       const targetScopeObject = scopes[targetScope]
       const storageAdapter = targetScopeObject.vars.storageAdapter
       const targetIdColumn = getIdColumnForScope(scopes, targetScope)
-    const fieldSelectionInfo = fields?.[targetScope]
-      ? await buildFieldSelection(targetScopeObject, {
+      const fieldSelectionInfo = fields?.[targetScope]
+        ? await buildFieldSelection(targetScopeObject, {
           context: {
             ...(requestContext || {}),
             scopeName: targetScope,
@@ -958,7 +954,7 @@ export const loadHasMany = async (scope, deps) => {
       const foreignKey = relDef.foreignKey
 
       if (!foreignKey) {
-        throw new Error(`Missing foreignKey in hasMany relationship '${relName}' for scope '${scopeName}'`)
+        throw new Error(`Missing foreignKey in hasMany relationship '${includeName}' for scope '${scopeName}'`)
       }
 
       log.debug(`[INCLUDE] Loading ${targetScope} records with foreign key ${foreignKey}:`, {
@@ -967,7 +963,6 @@ export const loadHasMany = async (scope, deps) => {
       })
 
       // Build field selection for sparse fieldsets
-      const targetSchema = scopes[targetScope].vars.schemaInfo.schemaInstance
       const targetScopeObject = scopes[targetScope]
       const storageAdapter = targetScopeObject.vars.storageAdapter
       const fieldSelectionInfo = fields?.[targetScope]
@@ -1199,10 +1194,10 @@ export const loadHasOne = async (scope, deps) => {
   const { records, scopeName, includeName, relDef, subIncludes, included, processedPaths, currentPath, fields } = scope
   const { context: { scopes, log, knex, capabilities, requestContext } } = deps
 
-  log = log?.child ? log.child({ method: 'loadHasOne' }) : console
+  const childLog = log?.child ? log.child({ method: 'loadHasOne' }) : console
 
   if (!records || records.length === 0) {
-    log.trace('[INCLUDE] No records to load hasOne for')
+    childLog.trace('[INCLUDE] No records to load hasOne for')
     return
   }
 
@@ -1210,7 +1205,7 @@ export const loadHasOne = async (scope, deps) => {
   const mainIds = [...new Set(records.map(record => getFieldValueForScope(scopes, scopeName, record, 'id')).filter(Boolean))]
 
   if (mainIds.length === 0) {
-    log.trace('[INCLUDE] No parent IDs found, skipping hasOne load')
+    childLog.trace('[INCLUDE] No parent IDs found, skipping hasOne load')
     return
   }
 
@@ -1222,12 +1217,11 @@ export const loadHasOne = async (scope, deps) => {
     throw new Error(`Missing foreignKey in hasOne relationship '${includeName}' for scope '${scopeName}'`)
   }
 
-  log.debug(`[INCLUDE] Loading ${targetScope} records with foreign key ${foreignKey}:`, {
+  childLog.debug(`[INCLUDE] Loading ${targetScope} records with foreign key ${foreignKey}:`, {
     whereIn: mainIds
   })
 
   // Build query for hasOne - expects single result per parent
-  const targetSchema = scopes[targetScope].vars.schemaInfo.schemaInstance
   const targetScopeObject = scopes[targetScope]
   const storageAdapter = targetScopeObject.vars.storageAdapter
 
@@ -1629,7 +1623,6 @@ export const loadReversePolymorphic = async (scope, deps) => {
     })
 
     // Build field selection for sparse fieldsets
-    const targetSchema = scopes[targetScope].vars.schemaInfo.schemaInstance
     const targetScopeObject = scopes[targetScope]
     const storageAdapter = targetScopeObject.vars.storageAdapter
     const targetIdColumn = getIdColumnForScope(scopes, targetScope)

package/plugins/core/lib/querying-writing/compile-schemas.js

@@ -1,6 +1,6 @@
 import { createSchema } from 'json-rest-schema'
 import { ensureSearchFieldsAreIndexed, generateSearchSchemaFromSchema, sortFieldsByDependencies } from './schema-helpers.js'
-import { buildStorageInfo } from '../storage/storage-mapping.js'
+import { buildStorageInfo, normalizeStorageConfig } from '../storage/storage-mapping.js'
 
 /**
  * Compiles and enriches schemas for a resource scope
@@ -155,7 +155,7 @@ export async function compileSchemas (scope, deps) {
   }
 
   // Default enrichment: Add type to belongsTo fields
-  for (const [fieldName, fieldDef] of Object.entries(enrichedFields)) {
+  for (const [, fieldDef] of Object.entries(enrichedFields)) {
     if (fieldDef.belongsTo && !fieldDef.type) {
       fieldDef.type = 'id'
     }
@@ -174,7 +174,7 @@ export async function compileSchemas (scope, deps) {
 
   if (isProbablyPivot) {
     // Make all belongsTo fields searchable for pivot operations
-    for (const [fieldName, fieldDef] of belongsToFields) {
+    for (const [, fieldDef] of belongsToFields) {
       if (!fieldDef.search) {
         fieldDef.search = true
       }
@@ -206,6 +206,7 @@ export async function compileSchemas (scope, deps) {
     scope.scopeOptions.searchSchema
   )
 
+  let searchSchemaObject
   if (rawSearchFields) {
     // Mark all search fields as indexed for database optimization.
     // This ensures that any field used for filtering will have a database index created,
@@ -224,9 +225,9 @@ export async function compileSchemas (scope, deps) {
     await runHooks('searchSchema:enrich', searchSchemaContext)
 
     // Create searchSchema object
-    var searchSchemaObject = createSchema(rawSearchFields)
+    searchSchemaObject = createSchema(rawSearchFields)
   } else {
-    var searchSchemaObject = createSchema({})
+    searchSchemaObject = createSchema({})
   }
 
   // Build schemaRelationships including polymorphic fields from schema
@@ -317,9 +318,11 @@ export async function compileSchemas (scope, deps) {
   }
 
   const idProperty = scope.scopeOptions.idProperty || scope.vars.idProperty || 'id'
+  const storage = normalizeStorageConfig(scope.scopeOptions.storage)
   const storageInfo = buildStorageInfo({
     schemaStructure: schemaObject.structure,
-    idProperty
+    idProperty,
+    storage
   })
 
   // Cache everything
@@ -335,6 +338,7 @@ export async function compileSchemas (scope, deps) {
     schemaRelationships,
     tableName: scope.scopeOptions.tableName || scopeName,
     idProperty,
+    storage,
     storageInfo,
     indexes: Array.isArray(scope.scopeOptions.indexes) ? [...scope.scopeOptions.indexes] : [],
     foreignKeys: Array.isArray(scope.scopeOptions.foreignKeys) ? [...scope.scopeOptions.foreignKeys] : [],

package/plugins/core/lib/querying-writing/knex-field-helpers.js

@@ -257,7 +257,6 @@ export const buildFieldSelection = async (scope, deps) => {
         }
       })
     }
-
   } else {
     // No sparse fieldsets - return all visible fields
     // This is the default behavior when no ?fields parameter is provided
@@ -283,7 +282,7 @@ export const buildFieldSelection = async (scope, deps) => {
     // So we need to include their dependencies even if normallyHidden
     // Example: profit_margin depends on 'cost' which is normallyHidden
     // We fetch 'cost' for calculation but don't return it in response
-    for (const [fieldName, fieldDef] of Object.entries(computedFields)) {
+    for (const [, fieldDef] of Object.entries(computedFields)) {
       if (fieldDef.dependencies) {
         for (const dep of fieldDef.dependencies) {
           const depFieldDef = schemaStructure[dep]