wip

packages/opencode/src/cli/cmd/github.ts

@@ -403,12 +403,12 @@ export const GithubRunCommand = cmd({
       let appToken: string
       let octoRest: Octokit
       let octoGraph: typeof graphql
+      let commentId: number
       let gitConfig: string
       let session: { id: string; title: string; version: string }
       let shareId: string | undefined
       let exitCode = 0
       type PromptFiles = Awaited<ReturnType<typeof getUserPrompt>>["promptFiles"]
-      const triggerCommentId = payload.comment.id
 
       try {
         const actionToken = isMock ? args.token! : await getOidcToken()
@@ -422,7 +422,8 @@ export const GithubRunCommand = cmd({
         await configureGit(appToken)
         await assertPermissions()
 
-        await addReaction("eyes")
+        const comment = await createComment()
+        commentId = comment.data.id
 
         // Setup opencode session
         const repoData = await fetchRepo()
@@ -454,8 +455,7 @@ export const GithubRunCommand = cmd({
               await pushToLocalBranch(summary, uncommittedChanges)
             }
             const hasShared = prData.comments.nodes.some((c) => c.body.includes(`${shareBaseUrl}/s/${shareId}`))
-            await createComment(`${response}${footer({ image: !hasShared })}`)
-            await removeReaction()
+            await updateComment(`${response}${footer({ image: !hasShared })}`)
           }
           // Fork PR
           else {
@@ -469,8 +469,7 @@ export const GithubRunCommand = cmd({
               await pushToForkBranch(summary, prData, uncommittedChanges)
             }
             const hasShared = prData.comments.nodes.some((c) => c.body.includes(`${shareBaseUrl}/s/${shareId}`))
-            await createComment(`${response}${footer({ image: !hasShared })}`)
-            await removeReaction()
+            await updateComment(`${response}${footer({ image: !hasShared })}`)
           }
         }
         // Issue
@@ -490,11 +489,9 @@ export const GithubRunCommand = cmd({
               summary,
               `${response}\n\nCloses #${issueId}${footer({ image: true })}`,
             )
-            await createComment(`Created PR #${pr}${footer({ image: true })}`)
-            await removeReaction()
+            await updateComment(`Created PR #${pr}${footer({ image: true })}`)
           } else {
-            await createComment(`${response}${footer({ image: true })}`)
-            await removeReaction()
+            await updateComment(`${response}${footer({ image: true })}`)
           }
         }
       } catch (e: any) {
@@ -506,8 +503,7 @@ export const GithubRunCommand = cmd({
         } else if (e instanceof Error) {
           msg = e.message
         }
-        await createComment(`${msg}${footer()}`)
-        await removeReaction()
+        await updateComment(`${msg}${footer()}`)
         core.setFailed(msg)
         // Also output the clean error message for the action to capture
         //core.setOutput("prepare_error", e.message);
@@ -935,41 +931,24 @@ Co-authored-by: ${actor} <${actor}@users.noreply.github.com>"`
         if (!["admin", "write"].includes(permission)) throw new Error(`User ${actor} does not have write permissions`)
       }
 
-      async function addReaction(reaction: "eyes") {
-        console.log("Adding reaction...")
-        return await octoRest.rest.reactions.createForIssueComment({
-          owner,
-          repo,
-          comment_id: triggerCommentId,
-          content: reaction,
-        })
-      }
-
-      async function removeReaction() {
-        console.log("Removing reaction...")
-        const reactions = await octoRest.rest.reactions.listForIssueComment({
-          owner,
-          repo,
-          comment_id: triggerCommentId,
-        })
-
-        const eyesReaction = reactions.data.find((r) => r.content === "eyes")
-        if (!eyesReaction) return
-
-        await octoRest.rest.reactions.deleteForIssueComment({
-          owner,
-          repo,
-          comment_id: triggerCommentId,
-          reaction_id: eyesReaction.id,
-        })
-      }
-
-      async function createComment(body: string) {
+      async function createComment() {
         console.log("Creating comment...")
         return await octoRest.rest.issues.createComment({
           owner,
           repo,
           issue_number: issueId,
+          body: `[Working...](${runUrl})`,
+        })
+      }
+
+      async function updateComment(body: string) {
+        if (!commentId) return
+
+        console.log("Updating comment...")
+        return await octoRest.rest.issues.updateComment({
+          owner,
+          repo,
+          comment_id: commentId,
           body,
         })
       }
@@ -1050,7 +1029,7 @@ query($owner: String!, $repo: String!, $number: Int!) {
         const comments = (issue.comments?.nodes || [])
           .filter((c) => {
             const id = parseInt(c.databaseId)
-            return id !== payload.comment.id
+            return id !== commentId && id !== payload.comment.id
           })
           .map((c) => `  - ${c.author.login} at ${c.createdAt}: ${c.body}`)
 
@@ -1169,7 +1148,7 @@ query($owner: String!, $repo: String!, $number: Int!) {
         const comments = (pr.comments?.nodes || [])
           .filter((c) => {
             const id = parseInt(c.databaseId)
-            return id !== payload.comment.id
+            return id !== commentId && id !== payload.comment.id
           })
           .map((c) => `- ${c.author.login} at ${c.createdAt}: ${c.body}`)
 

packages/opencode/src/provider/models.ts

@@ -12,7 +12,6 @@ export namespace ModelsDev {
   export const Model = z.object({
     id: z.string(),
     name: z.string(),
-    family: z.string().optional(),
     release_date: z.string(),
     attachment: z.boolean(),
     reasoning: z.boolean(),

packages/opencode/src/provider/provider.ts

@@ -330,7 +330,6 @@ export namespace Provider {
         npm: z.string(),
       }),
       name: z.string(),
-      family: z.string().optional(),
       capabilities: z.object({
         temperature: z.boolean(),
         reasoning: z.boolean(),
@@ -408,7 +407,6 @@ export namespace Provider {
       id: model.id,
       providerID: provider.id,
       name: model.name,
-      family: model.family,
       api: {
         id: model.id,
         url: provider.api!,

packages/opencode/test/provider/transform.test.ts

@@ -158,6 +158,54 @@ describe("ProviderTransform.message - DeepSeek reasoning content", () => {
     expect(result[0].providerOptions?.openaiCompatible?.reasoning_content).toBe("Let me think about this...")
   })
 
+  test("DeepSeek without tool calls strips reasoning from content", () => {
+    const msgs = [
+      {
+        role: "assistant",
+        content: [
+          { type: "reasoning", text: "Let me think about this..." },
+          { type: "text", text: "Final answer" },
+        ],
+      },
+    ] as any[]
+
+    const result = ProviderTransform.message(msgs, {
+      id: "deepseek/deepseek-chat",
+      providerID: "deepseek",
+      api: {
+        id: "deepseek-chat",
+        url: "https://api.deepseek.com",
+        npm: "@ai-sdk/openai-compatible",
+      },
+      name: "DeepSeek Chat",
+      capabilities: {
+        temperature: true,
+        reasoning: true,
+        attachment: false,
+        toolcall: true,
+        input: { text: true, audio: false, image: false, video: false, pdf: false },
+        output: { text: true, audio: false, image: false, video: false, pdf: false },
+        interleaved: false,
+      },
+      cost: {
+        input: 0.001,
+        output: 0.002,
+        cache: { read: 0.0001, write: 0.0002 },
+      },
+      limit: {
+        context: 128000,
+        output: 8192,
+      },
+      status: "active",
+      options: {},
+      headers: {},
+    })
+
+    expect(result).toHaveLength(1)
+    expect(result[0].content).toEqual([{ type: "text", text: "Final answer" }])
+    expect(result[0].providerOptions?.openaiCompatible?.reasoning_content).toBeUndefined()
+  })
+
   test("DeepSeek model ID containing 'deepseek' matches (case insensitive)", () => {
     const msgs = [
       {

packages/sdk/js/src/v2/gen/types.gen.ts

@@ -1035,7 +1035,6 @@ export type ProviderConfig = {
     [key: string]: {
       id?: string
       name?: string
-      family?: string
       release_date?: string
       attachment?: boolean
       reasoning?: boolean
@@ -1462,7 +1461,6 @@ export type Model = {
     npm: string
   }
   name: string
-  family?: string
   capabilities: {
     temperature: boolean
     reasoning: boolean
@@ -3029,7 +3027,6 @@ export type ProviderListResponses = {
         [key: string]: {
           id: string
           name: string
-          family?: string
           release_date: string
           attachment: boolean
           reasoning: boolean

packages/sdk/openapi.json

@@ -2788,9 +2788,6 @@
                                 "name": {
                                   "type": "string"
                                 },
-                                "family": {
-                                  "type": "string"
-                                },
                                 "release_date": {
                                   "type": "string"
                                 },
@@ -7303,9 +7300,6 @@
                 "name": {
                   "type": "string"
                 },
-                "family": {
-                  "type": "string"
-                },
                 "release_date": {
                   "type": "string"
                 },
@@ -8293,9 +8287,6 @@
           "name": {
             "type": "string"
           },
-          "family": {
-            "type": "string"
-          },
           "capabilities": {
             "type": "object",
             "properties": {