fast-json-stringify 3.0.1 → 3.1.0

package/.github/dependabot.yml

@@ -1,16 +1,13 @@
 version: 2
 updates:
-  - package-ecosystem: github-actions
-    directory: '/'
-    ignore:
-      - dependency-name: "actions/*"
-        update-types:
-            ["version-update:semver-minor", "version-update:semver-patch"]
+  - package-ecosystem: "github-actions"
+    directory: "/"
     schedule:
-      interval: daily
+      interval: "monthly"
     open-pull-requests-limit: 10
-  - package-ecosystem: npm
-    directory: '/'
+
+  - package-ecosystem: "npm"
+    directory: "/"
     schedule:
-      interval: daily
+      interval: "weekly"
     open-pull-requests-limit: 10

package/.github/workflows/benchmark.yml

@@ -8,6 +8,8 @@ jobs:
   benchmark:
     if: ${{ github.event.label.name == 'benchmark' }}
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     outputs:
       PR-BENCH-12: ${{ steps.benchmark-pr.outputs.BENCH_RESULT12 }}
       PR-BENCH-14: ${{ steps.benchmark-pr.outputs.BENCH_RESULT14 }}
@@ -21,7 +23,8 @@ jobs:
     steps:
       - uses: actions/checkout@v2
         with:
-          ref: ${{github.event.pull_request.head.ref}}
+          persist-credentials: false
+          ref: ${{github.event.pull_request.head.sha}}
           repository: ${{github.event.pull_request.head.repo.full_name}}
 
       - uses: actions/setup-node@v2
@@ -64,6 +67,8 @@ jobs:
   output-benchmark:
     needs: [benchmark]
     runs-on: ubuntu-latest
+    permissions:
+      pull-requests: write
     steps:
       - name: Comment PR
         uses: thollander/actions-comment-pull-request@v1

package/README.md

@@ -56,11 +56,12 @@ compile-json-stringify date format x 1,086,187 ops/sec ±0.16% (99 runs sampled)
  - <a href="#missingFields">`Missing fields`</a>
  - <a href="#patternProperties">`Pattern Properties`</a>
  - <a href="#additionalProperties">`Additional Properties`</a>
- - <a href="#anyof">`AnyOf` and `OneOf`</a>
+ - <a href="#AnyOf-and-OneOf">`AnyOf` and `OneOf`</a>
  - <a href="#ref">`Reuse - $ref`</a>
  - <a href="#long">`Long integers`</a>
  - <a href="#integer">`Integers`</a>
  - <a href="#nullable">`Nullable`</a>
+ - <a href="#largearrays">`Large Arrays`</a>
 - <a href="#security">`Security Notice`</a>
 - <a href="#acknowledgements">`Acknowledgements`</a>
 - <a href="#license">`License`</a>
