From 4d464b069771bb6541931592879afc5c59cd9823 Mon Sep 17 00:00:00 2001 From: Mario Campos Date: Fri, 9 Oct 2026 10:35:34 -0500 Subject: [PATCH] Improve issue ID parsing and validation logic Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- pr-checks/post-comment.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pr-checks/post-comment.ts b/pr-checks/post-comment.ts index f55da65d2..28e065786 100644 --- a/pr-checks/post-comment.ts +++ b/pr-checks/post-comment.ts @@ -85,9 +85,9 @@ function parseBoolean(flag: string, value: string | undefined): boolean { /** Parses and validates `value` as an issue/pull request number. */ function parseIssueId(value: string | undefined): number { return parseFlag("issue-id", value, (raw) => { - const parsed = Number.parseInt(raw, 10); - if (Number.isNaN(parsed)) { - throw new Error("Must be an integer."); + const parsed = Number(raw); + if (!Number.isInteger(parsed) || parsed <= 0) { + throw new Error("Must be a positive integer."); } return parsed; });