feat: update to not post comment on workflows when no duplicates found

.github/VOUCHED.td

@@ -8,7 +8,6 @@
 #   - Denounce with minus prefix: -username or -platform:username.
 #   - Optional details after a space following the handle.
 adamdotdevin
-ariane-emory
 -florianleibert
 fwang
 iamdavidhill

.github/workflows/pr-management.yml

@@ -60,9 +60,11 @@ jobs:
         run: |
           COMMENT=$(bun script/duplicate-pr.ts -f pr_info.txt "Check the attached file for PR details and search for duplicates")
 
-          gh pr comment "$PR_NUMBER" --body "_The following comment was made by an LLM, it may be inaccurate:_
+          if [ "$COMMENT" != "No duplicate PRs found" ]; then
+            gh pr comment "$PR_NUMBER" --body "_The following comment was made by an LLM, it may be inaccurate:_
 
           $COMMENT"
+          fi
 
   add-contributor-label:
     runs-on: ubuntu-latest

packages/app/src/components/prompt-input/attachments.ts

@@ -2,7 +2,6 @@ import { onCleanup, onMount } from "solid-js"
 import { showToast } from "@opencode-ai/ui/toast"
 import { usePrompt, type ContentPart, type ImageAttachmentPart } from "@/context/prompt"
 import { useLanguage } from "@/context/language"
-import { uuid } from "@/utils/uuid"
 import { getCursorPosition } from "./editor-dom"
 
 export const ACCEPTED_IMAGE_TYPES = ["image/png", "image/jpeg", "image/gif", "image/webp"]
