diff --git a/packages/app/public/release/release-example.mp4 b/packages/app/public/release/release-example.mp4 deleted file mode 100755 index 6cb4dd585..000000000 Binary files a/packages/app/public/release/release-example.mp4 and /dev/null differ diff --git a/packages/app/public/release/release-share.png b/packages/app/public/release/release-share.png deleted file mode 100644 index e4b99d2db..000000000 Binary files a/packages/app/public/release/release-share.png and /dev/null differ diff --git a/packages/app/src/app.tsx b/packages/app/src/app.tsx index 0980bedb9..ba0d1e7aa 100644 --- a/packages/app/src/app.tsx +++ b/packages/app/src/app.tsx @@ -26,11 +26,10 @@ import { DialogProvider } from "@opencode-ai/ui/context/dialog" import { CommandProvider } from "@/context/command" import { LanguageProvider, useLanguage } from "@/context/language" import { usePlatform } from "@/context/platform" -import { Logo } from "@opencode-ai/ui/logo" +import { HighlightsProvider } from "@/context/highlights" import Layout from "@/pages/layout" import DirectoryLayout from "@/pages/directory-layout" import { ErrorPage } from "./pages/error" -import { iife } from "@opencode-ai/util/iife" import { Suspense } from "solid-js" const Home = lazy(() => import("@/pages/home")) @@ -119,7 +118,9 @@ export function AppInterface(props: { defaultUrl?: string }) { - {props.children} + + {props.children} + diff --git a/packages/app/src/components/dialog-release-notes.tsx b/packages/app/src/components/dialog-release-notes.tsx index d6375dbbc..c62cbc188 100644 --- a/packages/app/src/components/dialog-release-notes.tsx +++ b/packages/app/src/components/dialog-release-notes.tsx @@ -2,110 +2,8 @@ import { createSignal, createEffect, onMount, onCleanup } from "solid-js" import { Dialog } from "@opencode-ai/ui/dialog" import { Button } from "@opencode-ai/ui/button" import { useDialog } from "@opencode-ai/ui/context/dialog" -import { markReleaseNotesSeen } from "@/lib/release-notes" -const CHANGELOG_URL = "https://opencode.ai/changelog.json" - -function isRecord(value: unknown): value is Record { - return typeof value === "object" && value !== null -} - -function getText(value: unknown): string | undefined { - if (typeof value === "string") { - const text = value.trim() - return text.length > 0 ? text : undefined - } - - if (!Array.isArray(value)) return - const parts = value.map((item) => (typeof item === "string" ? item.trim() : "")).filter((item) => item.length > 0) - if (parts.length === 0) return - return parts.join(" ") -} - -function normalizeRemoteUrl(url: string): string { - if (url.startsWith("https://") || url.startsWith("http://")) return url - if (url.startsWith("/")) return `https://opencode.ai${url}` - return `https://opencode.ai/${url}` -} - -function parseMedia(value: unknown): ReleaseFeature["media"] | undefined { - if (!isRecord(value)) return - - const type = getText(value.type)?.toLowerCase() - const src = getText(value.src) - if (!src) return - if (type !== "image" && type !== "video") return - - return { - type, - src: normalizeRemoteUrl(src), - alt: getText(value.alt), - } -} - -function parseFeature(value: unknown): ReleaseFeature | undefined { - if (!isRecord(value)) return - - const title = getText(value.title) ?? getText(value.name) ?? getText(value.heading) - const description = getText(value.description) ?? getText(value.body) ?? getText(value.text) - - if (!title) return - if (!description) return - - const tag = getText(value.tag) ?? getText(value.label) ?? "New" - - const media = (() => { - const parsed = parseMedia(value.media) - if (parsed) return parsed - - const alt = getText(value.alt) - const image = getText(value.image) - if (image) return { type: "image" as const, src: normalizeRemoteUrl(image), alt } - - const video = getText(value.video) - if (video) return { type: "video" as const, src: normalizeRemoteUrl(video), alt } - })() - - return { title, description, tag, media } -} - -function parseChangelog(value: unknown): ReleaseNote | undefined { - const releases = (() => { - if (Array.isArray(value)) return value - if (!isRecord(value)) return - if (Array.isArray(value.releases)) return value.releases - if (Array.isArray(value.versions)) return value.versions - if (Array.isArray(value.changelog)) return value.changelog - })() - - if (!releases) { - if (!isRecord(value)) return - if (!Array.isArray(value.highlights)) return - const features = value.highlights.map(parseFeature).filter((item): item is ReleaseFeature => item !== undefined) - if (features.length === 0) return - return { version: CURRENT_RELEASE.version, features: features.slice(0, 3) } - } - - const version = (() => { - const head = releases[0] - if (!isRecord(head)) return - return getText(head.version) ?? getText(head.tag_name) ?? getText(head.tag) ?? getText(head.name) - })() - - const features = releases - .flatMap((item) => { - if (!isRecord(item)) return [] - const highlights = item.highlights - if (!Array.isArray(highlights)) return [] - return highlights.map(parseFeature).filter((feature): feature is ReleaseFeature => feature !== undefined) - }) - .slice(0, 3) - - if (features.length === 0) return - return { version: version ?? CURRENT_RELEASE.version, features } -} - -export interface ReleaseFeature { +export type Highlight = { title: string description: string tag?: string @@ -116,74 +14,30 @@ export interface ReleaseFeature { } } -export interface ReleaseNote { - version: string - features: ReleaseFeature[] -} - -// Current release notes - update this with each release -export const CURRENT_RELEASE: ReleaseNote = { - version: "1.0.0", - features: [ - { - title: "Cleaner tab experience", - description: "Chat is now fixed to the side of your tabs, and review is now available as a dedicated tab. ", - tag: "New", - media: { - type: "video", - src: "/release/release-example.mp4", - alt: "Cleaner tab experience", - }, - }, - { - title: "Share with control", - description: "Keep your sessions private by default, or publish them to the web with a shareable URL.", - tag: "New", - media: { - type: "image", - src: "/release/release-share.png", - alt: "Share with control", - }, - }, - { - title: "Improved attachment management", - description: "Upload and manage attachments more easily, to help build and maintain context.", - tag: "New", - media: { - type: "video", - src: "/release/release-example.mp4", - alt: "Improved attachment management", - }, - }, - ], -} - -export function DialogReleaseNotes(props: { release?: ReleaseNote }) { +export function DialogReleaseNotes(props: { highlights: Highlight[] }) { const dialog = useDialog() - const [note, setNote] = createSignal(props.release ?? CURRENT_RELEASE) const [index, setIndex] = createSignal(0) - const feature = () => note().features[index()] ?? note().features[0] ?? CURRENT_RELEASE.features[0]! - const total = () => note().features.length + const total = () => props.highlights.length + const last = () => Math.max(0, total() - 1) + const feature = () => props.highlights[index()] ?? props.highlights[last()] const isFirst = () => index() === 0 - const isLast = () => index() === total() - 1 + const isLast = () => index() >= last() + const paged = () => total() > 1 function handleNext() { - if (!isLast()) setIndex(index() + 1) - } - - function handleBack() { - if (!isFirst()) setIndex(index() - 1) + if (isLast()) return + setIndex(index() + 1) } function handleClose() { - markReleaseNotesSeen() dialog.close() } let focusTrap: HTMLDivElement | undefined function handleKeyDown(e: KeyboardEvent) { + if (!paged()) return if (e.key === "ArrowLeft" && !isFirst()) { e.preventDefault() setIndex(index() - 1) @@ -196,28 +50,10 @@ export function DialogReleaseNotes(props: { release?: ReleaseNote }) { onMount(() => { focusTrap?.focus() + + if (!paged()) return document.addEventListener("keydown", handleKeyDown) onCleanup(() => document.removeEventListener("keydown", handleKeyDown)) - - const controller = new AbortController() - fetch(CHANGELOG_URL, { - signal: controller.signal, - headers: { Accept: "application/json" }, - }) - .then((response) => (response.ok ? (response.json() as Promise) : undefined)) - .then((json) => { - if (!json) return - const parsed = parseChangelog(json) - if (!parsed) return - setNote({ - version: parsed.version, - features: parsed.features, - }) - setIndex(0) - }) - .catch(() => undefined) - - onCleanup(() => controller.abort()) }) // Refocus the trap when index changes to ensure escape always works @@ -235,17 +71,17 @@ export function DialogReleaseNotes(props: { release?: ReleaseNote }) { {/* Top section - feature content (fixed position from top) */}
-

{feature().title}

- {feature().tag && ( +

{feature()?.title ?? ""}

+ {feature()?.tag && ( - {feature().tag} + {feature()!.tag} )}
-

{feature().description}

+

{feature()?.description ?? ""}

{/* Spacer to push buttons to bottom */} @@ -265,9 +101,9 @@ export function DialogReleaseNotes(props: { release?: ReleaseNote }) { )} - {total() > 1 && ( + {paged() && (
- {note().features.map((_, i) => ( + {props.highlights.map((_, i) => (
- ) }