fast-json-stringify - npm Package Compare versions

Comparing version 5.15.1 to 5.16.0

lib/standalone.js

@@ -6,2 +6,3 @@ 'use strict'
   if (context.validatorSchemasIds.size > 0) {
+    ajvDependencyCode += 'const Validator = require(\'fast-json-stringify/lib/validator\')\n'
     ajvDependencyCode += `const validatorState = ${JSON.stringify(validator.getState())}\n`
@@ -13,9 +14,11 @@ ajvDependencyCode += 'const validator = Validator.restoreFromState(validatorState)\n'
 
+  // Don't need to keep external schemas once compiled
+  // validatorState will hold external schemas if it needs them
+  const { schema, ...serializerState } = serializer.getState()
+
   return `
   'use strict'
-  const { dependencies } = require('fast-json-stringify/lib/standalone')
 
-  const { Serializer, Validator } = dependencies
-
-  const serializerState = ${JSON.stringify(serializer.getState())}
+  const Serializer = require('fast-json-stringify/lib/serializer')
+  const serializerState = ${JSON.stringify(serializerState)}
   const serializer = Serializer.restoreFromState(serializerState)
@@ -22,0 +25,0 @@

package.json

 {
   "name": "fast-json-stringify",
-  "version": "5.15.1",
+  "version": "5.16.0",
   "description": "Stringify your JSON at max speed",
@@ -5,0 +5,0 @@ "main": "index.js",

test/standalone-mode.test.js

@@ -29,10 +29,10 @@ 'use strict'
 
-  const destionation = path.resolve(tmpDir, 'standalone.js')
+  const destination = path.resolve(tmpDir, 'standalone.js')
 
   t.teardown(async () => {
-    await fs.promises.rm(destionation, { force: true })
+    await fs.promises.rm(destination, { force: true })
   })
 
-  await fs.promises.writeFile(destionation, code)
-  const standalone = require(destionation)
+  await fs.promises.writeFile(destination, code)
+  const standalone = require(destination)
   t.same(standalone({ firstName: 'Foo', surname: 'bar' }), JSON.stringify({ firstName: 'Foo' }), 'surname evicted')
@@ -93,10 +93,10 @@ })
 
-  const destionation = path.resolve(tmpDir, 'standalone2.js')
+  const destination = path.resolve(tmpDir, 'standalone2.js')
 
   t.teardown(async () => {
-    await fs.promises.rm(destionation, { force: true })
+    await fs.promises.rm(destination, { force: true })
   })
 
-  await fs.promises.writeFile(destionation, code)
-  const standalone = require(destionation)
+  await fs.promises.writeFile(destination, code)
+  const standalone = require(destination)
   t.same(standalone({
@@ -124,1 +124,96 @@ kind: 'foobar',
 })
+
+test('no need to keep external schemas once compiled', async (t) => {
+  t.plan(1)
+  const externalSchema = {
+    first: {
+      definitions: {
+        id1: {
+          type: 'object',
+          properties: {
+            id1: {
+              type: 'integer'
+            }
+          }
+        }
+      }
+    }
+  }
+  const code = fjs({
+    $ref: 'first#/definitions/id1'
+  }, {
+    mode: 'standalone',
+    schema: externalSchema
+  })
+
+  const destination = path.resolve(tmpDir, 'standalone3.js')
+
+  t.teardown(async () => {
+    await fs.promises.rm(destination, { force: true })
+  })
+
+  await fs.promises.writeFile(destination, code)
+  const standalone = require(destination)
+
+  t.same(standalone({ id1: 5 }), JSON.stringify({ id1: 5 }), 'serialization works with external schemas')
+})
+
+test('no need to keep external schemas once compiled - with oneOf validator', async (t) => {
+  t.plan(2)
+
+  const externalSchema = {
+    ext: {
+      definitions: {
+        oBaz: {
+          type: 'object',
+          properties: {
+            baz: { type: 'number' }
+          },
+          required: ['baz']
+        },
+        oBar: {
+          type: 'object',
+          properties: {
+            bar: { type: 'string' }
+          },
+          required: ['bar']
+        },
+        other: {
+          type: 'string',
+          const: 'other'
+        }
+      }
+    }
+  }
+
+  const schema = {
+    title: 'object with oneOf property value containing refs to external schema',
+    type: 'object',
+    properties: {
+      oneOfSchema: {
+        oneOf: [
+          { $ref: 'ext#/definitions/oBaz' },
+          { $ref: 'ext#/definitions/oBar' }
+        ]
+      }
+    },
+    required: ['oneOfSchema']
+  }
+
+  const code = fjs(schema, {
+    mode: 'standalone',
+    schema: externalSchema
+  })
+
+  const destination = path.resolve(tmpDir, 'standalone-oneOf-ref.js')
+
+  t.teardown(async () => {
+    await fs.promises.rm(destination, { force: true })
+  })
+
+  await fs.promises.writeFile(destination, code)
+  const stringify = require(destination)
+
+  t.equal(stringify({ oneOfSchema: { baz: 5 } }), '{"oneOfSchema":{"baz":5}}')
+  t.equal(stringify({ oneOfSchema: { bar: 'foo' } }), '{"oneOfSchema":{"bar":"foo"}}')
+})

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

368278

increased by0.65%

Lines of code

13823

increased by0.62%

Worsened metrics

Number of low supply chain risk alerts

11

increased by22.22%

No dependency changes