Use version of prettier from package.json

Author: pokey

.pre-commit-config.yaml

@@ -36,10 +36,16 @@ repos:
         # document contents. For example
         # packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/ruby/changeCondition.yml
         exclude: ^packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/.*/[^/]*\.yml$
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v2.7.1"
+  - repo: local
     hooks:
       - id: prettier
+        name: prettier
+        types: [text]
+        language: node
+        entry: prettier --write --list-different --ignore-unknown
+        additional_dependencies:
+          - [email protected]
+        args: []
   - repo: https://github.com/ikamensh/flynt/
     rev: "0.78"
     hooks:

package.json

@@ -11,7 +11,7 @@
     "watch": "tsc --build --watch",
     "init-vscode-sandbox": "pnpm --filter=@cursorless/cursorless-vscode-core init-launch-sandbox",
     "meta-updater:base": "pnpm --filter=@cursorless/meta-updater build && meta-updater",
-    "meta-updater": "pnpm run meta-updater:base && pnpm -r exec prettier --write tsconfig.json package.json",
+    "meta-updater": "pnpm run meta-updater:base && pnpm -r exec prettier --write tsconfig.json package.json && pnpm -w exec prettier --write .pre-commit-config.yaml",
     "lint:meta": "pnpm run meta-updater:base --test",
     "lint": "pnpm run lint:meta && syncpack list-mismatches && pnpm run lint:ts",
     "lint:ts": "eslint packages --ext ts",
@@ -40,7 +40,8 @@
   },
   "pnpm": {
     "patchedDependencies": {
-      "@docusaurus/[email protected]": "patches/@[email protected]"
+      "@docusaurus/[email protected]": "patches/@[email protected]",
+      "@pnpm/[email protected]": "patches/@[email protected]"
     },
     "peerDependencyRules": {
       "ignoreMissing": [

packages/meta-updater/package.json

@@ -15,7 +15,9 @@
     "@types/normalize-path": "^3.0.0",
     "normalize-path": "^3.0.0",
     "path-exists": "^4.0.0",
-    "type-fest": "3.6.1"
+    "ramda": "0.29.0",
+    "type-fest": "3.6.1",
+    "yaml": "2.2.1"
   },
   "main": "./out/index.js",
   "types": "./out/index.d.ts",
@@ -27,6 +29,7 @@
     }
   },
   "devDependencies": {
+    "@types/ramda": "0.28.23",
     "esbuild": "^0.17.11"
   }
 }

packages/meta-updater/src/formats.ts

@@ -0,0 +1,24 @@
+import { createFormat } from "@pnpm/meta-updater";
+import { readFile, writeFile } from "fs/promises";
+import { equals } from "ramda";
+import yaml from "yaml";
+
+export const formats = {
+  [".yaml"]: createFormat({
+    async read({ resolvedPath }) {
+      return yaml.parseDocument(await readFile(resolvedPath, "utf-8")).clone();
+    },
+    update(actual, updater, options) {
+      return updater(actual, options);
+    },
+    equal(expected, actual) {
+      return equals(actual.toJS(), expected.toJS());
+    },
+    async write(expected, { resolvedPath }) {
+      await writeFile(resolvedPath, expected.toString());
+    },
+    clone(content) {
+      return content == null ? content : content.clone();
+    },
+  }),
+};

packages/meta-updater/src/getPackageDeps.ts

@@ -0,0 +1,27 @@
+import normalizePath from "normalize-path";
+import path from "path";
+import { Lockfile } from "@pnpm/lockfile-file";
+
+export function getPackageDeps(
+  workspaceDir: string,
+  packageDir: string,
+  pnpmLockfile: Lockfile,
+) {
+  const pathFromRootToPackage =
+    packageDir === workspaceDir
+      ? "."
+      : normalizePath(path.relative(workspaceDir, packageDir));
+
+  /** Info about package dependencies gleaned from lock file. */
+  const lockFilePackageInfo = pnpmLockfile.importers[pathFromRootToPackage];
+  if (!lockFilePackageInfo) {
+    // Raise an error here because there should always be an entry in the lockfile.
+    throw new Error(`No importer found for ${pathFromRootToPackage}`);
+  }
+
+  const deps = {
+    ...lockFilePackageInfo.dependencies,
+    ...lockFilePackageInfo.devDependencies,
+  };
+  return deps;
+}

packages/meta-updater/src/metaUpdater.ts

@@ -7,6 +7,8 @@ import { createUpdateOptions } from "@pnpm/meta-updater";
 import { Context } from "./Context";
 import { updatePackageJson } from "./updatePackageJson";
 import { updateTSConfig } from "./updateTSConfig";
