fix: allow additional top level fields (#460)

Uniswap · Aug 2, 2023 · 8e9f553 · antazoey · Sep 21, 2023 · 8e9f553
1 parent a273873
commit 8e9f553
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/src/tokenlist.schema.json b/src/tokenlist.schema.json
@@ -296,7 +296,6 @@
     }
   },
   "type": "object",
-  "additionalProperties": false,
   "properties": {
     "name": {
       "type": "string",
@@ -401,4 +400,4 @@
     "version",
     "tokens"
   ]
-}
+}
diff --git a/test/__snapshots__/tokenlist.schema.test.ts.snap b/test/__snapshots__/tokenlist.schema.test.ts.snap
@@ -1,5 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`schema allows additional top-level fields 1`] = `null`;
+
 exports[`schema allows up to 10k tokens 1`] = `null`;
 
 exports[`schema checks extensions 1`] = `null`;

diff --git a/test/tokenlist.schema.test.ts b/test/tokenlist.schema.test.ts
@@ -136,4 +136,12 @@ describe('schema', () => {
     };
     checkSchema(exampleListWith10kTokensPlusOne, false);
   });
+
+  it('allows additional top-level fields', () => {
+    const exampleListWithUnknownField = {
+      ...exampleList,
+      unknownField: 'foo',
+    };
+    checkSchema(exampleListWithUnknownField, true);
+  });
 });