virtual-code-owners 8.0.3 → 8.0.4

package/README.md

@@ -43,21 +43,20 @@ For example a CODEOWNERS file can look like this:
 
 # admin & ci stuff => transversal
 
-.github/                @ch/transversal
+.github/            @ch/transversal
 
 # generic stuff
 
-apps/framework/         @ch/transversal
-apps/ux-portal/         @ch/ux @ch/transversal
-libs/components/        @ch/ux
+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
-libs/ubc-baarden/       @ch/mannen-met-baarden
+libs/sales/         @ch/sales
+libs/after-sales/   @ch/after-sales
+libs/refund/        @ch/sales @ch/after-sales
+libs/baarden/       @ch/mannen-met-baarden
 ```
 
 ... where only the @cloud-heroes-all is a 'real' team on GitHub level. The other
@@ -74,7 +73,7 @@ to keep CODEOWNERS current.
 ch/after-sales:
   - john-doe-ch
   - pete-peterson-ch
-  - william-the-fourth-ch
+  - john-galt-ch
   - daisy-duck
   - donald-duck
 ch/sales:
@@ -83,11 +82,6 @@ ch/sales:
   - abraham-ableton-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
@@ -129,21 +123,20 @@ Running `npx virtual-code-owners` will combine these into a CODEOWNERS file like
 
 # admin & ci stuff => transversal
 
-.github/                @abraham-lincoln @benjamin-franklin @koos-koets @luke-the-lucky-ch @mary-the-merry-ch
+.github/            @abraham-lincoln @benjamin-franklin @koos-koets @luke-the-lucky-ch @mary-the-merry-ch
 
 # generic stuff
 
-apps/framework/         @abraham-lincoln @benjamin-franklin @koos-koets @luke-the-lucky-ch @mary-the-merry-ch
-apps/ux-portal/         @abraham-lincoln @benjamin-franklin @davy-davidson-ch @joe-dalton-ch @john-johnson-ch @koos-koets @luke-the-lucky-ch @mary-the-merry-ch
-libs/components/        @davy-davidson-ch @joe-dalton-ch @john-johnson-ch @koos-koets
+apps/framework/     @abraham-lincoln @benjamin-franklin @koos-koets @luke-the-lucky-ch @mary-the-merry-ch
+apps/ux-portal/     @abraham-lincoln @benjamin-franklin @davy-davidson-ch @joe-dalton-ch @john-johnson-ch @koos-koets @luke-the-lucky-ch @mary-the-merry-ch
+libs/components/    @davy-davidson-ch @joe-dalton-ch @john-johnson-ch @koos-koets
 
 # specific functionality
 
-libs/ubc-sales/         @abraham-ableton-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 @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
-libs/ubc-baarden/       jan@example.com korneel@example.com pier@example.com tjorus@example.com
+libs/sales/         @abraham-ableton-ch @dagny-taggert-ch @gregory-gregson-ch @jane-doe-ch @karl-marx-ch
+libs/after-sales/   @daisy-duck @donald-duck @john-doe-ch @john-galt @pete-peterson-ch
+libs/refund/        @abraham-ableton-ch @dagny-taggert-ch @daisy-duck @donald-duck @gregory-gregson-ch @jane-doe-ch @john-doe-ch @john-galt @karl-marx-ch @pete-peterson-ch
+libs/baarden/       jan@example.com korneel@example.com pier@example.com tjorus@example.com
 ```
 
 ## FAQ
@@ -252,16 +245,16 @@ the formatting of _those_.
 
 You should _totally_ use GitHub teams! If you can.
 
-Organizations sometimes have large mono repositories with loads of code owners.
-They or their bureaucracy haven't landed on actually using GitHub teams to clearly
+Organizations sometimes have large mono repositories with many code owners.
+They or their bureaucracy haven't landed on actually using GitHub teams to
 demarcate that. Or you're working on a cross-functional team that doesn't follow
 the organization chart (and hence the GitHub teams). Teams in those organizations
-who want to have clear code ownership have the following choices:
+who want to have clear code ownership can either:
 
 - Wrestle the bureaucracy.  