@@ -32,7 +31,7 @@ export function createPromptAttachments(input: PromptAttachmentsInput) {
       const dataUrl = reader.result as string
       const attachment: ImageAttachmentPart = {
         type: "image",
-        id: uuid(),
+        id: crypto.randomUUID?.() ?? Math.random().toString(16).slice(2),
         filename: file.name,
         mime: file.type,
         dataUrl,

packages/app/src/context/comments.tsx

@@ -4,7 +4,6 @@ import { createSimpleContext } from "@opencode-ai/ui/context"
 import { useParams } from "@solidjs/router"
 import { Persist, persisted } from "@/utils/persist"
 import { createScopedCache } from "@/utils/scoped-cache"
-import { uuid } from "@/utils/uuid"
 import type { SelectedLineRange } from "@/context/file"
 
 export type LineComment = {
@@ -54,7 +53,7 @@ function createCommentSessionState(store: Store<CommentStore>, setStore: SetStor
 
   const add = (input: Omit<LineComment, "id" | "time">) => {
     const next: LineComment = {
-      id: uuid(),
+      id: crypto.randomUUID?.() ?? Math.random().toString(16).slice(2),
       time: Date.now(),
       ...input,
     }

packages/app/src/utils/perf.ts

@@ -1,5 +1,3 @@
-import { uuid } from "@/utils/uuid"
-
 type Nav = {
   id: string
   dir?: string
@@ -18,6 +16,8 @@ const key = (dir: string | undefined, to: string) => `${dir ?? ""}:${to}`
 
 const now = () => performance.now()
 
+const uid = () => crypto.randomUUID?.() ?? Math.random().toString(16).slice(2)
+
 const navs = new Map<string, Nav>()
 const pending = new Map<string, string>()
 const active = new Map<string, string>()
@@ -94,7 +94,7 @@ function ensure(id: string, data: Omit<Nav, "marks" | "logged" | "timer">) {
 export function navStart(input: { dir?: string; from?: string; to: string; trigger?: string }) {
   if (!dev) return
 
-  const id = uuid()
+  const id = uid()
   const start = now()
   const nav = ensure(id, { ...input, id, start })
   nav.marks["navigate:start"] = start
@@ -109,7 +109,7 @@ export function navParams(input: { dir?: string; from?: string; to: string }) {
   const k = key(input.dir, input.to)
   const pendingId = pending.get(k)
   if (pendingId) pending.delete(k)
-  const id = pendingId ?? uuid()
+  const id = pendingId ?? uid()
 
   const start = now()
   const nav = ensure(id, { ...input, id, start, trigger: pendingId ? "key" : "route" })

packages/app/src/utils/uuid.test.ts

@@ -1,78 +0,0 @@
-import { afterEach, describe, expect, test } from "bun:test"
-import { uuid } from "./uuid"
-
-const cryptoDescriptor = Object.getOwnPropertyDescriptor(globalThis, "crypto")
-const secureDescriptor = Object.getOwnPropertyDescriptor(globalThis, "isSecureContext")
-const randomDescriptor = Object.getOwnPropertyDescriptor(Math, "random")
-
-const setCrypto = (value: Partial<Crypto>) => {
-  Object.defineProperty(globalThis, "crypto", {
-    configurable: true,
-    value: value as Crypto,
-  })
-}
-
-const setSecure = (value: boolean) => {
-  Object.defineProperty(globalThis, "isSecureContext", {
-    configurable: true,
-    value,
-  })
-}
-
-const setRandom = (value: () => number) => {
-  Object.defineProperty(Math, "random", {
-    configurable: true,
-    value,
-  })
-}
-
-afterEach(() => {
-  if (cryptoDescriptor) {
-    Object.defineProperty(globalThis, "crypto", cryptoDescriptor)
-  }
-
-  if (secureDescriptor) {
-    Object.defineProperty(globalThis, "isSecureContext", secureDescriptor)
-  }
-
-  if (!secureDescriptor) {
-    delete (globalThis as { isSecureContext?: boolean }).isSecureContext
-  }
-
-  if (randomDescriptor) {
-    Object.defineProperty(Math, "random", randomDescriptor)
-  }
-})
-
-describe("uuid", () => {
-  test("uses randomUUID in secure contexts", () => {
-    setCrypto({ randomUUID: () => "00000000-0000-0000-0000-000000000000" })
-    setSecure(true)
-    expect(uuid()).toBe("00000000-0000-0000-0000-000000000000")
-  })
-
-  test("falls back in insecure contexts", () => {
-    setCrypto({ randomUUID: () => "00000000-0000-0000-0000-000000000000" })
-    setSecure(false)
-    setRandom(() => 0.5)
-    expect(uuid()).toBe("8")
-  })
-
-  test("falls back when randomUUID throws", () => {
-    setCrypto({
-      randomUUID: () => {
-        throw new DOMException("Failed", "OperationError")
-      },
-    })
-    setSecure(true)
-    setRandom(() => 0.5)
-    expect(uuid()).toBe("8")
-  })
-
-  test("falls back when randomUUID is unavailable", () => {
-    setCrypto({})
-    setSecure(true)
-    setRandom(() => 0.5)
-    expect(uuid()).toBe("8")
-  })
-})

packages/app/src/utils/uuid.ts

@@ -1,12 +0,0 @@
-const fallback = () => Math.random().toString(16).slice(2)
-
-export function uuid() {
-  const c = globalThis.crypto
-  if (!c || typeof c.randomUUID !== "function") return fallback()
-  if (typeof globalThis.isSecureContext === "boolean" && !globalThis.isSecureContext) return fallback()
-  try {
-    return c.randomUUID()
-  } catch {
-    return fallback()
-  }
-}

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx

@@ -149,7 +149,6 @@ export function Session() {
   const [showDetails, setShowDetails] = kv.signal("tool_details_visibility", true)
   const [showAssistantMetadata, setShowAssistantMetadata] = kv.signal("assistant_metadata_visibility", true)
   const [showScrollbar, setShowScrollbar] = kv.signal("scrollbar_visible", false)
-  const [showHeader, setShowHeader] = kv.signal("header_visible", true)
   const [diffWrapMode] = kv.signal<"word" | "none">("diff_wrap_mode", "word")
   const [animationsEnabled, setAnimationsEnabled] = kv.signal("animations_enabled", true)
 
@@ -583,15 +582,6 @@ export function Session() {
         dialog.clear()
       },
     },
-    {
-      title: showHeader() ? "Hide header" : "Show header",
-      value: "session.toggle.header",
-      category: "Session",
-      onSelect: (dialog) => {
-        setShowHeader((prev) => !prev)
-        dialog.clear()
-      },
-    },
     {
       title: "Page up",
       value: "session.page.up",
@@ -973,7 +963,7 @@ export function Session() {
       <box flexDirection="row">
         <box flexGrow={1} paddingBottom={1} paddingTop={1} paddingLeft={2} paddingRight={2} gap={1}>
           <Show when={session()}>
-            <Show when={showHeader() && (!sidebarVisible() || !wide())}>
+            <Show when={!sidebarVisible() || !wide()}>
               <Header />
             </Show>
             <scrollbox

packages/opencode/src/provider/transform.ts

@@ -401,8 +401,6 @@ export namespace ProviderTransform {
       // https://v5.ai-sdk.dev/providers/ai-sdk-providers/xai
       case "@ai-sdk/deepinfra":
       // https://v5.ai-sdk.dev/providers/ai-sdk-providers/deepinfra
-      case "venice-ai-sdk-provider":
-      // https://docs.venice.ai/overview/guides/reasoning-models#reasoning-effort
       case "@ai-sdk/openai-compatible":
         return Object.fromEntries(WIDELY_SUPPORTED_EFFORTS.map((effort) => [effort, { reasoningEffort: effort }]))