b4836589f4
This reverts: -28b03595bresearch: delete Hono backend (do not merge) (#25667) -b24a4e897chore(server): clean up post-Hono-deletion scar tissue (#26542) v1.14.42 broke startup for users with plugins that depend on the Hono wire format (most visibly opencode-gemini-auth, see #26546). Restoring Hono as the default backend on stable channels while we investigate the actual plugin compatibility story. OPENCODE_EXPERIMENTAL_HTTPAPI flag and dual-backend selection come back. Stable installs default to Hono; dev/beta default to HTTP API. Conflict resolution: took the pre-deletion side for control-plane schemas and the seven test files where post-deletion follow-up PRs had also touched the conflicting lines. The HTTP API code added since the deletion (compression, cors-vary, fence, lifecycle log, account error mapping, etc.) is preserved as-is — those still apply on the HTTP API path for users on dev/beta channels.
86 lines
2.8 KiB
TypeScript
86 lines
2.8 KiB
TypeScript
import { afterEach, expect, test } from "bun:test"
|
|
import { Hono } from "hono"
|
|
import { existsSync } from "node:fs"
|
|
import path from "node:path"
|
|
import { pathToFileURL } from "node:url"
|
|
import { bootstrap as cliBootstrap } from "../../src/cli/bootstrap"
|
|
import { Instance } from "../../src/project/instance"
|
|
import { WithInstance } from "../../src/project/with-instance"
|
|
import { InstanceRuntime } from "../../src/project/instance-runtime"
|
|
import { InstanceMiddleware } from "../../src/server/routes/instance/middleware"
|
|
import { disposeAllInstances, tmpdir } from "../fixture/fixture"
|
|
|
|
// These regressions cover the legacy instance-loading paths fixed by PRs
|
|
// #25389 and #25449. The plugin config hook writes a marker file, and the test
|
|
// bodies deliberately avoid touching Plugin or config directly. The marker only
|
|
// exists if InstanceBootstrap ran at the instance boundary.
|
|
|
|
afterEach(async () => {
|
|
await disposeAllInstances()
|
|
})
|
|
|
|
async function bootstrapFixture() {
|
|
return tmpdir({
|
|
init: async (dir) => {
|
|
const marker = path.join(dir, "config-hook-fired")
|
|
const pluginFile = path.join(dir, "plugin.ts")
|
|
await Bun.write(
|
|
pluginFile,
|
|
[
|
|
`const MARKER = ${JSON.stringify(marker)}`,
|
|
"export default async () => ({",
|
|
" config: async () => {",
|
|
' await Bun.write(MARKER, "ran")',
|
|
" },",
|
|
"})",
|
|
"",
|
|
].join("\n"),
|
|
)
|
|
await Bun.write(
|
|
path.join(dir, "opencode.json"),
|
|
JSON.stringify({
|
|
$schema: "https://opencode.ai/config.json",
|
|
plugin: [pathToFileURL(pluginFile).href],
|
|
}),
|
|
)
|
|
return marker
|
|
},
|
|
})
|
|
}
|
|
|
|
test("Instance.provide runs InstanceBootstrap before fn (boundary invariant)", async () => {
|
|
await using tmp = await bootstrapFixture()
|
|
|
|
await WithInstance.provide({
|
|
directory: tmp.path,
|
|
fn: async () => "ok",
|
|
})
|
|
|
|
expect(existsSync(tmp.extra)).toBe(true)
|
|
})
|
|
|
|
test("CLI bootstrap runs InstanceBootstrap before callback", async () => {
|
|
await using tmp = await bootstrapFixture()
|
|
|
|
await cliBootstrap(tmp.path, async () => "ok")
|
|
|
|
expect(existsSync(tmp.extra)).toBe(true)
|
|
})
|
|
|
|
test("legacy Hono instance middleware runs InstanceBootstrap before next handler", async () => {
|
|
await using tmp = await bootstrapFixture()
|
|
const app = new Hono().use(InstanceMiddleware()).get("/probe", (c) => c.text("ok"))
|
|
|
|
const response = await app.request("/probe", { headers: { "x-opencode-directory": tmp.path } })
|
|
|
|
expect(response.status).toBe(200)
|
|
expect(existsSync(tmp.extra)).toBe(true)
|
|
})
|
|
|
|
test("InstanceRuntime.reloadInstance runs InstanceBootstrap", async () => {
|
|
await using tmp = await bootstrapFixture()
|
|
|
|
await InstanceRuntime.reloadInstance({ directory: tmp.path })
|
|
|
|
expect(existsSync(tmp.extra)).toBe(true)
|
|
})
|