chore: Cherry-picked changes from upstream

dist/index.js

@@ -67,16 +67,17 @@ exports.getBackoffOptions = exports.getConfig = void 0;
 const core = __importStar(__nccwpck_require__(2186));
 const action_types_1 = __nccwpck_require__(313);
 function getNumberFromValue(value) {
-    try {
-        const num = parseFloat(value);
-        if (isNaN(num)) {
-            throw new Error(`${value}: Parsed value is NaN`);
-        }
-        return num;
-    }
-    catch (_a) {
+    const num = parseFloat(value);
+    return isNaN(num) ? undefined : num;
+}
+function getWorkflowIdFromValue(value) {
+    // Only treat as a workflow ID if the entire string is a positive integer
+    // This prevents filenames like "1-release.yaml" from being parsed as workflow ID 1
+    if (!/^\d+$/.test(value)) {
         return undefined;
     }
+    const num = parseInt(value, 10);
+    return isNaN(num) ? undefined : num;
 }
 function getWorkflowInputs(dispatchMethod) {
     const workflowInputs = core.getInput('workflow-inputs');
@@ -182,7 +183,7 @@ The 'workflow' input is not supported for the repository_dispatch method and mus
         throw error;
     }
     if (dispatchMethod === action_types_1.DispatchMethod.WorkflowDispatch) {
-        return getNumberFromValue(workflow) || workflow;
+        return getWorkflowIdFromValue(workflow) || workflow;
     }
     return undefined;
 }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "dispatch-workflow",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "private": true,
   "description": "A GitHub action to dispatch a remote GitHub workflow and optionally retrieve its information",
   "main": "lib/index.js",

src/action/action.test.ts

@@ -139,6 +139,13 @@ describe('Action', () => {
 
         expect(config.workflow).toStrictEqual(123456)
       })
+
+      test('Should treat workflow filename starting with a number as a string, not a workflow ID', () => {
+        mockGitHubConfig.workflow = '1-release.yaml'
+        const config: ActionConfig = getConfig()
+
+        expect(config.workflow).toStrictEqual('1-release.yaml')
+      })
     })
 
     describe('repositoryDispatch', () => {

src/action/index.ts

@@ -8,15 +8,18 @@ import {
 import {BackoffOptions} from 'exponential-backoff'
 
 function getNumberFromValue(value: string): number | undefined {
-  try {
-    const num = parseFloat(value)
-    if (isNaN(num)) {
-      throw new Error(`${value}: Parsed value is NaN`)
-    }
-    return num
-  } catch {
+  const num = parseFloat(value)
+  return isNaN(num) ? undefined : num
+}
+
+function getWorkflowIdFromValue(value: string): number | undefined {
+  // Only treat as a workflow ID if the entire string is a positive integer
+  // This prevents filenames like "1-release.yaml" from being parsed as workflow ID 1
+  if (!/^\d+$/.test(value)) {
     return undefined
   }
+  const num = parseInt(value, 10)
+  return isNaN(num) ? undefined : num
 }
 
 function getWorkflowInputs(
@@ -134,7 +137,7 @@ The 'workflow' input is not supported for the repository_dispatch method and mus
   }
 
   if (dispatchMethod === DispatchMethod.WorkflowDispatch) {
-    return getNumberFromValue(workflow) || workflow
+    return getWorkflowIdFromValue(workflow) || workflow
   }
   return undefined
 }