summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorShoubhit Dash <[email protected]>2026-03-19 19:29:29 +0530
committerGitHub <[email protected]>2026-03-19 19:29:29 +0530
commite6f521477959b1009153d8310fb90ac41d766b8b (patch)
treea995a931aa546702cb87076ccac7e895e6a17927
parent84f60d97a0c37c9136dfd965a66a0c8685a19e71 (diff)
downloadopencode-e6f521477959b1009153d8310fb90ac41d766b8b.tar.gz
opencode-e6f521477959b1009153d8310fb90ac41d766b8b.zip
feat: add git-backed session review modes (#17961)
-rw-r--r--packages/app/src/context/global-sync/bootstrap.ts2
-rw-r--r--packages/app/src/context/global-sync/event-reducer.ts4
-rw-r--r--packages/app/src/i18n/en.ts2
-rw-r--r--packages/app/src/pages/session.tsx192
-rw-r--r--packages/app/src/pages/session/session-side-panel.tsx54
-rw-r--r--packages/app/src/pages/session/use-session-commands.tsx6
-rw-r--r--packages/opencode/src/cli/cmd/github.ts10
-rw-r--r--packages/opencode/src/cli/cmd/pr.ts8
-rw-r--r--packages/opencode/src/effect/instances.ts2
-rw-r--r--packages/opencode/src/effect/runtime.ts2
-rw-r--r--packages/opencode/src/file/index.ts14
-rw-r--r--packages/opencode/src/file/watcher.ts4
-rw-r--r--packages/opencode/src/git/effect.ts298
-rw-r--r--packages/opencode/src/git/index.ts64
-rw-r--r--packages/opencode/src/project/project.ts10
-rw-r--r--packages/opencode/src/project/vcs.ts169
-rw-r--r--packages/opencode/src/server/server.ts38
-rw-r--r--packages/opencode/src/storage/storage.ts4
-rw-r--r--packages/opencode/src/util/git.ts35
-rw-r--r--packages/opencode/src/worktree/index.ts44
-rw-r--r--packages/opencode/test/git/git.test.ts107
-rw-r--r--packages/opencode/test/project/project.test.ts17
-rw-r--r--packages/opencode/test/project/vcs.test.ts111
-rw-r--r--packages/sdk/js/src/v2/gen/sdk.gen.ts33
-rw-r--r--packages/sdk/js/src/v2/gen/types.gen.ts23
-rw-r--r--packages/ui/src/i18n/en.ts2
26 files changed, 1072 insertions, 183 deletions
diff --git a/packages/app/src/context/global-sync/bootstrap.ts b/packages/app/src/context/global-sync/bootstrap.ts
index 13494b7ad..c71e00f6f 100644
--- a/packages/app/src/context/global-sync/bootstrap.ts
+++ b/packages/app/src/context/global-sync/bootstrap.ts
@@ -158,7 +158,7 @@ export async function bootstrapDirectory(input: {
input.sdk.vcs.get().then((x) => {
const next = x.data ?? input.store.vcs
input.setStore("vcs", next)
- if (next?.branch) input.vcsCache.setStore("value", next)
+ if (next) input.vcsCache.setStore("value", next)
}),
input.sdk.permission.list().then((x) => {
const grouped = groupBySession(
diff --git a/packages/app/src/context/global-sync/event-reducer.ts b/packages/app/src/context/global-sync/event-reducer.ts
index b8eda0573..0e243e95a 100644
--- a/packages/app/src/context/global-sync/event-reducer.ts
+++ b/packages/app/src/context/global-sync/event-reducer.ts
@@ -268,9 +268,9 @@ export function applyDirectoryEvent(input: {
break
}
case "vcs.branch.updated": {
- const props = event.properties as { branch: string }
+ const props = event.properties as { branch?: string }
if (input.store.vcs?.branch === props.branch) break
- const next = { branch: props.branch }
+ const next = { ...input.store.vcs, branch: props.branch }
input.setStore("vcs", next)
if (input.vcsCache) input.vcsCache.setStore("value", next)
break
diff --git a/packages/app/src/i18n/en.ts b/packages/app/src/i18n/en.ts
index 72caed40a..c7e0c95ea 100644
--- a/packages/app/src/i18n/en.ts
+++ b/packages/app/src/i18n/en.ts
@@ -533,6 +533,8 @@ export const dict = {
"session.review.noVcs.createGit.action": "Create Git repository",
"session.review.noSnapshot": "Snapshot tracking is disabled in config, so session changes are unavailable",
"session.review.noChanges": "No changes",
+ "session.review.noUncommittedChanges": "No uncommitted changes yet",
+ "session.review.noBranchChanges": "No branch changes yet",
"session.files.selectToOpen": "Select a file to open",
"session.files.all": "All files",
diff --git a/packages/app/src/pages/session.tsx b/packages/app/src/pages/session.tsx
index 6d2917008..6fe2e56b8 100644
--- a/packages/app/src/pages/session.tsx
+++ b/packages/app/src/pages/session.tsx
@@ -1,4 +1,4 @@
-import type { Project, UserMessage } from "@opencode-ai/sdk/v2"
+import type { FileDiff, Project, UserMessage } from "@opencode-ai/sdk/v2"
import { useDialog } from "@opencode-ai/ui/context/dialog"
import {
batch,
@@ -57,6 +57,9 @@ import { formatServerError } from "@/utils/server-errors"
const emptyUserMessages: UserMessage[] = []
const emptyFollowups: (FollowupDraft & { id: string })[] = []
+type ChangeMode = "git" | "branch" | "session" | "turn"
+type VcsMode = "git" | "branch"
+
type SessionHistoryWindowInput = {
sessionID: () => string | undefined
messagesReady: () => boolean
@@ -415,15 +418,16 @@ export default function Page() {
const info = createMemo(() => (params.id ? sync.session.get(params.id) : undefined))
const diffs = createMemo(() => (params.id ? (sync.data.session_diff[params.id] ?? []) : []))
- const reviewCount = createMemo(() => Math.max(info()?.summary?.files ?? 0, diffs().length))
- const hasReview = createMemo(() => reviewCount() > 0)
+ const sessionCount = createMemo(() => Math.max(info()?.summary?.files ?? 0, diffs().length))
+ const hasSessionReview = createMemo(() => sessionCount() > 0)
+ const canReview = createMemo(() => !!params.id)
const reviewTab = createMemo(() => isDesktop())
const tabState = createSessionTabs({
tabs,
pathFromTab: file.pathFromTab,
normalizeTab,
review: reviewTab,
- hasReview,
+ hasReview: canReview,
})
const contextOpen = tabState.contextOpen
const openedTabs = tabState.openedTabs
@@ -499,11 +503,22 @@ export default function Page() {
const [store, setStore] = createStore({
messageId: undefined as string | undefined,
mobileTab: "session" as "session" | "changes",
- changes: "session" as "session" | "turn",
+ changes: "git" as ChangeMode,
newSessionWorktree: "main",
deferRender: false,
})
+ const [vcs, setVcs] = createStore({
+ diff: {
+ git: [] as FileDiff[],
+ branch: [] as FileDiff[],
+ },
+ ready: {
+ git: false,
+ branch: false,
+ },
+ })
+
const [followup, setFollowup] = createStore({
items: {} as Record<string, (FollowupDraft & { id: string })[] | undefined>,
sending: {} as Record<string, string | undefined>,
@@ -531,6 +546,40 @@ export default function Page() {
let refreshTimer: number | undefined
let diffFrame: number | undefined
let diffTimer: number | undefined
+ const vcsTask = new Map<VcsMode, Promise<void>>()
+
+ const resetVcs = () => {
+ vcsTask.clear()
+ setVcs({
+ diff: { git: [], branch: [] },
+ ready: { git: false, branch: false },
+ })
+ }
+
+ const loadVcs = (mode: VcsMode, force = false) => {
+ if (sync.project?.vcs !== "git") return Promise.resolve()
+ if (vcs.ready[mode] && !force) return Promise.resolve()
+ const current = vcsTask.get(mode)
+ if (current) return current
+
+ const task = sdk.client.vcs
+ .diff({ mode })
+ .then((result) => {
+ setVcs("diff", mode, result.data ?? [])
+ setVcs("ready", mode, true)
+ })
+ .catch((error) => {
+ console.debug("[session-review] failed to load vcs diff", { mode, error })
+ setVcs("diff", mode, [])
+ setVcs("ready", mode, true)
+ })
+ .finally(() => {
+ vcsTask.delete(mode)
+ })
+
+ vcsTask.set(mode, task)
+ return task
+ }
createComputed((prev) => {
const open = desktopReviewOpen()
@@ -546,7 +595,38 @@ export default function Page() {
}, desktopReviewOpen())
const turnDiffs = createMemo(() => lastUserMessage()?.summary?.diffs ?? [])
- const reviewDiffs = createMemo(() => (store.changes === "session" ? diffs() : turnDiffs()))
+ const changesOptions = createMemo<ChangeMode[]>(() => {
+ const list: ChangeMode[] = []
+ const git = sync.project?.vcs === "git"
+ if (git) list.push("git")
+ if (git && sync.data.vcs?.branch && sync.data.vcs?.default_branch && sync.data.vcs.branch !== sync.data.vcs.default_branch) {
+ list.push("branch")
+ }
+ list.push("session", "turn")
+ return list
+ })
+ const vcsMode = createMemo<VcsMode | undefined>(() => {
+ if (store.changes === "git" || store.changes === "branch") return store.changes
+ })
+ const reviewDiffs = createMemo(() => {
+ if (store.changes === "git") return vcs.diff.git
+ if (store.changes === "branch") return vcs.diff.branch
+ if (store.changes === "session") return diffs()
+ return turnDiffs()
+ })
+ const reviewCount = createMemo(() => {
+ if (store.changes === "git") return vcs.diff.git.length
+ if (store.changes === "branch") return vcs.diff.branch.length
+ if (store.changes === "session") return sessionCount()
+ return turnDiffs().length
+ })
+ const hasReview = createMemo(() => reviewCount() > 0)
+ const reviewReady = createMemo(() => {
+ if (store.changes === "git") return vcs.ready.git
+ if (store.changes === "branch") return vcs.ready.branch
+ if (store.changes === "session") return !hasSessionReview() || diffsReady()
+ return true
+ })
const newSessionWorktree = createMemo(() => {
if (store.newSessionWorktree === "create") return "create"
@@ -615,10 +695,10 @@ export default function Page() {
const diffsReady = createMemo(() => {
const id = params.id
if (!id) return true
- if (!hasReview()) return true
+ if (!hasSessionReview()) return true
return sync.data.session_diff[id] !== undefined
})
- const reviewEmptyKey = createMemo(() => {
+ const sessionEmptyKey = createMemo(() => {
const project = sync.project
if (project && !project.vcs) return "session.review.noVcs"
if (sync.data.config.snapshot === false) return "session.review.noSnapshot"
@@ -741,7 +821,7 @@ export default function Page() {
sessionKey,
() => {
setStore("messageId", undefined)
- setStore("changes", "session")
+ setStore("changes", "git")
setUi("pendingMessage", undefined)
},
{ defer: true },
@@ -750,6 +830,16 @@ export default function Page() {
createEffect(
on(
+ () => sdk.directory,
+ () => {
+ resetVcs()
+ },
+ { defer: true },
+ ),
+ )
+
+ createEffect(
+ on(
() => params.dir,
(dir) => {
if (!dir) return
@@ -870,6 +960,38 @@ export default function Page() {
}
const mobileChanges = createMemo(() => !isDesktop() && store.mobileTab === "changes")
+ const wantsReview = createMemo(() =>
+ isDesktop() ? desktopFileTreeOpen() || (desktopReviewOpen() && activeTab() === "review") : store.mobileTab === "changes",
+ )
+
+ createEffect(() => {
+ const list = changesOptions()
+ if (list.includes(store.changes)) return
+ const next = list[0]
+ if (!next) return
+ setStore("changes", next)
+ })
+
+ createEffect(() => {
+ const mode = vcsMode()
+ if (!mode) return
+ if (!wantsReview()) return
+ void loadVcs(mode)
+ })
+
+ createEffect(
+ on(
+ () => sync.data.session_status[params.id ?? ""]?.type,
+ (next, prev) => {
+ const mode = vcsMode()
+ if (!mode) return
+ if (!wantsReview()) return
+ if (next !== "idle" || prev === undefined || prev === "idle") return
+ void loadVcs(mode, true)
+ },
+ { defer: true },
+ ),
+ )
const fileTreeTab = () => layout.fileTree.tab()
const setFileTreeTab = (value: "changes" | "all") => layout.fileTree.setTab(value)
@@ -916,21 +1038,23 @@ export default function Page() {
loadFile: file.load,
})
- const changesOptions = ["session", "turn"] as const
- const changesOptionsList = [...changesOptions]
-
const changesTitle = () => {
- if (!hasReview()) {
+ if (!canReview()) {
return null
}
+ const label = (option: ChangeMode) => {
+ if (option === "git") return language.t("ui.sessionReview.title.git")
+ if (option === "branch") return language.t("ui.sessionReview.title.branch")
+ if (option === "session") return language.t("ui.sessionReview.title")
+ return language.t("ui.sessionReview.title.lastTurn")
+ }
+
return (
<Select
- options={changesOptionsList}
+ options={changesOptions()}
current={store.changes}
- label={(option) =>
- option === "session" ? language.t("ui.sessionReview.title") : language.t("ui.sessionReview.title.lastTurn")
- }
+ label={label}
onSelect={(option) => option && setStore("changes", option)}
variant="ghost"
size="small"
@@ -939,20 +1063,34 @@ export default function Page() {
)
}
- const emptyTurn = () => (
+ const empty = (text: string) => (
<div class="h-full pb-64 -mt-4 flex flex-col items-center justify-center text-center gap-6">
- <div class="text-14-regular text-text-weak max-w-56">{language.t("session.review.noChanges")}</div>
+ <div class="text-14-regular text-text-weak max-w-56">{text}</div>
</div>
)
+ const reviewEmptyText = createMemo(() => {
+ if (store.changes === "git") return language.t("session.review.noUncommittedChanges")
+ if (store.changes === "branch") return language.t("session.review.noBranchChanges")
+ if (store.changes === "turn") return language.t("session.review.noChanges")
+ return language.t(sessionEmptyKey())
+ })
+
const reviewEmpty = (input: { loadingClass: string; emptyClass: string }) => {
- if (store.changes === "turn") return emptyTurn()
+ if (store.changes === "git" || store.changes === "branch") {
+ if (!reviewReady()) return <div class={input.loadingClass}>{language.t("session.review.loadingChanges")}</div>
+ return empty(reviewEmptyText())
+ }
+
+ if (store.changes === "turn") {
+ return empty(reviewEmptyText())
+ }
- if (hasReview() && !diffsReady()) {
+ if (hasSessionReview() && !diffsReady()) {
return <div class={input.loadingClass}>{language.t("session.review.loadingChanges")}</div>
}
- if (reviewEmptyKey() === "session.review.noVcs") {
+ if (sessionEmptyKey() === "session.review.noVcs") {
return (
<div class={input.emptyClass}>
<div class="flex flex-col gap-3">
@@ -972,7 +1110,7 @@ export default function Page() {
return (
<div class={input.emptyClass}>
- <div class="text-14-regular text-text-weak max-w-56">{language.t(reviewEmptyKey())}</div>
+ <div class="text-14-regular text-text-weak max-w-56">{reviewEmptyText()}</div>
</div>
)
}
@@ -1076,7 +1214,7 @@ export default function Page() {
const pending = tree.pendingDiff
if (!pending) return
if (!tree.reviewScroll) return
- if (!diffsReady()) return
+ if (!reviewReady()) return
const attempt = (count: number) => {
if (tree.pendingDiff !== pending) return
@@ -1808,6 +1946,12 @@ export default function Page() {
</div>
<SessionSidePanel
+ canReview={canReview}
+ diffs={reviewDiffs}
+ diffsReady={reviewReady}
+ empty={reviewEmptyText}
+ hasReview={hasReview}
+ reviewCount={reviewCount}
reviewPanel={reviewPanel}
activeDiff={tree.activeDiff}
focusReviewDiff={focusReviewDiff}
diff --git a/packages/app/src/pages/session/session-side-panel.tsx b/packages/app/src/pages/session/session-side-panel.tsx
index 3b8b0c96b..ed90c97dc 100644
--- a/packages/app/src/pages/session/session-side-panel.tsx
+++ b/packages/app/src/pages/session/session-side-panel.tsx
@@ -8,6 +8,7 @@ import { ResizeHandle } from "@opencode-ai/ui/resize-handle"
import { Mark } from "@opencode-ai/ui/logo"
import { DragDropProvider, DragDropSensors, DragOverlay, SortableProvider, closestCenter } from "@thisbeyond/solid-dnd"
import type { DragEvent } from "@thisbeyond/solid-dnd"
+import type { FileDiff } from "@opencode-ai/sdk/v2"
import { ConstrainDragYAxis, getDraggableId } from "@/utils/solid-dnd"
import { useDialog } from "@opencode-ai/ui/context/dialog"
@@ -19,7 +20,6 @@ import { useCommand } from "@/context/command"
import { useFile, type SelectedLineRange } from "@/context/file"
import { useLanguage } from "@/context/language"
import { useLayout } from "@/context/layout"
-import { useSync } from "@/context/sync"
import { createFileTabListSync } from "@/pages/session/file-tab-scroll"
import { FileTabContent } from "@/pages/session/file-tabs"
import { createOpenSessionFileTab, createSessionTabs, getTabReorderIndex, type Sizing } from "@/pages/session/helpers"
@@ -27,6 +27,12 @@ import { setSessionHandoff } from "@/pages/session/handoff"
import { useSessionLayout } from "@/pages/session/session-layout"
export function SessionSidePanel(props: {
+ canReview: () => boolean
+ diffs: () => FileDiff[]
+ diffsReady: () => boolean
+ empty: () => string
+ hasReview: () => boolean
+ reviewCount: () => number
reviewPanel: () => JSX.Element
activeDiff?: string
focusReviewDiff: (path: string) => void
@@ -34,12 +40,11 @@ export function SessionSidePanel(props: {
size: Sizing
}) {
const layout = useLayout()
- const sync = useSync()
const file = useFile()
const language = useLanguage()
const command = useCommand()
const dialog = useDialog()
- const { params, sessionKey, tabs, view } = useSessionLayout()
+ const { sessionKey, tabs, view } = useSessionLayout()
const isDesktop = createMediaQuery("(min-width: 768px)")
@@ -54,24 +59,7 @@ export function SessionSidePanel(props: {
})
const treeWidth = createMemo(() => (fileOpen() ? `${layout.fileTree.width()}px` : "0px"))
- const info = createMemo(() => (params.id ? sync.session.get(params.id) : undefined))
- const diffs = createMemo(() => (params.id ? (sync.data.session_diff[params.id] ?? []) : []))
- const reviewCount = createMemo(() => Math.max(info()?.summary?.files ?? 0, diffs().length))
- const hasReview = createMemo(() => reviewCount() > 0)
- const diffsReady = createMemo(() => {
- const id = params.id
- if (!id) return true
- if (!hasReview()) return true
- return sync.data.session_diff[id] !== undefined
- })
-
- const reviewEmptyKey = createMemo(() => {
- if (sync.project && !sync.project.vcs) return "session.review.noVcs"
- if (sync.data.config.snapshot === false) return "session.review.noSnapshot"
- return "session.review.noChanges"
- })
-
- const diffFiles = createMemo(() => diffs().map((d) => d.file))
+ const diffFiles = createMemo(() => props.diffs().map((d) => d.file))
const kinds = createMemo(() => {
const merge = (a: "add" | "del" | "mix" | undefined, b: "add" | "del" | "mix") => {
if (!a) return b
@@ -82,7 +70,7 @@ export function SessionSidePanel(props: {
const normalize = (p: string) => p.replaceAll("\\\\", "/").replace(/\/+$/, "")
const out = new Map<string, "add" | "del" | "mix">()
- for (const diff of diffs()) {
+ for (const diff of props.diffs()) {
const file = normalize(diff.file)
const kind = diff.status === "added" ? "add" : diff.status === "deleted" ? "del" : "mix"
@@ -136,7 +124,7 @@ export function SessionSidePanel(props: {
pathFromTab: file.pathFromTab,
normalizeTab,
review: reviewTab,
- hasReview,
+ hasReview: props.canReview,
})
const contextOpen = tabState.contextOpen
const openedTabs = tabState.openedTabs
@@ -241,12 +229,12 @@ export function SessionSidePanel(props: {
onCleanup(stop)
}}
>
- <Show when={reviewTab()}>
+ <Show when={reviewTab() && props.canReview()}>
<Tabs.Trigger value="review">
<div class="flex items-center gap-1.5">
<div>{language.t("session.tab.review")}</div>
- <Show when={hasReview()}>
- <div>{reviewCount()}</div>
+ <Show when={props.hasReview()}>
+ <div>{props.reviewCount()}</div>
</Show>
</div>
</Tabs.Trigger>
@@ -303,7 +291,7 @@ export function SessionSidePanel(props: {
</Tabs.List>
</div>
- <Show when={reviewTab()}>
+ <Show when={reviewTab() && props.canReview()}>
<Tabs.Content value="review" class="flex flex-col h-full overflow-hidden contain-strict">
<Show when={activeTab() === "review"}>{props.reviewPanel()}</Show>
</Tabs.Content>
@@ -377,8 +365,8 @@ export function SessionSidePanel(props: {
>
<Tabs.List>
<Tabs.Trigger value="changes" class="flex-1" classes={{ button: "w-full" }}>
- {reviewCount()}{" "}
- {language.t(reviewCount() === 1 ? "session.review.change.one" : "session.review.change.other")}
+ {props.reviewCount()}{" "}
+ {language.t(props.reviewCount() === 1 ? "session.review.change.one" : "session.review.change.other")}
</Tabs.Trigger>
<Tabs.Trigger value="all" class="flex-1" classes={{ button: "w-full" }}>
{language.t("session.files.all")}
@@ -386,9 +374,9 @@ export function SessionSidePanel(props: {
</Tabs.List>
<Tabs.Content value="changes" class="bg-background-stronger px-3 py-0">
<Switch>
- <Match when={hasReview()}>
+ <Match when={props.hasReview() || !props.diffsReady()}>
<Show
- when={diffsReady()}
+ when={props.diffsReady()}
fallback={
<div class="px-2 py-2 text-12-regular text-text-weak">
{language.t("common.loading")}
@@ -408,9 +396,7 @@ export function SessionSidePanel(props: {
</Show>
</Match>
<Match when={true}>
- {empty(
- language.t(sync.project && !sync.project.vcs ? "session.review.noChanges" : reviewEmptyKey()),
- )}
+ {empty(props.empty())}
</Match>
</Switch>
</Tabs.Content>
diff --git a/packages/app/src/pages/session/use-session-commands.tsx b/packages/app/src/pages/session/use-session-commands.tsx
index 1a2e777f5..f45374359 100644
--- a/packages/app/src/pages/session/use-session-commands.tsx
+++ b/packages/app/src/pages/session/use-session-commands.tsx
@@ -56,11 +56,7 @@ export const useSessionCommands = (actions: SessionCommandContext) => {
if (!id) return
return sync.session.get(id)
}
- const hasReview = () => {
- const id = params.id
- if (!id) return false
- return Math.max(info()?.summary?.files ?? 0, (sync.data.session_diff[id] ?? []).length) > 0
- }
+ const hasReview = () => !!params.id
const normalizeTab = (tab: string) => {
if (!tab.startsWith("file://")) return tab
return file.tab(tab)
diff --git a/packages/opencode/src/cli/cmd/github.ts b/packages/opencode/src/cli/cmd/github.ts
index edd9d7561..1d8542133 100644
--- a/packages/opencode/src/cli/cmd/github.ts
+++ b/packages/opencode/src/cli/cmd/github.ts
@@ -30,7 +30,7 @@ import { MessageV2 } from "../../session/message-v2"
import { SessionPrompt } from "@/session/prompt"
import { setTimeout as sleep } from "node:timers/promises"
import { Process } from "@/util/process"
-import { git } from "@/util/git"
+import { Git } from "@/git"
type GitHubAuthor = {
login: string
@@ -257,7 +257,7 @@ export const GithubInstallCommand = cmd({
}
// Get repo info
- const info = (await git(["remote", "get-url", "origin"], { cwd: Instance.worktree })).text().trim()
+ const info = (await Git.run(["remote", "get-url", "origin"], { cwd: Instance.worktree })).text().trim()
const parsed = parseGitHubRemote(info)
if (!parsed) {
prompts.log.error(`Could not find git repository. Please run this command from a git repository.`)
@@ -496,20 +496,20 @@ export const GithubRunCommand = cmd({
: "issue"
: undefined
const gitText = async (args: string[]) => {
- const result = await git(args, { cwd: Instance.worktree })
+ const result = await Git.run(args, { cwd: Instance.worktree })
if (result.exitCode !== 0) {
throw new Process.RunFailedError(["git", ...args], result.exitCode, result.stdout, result.stderr)
}
return result.text().trim()
}
const gitRun = async (args: string[]) => {
- const result = await git(args, { cwd: Instance.worktree })
+ const result = await Git.run(args, { cwd: Instance.worktree })
if (result.exitCode !== 0) {
throw new Process.RunFailedError(["git", ...args], result.exitCode, result.stdout, result.stderr)
}
return result
}
- const gitStatus = (args: string[]) => git(args, { cwd: Instance.worktree })
+ const gitStatus = (args: string[]) => Git.run(args, { cwd: Instance.worktree })
const commitChanges = async (summary: string, actor?: string) => {
const args = ["commit", "-m", summary]
if (actor) args.push("-m", `Co-authored-by: ${actor} <${actor}@users.noreply.github.com>`)
diff --git a/packages/opencode/src/cli/cmd/pr.ts b/packages/opencode/src/cli/cmd/pr.ts
index 8826fe343..a3a15afbc 100644
--- a/packages/opencode/src/cli/cmd/pr.ts
+++ b/packages/opencode/src/cli/cmd/pr.ts
@@ -2,7 +2,7 @@ import { UI } from "../ui"
import { cmd } from "./cmd"
import { Instance } from "@/project/instance"
import { Process } from "@/util/process"
-import { git } from "@/util/git"
+import { Git } from "@/git"
export const PrCommand = cmd({
command: "pr <number>",
@@ -67,9 +67,9 @@ export const PrCommand = cmd({
const remoteName = forkOwner
// Check if remote already exists
- const remotes = (await git(["remote"], { cwd: Instance.worktree })).text().trim()
+ const remotes = (await Git.run(["remote"], { cwd: Instance.worktree })).text().trim()
if (!remotes.split("\n").includes(remoteName)) {
- await git(["remote", "add", remoteName, `https://github.com/${forkOwner}/${forkName}.git`], {
+ await Git.run(["remote", "add", remoteName, `https://github.com/${forkOwner}/${forkName}.git`], {
cwd: Instance.worktree,
})
UI.println(`Added fork remote: ${remoteName}`)
@@ -77,7 +77,7 @@ export const PrCommand = cmd({
// Set upstream to the fork so pushes go there
const headRefName = prInfo.headRefName
- await git(["branch", `--set-upstream-to=${remoteName}/${headRefName}`, localBranchName], {
+ await Git.run(["branch", `--set-upstream-to=${remoteName}/${headRefName}`, localBranchName], {
cwd: Instance.worktree,
})
}
diff --git a/packages/opencode/src/effect/instances.ts b/packages/opencode/src/effect/instances.ts
index c05458d5d..5ff1fc121 100644
--- a/packages/opencode/src/effect/instances.ts
+++ b/packages/opencode/src/effect/instances.ts
@@ -40,7 +40,7 @@ function lookup(_key: string) {
Layer.fresh(PermissionNext.layer),
Layer.fresh(ProviderAuth.defaultLayer),
Layer.fresh(FileWatcher.layer).pipe(Layer.orDie),
- Layer.fresh(Vcs.layer),
+ Layer.fresh(Vcs.defaultLayer),
Layer.fresh(FileTime.layer).pipe(Layer.orDie),
Layer.fresh(Format.layer),
Layer.fresh(File.layer),
diff --git a/packages/opencode/src/effect/runtime.ts b/packages/opencode/src/effect/runtime.ts
index f52203b22..02209e7a0 100644
--- a/packages/opencode/src/effect/runtime.ts
+++ b/packages/opencode/src/effect/runtime.ts
@@ -1,6 +1,7 @@
import { Effect, Layer, ManagedRuntime } from "effect"
import { AccountEffect } from "@/account/effect"
import { AuthEffect } from "@/auth/effect"
+import { GitEffect } from "@/git/effect"
import { Instances } from "@/effect/instances"
import type { InstanceServices } from "@/effect/instances"
import { TruncateEffect } from "@/tool/truncate-effect"
@@ -9,6 +10,7 @@ import { Instance } from "@/project/instance"
export const runtime = ManagedRuntime.make(
Layer.mergeAll(
AccountEffect.defaultLayer, //
+ GitEffect.defaultLayer,
TruncateEffect.defaultLayer,
Instances.layer,
).pipe(Layer.provideMerge(AuthEffect.layer)),
diff --git a/packages/opencode/src/file/index.ts b/packages/opencode/src/file/index.ts
index 6e9b91727..79f8d9f7d 100644
--- a/packages/opencode/src/file/index.ts
+++ b/packages/opencode/src/file/index.ts
@@ -1,7 +1,7 @@
import { BusEvent } from "@/bus/bus-event"
import { InstanceContext } from "@/effect/instance-context"
import { runPromiseInstance } from "@/effect/runtime"
-import { git } from "@/util/git"
+import { Git } from "@/git"
import { Effect, Fiber, Layer, Scope, ServiceMap } from "effect"
import { formatPatch, structuredPatch } from "diff"
import fs from "fs"
@@ -440,7 +440,7 @@ export namespace File {
return yield* Effect.promise(async () => {
const diffOutput = (
- await git(["-c", "core.fsmonitor=false", "-c", "core.quotepath=false", "diff", "--numstat", "HEAD"], {
+ await Git.run(["-c", "core.fsmonitor=false", "-c", "core.quotepath=false", "diff", "--numstat", "HEAD"], {
cwd: instance.directory,
})
).text()
@@ -460,7 +460,7 @@ export namespace File {
}
const untrackedOutput = (
- await git(
+ await Git.run(
[
"-c",
"core.fsmonitor=false",
@@ -493,7 +493,7 @@ export namespace File {
}
const deletedOutput = (
- await git(
+ await Git.run(
[
"-c",
"core.fsmonitor=false",
@@ -584,17 +584,17 @@ export namespace File {
if (instance.project.vcs === "git") {
let diff = (
- await git(["-c", "core.fsmonitor=false", "diff", "--", file], { cwd: instance.directory })
+ await Git.run(["-c", "core.fsmonitor=false", "diff", "--", file], { cwd: instance.directory })
).text()
if (!diff.trim()) {
diff = (
- await git(["-c", "core.fsmonitor=false", "diff", "--staged", "--", file], {
+ await Git.run(["-c", "core.fsmonitor=false", "diff", "--staged", "--", file], {
cwd: instance.directory,
})
).text()
}
if (diff.trim()) {
- const original = (await git(["show", `HEAD:${file}`], { cwd: instance.directory })).text()
+ const original = (await Git.run(["show", `HEAD:${file}`], { cwd: instance.directory })).text()
const patch = structuredPatch(file, file, original, content, "old", "new", {
context: Infinity,
ignoreWhitespace: true,
diff --git a/packages/opencode/src/file/watcher.ts b/packages/opencode/src/file/watcher.ts
index 16ab3c6d3..a689e6385 100644
--- a/packages/opencode/src/file/watcher.ts
+++ b/packages/opencode/src/file/watcher.ts
@@ -10,7 +10,7 @@ import { BusEvent } from "@/bus/bus-event"
import { InstanceContext } from "@/effect/instance-context"
import { Flag } from "@/flag/flag"
import { Instance } from "@/project/instance"
-import { git } from "@/util/git"
+import { Git } from "@/git"
import { lazy } from "@/util/lazy"
import { Config } from "../config/config"
import { FileIgnore } from "./ignore"
@@ -117,7 +117,7 @@ export namespace FileWatcher {
if (instance.project.vcs === "git") {
const result = yield* Effect.promise(() =>
- git(["rev-parse", "--git-dir"], {
+ Git.run(["rev-parse", "--git-dir"], {
cwd: instance.project.worktree,
}),
)
diff --git a/packages/opencode/src/git/effect.ts b/packages/opencode/src/git/effect.ts
new file mode 100644
index 000000000..feff72b44
--- /dev/null
+++ b/packages/opencode/src/git/effect.ts
@@ -0,0 +1,298 @@
+import { NodeChildProcessSpawner, NodeFileSystem, NodePath } from "@effect/platform-node"
+import { Effect, Layer, ServiceMap, Stream } from "effect"
+import { ChildProcess, ChildProcessSpawner } from "effect/unstable/process"
+
+export namespace GitEffect {
+ const cfg = [
+ "--no-optional-locks",
+ "-c",
+ "core.autocrlf=false",
+ "-c",
+ "core.fsmonitor=false",
+ "-c",
+ "core.longpaths=true",
+ "-c",
+ "core.symlinks=true",
+ "-c",
+ "core.quotepath=false",
+ ] as const
+
+ function out(result: { text(): string }) {
+ return result.text().trim()
+ }
+
+ function split(text: string) {
+ return text.split("\0").filter(Boolean)
+ }
+
+ export type Kind = "added" | "deleted" | "modified"
+
+ export type Base = {
+ readonly name: string
+ readonly ref: string
+ }
+
+ export type Item = {
+ readonly file: string
+ readonly code: string
+ readonly status: Kind
+ }
+
+ export type Stat = {
+ readonly file: string
+ readonly additions: number
+ readonly deletions: number
+ }
+
+ export interface Result {
+ readonly exitCode: number
+ readonly text: () => string
+ readonly stdout: Buffer
+ readonly stderr: Buffer
+ }
+
+ export interface Options {
+ readonly cwd: string
+ readonly env?: Record<string, string>
+ }
+
+ export interface Interface {
+ readonly run: (args: string[], opts: Options) => Effect.Effect<Result>
+ readonly text: (args: string[], opts: Options) => Effect.Effect<string>
+ readonly lines: (args: string[], opts: Options) => Effect.Effect<string[]>
+ readonly branch: (cwd: string) => Effect.Effect<string | undefined>
+ readonly prefix: (cwd: string) => Effect.Effect<string>
+ readonly defaultBranch: (cwd: string) => Effect.Effect<Base | undefined>
+ readonly hasHead: (cwd: string) => Effect.Effect<boolean>
+ readonly mergeBase: (cwd: string, base: string, head?: string) => Effect.Effect<string | undefined>
+ readonly show: (cwd: string, ref: string, file: string, prefix?: string) => Effect.Effect<string>
+ readonly status: (cwd: string) => Effect.Effect<Item[]>
+ readonly diff: (cwd: string, ref: string) => Effect.Effect<Item[]>
+ readonly stats: (cwd: string, ref: string) => Effect.Effect<Stat[]>
+ }
+
+ function kind(code: string | undefined): Kind {
+ if (code === "??") return "added"
+ if (code?.includes("U")) return "modified"
+ if (code?.includes("A") && !code.includes("D")) return "added"
+ if (code?.includes("D") && !code.includes("A")) return "deleted"
+ return "modified"
+ }
+
+ function parseStatus(text: string) {
+ return split(text).flatMap((item) => {
+ const file = item.slice(3)
+ if (!file) return []
+ const code = item.slice(0, 2)
+ return [{ file, code, status: kind(code) } satisfies Item]
+ })
+ }
+
+ function parseNames(text: string) {
+ const list = split(text)
+ const out: Item[] = []
+ for (let i = 0; i < list.length; i += 2) {
+ const code = list[i]
+ const file = list[i + 1]
+ if (!code || !file) continue
+ out.push({ file, code, status: kind(code) })
+ }
+ return out
+ }
+
+ function parseStats(text: string) {
+ const out: Stat[] = []
+ for (const item of split(text)) {
+ const a = item.indexOf("\t")
+ const b = item.indexOf("\t", a + 1)
+ if (a === -1 || b === -1) continue
+ const file = item.slice(b + 1)
+ if (!file) continue
+ const adds = item.slice(0, a)
+ const dels = item.slice(a + 1, b)
+ const additions = adds === "-" ? 0 : Number.parseInt(adds || "0", 10)
+ const deletions = dels === "-" ? 0 : Number.parseInt(dels || "0", 10)
+ out.push({
+ file,
+ additions: Number.isFinite(additions) ? additions : 0,
+ deletions: Number.isFinite(deletions) ? deletions : 0,
+ })
+ }
+ return out
+ }
+
+ export class Service extends ServiceMap.Service<Service, Interface>()("@opencode/Git") {}
+
+ export const layer = Layer.effect(
+ Service,
+ Effect.gen(function* () {
+ const spawner = yield* ChildProcessSpawner.ChildProcessSpawner
+
+ const run = Effect.fn("Git.run")(
+ function* (args: string[], opts: Options) {
+ const proc = ChildProcess.make("git", [...cfg, ...args], {
+ cwd: opts.cwd,
+ env: opts.env,
+ extendEnv: true,
+ })
+ const handle = yield* spawner.spawn(proc)
+ const [stdout, stderr] = yield* Effect.all(
+ [Stream.mkString(Stream.decodeText(handle.stdout)), Stream.mkString(Stream.decodeText(handle.stderr))],
+ { concurrency: 2 },
+ )
+ return {
+ exitCode: yield* handle.exitCode,
+ text: () => stdout,
+ stdout: Buffer.from(stdout),
+ stderr: Buffer.from(stderr),
+ } satisfies Result
+ },
+ Effect.scoped,
+ Effect.catch((err) =>
+ Effect.succeed({
+ exitCode: ChildProcessSpawner.ExitCode(1),
+ text: () => "",
+ stdout: Buffer.alloc(0),
+ stderr: Buffer.from(String(err)),
+ }),
+ ),
+ )
+
+ const text = Effect.fn("Git.text")(function* (args: string[], opts: Options) {
+ return (yield* run(args, opts)).text()
+ })
+
+ const lines = Effect.fn("Git.lines")(function* (args: string[], opts: Options) {
+ return (yield* text(args, opts))
+ .split(/\r?\n/)
+ .map((item) => item.trim())
+ .filter(Boolean)
+ })
+
+ const refs = Effect.fnUntraced(function* (cwd: string) {
+ return yield* lines(["for-each-ref", "--format=%(refname:short)", "refs/heads"], { cwd })
+ })
+
+ const configured = Effect.fnUntraced(function* (cwd: string, list: string[]) {
+ const result = yield* run(["config", "init.defaultBranch"], { cwd })
+ if (result.exitCode !== 0) return
+ const name = out(result)
+ if (!name || !list.includes(name)) return
+ const ref = yield* run(["rev-parse", "--verify", name], { cwd })
+ if (ref.exitCode !== 0) return
+ return { name, ref: name } satisfies Base
+ })
+
+ const remoteHead = Effect.fnUntraced(function* (cwd: string, remote: string) {
+ const result = yield* run(["ls-remote", "--symref", remote, "HEAD"], { cwd })
+ if (result.exitCode !== 0) return
+ for (const line of result.text().split("\n")) {
+ const match = /^ref: refs\/heads\/(.+)\tHEAD$/.exec(line.trim())
+ if (!match?.[1]) continue
+ return { name: match[1], ref: `${remote}/${match[1]}` } satisfies Base
+ }
+ })
+
+ const primary = Effect.fnUntraced(function* (cwd: string) {
+ const list = yield* lines(["remote"], { cwd })
+ if (list.includes("origin")) return "origin"
+ if (list.length === 1) return list[0]
+ if (list.includes("upstream")) return "upstream"
+ return list[0]
+ })
+
+ const branch = Effect.fn("Git.branch")(function* (cwd: string) {
+ const result = yield* run(["rev-parse", "--abbrev-ref", "HEAD"], { cwd })
+ if (result.exitCode !== 0) return
+ const text = out(result)
+ return text || undefined
+ })
+
+ const prefix = Effect.fn("Git.prefix")(function* (cwd: string) {
+ const result = yield* run(["rev-parse", "--show-prefix"], { cwd })
+ if (result.exitCode !== 0) return ""
+ return out(result)
+ })
+
+ const defaultBranch = Effect.fn("Git.defaultBranch")(function* (cwd: string) {
+ const remote = yield* primary(cwd)
+ if (remote) {
+ const head = yield* run(["symbolic-ref", `refs/remotes/${remote}/HEAD`], { cwd })
+ if (head.exitCode === 0) {
+ const ref = out(head).replace(/^refs\/remotes\//, "")
+ const name = ref.startsWith(`${remote}/`) ? ref.slice(`${remote}/`.length) : ""
+ if (name) return { name, ref } satisfies Base
+ }
+
+ const next = yield* remoteHead(cwd, remote)
+ if (next) return next
+ }
+
+ const list = yield* refs(cwd)
+ const next = yield* configured(cwd, list)
+ if (next) return next
+ for (const name of ["main", "master"]) {
+ if (list.includes(name)) return { name, ref: name } satisfies Base
+ }
+ })
+
+ const hasHead = Effect.fn("Git.hasHead")(function* (cwd: string) {
+ const result = yield* run(["rev-parse", "--verify", "HEAD"], { cwd })
+ return result.exitCode === 0
+ })
+
+ const mergeBase = Effect.fn("Git.mergeBase")(function* (cwd: string, base: string, head = "HEAD") {
+ const result = yield* run(["merge-base", base, head], { cwd })
+ if (result.exitCode !== 0) return
+ const text = out(result)
+ return text || undefined
+ })
+
+ const show = Effect.fn("Git.show")(function* (cwd: string, ref: string, file: string, prefix = "") {
+ const target = prefix ? `${prefix}${file}` : file
+ const result = yield* run(["show", `${ref}:${target}`], { cwd })
+ if (result.exitCode !== 0) return ""
+ return result.text()
+ })
+
+ const status = Effect.fn("Git.status")(function* (cwd: string) {
+ return parseStatus(
+ yield* text(["status", "--porcelain=v1", "--untracked-files=all", "--no-renames", "-z", "--", "."], { cwd }),
+ )
+ })
+
+ const diff = Effect.fn("Git.diff")(function* (cwd: string, ref: string) {
+ return parseNames(
+ yield* text(["diff", "--no-ext-diff", "--no-renames", "--name-status", "-z", ref, "--", "."], { cwd }),
+ )
+ })
+
+ const stats = Effect.fn("Git.stats")(function* (cwd: string, ref: string) {
+ return parseStats(
+ yield* text(["diff", "--no-ext-diff", "--no-renames", "--numstat", "-z", ref, "--", "."], { cwd }),
+ )
+ })
+
+ return Service.of({
+ run,
+ text,
+ lines,
+ branch,
+ prefix,
+ defaultBranch,
+ hasHead,
+ mergeBase,
+ show,
+ status,
+ diff,
+ stats,
+ })
+ }),
+ )
+
+ const platformLayer = NodeChildProcessSpawner.layer.pipe(
+ Layer.provideMerge(Layer.mergeAll(NodeFileSystem.layer, NodePath.layer)),
+ )
+
+ export const defaultLayer = layer.pipe(Layer.provide(platformLayer))
+}
diff --git a/packages/opencode/src/git/index.ts b/packages/opencode/src/git/index.ts
new file mode 100644
index 000000000..1b04353b1
--- /dev/null
+++ b/packages/opencode/src/git/index.ts
@@ -0,0 +1,64 @@
+import { Effect } from "effect"
+import { runtime } from "@/effect/runtime"
+import { GitEffect } from "./effect"
+
+function runPromise<A>(f: (service: GitEffect.Interface) => Effect.Effect<A>): Promise<A> {
+ return runtime.runPromise(GitEffect.Service.use(f))
+}
+
+export namespace Git {
+ export type Kind = GitEffect.Kind
+ export type Base = GitEffect.Base
+ export type Item = GitEffect.Item
+ export type Stat = GitEffect.Stat
+ export type Result = GitEffect.Result
+ export type Options = GitEffect.Options
+
+ export function run(args: string[], opts: Options) {
+ return runPromise((git) => git.run(args, opts))
+ }
+
+ export function text(args: string[], opts: Options) {
+ return runPromise((git) => git.text(args, opts))
+ }
+
+ export function lines(args: string[], opts: Options) {
+ return runPromise((git) => git.lines(args, opts))
+ }
+
+ export function branch(cwd: string) {
+ return runPromise((git) => git.branch(cwd))
+ }
+
+ export function prefix(cwd: string) {
+ return runPromise((git) => git.prefix(cwd))
+ }
+
+ export function defaultBranch(cwd: string) {
+ return runPromise((git) => git.defaultBranch(cwd))
+ }
+
+ export function hasHead(cwd: string) {
+ return runPromise((git) => git.hasHead(cwd))
+ }
+
+ export function mergeBase(cwd: string, base: string, head?: string) {
+ return runPromise((git) => git.mergeBase(cwd, base, head))
+ }
+
+ export function show(cwd: string, ref: string, file: string, prefix?: string) {
+ return runPromise((git) => git.show(cwd, ref, file, prefix))
+ }
+
+ export function status(cwd: string) {
+ return runPromise((git) => git.status(cwd))
+ }
+
+ export function diff(cwd: string, ref: string) {
+ return runPromise((git) => git.diff(cwd, ref))
+ }
+
+ export function stats(cwd: string, ref: string) {
+ return runPromise((git) => git.stats(cwd, ref))
+ }
+}
diff --git a/packages/opencode/src/project/project.ts b/packages/opencode/src/project/project.ts
index 1cef41c85..f2df9982e 100644
--- a/packages/opencode/src/project/project.ts
+++ b/packages/opencode/src/project/project.ts
@@ -11,7 +11,7 @@ import { BusEvent } from "@/bus/bus-event"
import { iife } from "@/util/iife"
import { GlobalBus } from "@/bus/global"
import { existsSync } from "fs"
-import { git } from "../util/git"
+import { Git } from "@/git"
import { Glob } from "../util/glob"
import { which } from "../util/which"
import { ProjectID } from "./schema"
@@ -119,7 +119,7 @@ export namespace Project {
}
}
- const worktree = await git(["rev-parse", "--git-common-dir"], {
+ const worktree = await Git.run(["rev-parse", "--git-common-dir"], {
cwd: sandbox,
})
.then(async (result) => {
@@ -147,7 +147,7 @@ export namespace Project {
// generate id from root commit
if (!id) {
- const roots = await git(["rev-list", "--max-parents=0", "HEAD"], {
+ const roots = await Git.run(["rev-list", "--max-parents=0", "HEAD"], {
cwd: sandbox,
})
.then(async (result) =>
@@ -184,7 +184,7 @@ export namespace Project {
}
}
- const top = await git(["rev-parse", "--show-toplevel"], {
+ const top = await Git.run(["rev-parse", "--show-toplevel"], {
cwd: sandbox,
})
.then(async (result) => gitpath(sandbox, await result.text()))
@@ -349,7 +349,7 @@ export namespace Project {
if (input.project.vcs === "git") return input.project
if (!which("git")) throw new Error("Git is not installed")
- const result = await git(["init", "--quiet"], {
+ const result = await Git.run(["init", "--quiet"], {
cwd: input.directory,
})
if (result.exitCode !== 0) {
diff --git a/packages/opencode/src/project/vcs.ts b/packages/opencode/src/project/vcs.ts
index 9e85571c4..7358ec414 100644
--- a/packages/opencode/src/project/vcs.ts
+++ b/packages/opencode/src/project/vcs.ts
@@ -1,16 +1,122 @@
+import { AppFileSystem } from "@/filesystem"
import { Effect, Layer, ServiceMap } from "effect"
import { Bus } from "@/bus"
import { BusEvent } from "@/bus/bus-event"
import { InstanceContext } from "@/effect/instance-context"
import { FileWatcher } from "@/file/watcher"
+import { GitEffect } from "@/git/effect"
+import { Snapshot } from "@/snapshot"
import { Log } from "@/util/log"
-import { git } from "@/util/git"
+import path from "path"
import { Instance } from "./instance"
import z from "zod"
+function count(text: string) {
+ if (!text) return 0
+ if (!text.endsWith("\n")) return text.split("\n").length
+ return text.slice(0, -1).split("\n").length
+}
+
+const work = Effect.fnUntraced(function* (fs: AppFileSystem.Interface, cwd: string, file: string) {
+ const full = path.join(cwd, file)
+ if (!(yield* fs.exists(full).pipe(Effect.orDie))) return ""
+ const buf = yield* fs.readFile(full).pipe(Effect.catch(() => Effect.succeed(new Uint8Array())))
+ if (Buffer.from(buf).includes(0)) return ""
+ return Buffer.from(buf).toString("utf8")
+})
+
+function stats(list: GitEffect.Stat[]) {
+ const out = new Map<string, { additions: number; deletions: number }>()
+ for (const item of list) {
+ out.set(item.file, {
+ additions: item.additions,
+ deletions: item.deletions,
+ })
+ }
+ return out
+}
+
+function merge(...lists: GitEffect.Item[][]) {
+ const out = new Map<string, GitEffect.Item>()
+ for (const list of lists) {
+ for (const item of list) {
+ if (!out.has(item.file)) out.set(item.file, item)
+ }
+ }
+ return [...out.values()]
+}
+
+const files = Effect.fnUntraced(function* (
+ fs: AppFileSystem.Interface,
+ git: GitEffect.Interface,
+ cwd: string,
+ ref: string | undefined,
+ list: GitEffect.Item[],
+ nums: Map<string, { additions: number; deletions: number }>,
+) {
+ const base = ref ? yield* git.prefix(cwd) : ""
+ const next = yield* Effect.forEach(
+ list,
+ (item) =>
+ Effect.gen(function* () {
+ const before = item.status === "added" || !ref ? "" : yield* git.show(cwd, ref, item.file, base)
+ const after = item.status === "deleted" ? "" : yield* work(fs, cwd, item.file)
+ const stat = nums.get(item.file)
+ return {
+ file: item.file,
+ before,
+ after,
+ additions: stat?.additions ?? (item.status === "added" ? count(after) : 0),
+ deletions: stat?.deletions ?? (item.status === "deleted" ? count(before) : 0),
+ status: item.status,
+ } satisfies Snapshot.FileDiff
+ }),
+ { concurrency: 8 },
+ )
+ return next.toSorted((a, b) => a.file.localeCompare(b.file))
+})
+
+const track = Effect.fnUntraced(function* (
+ fs: AppFileSystem.Interface,
+ git: GitEffect.Interface,
+ cwd: string,
+ ref: string | undefined,
+) {
+ if (!ref) {
+ return yield* files(fs, git, cwd, ref, yield* git.status(cwd), new Map())
+ }
+ const [list, nums] = yield* Effect.all([git.status(cwd), git.stats(cwd, ref)], { concurrency: 2 })
+ return yield* files(fs, git, cwd, ref, list, stats(nums))
+})
+
+const compare = Effect.fnUntraced(function* (
+ fs: AppFileSystem.Interface,
+ git: GitEffect.Interface,
+ cwd: string,
+ ref: string,
+) {
+ const [list, nums, extra] = yield* Effect.all([git.diff(cwd, ref), git.stats(cwd, ref), git.status(cwd)], {
+ concurrency: 3,
+ })
+ return yield* files(
+ fs,
+ git,
+ cwd,
+ ref,
+ merge(
+ list,
+ extra.filter((item) => item.code === "??"),
+ ),
+ stats(nums),
+ )
+})
+
export namespace Vcs {
const log = Log.create({ service: "vcs" })
+ export const Mode = z.enum(["git", "branch"])
+ export type Mode = z.infer<typeof Mode>
+
export const Event = {
BranchUpdated: BusEvent.define(
"vcs.branch.updated",
@@ -22,7 +128,8 @@ export namespace Vcs {
export const Info = z
.object({
- branch: z.string(),
+ branch: z.string().optional(),
+ default_branch: z.string().optional(),
})
.meta({
ref: "VcsInfo",
@@ -31,6 +138,8 @@ export namespace Vcs {
export interface Interface {
readonly branch: () => Effect.Effect<string | undefined>
+ readonly defaultBranch: () => Effect.Effect<string | undefined>
+ readonly diff: (mode: Mode) => Effect.Effect<Snapshot.FileDiff[]>
}
export class Service extends ServiceMap.Service<Service, Interface>()("@opencode/Vcs") {}
@@ -39,20 +148,18 @@ export namespace Vcs {
Service,
Effect.gen(function* () {
const instance = yield* InstanceContext
- let currentBranch: string | undefined
+ const fs = yield* AppFileSystem.Service
+ const git = yield* GitEffect.Service
+ let current: string | undefined
+ let root: GitEffect.Base | undefined
if (instance.project.vcs === "git") {
- const getCurrentBranch = async () => {
- const result = await git(["rev-parse", "--abbrev-ref", "HEAD"], {
- cwd: instance.project.worktree,
- })
- if (result.exitCode !== 0) return undefined
- const text = result.text().trim()
- return text || undefined
- }
-
- currentBranch = yield* Effect.promise(() => getCurrentBranch())
- log.info("initialized", { branch: currentBranch })
+ const get = () => Effect.runPromise(git.branch(instance.directory))
+
+ ;[current, root] = yield* Effect.all([git.branch(instance.directory), git.defaultBranch(instance.directory)], {
+ concurrency: 2,
+ })
+ log.info("initialized", { branch: current, default_branch: root?.name })
yield* Effect.acquireRelease(
Effect.sync(() =>
@@ -60,12 +167,11 @@ export namespace Vcs {
FileWatcher.Event.Updated,
Instance.bind(async (evt) => {
if (!evt.properties.file.endsWith("HEAD")) return
- const next = await getCurrentBranch()
- if (next !== currentBranch) {
- log.info("branch changed", { from: currentBranch, to: next })
- currentBranch = next
- Bus.publish(Event.BranchUpdated, { branch: next })
- }
+ const next = await get()
+ if (next === current) return
+ log.info("branch changed", { from: current, to: next })
+ current = next
+ Bus.publish(Event.BranchUpdated, { branch: next })
}),
),
),
@@ -75,9 +181,30 @@ export namespace Vcs {
return Service.of({
branch: Effect.fn("Vcs.branch")(function* () {
- return currentBranch
+ return current
+ }),
+ defaultBranch: Effect.fn("Vcs.defaultBranch")(function* () {
+ return root?.name
+ }),
+ diff: Effect.fn("Vcs.diff")(function* (mode: Mode) {
+ if (instance.project.vcs !== "git") return []
+ if (mode === "git") {
+ const ok = yield* git.hasHead(instance.directory)
+ return yield* track(fs, git, instance.directory, ok ? "HEAD" : undefined)
+ }
+
+ if (!root) return []
+ if (current && current === root.name) return []
+ const ref = yield* git.mergeBase(instance.directory, root.ref)
+ if (!ref) return []
+ return yield* compare(fs, git, instance.directory, ref)
}),
})
}),
)
+
+ export const defaultLayer = layer.pipe(
+ Layer.provide(GitEffect.defaultLayer),
+ Layer.provide(AppFileSystem.defaultLayer),
+ )
}
diff --git a/packages/opencode/src/server/server.ts b/packages/opencode/src/server/server.ts
index c485654fd..b0872b557 100644
--- a/packages/opencode/src/server/server.ts
+++ b/packages/opencode/src/server/server.ts
@@ -41,6 +41,7 @@ import { websocket } from "hono/bun"
import { HTTPException } from "hono/http-exception"
import { errors } from "./error"
import { Filesystem } from "@/util/filesystem"
+import { Snapshot } from "@/snapshot"
import { QuestionRoutes } from "./routes/question"
import { PermissionRoutes } from "./routes/permission"
import { GlobalRoutes } from "./routes/global"
@@ -332,10 +333,39 @@ export namespace Server {
},
}),
async (c) => {
- const branch = await runPromiseInstance(Vcs.Service.use((s) => s.branch()))
- return c.json({
- branch,
- })
+ const [branch, default_branch] = await Promise.all([
+ runPromiseInstance(Vcs.Service.use((s) => s.branch())),
+ runPromiseInstance(Vcs.Service.use((s) => s.defaultBranch())),
+ ])
+ return c.json({ branch, default_branch })
+ },
+ )
+ .get(
+ "/vcs/diff",
+ describeRoute({
+ summary: "Get VCS diff",
+ description: "Retrieve the current git diff for the working tree or against the default branch.",
+ operationId: "vcs.diff",
+ responses: {
+ 200: {
+ description: "VCS diff",
+ content: {
+ "application/json": {
+ schema: resolver(Snapshot.FileDiff.array()),
+ },
+ },
+ },
+ },
+ }),
+ validator(
+ "query",
+ z.object({
+ mode: Vcs.Mode,
+ }),
+ ),
+ async (c) => {
+ const mode = c.req.valid("query").mode
+ return c.json(await runPromiseInstance(Vcs.Service.use((s) => s.diff(mode))))
},
)
.get(
diff --git a/packages/opencode/src/storage/storage.ts b/packages/opencode/src/storage/storage.ts
index a78607cdf..99067594d 100644
--- a/packages/opencode/src/storage/storage.ts
+++ b/packages/opencode/src/storage/storage.ts
@@ -8,7 +8,7 @@ import { Lock } from "../util/lock"
import { NamedError } from "@opencode-ai/util/error"
import z from "zod"
import { Glob } from "../util/glob"
-import { git } from "@/util/git"
+import { Git } from "@/git"
export namespace Storage {
const log = Log.create({ service: "storage" })
@@ -49,7 +49,7 @@ export namespace Storage {
}
if (!worktree) continue
if (!(await Filesystem.isDir(worktree))) continue
- const result = await git(["rev-list", "--max-parents=0", "--all"], {
+ const result = await Git.run(["rev-list", "--max-parents=0", "--all"], {
cwd: worktree,
})
const [id] = result
diff --git a/packages/opencode/src/util/git.ts b/packages/opencode/src/util/git.ts
deleted file mode 100644
index 731131357..000000000
--- a/packages/opencode/src/util/git.ts
+++ /dev/null
@@ -1,35 +0,0 @@
-import { Process } from "./process"
-
-export interface GitResult {
- exitCode: number
- text(): string
- stdout: Buffer
- stderr: Buffer
-}
-
-/**
- * Run a git command.
- *
- * Uses Process helpers with stdin ignored to avoid protocol pipe inheritance
- * issues in embedded/client environments.
- */
-export async function git(args: string[], opts: { cwd: string; env?: Record<string, string> }): Promise<GitResult> {
- return Process.run(["git", ...args], {
- cwd: opts.cwd,
- env: opts.env,
- stdin: "ignore",
- nothrow: true,
- })
- .then((result) => ({
- exitCode: result.code,
- text: () => result.stdout.toString(),
- stdout: result.stdout,
- stderr: result.stderr,
- }))
- .catch((error) => ({
- exitCode: 1,
- text: () => "",
- stdout: Buffer.alloc(0),
- stderr: Buffer.from(error instanceof Error ? error.message : String(error)),
- }))
-}
diff --git a/packages/opencode/src/worktree/index.ts b/packages/opencode/src/worktree/index.ts
index 6ed0e4820..d1f52fff9 100644
--- a/packages/opencode/src/worktree/index.ts
+++ b/packages/opencode/src/worktree/index.ts
@@ -12,7 +12,7 @@ import type { ProjectID } from "../project/schema"
import { fn } from "../util/fn"
import { Log } from "../util/log"
import { Process } from "../util/process"
-import { git } from "../util/git"
+import { Git } from "@/git"
import { BusEvent } from "@/bus/bus-event"
import { GlobalBus } from "@/bus/global"
@@ -250,14 +250,14 @@ export namespace Worktree {
}
async function sweep(root: string) {
- const first = await git(["clean", "-ffdx"], { cwd: root })
+ const first = await Git.run(["clean", "-ffdx"], { cwd: root })
if (first.exitCode === 0) return first
const entries = failed(first)
if (!entries.length) return first
await prune(root, entries)
- return git(["clean", "-ffdx"], { cwd: root })
+ return Git.run(["clean", "-ffdx"], { cwd: root })
}
async function canonical(input: string) {
@@ -276,7 +276,7 @@ export namespace Worktree {
if (await exists(directory)) continue
const ref = `refs/heads/${branch}`
- const branchCheck = await git(["show-ref", "--verify", "--quiet", ref], {
+ const branchCheck = await Git.run(["show-ref", "--verify", "--quiet", ref], {
cwd: Instance.worktree,
})
if (branchCheck.exitCode === 0) continue
@@ -348,7 +348,7 @@ export namespace Worktree {
}
export async function createFromInfo(info: Info, startCommand?: string) {
- const created = await git(["worktree", "add", "--no-checkout", "-b", info.branch, info.directory], {
+ const created = await Git.run(["worktree", "add", "--no-checkout", "-b", info.branch, info.directory], {
cwd: Instance.worktree,
})
if (created.exitCode !== 0) {
@@ -362,7 +362,7 @@ export namespace Worktree {
return () => {
const start = async () => {
- const populated = await git(["reset", "--hard"], { cwd: info.directory })
+ const populated = await Git.run(["reset", "--hard"], { cwd: info.directory })
if (populated.exitCode !== 0) {
const message = errorText(populated) || "Failed to populate worktree"
log.error("worktree checkout failed", { directory: info.directory, message })
@@ -479,10 +479,10 @@ export namespace Worktree {
const stop = async (target: string) => {
if (!(await exists(target))) return
- await git(["fsmonitor--daemon", "stop"], { cwd: target })
+ await Git.run(["fsmonitor--daemon", "stop"], { cwd: target })
}
- const list = await git(["worktree", "list", "--porcelain"], { cwd: Instance.worktree })
+ const list = await Git.run(["worktree", "list", "--porcelain"], { cwd: Instance.worktree })
if (list.exitCode !== 0) {
throw new RemoveFailedError({ message: errorText(list) || "Failed to read git worktrees" })
}
@@ -499,11 +499,11 @@ export namespace Worktree {
}
await stop(entry.path)
- const removed = await git(["worktree", "remove", "--force", entry.path], {
+ const removed = await Git.run(["worktree", "remove", "--force", entry.path], {
cwd: Instance.worktree,
})
if (removed.exitCode !== 0) {
- const next = await git(["worktree", "list", "--porcelain"], { cwd: Instance.worktree })
+ const next = await Git.run(["worktree", "list", "--porcelain"], { cwd: Instance.worktree })
if (next.exitCode !== 0) {
throw new RemoveFailedError({
message: errorText(removed) || errorText(next) || "Failed to remove git worktree",
@@ -520,7 +520,7 @@ export namespace Worktree {
const branch = entry.branch?.replace(/^refs\/heads\//, "")
if (branch) {
- const deleted = await git(["branch", "-D", branch], { cwd: Instance.worktree })
+ const deleted = await Git.run(["branch", "-D", branch], { cwd: Instance.worktree })
if (deleted.exitCode !== 0) {
throw new RemoveFailedError({ message: errorText(deleted) || "Failed to delete worktree branch" })
}
@@ -540,7 +540,7 @@ export namespace Worktree {
throw new ResetFailedError({ message: "Cannot reset the primary workspace" })
}
- const list = await git(["worktree", "list", "--porcelain"], { cwd: Instance.worktree })
+ const list = await Git.run(["worktree", "list", "--porcelain"], { cwd: Instance.worktree })
if (list.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(list) || "Failed to read git worktrees" })
}
@@ -573,7 +573,7 @@ export namespace Worktree {
throw new ResetFailedError({ message: "Worktree not found" })
}
- const remoteList = await git(["remote"], { cwd: Instance.worktree })
+ const remoteList = await Git.run(["remote"], { cwd: Instance.worktree })
if (remoteList.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(remoteList) || "Failed to list git remotes" })
}
@@ -592,17 +592,17 @@ export namespace Worktree {
: ""
const remoteHead = remote
- ? await git(["symbolic-ref", `refs/remotes/${remote}/HEAD`], { cwd: Instance.worktree })
+ ? await Git.run(["symbolic-ref", `refs/remotes/${remote}/HEAD`], { cwd: Instance.worktree })
: { exitCode: 1, stdout: undefined, stderr: undefined }
const remoteRef = remoteHead.exitCode === 0 ? outputText(remoteHead.stdout) : ""
const remoteTarget = remoteRef ? remoteRef.replace(/^refs\/remotes\//, "") : ""
const remoteBranch = remote && remoteTarget.startsWith(`${remote}/`) ? remoteTarget.slice(`${remote}/`.length) : ""
- const mainCheck = await git(["show-ref", "--verify", "--quiet", "refs/heads/main"], {
+ const mainCheck = await Git.run(["show-ref", "--verify", "--quiet", "refs/heads/main"], {
cwd: Instance.worktree,
})
- const masterCheck = await git(["show-ref", "--verify", "--quiet", "refs/heads/master"], {
+ const masterCheck = await Git.run(["show-ref", "--verify", "--quiet", "refs/heads/master"], {
cwd: Instance.worktree,
})
const localBranch = mainCheck.exitCode === 0 ? "main" : masterCheck.exitCode === 0 ? "master" : ""
@@ -613,7 +613,7 @@ export namespace Worktree {
}
if (remoteBranch) {
- const fetch = await git(["fetch", remote, remoteBranch], { cwd: Instance.worktree })
+ const fetch = await Git.run(["fetch", remote, remoteBranch], { cwd: Instance.worktree })
if (fetch.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(fetch) || `Failed to fetch ${target}` })
}
@@ -625,7 +625,7 @@ export namespace Worktree {
const worktreePath = entry.path
- const resetToTarget = await git(["reset", "--hard", target], { cwd: worktreePath })
+ const resetToTarget = await Git.run(["reset", "--hard", target], { cwd: worktreePath })
if (resetToTarget.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(resetToTarget) || "Failed to reset worktree to target" })
}
@@ -635,26 +635,26 @@ export namespace Worktree {
throw new ResetFailedError({ message: errorText(clean) || "Failed to clean worktree" })
}
- const update = await git(["submodule", "update", "--init", "--recursive", "--force"], { cwd: worktreePath })
+ const update = await Git.run(["submodule", "update", "--init", "--recursive", "--force"], { cwd: worktreePath })
if (update.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(update) || "Failed to update submodules" })
}
- const subReset = await git(["submodule", "foreach", "--recursive", "git", "reset", "--hard"], {
+ const subReset = await Git.run(["submodule", "foreach", "--recursive", "git", "reset", "--hard"], {
cwd: worktreePath,
})
if (subReset.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(subReset) || "Failed to reset submodules" })
}
- const subClean = await git(["submodule", "foreach", "--recursive", "git", "clean", "-fdx"], {
+ const subClean = await Git.run(["submodule", "foreach", "--recursive", "git", "clean", "-fdx"], {
cwd: worktreePath,
})
if (subClean.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(subClean) || "Failed to clean submodules" })
}
- const status = await git(["-c", "core.fsmonitor=false", "status", "--porcelain=v1"], { cwd: worktreePath })
+ const status = await Git.run(["-c", "core.fsmonitor=false", "status", "--porcelain=v1"], { cwd: worktreePath })
if (status.exitCode !== 0) {
throw new ResetFailedError({ message: errorText(status) || "Failed to read git status" })
}
diff --git a/packages/opencode/test/git/git.test.ts b/packages/opencode/test/git/git.test.ts
new file mode 100644
index 000000000..3427d8eb5
--- /dev/null
+++ b/packages/opencode/test/git/git.test.ts
@@ -0,0 +1,107 @@
+import { $ } from "bun"
+import { describe, expect, test } from "bun:test"
+import fs from "fs/promises"
+import path from "path"
+import { ManagedRuntime } from "effect"
+import { GitEffect } from "../../src/git/effect"
+import { tmpdir } from "../fixture/fixture"
+
+const weird = process.platform === "win32" ? "space file.txt" : "tab\tfile.txt"
+
+async function withGit<T>(fn: (rt: ManagedRuntime.ManagedRuntime<GitEffect.Service, never>) => Promise<T>) {
+ const rt = ManagedRuntime.make(GitEffect.defaultLayer)
+ try {
+ return await fn(rt)
+ } finally {
+ await rt.dispose()
+ }
+}
+
+describe("Git", () => {
+ test("branch() returns current branch name", async () => {
+ await using tmp = await tmpdir({ git: true })
+
+ await withGit(async (rt) => {
+ const branch = await rt.runPromise(GitEffect.Service.use((git) => git.branch(tmp.path)))
+ expect(branch).toBeDefined()
+ expect(typeof branch).toBe("string")
+ })
+ })
+
+ test("branch() returns undefined for non-git directories", async () => {
+ await using tmp = await tmpdir()
+
+ await withGit(async (rt) => {
+ const branch = await rt.runPromise(GitEffect.Service.use((git) => git.branch(tmp.path)))
+ expect(branch).toBeUndefined()
+ })
+ })
+
+ test("defaultBranch() uses init.defaultBranch when available", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await $`git branch -M trunk`.cwd(tmp.path).quiet()
+ await $`git config init.defaultBranch trunk`.cwd(tmp.path).quiet()
+
+ await withGit(async (rt) => {
+ const branch = await rt.runPromise(GitEffect.Service.use((git) => git.defaultBranch(tmp.path)))
+ expect(branch?.name).toBe("trunk")
+ expect(branch?.ref).toBe("trunk")
+ })
+ })
+
+ test("status() handles special filenames", async () => {
+ await using tmp = await tmpdir({ git: true })
+ const file = weird
+ await fs.writeFile(path.join(tmp.path, file), "hello\n", "utf-8")
+
+ await withGit(async (rt) => {
+ const status = await rt.runPromise(GitEffect.Service.use((git) => git.status(tmp.path)))
+ expect(status).toEqual(
+ expect.arrayContaining([
+ expect.objectContaining({
+ file,
+ status: "added",
+ }),
+ ]),
+ )
+ })
+ })
+
+ test("diff(), stats(), and mergeBase() parse tracked changes", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await $`git branch -M main`.cwd(tmp.path).quiet()
+ const file = weird
+ await fs.writeFile(path.join(tmp.path, file), "before\n", "utf-8")
+ await $`git add .`.cwd(tmp.path).quiet()
+ await $`git commit --no-gpg-sign -m "add file"`.cwd(tmp.path).quiet()
+ await $`git checkout -b feature/test`.cwd(tmp.path).quiet()
+ await fs.writeFile(path.join(tmp.path, file), "after\n", "utf-8")
+
+ await withGit(async (rt) => {
+ const [base, diff, stats] = await Promise.all([
+ rt.runPromise(GitEffect.Service.use((git) => git.mergeBase(tmp.path, "main"))),
+ rt.runPromise(GitEffect.Service.use((git) => git.diff(tmp.path, "HEAD"))),
+ rt.runPromise(GitEffect.Service.use((git) => git.stats(tmp.path, "HEAD"))),
+ ])
+
+ expect(base).toBeTruthy()
+ expect(diff).toEqual(
+ expect.arrayContaining([
+ expect.objectContaining({
+ file,
+ status: "modified",
+ }),
+ ]),
+ )
+ expect(stats).toEqual(
+ expect.arrayContaining([
+ expect.objectContaining({
+ file,
+ additions: 1,
+ deletions: 1,
+ }),
+ ]),
+ )
+ })
+ })
+})
diff --git a/packages/opencode/test/project/project.test.ts b/packages/opencode/test/project/project.test.ts
index a71fe0528..43c86c32e 100644
--- a/packages/opencode/test/project/project.test.ts
+++ b/packages/opencode/test/project/project.test.ts
@@ -10,14 +10,16 @@ import { ProjectID } from "../../src/project/schema"
Log.init({ print: false })
-const gitModule = await import("../../src/util/git")
-const originalGit = gitModule.git
+const gitModule = await import("../../src/git")
+const originalGit = gitModule.Git.run
type Mode = "none" | "rev-list-fail" | "top-fail" | "common-dir-fail"
let mode: Mode = "none"
-mock.module("../../src/util/git", () => ({
- git: (args: string[], opts: { cwd: string; env?: Record<string, string> }) => {
+mock.module("../../src/git", () => ({
+ Git: {
+ ...gitModule.Git,
+ run: (args: string[], opts: { cwd: string; env?: Record<string, string> }) => {
const cmd = ["git", ...args].join(" ")
if (
mode === "rev-list-fail" &&
@@ -27,7 +29,7 @@ mock.module("../../src/util/git", () => ({
) {
return Promise.resolve({
exitCode: 128,
- text: () => Promise.resolve(""),
+ text: () => "",
stdout: Buffer.from(""),
stderr: Buffer.from("fatal"),
})
@@ -35,7 +37,7 @@ mock.module("../../src/util/git", () => ({
if (mode === "top-fail" && cmd.includes("git rev-parse") && cmd.includes("--show-toplevel")) {
return Promise.resolve({
exitCode: 128,
- text: () => Promise.resolve(""),
+ text: () => "",
stdout: Buffer.from(""),
stderr: Buffer.from("fatal"),
})
@@ -43,12 +45,13 @@ mock.module("../../src/util/git", () => ({
if (mode === "common-dir-fail" && cmd.includes("git rev-parse") && cmd.includes("--git-common-dir")) {
return Promise.resolve({
exitCode: 128,
- text: () => Promise.resolve(""),
+ text: () => "",
stdout: Buffer.from(""),
stderr: Buffer.from("fatal"),
})
}
return originalGit(args, opts)
+ },
},
}))
diff --git a/packages/opencode/test/project/vcs.test.ts b/packages/opencode/test/project/vcs.test.ts
index 90f445ed7..c0967993f 100644
--- a/packages/opencode/test/project/vcs.test.ts
+++ b/packages/opencode/test/project/vcs.test.ts
@@ -23,7 +23,7 @@ function withVcs(
) {
return withServices(
directory,
- Layer.merge(FileWatcher.layer, Vcs.layer),
+ Layer.merge(FileWatcher.layer, Vcs.defaultLayer),
async (rt) => {
await rt.runPromise(FileWatcher.Service.use(() => Effect.void))
await rt.runPromise(Vcs.Service.use(() => Effect.void))
@@ -34,7 +34,15 @@ function withVcs(
)
}
+function withVcsOnly(
+ directory: string,
+ body: (rt: ManagedRuntime.ManagedRuntime<Vcs.Service, never>) => Promise<void>,
+) {
+ return withServices(directory, Vcs.defaultLayer, body)
+}
+
type BranchEvent = { directory?: string; payload: { type: string; properties: { branch?: string } } }
+const weird = process.platform === "win32" ? "space file.txt" : "tab\tfile.txt"
/** Wait for a Vcs.Event.BranchUpdated event on GlobalBus, with retry polling as fallback */
function nextBranchUpdate(directory: string, timeout = 10_000) {
@@ -121,3 +129,104 @@ describeVcs("Vcs", () => {
})
})
})
+
+describe("Vcs diff", () => {
+ afterEach(() => Instance.disposeAll())
+
+ test("defaultBranch() falls back to main", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await $`git branch -M main`.cwd(tmp.path).quiet()
+
+ await withVcsOnly(tmp.path, async (rt) => {
+ const branch = await rt.runPromise(Vcs.Service.use((s) => s.defaultBranch()))
+ expect(branch).toBe("main")
+ })
+ })
+
+ test("defaultBranch() uses init.defaultBranch when available", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await $`git branch -M trunk`.cwd(tmp.path).quiet()
+ await $`git config init.defaultBranch trunk`.cwd(tmp.path).quiet()
+
+ await withVcsOnly(tmp.path, async (rt) => {
+ const branch = await rt.runPromise(Vcs.Service.use((s) => s.defaultBranch()))
+ expect(branch).toBe("trunk")
+ })
+ })
+
+ test("detects current branch from the active worktree", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await using wt = await tmpdir()
+ await $`git branch -M main`.cwd(tmp.path).quiet()
+ const dir = path.join(wt.path, "feature")
+ await $`git worktree add -b feature/test ${dir} HEAD`.cwd(tmp.path).quiet()
+
+ await withVcsOnly(dir, async (rt) => {
+ const [branch, base] = await Promise.all([
+ rt.runPromise(Vcs.Service.use((s) => s.branch())),
+ rt.runPromise(Vcs.Service.use((s) => s.defaultBranch())),
+ ])
+ expect(branch).toBe("feature/test")
+ expect(base).toBe("main")
+ })
+ })
+
+ test("diff('git') returns uncommitted changes", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await fs.writeFile(path.join(tmp.path, "file.txt"), "original\n", "utf-8")
+ await $`git add .`.cwd(tmp.path).quiet()
+ await $`git commit --no-gpg-sign -m "add file"`.cwd(tmp.path).quiet()
+ await fs.writeFile(path.join(tmp.path, "file.txt"), "changed\n", "utf-8")
+
+ await withVcsOnly(tmp.path, async (rt) => {
+ const diff = await rt.runPromise(Vcs.Service.use((s) => s.diff("git")))
+ expect(diff).toEqual(
+ expect.arrayContaining([
+ expect.objectContaining({
+ file: "file.txt",
+ status: "modified",
+ }),
+ ]),
+ )
+ })
+ })
+
+ test("diff('git') handles special filenames", async () => {
+ await using tmp = await tmpdir({ git: true })
+ const file = weird
+ await fs.writeFile(path.join(tmp.path, file), "hello\n", "utf-8")
+
+ await withVcsOnly(tmp.path, async (rt) => {
+ const diff = await rt.runPromise(Vcs.Service.use((s) => s.diff("git")))
+ expect(diff).toEqual(
+ expect.arrayContaining([
+ expect.objectContaining({
+ file,
+ status: "added",
+ }),
+ ]),
+ )
+ })
+ })
+
+ test("diff('branch') returns changes against default branch", async () => {
+ await using tmp = await tmpdir({ git: true })
+ await $`git branch -M main`.cwd(tmp.path).quiet()
+ await $`git checkout -b feature/test`.cwd(tmp.path).quiet()
+ await fs.writeFile(path.join(tmp.path, "branch.txt"), "hello\n", "utf-8")
+ await $`git add .`.cwd(tmp.path).quiet()
+ await $`git commit --no-gpg-sign -m "branch file"`.cwd(tmp.path).quiet()
+
+ await withVcsOnly(tmp.path, async (rt) => {
+ const diff = await rt.runPromise(Vcs.Service.use((s) => s.diff("branch")))
+ expect(diff).toEqual(
+ expect.arrayContaining([
+ expect.objectContaining({
+ file: "branch.txt",
+ status: "added",
+ }),
+ ]),
+ )
+ })
+ })
+})
diff --git a/packages/sdk/js/src/v2/gen/sdk.gen.ts b/packages/sdk/js/src/v2/gen/sdk.gen.ts
index aa759bb1e..6ec934e6b 100644
--- a/packages/sdk/js/src/v2/gen/sdk.gen.ts
+++ b/packages/sdk/js/src/v2/gen/sdk.gen.ts
@@ -172,6 +172,7 @@ import type {
TuiSelectSessionResponses,
TuiShowToastResponses,
TuiSubmitPromptResponses,
+ VcsDiffResponses,
VcsGetResponses,
WorktreeCreateErrors,
WorktreeCreateInput,
@@ -3661,6 +3662,38 @@ export class Vcs extends HeyApiClient {
...params,
})
}
+
+ /**
+ * Get VCS diff
+ *
+ * Retrieve the current git diff for the working tree or against the default branch.
+ */
+ public diff<ThrowOnError extends boolean = false>(
+ parameters: {
+ directory?: string
+ workspace?: string
+ mode: "git" | "branch"
+ },
+ options?: Options<never, ThrowOnError>,
+ ) {
+ const params = buildClientParams(
+ [parameters],
+ [
+ {
+ args: [
+ { in: "query", key: "directory" },
+ { in: "query", key: "workspace" },
+ { in: "query", key: "mode" },
+ ],
+ },
+ ],
+ )
+ return (options?.client ?? this.client).get<VcsDiffResponses, unknown, ThrowOnError>({
+ url: "/vcs/diff",
+ ...options,
+ ...params,
+ })
+ }
}
export class Command extends HeyApiClient {
diff --git a/packages/sdk/js/src/v2/gen/types.gen.ts b/packages/sdk/js/src/v2/gen/types.gen.ts
index 41aa24817..942e1c3e1 100644
--- a/packages/sdk/js/src/v2/gen/types.gen.ts
+++ b/packages/sdk/js/src/v2/gen/types.gen.ts
@@ -1889,7 +1889,8 @@ export type Path = {
}
export type VcsInfo = {
- branch: string
+ branch?: string
+ default_branch?: string
}
export type Command = {
@@ -4833,6 +4834,26 @@ export type VcsGetResponses = {
export type VcsGetResponse = VcsGetResponses[keyof VcsGetResponses]
+export type VcsDiffData = {
+ body?: never
+ path?: never
+ query: {
+ directory?: string
+ workspace?: string
+ mode: "git" | "branch"
+ }
+ url: "/vcs/diff"
+}
+
+export type VcsDiffResponses = {
+ /**
+ * VCS diff
+ */
+ 200: Array<FileDiff>
+}
+
+export type VcsDiffResponse = VcsDiffResponses[keyof VcsDiffResponses]
+
export type CommandListData = {
body?: never
path?: never
diff --git a/packages/ui/src/i18n/en.ts b/packages/ui/src/i18n/en.ts
index 18823aeaa..d9f724fce 100644
--- a/packages/ui/src/i18n/en.ts
+++ b/packages/ui/src/i18n/en.ts
@@ -1,5 +1,7 @@
export const dict: Record<string, string> = {
"ui.sessionReview.title": "Session changes",
+ "ui.sessionReview.title.git": "Git changes",
+ "ui.sessionReview.title.branch": "Branch changes",
"ui.sessionReview.title.lastTurn": "Last turn changes",
"ui.sessionReview.diffStyle.unified": "Unified",
"ui.sessionReview.diffStyle.split": "Split",