chore: apply AI code review suggestions

- Move PullRequestPayload and ReleasePayload interfaces to top of file
- Use PullRequestPayload type instead of inline type assertion
- Simplify parseArgs regex match logic (remove redundant fallback)

Author: amondnet

src/index.ts

@@ -17,6 +17,35 @@ import {
 
 type OctokitClient = ReturnType<typeof github.getOctokit>
 
+interface PullRequestPayload {
+  pull_request?: {
+    head: {
+      ref: string
+      sha: string
+      repo?: {
+        owner: { login: string }
+        name: string
+      }
+    }
+  }
+  pull_request_target?: {
+    head: {
+      ref: string
+      sha: string
+      repo?: {
+        owner: { login: string }
+        name: string
+      }
+    }
+  }
+}
+
+interface ReleasePayload {
+  release?: {
+    tag_name: string
+  }
+}
+
 const { context } = github
 
 const githubToken = core.getInput('github-token')
@@ -46,9 +75,8 @@ const aliasDomains = core
     let url = s
     let branch = slugify(context.ref.replace('refs/heads/', ''))
     if (isPullRequestType(context.eventName)) {
-      const pr = (context.payload.pull_request || context.payload.pull_request_target) as {
-        head: { ref: string }
-      } | undefined
+      const payload = context.payload as PullRequestPayload
+      const pr = payload.pull_request || payload.pull_request_target
       if (pr) {
         branch = slugify(pr.head.ref.replace('refs/heads/', ''))
         url = url.replace(prNumberRegExp, context.issue.number.toString())
@@ -322,35 +350,6 @@ async function aliasDomainsToDeployment(deploymentUrl: string): Promise<void> {
   await Promise.all(promises)
 }
 
-interface PullRequestPayload {
-  pull_request?: {
-    head: {
-      ref: string
-      sha: string
-      repo?: {
-        owner: { login: string }
-        name: string
-      }
-    }
-  }
-  pull_request_target?: {
-    head: {
-      ref: string
-      sha: string
-      repo?: {
-        owner: { login: string }
-        name: string
-      }
-    }
-  }
-}
-
-interface ReleasePayload {
-  release?: {
-    tag_name: string
-  }
-}
-
 async function run(): Promise<void> {
   core.debug(`action : ${context.action}`)
   core.debug(`ref : ${context.ref}`)

src/utils.ts

@@ -35,7 +35,7 @@ export function parseArgs(s: string): string[] {
   const args: string[] = []
 
   for (const match of s.matchAll(/'([^']*)'|"([^"]*)"|(\S+)/g)) {
-    args.push(match[1] ?? match[2] ?? match[3] ?? match[0])
+    args.push((match[1] ?? match[2] ?? match[3])!)
   }
   return args
 }