Merge pull request #17 from arkedge/renovate/prettier-3.x

Update dependency prettier to v3

package.json

@@ -45,7 +45,7 @@
     "eslint-plugin-prettier": "4.2.1",
     "eslint-plugin-promise": "6.1.1",
     "npm-run-all2": "5.0.2",
-    "prettier": "2.8.8",
+    "prettier": "3.2.5",
     "ts-node": "10.9.2",
     "typescript": "5.4.5",
     "vitest": "1.5.3"

src/check.ts

@@ -48,7 +48,7 @@ assert(input, "--input <PATH> or process.env.RULES_FILE is required");
 
 const printSemDiag = (
   filepath: string,
-  { range, msg, objPath, diagKind }: SemDiag
+  { range, msg, objPath, diagKind }: SemDiag,
 ) => {
   let semPos;
   if (objPath.length === 0) {
@@ -152,7 +152,7 @@ if (!docResult.is_ok) {
             docResult.docRaw.value,
             e.diag.objPath,
             key.value,
-            candidates[0]
+            candidates[0],
           );
           modified = true;
         }

src/config.test.ts

@@ -104,28 +104,28 @@ it("can match with readme rules", async () => {
   await expect(rules[2].test(obj)).resolves.toBeTruthy();
   expect(
     mock,
-    "acquire team member the first time we need"
+    "acquire team member the first time we need",
   ).toHaveBeenCalledTimes(1);
 
   await expect(
     getMatchedProj(
       rules,
       { kind: WebhookEventKind.Issue, action: "assigned" },
-      obj
-    )
+      obj,
+    ),
   ).resolves.toEqual([1, 4]);
 
   await expect(rules[2].test(obj2)).resolves.toBeFalsy();
   expect(
     mock,
-    "once called, it will not be called again"
+    "once called, it will not be called again",
   ).toHaveBeenCalledTimes(1);
   await expect(
     getMatchedProj(
       rules,
       { kind: WebhookEventKind.Issue, action: "assigned" },
-      obj2
-    )
+      obj2,
+    ),
   ).resolves.toEqual([]);
 
   await expect(rules[3].test(obj3)).resolves.toBeFalsy();
@@ -137,8 +137,8 @@ it("can match with readme rules", async () => {
     getMatchedProj(
       rules,
       { kind: WebhookEventKind.PullRequest, action: "assigned" },
-      obj4
-    )
+      obj4,
+    ),
   ).resolves.toEqual([9]);
 });
 
