Proceed with reopen when wontfix label check fails

Author: kzhou314

.github/actions/file/src/index.ts

@@ -108,7 +108,14 @@ export default async function () {
           }
         } else if (isRepeatedFiling(filing)) {
           const issue = new Issue(filing.issue)
-          if (await isWontfixIssue(octokit, issue)) {
+          let isWontfix = false
+          try {
+            isWontfix = await isWontfixIssue(octokit, issue)
+          } catch (error) {
+            // A failed label check shouldn't abort the run, so reopen as usual
+            core.warning(`Could not check labels for ${filing.issue.url}; proceeding with reopen: ${error}`)
+          }
+          if (isWontfix) {
             // The developer intentionally closed this issue and labeled it 'wontfix', so leave it closed
             core.info(`Skipping reopen of issue labeled '${WONTFIX_LABEL}': ${filing.issue.url}`)
           } else {

.github/actions/file/tests/wontfixReopen.test.ts

@@ -9,6 +9,7 @@ vi.mock('../src/closeIssue.js', () => ({closeIssue: (...args: unknown[]) => clos
 
 const inputs: Record<string, string> = {}
 const infoLines: string[] = []
+const warnLines: string[] = []
 const outputs: Record<string, string> = {}
 vi.mock('@actions/core', () => ({
   getInput: (name: string) => inputs[name] ?? '',
@@ -20,7 +21,9 @@ vi.mock('@actions/core', () => ({
     infoLines.push(msg)
   },
   debug: () => {},
-  warning: () => {},
+  warning: (msg: string) => {
+    warnLines.push(msg)
+  },
   setFailed: () => {},
 }))
 
@@ -89,6 +92,7 @@ describe('file action — wontfix label', () => {
   beforeEach(() => {
     vi.clearAllMocks()
     infoLines.length = 0
+    warnLines.length = 0
     for (const k of Object.keys(inputs)) delete inputs[k]
     for (const k of Object.keys(outputs)) delete outputs[k]
   })
@@ -117,4 +121,16 @@ describe('file action — wontfix label', () => {
       "Skipping reopen of issue labeled 'wontfix': https://github.com/org/repo/issues/1",
     )
   })
+
+  it('reopens as usual (and warns) when the label check fails', async () => {
+    setup()
+    // The label-check GET fails for every issue (e.g. transient API error)
+    octokitRequest.mockRejectedValue(new Error('boom'))
+
+    await runFileAction()
+
+    // Both repeated filings should still be reopened rather than aborting the run
+    expect(reopenIssue).toHaveBeenCalledTimes(2)
+    expect(warnLines.join('\n')).toContain('Could not check labels for')
+  })
 })