@keyou007/agent-bot 0.1.24-alpha.1 → 0.1.24-alpha.2

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "@keyou007/agent-bot",
-  "version": "0.1.24-alpha.1",
+  "version": "0.1.24-alpha.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@keyou007/agent-bot",
-      "version": "0.1.24-alpha.1",
+      "version": "0.1.24-alpha.2",
       "license": "GPL-3.0-only",
       "dependencies": {
         "@agentclientprotocol/sdk": "1.2.1",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@keyou007/agent-bot",
-  "version": "0.1.24-alpha.1",
+  "version": "0.1.24-alpha.2",
   "type": "module",
   "description": "Use Codex, TraeX, and compatible coding agents from Lark/Feishu.",
   "license": "GPL-3.0-only",

package/skills/agent-bot/SKILL.md

@@ -125,7 +125,7 @@ agentbot task permissions [auto|confirm]
 
 Omit the value to inspect the current setting and available choices. `agent` changes the default Agent for future tasks in that conversation. The other settings affect the specified task from its next request and become the saved defaults for that Agent.
 
-A successful Provider switch confirms settings, not inference connectivity. Model-list fallback warnings appear on settings cards; inspect the Provider service when availability is unconfirmed. App Server errors and native retry notices appear on the current progress card without terminating the turn. Do not stop or restart a retrying task unless the user requests it.
+A successful Provider switch confirms settings, not inference connectivity. Model-list fallback warnings appear on settings cards; inspect the Provider service when availability is unconfirmed. App Server errors and native retry notices show the message and supplied `additionalDetails` as literal text on the current progress card and Preview without terminating the turn. Use Preview for long reasons; `Reconnecting` alone does not identify the cause when the Agent supplies no details, and old snapshots are not backfilled. Do not stop or restart a retrying task unless the user requests it.
 
 Providers such as a Responses bridge may omit assistant message phases. Unphased text stays in the progress timeline; successful completion promotes only the last message not followed by a tool start. Do not diagnose progress text in the answer area as a CSS issue without checking message phases. Old snapshots are not automatically backfilled.
 

package/src/cli/SelfUpdateRunner.ts

@@ -571,6 +571,7 @@ export function runNpmCommand(args: string[], cwd: string): CommandResult {
     {
       cwd,
       env: process.env,
+      windowsHide: true,
       encoding: "utf8",
       maxBuffer: 32 * 1024 * 1024,
       shell: process.platform === "win32",
@@ -610,6 +611,7 @@ function runNode(args: string[], cwd: string): CommandResult {
     spawnSync(process.execPath, args, {
       cwd,
       env: process.env,
+      windowsHide: true,
       encoding: "utf8",
       maxBuffer: 32 * 1024 * 1024,
     }),

package/src/codex/CodexEventMapper.ts

@@ -73,7 +73,7 @@ export function mapCodexNotification(method: string, params: unknown): MappedCod
   if (!threadId || !turnId) return undefined;
 
   if (method === "error" && isRecord(params.error)) {
-    const message = stringValue(params.error.message)?.trim();
+    const message = formatCodexError(params.error);
     if (!message || typeof params.willRetry !== "boolean") return undefined;
     return { kind: "runtime_error", threadId, turnId, message, willRetry: params.willRetry };
   }
@@ -200,7 +200,7 @@ export function mapCodexNotification(method: string, params: unknown): MappedCod
   if (method === "turn/completed" && isRecord(params.turn)) {
     const status = params.turn.status;
     const mappedStatus = status === "interrupted" ? "cancelled" : status === "failed" ? "failed" : "completed";
-    const error = isRecord(params.turn.error) ? stringValue(params.turn.error.message) : undefined;
+    const error = formatCodexError(params.turn.error);
     return {
       kind: "terminal",
       threadId,
@@ -213,6 +213,15 @@ export function mapCodexNotification(method: string, params: unknown): MappedCod
   return undefined;
 }
 
+export function formatCodexError(error: unknown): string | undefined {
+  if (!isRecord(error)) return undefined;
+  const message = stringValue(error.message)?.trim();
+  const details = stringValue(error.additionalDetails)?.trim();
+  if (!message) return details || undefined;
+  if (!details || message.includes(details)) return message;
+  return details.includes(message) ? details : `${message}\n\n${details}`;
+}
+
 function mapTool(
   item: Record<string, unknown>,
   phase: "started" | "updated",

package/src/codex/CodexRuntime.ts

@@ -29,7 +29,7 @@ import type {
 import { appendGeneratedImageMarkdown } from "../utils/generatedImageMarkdown.js";
 import { normalizeTaskTitle } from "../utils/taskTitle.js";
 import { AppServerRequestError } from "./AppServerConnection.js";
-import { mapCodexNotification } from "./CodexEventMapper.js";
+import { formatCodexError, mapCodexNotification } from "./CodexEventMapper.js";
 import { CodexLocalActivityDetector } from "./CodexLocalActivityDetector.js";
 import { detectProjectlessWorkspace } from "./ProjectlessWorkspace.js";
 import { threadWriterLockPath } from "./ThreadWriterProcess.js";
@@ -1460,7 +1460,7 @@ export class CodexRuntime implements AgentRuntime {
         type: "turn_failed",
         sessionId: session.localSessionId,
         turnId: turn.id,
-        message: turn.error?.message ?? "App Server turn failed.",
+        message: formatCodexError(turn.error) ?? "App Server turn failed.",
       });
       return;
     }
@@ -1639,7 +1639,7 @@ interface CodexTurnSnapshot {
     status?: string;
     savedPath?: string;
   }>;
-  error?: { message?: string } | null;
+  error?: { message?: string; additionalDetails?: string | null } | null;
   startedAt?: number | null;
   durationMs?: number | null;
 }
@@ -1798,7 +1798,7 @@ function remoteSessionSummary(thread: CodexThreadSnapshot): RemoteSessionSummary
     finalResponse: lastTurn && lastTurn.status !== "inProgress"
       ? appendGeneratedImageMarkdown(extractFinalResponse(lastTurn), extractGeneratedImagePaths(lastTurn)) || undefined
       : undefined,
-    lastError: lastTurn?.error?.message,
+    lastError: formatCodexError(lastTurn?.error),
     lastTurnToolCount: toolCounts?.total,
     lastTurnCompletedToolCount: toolCounts?.completed,
     lastTurnFailedToolCount: toolCounts?.failed,

package/src/feishu/CardRenderer.ts

@@ -1847,6 +1847,10 @@ function renderActivity(
   if (activity.kind === "assistant" || (activity.kind === "reasoning" && activity.id.startsWith("commentary:"))) {
     const text = activity.text.trim();
     if (!text) return [];
+    if (activity.id.startsWith("commentary:runtime-error:")) {
+      const chunks = fullAssistantText ? splitText(text, ACTIVITY_TEXT_CHUNK) : [text];
+      return chunks.map((chunk) => markdown(codeBlock(chunk, fullAssistantText ? ACTIVITY_TEXT_CHUNK : MAX_LIVE_ASSISTANT_TEXT)));
+    }
     return fullAssistantText
       ? splitText(text, ACTIVITY_TEXT_CHUNK).map(markdown)
       : [markdown(truncateText(text, MAX_LIVE_ASSISTANT_TEXT))];

package/src/feishu/FeishuMarkdown.ts

@@ -54,11 +54,25 @@ function includeLocalFileLineReferences(
   projectCwd?: string,
   localFileUrl?: LocalFileUrlResolver,
 ): string {
-  return markdown.replace(MARKDOWN_LINK, (link, imageMarker: string, label: string, rawTarget: string) => {
-    if (imageMarker) return link;
+  const codeSpans = inlineCodeSpans(markdown);
+  return markdown.replace(MARKDOWN_LINK, (link, imageMarker: string, label: string, rawTarget: string, offset: number) => {
+    if (imageMarker || codeSpans.some(([start, end]) => offset >= start && offset < end)) return link;
 
     const target = unwrapMarkdownTarget(rawTarget.trim());
-    if (!isLocalFileTarget(target)) return link;
+    if (!isLocalFileTarget(target)) {
+      const relative = resolveRelativeFileTarget(target, projectCwd);
+      if (!relative || isLocalImageTarget(relative.filePath)) return link;
+      const viewerUrl = localFileUrl?.(relative.filePath, relative.reference);
+      if (!viewerUrl) {
+        const displayPath = displayAbsolutePath(relative.filePath) + (relative.reference ?? "");
+        return label ? `${label}(${inlineCode(displayPath)})` : inlineCode(displayPath);
+      }
+      const url = new URL(viewerUrl);
+      if (relative.hash) url.hash = relative.hash;
+      const normalizedLabel = relative.reference && !label.endsWith(relative.reference)
+        ? `${label}${relative.reference}` : label;
+      return `[${normalizedLabel}](${url.toString()})`;
+    }
     if (isLocalImageTarget(target)) return link;
 
     const reference = LOCAL_FILE_LINE_REFERENCE.exec(target)?.[1];
@@ -84,6 +98,58 @@ function includeLocalFileLineReferences(
   });
 }
 
+interface RelativeFileTarget {
+  filePath: string;
+  reference?: string;
+  hash?: string;
+}
+
+function resolveRelativeFileTarget(target: string, projectCwd?: string): RelativeFileTarget | undefined {
+  if (!projectCwd || !target || /^(?:#|\?|\/\/)/u.test(target)) return undefined;
+  const pathApi = usesWindowsPaths(projectCwd) ? path.win32 : path.posix;
+  if (!pathApi.isAbsolute(projectCwd)) return undefined;
+  const suffixStart = target.search(/[?#]/u);
+  const pathTarget = suffixStart < 0 ? target : target.slice(0, suffixStart);
+  const lineSuffix = LOCAL_FILE_LINE_REFERENCE.exec(pathTarget)?.[1];
+  // A basename with a line suffix (report.md:12) is a file, not a URI scheme.
+  const fileWithLine = /^[^:/\\]+\.[^:/\\]+:\d+(?::\d+)?$/u.test(pathTarget);
+  if (!fileWithLine && /^[a-z][a-z\d+.-]*:/iu.test(target)) return undefined;
+  const encodedPath = lineSuffix ? pathTarget.slice(0, -lineSuffix.length) : pathTarget;
+  try {
+    const decodedPath = decodeURIComponent(encodedPath);
+    if (!decodedPath || /[\x00-\x1f\x7f]/u.test(decodedPath)
+      || pathApi.isAbsolute(decodedPath) || /^[a-z]:/iu.test(decodedPath)) return undefined;
+    const filePath = pathApi.resolve(projectCwd, decodedPath);
+    const hashStart = target.indexOf("#");
+    const hash = hashStart < 0 ? undefined : new URL(target.slice(hashStart), "http://localhost/").hash;
+    const hashLine = /^#L(\d+)(?:C\d+)?(?:-L?\d+(?:C\d+)?)?$/u.exec(hash ?? "")?.[1];
+    const validHashLine = hashLine && Number.isSafeInteger(Number(hashLine)) && Number(hashLine) > 0;
+    return {
+      filePath,
+      reference: validHashLine ? `:${Number(hashLine)}` : hash ? undefined : lineSuffix,
+      hash: validHashLine ? `#L${Number(hashLine)}` : hash,
+    };
+  } catch {
+    return undefined;
+  }
+}
+
+function inlineCodeSpans(markdown: string): Array<[number, number]> {
+  const runs = [...markdown.matchAll(/`+/gu)];
+  const spans: Array<[number, number]> = [];
+  for (let index = 0; index < runs.length; index += 1) {
+    const opener = runs[index]!;
+    const backslashes = /\\*$/u.exec(markdown.slice(0, opener.index))?.[0].length ?? 0;
+    if (backslashes % 2 !== 0) continue;
+    const closing = runs.findIndex((run, candidate) => candidate > index && run[0].length === opener[0].length);
+    if (closing < 0) continue;
+    const closer = runs[closing]!;
+    spans.push([opener.index, closer.index + closer[0].length]);
+    index = closing;
+  }
+  return spans;
+}
+
 function projectFilePathLabel(
   target: string,
   projectCwd: string,

package/src/local-files/TurnPreviewPage.ts

@@ -362,6 +362,9 @@ function renderActivity(
 ): string {
   switch (activity.kind) {
     case "assistant":
+      if (activity.id.startsWith("commentary:runtime-error:")) {
+        return `<article class="activity message runtime-error" data-activity="${escapeAttribute(activity.id)}"><pre>${escapeHtml(activity.text)}</pre></article>`;
+      }
       return renderMessageActivity(activity.id, activity.text, "commentary", renderText);
     case "user":
       return renderMessageActivity(activity.id, activity.text, "user", renderText);
@@ -813,6 +816,7 @@ summary:focus-visible { outline: 2px solid var(--accent); outline-offset: -2px;
 .tool-meta span { white-space: nowrap; }
 .tool-header-timing { min-width: 0; overflow: hidden; text-overflow: ellipsis; }
 .tool-header-timing:empty, .tool-footer-timing:empty { display: none; }
+.runtime-error pre { white-space: pre-wrap; overflow-wrap: anywhere; border-left: 2px solid var(--danger); }
 .error-output, .error-result h2 { color: var(--danger); }
 .muted, .empty { color: var(--muted); }
 .notice, .empty { padding: 10px 0; }

package/src/presentation/TurnStateReducer.ts

@@ -179,7 +179,7 @@ export function reduceTurnEvent(state: TurnViewState, event: AgentEvent): TurnVi
       const activityUpdate = upsertReasoningActivity(
         warning ? activities.filter((activity) => activity.id !== (event.activityId ?? "progress")) : activities,
         event.activityId ?? "progress",
-        warning ? bound(event.text) : event.text,
+        warning && !event.activityId?.startsWith("commentary:runtime-error:") ? bound(event.text) : event.text,
         !warning && event.append === true,
         event.activityId?.startsWith("commentary:") ? "assistant" : "reasoning",
       );
@@ -252,7 +252,7 @@ export function reduceTurnEvent(state: TurnViewState, event: AgentEvent): TurnVi
         approval: undefined,
         completedAt,
         durationMs: Math.max(0, completedAt - state.startedAt),
-        error: bound(event.message),
+        error: event.message,
       };
     }
   }

package/src/proxy/ProxySessionController.ts

@@ -2158,7 +2158,7 @@ export class ProxySessionController {
         await this.openExecutionSettings(contextKey, "model");
         return;
       case "provider":
-        await this.openProviderSettings(contextKey);
+        await this.openExecutionSettings(contextKey, "provider");
         return;
       case "thinking":
         await this.openExecutionSettings(contextKey, "thinking");
@@ -5179,21 +5179,6 @@ export class ProxySessionController {
     await this.openExecutionSettings(contextKey, "agent");
   }
 
-  private async openProviderSettings(contextKey: string): Promise<void> {
-    const loaded = await this.loadSession(this.requireCurrentSession(contextKey));
-    const providers = loaded.runtime.kind === "codex" && loaded.runtime.listModelProviders
-      ? await this.modelProviderOptions(loaded)
-      : [];
-    const current = loaded.session.modelProvider?.trim()
-      || providers.find((provider) => provider.isDefault)?.id
-      || "运行时默认";
-    if (providers.length <= 1) {
-      await this.outbound.sendText(contextKey, `当前 Provider：${current}\n当前没有其他 Provider 可以切换。`);
-      return;
-    }
-    await this.openExecutionSettings(contextKey, "provider", { sessionId: loaded.record.localSessionId });
-  }
-
   private async selectProvider(
     contextKey: string,
     modelProvider: string,