fix(action): Change the way some fields are populated in the slack...

src/commonMain/kotlin/com/monta/slack/notifier/SlackClient.kt

@@ -103,11 +103,11 @@ class SlackClient(
                         ),
                         SlackBlock.Text(
                             type = "mrkdwn",
-                            text = " \n*Message:*\n<${githubEvent.getCommitUrl()}|${githubEvent.getCommitMessage()}>"
+                            text = " \n*Message:*\n<${githubEvent.getChangeUrl()}|${githubEvent.getChangeMessage()}>"
                         ),
                         SlackBlock.Text(
                             type = "mrkdwn",
-                            text = " \n*SHA:*\n<${githubEvent.getCommitUrl()}|${githubEvent.commitSHA}>"
+                            text = " \n*Change:*\n<${githubEvent.getChangeUrl()}|${githubEvent.getChangeIdentifier()}>"
                         )
                     )
                 ),

src/commonMain/kotlin/com/monta/slack/notifier/command/PublishSlackCommand.kt

@@ -105,8 +105,9 @@ class PublishSlackCommand : CliktCommand() {
             runId = githubRunId,
             displayName = baseGithubContext.displayName,
             commitSHA = baseGithubContext.sha,
-            commitMessage = baseGithubContext.message,
-            workflow = githubWorkflow
+            message = baseGithubContext.message,
+            workflow = githubWorkflow,
+            prUrl = baseGithubContext.prUrl
         )
     }
 

src/commonMain/kotlin/com/monta/slack/notifier/model/GithubEvent.kt

@@ -6,19 +6,42 @@ class GithubEvent(
     var runId: String,
     val displayName: String?,
     val commitSHA: String?,
-    val commitMessage: String?,
+    val message: String?,
     val workflow: String?,
+    val prUrl: String?,
 ) {
     fun getRunUrl(): String {
         return "https://github.com/$repository/actions/runs/$runId"
     }
-    fun getCommitUrl(): String {
-        return "https://github.com/$repository/commit/$commitSHA"
+    fun getChangeIdentifier(): String? {
+        if (commitSHA != null) {
+            return commitSHA
+        } else if (prUrl != null) {
+            return getPRidentifier(prUrl)
+        }
+        return null
     }
-    fun getCommitMessage(): String? {
-        return commitMessage
+    fun getChangeUrl(): String {
+        if (commitSHA != null) {
+            return "https://github.com/$repository/commit/$commitSHA"
+        } else if (prUrl != null) {
+            return prUrl
+        }
+        return "https://github.com/$repository/"
+    }
+    fun getChangeMessage(): String? {
+        return message
             ?.replace("\n", " ")
             ?.replace("\r", " ")
+            ?.replace("<", "")
+            ?.replace(">", "")
             ?.take(120)
     }
+    fun getPRidentifier(url: String): String? {
+        // Will extract the "pull/51" part of
+        // "https://github.com/monta-app/data-smart-charge/pull/51",
+        val regex = Regex("""pull/\d+""")
+        val matchResult = regex.find(url)
+        return matchResult?.value
+    }
 }

src/commonMain/kotlin/com/monta/slack/notifier/model/serializers/BaseGithubContext.kt

@@ -4,4 +4,5 @@ class BaseGithubContext(
     val displayName: String?,
     val sha: String?,
     val message: String?,
+    val prUrl: String?,
 )

src/commonMain/kotlin/com/monta/slack/notifier/model/serializers/GithubCreatedContext.kt

@@ -5,17 +5,17 @@ import kotlinx.serialization.Serializable
 
 @Serializable
 class GithubCreatedContext(
-    @SerialName("after") val sha: String,
-    @SerialName("pull_request") val pullRequest: PullRequest,
+    @SerialName("sender") val sender: Sender,
+    @SerialName("issue") val issue: Issue,
 ) {
     @Serializable
-    data class PullRequest(
-        @SerialName("title") val title: String,
-        @SerialName("user") val user: PullRequestUser,
+    data class Sender(
+        @SerialName("login") val login: String,
     )
 
     @Serializable
-    data class PullRequestUser(
-        @SerialName("login") val login: String,
+    data class Issue(
+        @SerialName("title") val title: String,
+        @SerialName("html_url") val url: String,
     )
 }

src/commonMain/kotlin/com/monta/slack/notifier/util/FileUtils.kt

@@ -51,7 +51,8 @@ private fun populateOnJsonPush(eventJson: String): BaseGithubContext? {
         return BaseGithubContext(
             displayName = event.pusher.displayName,
             sha = event.headCommit.id,
-            message = event.headCommit.message
+            message = event.headCommit.message,
+            prUrl = null
         )
     } catch (e: SerializationException) {
         null
@@ -65,7 +66,8 @@ private fun populateOnJsonOpened(eventJson: String): BaseGithubContext? {
         return BaseGithubContext(
             displayName = event.pullRequest.user.login,
             sha = event.pullRequest.head.sha,
-            message = event.pullRequest.title
+            message = event.pullRequest.title,
+            prUrl = null
         )
     } catch (e: SerializationException) {
         null
@@ -77,9 +79,10 @@ private fun populateOnJsonCreated(eventJson: String): BaseGithubContext? {
     return try {
         val event = JsonUtil.instance.decodeFromString<GithubCreatedContext>(eventJson)
         return BaseGithubContext(
-            displayName = event.pullRequest.user.login,
-            sha = event.sha,
-            message = event.pullRequest.title
+            displayName = event.sender.login,
+            sha = null,
+            message = event.issue.title,
+            prUrl = event.issue.url
         )
     } catch (e: SerializationException) {
         null
@@ -90,6 +93,7 @@ private fun handleFailure(): BaseGithubContext {
     return BaseGithubContext(
         displayName = null,
         sha = null,
-        message = null
+        message = null,
+        prUrl = null
     )
 }