@xmoxmo/bncr 0.5.6 → 0.5.8

package/src/plugin/file-transfer-orchestrator.ts

@@ -193,7 +193,38 @@ export function createBncrFileTransferOrchestrator(runtime: BncrFileTransferOrch
   };
   const HTTP_URL_RE = /^https?:\/\//i;
 
-  /** Try HEAD for Content-Type; fall back to Range+sniff for magic bytes when unclear. */
+  const EXT_TO_MIME: Record<string, string> = {
+    jpg: 'image/jpeg',
+    jpeg: 'image/jpeg',
+    png: 'image/png',
+    gif: 'image/gif',
+    webp: 'image/webp',
+    bmp: 'image/bmp',
+    heic: 'image/heic',
+    heif: 'image/heif',
+    mp4: 'video/mp4',
+    mov: 'video/quicktime',
+    mkv: 'video/x-matroska',
+    webm: 'video/webm',
+    avi: 'video/x-msvideo',
+    m4v: 'video/x-m4v',
+    mp3: 'audio/mpeg',
+    wav: 'audio/wav',
+    ogg: 'audio/ogg',
+    oga: 'audio/ogg',
+    opus: 'audio/opus',
+    m4a: 'audio/mp4',
+    aac: 'audio/aac',
+    flac: 'audio/flac',
+  };
+
+  function mimeFromUrlExt(url: string): string | undefined {
+    const clean = url.split(/[?#]/, 1)[0] || '';
+    const ext = clean.includes('.') ? clean.slice(clean.lastIndexOf('.') + 1).toLowerCase() : '';
+    return EXT_TO_MIME[ext];
+  }
+
+  /** Try HEAD for Content-Type; fall back to Range+sniff for magic bytes; then URL extension. */
   async function resolveRemoteMediaType(url: string): Promise<{ mimeType?: string }> {
     // 1) HEAD -> Content-Type
     let headerType = '';
@@ -229,8 +260,14 @@ export function createBncrFileTransferOrchestrator(runtime: BncrFileTransferOrch
       if (!sniffed) return {};
       return { mimeType: sniffed };
     } catch {
-      return {};
+      /* Range sniff failed, fall through to extension inference */
     }
+
+    // 3) URL extension -> mime type (fallback when HEAD and Range both fail)
+    const extMime = mimeFromUrlExt(url);
+    if (extMime) return { mimeType: extMime };
+
+    return {};
   }
 
   /** Minimal magic-byte MIME sniffer for the first ~512 bytes. */

package/src/plugin/media-dedupe-runtime.ts

@@ -1,3 +1,4 @@
+import { asString } from '../core/value-sanitize.ts';
 import {
   buildMediaTextFallback,
   type MediaDedupeCacheEntry,
@@ -5,12 +6,6 @@ import {
   normalizeReplyToId,
 } from '../messaging/outbound/media-dedupe.ts';
 
-function asString(v: unknown, fallback = ''): string {
-  if (typeof v === 'string') return v;
-  if (v == null) return fallback;
-  return String(v);
-}
-
 export function createBncrMediaDedupeRuntime(runtime: {
   now: () => number;
   recentMediaDedupeBySession: Map<string, Map<string, MediaDedupeCacheEntry>>;

package/src/plugin/media-orchestrators-runtime-group.ts

@@ -1,21 +1,13 @@
 import type { BncrRoute, FileSendTransferState, OutboxEntry } from '../core/types.ts';
-import { normalizeMessageText } from '../messaging/outbound/media-dedupe.ts';
 import type {
   NormalizedReplyPayload,
   OutboundReplyTargetPolicy,
-  ReplyMediaEntriesParams,
   ReplyPayloadInput,
 } from '../messaging/outbound/reply-enqueue.ts';
 import {
   enqueueNormalizedReplyPayload,
-  enqueueReplyTextEntry,
   normalizeReplyPayload,
 } from '../messaging/outbound/reply-enqueue.ts';
-import {
-  enqueueReplyMediaFallbackTextEntry,
-  enqueueReplyMediaFileTransferEntry,
-  enqueueSingleReplyMediaEntry,
-} from '../messaging/outbound/reply-enqueue-media.ts';
 import type { createBncrFileAckRuntime } from './file-ack-runtime.ts';
 import { createBncrFileTransferOrchestrator } from './file-transfer-orchestrator.ts';
 import type { createBncrFileTransferSetup } from './file-transfer-setup.ts';
@@ -27,92 +19,12 @@ type FileAckPayload = Awaited<
   ReturnType<ReturnType<typeof createBncrFileAckRuntime>['waitForFileAck']>
 >;
 
-function buildReplyMediaEntryHelpers(runtime: {
-  logInfo: (scope: string | undefined, message: string, options?: { debugOnly?: boolean }) => void;
-  enqueueOutbound: (entry: OutboxEntry) => void;
-  buildTextOutboxEntry: (args: {
-    accountId: string;
-    sessionKey: string;
-    route: BncrRoute;
-    text: string;
-    kind?: 'tool' | 'block' | 'final';
-    replyToId?: string;
-    replyTargetPolicy?: OutboundReplyTargetPolicy;
-  }) => OutboxEntry;
-  buildFileTransferOutboxEntry: (args: {
-    accountId: string;
-    sessionKey: string;
-    route: BncrRoute;
-    mediaUrl: string;
-    mediaLocalRoots?: readonly string[];
-    text: string;
-    asVoice: boolean;
-    audioAsVoice: boolean;
-    type?: string;
-    extra?: Record<string, unknown>;
-    kind?: 'tool' | 'block' | 'final';
-    replyToId?: string;
-    replyTargetPolicy?: OutboundReplyTargetPolicy;
-    downloadMedia?: boolean;
-  }) => OutboxEntry;
-  rememberRecentMediaSend: (args: {
-    sessionKey: string;
-    mediaUrl: string;
-    text: string;
-    replyToId: string;
-    createdAt: number;
-  }) => void;
-}) {
-  return {
-    enqueueReplyMediaFallbackTextEntry: (
-      params: Parameters<typeof enqueueReplyMediaFallbackTextEntry>[0],
-    ) =>
-      enqueueReplyMediaFallbackTextEntry(params, {
-        logInfo: runtime.logInfo,
-        enqueueOutbound: runtime.enqueueOutbound,
-        buildTextOutboxEntry: runtime.buildTextOutboxEntry,
-      }),
-    enqueueReplyMediaFileTransferEntry: (
-      params: Parameters<typeof enqueueReplyMediaFileTransferEntry>[0],
-    ) =>
-      enqueueReplyMediaFileTransferEntry(params, {
-        enqueueOutbound: runtime.enqueueOutbound,
-        buildFileTransferOutboxEntry: runtime.buildFileTransferOutboxEntry,
-        rememberRecentMediaSend: runtime.rememberRecentMediaSend,
-      }),
-  };
-}
-
-function buildReplyTextEntryHelper(runtime: {
-  enqueueOutbound: (entry: OutboxEntry) => void;
-  buildTextOutboxEntry: (args: {
-    accountId: string;
-    sessionKey: string;
-    route: BncrRoute;
-    text: string;
-    kind?: 'tool' | 'block' | 'final';
-    replyToId?: string;
-    replyTargetPolicy?: OutboundReplyTargetPolicy;
-  }) => OutboxEntry;
-}) {
-  return (params: {
-    accountId: string;
-    sessionKey: string;
-    route: BncrRoute;
-    payload: NormalizedReplyPayload;
-  }) =>
-    enqueueReplyTextEntry(params, {
-      enqueueOutbound: runtime.enqueueOutbound,
-      buildTextOutboxEntry: runtime.buildTextOutboxEntry,
-    });
-}
-
-function normalizeReplyOrchestratorPayload(args: {
+async function normalizeReplyOrchestratorPayload(args: {
   payload: ReplyPayloadInput;
   asString: (value: unknown, fallback?: string) => string;
   replyTargetPolicy?: OutboundReplyTargetPolicy;
 }) {
-  return normalizeReplyPayload(
+  return await normalizeReplyPayload(
     args.payload,
     { asString: args.asString },
     {
@@ -186,29 +98,21 @@ export function createBncrMediaOrchestratorsRuntimeGroup(runtime: {
     payload: NormalizedReplyPayload;
   }) => void;
   enqueueOutbound: (entry: OutboxEntry) => void;
-  buildTextOutboxEntry: (args: {
+  buildOutboxEntry: (args: {
     accountId: string;
     sessionKey: string;
     route: BncrRoute;
-    text: string;
-    kind?: 'tool' | 'block' | 'final';
-    replyToId?: string;
-    replyTargetPolicy?: OutboundReplyTargetPolicy;
-  }) => OutboxEntry;
-  buildFileTransferOutboxEntry: (args: {
-    accountId: string;
-    sessionKey: string;
-    route: BncrRoute;
-    mediaUrl: string;
-    mediaLocalRoots?: readonly string[];
-    text: string;
-    asVoice: boolean;
-    audioAsVoice: boolean;
     type?: string;
-    extra?: Record<string, unknown>;
+    msg: string;
     kind?: 'tool' | 'block' | 'final';
     replyToId?: string;
     replyTargetPolicy?: OutboundReplyTargetPolicy;
+    extra?: Record<string, unknown>;
+    transferMode?: 'text' | 'media';
+    mediaUrl?: string;
+    mediaLocalRoots?: readonly string[];
+    asVoice?: boolean;
+    audioAsVoice?: boolean;
     downloadMedia?: boolean;
   }) => OutboxEntry;
   rememberRecentMediaSend: (args: {
@@ -244,40 +148,7 @@ export function createBncrMediaOrchestratorsRuntimeGroup(runtime: {
     logFileTransferCompleteAck: runtime.logFileTransferCompleteAck,
   });
 
-  const replyMediaEntryHelpers = buildReplyMediaEntryHelpers(runtime);
-  const replyTextEntryHelper = buildReplyTextEntryHelper(runtime);
-
-  const enqueueReplyMediaEntries = (params: ReplyMediaEntriesParams) => {
-    let first = true;
-    const currentTime = runtime.now();
-
-    for (const mediaUrl of params.payload.mediaList) {
-      const normalizedText = normalizeMessageText(first ? params.payload.text : '');
-      const fallback = runtime.tryBuildMediaDedupeFallback({
-        sessionKey: params.sessionKey,
-        mediaUrl,
-        text: normalizedText,
-        replyToId: params.payload.replyToId,
-        currentTime,
-      });
-
-      enqueueSingleReplyMediaEntry(
-        {
-          params,
-          mediaUrl,
-          normalizedText,
-          text: first ? params.payload.text : '',
-          fallback,
-          currentTime,
-        },
-        replyMediaEntryHelpers,
-      );
-
-      first = false;
-    }
-  };
-
-  const enqueueFromReply = (params: {
+  const enqueueFromReply = async (params: {
     accountId: string;
     sessionKey: string;
     route: BncrRoute;
@@ -286,7 +157,7 @@ export function createBncrMediaOrchestratorsRuntimeGroup(runtime: {
     replyTargetPolicy?: OutboundReplyTargetPolicy;
   }) => {
     const { accountId, sessionKey, route, payload, mediaLocalRoots, replyTargetPolicy } = params;
-    const normalized = normalizeReplyOrchestratorPayload({
+    const normalized = await normalizeReplyOrchestratorPayload({
       payload,
       asString: runtime.asString,
       replyTargetPolicy,
@@ -302,19 +173,19 @@ export function createBncrMediaOrchestratorsRuntimeGroup(runtime: {
       },
       {
         logEnqueueFromReply: runtime.logEnqueueFromReply,
-        enqueueReplyMediaEntries,
-        enqueueReplyTextEntry: replyTextEntryHelper,
+        enqueueOutbound: runtime.enqueueOutbound,
+        buildOutboxEntry: runtime.buildOutboxEntry,
+        tryBuildMediaDedupeFallback: runtime.tryBuildMediaDedupeFallback,
+        rememberRecentMediaSend: runtime.rememberRecentMediaSend,
+        logInfo: runtime.logInfo,
       },
     );
   };
 
-  const replyMediaOrchestrator = {
-    enqueueFromReply,
-    enqueueReplyMediaEntries,
-  };
-
   return {
     fileTransferOrchestrator,
-    replyMediaOrchestrator,
+    replyMediaOrchestrator: {
+      enqueueFromReply,
+    },
   };
 }

package/src/plugin/message-send.ts

@@ -1,23 +1,22 @@
 import type { ChannelMessageSendResult } from 'openclaw/plugin-sdk/channel-message';
-
-type MessageSendContext = Record<string, unknown>;
+import type { BncrChannelSendContext } from './channel-runtime-types.ts';
 
 export type BncrMessageSendBridge = {
   channelMessageSendText: (
-    ctx: MessageSendContext,
+    ctx: BncrChannelSendContext,
   ) => ChannelMessageSendResult | Promise<ChannelMessageSendResult>;
   channelMessageSendMedia: (
-    ctx: MessageSendContext,
+    ctx: BncrChannelSendContext,
   ) => ChannelMessageSendResult | Promise<ChannelMessageSendResult>;
   channelMessageSendPayload: (
-    ctx: MessageSendContext,
+    ctx: BncrChannelSendContext,
   ) => ChannelMessageSendResult | Promise<ChannelMessageSendResult>;
 };
 
 export function createBncrMessageSend(getBridge: () => BncrMessageSendBridge) {
   return {
-    text: async (ctx: MessageSendContext) => getBridge().channelMessageSendText(ctx),
-    media: async (ctx: MessageSendContext) => getBridge().channelMessageSendMedia(ctx),
-    payload: async (ctx: MessageSendContext) => getBridge().channelMessageSendPayload(ctx),
+    text: async (ctx: BncrChannelSendContext) => getBridge().channelMessageSendText(ctx),
+    media: async (ctx: BncrChannelSendContext) => getBridge().channelMessageSendMedia(ctx),
+    payload: async (ctx: BncrChannelSendContext) => getBridge().channelMessageSendPayload(ctx),
   };
 }

package/src/plugin/messaging.ts

@@ -67,12 +67,12 @@ function isDisplayTarget(
   return Boolean(value && typeof value === 'object');
 }
 
-export function normalizeBncrMessagingTarget(raw: string) {
+function normalizeBncrMessagingTarget(raw: string) {
   const input = asSanitizedString(raw).trim();
   return input || undefined;
 }
 
-export function formatBncrMessagingTargetDisplay({ target }: BncrMessagingTargetDisplayInput) {
+function formatBncrMessagingTargetDisplay({ target }: BncrMessagingTargetDisplayInput) {
   if (typeof target === 'string') {
     const parsed = parseExplicitTarget(asSanitizedString(target).trim());
     return parsed?.route ? formatBncrHumanDisplay(parsed.route) : asSanitizedString(target).trim();
@@ -183,7 +183,7 @@ export function createBncrMessagingSurface(getBridge: () => BncrMessagingRuntime
   };
 }
 
-export function createBncrMessagingTargetResolver(getBridge: () => BncrMessagingRuntimeBridge) {
+function createBncrMessagingTargetResolver(getBridge: () => BncrMessagingRuntimeBridge) {
   return {
     looksLikeId: (raw: string, normalized?: string) => {
       return looksLikeBncrExplicitTarget(asSanitizedString(normalized || raw).trim());

package/src/plugin/outbound.ts

@@ -1,35 +1,15 @@
 import { normalizeAccountId } from '../core/accounts.ts';
+import { asString } from '../core/value-sanitize.ts';
 import {
   deleteBncrMessageAction,
   editBncrMessageAction,
   reactBncrMessageAction,
   sendBncrReplyAction,
 } from '../messaging/outbound/actions.ts';
-
-function asString(v: unknown, fallback = ''): string {
-  return typeof v === 'string' ? v : v == null ? fallback : String(v);
-}
+import type { BncrChannelSendContext } from './channel-runtime-types.ts';
 
 type BncrOutboundDeliveryResult = { channel: string; messageId: string; chatId: string };
 
-type BncrOutboundSendContext = {
-  accountId?: string | null;
-  to?: string;
-  text?: string;
-  mediaUrl?: string;
-  type?: string;
-  kind?: string;
-  replyToId?: string | null;
-  replyToMessageId?: string | null;
-  asVoice?: boolean;
-  audioAsVoice?: boolean;
-  mediaLocalRoots?: readonly string[];
-  /** Pass-through fields from host — bncr plugin does NOT consume these. */
-  forceDocument?: boolean;
-  gifPlayback?: boolean;
-  silent?: boolean;
-};
-
 type BncrOutboundReplyActionContext = {
   accountId?: string | null;
   to?: string;
@@ -54,18 +34,18 @@ type BncrOutboundEditActionContext = BncrOutboundTargetActionContext & {
 
 export type BncrOutboundBridge = {
   channelSendText: (
-    ctx: BncrOutboundSendContext,
+    ctx: BncrChannelSendContext,
   ) => BncrOutboundDeliveryResult | Promise<BncrOutboundDeliveryResult>;
   channelSendMedia: (
-    ctx: BncrOutboundSendContext,
+    ctx: BncrChannelSendContext,
   ) => BncrOutboundDeliveryResult | Promise<BncrOutboundDeliveryResult>;
 };
 
 export function createBncrOutboundRuntime(getBridge: () => BncrOutboundBridge) {
   return {
     deliveryMode: 'gateway' as const,
-    sendText: async (ctx: BncrOutboundSendContext) => getBridge().channelSendText(ctx),
-    sendMedia: async (ctx: BncrOutboundSendContext) => getBridge().channelSendMedia(ctx),
+    sendText: async (ctx: BncrChannelSendContext) => getBridge().channelSendText(ctx),
+    sendMedia: async (ctx: BncrChannelSendContext) => getBridge().channelSendMedia(ctx),
     replyAction: async (ctx: BncrOutboundReplyActionContext) =>
       sendBncrReplyAction({
         accountId: normalizeAccountId(ctx?.accountId),

package/src/plugin/outbox-drain-loop.ts

@@ -295,8 +295,8 @@ export function createBncrOutboxDrainLoop(
     try {
       pushed = await runtime.tryPushEntry(entry);
     } catch (error) {
-      const meta = runtime.isPlainObject(entry.payload?._meta) ? entry.payload._meta : null;
-      if (meta?.kind === 'file-transfer') {
+      const msg = runtime.isPlainObject(entry.payload?.message) ? entry.payload.message : null;
+      if (msg?.transferMode === 'media') {
         runtime.handleFileTransferPushFailure({ entry, error });
       } else {
         runtime.handleTextPushFailure({ entry, error });

package/src/plugin/outbox-push.ts

@@ -3,12 +3,7 @@ import {
   resolveFileTransferFailureState,
 } from '../core/outbox-file-transfer-failure.ts';
 import { buildTextPushFailureArgs } from '../core/outbox-text-push-failure.ts';
-import {
-  buildTextPushBroadcastPayload,
-  buildTextPushOkArgs,
-  buildTextPushRouteSelectArgs,
-  buildTextPushSuccessArgs,
-} from '../core/outbox-text-push-success.ts';
+
 import type { BncrConnection, OutboxEntry } from '../core/types.ts';
 import {
   buildOutboxPushOkDebugInfo,
@@ -137,51 +132,6 @@ export function createBncrOutboxPush(runtime: BncrOutboxPushRuntime) {
     );
   };
 
-  const pushTextSuccessPath = (args: {
-    entry: OutboxEntry;
-    owner: BncrConnection | null;
-    connIds: Iterable<string>;
-    recentInboundReachable: boolean;
-    routeReason: string;
-    ownerConnId?: string;
-  }) => {
-    runtime.gatewayBroadcastToConnIds(
-      runtime.pushEvent,
-      buildTextPushBroadcastPayload({
-        payload: args.entry.payload,
-        messageId: args.entry.messageId,
-      }),
-      new Set(args.connIds),
-    );
-    logOutboxRouteSelect(
-      buildTextPushRouteSelectArgs({
-        entry: args.entry,
-        connIds: args.connIds,
-        routeReason: args.routeReason,
-        recentInboundReachable: args.recentInboundReachable,
-        owner: args.owner,
-        event: runtime.pushEvent,
-      }),
-    );
-    runtime.recordOutboxPushSuccess(
-      buildTextPushSuccessArgs({
-        entry: args.entry,
-        connIds: args.connIds,
-        ownerConnId: args.ownerConnId,
-        ownerClientId: args.ownerConnId ? args.owner?.clientId : undefined,
-      }),
-    );
-    logOutboxPushOkSummary(args.entry.messageId);
-    logOutboxPushOk(
-      buildTextPushOkArgs({
-        entry: args.entry,
-        connIds: args.connIds,
-        recentInboundReachable: args.recentInboundReachable,
-        event: runtime.pushEvent,
-      }),
-    );
-  };
-
   const handleTextPushFailure = (args: { entry: OutboxEntry; error: unknown }) => {
     runtime.recordOutboxPushFailure({
       entry: args.entry,
@@ -253,7 +203,6 @@ export function createBncrOutboxPush(runtime: BncrOutboxPushRuntime) {
   };
 
   return {
-    pushTextSuccessPath,
     handleTextPushFailure,
     handleFileTransferPushFailure,
     handleFileTransferPushGuardFailure,