fix: logging

Author: coltenkrauter

dist/index.js

@@ -80770,7 +80770,7 @@ const { /* homepage */ "TB": homepage, /* name */ "UU": name, /* version */ "rE"
  */
 async function main() {
     try {
-        (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)('Starting main...');
+        console.log(`Starting GitHub Notifier version [${version}]...`);
         const { githubConfig, repositoryFilter, slackConfig, withArchived, withDrafts, withPublic, withTestData, withUserMentions, } = (0,_input_parser_js__WEBPACK_IMPORTED_MODULE_10__/* .parseInputs */ .T)();
         const slack = new _utils_slack_client_js__WEBPACK_IMPORTED_MODULE_8__/* .SlackClient */ .Q(slackConfig);
         const results = await githubConfig.tokens.reduce(async (accPromise, token) => {
@@ -80789,7 +80789,7 @@ async function main() {
                 });
                 const org = await client.getOrg();
                 const pulls = await client.getPulls({ repositories, state: _utils_github_structures_js__WEBPACK_IMPORTED_MODULE_6__/* .PullState */ .lT.Open, withDrafts });
-                console.log(`Found [${pulls.length}] pulls for [${org.name}]`);
+                console.log(`Found [${pulls.length}] ${(0,_krauters_utils__WEBPACK_IMPORTED_MODULE_2__.plural)('pull', pulls.length)} for [${org.name}]`);
                 return [...acc, { client, org: org.name, pulls }];
             }
             catch (error) {
@@ -80808,6 +80808,7 @@ async function main() {
         const dedupedPulls = [...new Map(pulls.map((pull) => [`${pull.org}/${pull.repo}/${pull.number}`, pull]))].map(
         // eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-unused-vars
         ([_, pull]) => pull);
+        console.log(`After deduplication, there are [${dedupedPulls.length}] unique pull requests`);
         let blocks = [];
         for (const pull of dedupedPulls) {
             (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Building Slack blocks from pull request [${pull.number}]`);
@@ -80842,7 +80843,9 @@ async function main() {
                 `/${_actions_github__WEBPACK_IMPORTED_MODULE_0__.context.payload.repository?.name}> (<${_constants_js__WEBPACK_IMPORTED_MODULE_4__/* .workflowLogsUrl */ .GM}|logs>) using <${homepage}|${name}>@<${homepage}/releases/tag/${version}|${version}>`,
             ].join('')),
         ];
+        console.log(`Posting [${blocks.length}] Slack blocks to [${slack.channels.length}] channels`);
         await slack.postMessage(header, blocks);
+        console.log('Successfully posted notification to Slack');
     }
     catch (error) {
         console.error(`Fatal error [${error}]`);
@@ -81092,22 +81095,25 @@ class GitHubClient {
      * @param props Configuration for retrieving pull requests.
      */
     async getPulls({ oldest = (0,_krauters_utils__WEBPACK_IMPORTED_MODULE_2__.snapDate)(new Date(), { months: -36, snap: _krauters_utils__WEBPACK_IMPORTED_MODULE_2__.SnapType.Month }), onlyGhReviews = false, repositories, state = _structures_js__WEBPACK_IMPORTED_MODULE_0__/* .PullState */ .lT.All, withCommits = true, withDrafts, withFilesAndChanges = true, withUser = true, }) {
+        const startTime = Date.now();
         const org = await this.getOrgName();
-        console.log('\n');
         if (state === _structures_js__WEBPACK_IMPORTED_MODULE_0__/* .PullState */ .lT.Open) {
-            console.log(`Getting [${state}] pulls in org [${org}]...`);
+            (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Getting [${state}] ${(0,_krauters_utils__WEBPACK_IMPORTED_MODULE_2__.plural)('pull', 2)} in org [${org}]...`);
         }
         else {
-            console.log(`Getting [${state}] pulls in org [${org}] that are newer than [${oldest}...`);
+            (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Getting [${state}] ${(0,_krauters_utils__WEBPACK_IMPORTED_MODULE_2__.plural)('pull', 2)} in org [${org}] that are newer than [${oldest}]...`);
         }
         const pullRequests = [];
+        let apiRequestCount = 0;
         for (const repo of repositories) {
             (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Getting [${state}] pulls in repository [${repo.name}]...`);
+            const repoStartTime = Date.now();
             const pulls = await this.client.paginate(this.client.rest.pulls.list, {
                 owner: org,
                 repo: repo.name,
                 state,
             }, (response, done) => {
+                apiRequestCount++;
                 (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Paginated response for repository [${repo.name}], status [${response.status}], items [${response.data.length}]`);
                 const found = response.data.find((pull) => new Date(pull.created_at) < oldest);
                 if (found && state !== _structures_js__WEBPACK_IMPORTED_MODULE_0__/* .PullState */ .lT.Open) {
@@ -81116,7 +81122,8 @@ class GitHubClient {
                 }
                 return response.data.filter((pull) => new Date(pull.created_at) > oldest);
             });
-            console.log(`Found [${pulls.length}] pull in repository [${repo.name}]`);
+            const repoDuration = (Date.now() - repoStartTime) / 1000;
+            console.log(`Found [${pulls.length}] pull in repository [${repo.name}] in [${repoDuration.toFixed(2)}] seconds`);
             for (const pull of pulls) {
                 const { base, closed_at, created_at, draft, html_url, merged_at, number, title, user } = pull;
                 (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Processing pull [${number}]...`);
@@ -81155,6 +81162,8 @@ class GitHubClient {
                 (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)(`Added pull [${number}] to response`);
             }
         }
+        const totalDuration = (Date.now() - startTime) / 1000;
+        console.log(`Completed processing [${pullRequests.length}] pull requests in [${totalDuration.toFixed(2)}] seconds with [${apiRequestCount}] API calls`);
         return pullRequests;
     }
     /**
@@ -81832,6 +81841,7 @@ class SlackClient {
         this.client = new dist.WebClient(token);
         this.channels = channels;
         this.userMappings = userMappings;
+        console.log(`Slack client initialized with [${channels.length}] channels and [${userMappings.length}] user mappings`);
     }
     /**
      * Ensure app name pattern.
@@ -81848,6 +81858,7 @@ class SlackClient {
             throw new Error(`Current app name [${name}] does not match the desired pattern [${pattern}]. Please update Slack app "Basic Information" > "Display Information" > "App name" to have the suffix "GitHub Notifier". Slack app url [${SlackAppUrl.Prefix}/${info?.bot?.app_id}/${SlackAppUrl.SuffixDisplayInfo}].`);
         }
         this.bot = info.bot;
+        console.log(`Verified Slack app name: [${name}]`);
     }
     /**
      * Get all Slack users.
@@ -81856,6 +81867,7 @@ class SlackClient {
         this.users = [];
         let cursor;
         console.log('Getting all Slack users...');
+        const startTime = Date.now();
         try {
             // Keep paginating until no more cursor is returned
             do {
@@ -81866,7 +81878,8 @@ class SlackClient {
                 cursor = result.response_metadata?.next_cursor;
                 (0,core.debug)(`Got [${result.members?.length}] users from Slack`);
             } while (cursor);
-            console.log(`Got a total of [${this.users.length}] active users from Slack`);
+            const duration = (Date.now() - startTime) / 1000;
+            console.log(`Got a total of [${this.users.length}] active users from Slack in [${duration.toFixed(2)}] seconds`);
             return this.users;
         }
         catch (error) {
@@ -81879,6 +81892,7 @@ class SlackClient {
      */
     async getBotInfo() {
         try {
+            console.log('Getting Slack bot information...');
             const authResponse = await this.client.auth.test({});
             const response = await this.client.bots.info({ bot: authResponse.bot_id });
             if (!response.ok) {
@@ -81928,10 +81942,16 @@ class SlackClient {
      * @param [channels=this.channels] Channels to post to.
      */
     async postMessage(text, blocks, channels = this.channels) {
+        const startTime = Date.now();
+        const totalBlocks = blocks.length;
+        const batches = (0,src.getBatches)(blocks);
+        (0,core.debug)(`Preparing to post [${totalBlocks}] blocks to [${channels.length}] channels`);
         for (const channel of channels) {
             let batchNumber = 0;
-            for (const batch of (0,src.getBatches)(blocks)) {
-                console.log(`Posting batch [${batchNumber++}] to Slack channel [${channel}]...`);
+            for (const batch of batches) {
+                batchNumber++;
+                const batchStartTime = Date.now();
+                (0,core.debug)(`Posting batch [${batchNumber}] to Slack channel [${channel}]...`);
                 const payload = {
                     blocks: batch.items,
                     channel,
@@ -81945,9 +81965,12 @@ class SlackClient {
                 (0,core.debug)(JSON.stringify(payload, null, 2));
                 const response = await this.client.chat.postMessage(payload);
                 (0,core.debug)(JSON.stringify(response, null, 2));
-                console.log(`Posted batch [${batchNumber++}] to Slack channel [${channel}] with success [${response.ok}]`);
+                const batchDuration = (Date.now() - batchStartTime) / 1000;
+                console.log(`Posted batch [${batchNumber}] to Slack channel [${channel}] with success [${response.ok}] in [${batchDuration.toFixed(2)}] seconds`);
             }
         }
+        const totalDuration = (Date.now() - startTime) / 1000;
+        console.log(`Completed posting all messages to Slack in [${totalDuration.toFixed(2)}] seconds`);
     }
 }
 
@@ -90605,7 +90628,7 @@ module.exports = {"version":"3.17.0"};
 /***/ 8330:
 /***/ ((module) => {
 
-module.exports = /*#__PURE__*/JSON.parse('{"UU":"@krauters/github-notifier","rE":"1.3.0","TB":"https://buymeacoffee.com/coltenkrauter"}');
+module.exports = /*#__PURE__*/JSON.parse('{"UU":"@krauters/github-notifier","rE":"1.3.1","TB":"https://buymeacoffee.com/coltenkrauter"}');
 
 /***/ })
 

dist/index.js.map

@@ -1,7 +1,7 @@
 {
   "name": "@krauters/github-notifier",
   "description": "GitHub Notifier by Krauters – Post Open Pull Requests to Slack",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "author": "Colten Krauter <coltenkrauter>",
   "type": "module",
   "homepage": "https://buymeacoffee.com/coltenkrauter",
@@ -47,7 +47,7 @@
   "devDependencies": {
     "@krauters/eslint-config": "^1.8.0",
     "@types/jest": "^29.5.14",
-    "@types/node": "^22.14.1",
+    "@types/node": "^22.15.0",
     "@vercel/ncc": "^0.38.3",
     "husky": "9.1.7",
     "jest": "^29.7.0",

package-lock.json

@@ -23,7 +23,8 @@ const { homepage, name, version } = pkg
  */
 async function main(): Promise<void> {
 	try {
-		debug('Starting main...')
+		console.log(`Starting GitHub Notifier version [${version}]...`)
+
 		const {
 			githubConfig,
 			repositoryFilter,
@@ -55,7 +56,7 @@ async function main(): Promise<void> {
 
 					const org = await client.getOrg()
 					const pulls = await client.getPulls({ repositories, state: PullState.Open, withDrafts })
-					console.log(`Found [${pulls.length}] pulls for [${org.name}]`)
+					console.log(`Found [${pulls.length}] ${plural('pull', pulls.length)} for [${org.name}]`)
 
 					return [...acc, { client, org: org.name, pulls }]
 				} catch (error: unknown) {
@@ -85,6 +86,7 @@ async function main(): Promise<void> {
 			// eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-unused-vars
 			([_, pull]) => pull,
 		)
+		console.log(`After deduplication, there are [${dedupedPulls.length}] unique pull requests`)
 
 		let blocks: KnownBlock[] = []
 		for (const pull of dedupedPulls) {
@@ -130,7 +132,9 @@ async function main(): Promise<void> {
 			),
 		]
 
+		console.log(`Posting [${blocks.length}] Slack blocks to [${slack.channels.length}] channels`)
 		await slack.postMessage(header, blocks)
+		console.log('Successfully posted notification to Slack')
 	} catch (error) {
 		console.error(`Fatal error [${error}]`)
 		process.exit(1)

package.json

@@ -27,7 +27,7 @@ import {
 	reviewText,
 } from './structures.js'
 import { debug } from '@actions/core'
-import { average, getHoursAgo, minutesBetweenDates, snapDate, SnapType } from '@krauters/utils'
+import { average, getHoursAgo, minutesBetweenDates, plural, snapDate, SnapType } from '@krauters/utils'
 import { ignoreFilenamesForChanges } from '../../constants.js'
 import { getRelativeHumanReadableAge } from '../misc.js'
 
@@ -205,17 +205,19 @@ export class GitHubClient {
 		withFilesAndChanges = true,
 		withUser = true,
 	}: GetPullsProps): Promise<Pull[]> {
+		const startTime = Date.now()
 		const org = await this.getOrgName()
-		console.log('\n')
 		if (state === PullState.Open) {
-			console.log(`Getting [${state}] pulls in org [${org}]...`)
+			debug(`Getting [${state}] ${plural('pull', 2)} in org [${org}]...`)
 		} else {
-			console.log(`Getting [${state}] pulls in org [${org}] that are newer than [${oldest}...`)
+			debug(`Getting [${state}] ${plural('pull', 2)} in org [${org}] that are newer than [${oldest}]...`)
 		}
 
 		const pullRequests = []
+		let apiRequestCount = 0
 		for (const repo of repositories) {
 			debug(`Getting [${state}] pulls in repository [${repo.name}]...`)
+			const repoStartTime = Date.now()
 			const pulls = await this.client.paginate(
 				this.client.rest.pulls.list,
 				{
@@ -224,6 +226,7 @@ export class GitHubClient {
 					state,
 				},
 				(response, done) => {
+					apiRequestCount++
 					debug(
 						`Paginated response for repository [${repo.name}], status [${response.status}], items [${response.data.length}]`,
 					)
@@ -237,7 +240,10 @@ export class GitHubClient {
 				},
 			)
 
-			console.log(`Found [${pulls.length}] pull in repository [${repo.name}]`)
+			const repoDuration = (Date.now() - repoStartTime) / 1000
+			console.log(
+				`Found [${pulls.length}] pull in repository [${repo.name}] in [${repoDuration.toFixed(2)}] seconds`,
+			)
 			for (const pull of pulls) {
 				const { base, closed_at, created_at, draft, html_url, merged_at, number, title, user } = pull
 				debug(`Processing pull [${number}]...`)
@@ -286,6 +292,11 @@ export class GitHubClient {
 			}
 		}
 
+		const totalDuration = (Date.now() - startTime) / 1000
+		console.log(
+			`Completed processing [${pullRequests.length}] pull requests in [${totalDuration.toFixed(2)}] seconds with [${apiRequestCount}] API calls`,
+		)
+
 		return pullRequests
 	}