Merge branch 'main' into ncalteen/event

Bump @github/local-action from 5.2.0 to 6.0.0

.github/workflows/check-dist.yml

@@ -32,7 +32,7 @@ jobs:
 
       - name: Setup Node.js
         id: setup-node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: .node-version
           cache: npm

.github/workflows/continuous-integration.yml

@@ -22,7 +22,7 @@ jobs:
 
       - name: Setup Node.js
         id: setup-node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: .node-version
           cache: npm

.github/workflows/licensed.yml

@@ -28,7 +28,7 @@ jobs:
 
       - name: Setup Node.js
         id: setup-node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: .node-version
           cache: npm

.github/workflows/linter.yml

@@ -27,7 +27,7 @@ jobs:
 
       - name: Setup Node.js
         id: setup-node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: .node-version
           cache: npm

__fixtures__/@actions/github.ts

@@ -8,6 +8,7 @@ export const context = {
     number: 10
   },
   payload: {
+    action: 'opened',
     number: 10,
     issue: {
       number: 10
@@ -19,7 +20,6 @@ export const context = {
       login: 'mona'
     }
   },
-  action: 'opened',
   repo: {
     owner: 'actions',
     repo: 'first-interaction'

__tests__/main.test.ts

@@ -30,7 +30,7 @@ describe('main.ts', () => {
   beforeEach(() => {
     // "Reset" the github context.
     github.context.eventName = 'pull_request'
-    github.context.action = 'opened'
+    github.context.payload.action = 'opened'
     github.context.payload.issue = undefined as any
     github.context.payload.pull_request = {
       number: 10
@@ -49,6 +49,7 @@ describe('main.ts', () => {
   describe('run()', () => {
     it('Skips invalid events', async () => {
       github.context.eventName = 'push'
+      github.context.payload = {} as any
 
       await main.run()
 
@@ -56,7 +57,7 @@ describe('main.ts', () => {
     })
 
     it('Skips invalid actions', async () => {
-      github.context.action = 'edited'
+      github.context.payload.action = 'edited'
 
       await main.run()
 
@@ -73,17 +74,6 @@ describe('main.ts', () => {
       )
     })
 
-    it('Fails if neither PR nor issue are provided', async () => {
-      github.context.payload.issue = undefined as any
-      github.context.payload.pull_request = undefined as any
-
-      await main.run()
-
-      expect(core.setFailed).toHaveBeenCalledWith(
-        'Internal Error...No Issue or PR Provided by GitHub'
-      )
-    })
-
     it('Fails if both PR and issue are provided', async () => {
       github.context.payload.issue = {
         number: 20

dist/index.js

@@ -34825,22 +34825,19 @@ const Octokit = Octokit$1.plugin(requestLog, legacyRestEndpointMethods, paginate
 
 async function run() {
     coreExports.info('Running actions/first-interaction!');
+    // Check if this is an issue or PR event.
+    const isIssue = githubExports.context.payload.issue !== undefined;
+    const isPullRequest = githubExports.context.payload.pull_request !== undefined;
     // Skip if this is not an issue or PR event.
-    if (githubExports.context.eventName !== 'issues' &&
-        githubExports.context.eventName !== 'pull_request')
+    if (!isIssue && !isPullRequest)
         return coreExports.info('Skipping...Not an Issue/PR Event');
     // Skip if this is not an issue/PR open event.
-    if (githubExports.context.action !== 'opened')
+    if (githubExports.context.payload.action !== 'opened')
         return coreExports.info('Skipping...Not an Opened Event');
     // Confirm the sender data is present.
     if (!githubExports.context.payload.sender)
         return coreExports.setFailed('Internal Error...No Sender Provided by GitHub');
-    // Check if this is an issue or PR event.
-    const isIssue = githubExports.context.payload.issue !== undefined;
-    const isPullRequest = githubExports.context.payload.pull_request !== undefined;
     // Confirm that only one of the two is present.
-    if (!isIssue && !isPullRequest)
-        return coreExports.setFailed('Internal Error...No Issue or PR Provided by GitHub');
     if (isIssue && isPullRequest)
         return coreExports.setFailed('Internal Error...Both Issue and PR Provided by GitHub');
     // Get the action inputs.

package.json

@@ -1,7 +1,7 @@
 {
   "name": "first-interaction-action",
   "description": "An action for greeting first time contributors.",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "author": "GitHub",
   "type": "module",
   "private": true,
@@ -45,29 +45,29 @@
   },
   "devDependencies": {
     "@eslint/compat": "^1.3.2",
-    "@github/local-action": "^5.1.0",
-    "@jest/globals": "^30.0.5",
+    "@github/local-action": "^6.0.0",
+    "@jest/globals": "^30.1.2",
     "@octokit/types": "^14.1.0",
     "@rollup/plugin-commonjs": "^28.0.6",
     "@rollup/plugin-node-resolve": "^16.0.1",
     "@rollup/plugin-typescript": "^12.1.4",
     "@types/jest": "^30.0.0",
-    "@types/node": "^24.3.0",
-    "@typescript-eslint/eslint-plugin": "^8.41.0",
+    "@types/node": "^24.5.0",
+    "@typescript-eslint/eslint-plugin": "^8.44.0",
     "@typescript-eslint/parser": "^8.24.1",
-    "eslint": "^9.34.0",
+    "eslint": "^9.35.0",
     "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-jest": "^29.0.1",
     "eslint-plugin-prettier": "^5.5.4",
-    "globals": "^16.3.0",
-    "jest": "^30.0.5",
+    "globals": "^16.4.0",
+    "jest": "^30.1.3",
     "make-coverage-badge": "^1.2.0",
     "prettier": "^3.6.2",
     "prettier-eslint": "^16.4.2",
-    "rollup": "^4.48.1",
-    "ts-jest": "^29.4.1",
+    "rollup": "^4.50.2",
+    "ts-jest": "^29.4.2",
     "ts-jest-resolver": "^2.0.1",
     "ts-node": "^10.9.2",
     "typescript": "^5.9.2"

src/main.ts

@@ -5,28 +5,23 @@ import { Octokit } from '@octokit/rest'
 export async function run() {
   core.info('Running actions/first-interaction!')
 
+  // Check if this is an issue or PR event.
+  const isIssue = github.context.payload.issue !== undefined
+  const isPullRequest = github.context.payload.pull_request !== undefined
+
   // Skip if this is not an issue or PR event.
-  if (
-    github.context.eventName !== 'issues' &&
-    github.context.eventName !== 'pull_request'
-  )
+  if (!isIssue && !isPullRequest)
     return core.info('Skipping...Not an Issue/PR Event')
 
   // Skip if this is not an issue/PR open event.
-  if (github.context.action !== 'opened')
+  if (github.context.payload.action !== 'opened')
     return core.info('Skipping...Not an Opened Event')
 
   // Confirm the sender data is present.
   if (!github.context.payload.sender)
     return core.setFailed('Internal Error...No Sender Provided by GitHub')
 
-  // Check if this is an issue or PR event.
-  const isIssue = github.context.payload.issue !== undefined
-  const isPullRequest = github.context.payload.pull_request !== undefined
-
   // Confirm that only one of the two is present.
-  if (!isIssue && !isPullRequest)
-    return core.setFailed('Internal Error...No Issue or PR Provided by GitHub')
   if (isIssue && isPullRequest)
     return core.setFailed(
       'Internal Error...Both Issue and PR Provided by GitHub'