@opendash-ai/cli 0.1.0-beta.1 → 0.1.0-beta.3

package/test/cli/cmd/tui/extmark-cjk.test.ts

@@ -1,96 +0,0 @@
-// @ts-nocheck — low-level FFI integration test against opentui internals
-import { describe, expect, test } from "bun:test"
-import { resolveRenderLib, EditBuffer, EditorView, createExtmarksController } from "@opentui/core"
-
-const lib = await resolveRenderLib()
-
-function setup() {
-  const buf = new EditBuffer(lib, lib.createEditBuffer(80, 10))
-  const view = new EditorView(lib, lib.createEditorView(buf.bufferPtr, 80, 10))
-  const extmarks = createExtmarksController(buf, view)
-  const typeId = extmarks.registerType("paste")
-  return { buf, view, extmarks, typeId }
-}
-
-describe("extmark CJK adjustment (patch)", () => {
-  test("insertChar shifts extmark by display width of CJK character", () => {
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("hello")
-    extmarks.create({ start: 5, end: 15, typeId, virtual: true })
-    view.setCursorByOffset(0)
-    buf.insertChar("中")
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(7)
-    expect(mark.end).toBe(17)
-  })
-
-  test("insertText shifts extmark by total display width", () => {
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("hello")
-    extmarks.create({ start: 5, end: 15, typeId, virtual: true })
-    view.setCursorByOffset(0)
-    buf.insertText("你好")
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(9)
-    expect(mark.end).toBe(19)
-  })
-
-  test("deleteCharBackward adjusts by display width of deleted CJK char", () => {
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("中hello")
-    extmarks.create({ start: 2, end: 12, typeId, virtual: true })
-    view.setCursorByOffset(2)
-    buf.deleteCharBackward()
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(0)
-    expect(mark.end).toBe(10)
-  })
-
-  test("deleteChar (forward) adjusts by display width of deleted CJK char", () => {
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("中hello")
-    extmarks.create({ start: 2, end: 12, typeId, virtual: true })
-    view.setCursorByOffset(0)
-    buf.deleteChar()
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(0)
-    expect(mark.end).toBe(10)
-  })
-
-  test("multiple CJK insertions accumulate correctly", () => {
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("test")
-    extmarks.create({ start: 4, end: 14, typeId, virtual: true })
-    view.setCursorByOffset(0)
-    buf.insertChar("你")
-    buf.insertChar("好")
-    buf.insertChar("世")
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(10)
-    expect(mark.end).toBe(20)
-  })
-
-  test("insertText with newlines counts newline as width 1", () => {
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("hello")
-    extmarks.create({ start: 5, end: 15, typeId, virtual: true })
-    view.setCursorByOffset(0)
-    buf.insertText("你\n好")
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(10)
-    expect(mark.end).toBe(20)
-  })
-
-  test("deleteChar works when cursor is past UTF-16 length but within display width", () => {
-    // "中中中" = UTF-16 length 3, display width 6
-    // Cursor at offset 4 (before 3rd char): 4 >= 3 in old code would skip adjustment
-    const { buf, view, extmarks, typeId } = setup()
-    buf.insertText("中中中end")
-    extmarks.create({ start: 9, end: 19, typeId, virtual: true })
-    view.setCursorByOffset(4)
-    buf.deleteChar()
-    const mark = extmarks.getAll()[0]
-    expect(mark.start).toBe(7)
-    expect(mark.end).toBe(17)
-  })
-})

package/test/cli/cmd/tui/offset.test.ts

