Merge pull request #43 from alexyaroshuk/add-pr-triage

adjust tool file

Author: alexyaroshuk

.github/workflows/pr-triage.yml

@@ -2,7 +2,7 @@ name: pr-triage
 
 on:
   pull_request_target:
-    types: [opened, edited, synchronize]
+    types: [opened]
 
 jobs:
   triage-pr:

.github/workflows/test-pr-triage.yml

@@ -10,69 +10,56 @@ tools:
 
 You are a triage agent responsible for triaging pull requests.
 
-Use your github-pr-triage tool to triage issues and PRs.
+Use your github-pr-triage tool to triage pull requests.
 
 ## Labels
 
 ### windows
 
-Use for any issue or PR that mentions Windows (the OS). Be sure they are saying that they are on Windows.
+Use for any pull request that mentions Windows (the OS). Be sure they are saying that they are on Windows.
 
 - Use if they mention WSL too
 
 #### perf
 
-Performance-related issues:
+Performance-related pull requests:
 
 - Slow performance
 - High RAM usage
 - High CPU usage
 
-**Only** add if it's likely a RAM or CPU issue. **Do not** add for LLM slowness.
+**Only** add if it's likely a RAM or CPU pull requests. **Do not** add for LLM slowness.
 
 #### desktop
 
-Desktop app issues:
+Desktop app pull requests:
 
 - `opencode web` command
 - The desktop app itself
 
-**Only** add if it's specifically about the Desktop application or `opencode web` view. **Do not** add for terminal, TUI, or general opencode issues.
+**Only** add if it's specifically about the Desktop application or `opencode web` view. **Do not** add for terminal, TUI, or general opencode pull requests.
 
 #### nix
 
-**Only** add if the issue or PR explicitly mentions nix.
+**Only** add if the pull request explicitly mentions nix.
 
 #### zen
 
-**Only** add if the issue or PR mentions "zen" or "opencode zen" or "opencode black".
+**Only** add if the pull request mentions "zen" or "opencode zen" or "opencode black".
 
-If the issue doesn't have "zen" or "opencode black" in it then don't add zen label
+If the pull request doesn't have "zen" or "opencode black" in it then don't add zen label
 
 #### docs
 
-Add if the issue or PR requests or contains documentation updates.
+Add if the pull request requests or contains documentation updates.
 
 #### opentui
 
-TUI issues potentially caused by our underlying TUI library:
+Add if the pull requests addresses TUI issues potentially caused by our underlying TUI library:
 
 - Keybindings not working
 - Scroll speed issues (too fast/slow/laggy)
 - Screen flickering
 - Crashes with opentui in the log
 
-**Do not** add for general TUI bugs.
-
-<!-- When assigning to people here are the following rules:
-
-adamdotdev:
-ONLY assign adam if the issue will have the "desktop" label.
-
-fwang:
-ONLY assign fwang if the issue will have the "zen" label.
-
-jayair:
-ONLY assign jayair if the issue will have the "docs" label.
-
-In all other cases use best judgment. Avoid assigning to kommander needlessly, when in doubt assign to rekram1-node. -->
+**Do not** add for general TUI bugfixes.

.opencode/agent/pr-triage.md

@@ -9,8 +9,7 @@ function getPRNumber(): number {
 }
 
 async function githubFetch(endpoint: string, options: RequestInit = {}) {
-  const url = `https://api.github.com${endpoint}`
-  const response = await fetch(url, {
+  const response = await fetch(`https://api.github.com${endpoint}`, {
     ...options,
     headers: {
       Authorization: `Bearer ${process.env.GITHUB_TOKEN}`,
@@ -20,8 +19,7 @@ async function githubFetch(endpoint: string, options: RequestInit = {}) {
     },
   })
   if (!response.ok) {
-    const error = await response.text()
-    throw new Error(`GitHub API ${response.status}: ${error}`)
+    throw new Error(`GitHub API error: ${response.status} ${response.statusText}`)
   }
   return response.json()
 }
@@ -41,43 +39,34 @@ async function ensureLabelExists(owner: string, repo: string, label: string) {
   }
 }
 
-function getRepoInfo(): { owner: string; repo: string } {
-  const repoFull = process.env.GITHUB_REPOSITORY ?? ""
-  const [owner, repo] = repoFull.split("/")
-  return { owner: owner ?? "anomalyco", repo: repo ?? "opencode" }
-}
-
 export default tool({
   description: DESCRIPTION,
   args: {
-    action: tool.schema.string().describe("Action to perform").optional(),
-    label: tool.schema.string().describe("Single label to add").optional(),
-    labels: tool.schema.array(tool.schema.string()).describe("Labels to add").optional(),
-    reason: tool.schema.string().describe("Reason for the label").optional(),
+    labels: tool.schema
+      .array(tool.schema.enum(["nix", "opentui", "perf", "desktop", "zen", "docs", "windows"]))
+      .describe("The label(s) to add to the PR")
+      .default([]),
   },
   async execute(args) {
     const pr = getPRNumber()
-    const { owner, repo } = getRepoInfo()
+    const owner = "alexyaroshuk"
+    const repo = "opencode"
 
-    let labels: string[] = []
+    const results: string[] = []
 
-    if (args.label) {
-      labels = [args.label]
-    } else if (args.labels) {
-      labels = args.labels
-    } else {
-      labels = ["zen"]
-    }
+    const labels: string[] = args.labels
 
-    for (const label of labels) {
-      await ensureLabelExists(owner, repo, label)
+    if (labels.length > 0) {
+      for (const label of labels) {
+        await ensureLabelExists(owner, repo, label)
+      }
+      await githubFetch(`/repos/${owner}/${repo}/issues/${pr}/labels`, {
+        method: "POST",
+        body: JSON.stringify({ labels }),
+      })
+      results.push(`Added labels: ${args.labels.join(", ")}`)
     }
 
-    await githubFetch(`/repos/${owner}/${repo}/issues/${pr}/labels`, {
-      method: "POST",
-      body: JSON.stringify({ labels }),
-    })
-
-    return `Added labels: ${labels.join(", ")}`
+    return results.join("\n")
   },
 })