Add ignore glob in config

package.json

@@ -23,6 +23,7 @@
   "devDependencies": {
     "@cloudflare/workers-types": "^4.20240208.0",
     "@octokit/webhooks-types": "^7.3.2",
+    "@types/picomatch": "^2.3.3",
     "eslint": "^8.56.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-unicorn": "^51.0.1",
@@ -40,6 +41,7 @@
     "js-toml": "^1.0.0",
     "octokit": "^3.1.2",
     "parse-git-diff": "^0.0.15",
+    "picomatch": "^4.0.1",
     "zod": "^3.22.4"
   }
 }

src/config.ts

@@ -4,6 +4,9 @@ import { load } from "js-toml";
 import type { Context } from "./context";
 
 const ConfigSchema = z.object({
+  /**
+   * List of labels for pr size.
+   */
   labels: z
     .array(
       z.object({
@@ -26,6 +29,11 @@ const ConfigSchema = z.object({
       clone.sort((a, b) => b.threshold - a.threshold);
       return clone;
     }),
+  /**
+   * Array of glob pattern to match against the file. If match, the file will
+   * get ignored in pr size calculation
+   */
+  ignore: z.array(z.string()).default(() => []),
 });
 
 export type ConfigType = z.infer<typeof ConfigSchema>;
@@ -40,6 +48,7 @@ const DEFAULT_CONFIG: ConfigType = {
     { name: "size/small", color: "008000", threshold: 100 },
     { name: "size/x-small", color: "008000", threshold: 0 },
   ],
+  ignore: ["package-lock.json", ".yarn/*", "yarn.lock", "pnpm-lock.yaml"],
 };
 
 export async function readConfig(ctx: Context): Promise<ConfigType> {

src/handler.ts

@@ -1,5 +1,6 @@
 import parseGitDiff, { type AnyChunk } from "parse-git-diff";
 import { ConfigType } from "./config";
+import picomatch from "picomatch";
 
 export interface HandlerResponse {
   label: string | null;
@@ -40,10 +41,15 @@ export class Handler {
     const summaryMap = this.genDiffSummary(diffFile);
     const summary: Summary = { add: 0, del: 0 };
 
-    // TODO: Ignore files base on config
-    for (const s of summaryMap.values()) {
-      summary.add += s.add;
-      summary.del += s.del;
+    const matcher = picomatch(this.config.ignore);
+
+    for (const [filepath, sum] of summaryMap) {
+      if (matcher(filepath)) {
+        continue;
+      }
+
+      summary.add += sum.add;
+      summary.del += sum.del;
     }
 
     const size = this.getSize(summary);