@@ -1,101 +0,0 @@
-import { describe, expect, test } from "bun:test"
-import {
-  charAfterCursor,
-  stringIndexToWidth,
-  widthToStringIndex,
-} from "../../../../src/cli/cmd/tui/component/prompt/offset"
-
-// The editor uses display-width offsets (a wide CJK char counts as 2 columns)
-// while plainText is a JS UTF-16 string (a CJK char is 1 unit). These helpers
-// translate between the two coordinate systems.
-describe("offset conversion", () => {
-  test("widthToStringIndex maps a width offset to a UTF-16 index", () => {
-    // "你好" is width 4 but 2 UTF-16 units
-    expect(widthToStringIndex("你好world", 4)).toBe(2)
-    expect(widthToStringIndex("你好world", 6)).toBe(4) // 你好wo
-    expect(widthToStringIndex("hello", 3)).toBe(3) // ascii: width == index
-  })
-
-  test("stringIndexToWidth maps a UTF-16 index to a width offset", () => {
-    expect(stringIndexToWidth("你好world", 2)).toBe(4) // 你好 -> width 4
-    expect(stringIndexToWidth("你好world", 4)).toBe(6) // 你好wo
-    expect(stringIndexToWidth("hello", 3)).toBe(3)
-  })
-
-  test("the two conversions round-trip on character boundaries", () => {
-    const text = "前缀@x后缀"
-    for (let i = 0; i <= text.length; i++) {
-      const width = stringIndexToWidth(text, i)
-      expect(widthToStringIndex(text, width)).toBe(i)
-    }
-  })
-
-  test("counts a newline as width 1 (matching the editor, not Bun.stringWidth)", () => {
-    // The editor advances its width offset by 1 per "\n", but Bun.stringWidth("\n")
-    // is 0. The converters must follow the editor, otherwise every newline before an
-    // offset desyncs the two coordinate systems by one.
-    // "你好\n[" — 你好=4, \n=1, so "[" sits at width 5 / string index 3
-    expect(stringIndexToWidth("你好\n[", 3)).toBe(5)
-    expect(widthToStringIndex("你好\n[", 5)).toBe(3)
-  })
-
-  test("round-trips across newlines and CJK together", () => {
-    const text = "你好\n世界\nA"
-    let index = 0
-    for (const ch of text) {
-      const width = stringIndexToWidth(text, index)
-      expect(widthToStringIndex(text, width)).toBe(index)
-      index += ch.length
-    }
-  })
-
-  test("counts a tab as width 2 (matching the editor, not Bun.stringWidth)", () => {
-    // The editor advances its width offset by 2 per "\t", but Bun.stringWidth("\t")
-    // is 0. Pasted code often contains tabs, so the converters must follow the editor.
-    // "ab\tc" — ab=2, \t=2, so "c" sits at width 4 / string index 3
-    expect(stringIndexToWidth("ab\tc", 3)).toBe(4)
-    expect(widthToStringIndex("ab\tc", 4)).toBe(3)
-  })
-
-  test("round-trips across tabs, newlines and CJK together", () => {
-    const text = "你好\t世界\nA\tB"
-    let index = 0
-    for (const ch of text) {
-      const width = stringIndexToWidth(text, index)
-      expect(widthToStringIndex(text, width)).toBe(index)
-      index += ch.length
-    }
-  })
-
-  test("round-trips across supplementary-plane (emoji) code-point boundaries", () => {
-    // 😀 is one code point but 2 UTF-16 units and display width 2. The converters
-    // are only contracted to agree on real code-point boundaries (the editor never
-    // emits an offset that splits a surrogate pair), so iterate by code point.
-    const text = "a😀好b"
-    let index = 0
-    for (const ch of text) {
-      const width = stringIndexToWidth(text, index)
-      expect(widthToStringIndex(text, width)).toBe(index)
-      index += ch.length
-    }
-  })
-})
-
-describe("charAfterCursor", () => {
-  test("reads the char right after the cursor in pure ascii", () => {
-    // "ab cd", cursor (width 2) sits before the space
-    expect(charAfterCursor("ab cd", 2)).toBe(" ")
-  })
-
-  test("reads the correct char when CJK precedes the cursor", () => {
-    // "你好 x" — cursor after 你好 (width 4) must land on the space, not be shifted
-    // by the width/UTF-16 mismatch.
-    expect(charAfterCursor("你好 x", 4)).toBe(" ")
-    // cursor after "你好 " (width 5) lands on "x"
-    expect(charAfterCursor("你好 x", 5)).toBe("x")
-  })
-
-  test("returns undefined at end of input", () => {
-    expect(charAfterCursor("你好", 4)).toBeUndefined()
-  })
-})

package/test/cli/cmd/tui/prompt-part.test.ts

