@ditojs/server 2.47.0 → 2.49.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ditojs/server",
-  "version": "2.47.0",
+  "version": "2.49.0",
   "type": "module",
   "description": "Dito.js Server – Dito.js is a declarative and modern web framework, based on Objection.js, Koa.js and Vue.js",
   "repository": "https://github.com/ditojs/dito/tree/master/packages/server",
@@ -25,15 +25,15 @@
     "node >= 18"
   ],
   "dependencies": {
-    "@ditojs/admin": "^2.47.0",
-    "@ditojs/build": "^2.47.0",
-    "@ditojs/router": "^2.47.0",
-    "@ditojs/utils": "^2.47.0",
+    "@ditojs/admin": "^2.49.0",
+    "@ditojs/build": "^2.49.0",
+    "@ditojs/router": "^2.49.0",
+    "@ditojs/utils": "^2.49.0",
     "@koa/cors": "^5.0.0",
     "@koa/multer": "^3.1.0",
     "@originjs/vite-plugin-commonjs": "^1.0.3",
     "ajv": "^8.17.1",
-    "ajv-formats": "^2.1.1",
+    "ajv-formats": "^3.0.1",
     "bcryptjs": "^3.0.2",
     "bytes": "^3.1.2",
     "data-uri-to-buffer": "^6.0.2",
@@ -66,7 +66,7 @@
     "pino-pretty": "^13.0.0",
     "pluralize": "^8.0.0",
     "repl": "^0.1.3",
-    "type-fest": "^4.40.1",
+    "type-fest": "^4.41.0",
     "uuid": "^11.1.0"
   },
   "peerDependencies": {
@@ -84,11 +84,11 @@
     "@types/koa-session": "^6.4.5",
     "@types/koa-static": "^4.0.4",
     "@types/koa__cors": "^5.0.0",
-    "@types/node": "^22.15.3",
+    "@types/node": "^22.15.17",
     "knex": "^3.1.0",
     "objection": "^3.1.5",
     "typescript": "^5.8.3"
   },
   "types": "types",
-  "gitHead": "9ce26a1dc412e67304e823c0206593ee4a9c8ae4"
+  "gitHead": "8276ff1db64a82c03e1569e2403f378b0201ea3c"
 }

package/src/app/Application.js

@@ -31,8 +31,7 @@ import {
   parseDataPath,
   normalizeDataPath,
   toPromiseCallback,
-  mapConcurrently,
-  deprecate
+  mapConcurrently
 } from '@ditojs/utils'
 import { Validator } from './Validator.js'
 import { EventEmitter } from '../lib/index.js'
@@ -879,11 +878,6 @@ export class Application extends Koa {
     }
   }
 
