virtual-code-owners 2.2.0 → 2.2.3

package/README.md

@@ -22,8 +22,8 @@ or, if you want to be verbose
 ```
 npx virtual-code-owners \
   --virtual-code-owners .github/VIRTUAL-CODEOWNERS.txt \
-  --virtual-teams .github/virtual-teams.yml \
-  --code-owners .github/CODEOWNERS
+  --virtual-teams       .github/virtual-teams.yml \
+  --code-owners         .github/CODEOWNERS
 ```
 
 ## Why?
@@ -38,8 +38,33 @@ have the following choices:
   though there are good people on many levels in bureaucracies, it might
   eventually not pan out because #reasons.
 - Maintain a CODEOWNERS file with code assigned to large lists of individuals.  
-  An option, but laborious to maintain.
-- Use `virtual-code-owners`.
+  An option, but laborious to maintain, even for smaller projects; for example:
+
+```CODEOWNERS
+# catch-all to ensure there at least _is_ a code owner, even when
+# it's _everyone_
+
+* @cloud-heroes-all
+
+# admin & ci stuff => transversal
+
+.github/                @abraham-lincoln @benjamin-franklin @koos-koets @luke-the-lucky-ch @mary-the-merry-ch @naomi-the-namegiver-ch
+
+# generic stuff
+
+apps/framework/         @abraham-lincoln @benjamin-franklin @koos-koets @luke-the-lucky-ch @mary-the-merry-ch @naomi-the-namegiver-ch
+apps/ux-portal/         @abraham-lincoln @benjamin-franklin @charlotte-de-bourbon-ch @davy-davidson-ch @joe-dalton-ch @john-johnson-ch @koos-koets @luke-the-lucky-ch @mary-the-merry-ch @naomi-the-namegiver-ch
+libs/components/        @charlotte-de-bourbon-ch @davy-davidson-ch @joe-dalton-ch @john-johnson-ch @koos-koets
+
+# specific functionality
+
+libs/ubc-sales/         @abraham-ableton-ch @boris-bubbleblower-ch @charlotte-charleston-ch @dagny-taggert-ch @gregory-gregson-ch @jane-doe-ch @karl-marx-ch
+libs/ubc-after-sales/   @daisy-duck @donald-duck @john-doe-ch @pete-peterson-ch @william-the-fourth-ch
+libs/ubc-pre-sales/     @averel-dalton-ch @jean-claude-ch @john-galt-ch @valerie-valerton-ch
+libs/ubc-refund/        @abraham-ableton-ch @boris-bubbleblower-ch @charlotte-charleston-ch @dagny-taggert-ch @daisy-duck @donald-duck @gregory-gregson-ch @jane-doe-ch @john-doe-ch @karl-marx-ch @pete-peterson-ch @william-the-fourth-ch
+```
+
+This is where `virtual-code-owners` comes in.
 
 ## Formats
 
@@ -48,29 +73,44 @@ have the following choices:
 `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.
 
-Example:
+For example the CODEOWNERS file above can then look like this:
 
-```
-#! comments that start with #! won't appear in the CODEOWNERS output e.g.
+```CODEOWNERS
+#! comments that start with #! won't appear in the CODEOWNERS output
+#!
 #! this is not the CODEOWNERS file - to get that one run
 #!   npx virtual-code-owners
-#! on this.
 #!
-# Regular comments are retained
+# catch-all to ensure there at least _is_ a code owner, even when
+# it's _everyone_
+
 * @cloud-heroes-all
-.github/ @ch/transversal
-apps/broker @ch/transversal
-apps/ux-portal/ @ch/ux @ch/transversal
-libs/ubc-sales/ @ch/sales
-libs/ubc-after-sales/ @ch/after-sales
-libs/ubc-pre-sales/ @ch/pre-sales
-libs/components/ @ch/ux
+
+# admin & ci stuff => transversal
+
+.github/                @ch/transversal
+
+# generic stuff
+
+apps/framework/         @ch/transversal
+apps/ux-portal/         @ch/ux @ch/transversal
+libs/components/        @ch/ux
+
+# specific functionality
+
+libs/ubc-sales/         @ch/sales
+libs/ubc-after-sales/   @ch/after-sales
+libs/ubc-pre-sales/     @ch/pre-sales
+libs/ubc-refund/        @ch/sales @ch/after-sales
 ```
 
 ### virtual-teams.yml
 
-A valid YAML file that contains a list of teams, with for each team its members:
+A valid YAML file that contains a list of teams, with for each team its members.
+If a new team member joins, you can enter it here, run `npx virtual-code-owners`
+to update CODEOWNERS.
 
 ```yaml
 # yaml-language-server: $schema=https://raw.githubusercontent.com/sverweij/virtual-code-owners/main/src/virtual-teams.schema.json
@@ -78,20 +118,34 @@ ch/after-sales:
   - john-doe-ch
   - pete-peterson-ch
   - william-the-fourth-ch
+  - daisy-duck
+  - donald-duck
 ch/sales:
   - gregory-gregson-ch
   - jane-doe-ch
+  - abraham-ableton-ch
+  - boris-bubbleblower-ch
+  - charlotte-charleston-ch
+  - dagny-taggert-ch
+  - karl-marx-ch
 ch/pre-sales:
   - jean-claude-ch
   - valerie-valerton-ch
   - averel-dalton-ch
+  - john-galt-ch
 ch/ux:
   - davy-davidson-ch
   - john-johnson-ch
   - joe-dalton-ch
+  - koos-koets
+  - charlotte-de-bourbon-ch
 ch/transversal:
+  - mary-the-merry-ch
   - luke-the-lucky-ch
-# etc
+  - naomi-the-namegiver-ch
+  - benjamin-franklin
+  - koos-koets
+  - abraham-lincoln
 ```
 
 ## FAQ