@@ -1,97 +0,0 @@
-import { describe, expect, test } from "bun:test"
-import type { PromptInfo } from "../../../../src/cli/cmd/tui/component/prompt/history"
-import { assign, expandPlaceholders, strip } from "../../../../src/cli/cmd/tui/component/prompt/part"
-
-describe("prompt part", () => {
-  test("strip removes persisted ids from reused file parts", () => {
-    const part = {
-      id: "prt_old",
-      sessionID: "ses_old",
-      messageID: "msg_old",
-      type: "file" as const,
-      mime: "image/png",
-      filename: "tiny.png",
-      url: "data:image/png;base64,abc",
-    }
-
-    expect(strip(part)).toEqual({
-      type: "file",
-      mime: "image/png",
-      filename: "tiny.png",
-      url: "data:image/png;base64,abc",
-    })
-  })
-
-  test("assign overwrites stale runtime ids", () => {
-    const part = {
-      id: "prt_old",
-      sessionID: "ses_old",
-      messageID: "msg_old",
-      type: "file" as const,
-      mime: "image/png",
-      filename: "tiny.png",
-      url: "data:image/png;base64,abc",
-    } as PromptInfo["parts"][number]
-
-    const next = assign(part)
-
-    expect(next.id).not.toBe("prt_old")
-    expect(next.id.startsWith("prt_")).toBe(true)
-    expect(next).toMatchObject({
-      type: "file",
-      mime: "image/png",
-      filename: "tiny.png",
-      url: "data:image/png;base64,abc",
-    })
-  })
-})
-
-describe("expandPlaceholders", () => {
-  // Editor extmark offsets are display-WIDTH based (CJK = 2 columns), while the
-  // plain text is a JS UTF-16 string (CJK = 1 unit). expandPlaceholders must
-  // bridge the two coordinate systems.
-
-  test("expands a single ascii placeholder", () => {
-    const result = expandPlaceholders("[Pasted ~3 lines] ", [
-      { start: 0, end: "[Pasted ~3 lines]".length, text: "line1\nline2\nline3" },
-    ])
-    expect(result).toBe("line1\nline2\nline3 ")
-  })
-
-  test("does not leave residue or swallow content when preceded by CJK", () => {
-    // User typed "你好" (width 4, but string index 2) then pasted.
-    // plainText shown in editor: "你好[Pasted ~3 lines] "
-    // extmark.start is width-based = 4, extmark.end = 4 + 17 = 21
-    const result = expandPlaceholders("你好[Pasted ~3 lines] ", [
-      { start: 4, end: 4 + "[Pasted ~3 lines]".length, text: "line1\nline2\nline3" },
-    ])
-    expect(result).toBe("你好line1\nline2\nline3 ")
-  })
-
-  test("handles multiple placeholders interleaved with CJK", () => {
-    const v1 = "[Pasted ~3 lines]"
-    const v2 = "[Pasted ~2 lines]"
-    // plainText: "你好" + v1 + " " + "世界" + v2 + " " + "末"
-    // widths: 你好=4 -> v1 start 4, end 4+17=21; after v1+space width=18 => 22; 世界=4 => 26; v2 start 26 end 43
-    const plain = `你好${v1} 世界${v2} 末`
-    const result = expandPlaceholders(plain, [
-      { start: 4, end: 4 + v1.length, text: "AAA" },
-      { start: 26, end: 26 + v2.length, text: "BBB" },
-    ])
-    expect(result).toBe("你好AAA 世界BBB 末")
-  })
-
-  test("expands a placeholder that sits on its own line after CJK", () => {
-    const v = "[Pasted ~3 lines]"
-    // User typed CJK on line 1, pasted on line 2, typed CJK on line 3:
-    //   "你好\n" + v + " " + "\n世界"
-    // Editor offsets count "\n" as width 1: 你好=4, \n=5, so the placeholder starts at width 5.
-    const plain = `你好\n${v} \n世界`
-    const result = expandPlaceholders(plain, [{ start: 5, end: 5 + v.length, text: "CONTENT" }])
-    expect(result).toBe("你好\nCONTENT \n世界")
-  })
-
-  test("returns input unchanged when there are no placeholders", () => {
-    expect(expandPlaceholders("你好世界", [])).toBe("你好世界")
-  })
-})

package/test/cli/error.test.ts

@@ -1,18 +0,0 @@
-import { describe, expect, test } from "bun:test"
-import { AccountTransportError } from "../../src/account/schema"
-import { FormatError } from "../../src/cli/error"
-
-describe("cli.error", () => {
-  test("formats account transport errors clearly", () => {
-    const error = new AccountTransportError({
-      method: "POST",
-      url: "https://console.opendash.ai/auth/device/code",
-    })
-
-    const formatted = FormatError(error)
-
-    expect(formatted).toContain("Could not reach POST https://console.opendash.ai/auth/device/code.")
-    expect(formatted).toContain("This failed before the server returned an HTTP response.")
-    expect(formatted).toContain("Check your network, proxy, or VPN configuration and try again.")
-  })
-})

package/test/cli/github-action.test.ts