-  This is the recommended approach. It might take a while, though - and even
-  though there are good people on many levels in bureaucracies, it might
-  eventually not pan out because #reasons.
+  Recommended! It might take a while, though - and even 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, even for smaller projects
 

package/dist/cli.js

@@ -25,64 +25,64 @@ Options:
                                        outputs (default: false)
   -h, --help                           display help for command`;
 export function cli(
-  pArguments = process.argv.slice(2),
-  pOutStream = process.stdout,
-  pErrorStream = process.stderr,
-  pErrorExitCode = 1,
+	pArguments = process.argv.slice(2),
+	pOutStream = process.stdout,
+	pErrorStream = process.stderr,
+	pErrorExitCode = 1,
 ) {
-  try {
-    const lOptions = getOptions(pArguments);
-    if (lOptions.help) {
-      pOutStream.write(`${HELP_MESSAGE}${EOL}`);
-      return;
-    }
-    if (lOptions.version) {
-      pOutStream.write(`${VERSION}${EOL}`);
-      return;
-    }
-    main(lOptions, pErrorStream);
-  } catch (pError) {
-    pErrorStream.write(`${EOL}ERROR: ${pError.message}${EOL}${EOL}`);
-    process.exitCode = pErrorExitCode;
-  }
+	try {
+		const lOptions = getOptions(pArguments);
+		if (lOptions.help) {
+			pOutStream.write(`${HELP_MESSAGE}${EOL}`);
+			return;
+		}
+		if (lOptions.version) {
+			pOutStream.write(`${VERSION}${EOL}`);
+			return;
+		}
+		main(lOptions, pErrorStream);
+	} catch (pError) {
+		pErrorStream.write(`${EOL}ERROR: ${pError.message}${EOL}${EOL}`);
+		process.exitCode = pErrorExitCode;
+	}
 }
 function getOptions(pArguments) {
-  return parseArgs({
-    args: pArguments,
-    options: {
-      virtualCodeOwners: {
-        type: "string",
-        short: "v",
-        default: ".github/VIRTUAL-CODEOWNERS.txt",
-      },
-      virtualTeams: {
-        type: "string",
-        short: "t",
-        default: ".github/virtual-teams.yml",
-      },
-      codeOwners: {
-        type: "string",
-        short: "c",
-        default: ".github/CODEOWNERS",
-      },
-      emitLabeler: {
-        type: "boolean",
-        short: "l",
-        default: false,
-      },
-      labelerLocation: {
-        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 },
-    },
-    strict: true,
-    allowPositionals: true,
-    tokens: false,
-  }).values;
+	return parseArgs({
+		args: pArguments,
+		options: {
+			virtualCodeOwners: {
+				type: "string",
+				short: "v",
+				default: ".github/VIRTUAL-CODEOWNERS.txt",
+			},
+			virtualTeams: {
+				type: "string",
+				short: "t",
+				default: ".github/virtual-teams.yml",
+			},
+			codeOwners: {
+				type: "string",
+				short: "c",
+				default: ".github/CODEOWNERS",
+			},
+			emitLabeler: {
+				type: "boolean",
+				short: "l",
+				default: false,
+			},
+			labelerLocation: {
+				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 },
+		},
+		strict: true,
+		allowPositionals: true,
+		tokens: false,
+	}).values;
 }

package/dist/codeowners/generate.js

@@ -1,63 +1,63 @@
 import { EOL } from "node:os";
 import { isEmailIshUsername } from "../utensils.js";
 const DEFAULT_WARNING =
-  `#${EOL}` +
-  `# DO NOT EDIT - this file is generated and your edits will be overwritten${EOL}` +
-  `#${EOL}` +
-  `# To make changes:${EOL}` +
-  `#${EOL}` +
-  `#   - edit .github/VIRTUAL-CODEOWNERS.txt${EOL}` +
-  `#   - and/ or add team members to .github/virtual-teams.yml${EOL}` +
-  `#   - run 'npx virtual-code-owners' (or 'npx virtual-code-owners --emitLabeler' if you also${EOL}` +
-  `#     want to generate a .github/labeler.yml)${EOL}` +
-  `#${EOL}${EOL}`;
+	`#${EOL}` +
+	`# DO NOT EDIT - this file is generated and your edits will be overwritten${EOL}` +
+	`#${EOL}` +
+	`# To make changes:${EOL}` +
+	`#${EOL}` +
+	`#   - edit .github/VIRTUAL-CODEOWNERS.txt${EOL}` +
+	`#   - and/ or add team members to .github/virtual-teams.yml${EOL}` +
+	`#   - run 'npx virtual-code-owners' (or 'npx virtual-code-owners --emitLabeler' if you also${EOL}` +
+	`#     want to generate a .github/labeler.yml)${EOL}` +
+	`#${EOL}${EOL}`;
 export default function generateCodeOwners(
-  pVirtualCodeOwners,
-  pTeamMap,
-  pGeneratedWarning = DEFAULT_WARNING,
+	pVirtualCodeOwners,
+	pTeamMap,
+	pGeneratedWarning = DEFAULT_WARNING,
 ) {
-  return (
-    pGeneratedWarning +
-    pVirtualCodeOwners
-      .filter((pLine) => pLine.type !== "ignorable-comment")
-      .map((pLine) => generateLine(pLine, pTeamMap))
-      .join(EOL)
-  );
+	return (
+		pGeneratedWarning +
+		pVirtualCodeOwners
+			.filter((pLine) => pLine.type !== "ignorable-comment")
+			.map((pLine) => generateLine(pLine, pTeamMap))
+			.join(EOL)
+	);
 }
 function generateLine(pCSTLine, pTeamMap) {
-  if (pCSTLine.type === "rule") {
-    const lUserNames = uniq(
-      pCSTLine.users.flatMap((pUser) => expandTeamToUserNames(pUser, pTeamMap)),
-    )
-      .sort(compareUserNames)
-      .join(" ");
-    return (
-      pCSTLine.filesPattern +
-      pCSTLine.spaces +
-      lUserNames +
-      (pCSTLine.inlineComment ? ` #${pCSTLine.inlineComment}` : "")
-    );
-  }
-  return pCSTLine.raw;
+	if (pCSTLine.type === "rule") {
+		const lUserNames = uniq(
+			pCSTLine.users.flatMap((pUser) => expandTeamToUserNames(pUser, pTeamMap)),
+		)
+			.sort(compareUserNames)
+			.join(" ");
+		return (
+			pCSTLine.filesPattern +
+			pCSTLine.spaces +
+			lUserNames +
+			(pCSTLine.inlineComment ? ` #${pCSTLine.inlineComment}` : "")
+		);
+	}
+	return pCSTLine.raw;
 }
 function expandTeamToUserNames(pUser, pTeamMap) {
-  if (pUser.type === "virtual-team-name") {
-    return stringifyTeamMembers(pTeamMap, pUser.bareName);
-  }
-  return [pUser.raw];
+	if (pUser.type === "virtual-team-name") {
+		return stringifyTeamMembers(pTeamMap, pUser.bareName);
+	}
+	return [pUser.raw];
 }
 function stringifyTeamMembers(pTeamMap, pTeamName) {
-  return (pTeamMap[pTeamName] ?? []).map(userNameToCodeOwner);
+	return (pTeamMap[pTeamName] ?? []).map(userNameToCodeOwner);
 }
 function userNameToCodeOwner(pUserName) {
-  if (isEmailIshUsername(pUserName)) {
-    return pUserName;
-  }
-  return `@${pUserName}`;
+	if (isEmailIshUsername(pUserName)) {
+		return pUserName;
+	}
+	return `@${pUserName}`;
 }
 function compareUserNames(pLeftName, pRightName) {
-  return pLeftName.toLowerCase() > pRightName.toLowerCase() ? 1 : -1;
+	return pLeftName.toLowerCase() > pRightName.toLowerCase() ? 1 : -1;
 }
 function uniq(pUserNames) {
-  return Array.from(new Set(pUserNames));
+	return Array.from(new Set(pUserNames));
 }

