fix: trim slack allowed users entries in audit check

Author: benvinegar

bin/checks/slack-allowed-users.mjs

@@ -8,6 +8,7 @@ function countUsers(rawValue) {
   if (!rawValue) return 0;
   return rawValue
     .split(",")
+    .map((entry) => entry.trim())
     .filter((entry) => entry.length > 0).length;
 }
 

test/slack-allowed-users-check.test.mjs

@@ -48,6 +48,19 @@ describe("bin/checks/slack-allowed-users.mjs", () => {
     expect(payload.count).toBe("3");
   });
 
+  it("ignores whitespace-only entries", () => {
+    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "baudbot-slack-users-check-"));
+    tmpDirs.push(tmpDir);
+
+    const envPath = path.join(tmpDir, ".env");
+    fs.writeFileSync(envPath, "SLACK_ALLOWED_USERS=U1, , U2\n");
+
+    const payload = runCheck(envPath);
+    expect(payload.defined).toBe("1");
+    expect(payload.raw_non_empty).toBe("1");
+    expect(payload.count).toBe("2");
+  });
+
   it("handles empty allowed users setting", () => {
     const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "baudbot-slack-users-check-"));
     tmpDirs.push(tmpDir);