@@ -118,7 +172,7 @@ team name and _virtual-code-owners_ will leave the real name alone.
 - _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 not be cryptic.
+  assumption is not met, but the error-messages might be cryptic.
 
 ### Why the `.txt` extension?
 

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

@@ -1,5 +1,5 @@
 import { EOL } from "node:os";
-const DEFAULT_GENERATED_WARNING = `#${EOL}` +
+const DEFAULT_WARNING = `#${EOL}` +
     `# DO NOT EDIT - this file is generated and your edits will be overwritten${EOL}` +
     `#${EOL}` +
     `# To make changes:${EOL}` +
@@ -8,14 +8,13 @@ const DEFAULT_GENERATED_WARNING = `#${EOL}` +
     `#   - and/ or add team members to .github/virtual-teams.yml${EOL}` +
     `#   - run 'npx virtual-code-owners'${EOL}` +
     `#${EOL}${EOL}`;
-const LINE_PATTERN = /^(?<filesPattern>[^\s]+\s+)(?<userNames>.*)$/;
-export function convert(pCodeOwnersFileAsString, pTeamMap, pGeneratedWarning = DEFAULT_GENERATED_WARNING) {
-    return `${pGeneratedWarning}${pCodeOwnersFileAsString
-        .split(EOL)
-        .filter(shouldAppearInResult)
-        .map(convertLine(pTeamMap))
-        .map(deduplicateUserNames)
-        .join(EOL)}`;
+export function convert(pCodeOwnersFileAsString, pTeamMap, pGeneratedWarning = DEFAULT_WARNING) {
+    return (pGeneratedWarning +
+        pCodeOwnersFileAsString
+            .split(EOL)
+            .filter(shouldAppearInResult)
+            .map(convertLine(pTeamMap))
+            .join(EOL));
 }
 function shouldAppearInResult(pLine) {
     return !pLine.trimStart().startsWith("#!");
@@ -23,34 +22,26 @@ function shouldAppearInResult(pLine) {
 function convertLine(pTeamMap) {
     return (pUntreatedLine) => {
         const lTrimmedLine = pUntreatedLine.trim();
-        if (lTrimmedLine.startsWith("#") || lTrimmedLine === "") {
+        const lSplitLine = lTrimmedLine.match(/^(?<filesPattern>[^\s]+\s+)(?<userNames>.*)$/);
+        if (lTrimmedLine.startsWith("#") || !lSplitLine?.groups) {
             return pUntreatedLine;
         }
-        else {
-            return replaceTeamNames(lTrimmedLine, pTeamMap);
-        }
+        const lUserNames = replaceTeamNames(lSplitLine.groups.userNames, pTeamMap);
+        return lSplitLine.groups.filesPattern + uniqAndSortUserNames(lUserNames);
     };
 }
-function replaceTeamNames(pTrimmedLine, pTeamMap) {
-    const lSplitLine = pTrimmedLine.match(LINE_PATTERN);
-    if (!lSplitLine?.groups) {
-        return pTrimmedLine;
-    }
-    let lUserNames = lSplitLine.groups.userNames.trim();
+function replaceTeamNames(pUserNames, pTeamMap) {
+    let lReturnValue = pUserNames;
     for (let lTeamName of Object.keys(pTeamMap)) {
-        lUserNames = lUserNames.replace(new RegExp(`(\\s|^)@${lTeamName}(\\s|$)`, "g"), `$1${stringifyTeamMembers(pTeamMap, lTeamName)}$2`);
+        lReturnValue = lReturnValue.replace(new RegExp(`(\\s|^)@${lTeamName}(\\s|$)`, "g"), `$1${stringifyTeamMembers(pTeamMap, lTeamName)}$2`);
     }
-    return `${lSplitLine.groups.filesPattern}${lUserNames}`;
+    return lReturnValue;
 }
 function stringifyTeamMembers(pTeamMap, pTeamName) {
     return pTeamMap[pTeamName].map((pUserName) => `@${pUserName}`).join(" ");
 }
-function deduplicateUserNames(pTrimmedLine) {
-    const lSplitLine = pTrimmedLine.match(LINE_PATTERN);
-    if (pTrimmedLine.startsWith("#") || !lSplitLine?.groups) {
-        return pTrimmedLine;
-    }
-    return `${lSplitLine.groups.filesPattern}${Array.from(new Set(lSplitLine.groups.userNames.trim().split(/\s+/)))
+function uniqAndSortUserNames(pUserNames) {
+    return Array.from(new Set(pUserNames.split(/\s+/)))
         .sort()
-        .join(" ")}`;
+        .join(" ");
 }

package/dist/read-and-convert.js

@@ -2,12 +2,12 @@ import { readFileSync } from "node:fs";
 import yaml from "js-yaml";
 import { convert } from "./convert-virtual-code-owners.js";
 export function readAndConvert(pVirtualCodeOwnersFileName, pVirtualTeamsFileName) {
-    const lVirtualCodeOwnersFileAsAString = readFileSync(pVirtualCodeOwnersFileName, {
+    const lVirtualCodeOwnersAsAString = readFileSync(pVirtualCodeOwnersFileName, {
         encoding: "utf-8",
     });
-    const lVirtualTeamsYamlAsAString = readFileSync(pVirtualTeamsFileName, {
+    const lVirtualTeamsAsAString = readFileSync(pVirtualTeamsFileName, {
         encoding: "utf-8",
     });
-    const lTeamMap = yaml.load(lVirtualTeamsYamlAsAString);
-    return convert(lVirtualCodeOwnersFileAsAString, lTeamMap);
+    const lTeamMap = yaml.load(lVirtualTeamsAsAString);
+    return convert(lVirtualCodeOwnersAsAString, lTeamMap);
 }

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "2.2.0";
+export const VERSION = "2.2.3";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "virtual-code-owners",
-  "version": "2.2.0",
+  "version": "2.2.3",
   "description": "Merges a VIRTUAL-CODEOWNERS.txt and a virtual-teams.yml into CODEOWNERS",
   "type": "module",
   "exports": {
@@ -22,12 +22,12 @@
   "scripts": {
     "build": "rm -rf dist && ts-node --esm tools/get-version.ts > src/version.ts && tsc",
     "check": "npm run format && npm run build && npm run depcruise -- --no-progress && npm test",
-    "depcruise": "depcruise src tools --config .dependency-cruiser.cjs",
-    "depcruise:graph": "depcruise src --include-only '^(src)' --config --output-type dot | dot -T svg | depcruise-wrap-stream-in-html > dependency-graph.html",
-    "depcruise:graph:dev": "depcruise src --prefix vscode://file/$(pwd)/ --config --output-type dot | dot -T svg | depcruise-wrap-stream-in-html | browser",
-    "depcruise:graph:diff:dev": "depcruise src --prefix vscode://file/$(pwd)/ --config --output-type dot --reaches \"$(watskeburt $SHA -T regex)\"| dot -T svg | depcruise-wrap-stream-in-html | browser",
-    "depcruise:graph:diff:mermaid": "depcruise src tools --config --output-type mermaid --output-to - --reaches \"$(watskeburt $SHA -T regex)\"",
-    "depcruise:html": "depcruise src tools --config --output-type err-html --output-to dependency-violation-report.html",
+    "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",
+    "depcruise:graph:dev": "depcruise src --prefix vscode://file/$(pwd)/ --output-type dot | dot -T svg | depcruise-wrap-stream-in-html | browser",
+    "depcruise:graph:diff:dev": "depcruise src --prefix vscode://file/$(pwd)/ --output-type dot --reaches \"$(watskeburt $SHA -T regex)\"| dot -T svg | depcruise-wrap-stream-in-html | browser",
+    "depcruise:graph:diff:mermaid": "depcruise src tools --output-type mermaid --output-to - --reaches \"$(watskeburt $SHA -T regex)\"",
+    "depcruise:html": "depcruise src tools --output-type err-html --output-to dependency-violation-report.html",
     "format": "prettier --loglevel warn --write \"**/*.{md,ts,json,yml}\"",
     "prepare": "husky install",
     "scm:stage": "git add .",
@@ -54,22 +54,31 @@
   "devDependencies": {
     "@types/js-yaml": "4.0.5",
     "@types/mocha": "10.0.1",
-    "@types/node": "18.15.3",
+    "@types/node": "18.15.11",
     "c8": "7.13.0",
-    "dependency-cruiser": "12.10.2",
+    "dependency-cruiser": "^13.0.0-beta-2",
     "husky": "8.0.3",
-    "lint-staged": "13.2.0",
+    "lint-staged": "13.2.1",
     "mocha": "10.2.0",
-    "prettier": "2.8.4",
+    "prettier": "2.8.7",
     "ts-node": "10.9.1",
-    "typescript": "5.0.2",
+    "typescript": "5.0.4",
     "upem": "7.3.2",
-    "watskeburt": "0.10.1"
+    "watskeburt": "0.10.2"
   },
   "dependencies": {
-    "commander": "10.0.0",
+    "commander": "10.0.1",
     "js-yaml": "4.1.0"
   },
+  "upem": {
+    "policies": [
+      {
+        "package": "dependency-cruiser",
+        "policy": "wanted",
+        "because": "we're beta testing dependency-cruiser v13 and the _latest_ policy would bump down to v12 again"
+      }
+    ]
+  },
   "engines": {
     "node": "^14.13.1||^16||>=18"
   }