feat: image_72 fallbacks

Author: coltenkrauter

dist/index.js

@@ -80746,10 +80746,10 @@ __nccwpck_require__.a(module, async (__webpack_handle_async_dependencies__, __we
 /* harmony import */ var _krauters_utils__WEBPACK_IMPORTED_MODULE_2___default = /*#__PURE__*/__nccwpck_require__.n(_krauters_utils__WEBPACK_IMPORTED_MODULE_2__);
 /* harmony import */ var _package_json__WEBPACK_IMPORTED_MODULE_3__ = __nccwpck_require__(8330);
 /* harmony import */ var _constants_js__WEBPACK_IMPORTED_MODULE_4__ = __nccwpck_require__(27242);
-/* harmony import */ var _utils_github_github_client_js__WEBPACK_IMPORTED_MODULE_5__ = __nccwpck_require__(32031);
+/* harmony import */ var _utils_github_client_js__WEBPACK_IMPORTED_MODULE_5__ = __nccwpck_require__(10598);
 /* harmony import */ var _utils_github_structures_js__WEBPACK_IMPORTED_MODULE_6__ = __nccwpck_require__(28315);
 /* harmony import */ var _utils_slack_blocks_js__WEBPACK_IMPORTED_MODULE_7__ = __nccwpck_require__(96114);
-/* harmony import */ var _utils_slack_slack_client_js__WEBPACK_IMPORTED_MODULE_8__ = __nccwpck_require__(70741);
+/* harmony import */ var _utils_slack_client_js__WEBPACK_IMPORTED_MODULE_8__ = __nccwpck_require__(90294);
 /* harmony import */ var _utils_test_data_js__WEBPACK_IMPORTED_MODULE_9__ = __nccwpck_require__(23930);
 /* harmony import */ var _input_parser_js__WEBPACK_IMPORTED_MODULE_10__ = __nccwpck_require__(40935);
 // https://github.com/actions/github-script
@@ -80772,12 +80772,12 @@ async function main() {
     try {
         (0,_actions_core__WEBPACK_IMPORTED_MODULE_1__.debug)('Starting main...');
         const { githubConfig, repositoryFilter, slackConfig, withArchived, withDrafts, withPublic, withTestData, withUserMentions, } = (0,_input_parser_js__WEBPACK_IMPORTED_MODULE_10__/* .parseInputs */ .T)();
-        const slack = new _utils_slack_slack_client_js__WEBPACK_IMPORTED_MODULE_8__/* .SlackClient */ .Q(slackConfig);
+        const slack = new _utils_slack_client_js__WEBPACK_IMPORTED_MODULE_8__/* .SlackClient */ .Q(slackConfig);
         const results = await githubConfig.tokens.reduce(async (accPromise, token) => {
             const acc = await accPromise;
             try {
                 // TODO - Consider making this thread safe so requests can be made in parallel
-                const client = new _utils_github_github_client_js__WEBPACK_IMPORTED_MODULE_5__/* .GitHubClient */ .j({
+                const client = new _utils_github_client_js__WEBPACK_IMPORTED_MODULE_5__/* .GitHubClient */ .j({
                     options: githubConfig.options,
                     token,
                 });
@@ -80933,7 +80933,7 @@ function parseInputs() {
 
 /***/ }),
 
-/***/ 32031:
+/***/ 10598:
 /***/ ((__unused_webpack_module, __webpack_exports__, __nccwpck_require__) => {
 
 /* harmony export */ __nccwpck_require__.d(__webpack_exports__, {
@@ -81376,7 +81376,7 @@ var PullState;
 /**
  * Get an emoji based on the age of something.
  *
- * @param {number} hoursAgo - The number of hours ago since something happened.
+ * @param {number} hoursAgo The number of hours ago since something happened.
  * @returns {string}
  */
 function getAgeBasedEmoji(hoursAgo) {
@@ -81391,8 +81391,8 @@ function getAgeBasedEmoji(hoursAgo) {
 /**
  * Get the relative human readable age of something.
  *
- * @param {number} hoursAgo - The number of hours ago since something happened.
- * @param {boolean} [withAgo=true] - Whether to include the ' ago' suffix when appropriate.
+ * @param {number} hoursAgo The number of hours ago since something happened.
+ * @param {boolean} [withAgo=true] Whether to include the ' ago' suffix when appropriate.
  * @returns {string}
  */
 function getRelativeHumanReadableAge(hoursAgo, withAgo = true) {
@@ -81409,7 +81409,7 @@ function getRelativeHumanReadableAge(hoursAgo, withAgo = true) {
 }
 /**
  * Get have or has depending on quantity context.
- * @param {number} number - The number of entities in question.
+ * @param {number} number The number of entities in question.
  * @returns {string}
  */
 function haveOrHas(number) {
@@ -81588,9 +81588,13 @@ async function getPullBlocks(pull, slack, withUserMentions) {
             email,
             username,
         });
-        // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
         const displayName = slackUser?.profile?.display_name || slackUser?.profile?.real_name_normalized || username;
-        const imageUrl = slackUser?.profile?.image_72;
+        const imageUrl = slackUser?.profile?.image_512 ||
+            slackUser?.profile?.image_192 ||
+            slackUser?.profile?.image_72 ||
+            slackUser?.profile?.image_48 ||
+            slackUser?.profile?.image_32 ||
+            slackUser?.profile?.image_24;
         activityBlocks.push({
             elements: [
                 {
@@ -81614,7 +81618,6 @@ async function getPullBlocks(pull, slack, withUserMentions) {
         const slackUserIdsOrLogins = [];
         for (const username of requestedReviewers) {
             const slackUser = await slack.getSlackUser({ username });
-            // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
             slackUserIdsOrLogins.push((withUserMentions && slackUser?.id && `<@${slackUser.id}>`) || username);
         }
         activityBlocks.unshift(...getContextMarkdownBlock((0,_krauters_utils__WEBPACK_IMPORTED_MODULE_0__.formatStringList)(slackUserIdsOrLogins) +
@@ -81660,7 +81663,7 @@ async function getPullBlocks(pull, slack, withUserMentions) {
 
 /***/ }),
 
-/***/ 70741:
+/***/ 90294:
 /***/ ((__unused_webpack_module, __webpack_exports__, __nccwpck_require__) => {
 
 
@@ -81680,7 +81683,7 @@ var SlackAppUrl;
     SlackAppUrl["SuffixDisplayInfo"] = "general#display_info_form";
 })(SlackAppUrl || (SlackAppUrl = {}));
 
-;// CONCATENATED MODULE: ./src/utils/slack/slack-client.ts
+;// CONCATENATED MODULE: ./src/utils/slack/client.ts
 
 
 
@@ -81762,24 +81765,36 @@ class SlackClient {
     async getSlackUser({ email, userId, username }) {
         console.log(`Getting Slack UserId for email [${email}], username [${username}], and userId [${userId}]...`);
         const users = this.users ?? (await this.getAllusers());
+        // Define matching functions for better readability and extensibility
+        const matchById = (user) => userId && user.id === userId;
+        const matchByEmail = (user) => email && user.profile?.email === email;
+        const matchByEmailContainsUsername = (user) => username && String(user.profile?.email ?? '').includes(username);
+        const matchByDisplayName = (user) => username && user.profile?.display_name === username;
+        const matchByRealName = (user) => username && user.profile?.real_name === username;
         const user = users.find((user) => {
-            if (userId) {
-                return user?.id === userId;
-            }
-            const profile = user.profile;
             return (
             // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
-            (email && profile?.email === email) ||
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-                String(profile?.email).includes(username) ||
-                profile?.display_name === username ||
-                profile?.real_name === username);
+            matchById(user) ||
+                // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
+                matchByEmail(user) ||
+                // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
+                matchByEmailContainsUsername(user) ||
+                // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
+                matchByDisplayName(user) ||
+                matchByRealName(user));
         });
         if (user) {
             console.log(`User found with userId [${user.id}]`);
             return user;
         }
-        console.log('User not found, returning undefined');
+        console.log(`No user match found after checking [${users.length}] users with criteria:`);
+        if (userId)
+            console.log(`- user id [${userId}]`);
+        if (email)
+            console.log(`- email [${email}]`);
+        if (username)
+            console.log(`- username [${username}] (checking display name, real name, and email)`);
+        return undefined;
     }
     /**
      * Post a message with blocks to Slack channels.
@@ -90365,7 +90380,7 @@ module.exports = {"version":"3.17.0"};
 /***/ 8330:
 /***/ ((module) => {
 
-module.exports = /*#__PURE__*/JSON.parse('{"UU":"@krauters/github-notifier","rE":"1.1.0","TB":"https://buymeacoffee.com/coltenkrauter"}');
+module.exports = /*#__PURE__*/JSON.parse('{"UU":"@krauters/github-notifier","rE":"1.2.0","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.1.0",
+  "version": "1.2.0",
   "author": "Colten Krauter <coltenkrauter>",
   "type": "module",
   "homepage": "https://buymeacoffee.com/coltenkrauter",

package-lock.json

@@ -8,11 +8,11 @@ import { formatStringList, plural } from '@krauters/utils'
 
 import pkg from '../package.json' with { type: 'json' }
 import { workflowLogsUrl, workflowUrl } from './constants.js'
-import { GitHubClient } from './utils/github/github-client.js'
+import { GitHubClient } from './utils/github/client.js'
 import type { Pull } from './utils/github/structures.js'
 import { PullState, RepositoryType } from './utils/github/structures.js'
 import { getFirstBlocks, getLastBlocks, getPullBlocks } from './utils/slack/blocks.js'
-import { SlackClient } from './utils/slack/slack-client.js'
+import { SlackClient } from './utils/slack/client.js'
 import { getApprovedPullRequest } from './utils/test-data.js'
 import { parseInputs as getInputs } from './input-parser.js'
 

package.json

@@ -3,7 +3,7 @@ import { plural } from '@krauters/utils'
 /**
  * Get an emoji based on the age of something.
  *
- * @param {number} hoursAgo - The number of hours ago since something happened.
+ * @param {number} hoursAgo The number of hours ago since something happened.
  * @returns {string}
  */
 export function getAgeBasedEmoji(hoursAgo: number): string {
@@ -22,8 +22,8 @@ export function getAgeBasedEmoji(hoursAgo: number): string {
 /**
  * Get the relative human readable age of something.
  *
- * @param {number} hoursAgo - The number of hours ago since something happened.
- * @param {boolean} [withAgo=true] - Whether to include the ' ago' suffix when appropriate.
+ * @param {number} hoursAgo The number of hours ago since something happened.
+ * @param {boolean} [withAgo=true] Whether to include the ' ago' suffix when appropriate.
  * @returns {string}
  */
 export function getRelativeHumanReadableAge(hoursAgo: number, withAgo = true): string {
@@ -41,7 +41,7 @@ export function getRelativeHumanReadableAge(hoursAgo: number, withAgo = true): s
 
 /**
  * Get have or has depending on quantity context.
- * @param {number} number - The number of entities in question.
+ * @param {number} number The number of entities in question.
  * @returns {string}
  */
 export function haveOrHas(number: number): string {

src/app.ts

@@ -1,11 +1,12 @@
 /* eslint-disable @typescript-eslint/naming-convention */
+/* eslint-disable @typescript-eslint/prefer-nullish-coalescing */
 import type { KnownBlock, PlainTextElement, RichTextElement } from '@slack/web-api'
 import type { Button } from '@slack/types'
 
 import { capitalize, formatStringList, plural } from '@krauters/utils'
 
 import type { Pull } from '../github/structures.js'
-import type { SlackClient } from './slack-client.js'
+import type { SlackClient } from './client.js'
 
 import { prBaseUrl, scmUrl } from '../../constants.js'
 import { getAgeBasedEmoji, haveOrHas } from '../misc.js'
@@ -195,9 +196,14 @@ export async function getPullBlocks(pull: Pull, slack: SlackClient, withUserMent
 			username,
 		})
 
-		// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
 		const displayName = slackUser?.profile?.display_name || slackUser?.profile?.real_name_normalized || username
-		const imageUrl = slackUser?.profile?.image_72
+		const imageUrl =
+			slackUser?.profile?.image_512 ||
+			slackUser?.profile?.image_192 ||
+			slackUser?.profile?.image_72 ||
+			slackUser?.profile?.image_48 ||
+			slackUser?.profile?.image_32 ||
+			slackUser?.profile?.image_24
 
 		activityBlocks.push({
 			elements: [
@@ -223,7 +229,6 @@ export async function getPullBlocks(pull: Pull, slack: SlackClient, withUserMent
 		const slackUserIdsOrLogins: string[] = []
 		for (const username of requestedReviewers) {
 			const slackUser = await slack.getSlackUser({ username })
-			// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
 			slackUserIdsOrLogins.push((withUserMentions && slackUser?.id && `<@${slackUser.id}>`) || username)
 		}
 		activityBlocks.unshift(

src/utils/github/github-client.ts src/utils/github/client.tssrc/utils/github/github-client.ts renamed to src/utils/github/client.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 import type { Block, BotsInfoResponse } from '@slack/web-api'
 import type { Bot } from '@slack/web-api/dist/types/response/BotsInfoResponse.js'
-import type { Member, Profile } from '@slack/web-api/dist/types/response/UsersListResponse.js'
+import type { Member } from '@slack/web-api/dist/types/response/UsersListResponse.js'
 import type { User } from '@slack/web-api/dist/types/response/UsersLookupByEmailResponse.js'
 
 import { getBatches } from '@krauters/utils'
@@ -104,20 +104,26 @@ export class SlackClient {
 		console.log(`Getting Slack UserId for email [${email}], username [${username}], and userId [${userId}]...`)
 
 		const users = this.users ?? (await this.getAllusers())
-		const user = users.find((user: Member) => {
-			if (userId) {
-				return user?.id === userId
-			}
 
-			const profile: Profile | undefined = user.profile
+		// Define matching functions for better readability and extensibility
+		const matchById = (user: Member) => userId && user.id === userId
+		const matchByEmail = (user: Member) => email && user.profile?.email === email
+		const matchByEmailContainsUsername = (user: Member) =>
+			username && String(user.profile?.email ?? '').includes(username)
+		const matchByDisplayName = (user: Member) => username && user.profile?.display_name === username
+		const matchByRealName = (user: Member) => username && user.profile?.real_name === username
 
+		const user = users.find((user: Member) => {
 			return (
 				// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
-				(email && profile?.email === email) ||
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				String(profile?.email).includes(username!) ||
-				profile?.display_name === username ||
-				profile?.real_name === username
+				matchById(user) ||
+				// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
+				matchByEmail(user) ||
+				// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
+				matchByEmailContainsUsername(user) ||
+				// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
+				matchByDisplayName(user) ||
+				matchByRealName(user)
 			)
 		})
 
@@ -127,7 +133,12 @@ export class SlackClient {
 			return user
 		}
 
-		console.log('User not found, returning undefined')
+		console.log(`No user match found after checking [${users.length}] users with criteria:`)
+		if (userId) console.log(`- user id [${userId}]`)
+		if (email) console.log(`- email [${email}]`)
+		if (username) console.log(`- username [${username}] (checking display name, real name, and email)`)
+
+		return undefined
 	}
 
 	/**

src/utils/misc.ts

@@ -1,2 +1,2 @@
 export * from './blocks.js'
-export * from './slack-client.js'
+export * from './client.js'