Merge pull request #2297 from o1-labs/dw/stop-running-prettier-js-layout

Delegate call to prettier from js-layout to the user

Author: dannywillems

.prettierignore

@@ -4,7 +4,6 @@ benchmark/
 dist/
 node_modules/
 o1js-reference/
-src/bindings/mina-transaction/gen/
 src/bindings/compiled/
 src/mina/
 **/*.d.ts

src/bindings/mina-transaction/gen/v1/dune

@@ -9,9 +9,6 @@
   (<> %{env:NO_JS_BUILD=0} 1))
  (deps
   (:script ../../../../build/js-layout-to-types.mjs)
-  ../../../ocaml/jsLayout.json
-  ../../../../../.prettierrc.cjs
-  ../../../../../node_modules/prettier/standalone.js
-  (glob_files ../../../../../node_modules/prettier/plugins/*))
+  ../../../ocaml/jsLayout.json)
  (action
   (run node %{script})))

src/bindings/mina-transaction/gen/v2/dune

@@ -6,9 +6,6 @@
   (<> %{env:NO_JS_BUILD=0} 1))
  (deps
   (:script ../../../../build/js-layout-to-types-v2.mjs)
-  (:data ../../../ocaml/jsLayout.json)
-  ../../../../../.prettierrc.cjs
-  ../../../../../node_modules/prettier/standalone.js
-  (glob_files ../../../../../node_modules/prettier/plugins/*))
+  (:data ../../../ocaml/jsLayout.json))
  (action
   (run node %{script} %{data} js-layout.ts)))

src/build/js-layout-to-types-v2.mjs

@@ -2,10 +2,6 @@
 
 import assert from 'node:assert';
 import fs from 'node:fs/promises';
-import prettier from 'prettier/standalone.js';
-import prettierRc from '../../.prettierrc.cjs';
-import parserTypeScript from 'prettier/plugins/typescript.js';
-import prettierPluginEstree from 'prettier/plugins/estree.js';
 
 const leafTypes = {
   number: 'number',
@@ -343,14 +339,8 @@ for (const typeName of depOrderTypeNames) {
 out += `\
   \ const Types: {[key: string]: BindingsType<any>} = {${Object.keys(jsTypes).join(', ')}};\n`;
 
-const prettyOut = await prettier.format(out, {
-  parser: 'typescript',
-  plugins: [parserTypeScript, prettierPluginEstree],
-  ...prettierRc,
-});
-
 if (outputFilepath !== null) {
-  await fs.writeFile(outputFilepath, prettyOut);
+  await fs.writeFile(outputFilepath, out);
 } else {
-  console.log(prettyOut);
+  console.log(out);
 }

src/build/js-layout-to-types.mjs

@@ -1,11 +1,6 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
-import prettier from 'prettier/standalone.js';
-import parserTypeScript from 'prettier/plugins/typescript.js';
-import prettierPluginEstree from 'prettier/plugins/estree.js';
-
-import prettierRc from '../../.prettierrc.cjs';
 
 // let jsLayout = JSON.parse(process.argv[2]);
 let selfPath = fileURLToPath(import.meta.url);
@@ -219,11 +214,6 @@ ${output}`;
 
 async function writeTsFile(content, relPath) {
   let absPath = path.resolve(selfPath, relPath);
-  content = await prettier.format(content, {
-    filepath: absPath,
-    plugins: [parserTypeScript, prettierPluginEstree],
-    ...prettierRc,
-  });
   await fs.writeFile(absPath, content);
 }
 let genPath = '../../bindings/mina-transaction/gen/v1';