@@ -217,7 +217,7 @@ it("treat rules about issue and pr exclusively", () => {
         pr:
           - assigned
       project: 1`,
-    "issue rule on pr event"
+    "issue rule on pr event",
   );
 });
 
@@ -253,8 +253,8 @@ it("can limit firing event", async () => {
         kind: WebhookEventKind.Issue,
         action: "assigned",
       },
-      arg("assignee")
-    )
+      arg("assignee"),
+    ),
   ).resolves.toEqual([1]);
   await expect(
     getMatchedProj(
@@ -263,8 +263,8 @@ it("can limit firing event", async () => {
         kind: WebhookEventKind.Issue,
         action: "opened",
       },
-      arg("assignee")
-    )
+      arg("assignee"),
+    ),
   ).resolves.toEqual([1, 2]);
   await expect(
     getMatchedProj(
@@ -273,8 +273,8 @@ it("can limit firing event", async () => {
         kind: WebhookEventKind.PullRequest,
         action: "opened",
       },
-      arg("assignee")
-    )
+      arg("assignee"),
+    ),
   ).resolves.toEqual([2]);
   await expect(
     getMatchedProj(
@@ -283,7 +283,7 @@ it("can limit firing event", async () => {
         kind: WebhookEventKind.PullRequest,
         action: "assigned",
       },
-      arg("assignee")
-    )
+      arg("assignee"),
+    ),
   ).resolves.toEqual([]);
 });

src/config.ts

@@ -94,7 +94,7 @@ interface Rule {
 export async function getMatchedProj(
   rules: Rule[],
   webhookEvent: WebhookEventAction,
-  obj: MatchArg
+  obj: MatchArg,
 ): Promise<number[]> {
   const ruleMatchedProjectNumbers = await Promise.all(
     rules.map(async (rule) => {
@@ -104,7 +104,7 @@ export async function getMatchedProj(
         }
       }
       return [];
-    })
+    }),
   );
   const targetProjs: TargetProj[] = ruleMatchedProjectNumbers.flat();
   return filterTargetProjs(targetProjs);
@@ -114,7 +114,7 @@ function filterTargetProjs(targetProjs: TargetProj[]): number[] {
   // lift `priority` from `TargetProjNumber`
   const targetProjsFlat = targetProjs.flatMap(
     (
-      p: TargetProj
+      p: TargetProj,
     ): Array<
       | ({ kind: TargetProjKind.Number } & TargetProjNumber)
       | TargetProjOnly
@@ -127,7 +127,7 @@ function filterTargetProjs(targetProjs: TargetProj[]): number[] {
         }));
       }
       return [p];
-    }
+    },
   );
   // dictionary order (priority >> kind)
   targetProjsFlat.sort((a, b) => {
@@ -172,13 +172,13 @@ function filterTargetProjs(targetProjs: TargetProj[]): number[] {
 export function processRulesV0(
   format: EitherFormat,
   filepath: string,
-  src: string
+  src: string,
 ): ParseResult<Config> {
   const { docResult, content, error } = parseFile(
     format,
     configSchema,
     filepath,
-    src
+    src,
   );
   return {
     docResult,
@@ -229,7 +229,7 @@ function parseFile<Schema extends z.ZodType<any, any, any>>(
   format: EitherFormat,
   schema: Schema,
   filepath: string,
-  yaml: string
+  yaml: string,
 ): ParseResult<Schema["_output"]> {
   const result = format.parse(yaml);
   if (!result.is_ok) {
@@ -336,13 +336,13 @@ function getWebhookEventTarget(parsed: RuleSchema): WebhookEventActionTarget {
 export function processRules(
   format: EitherFormat,
   filepath: string,
-  src: string
+  src: string,
 ): ParseResult<Rule[]> {
   const { docResult, content, error } = parseFile(
     format,
     versionedSchema,
     filepath,
-    src
+    src,
   );
   return {
     docResult,
@@ -426,7 +426,7 @@ const extractSemDiag = (doc: Document, error: z.ZodError) => {
     if (
       (issue.code === "invalid_union" &&
         issue.unionErrors.every((e) =>
-          e.errors.every((e) => e.message === "Required")
+          e.errors.every((e) => e.message === "Required"),
         )) ||
       issue.message === "Required"
     ) {

src/configFormat/jsonFormat.ts

@@ -10,7 +10,7 @@ import {
 import { type ObjPath } from "../types";
 
 const parserGrammar = nearley.Grammar.fromCompiled(
-  grammar as nearley.CompiledRules
+  grammar as nearley.CompiledRules,
 );
 
 export type JsonTagged<T> = {
@@ -215,7 +215,7 @@ function findDelegator(node: Value | null) {
   return (keyName: string): DocumentNode | null => {
     if (node?.type === "Object") {
       const key = node?.children.find(
-        (pair) => pair.key.value.toString() === keyName
+        (pair) => pair.key.value.toString() === keyName,
       )?.key;
       if (typeof key === "undefined") {
         return null;

src/configFormat/yamlFormat.ts

@@ -24,7 +24,7 @@ type ParseResultWrapped<T, U> = ParseResult<YamlTagged<T>, YamlTagged<U>>;
 
 export class YamlFormat implements Format {
   parse(
-    s: string
+    s: string,
   ): ParseResultWrapped<YamlDocument, YAML.Document.Parsed<YAML.ParsedNode>> {
     const lineCounter = new LineCounter();
     const doc = YAML.parseDocument(s, { lineCounter, prettyErrors: true });
@@ -45,7 +45,7 @@ export class YamlFormat implements Format {
 export class YamlDocument implements Document {
   constructor(
     private readonly doc: YAML.Document.Parsed,
-    private readonly lineCounter: LineCounter
+    private readonly lineCounter: LineCounter,
   ) {}
 
   toJS() {
@@ -77,7 +77,7 @@ export class YamlDocument implements Document {
 
 function toDocumentNode(
   node: YAML.ParsedNode,
-  lineCounter: LineCounter
+  lineCounter: LineCounter,
 ): DocumentNode & FindKey {
   return {
     loc: {
@@ -92,7 +92,7 @@ function findDelegator(node: YAML.ParsedNode | null, lineCounter: LineCounter) {
   return (keyName: string) => {
     if (YAML.isMap(node)) {
       const key = node?.items.find(
-        (pair) => pair.key.toString() === keyName
+        (pair) => pair.key.toString() === keyName,
       )?.key;
       if (typeof key === "undefined") {
         return null;

src/configSchema.ts

@@ -61,7 +61,7 @@ const composeEAsync =
     const y = flatten(f(x));
     return await existsAsync(
       g,
-      async (g) => await existsAsync(y, async (y) => await g(y))
+      async (g) => await existsAsync(y, async (y) => await g(y)),
     );
   };
 
@@ -78,7 +78,7 @@ const composeAAsync =
     const y = flatten(f(x));
     return await forallAsync(
       g,
-      async (g) => await existsAsync(y, async (y) => await g(y))
+      async (g) => await existsAsync(y, async (y) => await g(y)),
     );
   };
 
@@ -147,7 +147,7 @@ const numberParser = z.union([_number.transform(unit), _number.array()], {
             (i) =>
               // i.code === z.ZodIssueCode.invalid_type is required because of type inference
               i.code === z.ZodIssueCode.invalid_type &&
-              i.received === z.ZodParsedType.array
+              i.received === z.ZodParsedType.array,
           )
         ) {
           // error is array of something
@@ -239,7 +239,7 @@ const loginEq =
       return async (x: X & GetTeamMemberProp) => {
         const y = await p(x);
         return proj(x).some((g) =>
-          y.some((y) => curryingStrictEq(g.login)(y.login))
+          y.some((y) => curryingStrictEq(g.login)(y.login)),
         );
       };
     }
@@ -258,7 +258,7 @@ const labelParser = stringish
 
 const issueSchema = z.object({
   assignees: loginParser(
-    (x: Issue & GetTeamMemberProp) => x.assignees
+    (x: Issue & GetTeamMemberProp) => x.assignees,
   ).transform(extendEAsync((x: Issue & GetTeamMemberProp) => x)),
   labels: labelParser
     .transform(extendPropE<Issue>()("labels"))
@@ -274,10 +274,10 @@ const prSchema = z.object({
       } else {
         return [];
       }
-    })
+    }),
   ).transform(extendEAsync((x: PullRequest & GetTeamMemberProp) => x)),
   assignees: loginParser(
-    (x: PullRequest & GetTeamMemberProp) => x.assignees
+    (x: PullRequest & GetTeamMemberProp) => x.assignees,
   ).transform(extendEAsync((x: PullRequest & GetTeamMemberProp) => x)),
   labels: labelParser
     .transform(extendPropE<PullRequest>()("labels"))
@@ -363,7 +363,7 @@ const targetProjNumberSchema = _number
           return projectNumber;
         }),
       })
-      .transform((x) => x.not)
+      .transform((x) => x.not),
   );
 
 const targetProjSchema = targetProjNumberSchema
@@ -387,7 +387,7 @@ const targetProjSchema = targetProjNumberSchema
           return r;
         }),
       })
-      .transform((x) => x.only)
+      .transform((x) => x.only),
   )
   .or(
     z
@@ -400,7 +400,7 @@ const targetProjSchema = targetProjNumberSchema
           return r;
         }),
       })
-      .transform((x) => x.reject)
+      .transform((x) => x.reject),
   );
 
 const ruleSchema = payloadSchema
@@ -411,7 +411,7 @@ const ruleSchema = payloadSchema
   .strict()
   .refine(
     (p) => typeof p.issue === "undefined" || typeof p.pr === "undefined",
-    "issue and pr cannot be specified together"
+    "issue and pr cannot be specified together",
   )
   .refine((p) => {
     // issue rule