package/dist/labeler-yml/generate.js

@@ -1,56 +1,56 @@
 import { EOL } from "node:os";
 const DEFAULT_WARNING =
-  `#${EOL}` +
-  `# DO NOT EDIT - this file is generated and your edits will be overwritten${EOL}` +
-  `#${EOL}` +
-  `# To make changes:${EOL}` +
-  `#${EOL}` +
-  `#   - edit .github/VIRTUAL-CODEOWNERS.txt${EOL}` +
-  `#   - and/ or add teams (& members) to .github/virtual-teams.yml${EOL}` +
-  `#   - run 'npx virtual-code-owners --emitLabeler'${EOL}` +
-  `#${EOL}${EOL}`;
+	`#${EOL}` +
+	`# DO NOT EDIT - this file is generated and your edits will be overwritten${EOL}` +
+	`#${EOL}` +
+	`# To make changes:${EOL}` +
+	`#${EOL}` +
+	`#   - edit .github/VIRTUAL-CODEOWNERS.txt${EOL}` +
+	`#   - and/ or add teams (& members) to .github/virtual-teams.yml${EOL}` +
+	`#   - run 'npx virtual-code-owners --emitLabeler'${EOL}` +
+	`#${EOL}${EOL}`;
 export default function generateLabelerYml(
-  pCodeOwners,
-  pTeamMap,
-  pGeneratedWarning = DEFAULT_WARNING,
+	pCodeOwners,
+	pTeamMap,
+	pGeneratedWarning = DEFAULT_WARNING,
 ) {
-  let lReturnValue = pGeneratedWarning;
-  for (const lTeamName in pTeamMap) {
-    const lPatternsForTeam = getPatternsForTeam(pCodeOwners, lTeamName)
-      .map(
-        (pPattern) =>
-          `      - any-glob-to-any-file: ${transformForYamlAndMinimatch(pPattern)}${EOL}`,
-      )
-      .join("");
-    if (lPatternsForTeam) {
-      lReturnValue += `${lTeamName}:${EOL}  - changed-files:${EOL}${lPatternsForTeam}${EOL}`;
-    }
-  }
-  return lReturnValue;
+	let lReturnValue = pGeneratedWarning;
+	for (const lTeamName in pTeamMap) {
+		const lPatternsForTeam = getPatternsForTeam(pCodeOwners, lTeamName)
+			.map(
+				(pPattern) =>
+					`      - any-glob-to-any-file: ${transformForYamlAndMinimatch(pPattern)}${EOL}`,
+			)
+			.join("");
+		if (lPatternsForTeam) {
+			lReturnValue += `${lTeamName}:${EOL}  - changed-files:${EOL}${lPatternsForTeam}${EOL}`;
+		}
+	}
+	return lReturnValue;
 }
 function getPatternsForTeam(pCodeOwners, pTeamName) {
-  return pCodeOwners
-    .filter((pLine) => {
-      return pLine.type === "rule" && lineContainsTeamName(pLine, pTeamName);
-    })
-    .map((pLine) => pLine.filesPattern);
+	return pCodeOwners
+		.filter((pLine) => {
+			return pLine.type === "rule" && lineContainsTeamName(pLine, pTeamName);
+		})
+		.map((pLine) => pLine.filesPattern);
 }
 function transformForYamlAndMinimatch(pOriginalString) {
-  let lReturnValue = pOriginalString;
-  if (pOriginalString === "*") {
-    lReturnValue = "**";
-  }
-  if (lReturnValue.startsWith("*")) {
-    lReturnValue = `"${lReturnValue}"`;
-  }
-  if (pOriginalString.endsWith("/")) {
-    lReturnValue = `${lReturnValue}**`;
-  }
-  return lReturnValue;
+	let lReturnValue = pOriginalString;
+	if (pOriginalString === "*") {
+		lReturnValue = "**";
+	}
+	if (lReturnValue.startsWith("*")) {
+		lReturnValue = `"${lReturnValue}"`;
+	}
+	if (pOriginalString.endsWith("/")) {
+		lReturnValue = `${lReturnValue}**`;
+	}
+	return lReturnValue;
 }
 function lineContainsTeamName(pLine, pTeamName) {
-  return pLine.users.some(
-    (pUser) =>
-      pUser.type === "virtual-team-name" && pUser.bareName === pTeamName,
-  );
+	return pLine.users.some(
+		(pUser) =>
+			pUser.type === "virtual-team-name" && pUser.bareName === pTeamName,
+	);
 }

