Skip to content

Commit d9f7c94

Browse files
Apply PR #22753: core: move plugin intialisation to config layer override
2 parents 5d9886d + b1db69f commit d9f7c94

5 files changed

Lines changed: 42 additions & 17 deletions

File tree

packages/opencode/src/cli/cmd/serve.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import os from "node:os"
44
import { Server } from "../../server/server"
55
import { cmd } from "./cmd"
66
import { withNetworkOptions, resolveNetworkOptions } from "../network"
7+
import { bootstrap } from "../bootstrap"
78
import { Flag } from "../../flag/flag"
89
import { Workspace } from "../../control-plane/workspace"
910
import { Project } from "../../project"
@@ -199,7 +200,7 @@ export const ServeCommand = cmd({
199200
}),
200201
describe: "starts a headless opencode server",
201202
handler: async (args) => {
202-
const opts = await resolveNetworkOptions(args)
203+
const opts = await bootstrap(process.cwd(), () => resolveNetworkOptions(args))
203204
const relayURL = (
204205
args["relay-url"] ??
205206
process.env.OPENCODE_EXPERIMENTAL_PUSH_RELAY_URL ??

packages/opencode/src/cli/cmd/tui/thread.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { UI } from "@/cli/ui"
88
import { Log } from "@/util"
99
import { errorMessage } from "@/util/error"
1010
import { withTimeout } from "@/util/timeout"
11+
import { Instance } from "@/project/instance"
1112
import { withNetworkOptions, resolveNetworkOptionsNoConfig } from "@/cli/network"
1213
import { Filesystem } from "@/util"
1314
import type { GlobalEvent } from "@opencode-ai/sdk/v2"
@@ -178,7 +179,11 @@ export const TuiThreadCommand = cmd({
178179
const prompt = await input(args.prompt)
179180
const config = await TuiConfig.get()
180181

181-
const network = resolveNetworkOptionsNoConfig(args)
182+
const network = await Instance.provide({
183+
directory: cwd,
184+
fn: () => resolveNetworkOptionsNoConfig(args),
185+
})
186+
182187
const external =
183188
process.argv.includes("--port") ||
184189
process.argv.includes("--hostname") ||

packages/opencode/src/cli/cmd/web.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { withNetworkOptions, resolveNetworkOptions } from "../network"
55
import { Flag } from "../../flag/flag"
66
import open from "open"
77
import { networkInterfaces } from "os"
8+
import { bootstrap } from "../bootstrap"
89

910
function getNetworkIPs() {
1011
const nets = networkInterfaces()
@@ -36,7 +37,7 @@ export const WebCommand = cmd({
3637
if (!Flag.OPENCODE_SERVER_PASSWORD) {
3738
UI.println(UI.Style.TEXT_WARNING_BOLD + "! OPENCODE_SERVER_PASSWORD is not set; server is unsecured.")
3839
}
39-
const opts = await resolveNetworkOptions(args)
40+
const opts = await bootstrap(process.cwd(), () => resolveNetworkOptions(args))
4041
const server = await Server.listen(opts)
4142
UI.empty()
4243
UI.println(UI.logo(" "))

packages/opencode/src/effect/app-runtime.ts

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,14 +47,32 @@ import { Installation } from "@/installation"
4747
import { ShareNext } from "@/share"
4848
import { SessionShare } from "@/share"
4949
import { Npm } from "@opencode-ai/shared/npm"
50+
import * as Effect from "effect/Effect"
51+
52+
// Adjusts the default Config layer to ensure that plugins are always initialised before
53+
// any other layers read the current config
54+
const ConfigWithPluginPriority = Layer.effect(
55+
Config.Service,
56+
Effect.gen(function* () {
57+
const config = yield* Config.Service
58+
const plugin = yield* Plugin.Service
59+
60+
return {
61+
...config,
62+
get: () => Effect.andThen(plugin.init(), config.get),
63+
getGlobal: () => Effect.andThen(plugin.init(), config.getGlobal),
64+
getConsoleState: () => Effect.andThen(plugin.init(), config.getConsoleState),
65+
}
66+
}),
67+
).pipe(Layer.provide(Layer.merge(Plugin.defaultLayer, Config.defaultLayer)))
5068

5169
export const AppLayer = Layer.mergeAll(
5270
Npm.defaultLayer,
5371
AppFileSystem.defaultLayer,
5472
Bus.defaultLayer,
5573
Auth.defaultLayer,
5674
Account.defaultLayer,
57-
Config.defaultLayer,
75+
ConfigWithPluginPriority,
5876
Git.defaultLayer,
5977
Ripgrep.defaultLayer,
6078
File.defaultLayer,

packages/opencode/src/project/bootstrap.ts

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import { Plugin } from "../plugin"
21
import { Format } from "../format"
32
import { LSP } from "../lsp"
43
import { File } from "../file"
@@ -8,6 +7,7 @@ import * as Vcs from "./vcs"
87
import { Bus } from "../bus"
98
import { Command } from "../command"
109
import { Instance } from "./instance"
10+
import { Plugin } from "../plugin"
1111
import { Log } from "@/util"
1212
import { FileWatcher } from "@/file/watcher"
1313
import { ShareNext } from "@/share"
@@ -16,20 +16,20 @@ import { Config } from "@/config"
1616

1717
export const InstanceBootstrap = Effect.gen(function* () {
1818
Log.Default.info("bootstrapping", { directory: Instance.directory })
19-
// everything depends on config so eager load it for nice traces
20-
yield* Config.Service.use((svc) => svc.get())
21-
// Plugin can mutate config so it has to be initialized before anything else.
22-
yield* Plugin.Service.use((svc) => svc.init())
2319
yield* Effect.all(
2420
[
25-
LSP.Service,
26-
ShareNext.Service,
27-
Format.Service,
28-
File.Service,
29-
FileWatcher.Service,
30-
Vcs.Service,
31-
Snapshot.Service,
32-
].map((s) => Effect.forkDetach(s.use((i) => i.init()))),
21+
Config.Service.use((i) => i.get()),
22+
...[
23+
Plugin.Service,
24+
LSP.Service,
25+
ShareNext.Service,
26+
Format.Service,
27+
File.Service,
28+
FileWatcher.Service,
29+
Vcs.Service,
30+
Snapshot.Service,
31+
].map((s) => s.use((i) => i.init())),
32+
].map((e) => Effect.forkDetach(e)),
3333
).pipe(Effect.withSpan("InstanceBootstrap.init"))
3434

3535
yield* Bus.Service.use((svc) =>

0 commit comments

Comments
 (0)