Change default maxSize to 1e7

Author: ehmicky

README.md

@@ -75,7 +75,7 @@ Object with the following properties.
 #### maxSize
 
 _Type_: `number`\
-_Default_: `Number.POSITIVE_INFINITY` (no maximum size)
+_Default_: `1e7`
 
 Big JSON strings can make a process, filesystem operation or network request
 crash. `maxSize` prevents it by setting a maximum

src/main.d.ts

@@ -10,7 +10,7 @@ export interface Options {
    * Additional properties beyond the size limit are omitted.
    * They are completely removed, not truncated (including strings).
    *
-   * @default Number.POSITIVE_INFINITY (no maximum size)
+   * @default 1e7
    *
    * @example
    * ```js

src/size.js

@@ -54,8 +54,10 @@ export const addSize = function ({
 
 // Skip checking for size when `maxSize` option equals this value
 const SKIP_MAX_SIZE = Number.POSITIVE_INFINITY
-// Default value for `maxSize` option
-export const DEFAULT_MAX_SIZE = SKIP_MAX_SIZE
+
+// Default value for `maxSize` option.
+// Chosen based on v8 max string length, which is ~5e8.
+export const DEFAULT_MAX_SIZE = 1e7
 
 const SIZED_TYPES = {
   value: {

test/size/large.js

@@ -1,5 +1,6 @@
 import test from 'ava'
 import safeJsonValue from 'safe-json-value'
+import { each } from 'test-each'
 
 const V8_MAX_STRING_LENGTH = 5e8
 const largeString = '\n'.repeat(V8_MAX_STRING_LENGTH)
@@ -32,3 +33,26 @@ test('Handles very large object properties', (t) => {
     ],
   })
 })
+
+test('Does not apply options.maxSize if infinite', (t) => {
+  t.deepEqual(
+    safeJsonValue(largeString, { maxSize: Number.POSITIVE_INFINITY }),
+    { value: largeString, changes: [] },
+  )
+})
+
+each([undefined, { maxSize: undefined }], ({ title }, options) => {
+  test(`Applies options.maxSize by default | ${title}`, (t) => {
+    t.deepEqual(safeJsonValue(largeString, options), {
+      value: undefined,
+      changes: [
+        {
+          path: [],
+          oldValue: largeString,
+          newValue: undefined,
+          reason: 'unsafeSize',
+        },
+      ],
+    })
+  })
+})

test/size/omit.js

@@ -2,15 +2,6 @@ import test from 'ava'
 import safeJsonValue from 'safe-json-value'
 import { each } from 'test-each'
 
-each(
-  [undefined, { maxSize: undefined }, { maxSize: Number.POSITIVE_INFINITY }],
-  ({ title }, options) => {
-    test(`Does not apply options.maxSize by default | ${title}`, (t) => {
-      t.deepEqual(safeJsonValue({}, options), { value: {}, changes: [] })
-    })
-  },
-)
-
 const symbol = Symbol('test')
 each(
   [