package/dist/main.js

@@ -5,30 +5,30 @@ import generateLabelerYml from "./labeler-yml/generate.js";
 import readTeamMap from "./team-map/read.js";
 import readVirtualCodeOwners from "./virtual-code-owners/read.js";
 export function main(pOptions, pErrorStream) {
-  const lTeamMap = readTeamMap(pOptions.virtualTeams);
-  const lVirtualCodeOwners = readVirtualCodeOwners(
-    pOptions.virtualCodeOwners,
-    lTeamMap,
-  );
-  const lCodeOwnersContent = generateCodeOwners(lVirtualCodeOwners, lTeamMap);
-  if (!pOptions.dryRun) {
-    writeFileSync(pOptions.codeOwners, lCodeOwnersContent, {
-      encoding: "utf-8",
-      flag: "w",
-    });
-  }
-  if (pOptions.emitLabeler) {
-    const lLabelerContent = generateLabelerYml(lVirtualCodeOwners, lTeamMap);
-    if (!pOptions.dryRun) {
-      writeFileSync(pOptions.labelerLocation, lLabelerContent, {
-        encoding: "utf-8",
-        flag: "w",
-      });
-    }
-    pErrorStream.write(
-      `${EOL}Wrote '${pOptions.codeOwners}' AND '${pOptions.labelerLocation}'${EOL}${EOL}`,
-    );
-  } else {
-    pErrorStream.write(`${EOL}Wrote '${pOptions.codeOwners}'${EOL}${EOL}`);
-  }
+	const lTeamMap = readTeamMap(pOptions.virtualTeams);
+	const lVirtualCodeOwners = readVirtualCodeOwners(
+		pOptions.virtualCodeOwners,
+		lTeamMap,
+	);
+	const lCodeOwnersContent = generateCodeOwners(lVirtualCodeOwners, lTeamMap);
+	if (!pOptions.dryRun) {
+		writeFileSync(pOptions.codeOwners, lCodeOwnersContent, {
+			encoding: "utf-8",
+			flag: "w",
+		});
+	}
+	if (pOptions.emitLabeler) {
+		const lLabelerContent = generateLabelerYml(lVirtualCodeOwners, lTeamMap);
+		if (!pOptions.dryRun) {
+			writeFileSync(pOptions.labelerLocation, lLabelerContent, {
+				encoding: "utf-8",
+				flag: "w",
+			});
+		}
+		pErrorStream.write(
+			`${EOL}Wrote '${pOptions.codeOwners}' AND '${pOptions.labelerLocation}'${EOL}${EOL}`,
+		);
+	} else {
+		pErrorStream.write(`${EOL}Wrote '${pOptions.codeOwners}'${EOL}${EOL}`);
+	}
 }