@@ -117,6 +118,8 @@ const stringify = fastJson(mySchema, {
 - `schema`: external schemas references by $ref property. [More details](#ref)
 - `ajv`: [ajv v8 instance's settings](https://ajv.js.org/options.html) for those properties that require `ajv`. [More details](#anyof)
 - `rounding`: setup how the `integer` types will be rounded when not integers. [More details](#integer)
+- `largeArrayMechanism`: set the mechanism that should be used to handle large
+(by default `20000` or more items) arrays. [More details](#largearrays)
 
 
 <a name="api"></a>
@@ -268,7 +271,7 @@ const stringify = fastJson({
 const obj = {
   nickname: 'nick',
   matchfoo: 42,
-  otherfoo: 'str'
+  otherfoo: 'str',
   matchnum: 3
 }
 
@@ -311,7 +314,7 @@ const stringify = fastJson({
 const obj = {
   nickname: 'nick',
   matchfoo: 42,
-  otherfoo: 'str'
+  otherfoo: 'str',
   matchnum: 3,
   nomatchstr: 'valar morghulis',
   nomatchint: 313
@@ -582,6 +585,59 @@ Otherwise, instead of raising an error, null values will be coerced as follows:
 - `string` -> `""`
 - `boolean` -> `false`
 
+<a name="largearrays"></a>
+#### Large Arrays
+
+Large arrays are, for the scope of this document, defined as arrays containing,
+by default, `20000` elements or more. That value can be adjusted via the option
+parameter `largeArraySize`.
+
+At some point the overhead caused by the default mechanism used by
+`fast-json-stringify` to handle arrays starts increasing exponentially, leading
+to slow overall executions.
+
+##### Settings
+
+In order to improve that the user can set the `largeArrayMechanism` and
+`largeArraySize` options.
+
+`largeArrayMechanism`'s default value is `default`. Valid values for it are:
+
+- `default` - This option is a compromise between performance and feature set by
+still providing the expected functionality out of this lib but giving up some
+possible performance gain. With this option set, **large arrays** would be
+stringified by joining their stringified elements using `Array.join` instead of
+string concatenation for better performance
+- `json-stringify` - This option will remove support for schema validation
+within **large arrays** completely. By doing so the overhead previously
+mentioned is nulled, greatly improving execution time. Mind there's no change
+in behavior for arrays not considered _large_
+
+`largeArraySize`'s default value is `20000`. Valid values for it are
+integer-like values, such as:
+
+- `20000`
+- `2e4`
+- `'20000'`
+- `'2e4'` - _note this will be converted to `2`, not `20000`_
+- `1.5` - _note this will be converted to `1`_
+
+##### Benchmarks
+
+For reference, here goes some benchmarks for comparison over the three
+mechanisms. Benchmarks conducted on an old machine.
+
+- Machine: `ST1000LM024 HN-M 1TB HDD, Intel Core i7-3610QM @ 2.3GHz, 12GB RAM, 4C/8T`.
+- Node.js `v16.13.1`
+
+```
+JSON.stringify large array x 157 ops/sec ±0.73% (86 runs sampled)
+fast-json-stringify large array default x 48.72 ops/sec ±4.92% (48 runs sampled)
+fast-json-stringify large array json-stringify x 157 ops/sec ±0.76% (86 runs sampled)
+compile-json-stringify large array x 175 ops/sec ±4.47% (79 runs sampled)
+AJV Serialize large array x 58.76 ops/sec ±4.59% (60 runs sampled)
+```
+
 <a name="security"></a>
 ## Security notice
 

package/bench.js

@@ -3,6 +3,10 @@
 const benchmark = require('benchmark')
 const suite = new benchmark.Suite()
 
+const STR_LEN = 1e4
+const LARGE_ARRAY_SIZE = 2e4
+const MULTI_ARRAY_LENGHT = 1e3
+
 const schema = {
   title: 'Example Schema',
   type: 'object',
@@ -89,7 +93,8 @@ const obj = {
 
 const date = new Date()
 
-const multiArray = []
+const multiArray = new Array(MULTI_ARRAY_LENGHT)
+const largeArray = new Array(LARGE_ARRAY_SIZE)
 
 const CJS = require('compile-json-stringify')
 const CJSStringify = CJS(schemaCJS)
@@ -99,7 +104,10 @@ const CJSStringifyString = CJS({ type: 'string' })
 
 const FJS = require('.')
 const stringify = FJS(schema)
-const stringifyArray = FJS(arraySchema)
+const stringifyArrayDefault = FJS(arraySchema)
+const stringifyArrayJSONStringify = FJS(arraySchema, {
+  largeArrayMechanism: 'json-stringify'
+})
 const stringifyDate = FJS(dateFormatSchema)
 const stringifyString = FJS({ type: 'string' })
 let str = ''
@@ -110,18 +118,48 @@ const ajvSerialize = ajv.compileSerializer(schemaAJVJTD)
 const ajvSerializeArray = ajv.compileSerializer(arraySchemaAJVJTD)
 const ajvSerializeString = ajv.compileSerializer({ type: 'string' })
 
+const getRandomString = (length) => {
+  if (!Number.isInteger(length)) {
+    throw new Error('Expected integer length')
+  }
+
+  const validCharacters = 'abcdefghijklmnopqrstuvwxyz'
+  const nValidCharacters = 26
+
+  let result = ''
+  for (let i = 0; i < length; ++i) {
+    result += validCharacters[Math.floor(Math.random() * nValidCharacters)]
+  }
+
+  return result[0].toUpperCase() + result.slice(1)
+}
+
 // eslint-disable-next-line
-for (var i = 0; i < 10000; i++) {
+for (let i = 0; i < STR_LEN; i++) {
+  largeArray[i] = {
+    firstName: getRandomString(8),
+    lastName: getRandomString(6),
+    age: Math.ceil(Math.random() * 99)
+  }
+
   str += i
   if (i % 100 === 0) {
     str += '"'
   }
 }
 
+for (let i = STR_LEN; i < LARGE_ARRAY_SIZE; ++i) {
+  largeArray[i] = {
+    firstName: getRandomString(10),
+    lastName: getRandomString(4),
+    age: Math.ceil(Math.random() * 99)
+  }
+}
+
 Number(str)
 
-for (i = 0; i < 1000; i++) {
-  multiArray.push(obj)
+for (let i = 0; i < MULTI_ARRAY_LENGHT; i++) {
+  multiArray[i] = obj
 }
 
 suite.add('FJS creation', function () {
@@ -138,8 +176,12 @@ suite.add('JSON.stringify array', function () {
   JSON.stringify(multiArray)
 })
 
-suite.add('fast-json-stringify array', function () {
-  stringifyArray(multiArray)
+suite.add('fast-json-stringify array default', function () {
+  stringifyArrayDefault(multiArray)
+})
+
+suite.add('fast-json-stringify array json-stringify', function () {
+  stringifyArrayJSONStringify(multiArray)
 })
 
 suite.add('compile-json-stringify array', function () {
@@ -150,6 +192,26 @@ suite.add('AJV Serialize array', function () {
   ajvSerializeArray(multiArray)
 })
 
+suite.add('JSON.stringify large array', function () {
+  JSON.stringify(largeArray)
+})
+
+suite.add('fast-json-stringify large array default', function () {
+  stringifyArrayDefault(largeArray)
+})
+
+suite.add('fast-json-stringify large array json-stringify', function () {
+  stringifyArrayJSONStringify(largeArray)
+})
+
+suite.add('compile-json-stringify large array', function () {
+  CJSStringifyArray(largeArray)
+})
+
+suite.add('AJV Serialize large array', function () {
+  ajvSerializeArray(largeArray)
+})
+
 suite.add('JSON.stringify long string', function () {
   JSON.stringify(str)
 })

package/example.js

@@ -1,5 +1,5 @@
 const moment = require('moment')
-const fastJson = require('fast-json-stringify')
+const fastJson = require('.')
 const stringify = fastJson({
   title: 'Example Schema',
   type: 'object',

package/index.js

@@ -3,13 +3,22 @@
 /* eslint no-prototype-builtins: 0 */
 
 const Ajv = require('ajv')
+const fastUri = require('fast-uri')
 const ajvFormats = require('ajv-formats')
 const merge = require('deepmerge')
 const clone = require('rfdc')({ proto: true })
 const fjsCloned = Symbol('fast-json-stringify.cloned')
+const { randomUUID } = require('crypto')
 
 const validate = require('./schema-validator')
+
+let largeArraySize = 2e4
 let stringSimilarity = null
+let largeArrayMechanism = 'default'
+const validLargeArrayMechanisms = [
+  'default',
+  'json-stringify'
+]
 
 const addComma = `
   if (addComma) {
@@ -43,11 +52,17 @@ function mergeLocation (source, dest) {
 
 const arrayItemsReferenceSerializersMap = new Map()
 const objectReferenceSerializersMap = new Map()
+let ajvInstance = null
 
 function build (schema, options) {
   arrayItemsReferenceSerializersMap.clear()
   objectReferenceSerializersMap.clear()
+
   options = options || {}
+
+  ajvInstance = new Ajv({ ...options.ajv, strictSchema: false, uriResolver: fastUri })
+  ajvFormats(ajvInstance)
+
   isValidSchema(schema)
   if (options.schema) {
     // eslint-disable-next-line
@@ -65,6 +80,22 @@ function build (schema, options) {
     }
   }
 
+  if (options.largeArrayMechanism) {
+    if (validLargeArrayMechanisms.includes(options.largeArrayMechanism)) {
+      largeArrayMechanism = options.largeArrayMechanism
+    } else {
+      throw new Error(`Unsupported large array mechanism ${options.rounding}`)
+    }
+  }
+
+  if (options.largeArraySize) {
+    if (!Number.isNaN(Number.parseInt(options.largeArraySize, 10))) {
+      largeArraySize = options.largeArraySize
+    } else {
+      throw new Error(`Unsupported large array size. Expected integer-like, got ${options.largeArraySize}`)
+    }
+  }
+
   /* eslint no-new-func: "off" */
   let code = `
     'use strict'
@@ -72,41 +103,6 @@ function build (schema, options) {
 
   code += `
     ${asFunctions}
-
-    
-    /**
-     * Used by schemas that are dependant on calling 'ajv.validate' during runtime,
-     * it stores the value of the '$id' property of the schema (if it has it) inside
-     * a cache which is used to figure out if the schema was compiled into a validator
-     * by ajv on a previous call, if it was then the '$id' string will be used to 
-     * invoke 'ajv.validate', this allows:
-     * 
-     * 1. Schemas that depend on ajv.validate calls to leverage ajv caching system.
-     * 2. To avoid errors, since directly invoking 'ajv.validate' with the same 
-     * schema (that contains an '$id' property) twice will throw an error.
-     */
-    const $validateWithAjv = (function() {
-      const cache = new Set()
-
-      return function (schema, target) {
-        const id = schema.$id
-        
-        if (!id) {
-          return ajv.validate(schema, target)
-        }
-
-        const cached = cache.has(id)
-
-        if (cached) {
-          return ajv.validate(id, target)
-        } else {
-          cache.add(id)
-          return ajv.validate(schema, target)
-        }
-      }
-    })()
-
-
     function parseInteger(int) { return Math.${intParseFunctionName}(int) }
     `
 
@@ -163,18 +159,17 @@ function build (schema, options) {
     ;
      return ${main}
   `
-
-  const ajvInstance = new Ajv(options.ajv)
-  ajvFormats(ajvInstance)
   const dependencies = [ajvInstance]
   const dependenciesName = ['ajv']
+  ajvInstance = null
+
   dependenciesName.push(code)
 
   if (options.debugMode) {
-    dependenciesName.toString = function () {
-      return dependenciesName.join('\n')
+    return {
+      code: dependenciesName.join('\n'),
+      ajv: dependencies[0]
     }
-    return dependenciesName
   }
 
   arrayItemsReferenceSerializersMap.clear()
@@ -218,7 +213,7 @@ const numberKeywords = [
 
 /**
  * Infer type based on keyword in order to generate optimized code
- * https://json-schema.org/latest/json-schema-validation.html#rfc.section.6
+ * https://datatracker.ietf.org/doc/html/draft-handrews-json-schema-validation-01#section-6
  */
 function inferTypeByKeyword (schema) {
   // eslint-disable-next-line
@@ -874,8 +869,11 @@ function addIfThenElse (location, name) {
   let merged = merge(copy, then)
   let mergedLocation = mergeLocation(location, { schema: merged })
 
+  const schemaKey = i.$id || randomUUID()
+  ajvInstance.addSchema(i, schemaKey)
+
   code += `
-    valid = $validateWithAjv(${JSON.stringify(i)}, obj)
+    valid = ajv.validate("${schemaKey}", obj)
     if (valid) {
   `
   if (merged.if && merged.then) {
@@ -935,7 +933,7 @@ function buildObject (location, code, name) {
   `
   }
 
-  if (objectReferenceSerializersMap.has(schema)) {
+  if (objectReferenceSerializersMap.has(schema) && objectReferenceSerializersMap.get(schema) !== name) {
     code += `
       return ${objectReferenceSerializersMap.get(schema)}(input)
     }
@@ -1054,6 +1052,11 @@ function buildArray (location, code, name, key = null) {
 
   code += `
     var l = obj.length
+    if (l && l >= ${largeArraySize}) {`
+
+  const concatSnippet = `
+    }
+
     var jsonOutput= ''
     for (var i = 0; i < l; i++) {
       var json = ''
@@ -1065,7 +1068,25 @@ function buildArray (location, code, name, key = null) {
       }
     }
     return \`[\${jsonOutput}]\`
+  }`
+
+  switch (largeArrayMechanism) {
+    case 'default':
+      code += `
+      return \`[\${obj.map(${result.mapFnName}).join(',')}]\``
+      break
+
+    case 'json-stringify':
+      code += `
+      return JSON.stringify(obj)`
+      break
+
+    default:
+      throw new Error(`Unsupported large array mechanism ${largeArrayMechanism}`)
   }
+
+  code += `
+  ${concatSnippet}
   ${result.laterCode}
   `
 
@@ -1173,22 +1194,27 @@ function nested (laterCode, name, key, location, subKey, isArray) {
 
   switch (type) {
     case 'null':
+      funcName = '$asNull'
       code += `
         json += $asNull()
       `
       break
     case 'string': {
+      funcName = '$asString'
       const stringSerializer = getStringSerializer(schema.format)
       code += nullable ? `json += obj${accessor} === null ? null : ${stringSerializer}(obj${accessor})` : `json += ${stringSerializer}(obj${accessor})`
       break
     }
     case 'integer':
+      funcName = '$asInteger'
       code += nullable ? `json += obj${accessor} === null ? null : $asInteger(obj${accessor})` : `json += $asInteger(obj${accessor})`
       break
     case 'number':
+      funcName = '$asNumber'
       code += nullable ? `json += obj${accessor} === null ? null : $asNumber(obj${accessor})` : `json += $asNumber(obj${accessor})`
       break
     case 'boolean':
+      funcName = '$asBoolean'
       code += nullable ? `json += obj${accessor} === null ? null : $asBoolean(obj${accessor})` : `json += $asBoolean(obj${accessor})`
       break
     case 'object':
@@ -1206,6 +1232,7 @@ function nested (laterCode, name, key, location, subKey, isArray) {
       `
       break
     case undefined:
+      funcName = '$asNull'
       if ('anyOf' in schema) {
         // beware: dereferenceOfRefs has side effects and changes schema.anyOf
         const anyOfLocations = dereferenceOfRefs(location, 'anyOf')
@@ -1224,8 +1251,12 @@ function nested (laterCode, name, key, location, subKey, isArray) {
           // with the actual schema
           // 2. `nested`, through `buildCode`, replaces any reference in object properties with the actual schema
           // (see https://github.com/fastify/fast-json-stringify/blob/6da3b3e8ac24b1ca5578223adedb4083b7adf8db/index.js#L631)
+
+          const schemaKey = location.schema.$id || randomUUID()
+          ajvInstance.addSchema(location.schema, schemaKey)
+
           code += `
-            ${index === 0 ? 'if' : 'else if'}($validateWithAjv(${JSON.stringify(location.schema)}, ${testValue}))
+            ${index === 0 ? 'if' : 'else if'}(ajv.validate("${schemaKey}", ${testValue}))
               ${nestedResult.code}
           `
           laterCode = nestedResult.laterCode
@@ -1241,8 +1272,12 @@ function nested (laterCode, name, key, location, subKey, isArray) {
           const testSerializer = getTestSerializer(location.schema.format)
           const testValue = testSerializer !== undefined ? `${testSerializer}(obj${accessor}, true)` : `obj${accessor}`
           // see comment on anyOf about dereferencing the schema before calling ajv.validate
+
+          const schemaKey = location.schema.$id || randomUUID()
+          ajvInstance.addSchema(location.schema, schemaKey)
+
           code += `
-            ${index === 0 ? 'if' : 'else if'}($validateWithAjv(${JSON.stringify(location.schema)}, ${testValue}))
+            ${index === 0 ? 'if' : 'else if'}(ajv.validate("${schemaKey}", ${testValue}))
               ${nestedResult.code}
           `
           laterCode = nestedResult.laterCode
@@ -1336,7 +1371,8 @@ function nested (laterCode, name, key, location, subKey, isArray) {
 
   return {
     code,
-    laterCode
+    laterCode,
+    mapFnName: funcName
   }
 }
 
@@ -1352,17 +1388,10 @@ function isEmpty (schema) {
 
 module.exports = build
 
-module.exports.restore = function (debugModeStr, options = {}) {
-  const dependencies = [debugModeStr]
-  const args = []
-  if (debugModeStr.startsWith('ajv')) {
-    dependencies.unshift('ajv')
-    const ajvInstance = new Ajv(options.ajv)
-    ajvFormats(ajvInstance)
-    args.push(ajvInstance)
-  }
+module.exports.validLargeArrayMechanisms = validLargeArrayMechanisms
 
+module.exports.restore = function ({ code, ajv }) {
   // eslint-disable-next-line
-  return (Function.apply(null, ['ajv', debugModeStr])
-    .apply(null, args))
+  return (Function.apply(null, ['ajv', code])
+    .apply(null, [ajv]))
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fast-json-stringify",
-  "version": "3.0.1",
+  "version": "3.1.0",
   "description": "Stringify your JSON at max speed",
   "main": "index.js",
   "types": "index.d.ts",
@@ -30,6 +30,7 @@
   },
   "homepage": "https://github.com/fastify/fast-json-stringify#readme",
   "devDependencies": {
+    "@sinclair/typebox": "^0.23.3",
     "benchmark": "^2.1.4",
     "compile-json-stringify": "^0.1.2",
     "is-my-json-valid": "^2.20.0",
@@ -38,14 +39,15 @@
     "proxyquire": "^2.1.3",
     "semver": "^7.1.0",
     "standard": "^16.0.1",
-    "tap": "^15.0.0",
+    "tap": "^16.0.1",
     "typescript": "^4.0.2",
     "webpack": "^5.40.0"
   },
   "dependencies": {
-    "ajv": "^8.6.2",
+    "ajv": "^8.10.0",
     "ajv-formats": "^2.1.1",
     "deepmerge": "^4.2.2",
+    "fast-uri": "^1.0.1",
     "rfdc": "^1.2.0",
     "string-similarity": "^4.0.1"
   },

package/test/array.test.js

@@ -5,12 +5,12 @@ const test = require('tap').test
 const validator = require('is-my-json-valid')
 const build = require('..')
 
-function buildTest (schema, toStringify) {
+function buildTest (schema, toStringify, options) {
   test(`render a ${schema.title} as JSON`, (t) => {
     t.plan(3)
 
     const validate = validator(schema)
-    const stringify = build(schema)
+    const stringify = build(schema, options)
     const output = stringify(toStringify)
 
     t.same(JSON.parse(output), toStringify)
@@ -319,3 +319,47 @@ test('object array with anyOf and symbol', (t) => {
   ])
   t.equal(value, '[{"name":"name-0","option":"Foo"},{"name":"name-1","option":"Bar"}]')
 })
+
+const largeArray = new Array(2e4).fill({ a: 'test', b: 1 })
+buildTest({
+  title: 'large array with default mechanism',
+  type: 'object',
+  properties: {
+    ids: {
+      type: 'array',
+      items: {
+        type: 'object',
+        properties: {
+          a: { type: 'string' },
+          b: { type: 'number' }
+        }
+      }
+    }
+  }
+}, {
+  ids: largeArray
+}, {
+  largeArraySize: 2e4,
+  largeArrayMechanism: 'default'
+})
+
+buildTest({
+  title: 'large array with json-stringify mechanism',
+  type: 'object',
+  properties: {
+    ids: {
+      type: 'array',
+      items: {
+        type: 'object',
+        properties: {
+          a: { type: 'string' },
+          b: { type: 'number' }
+        }
+      }
+    }
+  }
+}, {
+  ids: largeArray
+}, {
+  largeArrayMechanism: 'json-stringify'
+})

package/test/debug-mode.test.js

@@ -2,6 +2,7 @@
 
 const test = require('tap').test
 const fjs = require('..')
+const Ajv = require('ajv').default
 
 function build (opts) {
   return fjs({
@@ -17,32 +18,40 @@ function build (opts) {
 }
 
 test('activate debug mode', t => {
-  t.plan(2)
+  t.plan(3)
   const debugMode = build({ debugMode: true })
-  t.type(debugMode, Array)
-  t.match(debugMode.toString.toString(), 'join', 'to string override')
+
+  t.type(debugMode, 'object')
+  t.ok(debugMode.ajv instanceof Ajv)
+  t.type(debugMode.code, 'string')
 })
 
 test('activate debug mode truthy', t => {
-  t.plan(2)
+  t.plan(3)
+
   const debugMode = build({ debugMode: 'yes' })
-  t.type(debugMode, Array)
-  t.match(debugMode.toString.toString(), 'join', 'to string override')
+
+  t.type(debugMode, 'object')
+  t.type(debugMode.code, 'string')
+  t.ok(debugMode.ajv instanceof Ajv)
 })
 
 test('to string auto-consistent', t => {
-  t.plan(2)
+  t.plan(4)
   const debugMode = build({ debugMode: 1 })
-  t.type(debugMode, Array)
 
-  const str = debugMode.toString()
-  const compiled = fjs.restore(str)
+  t.type(debugMode, 'object')
+  t.type(debugMode.code, 'string')
+  t.ok(debugMode.ajv instanceof Ajv)
+
+  const compiled = fjs.restore(debugMode)
   const tobe = JSON.stringify({ firstName: 'Foo' })
   t.same(compiled({ firstName: 'Foo', surname: 'bar' }), tobe, 'surname evicted')
 })
 
 test('to string auto-consistent with ajv', t => {
-  t.plan(2)
+  t.plan(4)
+
   const debugMode = fjs({
     title: 'object with multiple types field',
     type: 'object',
@@ -56,16 +65,19 @@ test('to string auto-consistent with ajv', t => {
       }
     }
   }, { debugMode: 1 })
-  t.type(debugMode, Array)
 
-  const str = debugMode.toString()
-  const compiled = fjs.restore(str)
+  t.type(debugMode, 'object')
+  t.type(debugMode.code, 'string')
+  t.ok(debugMode.ajv instanceof Ajv)
+
+  const compiled = fjs.restore(debugMode)
   const tobe = JSON.stringify({ str: 'Foo' })
   t.same(compiled({ str: 'Foo', void: 'me' }), tobe)
 })
 
 test('to string auto-consistent with ajv-formats', t => {
   t.plan(3)
+
   const debugMode = fjs({
     title: 'object with multiple types field and format keyword',
     type: 'object',
@@ -80,11 +92,10 @@ test('to string auto-consistent with ajv-formats', t => {
       }
     }
   }, { debugMode: 1 })
-  t.type(debugMode, Array)
 
-  const str = debugMode.toString()
+  t.type(debugMode, 'object')
 
-  const compiled = fjs.restore(str)
+  const compiled = fjs.restore(debugMode)
   const tobe = JSON.stringify({ str: 'foo@bar.com' })
   t.same(compiled({ str: 'foo@bar.com' }), tobe)
   t.same(compiled({ str: 'foo' }), JSON.stringify({ str: null }), 'invalid format is ignored')

package/test/typebox.test.js

@@ -0,0 +1,36 @@
+'use strict'
+
+const test = require('tap').test
+const build = require('..')
+
+test('nested object in pattern properties for typebox', (t) => {
+  const { Type } = require('@sinclair/typebox')
+
+  t.plan(1)
+
+  const nestedSchema = Type.Object({
+    nestedKey1: Type.String()
+  })
+
+  const RootSchema = Type.Object({
+    key1: Type.Record(Type.String(), nestedSchema),
+    key2: Type.Record(Type.String(), nestedSchema)
+  })
+
+  const schema = RootSchema
+  const stringify = build(schema)
+
+  const value = stringify({
+    key1: {
+      nestedKey: {
+        nestedKey1: 'value1'
+      }
+    },
+    key2: {
+      nestedKey: {
+        nestedKey1: 'value2'
+      }
+    }
+  })
+  t.equal(value, '{"key1":{"nestedKey":{"nestedKey1":"value1"}},"key2":{"nestedKey":{"nestedKey1":"value2"}}}')
+})