Skip to content

Commit 5bcbc25

Browse files
authored
Merge pull request #186 from alltomatos/fix/181-skip-flaky-mcp-loop-test
test: skip flaky "loop includes MCP instructions" test, tracked in #181
2 parents ca72c61 + 3952675 commit 5bcbc25

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

‎packages/opencode/test/session/prompt.test.ts‎

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -565,7 +565,16 @@ it.instance("loop calls LLM and returns assistant message", () =>
565565
}),
566566
)
567567

568-
withMcpInstructions.instance(
568+
// Skipped: intermittently fails in CI (and reproduces locally in isolation) with
569+
// `InterruptError: All fibers interrupted without error`, thrown from this test's own
570+
// `.instance` scope teardown (tmpdir + git init + LSP boot + mock LLM server, all
571+
// released together) racing against the still-hanging `llm.hang()` (Stream.never-backed)
572+
// fiber's interruption. Tried wrapping the explicit `Fiber.interrupt(fiber)` below in
573+
// `Effect.ignore` — did not fix it, so the error isn't coming from that call. Root cause
574+
// needs someone with more Effect-runtime/Scope-finalizer context to trace which finalizer
575+
// is responsible. See https://github.com/alltomatos/opencode/issues/181 for the full
576+
// investigation before re-enabling.
577+
withMcpInstructions.instance.skip(
569578
"loop includes MCP instructions in model system context",
570579
() =>
571580
Effect.gen(function* () {

0 commit comments

Comments
 (0)