package/dist/team-map/read.js

@@ -4,29 +4,29 @@ import { EOL } from "node:os";
 import { parse as parseYaml } from "yaml";
 import virtualTeamsSchema from "./virtual-teams.schema.js";
 export default function readTeamMap(pVirtualTeamsFileName) {
-  const lVirtualTeamsAsAString = readFileSync(pVirtualTeamsFileName, {
-    encoding: "utf-8",
-  });
-  const lTeamMap = parseYaml(lVirtualTeamsAsAString);
-  assertTeamMapValid(lTeamMap, pVirtualTeamsFileName);
-  return lTeamMap;
+	const lVirtualTeamsAsAString = readFileSync(pVirtualTeamsFileName, {
+		encoding: "utf-8",
+	});
+	const lTeamMap = parseYaml(lVirtualTeamsAsAString);
+	assertTeamMapValid(lTeamMap, pVirtualTeamsFileName);
+	return lTeamMap;
 }
 function assertTeamMapValid(pTeamMap, pVirtualTeamsFileName) {
-  const ajv = new Ajv({
-    allErrors: true,
-    verbose: true,
-  });
-  if (!ajv.validate(virtualTeamsSchema, pTeamMap)) {
-    throw new Error(
-      `This is not a valid virtual-teams.yml:${EOL}${formatAjvErrors(ajv.errors, pVirtualTeamsFileName)}.\n`,
-    );
-  }
+	const ajv = new Ajv({
+		allErrors: true,
+		verbose: true,
+	});
+	if (!ajv.validate(virtualTeamsSchema, 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);
+	return pAjvErrors
+		.map((pAjvError) => formatAjvError(pAjvError, pVirtualTeamsFileName))
+		.join(EOL);
 }
 function formatAjvError(pAjvError, pVirtualTeamsFileName) {
-  return `${pVirtualTeamsFileName}: ${pAjvError.instancePath} - ${JSON.stringify(pAjvError.data)} ${pAjvError.message}`;
+	return `${pVirtualTeamsFileName}: ${pAjvError.instancePath} - ${JSON.stringify(pAjvError.data)} ${pAjvError.message}`;
 }

package/dist/team-map/virtual-teams.schema.js

@@ -1,16 +1,16 @@
 export default {
-  $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/7.0.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]+$",
-    },
-  },
+	$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/7.0.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/dist/utensils.js

