deploy-vir 3.4.0 → 3.4.2

package/LICENSE-MIT

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2024 electrovir
+Copyright (c) 2026 electrovir
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/dist/commit.d.ts

@@ -0,0 +1,15 @@
+import { type DefaultLogFields, type ListLogLine } from 'simple-git';
+/**
+ * A git commit.
+ *
+ * @category Internal
+ */
+export type Commit = DefaultLogFields & ListLogLine;
+/**
+ * Extract the human author name for a commit. Commits authored by a bot (a name with a `[bot]`
+ * suffix, like GitHub's merge queue bot) are attributed to their first non-bot `Co-authored-by:`
+ * trailer instead.
+ *
+ * @category Internal
+ */
+export declare function getCommitAuthorName(commit: Readonly<Pick<Commit, 'author_name' | 'body'>>): string;

package/dist/commit.js

@@ -0,0 +1,19 @@
+const coAuthorRegExp = /^co[- \t]*authored[- \t]*by:(.*)$/gim;
+const botNameSuffix = '[bot]';
+/**
+ * Extract the human author name for a commit. Commits authored by a bot (a name with a `[bot]`
+ * suffix, like GitHub's merge queue bot) are attributed to their first non-bot `Co-authored-by:`
+ * trailer instead.
+ *
+ * @category Internal
+ */
+export function getCommitAuthorName(commit) {
+    if (!commit.author_name.endsWith(botNameSuffix)) {
+        return commit.author_name;
+    }
+    const coAuthorName = Array.from(commit.body.matchAll(coAuthorRegExp))
+        /** Strip the trailing `<email>` from each trailer. */
+        .map((match) => (match[1] || '').split('<')[0]?.trim())
+        .find((name) => name && !name.endsWith(botNameSuffix));
+    return coAuthorName || commit.author_name;
+}

package/dist/config.d.ts

@@ -1,5 +1,6 @@
 import { type MaybePromise, type PartialWithUndefined } from '@augment-vir/common';
-import { type Commit, type DeployResult } from './git.js';
+import { type Commit } from './commit.js';
+import { type DeployResult } from './git.js';
 /**
  * The full deploy-vir config object.
  *

package/dist/git.d.ts

@@ -1,4 +1,5 @@
-import { type DefaultLogFields, type ListLogLine, type SimpleGit } from 'simple-git';
+import { type SimpleGit } from 'simple-git';
+import { type Commit } from './commit.js';
 import { type DeployVirBranchConfig, type DeployVirHooks, type DeployVirRepoConfig, type NotificationConfig } from './config.js';
 /**
  * Find the git remote's name, or create a new one.
@@ -6,12 +7,6 @@ import { type DeployVirBranchConfig, type DeployVirHooks, type DeployVirRepoConf
  * @category Internal
  */
 export declare function getGitRemoteName(git: Readonly<SimpleGit>, repoConfig: Readonly<Pick<DeployVirRepoConfig, 'name' | 'gitUrl'>>): Promise<string>;
-/**
- * A git commit.
- *
- * @category Internal
- */
-export type Commit = DefaultLogFields & ListLogLine;
 /**
  * All commits involved in a deploy.
  *

package/dist/git.js

@@ -1,6 +1,7 @@
 import { assert, assertWrap, check } from '@augment-vir/assert';
 import { awaitedBlockingMap, log, logColors } from '@augment-vir/common';
 import { confirm } from '@inquirer/prompts';
+import { getCommitAuthorName } from './commit.js';
 import { KnownError } from './known.error.js';
 import { sendNotifications } from './notifications/send-notifications.js';
 /**
@@ -31,6 +32,7 @@ export async function getGitRemoteName(git, repoConfig) {
  *
  * @category Internal
  */