+import { updatePreCommit } from "./updatePreCommit";
+import { formats } from "./formats";
 
 export const updater = async (workspaceDir: string) => {
   const pnpmLockfile = await readWantedLockfile(workspaceDir, {
@@ -23,7 +25,11 @@ export const updater = async (workspaceDir: string) => {
   };
 
   return createUpdateOptions({
-    ["package.json"]: updatePackageJson.bind(null, context),
-    ["tsconfig.json"]: updateTSConfig.bind(null, context),
+    files: {
+      ["package.json"]: updatePackageJson.bind(null, context),
+      ["tsconfig.json"]: updateTSConfig.bind(null, context),
+      [".pre-commit-config.yaml"]: updatePreCommit.bind(null, context),
+    },
+    formats,
   });
 };

packages/meta-updater/src/updatePreCommit.ts

@@ -0,0 +1,67 @@
+import type { FormatPluginFnOptions } from "@pnpm/meta-updater";
+import { Document, ParsedNode } from "yaml";
+import { Context } from "./Context";
+import { getPackageDeps } from "./getPackageDeps";
+
+interface PreCommitConfig {
+  repos: {
+    hooks: {
+      id: string;
+      // eslint-disable-next-line @typescript-eslint/naming-convention
+      additional_dependencies: string[];
+    }[];
+  }[];
+}
+
+/**
+ * Given a tsconfig.json, update it to match our conventions.  This function is
+ * called by the pnpm `meta-updater` plugin either to check if the tsconfig.json
+ * is up to date or to update it, depending on flags.
+ * @param context Contains context such as workspace dir and parsed pnpm
+ * lockfile
+ * @param rawInput The input tsconfig.json that should be checked / updated
+ * @param options Extra information provided by pnpm; mostly just the directory
+ * of the package whose tsconfig.json we are updating
+ * @returns The updated tsconfig.json
+ */
+export async function updatePreCommit(
+  { workspaceDir, pnpmLockfile }: Context,
+  rawInput: Document<ParsedNode> | null,
+  options: FormatPluginFnOptions,
+): Promise<Document<ParsedNode> | null> {
+  if (rawInput == null) {
+    return null;
+  }
+  /** Directory of the package whose tsconfig.json we are updating */
+  const packageDir = options.dir;
+
+  if (packageDir !== workspaceDir) {
+    throw new Error("updatePreCommit should only be called on root");
+  }
+
+  const deps = getPackageDeps(workspaceDir, packageDir, pnpmLockfile);
+  const prettierVersion = deps["prettier"];
+
+  const prettierHookIndex = (rawInput.toJS() as PreCommitConfig).repos
+    .flatMap(({ hooks }, repoIndex) =>
+      hooks.map((hook, hookIndex) => ({ hook, repoIndex, hookIndex })),
+    )
+    .filter(({ hook }) => hook.id === "prettier");
+
+  if (prettierHookIndex.length === 0) {
+    throw new Error("No prettier hook found");
+  }
+
+  if (prettierHookIndex.length > 1) {
+    throw new Error("Multiple prettier hooks found");
+  }
+
+  const { repoIndex, hookIndex } = prettierHookIndex[0];
+
+  rawInput.setIn(
+    ["repos", repoIndex, "hooks", hookIndex, "additional_dependencies"],
+    rawInput.createNode([`prettier@${prettierVersion}`]),
+  );
+
+  return rawInput;
+}

packages/meta-updater/src/updateTSConfig.ts

@@ -5,6 +5,7 @@ import exists from "path-exists";
 import { TsConfigJson } from "type-fest";
 import { toPosixPath } from "./toPosixPath";
 import { Context } from "./Context";
+import { getPackageDeps } from "./getPackageDeps";
 
 /**
  * Given a tsconfig.json, update it to match our conventions.  This function is
@@ -41,24 +42,11 @@ export async function updateTSConfig(
     };
   }
 
-  const pathFromRootToPackage = normalizePath(
-    path.relative(workspaceDir, packageDir),
-  );
   const pathFromPackageToRoot = normalizePath(
     path.relative(packageDir, workspaceDir),
   );
 
-  /** Info about package dependencies gleaned from lock file. */
-  const lockFilePackageInfo = pnpmLockfile.importers[pathFromRootToPackage];
-  if (!lockFilePackageInfo) {
-    // Raise an error here because there should always be an entry in the lockfile.
-    throw new Error(`No importer found for ${pathFromRootToPackage}`);
-  }
-
-  const deps = {
-    ...lockFilePackageInfo.dependencies,
-    ...lockFilePackageInfo.devDependencies,
-  };
+  const deps = getPackageDeps(workspaceDir, packageDir, pnpmLockfile);
 
   /** Computed tsconfig.json references based on dependencies. */
   const references = [] as Array<{ path: string }>;

patches/@[email protected]

@@ -0,0 +1,27 @@
+diff --git a/lib/index.js b/lib/index.js
+index d24112f5fedba5a9d7c5bd85221652a8f0bf09e1..2af95628f80926b8c4c9c774d9135a95b465959c 100644
+--- a/lib/index.js
++++ b/lib/index.js
+@@ -48,7 +48,8 @@ export async function performUpdates(workspaceDir, updateParam, opts) {
+                 _writeProjectManifest: writeProjectManifest,
+             };
+             const actual = (await fileExists(resolvedPath)) ? await formatPlugin.read(formatHandlerOptions) : null;
+-            const expected = await formatPlugin.update(clone(actual), updateFile, formatHandlerOptions);
++            const customClone = formatPlugin.clone == null ? clone : formatPlugin.clone;
++            const expected = await formatPlugin.update(customClone(actual), updateFile, formatHandlerOptions);
+             const equal = (actual == null && expected == null) ||
+                 (actual != null && expected != null && (await formatPlugin.equal(expected, actual, formatHandlerOptions)));
+             if (equal) {
+diff --git a/lib/updater/formatPlugin.d.ts b/lib/updater/formatPlugin.d.ts
+index 887845721b09e17b95dd0789a5db8de72c04a654..2e47ba30b25661cec37d3ab1dcca1d8908a1eb21 100644
+--- a/lib/updater/formatPlugin.d.ts
++++ b/lib/updater/formatPlugin.d.ts
+@@ -12,6 +12,8 @@ export interface FormatPlugin<Content> {
+     equal(expected: Content, actual: Content, options: FormatPluginFnOptions): PromiseOrValue<boolean>;
+     /** Called only if write is required (`--test` isn't specified, `expected != null` and `expected` is not equal to `actual`) */
+     write(expected: Content, options: FormatPluginFnOptions): PromiseOrValue<void>;
++    /** Can be used to override the built-in clone functionality */
++    clone?(content: Content): PromiseOrValue<Content>;
+ }
+ export interface FormatPluginFnOptions {
+     file: string;