@@ -1,4 +1,4 @@
 export function isEmailIshUsername(pUsername) {
-  const lEmailIshUsernameRE = /^.+@.+$/;
-  return Boolean(pUsername.match(lEmailIshUsernameRE));
+	const lEmailIshUsernameRE = /^.+@.+$/;
+	return Boolean(pUsername.match(lEmailIshUsernameRE));
 }

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "8.0.3";
+export const VERSION = "8.0.4";

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

@@ -1,32 +1,32 @@
 export function getAnomalies(pVirtualCodeOwners) {
-  const weirdLines = pVirtualCodeOwners
-    .filter((pLine) => pLine.type === "unknown")
-    .map((pLine) => ({
-      ...pLine,
-      type: "invalid-line",
-    }));
-  const weirdUsers = pVirtualCodeOwners.flatMap((pLine) => {
-    if (pLine.type === "rule") {
-      return pLine.users
-        .filter((pUser) => pUser.type === "invalid")
-        .map((pUser) => ({
-          ...pUser,
-          line: pLine.line,
-          type: "invalid-user",
-        }));
-    }
-    return [];
-  });
-  return weirdLines.concat(weirdUsers).sort(orderAnomaly);
+	const weirdLines = pVirtualCodeOwners
+		.filter((pLine) => pLine.type === "unknown")
+		.map((pLine) => ({
+			...pLine,
+			type: "invalid-line",
+		}));
+	const weirdUsers = pVirtualCodeOwners.flatMap((pLine) => {
+		if (pLine.type === "rule") {
+			return pLine.users
+				.filter((pUser) => pUser.type === "invalid")
+				.map((pUser) => ({
+					...pUser,
+					line: pLine.line,
+					type: "invalid-user",
+				}));
+		}
+		return [];
+	});
+	return weirdLines.concat(weirdUsers).sort(orderAnomaly);
 }
 function orderAnomaly(pLeft, pRight) {
-  if (
-    pLeft.line === pRight.line &&
-    pLeft.type === "invalid-user" &&
-    pRight.type === "invalid-user"
-  ) {
-    return pLeft.userNumberWithinLine > pRight.userNumberWithinLine ? 1 : -1;
-  } else {
-    return pLeft.line > pRight.line ? 1 : -1;
-  }
+	if (
+		pLeft.line === pRight.line &&
+		pLeft.type === "invalid-user" &&
+		pRight.type === "invalid-user"
+	) {
+		return pLeft.userNumberWithinLine > pRight.userNumberWithinLine ? 1 : -1;
+	} else {
+		return pLeft.line > pRight.line ? 1 : -1;
+	}
 }

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

