antechamber 1.0.25 → 1.0.27
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +16 -0
- package/dist/mcp.d.ts +2 -0
- package/dist/mcp.js +17 -8
- package/dist/pi-event-translation.d.ts +7 -0
- package/dist/pi-event-translation.js +32 -5
- package/extensions/omo-compat/package.json +1 -1
- package/extensions/omo-dag/package.json +1 -1
- package/extensions/omo-work/package.json +1 -1
- package/package.json +1 -1
package/CHANGELOG.md
CHANGED
|
@@ -7,6 +7,22 @@ and this project uses [Semantic Versioning](https://semver.org/).
|
|
|
7
7
|
|
|
8
8
|
## [Unreleased]
|
|
9
9
|
|
|
10
|
+
## [1.0.27] - 2026-10-28
|
|
11
|
+
|
|
12
|
+
### Fixed
|
|
13
|
+
|
|
14
|
+
- Find OmO's builtin `/mcp` command when an extension such as `pi-mcp-adapter` also registers `mcp` and OmO renames
|
|
15
|
+
the builtin to `mcp:1`. MCP sign-in used to fail there with "Unknown command /mcp".
|
|
16
|
+
|
|
17
|
+
## [1.0.26] - 2026-10-28
|
|
18
|
+
|
|
19
|
+
### Fixed
|
|
20
|
+
|
|
21
|
+
- Stop showing "Opencode failed to send message" for a provider error that OmO retries on its own, such as
|
|
22
|
+
Anthropic's 400 for a `web_search` tool use with no `web_search_tool_result`, which OmO retries without web search
|
|
23
|
+
replay data. The errored reply now waits for OmO's retry decision: a retried step completes without the error and
|
|
24
|
+
v2 no longer reports `session.step.failed`. An error that is not retried still fails the message, step and run.
|
|
25
|
+
|
|
10
26
|
## [1.0.25] - 2026-10-27
|
|
11
27
|
|
|
12
28
|
### Added
|
package/dist/mcp.d.ts
CHANGED
|
@@ -17,6 +17,8 @@
|
|
|
17
17
|
* Commands are sent straight to the agent, outside the conversation queue: they neither
|
|
18
18
|
* wait behind a running turn nor create a user message, and the session is held active so
|
|
19
19
|
* the reaper cannot stop the process (and its listener or pending state) mid-flow.
|
|
20
|
+
* An extension that registers its own `mcp` command makes OmO rename the builtin (`mcp:1`),
|
|
21
|
+
* so the builtin is found by its source, not by name.
|
|
20
22
|
*/
|
|
21
23
|
import type { RouteServices } from "./route-services.js";
|
|
22
24
|
type OCMcpStatus = {
|
package/dist/mcp.js
CHANGED
|
@@ -17,6 +17,8 @@
|
|
|
17
17
|
* Commands are sent straight to the agent, outside the conversation queue: they neither
|
|
18
18
|
* wait behind a running turn nor create a user message, and the session is held active so
|
|
19
19
|
* the reaper cannot stop the process (and its listener or pending state) mid-flow.
|
|
20
|
+
* An extension that registers its own `mcp` command makes OmO rename the builtin (`mcp:1`),
|
|
21
|
+
* so the builtin is found by its source, not by name.
|
|
20
22
|
*/
|
|
21
23
|
import { randomUUID } from "node:crypto";
|
|
22
24
|
import { errorBody } from "./http.js";
|
|
@@ -32,6 +34,7 @@ const MCP_AUTH_ATTEMPT_TTL_MS = 10 * 60_000;
|
|
|
32
34
|
const SERVER_NAME = /^[^\s"'\\]+$/;
|
|
33
35
|
const INTEGRATION_PREFIX = "mcp_";
|
|
34
36
|
const OAUTH_METHOD_ID = "oauth";
|
|
37
|
+
const BUILTIN_MCP_PATH = "<builtin:mcp>";
|
|
35
38
|
const flows = new Map();
|
|
36
39
|
const admissions = new Map();
|
|
37
40
|
function isRecord(value) {
|
|
@@ -95,11 +98,15 @@ async function directoryServers(sessions, directory) {
|
|
|
95
98
|
const live = liveSession(sessions, directory) ?? (fallback === undefined ? undefined : { pi: fallback });
|
|
96
99
|
return live === undefined ? [] : loadedServers(live.pi).catch(() => []);
|
|
97
100
|
}
|
|
98
|
-
|
|
101
|
+
/** The builtin MCP command's name in this agent; builds without `sourceInfo` only know it as `mcp`. */
|
|
102
|
+
async function mcpCommandName(pi) {
|
|
99
103
|
const response = await pi.probe({ type: "get_commands" }, undefined, AbortSignal.timeout(MCP_READ_WAIT_MS));
|
|
100
104
|
if (!response.success || !isRecord(response.data) || !Array.isArray(response.data.commands))
|
|
101
|
-
return
|
|
102
|
-
|
|
105
|
+
return undefined;
|
|
106
|
+
const commands = response.data.commands.flatMap((command) => isRecord(command) && command.source === "extension" && typeof command.name === "string"
|
|
107
|
+
? [{ name: command.name, builtin: isRecord(command.sourceInfo) && command.sourceInfo.path === BUILTIN_MCP_PATH }]
|
|
108
|
+
: []);
|
|
109
|
+
return (commands.find((command) => command.builtin) ?? commands.find((command) => command.name === "mcp"))?.name;
|
|
103
110
|
}
|
|
104
111
|
/** The URL OmO prints on its own line after naming the server (`auth` and `auth-start` both do). */
|
|
105
112
|
export function authorizationUrl(text, name) {
|
|
@@ -151,7 +158,7 @@ function sameCallback(target, authorization) {
|
|
|
151
158
|
const expected = new URL(announced);
|
|
152
159
|
return target.origin === expected.origin && target.pathname === expected.pathname;
|
|
153
160
|
}
|
|
154
|
-
function runMcpCommand(sessions, s, pi, name, args, observe = {}) {
|
|
161
|
+
function runMcpCommand(sessions, s, pi, name, commandName, args, observe = {}) {
|
|
155
162
|
const errors = [];
|
|
156
163
|
const onEvent = (event) => {
|
|
157
164
|
const entry = event.entry;
|
|
@@ -175,7 +182,7 @@ function runMcpCommand(sessions, s, pi, name, args, observe = {}) {
|
|
|
175
182
|
pi.onEvent(onEvent);
|
|
176
183
|
pi.onExtensionUI(onNotify);
|
|
177
184
|
return sessions.lifecycle
|
|
178
|
-
.use(s, () => pi.command({ type: "prompt", message:
|
|
185
|
+
.use(s, () => pi.command({ type: "prompt", message: `/${commandName} ${args}`, expandPromptTemplates: true }, MCP_AUTH_COMMAND_TIMEOUT_MS))
|
|
179
186
|
.then((response) => {
|
|
180
187
|
if (!response.success)
|
|
181
188
|
throw new Error(response.error ?? "OmO rejected the /mcp command");
|
|
@@ -233,12 +240,14 @@ async function startFlow(services, directory, name, command) {
|
|
|
233
240
|
const servers = await loadedServers(pi);
|
|
234
241
|
if (!servers.some((server) => server.name === name))
|
|
235
242
|
return { error: { status: 404, body: errorBody("McpServerNotFoundError", `MCP server not found: ${name}`) } };
|
|
236
|
-
|
|
243
|
+
const commandName = await mcpCommandName(pi);
|
|
244
|
+
if (commandName === undefined)
|
|
237
245
|
return { error: { status: 400, body: errorBody("UnsupportedError", "This OmO build has no /mcp command") } };
|
|
238
246
|
const url = Promise.withResolvers();
|
|
239
247
|
const flow = {
|
|
240
248
|
s,
|
|
241
249
|
pi,
|
|
250
|
+
commandName,
|
|
242
251
|
command,
|
|
243
252
|
attemptID: randomUUID(),
|
|
244
253
|
created: Date.now(),
|
|
@@ -247,7 +256,7 @@ async function startFlow(services, directory, name, command) {
|
|
|
247
256
|
running: true,
|
|
248
257
|
authorized: false,
|
|
249
258
|
};
|
|
250
|
-
flow.done = runMcpCommand(services.sessions, s, pi, name, `${command} ${name}`, {
|
|
259
|
+
flow.done = runMcpCommand(services.sessions, s, pi, name, commandName, `${command} ${name}`, {
|
|
251
260
|
url: (found) => {
|
|
252
261
|
flow.authorizationUrl ??= found;
|
|
253
262
|
url.resolve(found);
|
|
@@ -326,7 +335,7 @@ async function completeFlow(services, directory, name, input) {
|
|
|
326
335
|
await flow.done;
|
|
327
336
|
}
|
|
328
337
|
else {
|
|
329
|
-
await runMcpCommand(services.sessions, flow.s, flow.pi, name, `auth-complete ${name} ${redirect}`, {
|
|
338
|
+
await runMcpCommand(services.sessions, flow.s, flow.pi, name, flow.commandName, `auth-complete ${name} ${redirect}`, {
|
|
330
339
|
authorized: () => {
|
|
331
340
|
flow.authorized = true;
|
|
332
341
|
},
|
|
@@ -57,6 +57,11 @@ export declare class PiEventTranslator {
|
|
|
57
57
|
private services;
|
|
58
58
|
/** Ended assistant messages whose completion waits on their tools. */
|
|
59
59
|
private awaitingTools;
|
|
60
|
+
/**
|
|
61
|
+
* Errored assistant messages held open until agent_end says whether OmO retries them. OpenChamber prints a
|
|
62
|
+
* completed message's error as "failed to send message", and v2 cannot retract a `session.step.failed`.
|
|
63
|
+
*/
|
|
64
|
+
private heldFailures;
|
|
60
65
|
constructor(services: PiEventServices);
|
|
61
66
|
/** Close dead-runtime UI state before an error or idle event can reach the client. */
|
|
62
67
|
finishInterruptedRun(s: ManagedSession, error: NonNullable<OCAssistantMessage["error"]>): void;
|
|
@@ -93,6 +98,8 @@ export declare class PiEventTranslator {
|
|
|
93
98
|
private completeSettled;
|
|
94
99
|
/** Tool parts that never received a result (abort, crash) must not spin forever. */
|
|
95
100
|
private finalizeDanglingTools;
|
|
101
|
+
/** Complete held errored messages: with their error when the run failed, without it when OmO retries the step. */
|
|
102
|
+
private completeHeldFailures;
|
|
96
103
|
/** Complete every ended message of this session still waiting on its tools. */
|
|
97
104
|
private completeAwaitingTools;
|
|
98
105
|
}
|
|
@@ -168,6 +168,11 @@ export class PiEventTranslator {
|
|
|
168
168
|
services;
|
|
169
169
|
/** Ended assistant messages whose completion waits on their tools. */
|
|
170
170
|
awaitingTools = new WeakSet();
|
|
171
|
+
/**
|
|
172
|
+
* Errored assistant messages held open until agent_end says whether OmO retries them. OpenChamber prints a
|
|
173
|
+
* completed message's error as "failed to send message", and v2 cannot retract a `session.step.failed`.
|
|
174
|
+
*/
|
|
175
|
+
heldFailures = new WeakMap();
|
|
171
176
|
constructor(services) {
|
|
172
177
|
this.services = services;
|
|
173
178
|
}
|
|
@@ -394,8 +399,11 @@ export class PiEventTranslator {
|
|
|
394
399
|
this.setStatus(s, { type: "busy" });
|
|
395
400
|
return;
|
|
396
401
|
case "agent_end":
|
|
397
|
-
if (ev.willRetry === true)
|
|
398
|
-
|
|
402
|
+
if (ev.willRetry === true) {
|
|
403
|
+
// auto-retry keeps the session busy, and the failed step it retries is not a failure to report.
|
|
404
|
+
this.completeHeldFailures(s, false);
|
|
405
|
+
return;
|
|
406
|
+
}
|
|
399
407
|
if (s.interruptRequested)
|
|
400
408
|
this.finishInterruptedRun(s, { name: "MessageAbortedError", data: { message: "Cancelled" } });
|
|
401
409
|
s.approvals?.finishTurn();
|
|
@@ -599,15 +607,19 @@ export class PiEventTranslator {
|
|
|
599
607
|
s.streaming.message.modelID = s.model.modelID;
|
|
600
608
|
}
|
|
601
609
|
applyUsage(s.streaming.message, ended.usage);
|
|
602
|
-
|
|
610
|
+
const stop = piStopReason(ended.stopReason, ended.errorMessage);
|
|
611
|
+
if (ended.stopReason === "error")
|
|
612
|
+
this.heldFailures.set(s.streaming.message, stop);
|
|
613
|
+
else
|
|
614
|
+
Object.assign(s.streaming.message, stop);
|
|
603
615
|
// OpenCode completes an assistant message only after its tools settle, and
|
|
604
616
|
// OpenChamber marks active tools under a completed message as Interrupted.
|
|
605
617
|
// pi ends a toolUse message before executing its tool calls, so defer its
|
|
606
618
|
// completion to the last tool result (applyToolResults), the next step, or
|
|
607
|
-
// the run end. Errored/aborted messages never run their tools.
|
|
619
|
+
// the run end. Errored/aborted messages never run their tools; an errored one completes at agent_end.
|
|
608
620
|
if (ended.stopReason === "toolUse" && hasActiveTools(s, s.streaming.message.id))
|
|
609
621
|
this.awaitingTools.add(s.streaming.message);
|
|
610
|
-
else
|
|
622
|
+
else if (ended.stopReason !== "error")
|
|
611
623
|
s.streaming.message.time.completed = Date.now();
|
|
612
624
|
this.services.emit(s.info.directory, { type: "message.updated", properties: { info: s.streaming.message } });
|
|
613
625
|
this.finishStreaming(s);
|
|
@@ -968,8 +980,23 @@ export class PiEventTranslator {
|
|
|
968
980
|
s.toolByCallId.clear();
|
|
969
981
|
this.completeAwaitingTools(s); // the run is over
|
|
970
982
|
}
|
|
983
|
+
/** Complete held errored messages: with their error when the run failed, without it when OmO retries the step. */
|
|
984
|
+
completeHeldFailures(s, failed) {
|
|
985
|
+
const now = Date.now();
|
|
986
|
+
for (const { info } of s.messages ?? []) {
|
|
987
|
+
const stop = this.heldFailures.get(info);
|
|
988
|
+
if (stop === undefined || info.role !== "assistant")
|
|
989
|
+
continue;
|
|
990
|
+
this.heldFailures.delete(info);
|
|
991
|
+
if (failed)
|
|
992
|
+
Object.assign(info, stop);
|
|
993
|
+
info.time.completed = now;
|
|
994
|
+
this.services.emit(s.info.directory, { type: "message.updated", properties: { info } });
|
|
995
|
+
}
|
|
996
|
+
}
|
|
971
997
|
/** Complete every ended message of this session still waiting on its tools. */
|
|
972
998
|
completeAwaitingTools(s) {
|
|
999
|
+
this.completeHeldFailures(s, true);
|
|
973
1000
|
const now = Date.now();
|
|
974
1001
|
for (const { info } of s.messages ?? []) {
|
|
975
1002
|
if (!this.awaitingTools.has(info) || info.role !== "assistant")
|