Skip to content

Commit a4c6860

Browse files
authored
refactor: destroy Todo facade (#21962)
1 parent face879 commit a4c6860

3 files changed

Lines changed: 9 additions & 17 deletions

File tree

packages/opencode/src/effect/oltp.ts

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -31,14 +31,11 @@ export namespace Observability {
3131

3232
export const layer = !base
3333
? EffectLogger.layer
34-
: Layer.mergeAll(
35-
EffectLogger.layer,
36-
Otlp.layerJson({
37-
baseUrl: base,
38-
loggerExportInterval: Duration.seconds(1),
39-
loggerMergeWithExisting: true,
40-
resource,
41-
headers,
42-
}),
43-
).pipe(Layer.provide(FetchHttpClient.layer))
34+
: Otlp.layerJson({
35+
baseUrl: base,
36+
loggerExportInterval: Duration.seconds(1),
37+
loggerMergeWithExisting: true,
38+
resource,
39+
headers,
40+
}).pipe(Layer.provide(EffectLogger.layer), Layer.provide(FetchHttpClient.layer))
4441
}

packages/opencode/src/server/routes/session.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import { SessionShare } from "@/share/session"
1313
import { SessionStatus } from "@/session/status"
1414
import { SessionSummary } from "@/session/summary"
1515
import { Todo } from "../../session/todo"
16+
import { AppRuntime } from "../../effect/app-runtime"
1617
import { Agent } from "../../agent/agent"
1718
import { Snapshot } from "@/snapshot"
1819
import { Command } from "../../command"
@@ -185,7 +186,7 @@ export const SessionRoutes = lazy(() =>
185186
),
186187
async (c) => {
187188
const sessionID = c.req.valid("param").sessionID
188-
const todos = await Todo.get(sessionID)
189+
const todos = await AppRuntime.runPromise(Todo.Service.use((svc) => svc.get(sessionID)))
189190
return c.json(todos)
190191
},
191192
)

packages/opencode/src/session/todo.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { BusEvent } from "@/bus/bus-event"
22
import { Bus } from "@/bus"
3-
import { makeRuntime } from "@/effect/run-service"
43
import { SessionID } from "./schema"
54
import { Effect, Layer, ServiceMap } from "effect"
65
import z from "zod"
@@ -83,9 +82,4 @@ export namespace Todo {
8382
)
8483

8584
export const defaultLayer = layer.pipe(Layer.provide(Bus.layer))
86-
const { runPromise } = makeRuntime(Service, defaultLayer)
87-
88-
export async function get(sessionID: SessionID) {
89-
return runPromise((svc) => svc.get(sessionID))
90-
}
9185
}

0 commit comments

Comments
 (0)