fix: Autodetect PR Triggering Context

Only attempt commenting if in a pr context. Otherwise don't

action.yml

@@ -27,10 +27,6 @@ inputs:
     description: Whether to skip posting a pull request comment when no changes need to be performed.
     required: true
     default: "false"
-  should-comment:
-    description: Whether to attempt outputting the terraform plan as a PR comment
-    required: false
-    default: "true"
 outputs:
   plan-markdown:
     description: The raw markdown output of the terraform plan

src/comment.ts

@@ -1,4 +1,3 @@
-import * as core from '@actions/core'
 import type { GitHub } from '@actions/github/lib/utils'
 import * as github from '@actions/github'
 import type { PullRequestEvent } from '@octokit/webhooks-types'

src/index.ts

@@ -11,8 +11,7 @@ async function run() {
     terraformCmd: core.getInput('terraform-cmd', { required: true }),
     workingDirectory: core.getInput('working-directory', { required: true }),
     header: core.getInput('header', { required: true }),
-    skipEmpty: core.getBooleanInput('skip-empty', { required: true }),
-    shouldComment: core.getInput('should-comment', { required: false })
+    skipEmpty: core.getBooleanInput('skip-empty', { required: true })
   }
   const octokit = github.getOctokit(inputs.token)
 
@@ -26,6 +25,8 @@ async function run() {
   )
 
   if (!inputs.skipEmpty || !planIsEmpty(plan)) {
+    const prContext = 'pull_request' in github.context.payload
+
     // 3) Render the plan diff markdown and set it as output
     const planMarkdown = await core.group('Render plan diff markdown', () => {
       const markdown = renderMarkdown({ plan, header: inputs.header })
@@ -34,7 +35,7 @@ async function run() {
     })
 
     // 4) Post comment with markdown (if applicable)
-    if (inputs.shouldComment === 'true') {
+    if (prContext === true) {
       await core.group('Render comment', () => {
         return createOrUpdateComment({ octokit, content: planMarkdown })
       })