fast-json-stringify 5.9.0 → 5.9.2

package/README.md

@@ -692,7 +692,7 @@ The standalone mode is used to compile the code that can be directly run by `nod
 itself. You need to have `fast-json-stringify` installed for the standalone code to work.
 
 ```js
-const fs = require('node:fs')
+const fs = require('fs')
 const code = fastJson({
   title: 'default string',
   type: 'object',

package/benchmark/bench-cmp-branch.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { spawn } = require('node:child_process')
+const { spawn } = require('child_process')
 
 const cliSelect = require('cli-select')
 const simpleGit = require('simple-git')

package/benchmark/bench-thread.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { workerData: benchmark, parentPort } = require('node:worker_threads')
+const { workerData: benchmark, parentPort } = require('worker_threads')
 
 const Benchmark = require('benchmark')
 Benchmark.options.minSamples = 100

package/benchmark/bench.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const path = require('node:path')
-const { Worker } = require('node:worker_threads')
+const path = require('path')
+const { Worker } = require('worker_threads')
 
 const BENCH_THREAD_PATH = path.join(__dirname, 'bench-thread.js')
 

package/build/build-schema-validator.js

@@ -3,8 +3,8 @@
 const Ajv = require('ajv')
 const standaloneCode = require('ajv/dist/standalone').default
 const ajvFormats = require('ajv-formats')
-const fs = require('node:fs')
-const path = require('node:path')
+const fs = require('fs')
+const path = require('path')
 
 const ajv = new Ajv({
   addUsedSchema: false,

package/examples/server.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const http = require('node:http')
+const http = require('http')
 
 const stringify = require('fast-json-stringify')({
   type: 'object',

package/index.js

@@ -4,7 +4,6 @@
 
 const merge = require('@fastify/deepmerge')()
 const clone = require('rfdc')({ proto: true })
-const { randomUUID } = require('node:crypto')
 const { RefResolver } = require('json-schema-ref-resolver')
 
 const validate = require('./lib/schema-validator')
@@ -31,6 +30,8 @@ const validLargeArrayMechanisms = [
 
 const addComma = '!addComma && (addComma = true) || (json += \',\')'
 
+let schemaIdCounter = 0
+
 function isValidSchema (schema, name) {
   if (!validate(schema)) {
     if (name) {
@@ -86,7 +87,7 @@ function build (schema, options) {
     options,
     wrapObjects: true,
     refResolver: new RefResolver(),
-    rootSchemaId: schema.$id || randomUUID(),
+    rootSchemaId: schema.$id || `__fjs_root_${schemaIdCounter++}`,
     validatorSchemasIds: new Set()
   }
 
@@ -500,7 +501,7 @@ function mergeAllOfSchema (context, location, schema, mergedSchema) {
   }
   delete mergedSchema.allOf
 
-  mergedSchema.$id = `merged_${randomUUID()}`
+  mergedSchema.$id = `__fjs_merged_${schemaIdCounter++}`
   context.refResolver.addSchema(mergedSchema)
   location.addMergedSchema(mergedSchema, mergedSchema.$id)
 }
@@ -680,7 +681,15 @@ function buildArrayTypeCondition (type, accessor) {
       condition = `obj${accessor} === null`
       break
     case 'string':
-      condition = `typeof obj${accessor} === 'string'`
+      condition = `typeof obj${accessor} === 'string' ||
+      obj${accessor} === null ||
+      obj${accessor} instanceof Date ||
+      obj${accessor} instanceof RegExp ||
+      (
+        typeof obj${accessor} === "object" &&
+        typeof obj${accessor}.toString === "function" &&
+        obj${accessor}.toString !== Object.prototype.toString
+      )`
       break
     case 'integer':
       condition = `Number.isInteger(obj${accessor})`
@@ -740,8 +749,7 @@ function buildMultiTypeSerializer (context, location, input) {
             (
               typeof ${input} === "object" &&
               typeof ${input}.toString === "function" &&
-              ${input}.toString !== Object.prototype.toString &&
-              !(${input} instanceof Date)
+              ${input}.toString !== Object.prototype.toString
             )
           )
             ${nestedResult}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fast-json-stringify",
-  "version": "5.9.0",
+  "version": "5.9.2",
   "description": "Stringify your JSON at max speed",
   "main": "index.js",
   "type": "commonjs",
@@ -16,7 +16,10 @@
     "test:unit": "tap",
     "test": "npm run test:unit && npm run test:typescript"
   },
-  "precommit": ["lint", "test"],
+  "precommit": [
+    "lint",
+    "test"
+  ],
   "repository": {
     "type": "git",
     "url": "git+https://github.com/fastify/fast-json-stringify.git"
@@ -35,7 +38,7 @@
   "homepage": "https://github.com/fastify/fast-json-stringify#readme",
   "devDependencies": {
     "@fastify/pre-commit": "^2.0.2",
-    "@sinclair/typebox": "^0.31.1",
+    "@sinclair/typebox": "^0.32.3",
     "benchmark": "^2.1.4",
     "cli-select": "^1.1.2",
     "compile-json-stringify": "^0.1.2",

package/test/array.test.js

@@ -31,12 +31,36 @@ function buildTest (schema, toStringify, options) {
     const stringify = build(schema, options)
     const output = stringify(toStringify)
 
-    t.same(JSON.parse(output), toStringify)
+    t.same(JSON.parse(output), JSON.parse(JSON.stringify(toStringify)))
     t.equal(output, JSON.stringify(toStringify))
     t.ok(validate(JSON.parse(output)), 'valid schema')
   })
 }
 
+buildTest({
+  title: 'dates tuple',
+  type: 'object',
+  properties: {
+    dates: {
+      type: 'array',
+      minItems: 2,
+      maxItems: 2,
+      items: [
+        {
+          type: 'string',
+          format: 'date-time'
+        },
+        {
+          type: 'string',
+          format: 'date-time'
+        }
+      ]
+    }
+  }
+}, {
+  dates: [new Date(1), new Date(2)]
+})
+
 buildTest({
   title: 'string array',
   type: 'object',

package/test/standalone-mode.test.js

@@ -2,8 +2,8 @@
 
 const test = require('tap').test
 const fjs = require('..')
-const fs = require('node:fs')
-const path = require('node:path')
+const fs = require('fs')
+const path = require('path')
 
 function build (opts, schema) {
   return fjs(schema || {

package/test/webpack.test.js

@@ -2,7 +2,7 @@
 
 const test = require('tap').test
 const webpack = require('webpack')
-const path = require('node:path')
+const path = require('path')
 
 test('the library should work with webpack', async (t) => {
   t.plan(1)