virtual-code-owners 6.0.0 → 6.1.0

package/README.md

@@ -71,7 +71,7 @@ This is where `virtual-code-owners` comes in.
 
 ### VIRTUAL-CODEOWNERS.txt
 
-`VIRTUAL_CODEOWNERS.txt` is a regular, valid GitHub [CODEOWNERS](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners) file.
+`VIRTUAL-CODEOWNERS.txt` is a regular, valid GitHub [CODEOWNERS](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners) file.
 The only difference between VIRTUAL-CODEOWNERS.txt and a CODEOWNERS file is that
 the _teams_ the former uses might not exist yet, except in a `virtual-teams.yml`.
 This enables you to write a _much_ easier to maintain list of code owners.
@@ -189,10 +189,10 @@ file that will check that for you.
 - ~~Currently only works for _user names_ to identify team members - not for e-mail
   addresses.~~
   Works with both user names and e-mail addresses
-- _virtual-code-owners_ assumes the VIRTUAL-CODEOWNERS.txt is a valid CODEOWNERS
-  file and the virtual-teams.yml is a valid yaml file with teams names as keys
-  and team members as arrays under these. It will likely throw errors when this
-  assumption is not met, but the error-messages might be cryptic.
+- Although _virtual-code-owners_ performs basic validations on the CODEOWNER
+  format and tries to emit clear and actionable error messages about them, it
+  might not catch all of them - e.g. it doesn't check if the users in
+  virtual-teams.yml actually exist.
 
 ### Why the `.txt` extension?
 
@@ -236,3 +236,13 @@ npx virtual-code-owners --emitLabeler
 
 If you have an alternate file location for the `labeler.yml` you can specify that
 with virtual-code-owner's `--labelerLocation` parameter.
+
+### Can I just run virtual-code-owners to validate VIRTUAL-CODEOWNERS.txt & virtual-teams.yml?
+
+So _without_ generating any output?
+
+Yes. Use the `--dryRun` command line option:
+
+```
+npx virtual-code-owners --dryRun
+```

package/dist/generate-codeowners.js

@@ -20,7 +20,7 @@ export default function generateCodeOwners(pVirtualCodeOwners, pTeamMap, pGenera
 function generateLine(pCSTLine, pTeamMap) {
     if (pCSTLine.type === "rule") {
         const lUserNames = uniq(pCSTLine.users.flatMap((pUser) => expandTeamToUserNames(pUser, pTeamMap)))
-            .sort()
+            .sort(compareUserNames)
             .join(" ");
         return pCSTLine.filesPattern + pCSTLine.spaces + lUserNames;
     }
@@ -41,6 +41,9 @@ function userNameToCodeOwner(pUserName) {
     }
     return `@${pUserName}`;
 }
+function compareUserNames(pLeftName, pRightName) {
+    return pLeftName.toLowerCase() > pRightName.toLowerCase() ? 1 : -1;
+}
 function uniq(pUserNames) {
     return Array.from(new Set(pUserNames));
 }

package/dist/main.js

@@ -25,6 +25,8 @@ Options:
                                        (default: false)
   --labelerLocation [file-name]        The location of the labeler.yml file
                                        (default: ".github/labeler.yml")
+  --dryRun                             Just validate inputs, don't generate
+                                       outputs (default: false)
   -h, --help                           display help for command`;
 export function cli(pArguments = process.argv.slice(2), pOutStream = process.stdout, pErrorStream = process.stderr) {
     try {
@@ -72,6 +74,10 @@ function getOptions(pArguments) {
                 type: "string",
                 default: ".github/labeler.yml",
             },
+            dryRun: {
+                type: "boolean",
+                default: false,
+            },
             help: { type: "boolean", short: "h", default: false },
             version: { type: "boolean", short: "V", default: false },
         },
@@ -84,14 +90,18 @@ 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, {
+    if (!pOptions.dryRun) {
+        writeFileSync(pOptions.codeOwners, lCodeOwnersContent, {
             encoding: "utf-8",
         });
+    }
+    if (pOptions.emitLabeler) {
+        const lLabelerContent = generateLabelerYml(lVirtualCodeOwners, lTeamMap);
+        if (!pOptions.dryRun) {
+            writeFileSync(pOptions.labelerLocation, lLabelerContent, {
+                encoding: "utf-8",
+            });
+        }
         pErrorStream.write(`${EOL}Wrote '${pOptions.codeOwners}' AND '${pOptions.labelerLocation}'${EOL}${EOL}`);
     }
     else {

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

@@ -1,6 +1,6 @@
 import { readFileSync } from "node:fs";
 import { EOL } from "node:os";
-import { getAnomalies, parse as parseVirtualCodeOwners } from "./parse.js";
+import { getAnomalies, parse as parseVirtualCodeOwners, } from "./parse-virtual-code-owners.js";
 export default function readVirtualCodeOwners(pVirtualCodeOwnersFileName, pTeamMap) {
     const lVirtualCodeOwnersAsAString = readFileSync(pVirtualCodeOwnersFileName, {
         encoding: "utf-8",

package/dist/version.js

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

package/dist/virtual-teams.schema.json

@@ -2,7 +2,7 @@
   "$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",
+  "$id": "org.js.virtual-code-owners/6.0.0",
   "type": "object",
   "additionalProperties": {
     "type": "array",

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "virtual-code-owners",
-  "version": "6.0.0",
-  "description": "Merges a VIRTUAL-CODEOWNERS.txt and a virtual-teams.yml into CODEOWNERS",
+  "version": "6.1.0",
+  "description": "Makes your CODEOWNERS file liveable again",
   "type": "module",
   "exports": {
     "parse": [
       {
-        "import": "./dist/parse.js",
-        "types": "./types/parse.d.ts"
+        "import": "./dist/parse-virtual-code-owners.js",
+        "types": "./types/parse-virtual-code-owners.d.ts"
       },
-      "./dist/parse.js"
+      "./dist/parse-virtual-code-owners.js"
     ],
     "generateCodeOwners": [
       {