🏗 Remove freeze optiion from babel-plugin-transform-json-import (#3…

…6691)

build-system/babel-config/minified-config.js

@@ -40,10 +40,7 @@ function getMinifiedConfig() {
       './build-system/babel-plugins/babel-plugin-transform-log-methods',
       {replaceCallArguments: false},
     ],
-    [
-      './build-system/babel-plugins/babel-plugin-transform-json-import',
-      {freeze: false},
-    ],
+    './build-system/babel-plugins/babel-plugin-transform-json-import',
     './build-system/babel-plugins/babel-plugin-transform-amp-extension-call',
     './build-system/babel-plugins/babel-plugin-transform-html-template',
     './build-system/babel-plugins/babel-plugin-transform-jss',

build-system/babel-config/pre-closure-config.js

@@ -41,10 +41,7 @@ function getPreClosureConfig() {
       {replaceCallArguments: false},
     ],
     './build-system/babel-plugins/babel-plugin-transform-parenthesize-expression',
-    [
-      './build-system/babel-plugins/babel-plugin-transform-json-import',
-      {freeze: false},
-    ],
+    './build-system/babel-plugins/babel-plugin-transform-json-import',
     './build-system/babel-plugins/babel-plugin-transform-amp-extension-call',
     './build-system/babel-plugins/babel-plugin-transform-html-template',
     './build-system/babel-plugins/babel-plugin-transform-jss',

build-system/babel-plugins/babel-plugin-transform-json-import/index.js

@@ -15,12 +15,10 @@ const {readFileSync} = require('fs');
  * ```
  *
  * @param {babel} babel
- * @param {any} options
  * @return {babel.PluginObj}
  */
-module.exports = function (babel, options) {
+module.exports = function (babel) {
   const {template, types: t} = babel;
-  const {freeze = true} = options;
 
   /**
    * JSON reviver that converts {"string": "foo", ...} to just "foo".
@@ -99,19 +97,6 @@ module.exports = function (babel, options) {
           );
         }
 
-        if (freeze) {
-          path.replaceWith(
-            template.statement
-              .ast`const ${specifier} = JSON.parse('${JSON.stringify(
-              json
-            )}', function(key, val) {
-                if (typeof val === 'object') Object.freeze(val);
-                return val;
-              });`
-          );
-          return;
-        }
-
         path.replaceWith(
           template.statement
             .ast`const ${specifier} = JSON.parse('${JSON.stringify(json)}');`

build-system/babel-plugins/babel-plugin-transform-json-import/test/fixtures/transform-assertions/transform-minify-locales/output.mjs

@@ -1,10 +1,4 @@
 // options.js is left as is
 // lang.json is minified because the path includes /_locales/
-const options = JSON.parse("{\"plugins\":[\"../../../..\"],\"sourceType\":\"module\"}", function (key, val) {
-  if (typeof val === 'object') Object.freeze(val);
-  return val;
-});
-const lang = JSON.parse("{\"2\":\"foo\",\"15\":\"bar\"}", function (key, val) {
-  if (typeof val === 'object') Object.freeze(val);
-  return val;
-});
+const options = JSON.parse("{\"plugins\":[\"../../../..\"],\"sourceType\":\"module\"}");
+const lang = JSON.parse("{\"2\":\"foo\",\"15\":\"bar\"}");

build-system/babel-plugins/babel-plugin-transform-json-import/test/fixtures/transform-assertions/transform/output.mjs

@@ -1,8 +1,2 @@
-const key = JSON.parse("{\"plugins\":[\"../../../..\"],\"sourceType\":\"module\"}", function (key, val) {
-  if (typeof val === 'object') Object.freeze(val);
-  return val;
-});
-const string = JSON.parse("{\"plugins\":[\"../../../..\"],\"sourceType\":\"module\"}", function (key, val) {
-  if (typeof val === 'object') Object.freeze(val);
-  return val;
-});
+const key = JSON.parse("{\"plugins\":[\"../../../..\"],\"sourceType\":\"module\"}");
+const string = JSON.parse("{\"plugins\":[\"../../../..\"],\"sourceType\":\"module\"}");