fix(settings): handle nop mode results when check run is missing

lib/settings.js

@@ -203,6 +203,14 @@ class Settings {
       })
     })
 
+    // A nop run triggered outside the webhook flow (e.g. the full-sync entrypoint)
+    // has no check run or repository in its payload, so there is nowhere to post
+    // the results other than the log.
+    if (!payload?.check_run) {
+      this.log.info(`Dry-run results:\n${JSON.stringify(this.results, null, 2)}`)
+      return
+    }
+
     let error = false
     // Different logic
     const stats = {

test/unit/lib/settings.test.js

@@ -549,4 +549,48 @@ repository:
       expect(mockRepoSync).toHaveBeenCalledTimes(1)
     })
   }) // updateRepos - archived repo skipping
+
+  describe('handleResults', () => {
+    let settings
+
+    beforeEach(() => {
+      stubContext.octokit.rest.checks = { update: jest.fn().mockResolvedValue({}) }
+      stubContext.octokit.rest.issues = { createComment: jest.fn().mockResolvedValue({}) }
+      settings = new Settings(true, stubContext, mockRepo, {}, mockRef)
+      settings.results = [
+        {
+          type: 'INFO',
+          plugin: 'Branches',
+          repo: 'test/test-repo',
+          endpoint: '',
+          body: '',
+          action: { msg: 'Changes found', additions: {}, modifications: { branch: {} }, deletions: {} }
+        }
+      ]
+    })
+
+    describe('in nop mode without a check run in the payload (full sync)', () => {
+      it('logs the results instead of updating a check run', async () => {
+        // stubContext.payload only contains `installation`, like a full-sync context
+        await settings.handleResults()
+
+        expect(stubContext.log.info).toHaveBeenCalledWith(expect.stringContaining('Changes found'))
+        expect(stubContext.octokit.rest.checks.update).not.toHaveBeenCalled()
+      })
+    })
+
+    describe('in nop mode with a check run in the payload (webhook flow)', () => {
+      it('completes the check run', async () => {
+        stubContext.payload.check_run = { id: 42, check_suite: { pull_requests: [{ number: 1 }] } }
+        stubContext.payload.repository = { owner: { login: 'test' }, name: 'test-repo' }
+
+        await settings.handleResults()
+
+        expect(stubContext.octokit.rest.checks.update).toHaveBeenCalledWith(expect.objectContaining({
+          check_run_id: 42,
+          status: 'completed'
+        }))
+      })
+    })
+  }) // handleResults
 }) // Settings Tests