virtual-code-owners 5.1.0 → 6.0.0

package/dist/cli.js

@@ -1,3 +1,3 @@
 #!/usr/bin/env node
-import { main } from "./main.js";
-main();
+import { cli } from "./main.js";
+cli();

package/dist/generate-labeler-yml.js

@@ -35,7 +35,7 @@ function transformForYamlAndMinimatch(pOriginalString) {
         lReturnValue = "**";
     }
     if (lReturnValue.startsWith("*")) {
-        lReturnValue = `'${lReturnValue}'`;
+        lReturnValue = `"${lReturnValue}"`;
     }
     if (pOriginalString.endsWith("/")) {
         lReturnValue = `${lReturnValue}**`;

package/dist/main.js

@@ -6,7 +6,6 @@ import generateLabelerYml from "./generate-labeler-yml.js";
 import readTeamMap from "./read-team-map.js";
 import readVirtualCodeOwners from "./read-virtual-code-owners.js";
 import { VERSION } from "./version.js";
-import { getAnomalies } from "./parse.js";
 const HELP_MESSAGE = `Usage: virtual-code-owners [options]
 
 Merges a VIRTUAL-CODEOWNERS.txt and a virtual-teams.yml into CODEOWNERS
@@ -27,7 +26,7 @@ Options:
   --labelerLocation [file-name]        The location of the labeler.yml file
                                        (default: ".github/labeler.yml")
   -h, --help                           display help for command`;
-export function main(pArguments = process.argv.slice(2), pOutStream = process.stdout, pErrorStream = process.stderr) {
+export function cli(pArguments = process.argv.slice(2), pOutStream = process.stdout, pErrorStream = process.stderr) {
     try {
         const lOptions = getOptions(pArguments);
         if (lOptions.help) {
@@ -38,45 +37,13 @@ export function main(pArguments = process.argv.slice(2), pOutStream = process.st
             pOutStream.write(`${VERSION}${EOL}`);
             return;
         }
-        const lTeamMap = readTeamMap(lOptions.virtualTeams);
-        const lVirtualCodeOwners = readVirtualCodeOwners(lOptions.virtualCodeOwners, lTeamMap);
-        const lAnomalies = getAnomalies(lVirtualCodeOwners);
-        if (lAnomalies.length > 0) {
-            throw new Error(`${EOL}${reportAnomalies(lOptions.virtualCodeOwners, lAnomalies)}`);
-        }
-        const lCodeOwnersContent = generateCodeOwners(lVirtualCodeOwners, lTeamMap);
-        writeFileSync(lOptions.codeOwners, lCodeOwnersContent, {
-            encoding: "utf-8",
-        });
-        if (lOptions.emitLabeler) {
-            const lLabelerContent = generateLabelerYml(lVirtualCodeOwners, lTeamMap);
-            writeFileSync(lOptions.labelerLocation, lLabelerContent, {
-                encoding: "utf-8",
-            });
-            pErrorStream.write(`${EOL}Wrote ${lOptions.codeOwners} AND ${lOptions.labelerLocation}${EOL}${EOL}`);
-        }
-        else {
-            pErrorStream.write(`${EOL}Wrote ${lOptions.codeOwners}${EOL}${EOL}`);
-        }
+        main(lOptions, pErrorStream);
     }
     catch (pError) {
         pErrorStream.write(`${EOL}ERROR: ${pError.message}${EOL}${EOL}`);
         process.exitCode = 1;
     }
 }
-function reportAnomalies(pFileName, pAnomalies) {
-    return pAnomalies
-        .map((pAnomaly) => {
-        if (pAnomaly.type === "invalid-line") {
-            return `${pFileName}:${pAnomaly.line}:1 invalid line - neither a rule, comment nor empty: '${pAnomaly.raw}'`;
-        }
-        else {
-            return (`${pFileName}:${pAnomaly.line}:1 invalid user or team name '${pAnomaly.raw}' (# ${pAnomaly.userNumberWithinLine} on this line). ` +
-                `It should either start with '@' or be an e-mail address.`);
-        }
-    })
-        .join(EOL);
-}
 function getOptions(pArguments) {
     return parseArgs({
         args: pArguments,
@@ -113,3 +80,21 @@ function getOptions(pArguments) {
         tokens: false,
     }).values;
 }
+function main(pOptions, pErrorStream) {
+    const lTeamMap = readTeamMap(pOptions.virtualTeams);
+    const lVirtualCodeOwners = readVirtualCodeOwners(pOptions.virtualCodeOwners, lTeamMap);
+    const lCodeOwnersContent = generateCodeOwners(lVirtualCodeOwners, lTeamMap);
+    writeFileSync(pOptions.codeOwners, lCodeOwnersContent, {
+        encoding: "utf-8",
+    });
+    if (pOptions.emitLabeler) {
+        const lLabelerContent = generateLabelerYml(lVirtualCodeOwners, lTeamMap);
+        writeFileSync(pOptions.labelerLocation, lLabelerContent, {
+            encoding: "utf-8",
+        });
+        pErrorStream.write(`${EOL}Wrote '${pOptions.codeOwners}' AND '${pOptions.labelerLocation}'${EOL}${EOL}`);
+    }
+    else {
+        pErrorStream.write(`${EOL}Wrote '${pOptions.codeOwners}'${EOL}${EOL}`);
+    }
+}

package/dist/read-team-map.js

@@ -1,8 +1,30 @@
+import Ajv from "ajv";
 import { readFileSync } from "node:fs";
+import { EOL } from "node:os";
 import { parse as parseYaml } from "yaml";
+const TEAM_MAP_SCHEMA = JSON.parse(readFileSync(new URL("./virtual-teams.schema.json", import.meta.url), "utf-8"));
 export default function readTeamMap(pVirtualTeamsFileName) {
     const lVirtualTeamsAsAString = readFileSync(pVirtualTeamsFileName, {
         encoding: "utf-8",
     });
-    return parseYaml(lVirtualTeamsAsAString);
+    const lTeamMap = parseYaml(lVirtualTeamsAsAString);
+    validateTeamMap(lTeamMap, pVirtualTeamsFileName);
+    return lTeamMap;
+}
+function validateTeamMap(pTeamMap, pVirtualTeamsFileName) {
+    const ajv = new Ajv({
+        allErrors: true,
+        verbose: true,
+    });
+    if (!ajv.validate(TEAM_MAP_SCHEMA, pTeamMap)) {
+        throw new Error(`This is not a valid virtual-teams.yml:${EOL}${formatAjvErrors(ajv.errors, pVirtualTeamsFileName)}.\n`);
+    }
+}
+function formatAjvErrors(pAjvErrors, pVirtualTeamsFileName) {
+    return pAjvErrors
+        .map((pAjvError) => formatAjvError(pAjvError, pVirtualTeamsFileName))
+        .join(EOL);
+}
+function formatAjvError(pAjvError, pVirtualTeamsFileName) {
+    return `${pVirtualTeamsFileName}: ${pAjvError.instancePath} - ${JSON.stringify(pAjvError.data)} ${pAjvError.message}`;
 }

package/dist/read-virtual-code-owners.js

@@ -1,8 +1,27 @@
 import { readFileSync } from "node:fs";
-import { parse as parseVirtualCodeOwners } from "./parse.js";
+import { EOL } from "node:os";
+import { getAnomalies, parse as parseVirtualCodeOwners } from "./parse.js";
 export default function readVirtualCodeOwners(pVirtualCodeOwnersFileName, pTeamMap) {
     const lVirtualCodeOwnersAsAString = readFileSync(pVirtualCodeOwnersFileName, {
         encoding: "utf-8",
     });
-    return parseVirtualCodeOwners(lVirtualCodeOwnersAsAString, pTeamMap);
+    const lVirtualCodeOwners = parseVirtualCodeOwners(lVirtualCodeOwnersAsAString, pTeamMap);
+    const lAnomalies = getAnomalies(lVirtualCodeOwners);
+    if (lAnomalies.length > 0) {
+        throw new Error(`This is not a valid virtual code-owners file:${EOL}${reportAnomalies(pVirtualCodeOwnersFileName, lAnomalies)}`);
+    }
+    return lVirtualCodeOwners;
+}
+function reportAnomalies(pFileName, pAnomalies) {
+    return pAnomalies
+        .map((pAnomaly) => {
+        if (pAnomaly.type === "invalid-line") {
+            return `${pFileName}:${pAnomaly.line}:1 invalid line - neither a rule, comment nor empty: "${pAnomaly.raw}"`;
+        }
+        else {
+            return (`${pFileName}:${pAnomaly.line}:1 invalid user or team name "${pAnomaly.raw}" (#${pAnomaly.userNumberWithinLine} on this line). ` +
+                `It should either start with "@" or be an e-mail address.`);
+        }
+    })
+        .join(EOL);
 }

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "5.1.0";
+export const VERSION = "6.0.0";

package/dist/virtual-teams.schema.json

@@ -0,0 +1,15 @@
+{
+  "$schema": "http://json-schema.org/draft-07/schema#",
+  "title": "virtual teams schema for virtual-code-owners",
+  "description": "a list of teams and their team members",
+  "$id": "org.js.virtual-code-owners/4.1.0",
+  "type": "object",
+  "additionalProperties": {
+    "type": "array",
+    "items": {
+      "type": "string",
+      "description": "Username or e-mail address of a team member. (Don't prefix usernames with '@')",
+      "pattern": "^[^@][^\\s]+$"
+    }
+  }
+}

package/package.json

@@ -1,18 +1,33 @@
 {
   "name": "virtual-code-owners",
-  "version": "5.1.0",
+  "version": "6.0.0",
   "description": "Merges a VIRTUAL-CODEOWNERS.txt and a virtual-teams.yml into CODEOWNERS",
   "type": "module",
   "exports": {
-    ".": [
+    "parse": [
       {
-        "import": "./dist/parse-and-generate.js",
-        "types": "./types/parse-and-generate.d.ts"
+        "import": "./dist/parse.js",
+        "types": "./types/parse.d.ts"
       },
-      "./dist/parse-and-generate.js"
+      "./dist/parse.js"
+    ],
+    "generateCodeOwners": [
+      {
+        "import": "./dist/generate-codeowners.js",
+        "types": "./types/generate-codeowners.d.ts"
+      },
+      "./dist/generate-codeowners.js"
+    ],
+    "generateLabelerYml": [
+      {
+        "import": "./dist/generate-labeler-yml.js",
+        "types": "./types/generate-labeler-yml.d.ts"
+      },
+      "./dist/generate-labeler-yml.js"
     ]
   },
-  "main": "dist/parse-and-generate.js",
+  "main": "dist/main.js",
+  "types": "types/types.d.ts",
   "bin": "dist/cli.js",
   "files": [
     "dist",
@@ -21,7 +36,7 @@
     "LICENSE"
   ],
   "scripts": {
-    "build": "rm -rf dist && node --no-warnings --loader ts-node/esm tools/get-version.ts > src/version.ts && tsc",
+    "build": "rm -rf dist && node --no-warnings --loader ts-node/esm tools/get-version.ts > src/version.ts && tsc && cp -f src/*.json dist/.",
     "check": "npm run format && npm run build && npm run depcruise -- --no-progress && npm test",
     "depcruise": "depcruise src tools",
     "depcruise:graph": "depcruise src --include-only '^(src)' --output-type dot | dot -T svg | depcruise-wrap-stream-in-html > dependency-graph.html",
@@ -54,8 +69,8 @@
   },
   "devDependencies": {
     "@types/mocha": "10.0.1",
-    "@types/node": "20.3.0",
-    "c8": "7.14.0",
+    "@types/node": "20.3.1",
+    "c8": "8.0.0",
     "dependency-cruiser": "13.0.3",
     "husky": "8.0.3",
     "lint-staged": "13.2.2",
@@ -67,6 +82,7 @@
     "watskeburt": "0.11.3"
   },
   "dependencies": {
+    "ajv": "8.12.0",
     "yaml": "2.3.1"
   },
   "engines": {