+21

![opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com>](/assets/img/avatar_default.png)



![opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>](/assets/img/avatar_default.png)



James Long
Brendan Allan
Kit Langton
opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>
Affan Ali
affanali2k3
Frank
opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com>
𝓛𝓲𝓽𝓽𝓵𝓮 𝓕𝓻𝓪𝓷𝓴
Aiden Cline
Jay V
Dax Raad
Aarav Sareen
OpeOginni
Luke Parker
Ben Guthrie
Dax
Filip
Max Anderson
Brendan Allan
Jack
Shoubhit Dash
Dustin Deus
starptech
Aiden Cline
usrnk1
Jay
runvip
opencode
Julian Coy
Vladimir Glafirov
8c94e9005f
Co-authored-by: Brendan Allan <14191578+Brendonovich@users.noreply.github.com> Co-authored-by: Kit Langton <kit.langton@gmail.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: Affan Ali <93028901+affanali2k3@users.noreply.github.com> Co-authored-by: affanali2k3 <affanalikhanxx@gmail.com> Co-authored-by: Frank <frank@anoma.ly> Co-authored-by: opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com> Co-authored-by: 𝓛𝓲𝓽𝓽𝓵𝓮 𝓕𝓻𝓪𝓷𝓴 <little-frank@opencord.local> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: Jay V <air@live.ca> Co-authored-by: Dax Raad <d@ironbay.co> Co-authored-by: Aarav Sareen <96787824+arvsrn@users.noreply.github.com> Co-authored-by: OpeOginni <107570612+OpeOginni@users.noreply.github.com> Co-authored-by: Luke Parker <10430890+Hona@users.noreply.github.com> Co-authored-by: Ben Guthrie <benjee.012@gmail.com> Co-authored-by: Dax <mail@thdxr.com> Co-authored-by: Filip <34747899+neriousy@users.noreply.github.com> Co-authored-by: Max Anderson <max.a.anderson95@gmail.com> Co-authored-by: Brendan Allan <git@brendonovich.dev> Co-authored-by: Jack <jack@anoma.ly> Co-authored-by: Shoubhit Dash <shoubhit2005@gmail.com> Co-authored-by: Dustin Deus <deusdustin@gmail.com> Co-authored-by: starptech <starptech@starptechs-MBP.fritz.box> Co-authored-by: Aiden Cline <aidenpcline@gmail.com> Co-authored-by: usrnk1 <7547651+usrnk1@users.noreply.github.com> Co-authored-by: Jay <53023+jayair@users.noreply.github.com> Co-authored-by: runvip <164729189+runvip@users.noreply.github.com> Co-authored-by: opencode <opencode@sst.dev> Co-authored-by: Julian Coy <julian@ex-machina.co> Co-authored-by: Vladimir Glafirov <vglafirov@gitlab.com>
249 lines
7.8 KiB
TypeScript
249 lines
7.8 KiB
TypeScript
import { describe, expect } from "bun:test"
|
|
import { LayerNode } from "@opencode-ai/core/effect/layer-node"
|
|
import { CrossSpawnSpawner } from "@opencode-ai/core/cross-spawn-spawner"
|
|
import { Deferred, Effect, Fiber, Layer } from "effect"
|
|
import { InstanceRef } from "../../src/effect/instance-ref"
|
|
import { registerDisposer } from "../../src/effect/instance-registry"
|
|
import { InstanceBootstrap } from "../../src/project/bootstrap"
|
|
import { InstanceStore } from "../../src/project/instance-store"
|
|
import { tmpdirScoped } from "../fixture/fixture"
|
|
import { testEffect } from "../lib/effect"
|
|
|
|
let bootstrapRun: Effect.Effect<void> = Effect.void
|
|
const noopBootstrap = Layer.succeed(
|
|
InstanceBootstrap.Service,
|
|
InstanceBootstrap.Service.of({ run: Effect.suspend(() => bootstrapRun) }),
|
|
)
|
|
|
|
const it = testEffect(
|
|
LayerNode.compile(LayerNode.group([InstanceStore.node, CrossSpawnSpawner.node]), [
|
|
[InstanceStore.bootstrapNode, noopBootstrap],
|
|
]),
|
|
)
|
|
|
|
const setBootstrap = (run: Effect.Effect<void>) =>
|
|
Effect.acquireRelease(
|
|
Effect.sync(() => {
|
|
bootstrapRun = run
|
|
}),
|
|
() =>
|
|
Effect.sync(() => {
|
|
bootstrapRun = Effect.void
|
|
}),
|
|
)
|
|
|
|
const registerDisposerScoped = (disposer: (directory: string) => Promise<void>) =>
|
|
Effect.acquireRelease(
|
|
Effect.sync(() => registerDisposer(disposer)),
|
|
(off) => Effect.sync(off),
|
|
)
|
|
|
|
describe("InstanceStore", () => {
|
|
it.live("loads instance context", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
const ctx = yield* store.load({ directory: dir })
|
|
|
|
expect(ctx.directory).toBe(dir)
|
|
expect(ctx.worktree).toBe(dir)
|
|
}),
|
|
)
|
|
|
|
it.live("runs bootstrap with InstanceRef provided", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
let initializedDirectory: string | undefined
|
|
|
|
yield* setBootstrap(
|
|
Effect.gen(function* () {
|
|
initializedDirectory = (yield* InstanceRef)?.directory
|
|
}),
|
|
)
|
|
yield* store.load({ directory: dir })
|
|
|
|
expect(initializedDirectory).toBe(dir)
|
|
}),
|
|
)
|
|
|
|
it.live("caches loaded instance context by directory", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
let initialized = 0
|
|
|
|
yield* setBootstrap(
|
|
Effect.sync(() => {
|
|
initialized++
|
|
}),
|
|
)
|
|
const first = yield* store.load({ directory: dir })
|
|
const second = yield* store.load({ directory: dir })
|
|
|
|
expect(second).toBe(first)
|
|
expect(initialized).toBe(1)
|
|
}),
|
|
)
|
|
|
|
it.live("dedupes concurrent loads while init is in flight", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
const started = yield* Deferred.make<void>()
|
|
const release = yield* Deferred.make<void>()
|
|
let initialized = 0
|
|
|
|
yield* setBootstrap(
|
|
Effect.gen(function* () {
|
|
initialized++
|
|
yield* Deferred.succeed(started, undefined)
|
|
yield* Deferred.await(release)
|
|
}),
|
|
)
|
|
const first = yield* store.load({ directory: dir }).pipe(Effect.forkScoped)
|
|
|
|
yield* Deferred.await(started)
|
|
|
|
yield* setBootstrap(
|
|
Effect.sync(() => {
|
|
initialized++
|
|
}),
|
|
)
|
|
const second = yield* store.load({ directory: dir }).pipe(Effect.forkScoped)
|
|
|
|
expect(initialized).toBe(1)
|
|
yield* Deferred.succeed(release, undefined)
|
|
|
|
const [firstCtx, secondCtx] = yield* Effect.all([Fiber.join(first), Fiber.join(second)])
|
|
expect(secondCtx).toBe(firstCtx)
|
|
expect(initialized).toBe(1)
|
|
}),
|
|
)
|
|
|
|
it.live("removes failed loads from the cache", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
let attempts = 0
|
|
|
|
yield* setBootstrap(
|
|
Effect.sync(() => {
|
|
attempts++
|
|
throw new Error("init failed")
|
|
}),
|
|
)
|
|
const failed = yield* store.load({ directory: dir }).pipe(
|
|
Effect.as(false),
|
|
Effect.catchCause(() => Effect.succeed(true)),
|
|
)
|
|
|
|
expect(failed).toBe(true)
|
|
|
|
yield* setBootstrap(
|
|
Effect.sync(() => {
|
|
attempts++
|
|
}),
|
|
)
|
|
const ctx = yield* store.load({ directory: dir })
|
|
|
|
expect(ctx.directory).toBe(dir)
|
|
expect(attempts).toBe(2)
|
|
}),
|
|
)
|
|
|
|
it.live("reload replaces the cached context", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
|
|
const first = yield* store.load({ directory: dir })
|
|
const second = yield* store.reload({ directory: dir })
|
|
const cached = yield* store.load({ directory: dir })
|
|
|
|
expect(second).not.toBe(first)
|
|
expect(cached).toBe(second)
|
|
}),
|
|
)
|
|
|
|
it.live("stale dispose does not delete an in-flight reload", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
const reloading = yield* Deferred.make<void>()
|
|
const releaseReload = yield* Deferred.make<void>()
|
|
const disposed: Array<string> = []
|
|
yield* registerDisposerScoped(async (directory) => {
|
|
disposed.push(directory)
|
|
})
|
|
|
|
const first = yield* store.load({ directory: dir })
|
|
yield* setBootstrap(
|
|
Effect.gen(function* () {
|
|
yield* Deferred.succeed(reloading, undefined)
|
|
yield* Deferred.await(releaseReload)
|
|
}),
|
|
)
|
|
const reload = yield* store.reload({ directory: dir }).pipe(Effect.forkScoped)
|
|
|
|
yield* Deferred.await(reloading)
|
|
const staleDispose = yield* store.dispose(first).pipe(Effect.forkScoped)
|
|
yield* Deferred.succeed(releaseReload, undefined)
|
|
|
|
const second = yield* Fiber.join(reload)
|
|
yield* Fiber.join(staleDispose)
|
|
|
|
expect(disposed).toEqual([dir])
|
|
expect(yield* store.load({ directory: dir })).toBe(second)
|
|
}),
|
|
)
|
|
|
|
it.live("dedupes concurrent disposeAll calls", () =>
|
|
Effect.gen(function* () {
|
|
const dir = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
const disposing = yield* Deferred.make<void>()
|
|
const releaseDispose = yield* Deferred.make<() => void>()
|
|
const disposed: Array<string> = []
|
|
yield* registerDisposerScoped((directory) => {
|
|
disposed.push(directory)
|
|
Deferred.doneUnsafe(disposing, Effect.void)
|
|
return new Promise<void>((resolve) => {
|
|
Deferred.doneUnsafe(releaseDispose, Effect.succeed(resolve))
|
|
})
|
|
})
|
|
|
|
yield* store.load({ directory: dir })
|
|
const first = yield* store.disposeAll().pipe(Effect.forkScoped)
|
|
yield* Deferred.await(disposing)
|
|
const release = yield* Deferred.await(releaseDispose)
|
|
const second = yield* store.disposeAll().pipe(Effect.forkScoped)
|
|
|
|
expect(disposed).toEqual([dir])
|
|
yield* Effect.sync(release)
|
|
yield* Effect.all([Fiber.join(first), Fiber.join(second)])
|
|
expect(disposed).toEqual([dir])
|
|
}),
|
|
)
|
|
|
|
it.live("re-arms disposeAll after completion", () =>
|
|
Effect.gen(function* () {
|
|
const dir1 = yield* tmpdirScoped({ git: true })
|
|
const dir2 = yield* tmpdirScoped({ git: true })
|
|
const store = yield* InstanceStore.Service
|
|
const disposed: Array<string> = []
|
|
yield* registerDisposerScoped(async (directory) => {
|
|
disposed.push(directory)
|
|
})
|
|
|
|
yield* store.load({ directory: dir1 })
|
|
yield* store.disposeAll()
|
|
expect(disposed).toEqual([dir1])
|
|
|
|
yield* store.load({ directory: dir2 })
|
|
yield* store.disposeAll()
|
|
expect(disposed).toEqual([dir1, dir2])
|
|
}),
|
|
)
|
|
})
|