+// eslint-disable-next-line @virmator/prefer-params-object
 export async function pushDeploy(git, branchConfig, repoConfig, remoteName, notifications, bypassConfirmation = false, hooks) {
     const { deployName, branches } = branchConfig;
     return await awaitedBlockingMap(branches, async ({ fromBranch, toBranch, enableNotifications }) => {
@@ -64,12 +66,12 @@ export async function pushDeploy(git, branchConfig, repoConfig, remoteName, noti
         const requiresForcePush = commitsBehind.length > 0;
         log.info(`\n${requiresForcePush ? 'Only on' : 'Releasing from'} ${logColors.bold}${fromBranch}${logColors.reset}:`);
         commitsAhead.forEach((commit, index) => {
-            log.faint(`    ${index + 1}. ${commit.hash.slice(0, 7)} (${commit.author_name}) - ${commit.message}`);
+            log.faint(`    ${index + 1}. ${commit.hash.slice(0, 7)} (${getCommitAuthorName(commit)}) - ${commit.message}`);
         });
         if (requiresForcePush) {
             log.info(`\nOnly on ${logColors.bold}${toBranch}${logColors.reset}:`);
             commitsBehind.forEach((commit, index) => {
-                log.faint(`    ${index + 1}. ${commit.hash.slice(0, 7)} (${commit.author_name}) - ${commit.message}`);
+                log.faint(`    ${index + 1}. ${commit.hash.slice(0, 7)} (${getCommitAuthorName(commit)}) - ${commit.message}`);
             });
         }
         const shouldPush = bypassConfirmation ||
@@ -80,7 +82,7 @@ export async function pushDeploy(git, branchConfig, repoConfig, remoteName, noti
                 default: false,
             }));
         if (!shouldPush) {
-            throw new KnownError(`Deploy aborted.`);
+            throw new KnownError('Deploy aborted.');
         }
         if (requiresForcePush) {
             await git.push(remoteName, pushString, ['--force']);

package/dist/index.d.ts

@@ -1,4 +1,5 @@
 export * from './cli/cli.js';
+export * from './commit.js';
 export * from './config.js';
 export * from './git.js';
 export * from './known.error.js';

package/dist/index.js

@@ -1,4 +1,5 @@
 export * from './cli/cli.js';
+export * from './commit.js';
 export * from './config.js';
 export * from './git.js';
 export * from './known.error.js';

package/dist/notifications/slack.js

@@ -1,5 +1,6 @@
 import { addPrefix, log, setFirstLetterCasing, StringCase } from '@augment-vir/common';
 import { joinUrlPaths } from 'url-vir';
+import { getCommitAuthorName } from '../commit.js';
 function truncateString(value, truncateAt = 100) {
     if (value.length > truncateAt) {
         return `${value.slice(0, Math.max(0, truncateAt))}...`;
@@ -10,10 +11,10 @@ function truncateString(value, truncateAt = 100) {
 }
 function formatCommit(baseCommitUrl, commit) {
     if (baseCommitUrl) {
-        return `<${joinUrlPaths(baseCommitUrl, commit.hash)}|${commit.hash.slice(0, 7)}> (${commit.author_name}) ${truncateString(commit.message)}`;
+        return `<${joinUrlPaths(baseCommitUrl, commit.hash)}|${commit.hash.slice(0, 7)}> (${getCommitAuthorName(commit)}) ${truncateString(commit.message)}`;
     }
     else {
-        return `${commit.hash.slice(0, 7)} (${commit.author_name}) ${truncateString(commit.message)}`;
+        return `${commit.hash.slice(0, 7)} (${getCommitAuthorName(commit)}) ${truncateString(commit.message)}`;
     }
 }
 const slackMaxSectionLength = 2500;
@@ -86,7 +87,7 @@ export async function sendNotificationToSlack({ deployResult: { deployCommits: {
             type: 'section',
             text: {
                 type: 'mrkdwn',
-                text: `\n*Deployed Commits*\n`,
+                text: '\n*Deployed Commits*\n',
             },
         },
         ...chunkSectionLines(deployedCommitBulletLines).map((lines) => {
@@ -104,7 +105,7 @@ export async function sendNotificationToSlack({ deployResult: { deployCommits: {
                     type: 'section',
                     text: {
                         type: 'mrkdwn',
-                        text: `\n*Overwritten Commits*\n`,
+                        text: '\n*Overwritten Commits*\n',
                     },
                 },
                 ...chunkSectionLines(overwrittenCommitBulletLines).map((lines) => {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "deploy-vir",
-    "version": "3.4.0",
+    "version": "3.4.2",
     "description": "Deploy a branch to another branch with notifications.",
     "keywords": [
         "deploy",
@@ -15,7 +15,7 @@
         "type": "git",
         "url": "git+https://github.com/electrovir/deploy-vir.git"
     },
-    "license": "(MIT or CC0 1.0)",
+    "license": "(MIT OR CC0-1.0)",
     "author": {
         "name": "electrovir",
         "url": "https://github.com/electrovir"
@@ -32,7 +32,8 @@
         "format": "virmator format",
         "lint": "virmator lint fix",
         "publish": "virmator publish npm run test:all",
-        "test:all": "npm run compile && runstorm --names spelling,format,docs,deps,lint \"npm run test:spelling\" \"npm run test:format\" \"npm run test:docs\" \"npm run test:deps\" \"npm run test:lint\"",
+        "test": "virmator test node",
+        "test:all": "npm run compile && runstorm --names spelling,format,docs,deps,lint,test \"npm run test:spelling\" \"npm run test:format\" \"npm run test:docs\" \"npm run test:deps\" \"npm run test:lint\" \"npm test\"",
         "test:deps": "virmator deps check",
         "test:docs": "virmator docs check",
         "test:format": "virmator format check",
@@ -41,51 +42,52 @@
         "test:update": "npm run test update"
     },
     "dependencies": {
-        "@augment-vir/assert": "^31.68.4",
-        "@augment-vir/common": "^31.68.4",
-        "@augment-vir/node": "^31.68.4",
-        "@inquirer/prompts": "^8.4.2",
-        "@slack/web-api": "^7.15.1",
-        "cli-vir": "^1.0.3",
+        "@augment-vir/assert": "^32.2.1",
+        "@augment-vir/common": "^32.2.1",
+        "@augment-vir/node": "^32.2.1",
+        "@inquirer/prompts": "^8.5.2",
+        "@slack/web-api": "^8.0.0",
+        "cli-vir": "^1.0.4",
         "simple-git": "^3.36.0",
-        "tsx": "^4.21.0",
-        "url-vir": "^2.1.8"
+        "tsx": "^4.23.1",
+        "url-vir": "^2.1.10"
     },
     "devDependencies": {
-        "@eslint/eslintrc": "^3.3.5",
-        "@eslint/js": "^9.39.2",
+        "@augment-vir/test": "^32.2.1",
+        "@eslint/eslintrc": "^3.3.6",
+        "@eslint/js": "^10.0.1",
         "@stylistic/eslint-plugin": "^5.10.0",
         "@stylistic/eslint-plugin-ts": "^4.4.1",
-        "@types/node": "^25.6.0",
-        "@typescript-eslint/eslint-plugin": "^8.59.0",
-        "cspell": "^10.0.0",
-        "dependency-cruiser": "^17.3.10",
-        "esbuild": "^0.28.0",
-        "eslint": "^9.39.2",
+        "@types/node": "^26.1.1",
+        "@typescript-eslint/eslint-plugin": "^8.65.0",
+        "cspell": "^10.0.1",
+        "dependency-cruiser": "^18.1.0",
+        "esbuild": "^0.28.1",
+        "eslint": "^10.8.0",
         "eslint-config-prettier": "^10.1.8",
-        "eslint-plugin-jsdoc": "^62.9.0",
+        "eslint-plugin-jsdoc": "^63.2.2",
         "eslint-plugin-monorepo-cop": "^1.0.2",
-        "eslint-plugin-playwright": "^2.10.2",
-        "eslint-plugin-prettier": "^5.5.5",
+        "eslint-plugin-playwright": "^2.10.5",
+        "eslint-plugin-prettier": "^5.5.6",
         "eslint-plugin-require-extensions": "^0.1.3",
-        "eslint-plugin-sonarjs": "^4.0.3",
-        "eslint-plugin-unicorn": "^64.0.0",
+        "eslint-plugin-sonarjs": "^4.2.0",
+        "eslint-plugin-unicorn": "^72.0.0",
         "istanbul-smart-text-reporter": "^1.1.5",
-        "markdown-code-example-inserter": "^3.0.5",
-        "npm-check-updates": "^21.0.3",
+        "markdown-code-example-inserter": "^3.0.6",
+        "npm-check-updates": "^22.2.9",
         "prettier": "~3.3.3",
-        "prettier-plugin-interpolated-html-tags": "^2.0.1",
-        "prettier-plugin-jsdoc": "^1.8.0",
-        "prettier-plugin-multiline-arrays": "^4.1.7",
+        "prettier-plugin-interpolated-html-tags": "^2.0.2",
+        "prettier-plugin-jsdoc": "^1.8.1",
+        "prettier-plugin-multiline-arrays": "^4.1.11",
         "prettier-plugin-organize-imports": "^4.3.0",
         "prettier-plugin-packagejson": "^3.0.2",
         "prettier-plugin-sort-json": "^4.2.0",
         "prettier-plugin-toml": "^2.0.6",
-        "runstorm": "^1.0.0",
-        "typedoc": "^0.28.19",
-        "typescript": "^5.9.3",
-        "typescript-eslint": "^8.57.1",
-        "virmator": "^14.12.2"
+        "runstorm": "^1.0.6",
+        "typedoc": "^0.28.20",
+        "typescript": "^6.0.3",
+        "typescript-eslint": "^8.65.0",
+        "virmator": "^14.30.1"
     },
     "engines": {
         "node": ">=22"

package/src/commit.ts

@@ -0,0 +1,33 @@
+import {type DefaultLogFields, type ListLogLine} from 'simple-git';
+
+/**
+ * A git commit.
+ *
+ * @category Internal
+ */
+export type Commit = DefaultLogFields & ListLogLine;
+
+const coAuthorRegExp = /^co[- \t]*authored[- \t]*by:(.*)$/gim;
+const botNameSuffix = '[bot]';
+
+/**
+ * Extract the human author name for a commit. Commits authored by a bot (a name with a `[bot]`
+ * suffix, like GitHub's merge queue bot) are attributed to their first non-bot `Co-authored-by:`
+ * trailer instead.
+ *
+ * @category Internal
+ */
+export function getCommitAuthorName(
+    commit: Readonly<Pick<Commit, 'author_name' | 'body'>>,
+): string {
+    if (!commit.author_name.endsWith(botNameSuffix)) {
+        return commit.author_name;
+    }
+
+    const coAuthorName = Array.from(commit.body.matchAll(coAuthorRegExp))
+        /** Strip the trailing `<email>` from each trailer. */
+        .map((match) => (match[1] || '').split('<')[0]?.trim())
+        .find((name) => name && !name.endsWith(botNameSuffix));
+
+    return coAuthorName || commit.author_name;
+}

package/src/config.ts

@@ -1,5 +1,6 @@
 import {type MaybePromise, type PartialWithUndefined} from '@augment-vir/common';
-import {type Commit, type DeployResult} from './git.js';
+import {type Commit} from './commit.js';
+import {type DeployResult} from './git.js';
 
 /**
  * The full deploy-vir config object.

package/src/git.ts

@@ -1,7 +1,8 @@
 import {assert, assertWrap, check} from '@augment-vir/assert';
 import {awaitedBlockingMap, log, logColors} from '@augment-vir/common';
 import {confirm} from '@inquirer/prompts';
-import {type DefaultLogFields, type ListLogLine, type SimpleGit} from 'simple-git';
+import {type SimpleGit} from 'simple-git';
+import {getCommitAuthorName, type Commit} from './commit.js';
 import {
     type DeployNotificationConfig,
     type DeployVirBranchConfig,
@@ -52,13 +53,6 @@ export async function getGitRemoteName(
     return newRemoteName;
 }
 
-/**
- * A git commit.
- *
- * @category Internal
- */
-export type Commit = DefaultLogFields & ListLogLine;
-
 /**
  * All commits involved in a deploy.
  *
@@ -94,6 +88,7 @@ export type DeployResult = {
  *
  * @category Internal
  */
+// eslint-disable-next-line @virmator/prefer-params-object
 export async function pushDeploy(
     git: Readonly<SimpleGit>,
     branchConfig: Readonly<DeployVirBranchConfig>,
@@ -164,7 +159,7 @@ export async function pushDeploy(
             );
             commitsAhead.forEach((commit, index) => {
                 log.faint(
-                    `    ${index + 1}. ${commit.hash.slice(0, 7)} (${commit.author_name}) - ${commit.message}`,
+                    `    ${index + 1}. ${commit.hash.slice(0, 7)} (${getCommitAuthorName(commit)}) - ${commit.message}`,
                 );
             });
 
@@ -172,7 +167,7 @@ export async function pushDeploy(
                 log.info(`\nOnly on ${logColors.bold}${toBranch}${logColors.reset}:`);
                 commitsBehind.forEach((commit, index) => {
                     log.faint(
-                        `    ${index + 1}. ${commit.hash.slice(0, 7)} (${commit.author_name}) - ${commit.message}`,
+                        `    ${index + 1}. ${commit.hash.slice(0, 7)} (${getCommitAuthorName(commit)}) - ${commit.message}`,
                     );
                 });
             }
@@ -187,7 +182,7 @@ export async function pushDeploy(
                 }));
 
             if (!shouldPush) {
-                throw new KnownError(`Deploy aborted.`);
+                throw new KnownError('Deploy aborted.');
             }
 
             if (requiresForcePush) {

package/src/index.ts

@@ -1,4 +1,5 @@
 export * from './cli/cli.js';
+export * from './commit.js';
 export * from './config.js';
 export * from './git.js';
 export * from './known.error.js';

package/src/notifications/slack.ts

@@ -1,8 +1,8 @@
 import {addPrefix, log, setFirstLetterCasing, StringCase} from '@augment-vir/common';
 import {type ChatPostMessageArguments, type KnownBlock} from '@slack/web-api';
 import {joinUrlPaths} from 'url-vir';
+import {getCommitAuthorName, type Commit} from '../commit.js';
 import {type SlackNotificationConfig} from '../config.js';
-import {type Commit} from '../git.js';
 import {type NotificationParams} from './notification-params.js';
 
 export {type ChatPostMessageArguments} from '@slack/web-api';
@@ -17,9 +17,9 @@ function truncateString(value: string, truncateAt = 100): string {
 
 function formatCommit(baseCommitUrl: string | undefined, commit: Readonly<Commit>) {
     if (baseCommitUrl) {
-        return `<${joinUrlPaths(baseCommitUrl, commit.hash)}|${commit.hash.slice(0, 7)}> (${commit.author_name}) ${truncateString(commit.message)}`;
+        return `<${joinUrlPaths(baseCommitUrl, commit.hash)}|${commit.hash.slice(0, 7)}> (${getCommitAuthorName(commit)}) ${truncateString(commit.message)}`;
     } else {
-        return `${commit.hash.slice(0, 7)} (${commit.author_name}) ${truncateString(commit.message)}`;
+        return `${commit.hash.slice(0, 7)} (${getCommitAuthorName(commit)}) ${truncateString(commit.message)}`;
     }
 }
 
@@ -117,7 +117,7 @@ export async function sendNotificationToSlack({
             type: 'section',
             text: {
                 type: 'mrkdwn',
-                text: `\n*Deployed Commits*\n`,
+                text: '\n*Deployed Commits*\n',
             },
         },
         ...chunkSectionLines(deployedCommitBulletLines).map((lines): KnownBlock => {
@@ -135,7 +135,7 @@ export async function sendNotificationToSlack({
                       type: 'section',
                       text: {
                           type: 'mrkdwn',
-                          text: `\n*Overwritten Commits*\n`,
+                          text: '\n*Overwritten Commits*\n',
                       },
                   },