fix: configuration load

package.json

@@ -22,7 +22,8 @@
     "worker": "wrangler dev --env dev --port 8787",
     "proxy": "tsx src/proxy.ts",
     "knip": "knip --config .github/knip.ts",
-    "knip-ci": "knip --no-exit-code --reporter json --config .github/knip.ts"
+    "knip-ci": "knip --no-exit-code --reporter json --config .github/knip.ts",
+		"test": "bun test"
   },
   "keywords": [
     "typescript",

src/github/utils/config.ts

@@ -8,7 +8,7 @@ import { BotConfig, generateConfiguration } from "@ubiquibot/configuration";
 
 const UBIQUIBOT_CONFIG_FULL_PATH = ".github/.ubiquibot-config.yml";
 
-export async function getConfig(context: GitHubContext): Promise<BotConfig | null> {
+export async function getConfig(context: GitHubContext): Promise<BotConfig> {
   const payload = context.payload;
   const defaultConfiguration = generateConfiguration();
   if (!("repository" in payload) || !payload.repository) {
@@ -29,13 +29,13 @@ export async function getConfig(context: GitHubContext): Promise<BotConfig | nul
   try {
     config = Value.Decode(configSchema, Value.Default(configSchema, _repoConfig));
   } catch (error) {
-    console.error("Error decoding config", error);
-    return null;
+    console.error("Error decoding config, will use default.", error);
+    return defaultConfiguration;
   }
 
   checkPluginChains(config);
 
-  return generateConfiguration(config as BotConfig);
+  return generateConfiguration(Value.Default(configSchema, _repoConfig) as BotConfig);
 }
 
 function checkPluginChains(config: PluginConfiguration) {

tests/main.test.ts

@@ -6,6 +6,8 @@ mock.module("@octokit/webhooks", () => ({
   Webhooks: WebhooksMocked,
 }));
 
+const issueOpened = "issues.opened";
+
 class WebhooksMocked {
   constructor(_: unknown) {}
   verifyAndReceive(_: unknown) {
@@ -21,6 +23,9 @@ class WebhooksMocked {
 }
 
 import { config } from "dotenv";
+import { GitHubContext } from "../src/github/github-context";
+import { GitHubEventHandler } from "../src/github/github-event-handler";
+import { getConfig } from "../src/github/utils/config";
 import worker from "../src/worker";
 import { server } from "./__mocks__/node";
 
@@ -53,7 +58,7 @@ describe("Worker tests", () => {
   it("Should start a worker", async () => {
     const req = new Request("http://localhost:8080", {
       headers: {
-        "x-github-event": "issues.opened",
+        "x-github-event": issueOpened,
         "x-github-delivery": "1",
         "x-hub-signature-256": "123456",
       },
@@ -66,4 +71,73 @@ describe("Worker tests", () => {
     });
     expect(res.status).toEqual(200);
   });
+
+  describe("Configuration tests", () => {
+    it("Should generate a default configuration when no repo is defined", async () => {
+      const cfg = await getConfig({
+        key: issueOpened,
+        name: issueOpened,
+        id: "",
+        payload: {
+          repository: "",
+        },
+        octokit: {},
+        eventHandler: {} as GitHubEventHandler,
+      } as unknown as GitHubContext);
+      expect(cfg).toBeTruthy();
+    });
+    it("Should generate a default configuration when the target repo does not contain one", async () => {
+      const cfg = await getConfig({
+        key: issueOpened,
+        name: issueOpened,
+        id: "",
+        payload: {
+          repository: {
+            owner: { login: "ubiquity" },
+            name: "ubiquibot-kernel",
+          },
+        } as unknown as GitHubContext<"issues.closed">["payload"],
+        octokit: {
+          rest: {
+            repos: {
+              getContent() {
+                return { data: null };
+              },
+            },
+          },
+        },
+        eventHandler: {} as GitHubEventHandler,
+      } as unknown as GitHubContext);
+      expect(cfg).toBeTruthy();
+    });
+    it("Should merge the configuration when found", async () => {
+      const cfg = await getConfig({
+        key: issueOpened,
+        name: issueOpened,
+        id: "",
+        payload: {
+          repository: {
+            owner: { login: "ubiquity" },
+            name: "conversation-rewards",
+          },
+        } as unknown as GitHubContext<"issues.closed">["payload"],
+        octokit: {
+          rest: {
+            repos: {
+              getContent() {
+                return {
+                  data: `
+incentives:
+  enabled: false`,
+                };
+              },
+            },
+          },
+        },
+        eventHandler: {} as GitHubEventHandler,
+      } as unknown as GitHubContext);
+      expect(cfg).toBeTruthy();
+      expect(cfg.incentives.enabled).toBeFalse();
+    });
+  });
 });