@@ -1,67 +1,67 @@
 import { EOL } from "node:os";
 import { isEmailIshUsername } from "../utensils.js";
 export function parse(pVirtualCodeOwnersAsString, pTeamMap = {}) {
-  return pVirtualCodeOwnersAsString
-    .split(EOL)
-    .map((pUntreatedLine, pLineNo) =>
-      parseLine(pUntreatedLine, pTeamMap, pLineNo + 1),
-    );
+	return pVirtualCodeOwnersAsString
+		.split(EOL)
+		.map((pUntreatedLine, pLineNo) =>
+			parseLine(pUntreatedLine, pTeamMap, pLineNo + 1),
+		);
 }
 function parseLine(pUntreatedLine, pTeamMap, pLineNo) {
-  const lTrimmedLine = pUntreatedLine.trim();
-  const lCommentSplitLine = lTrimmedLine.split(/\s*#/);
-  const lRule = lCommentSplitLine[0]?.match(
-    /^(?<filesPattern>[^\s]+)(?<spaces>\s+)(?<userNames>.*)$/,
-  );
-  if (lTrimmedLine.startsWith("#!")) {
-    return { type: "ignorable-comment", line: pLineNo, raw: pUntreatedLine };
-  }
-  if (lTrimmedLine.startsWith("#")) {
-    return { type: "comment", line: pLineNo, raw: pUntreatedLine };
-  }
-  if (!lRule?.groups) {
-    if (lTrimmedLine === "") {
-      return { type: "empty", line: pLineNo, raw: pUntreatedLine };
-    }
-    return { type: "unknown", line: pLineNo, raw: pUntreatedLine };
-  }
-  return {
-    type: "rule",
-    line: pLineNo,
-    filesPattern: lRule.groups.filesPattern,
-    spaces: lRule.groups.spaces,
-    users: parseUsers(lRule.groups.userNames, pTeamMap),
-    inlineComment: lCommentSplitLine[1] ?? "",
-    raw: pUntreatedLine,
-  };
+	const lTrimmedLine = pUntreatedLine.trim();
+	const lCommentSplitLine = lTrimmedLine.split(/\s*#/);
+	const lRule = lCommentSplitLine[0]?.match(
+		/^(?<filesPattern>[^\s]+)(?<spaces>\s+)(?<userNames>.*)$/,
+	);
+	if (lTrimmedLine.startsWith("#!")) {
+		return { type: "ignorable-comment", line: pLineNo, raw: pUntreatedLine };
+	}
+	if (lTrimmedLine.startsWith("#")) {
+		return { type: "comment", line: pLineNo, raw: pUntreatedLine };
+	}
+	if (!lRule?.groups) {
+		if (lTrimmedLine === "") {
+			return { type: "empty", line: pLineNo, raw: pUntreatedLine };
+		}
+		return { type: "unknown", line: pLineNo, raw: pUntreatedLine };
+	}
+	return {
+		type: "rule",
+		line: pLineNo,
+		filesPattern: lRule.groups.filesPattern,
+		spaces: lRule.groups.spaces,
+		users: parseUsers(lRule.groups.userNames, pTeamMap),
+		inlineComment: lCommentSplitLine[1] ?? "",
+		raw: pUntreatedLine,
+	};
 }
 function parseUsers(pUserNamesString, pTeamMap) {
-  const lUserNames = pUserNamesString.split(/\s+/);
-  return lUserNames.map((pUserName, pIndex) => {
-    const lBareName = getBareUserName(pUserName);
-    return {
-      type: getUserNameType(pUserName, lBareName, pTeamMap),
-      userNumberWithinLine: pIndex + 1,
-      bareName: lBareName,
-      raw: pUserName,
-    };
-  });
+	const lUserNames = pUserNamesString.split(/\s+/);
+	return lUserNames.map((pUserName, pIndex) => {
+		const lBareName = getBareUserName(pUserName);
+		return {
+			type: getUserNameType(pUserName, lBareName, pTeamMap),
+			userNumberWithinLine: pIndex + 1,
+			bareName: lBareName,
+			raw: pUserName,
+		};
+	});
 }
 function getUserNameType(pUserName, pBareName, pTeamMap) {
-  if (isEmailIshUsername(pUserName)) {
-    return "e-mail-address";
-  }
-  if (pUserName.startsWith("@")) {
-    if (pTeamMap.hasOwnProperty(pBareName)) {
-      return "virtual-team-name";
-    }
-    return "other-user-or-team";
-  }
-  return "invalid";
+	if (isEmailIshUsername(pUserName)) {
+		return "e-mail-address";
+	}
+	if (pUserName.startsWith("@")) {
+		if (pTeamMap.hasOwnProperty(pBareName)) {
+			return "virtual-team-name";
+		}
+		return "other-user-or-team";
+	}
+	return "invalid";
 }
 function getBareUserName(pUserName) {
-  if (pUserName.startsWith("@")) {
-    return pUserName.slice(1);
-  }
-  return pUserName;
+	if (pUserName.startsWith("@")) {
+		return pUserName.slice(1);
+	}
+	return pUserName;
 }

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

@@ -3,35 +3,35 @@ import { EOL } from "node:os";
 import { getAnomalies } from "./anomalies.js";
 import { parse as parseVirtualCodeOwners } from "./parse.js";
 export default function readVirtualCodeOwners(
-  pVirtualCodeOwnersFileName,
-  pTeamMap,
+	pVirtualCodeOwnersFileName,
+	pTeamMap,
 ) {
-  const lVirtualCodeOwnersAsAString = readFileSync(pVirtualCodeOwnersFileName, {
-    encoding: "utf-8",
-  });
-  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;
+	const lVirtualCodeOwnersAsAString = readFileSync(pVirtualCodeOwnersFileName, {
+		encoding: "utf-8",
+	});
+	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);
+	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/package.json

@@ -1,38 +1,38 @@
 {
-  "name": "virtual-code-owners",
-  "version": "8.0.3",
-  "description": "CODEOWNERS with teams for teams that can't use GitHub teams",
-  "type": "module",
-  "bin": {
-    "virtual-code-owners": "dist/run-cli.js"
-  },
-  "files": [
-    "dist/",
-    "package.json",
-    "README.md",
-    "LICENSE"
-  ],
-  "keywords": [
-    "CODEOWNERS"
-  ],
-  "author": "Sander Verweij (https://sverweij.github.io/)",
-  "license": "MIT",
-  "homepage": "https://github.com/sverweij/virtual-code-owners",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/sverweij/virtual-code-owners.git"
-  },
-  "bugs": {
-    "url": "https://github.com/sverweij/virtual-code-owners/issues"
-  },
-  "dependencies": {
-    "ajv": "8.12.0",
-    "yaml": "2.4.1"
-  },
-  "engines": {
-    "node": "^18.11.0||>=20.0.0"
-  },
-  "scripts": {
-    "test": "echo for test, build and static analysis scripts: see the github repository"
-  }
+	"name": "virtual-code-owners",
+	"version": "8.0.4",
+	"description": "CODEOWNERS with teams for teams that can't use GitHub teams",
+	"type": "module",
+	"bin": {
+		"virtual-code-owners": "dist/run-cli.js"
+	},
+	"files": [
+		"dist/",
+		"package.json",
+		"README.md",
+		"LICENSE"
+	],
+	"keywords": [
+		"CODEOWNERS"
+	],
+	"author": "Sander Verweij (https://sverweij.github.io/)",
+	"license": "MIT",
+	"homepage": "https://github.com/sverweij/virtual-code-owners",
+	"repository": {
+		"type": "git",
+		"url": "git+https://github.com/sverweij/virtual-code-owners.git"
+	},
+	"bugs": {
+		"url": "https://github.com/sverweij/virtual-code-owners/issues"
+	},
+	"dependencies": {
+		"ajv": "8.12.0",
+		"yaml": "2.4.1"
+	},
+	"engines": {
+		"node": "^18.11.0||>=20.0.0"
+	},
+	"scripts": {
+		"test": "echo for test, build and static analysis scripts: see the github repository"
+	}
 }