-  startOrExit() {
-    deprecate(`app.startOrExit() is deprecated. Use app.execute() instead.`)
-    return this.execute()
-  }
-
   // Assets handling
 
   async createAssets(storage, files, count = 0, transaction = null) {

package/src/app/Validator.js

@@ -3,10 +3,15 @@ import Ajv from 'ajv/dist/2020.js'
 import addFormats from 'ajv-formats'
 import { isArray, isObject, clone, isAsync, isPromise } from '@ditojs/utils'
 import { formatJson } from '../utils/json.js'
-import * as schema from '../schema/index.js'
+import {
+  keywords as defaultKeywords,
+  formats as defaultFormats,
+  types as defaultTypes,
+  convertSchema
+} from '../schema/index.js'
 
-// Dito does not rely on objection.AjvValidator but instead implements its own
-// validator instance that is shared across the whole app and handles schema
+// Dito.js does not rely on objection.AjvValidator but instead implements its
+// own validator instance that is shared across the whole app and handles schema
 // compilation and caching differently:
 // It relies on Ajv's addSchema() / getSchema() pattern in conjunction with the
 // `schemaId: '$id'` option, and each schema is assigned an $id based on the
@@ -14,23 +19,25 @@ import * as schema from '../schema/index.js'
 // easily validate nested structures.
 
 export class Validator extends objection.Validator {
-  constructor({ options, keywords, formats } = {}) {
+  constructor({ options, keywords, formats, types } = {}) {
     super()
 
     this.options = {
       ...defaultOptions,
       ...options
     }
-
     this.keywords = {
-      ...schema.keywords,
+      ...defaultKeywords,
       ...keywords
     }
-
     this.formats = {
-      ...schema.formats,
+      ...defaultFormats,
       ...formats
     }
+    this.types = {
+      ...defaultTypes,
+      ...types
+    }
 
     this.schemas = []
 
@@ -71,6 +78,13 @@ export class Validator extends objection.Validator {
       })
     )
 
+    addSchemas(this.types, (type, schema) => {
+      ajv.addSchema({
+        $id: type,
+        ...this.processSchema(convertSchema(schema), options)
+      })
+    })
+
     // Also add all model schemas that were already compiled so far.
     for (const schema of this.schemas) {
       ajv.addSchema(this.processSchema(schema, options))

package/src/cli/console.js

@@ -32,7 +32,7 @@ export default async function startConsole(app, config) {
   server.eval = wrapEval(server)
 
   server.defineCommand('usage', {
-    help: 'Detailed Dito Console usage information',
+    help: 'Detailed Dito.js Console usage information',
     action() {
       displayUsage(app, config, true)
       this.displayPrompt()
@@ -40,7 +40,7 @@ export default async function startConsole(app, config) {
   })
 
   server.defineCommand('models', {
-    help: 'Display available Dito models',
+    help: 'Display available Dito.js models',
     action() {
       console.info(Object.keys(app.models).join(', '))
       this.displayPrompt()
@@ -59,7 +59,7 @@ export default async function startConsole(app, config) {
       .slice(0, config.historySize)
       .filter(line => line.trim())
       .map(line => server.history.push(line))
-  } catch (e) {
+  } catch {
     console.info(deindent`
       Unable to REPL history file at ${historyFile}.
       A history file will be created on shutdown
@@ -99,13 +99,13 @@ function displayUsage(app, config, details) {
   console.info(deindent`
 
     ------------------------------------------------------------
-    Dito Console
+    Dito.js Console
 
     Available references:
-     - Dito app: ${pico.cyan('app')}
+     - Dito.js app: ${pico.cyan('app')}
     ${
       modelHandleNames.length > 0
-        ? ` - Dito models: ${
+        ? ` - Dito.js models: ${
             modelHandleNames.map(m => pico.cyan(m)).join(', ')
           }`
         : ''

package/src/controllers/CollectionController.js

@@ -182,19 +182,7 @@ export class CollectionController extends Controller {
     )
   }
 
-  convertToCoreActions(actions) {
-    // Mark action object and methods as core, so `Controller.processValues()`
-    // can filter correctly.
-    for (const action of Object.values(actions)) {
-      // Mark action functions also, so ControllerAction can use it to determine
-      // value for `transacted`.
-      action.core = true
-    }
-    actions.$core = true
-    return actions
-  }
-
-  collection = this.convertToCoreActions({
+  collection = this.markAsCoreActions({
     async get(ctx, modify) {
       const result = await this.execute(ctx, (query, trx) => {
         query
@@ -245,7 +233,7 @@ export class CollectionController extends Controller {
     }
   })
 
-  member = this.convertToCoreActions({
+  member = this.markAsCoreActions({
     async get(ctx, modify) {
       return this.execute(ctx, (query, trx) =>
         query

package/src/controllers/Controller.js

@@ -32,8 +32,8 @@ export class Controller {
   name = null
   path = null
   url = null
-  actions = null
   assets = null
+  actions = null
   transacted = null
   initialized = false
 
@@ -80,7 +80,6 @@ export class Controller {
   // @overridable
   setup() {
     this.logController()
-    this.setProperty('actions', this.actions || this.reflectActionsObject())
     // Now that the instance fields are reflected in the `controller` object
     // we can use the normal inheritance mechanism through `setupActions()`:
     this.setProperty('actions', this.setupActions('actions'))
@@ -142,30 +141,15 @@ export class Controller {
     return value
   }
 
-  reflectActionsObject() {
-    // On base controllers, the actions can be defined directly in the class
-    // instead of inside an actions object, as is done with model and relation
-    // controllers. But in order to use the same structure for inheritance as
-    // these other controllers, we reflect these instance fields in a separate
-    // `actions` object.
-    const { allow } = this
-    const actions = allow ? { allow } : {}
-
-    const addAction = key => {
-      const value = this[key]
-      // NOTE: Only add instance methods that have a @action() decorator, which
-      // in turn sets the `method` property on the method, as well as action
-      // objects which provide the `method` property:
-      if (value?.method) {
-        actions[key] = value
-      }
+  markAsCoreActions(actions) {
+    // Mark action object and methods as core, so `Controller.processValues()`
+    // can filter correctly.
+    for (const action of Object.values(actions)) {
+      // Mark action functions also, so ControllerAction can use it to determine
+      // value for `transacted`.
+      action.core = true
     }
-    // Use `Object.getOwnPropertyNames()` to get the fields, in order to
-    // not also receive values from parents (those are fetched later in
-    // `inheritValues()`, see `getParentValues()`).
-    const proto = Object.getPrototypeOf(this)
-    Object.getOwnPropertyNames(proto).forEach(addAction)
-    Object.getOwnPropertyNames(this).forEach(addAction)
+    actions.$core = true
     return actions
   }
 
@@ -190,26 +174,28 @@ export class Controller {
       values: actions,
       authorize
     } = this.processValues(this.inheritValues(type))
-    for (const [name, action] of Object.entries(actions)) {
-      // Replace the action object with the converted action handler, so they
-      // too can benefit from prototypal inheritance:
-      actions[name] = this.setupAction(
-        type,
-        actions,
-        name,
-        action,
-        authorize[name]
-      )
+    if (actions) {
+      for (const [name, action] of Object.entries(actions)) {
+        // Replace the action object with the converted action handler, so they
+        // too can benefit from prototypal inheritance:
+        actions[name] = this.setupAction(
+          type,
+          actions,
+          name,
+          action,
+          authorize[name]
+        )
+      }
+      // Expose a direct reference to the controller on the action object, but
+      // also make it inherit from the controller so that all its public fields
+      // and functions (`app`, `query()`, `execute()`, etc.) can be accessed
+      // directly through `this` from actions.
+      // NOTE: Inheritance is also set up by `inheritValues()` so that from the
+      // handlers, `super` points to the parent controller's actions object, so
+      // that calling `super.patch()` from a patch handler magically works.
+      actions.controller = this
+      Object.setPrototypeOf(actions, this)
     }
-    // Expose a direct reference to the controller on the action object, but
-    // also make it inherit from the controller so that all its public fields
-    // and functions (`app`, `query()`, `execute()`, etc.) can be accessed
-    // directly through `this` from actions.
-    // NOTE: Inheritance is also set up by `inheritValues()` so that from inside
-    // the handlers, `super` points to the parent controller's actions object,
-    // so that calling `super.patch()` from a patch handler magically works.
-    actions.controller = this
-    Object.setPrototypeOf(actions, this)
     return actions
   }
 

package/src/controllers/ControllerAction.js

@@ -7,18 +7,12 @@ export default class ControllerAction {
     handler,
     type,
     name,
-    _method,
-    _path,
+    method,
+    path,
     _authorize
   ) {
     const {
       core = false,
-      // Allow decorators on actions to override the predetermined defaults for
-      // `method`, `path` and `authorize`:
-      // TODO: `handler.method` and `handler.path` were deprecated in March
-      // 2022, remove later and only set the valued passed to constructor then.
-      method = _method,
-      path = _path,
       scope,
       authorize,
       transacted,
@@ -28,6 +22,7 @@ export default class ControllerAction {
       ...additional
     } = handler
 
+    this.app = controller.app
     this.controller = controller
     this.actions = actions
     this.handler = handler
@@ -37,6 +32,8 @@ export default class ControllerAction {
     this.method = method
     this.path = path
     this.scope = scope
+    // Allow action handlers to override the predetermined defaults for
+    // `authorize`:
     this.authorize = authorize || _authorize
     this.transacted = !!(
       transacted ??
@@ -52,7 +49,6 @@ export default class ControllerAction {
       )
     )
     this.authorization = controller.processAuthorize(this.authorize)
-    this.app = controller.app
     this.paramsName = ['post', 'put', 'patch'].includes(this.method)
       ? 'body'
       : 'query'
@@ -304,7 +300,7 @@ export default class ControllerAction {
       if (objectType) {
         if (value && isString(value)) {
           if (!/^\{.*\}$/.test(value)) {
-            // Convert simplified Dito object notation to JSON, supporting:
+            // Convert simplified Dito.js object notation to JSON, supporting:
             // - `"key1":X, "key2":Y` (curly braces are added and parsed through
             //   `JSON.parse()`)
             // - `key1:X,key2:Y` (a simple parser is applied, splitting into
@@ -333,7 +329,7 @@ export default class ControllerAction {
           }
         }
         if (objectType !== 'object' && isObject(value)) {
-          // Convert the Pojo to the desired Dito model:
+          // Convert the Pojo to the desired Dito.js model:
           const modelClass = this.app.models[objectType]
           if (modelClass && !(value instanceof modelClass)) {
             value = modelClass.fromJson(value, modelOptions)

package/src/controllers/RelationController.js

@@ -87,7 +87,7 @@ export class RelationController extends CollectionController {
     })
   }
 
-  collection = this.convertToCoreActions({})
+  collection = this.markAsCoreActions({})
 
-  member = this.convertToCoreActions({})
+  member = this.markAsCoreActions({})
 }

package/src/index.js

@@ -6,5 +6,4 @@ export * from './mixins/index.js'
 export * from './models/index.js'
 export * from './controllers/index.js'
 export * from './services/index.js'
-export * from './decorators/index.js'
 export * from './storage/index.js'

package/src/middleware/handleSession.js

@@ -11,7 +11,6 @@ export function handleSession(app, {
     // Create a ContextStore that resolved the specified model class,
     // uses it to persist and retrieve the session, and automatically
     // binds all db operations to `ctx.transaction`, if it is set.
-    // eslint-disable-next-line new-cap
     options.ContextStore = createSessionStore(modelClass)
   }
   options.autoCommit = false

package/src/models/Model.js

@@ -343,7 +343,7 @@ export class Model extends objection.Model {
                 },
                 {}
               ),
-              additionalProperties: false
+              unevaluatedProperties: false
             },
             {
               type: 'object',
@@ -352,7 +352,7 @@ export class Model extends objection.Model {
                   type: 'string'
                 }
               },
-              additionalProperties: false
+              unevaluatedProperties: false
             }
           ]
         },
@@ -387,13 +387,21 @@ export class Model extends objection.Model {
     return this._getCached(
       'jsonSchema',
       () => {
-        const schema = convertSchema({
-          type: 'object',
-          properties: this.definition.properties
-        })
+        const definitions = {}
+        const schema = convertSchema(
+          {
+            type: 'object',
+            properties: this.definition.properties
+          },
+          { definitions }
+        )
         addRelationSchemas(this, schema.properties)
         // Merge in root-level schema additions
         assignDeeply(schema, this.definition.schema)
+        // Merge in definitions
+        if (Object.keys(definitions).length > 0) {
+          schema.definitions = definitions
+        }
         return {
           $id: this.name,
           ...schema
@@ -467,7 +475,17 @@ export class Model extends objection.Model {
   static getAttributes(filter) {
     const attributes = []
     const { properties } = this.definition
-    for (const [name, property] of Object.entries(properties)) {
+    const { definitions } = this.jsonSchema
+    for (let [name, property] of Object.entries(properties)) {
+      // Expand $refs so we can even find properties that uses definitions:
+      const { $ref, ...schema } = property
+      const definition = $ref && definitions?.[$ref]
+      if (definition) {
+        property = {
+          ...schema,
+          ...definition
+        }
+      }
       if (filter(property)) {
         attributes.push(name)
       }
@@ -814,8 +832,8 @@ export class Model extends objection.Model {
   // @override
   static createValidator() {
     // Use a shared validator per app, so model schema can reference each other.
-    // NOTE: The Dito Validator class creates and manages this shared Objection
-    // Validator instance for us, we just need to return it here:
+    // NOTE: The Dito.js Validator class creates and manages this shared
+    // Objection Validator instance for us, we just need to return it here:
     return this.app.validator
   }
 

package/src/query/QueryBuilder.js

@@ -1065,19 +1065,5 @@ const mixinMethods = [
   'havingBetween',
   'havingNotBetween',
   'havingRaw',
-  'havingWrapped',
-
-  // deprecated methods that are still supported at the moment.
-  // TODO: Remove once we move to Objection 3.0
-  'eager',
-  'joinEager',
-  'naiveEager',
-  'mergeEager',
-  'mergeJoinEager',
-  'mergeNaiveEager',
-  'clearEager',
-
-  'scope',
-  'mergeScope',
-  'clearScope'
+  'havingWrapped'
 ]

package/src/schema/index.js

@@ -1,4 +1,5 @@
 export * as keywords from './keywords/index.js'
 export * as formats from './formats/index.js'
+export * as types from './types/index.js'
 export * from './properties.js'
 export * from './relations.js'

package/src/schema/keywords/_extend.js

@@ -0,0 +1,31 @@
+import { resolve } from 'url'
+import { MissingRefError } from 'ajv'
+import { clone, mergeDeeply } from '@ditojs/utils'
+
+export const $extend = {
+  macro(schemas, parentSchema, ctx) {
+    const [source, ...patch] = schemas.map(schema => {
+      const { $ref } = schema
+      if ($ref) {
+        const { baseId, self } = ctx
+        const id =
+          baseId && baseId !== '#'
+            ? resolve(baseId, $ref)
+            : $ref
+        const validate = self.getSchema(id)
+        if (!validate) {
+          throw new MissingRefError(baseId, $ref)
+        }
+        schema = validate.schema
+      }
+      return schema
+    })
+    return mergeDeeply(clone(source), ...patch)
+  },
+
+  metaSchema: {
+    type: 'array',
+    items: { type: 'object' },
+    minItems: 2
+  }
+}

package/src/schema/keywords/_instanceof.js

@@ -14,8 +14,8 @@ export const _instanceof = {
   },
 
   validate(schema, data) {
-    // Support instanceof for basic JS types and Dito models. If `this` is the
-    // validator's ctx (see passContext), then we can access the models and
+    // Support instanceof for basic JS types and Dito.js models. If `this` is
+    // the validator's ctx (see passContext), then we can access the models and
     // check.
     const models = this?.app?.models
     for (const type of asArray(schema)) {

package/src/schema/keywords/index.js

@@ -10,3 +10,4 @@ export * from './_instanceof.js'
 export * from './_validate.js'
 export * from './_relate.js'
 export * from './_range.js'
+export * from './_extend.js'

package/src/schema/properties.js

@@ -1,13 +1,17 @@
-import { isObject, isArray, isString } from '@ditojs/utils'
+import { isObject, isArray, isString, equals } from '@ditojs/utils'
 
 export function convertSchema(schema, options = {}) {
   if (isArray(schema)) {
-    // Needed for allOf, anyOf, oneOf, not, items:
+    // Needed for allOf, anyOf, oneOf, not, items, see below:
     schema = schema.map(entry => convertSchema(entry, options))
   } else if (isObject(schema)) {
     // Create a shallow clone so we can modify and return:
-    schema = { ...schema }
-    const { type } = schema
+    // Also collect and propagate the definitions up to the root schema through
+    // `options.definitions`, as passed from `Model static get jsonSchema()`:
+    const { definitions, ...rest } = schema
+    mergeDefinitions(options.definitions, definitions, options)
+    schema = rest
+    const { $ref, type } = schema
     if (schema.required === true) {
       // Our 'required' is not the same as JSON Schema's: Use the 'required'
       // format instead that only validates if the required value is not empty,
@@ -44,22 +48,31 @@ export function convertSchema(schema, options = {}) {
     schema.prefixItems &&= convertSchema(schema.prefixItems, options)
     schema.items &&= convertSchema(schema.items, options)
 
-    // Handle nested allOf, anyOf, oneOf, not fields
-    for (const key of ['allOf', 'anyOf', 'oneOf', 'not']) {
+    // Handle nested allOf, anyOf, oneOf & co. fields
+    for (const key of ['allOf', 'anyOf', 'oneOf', 'not', '$extend']) {
       if (key in schema) {
         schema[key] = convertSchema(schema[key], options)
       }
     }
 
-    if (isString(type)) {
+    if (isString($ref)) {
+      // If the $ref is a nested Dito.js definition, convert it to a JSON schema
+      // reference. If it is a full URL, use it as is.
+      schema.$ref = $ref.startsWith('#')
+        ? `#/definitions/${$ref}`
+        : $ref
+    } else if (isString(type)) {
       // Convert schema property notation to JSON schema
       const jsonType = jsonTypes[type]
       if (jsonType) {
         schema.type = jsonType
-        if (hasConvertedProperties && !('additionalProperties' in schema)) {
-          // Invert the logic of `additionalProperties` so that it needs to be
+        if (
+          (hasConvertedProperties || schema.discriminator) &&
+          !('unevaluatedProperties' in schema)
+        ) {
+          // Invert the logic of `unevaluatedProperties` so that it needs to be
           // explicitly set to `true`:
-          schema.additionalProperties = false
+          schema.unevaluatedProperties = false
         }
       } else if (['date', 'datetime', 'timestamp'].includes(type)) {
         // Date properties can be submitted both as a string or a Date object.
@@ -105,7 +118,7 @@ export function convertSchema(schema, options = {}) {
   return schema
 }
 
-export function convertProperties(schemaProperties, options) {
+function convertProperties(schemaProperties, options) {
   const properties = {}
   const required = []
   for (const [key, property] of Object.entries(schemaProperties)) {
@@ -117,6 +130,32 @@ export function convertProperties(schemaProperties, options) {
   return { properties, required }
 }
 
+function mergeDefinitions(definitions, defs, options) {
+  if (definitions && defs) {
+    for (const [key, def] of Object.entries(defs)) {
+      if (!key.startsWith('#')) {
+        throw new Error(
+          `Invalid definition '${
+            key
+          }', the name of nested Dito.js definitions must start with '#': ${
+            JSON.stringify(def)
+          }`
+        )
+      }
+      const definition = definitions[key]
+      const converted = convertSchema(def, options)
+      if (definition && !equals(definition, converted)) {
+        throw new Error(
+          `Duplicate nested definition for '${key}' with different schema: ${
+            JSON.stringify(def)
+          }`
+        )
+      }
+      definitions[key] = converted
+    }
+  }
+}
+
 function addFormat(schema, newFormat) {
   // Support multiple `format` keywords through `allOf`:
   const { allOf, format, ...rest } = schema

package/src/schema/properties.test.js

@@ -30,7 +30,7 @@ describe('convertSchema()', () => {
     ).toEqual({
       type: 'object',
       properties,
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -69,7 +69,7 @@ describe('convertSchema()', () => {
     ).toEqual({
       type: 'object',
       properties,
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -90,7 +90,7 @@ describe('convertSchema()', () => {
           type: 'string'
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -121,7 +121,7 @@ describe('convertSchema()', () => {
           format: 'required'
         }
       },
-      additionalProperties: false,
+      unevaluatedProperties: false,
       required: ['myString', 'myNumber']
     })
   })
@@ -142,7 +142,7 @@ describe('convertSchema()', () => {
         myString: { type: 'string' },
         myNumber: { type: 'number' }
       },
-      additionalProperties: false,
+      unevaluatedProperties: false,
       required: ['myString', 'myNumber']
     })
   })
@@ -163,11 +163,11 @@ describe('convertSchema()', () => {
       properties: {
         myText: {
           type: 'object',
-          additionalProperties: false,
+          unevaluatedProperties: false,
           properties: {}
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -178,7 +178,7 @@ describe('convertSchema()', () => {
         properties: {
           myText: {
             type: 'object',
-            additionalProperties: true,
+            unevaluatedProperties: true,
             properties: {}
           }
         }
@@ -188,11 +188,11 @@ describe('convertSchema()', () => {
       properties: {
         myText: {
           type: 'object',
-          additionalProperties: true,
+          unevaluatedProperties: true,
           properties: {}
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -223,11 +223,11 @@ describe('convertSchema()', () => {
               format: 'required'
             }
           },
-          additionalProperties: false,
+          unevaluatedProperties: false,
           required: ['myProperty']
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -256,10 +256,10 @@ describe('convertSchema()', () => {
               type: 'string'
             }
           },
-          additionalProperties: false
+          unevaluatedProperties: false
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -295,7 +295,7 @@ describe('convertSchema()', () => {
           format: 'date-time'
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -316,7 +316,7 @@ describe('convertSchema()', () => {
           $ref: 'MyModel'
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -343,7 +343,7 @@ describe('convertSchema()', () => {
           instanceof: 'MyModel'
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -366,7 +366,7 @@ describe('convertSchema()', () => {
           nullable: true
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -391,7 +391,7 @@ describe('convertSchema()', () => {
           ]
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -415,7 +415,7 @@ describe('convertSchema()', () => {
           nullable: true
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -440,7 +440,7 @@ describe('convertSchema()', () => {
           nullable: true
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -504,7 +504,7 @@ describe('convertSchema()', () => {
                   }
                 },
                 required: ['prop1', 'prop2'],
-                additionalProperties: false
+                unevaluatedProperties: false
               },
               {
                 type: 'object',
@@ -519,13 +519,13 @@ describe('convertSchema()', () => {
                   }
                 },
                 required: ['prop3', 'prop4'],
-                additionalProperties: false
+                unevaluatedProperties: false
               }
             ]
           }
         }
       },
-      additionalProperties: false
+      unevaluatedProperties: false
     })
   })
 
@@ -566,11 +566,11 @@ describe('convertSchema()', () => {
               type: 'number'
             }
           },
-          additionalProperties: false,
+          unevaluatedProperties: false,
           required: ['prop1', 'prop2']
         }
       },
-      additionalProperties: false,
+      unevaluatedProperties: false,
       required: ['myObject']
     })
   })
@@ -605,6 +605,7 @@ describe('convertSchema()', () => {
     ).toEqual({
       type: 'object',
       discriminator: { propertyName: 'foo' },
+      unevaluatedProperties: false,
       required: ['foo'],
       oneOf: [
         {
@@ -630,4 +631,76 @@ describe('convertSchema()', () => {
       ]
     })
   })
+
+  it('supports nested Dito.js definitions', () => {
+    const definitions = {}
+    expect(
+      convertSchema(
+        {
+          type: 'object',
+          properties: {
+            prop1: {
+              $ref: '#type1'
+            },
+            prop2: {
+              type: 'object',
+              properties: {
+                prop3: {
+                  $ref: '#type2'
+                }
+              },
+              definitions: {
+                '#type2': {
+                  type: 'object',
+                  properties: {
+                    prop3: {
+                      type: 'string'
+                    }
+                  }
+                }
+              }
+            }
+          },
+
+          definitions: {
+            '#type1': {
+              type: 'integer'
+            }
+          }
+        },
+        { definitions }
+      )
+    ).toEqual({
+      type: 'object',
+      unevaluatedProperties: false,
+      properties: {
+        prop1: {
+          $ref: '#/definitions/#type1'
+        },
+        prop2: {
+          type: 'object',
+          unevaluatedProperties: false,
+          properties: {
+            prop3: {
+              $ref: '#/definitions/#type2'
+            }
+          }
+        }
+      }
+    })
+    expect(definitions).toEqual({
+      '#type1': {
+        type: 'integer'
+      },
+      '#type2': {
+        type: 'object',
+        unevaluatedProperties: false,
+        properties: {
+          prop3: {
+            type: 'string'
+          }
+        }
+      }
+    })
+  })
 })

package/src/schema/types/_asset.js

@@ -0,0 +1,31 @@
+export const asset = {
+  type: 'object',
+  properties: {
+    key: {
+      type: 'string',
+      required: true
+    },
+    name: {
+      type: 'string',
+      required: true
+    },
+    type: {
+      type: 'string',
+      required: true
+    },
+    size: {
+      type: 'integer',
+      required: true
+    },
+    url: {
+      type: 'string',
+      format: 'uri'
+    },
+    width: {
+      type: 'integer'
+    },
+    height: {
+      type: 'integer'
+    }
+  }
+}

package/src/schema/types/index.js

@@ -0,0 +1 @@
+export * from './_asset.js'

package/src/services/Service.js

@@ -28,11 +28,6 @@ export class Service {
   // @overridable
   async stop() {}
 
-  /** @deprecated Use `instance.logger` instead. */
-  getLogger() {
-    return this.logger
-  }
-
   get logger() {
     const logger = this.app.requestLocals.logger ?? this.app.logger
     return logger.child({ name: this.#loggerName })

package/src/storage/AssetFile.js

@@ -60,7 +60,6 @@ export class AssetFile {
   static convert(object, storage) {
     Object.setPrototypeOf(object, AssetFile.prototype)
     setHiddenProperty(object, SYMBOL_STORAGE, storage)
-    return object
   }
 
   static create(options) {

package/src/storage/S3Storage.js

@@ -164,6 +164,6 @@ function getFileTypeFromBuffer(buffer) {
   try {
     // Use leather as fall-back for better media file mime type detection.
     return fileTypeFromBuffer(buffer)?.mime || readMediaAttributes(buffer)?.mime
-  } catch (err) {}
+  } catch {}
   return null
 }

package/src/storage/Storage.js

@@ -4,7 +4,7 @@ import multer from '@koa/multer'
 import picomatch from 'picomatch'
 import { PassThrough } from 'stream'
 import { readMediaAttributes } from 'leather'
-import { hyphenate, toPromiseCallback, deprecate } from '@ditojs/utils'
+import { hyphenate, toPromiseCallback } from '@ditojs/utils'
 import { AssetFile } from './AssetFile.js'
 
 const storageClasses = {}
@@ -77,7 +77,7 @@ export class Storage {
   }
 
   convertAssetFile(file) {
-    return AssetFile.convert(file, this)
+    AssetFile.convert(file, this)
   }
 
   convertStorageFile(storageFile) {
@@ -158,18 +158,8 @@ export class Storage {
   async _listKeys() {}
 
   async _handleUpload(req, file, config) {
-    if (config.readImageSize) {
-      deprecate(
-        `config.readImageSize is deprecated in favour of config.readDimensions`
-      )
-    }
     if (
-      (
-        config.readDimensions ||
-        // TODO: `config.readImageSize` was deprecated in favour of
-        // `config.readDimensions` in March 2023. Remove in 1 year.
-        config.readImageSize
-      ) &&
+      config.readDimensions &&
       /^(image|video)\//.test(file.mimetype)
     ) {
       return this._handleMediaFile(req, file)

package/src/utils/duration.js

@@ -1,5 +1,4 @@
 import { isNumber } from '@ditojs/utils'
-// eslint-disable-next-line import/default
 import parseDuration from 'parse-duration'
 
 export function getDuration(duration) {

package/types/index.d.ts

@@ -3,7 +3,7 @@
 // Type definitions for Dito.js server
 // Project: <https://github.com/ditojs/dito/>
 
-// Export the entire Dito namespace.
+// Export the entire Dito.js namespace.
 
 import { ObjectCannedACL, S3ClientConfig } from '@aws-sdk/client-s3'
 import { DateFormat } from '@ditojs/utils'
@@ -645,14 +645,15 @@ export class Model extends objection.Model {
   }
 
   /**
-   * Dito automatically adds an `id` property if a model property with the
+   * Dito.js automatically adds an `id` property if a model property with the
    * `primary: true` setting is not already explicitly defined.
    */
   readonly id: Id
 
   /**
-   * Dito automatically adds a `foreignKeyId` property if foreign keys occurring
-   * in relations definitions are not explicitly defined in the properties.
+   * Dito.js automatically adds a `foreignKeyId` property if foreign keys
+   * occurring in relations definitions are not explicitly defined in the
+   * properties.
    */
   readonly foreignKeyId: Id
 
@@ -1442,8 +1443,6 @@ export class Service {
   /** @overridable */
   stop(): Promise<void>
   get logger(): PinoLogger
-  /** @deprecated Use `instance.logger` instead. */
-  getLogger(ctx: KoaContext): PinoLogger
 }
 export type Services = Record<string, Class<Service> | Service>
 

package/src/decorators/action.js

@@ -1,8 +0,0 @@
-import { createDecorator } from '../utils/decorator.js'
-
-export function action(method, path) {
-  return createDecorator(value => {
-    value.method = method
-    value.path = path
-  })
-}

package/src/decorators/authorize.js

@@ -1,7 +0,0 @@
-import { createDecorator } from '../utils/decorator.js'
-
-export function authorize(authorize) {
-  return createDecorator(value => {
-    value.authorize = authorize
-  })
-}

package/src/decorators/index.js

@@ -1,6 +0,0 @@
-export * from './action.js'
-export * from './authorize.js'
-export * from './parameters.js'
-export * from './returns.js'
-export * from './scope.js'
-export * from './transacted.js'

package/src/decorators/parameters.js

@@ -1,20 +0,0 @@
-import { isArray, isObject } from '@ditojs/utils'
-import { createDecorator } from '../utils/decorator.js'
-
-export function parameters(parameters, options) {
-  if (!isArray(parameters) && !isObject(parameters)) {
-    throw new Error(
-      `@parameters() need to be defined using array or object definitions`
-    )
-  }
-
-  return createDecorator(value => {
-    value.parameters = parameters
-    // If validation options are provided, expose them through
-    // `handler.options.parameters`, see ControllerAction
-    if (options) {
-      value.options ||= {}
-      value.options.parameters = options
-    }
-  })
-}

package/src/decorators/returns.js

@@ -1,23 +0,0 @@
-import { isObject } from '@ditojs/utils'
-import { createDecorator } from '../utils/decorator.js'
-import { formatJson } from '../utils/json.js'
-
-export function returns(returns, options) {
-  if (!isObject(returns)) {
-    throw new Error(
-      `@returns(${
-        formatJson(returns, false)
-      }) needs to be defined using an object parameter definition`
-    )
-  }
-
-  return createDecorator(value => {
-    value.returns = returns
-    // If validation options are provided, expose them through
-    // `handler.options.returns`, see ControllerAction
-    if (options) {
-      value.options ||= {}
-      value.options.returns = options
-    }
-  })
-}

package/src/decorators/scope.js

@@ -1,8 +0,0 @@
-import { createDecorator } from '../utils/decorator.js'
-
-export function scope(...scopes) {
-  return createDecorator(value => {
-    const scope = (value.scope ||= [])
-    scope.push(...scopes)
-  })
-}

package/src/decorators/transacted.js

@@ -1,5 +0,0 @@
-import { createDecorator } from '../utils/decorator.js'
-
-export const transacted = createDecorator(value => {
-  value.transacted = true
-})