@@ -1,198 +0,0 @@
-import { test, expect, describe } from "bun:test"
-import { extractResponseText, formatPromptTooLargeError } from "../../src/cli/cmd/github"
-import type { MessageV2 } from "../../src/session/message-v2"
-import { SessionID, MessageID, PartID } from "../../src/session/schema"
-
-// Helper to create minimal valid parts
-function createTextPart(text: string): MessageV2.Part {
-  return {
-    id: PartID.ascending(),
-    sessionID: SessionID.make("s"),
-    messageID: MessageID.make("m"),
-    type: "text" as const,
-    text,
-  }
-}
-
-function createReasoningPart(text: string): MessageV2.Part {
-  return {
-    id: PartID.ascending(),
-    sessionID: SessionID.make("s"),
-    messageID: MessageID.make("m"),
-    type: "reasoning" as const,
-    text,
-    time: { start: 0 },
-  }
-}
-
-function createToolPart(tool: string, title: string, status: "completed" | "running" = "completed"): MessageV2.Part {
-  if (status === "completed") {
-    return {
-      id: PartID.ascending(),
-      sessionID: SessionID.make("s"),
-      messageID: MessageID.make("m"),
-      type: "tool" as const,
-      callID: "c1",
-      tool,
-      state: {
-        status: "completed",
-        input: {},
-        output: "",
-        title,
-        metadata: {},
-        time: { start: 0, end: 1 },
-      },
-    }
-  }
-  return {
-    id: PartID.ascending(),
-    sessionID: SessionID.make("s"),
-    messageID: MessageID.make("m"),
-    type: "tool" as const,
-    callID: "c1",
-    tool,
-    state: {
-      status: "running",
-      input: {},
-      time: { start: 0 },
-    },
-  }
-}
-
-function createStepStartPart(): MessageV2.Part {
-  return {
-    id: PartID.ascending(),
-    sessionID: SessionID.make("s"),
-    messageID: MessageID.make("m"),
-    type: "step-start" as const,
-  }
-}
-
-function createStepFinishPart(): MessageV2.Part {
-  return {
-    id: PartID.ascending(),
-    sessionID: SessionID.make("s"),
-    messageID: MessageID.make("m"),
-    type: "step-finish" as const,
-    reason: "done",
-    cost: 0,
-    tokens: { input: 0, output: 0, reasoning: 0, cache: { read: 0, write: 0 } },
-  }
-}
-
-describe("extractResponseText", () => {
-  test("returns text from text part", () => {
-    const parts = [createTextPart("Hello world")]
-    expect(extractResponseText(parts)).toBe("Hello world")
-  })
-
-  test("returns last text part when multiple exist", () => {
-    const parts = [createTextPart("First"), createTextPart("Last")]
-    expect(extractResponseText(parts)).toBe("Last")
-  })
-
-  test("returns text even when tool parts follow", () => {
-    const parts = [createTextPart("I'll help with that."), createToolPart("todowrite", "3 todos")]
-    expect(extractResponseText(parts)).toBe("I'll help with that.")
-  })
-
-  test("returns null for reasoning-only response (signals summary needed)", () => {
-    const parts = [createReasoningPart("Let me think about this...")]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("returns null for tool-only response (signals summary needed)", () => {
-    // This is the exact scenario from the bug report - todowrite with no text
-    const parts = [createToolPart("todowrite", "8 todos")]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("returns null for multiple completed tools", () => {
-    const parts = [
-      createToolPart("read", "src/file.ts"),
-      createToolPart("edit", "src/file.ts"),
-      createToolPart("bash", "bun test"),
-    ]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("returns null for running tool parts (signals summary needed)", () => {
-    const parts = [createToolPart("bash", "", "running")]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("throws on empty array", () => {
-    expect(() => extractResponseText([])).toThrow("no parts returned")
-  })
-
-  test("returns null for step-start only", () => {
-    const parts = [createStepStartPart()]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("returns null for step-finish only", () => {
-    const parts = [createStepFinishPart()]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("returns null for step-start and step-finish", () => {
-    const parts = [createStepStartPart(), createStepFinishPart()]
-    expect(extractResponseText(parts)).toBeNull()
-  })
-
-  test("returns text from multi-step response", () => {
-    const parts = [
-      createStepStartPart(),
-      createToolPart("read", "src/file.ts"),
-      createTextPart("Done"),
-      createStepFinishPart(),
-    ]
-    expect(extractResponseText(parts)).toBe("Done")
-  })
-
-  test("prefers text over reasoning when both present", () => {
-    const parts = [createReasoningPart("Internal thinking..."), createTextPart("Final answer")]
-    expect(extractResponseText(parts)).toBe("Final answer")
-  })
-
-  test("prefers text over tools when both present", () => {
-    const parts = [createToolPart("read", "src/file.ts"), createTextPart("Here's what I found")]
-    expect(extractResponseText(parts)).toBe("Here's what I found")
-  })
-})
-
-describe("formatPromptTooLargeError", () => {
-  test("formats error without files", () => {
-    const result = formatPromptTooLargeError([])
-    expect(result).toBe("PROMPT_TOO_LARGE: The prompt exceeds the model's context limit.")
-  })
-
-  test("formats error with files (base64 content)", () => {
-    // Base64 is ~33% larger than original, so we multiply by 0.75 to get original size
-    // 400 KB base64 = 300 KB original, 200 KB base64 = 150 KB original
-    const files = [
-      { filename: "screenshot.png", content: "a".repeat(400 * 1024) },
-      { filename: "diagram.png", content: "b".repeat(200 * 1024) },
-    ]
-    const result = formatPromptTooLargeError(files)
-
-    expect(result).toStartWith("PROMPT_TOO_LARGE: The prompt exceeds the model's context limit.")
-    expect(result).toInclude("Files in prompt:")
-    expect(result).toInclude("screenshot.png (300 KB)")
-    expect(result).toInclude("diagram.png (150 KB)")
-  })
-
-  test("lists all files when multiple present", () => {
-    // Base64 sizes: 4KB -> 3KB, 8KB -> 6KB, 12KB -> 9KB
-    const files = [
-      { filename: "img1.png", content: "x".repeat(4 * 1024) },
-      { filename: "img2.jpg", content: "y".repeat(8 * 1024) },
-      { filename: "img3.gif", content: "z".repeat(12 * 1024) },
-    ]
-    const result = formatPromptTooLargeError(files)
-
-    expect(result).toInclude("img1.png (3 KB)")
-    expect(result).toInclude("img2.jpg (6 KB)")
-    expect(result).toInclude("img3.gif (9 KB)")
-  })
-})

package/test/cli/github-remote.test.ts

@@ -1,80 +0,0 @@
-import { test, expect } from "bun:test"
-import { parseGitHubRemote } from "../../src/cli/cmd/github"
-
-test("parses https URL with .git suffix", () => {
-  expect(parseGitHubRemote("https://github.com/sst/opendash.git")).toEqual({ owner: "sst", repo: "opendash" })
-})
-
-test("parses https URL without .git suffix", () => {
-  expect(parseGitHubRemote("https://github.com/sst/opendash")).toEqual({ owner: "sst", repo: "opendash" })
-})
-
-test("parses git@ URL with .git suffix", () => {
-  expect(parseGitHubRemote("git@github.com:sst/opendash.git")).toEqual({ owner: "sst", repo: "opendash" })
-})
-
-test("parses git@ URL without .git suffix", () => {
-  expect(parseGitHubRemote("git@github.com:sst/opendash")).toEqual({ owner: "sst", repo: "opendash" })
-})
-
-test("parses ssh:// URL with .git suffix", () => {
-  expect(parseGitHubRemote("ssh://git@github.com/sst/opendash.git")).toEqual({ owner: "sst", repo: "opendash" })
-})
-
-test("parses ssh:// URL without .git suffix", () => {
-  expect(parseGitHubRemote("ssh://git@github.com/sst/opendash")).toEqual({ owner: "sst", repo: "opendash" })
-})
-
-test("parses http URL", () => {
-  expect(parseGitHubRemote("http://github.com/owner/repo")).toEqual({ owner: "owner", repo: "repo" })
-})
-
-test("parses URL with hyphenated owner and repo names", () => {
-  expect(parseGitHubRemote("https://github.com/my-org/my-repo.git")).toEqual({ owner: "my-org", repo: "my-repo" })
-})
-
-test("parses URL with underscores in names", () => {
-  expect(parseGitHubRemote("git@github.com:my_org/my_repo.git")).toEqual({ owner: "my_org", repo: "my_repo" })
-})
-
-test("parses URL with numbers in names", () => {
-  expect(parseGitHubRemote("https://github.com/org123/repo456")).toEqual({ owner: "org123", repo: "repo456" })
-})
-
-test("parses repos with dots in the name", () => {
-  expect(parseGitHubRemote("https://github.com/socketio/socket.io.git")).toEqual({
-    owner: "socketio",
-    repo: "socket.io",
-  })
-  expect(parseGitHubRemote("https://github.com/vuejs/vue.js")).toEqual({
-    owner: "vuejs",
-    repo: "vue.js",
-  })
-  expect(parseGitHubRemote("git@github.com:mrdoob/three.js.git")).toEqual({
-    owner: "mrdoob",
-    repo: "three.js",
-  })
-  expect(parseGitHubRemote("https://github.com/jashkenas/backbone.git")).toEqual({
-    owner: "jashkenas",
-    repo: "backbone",
-  })
-})
-
-test("returns null for non-github URLs", () => {
-  expect(parseGitHubRemote("https://gitlab.com/owner/repo.git")).toBeNull()
-  expect(parseGitHubRemote("git@gitlab.com:owner/repo.git")).toBeNull()
-  expect(parseGitHubRemote("https://bitbucket.org/owner/repo")).toBeNull()
-})
-
-test("returns null for invalid URLs", () => {
-  expect(parseGitHubRemote("not-a-url")).toBeNull()
-  expect(parseGitHubRemote("")).toBeNull()
-  expect(parseGitHubRemote("github.com")).toBeNull()
-  expect(parseGitHubRemote("https://github.com/")).toBeNull()
-  expect(parseGitHubRemote("https://github.com/owner")).toBeNull()
-})
-
-test("returns null for URLs with extra path segments", () => {
-  expect(parseGitHubRemote("https://github.com/owner/repo/tree/main")).toBeNull()
-  expect(parseGitHubRemote("https://github.com/owner/repo/blob/main/file.ts")).toBeNull()
-})

package/test/cli/import.test.ts

@@ -1,54 +0,0 @@
-import { test, expect } from "bun:test"
-import {
-  parseShareUrl,
-  shouldAttachShareAuthHeaders,
-  transformShareData,
-  type ShareData,
-} from "../../src/cli/cmd/import"
-
-// parseShareUrl tests
-test("parses valid share URLs", () => {
-  expect(parseShareUrl("https://opncd.ai/share/Jsj3hNIW")).toBe("Jsj3hNIW")
-  expect(parseShareUrl("https://custom.example.com/share/abc123")).toBe("abc123")
-  expect(parseShareUrl("http://localhost:3000/share/test_id-123")).toBe("test_id-123")
-})
-
-test("rejects invalid URLs", () => {
-  expect(parseShareUrl("https://opncd.ai/s/Jsj3hNIW")).toBeNull() // legacy format
-  expect(parseShareUrl("https://opncd.ai/share/")).toBeNull()
-  expect(parseShareUrl("https://opncd.ai/share/id/extra")).toBeNull()
-  expect(parseShareUrl("not-a-url")).toBeNull()
-})
-
-test("only attaches share auth headers for same-origin URLs", () => {
-  expect(shouldAttachShareAuthHeaders("https://control.example.com/share/abc", "https://control.example.com")).toBe(
-    true,
-  )
-  expect(shouldAttachShareAuthHeaders("https://other.example.com/share/abc", "https://control.example.com")).toBe(false)
-  expect(shouldAttachShareAuthHeaders("https://control.example.com:443/share/abc", "https://control.example.com")).toBe(
-    true,
-  )
-  expect(shouldAttachShareAuthHeaders("not-a-url", "https://control.example.com")).toBe(false)
-})
-
-// transformShareData tests
-test("transforms share data to storage format", () => {
-  const data: ShareData[] = [
-    { type: "session", data: { id: "sess-1", title: "Test" } as any },
-    { type: "message", data: { id: "msg-1", sessionID: "sess-1" } as any },
-    { type: "part", data: { id: "part-1", messageID: "msg-1" } as any },
-    { type: "part", data: { id: "part-2", messageID: "msg-1" } as any },
-  ]
-
-  const result = transformShareData(data)!
-
-  expect(result.info.id).toBe("sess-1")
-  expect(result.messages).toHaveLength(1)
-  expect(result.messages[0].parts).toHaveLength(2)
-})
-
-test("returns null for invalid share data", () => {
-  expect(transformShareData([])).toBeNull()
-  expect(transformShareData([{ type: "message", data: {} as any }])).toBeNull()
-  expect(transformShareData([{ type: "session", data: { id: "s" } as any }])).toBeNull() // no messages
-})