@xmoxmo/bncr 0.5.6 → 0.5.7

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/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/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-file-push-flow.ts

@@ -1,11 +1,11 @@
 import { resolveFileTransferGuard } from '../core/outbox-file-transfer-guards.ts';
-import { prepareFileTransferRouteSelection } from '../core/outbox-file-transfer-prep.ts';
+import { prepareRouteSelection } from '../core/outbox-file-transfer-prep.ts';
 import type { BncrConnection, OutboxEntry } from '../core/types.ts';
 import { selectOutboxFileTransferRouteCandidates } from '../messaging/outbound/queue-selectors.ts';
 
 export async function runBncrFileTransferOutboxPush(args: {
   entry: OutboxEntry;
-  meta: Record<string, unknown>;
+  msg: Record<string, unknown>;
   gatewayContext: unknown;
   owner: BncrConnection | null;
   resolvePushConnIds: (accountId: string) => Set<string>;
@@ -18,7 +18,7 @@ export async function runBncrFileTransferOutboxPush(args: {
   }) => void;
   pushFileTransferSuccessPath: (args: {
     entry: OutboxEntry;
-    meta: Record<string, unknown>;
+    msg: Record<string, unknown>;
     owner: BncrConnection | null;
     connIds: Iterable<string>;
     recentInboundReachable: boolean;
@@ -27,21 +27,21 @@ export async function runBncrFileTransferOutboxPush(args: {
   }) => Promise<void>;
   handleFileTransferPushFailure: (args: { entry: OutboxEntry; error: unknown }) => void;
 }) {
-  const selection = prepareFileTransferRouteSelection({
+  const selection = prepareRouteSelection({
     entry: args.entry,
     owner: args.owner,
     resolvePushConnIds: args.resolvePushConnIds,
     resolveRecentInboundConnIds: args.resolveRecentInboundConnIds,
     hasRecentInboundReachability: args.hasRecentInboundReachability,
     isRevalidatedAttemptedConn: args.isRevalidatedAttemptedConn,
-    selectOutboxFileTransferRouteCandidates,
+    selectRouteCandidates: selectOutboxFileTransferRouteCandidates,
   });
   const guard = resolveFileTransferGuard({
     gatewayContext: args.gatewayContext,
     entry: args.entry,
     owner: args.owner,
     routeSelection: selection,
-    mediaUrl: String(args.meta.mediaUrl || '').trim(),
+    mediaUrl: String(args.msg.mediaUrl || '').trim(),
   });
   if (!guard.ok) {
     args.handleFileTransferPushGuardFailure({
@@ -54,7 +54,7 @@ export async function runBncrFileTransferOutboxPush(args: {
   try {
     await args.pushFileTransferSuccessPath({
       entry: args.entry,
-      meta: args.meta,
+      msg: args.msg,
       owner: args.owner,
       connIds: guard.connIds,
       recentInboundReachable: guard.recentInboundReachable,

package/src/plugin/outbox-text-push-flow.ts

@@ -43,7 +43,7 @@ export async function runBncrTextOutboxPush(args: {
     resolveRecentInboundConnIds: args.resolveRecentInboundConnIds,
     hasRecentInboundReachability: args.hasRecentInboundReachability,
     isRevalidatedAttemptedConn: args.isRevalidatedAttemptedConn,
-    selectOutboxRouteCandidates,
+    selectRouteCandidates: selectOutboxRouteCandidates,
   });
   const guard = resolveTextPushGuard({
     gatewayContext: args.gatewayContext,

package/src/plugin/target-runtime.ts

@@ -6,15 +6,10 @@ import {
   routeKey,
 } from '../core/targets.ts';
 import type { BncrRoute } from '../core/types.ts';
+import { asString } from '../core/value-sanitize.ts';
 import { getOpenClawRuntimeConfigOrDefault } from '../openclaw/config-runtime.ts';
 import type { BncrChannelConfigRoot } from './channel-runtime-types.ts';
 
-function asString(v: unknown, fallback = ''): string {
-  if (typeof v === 'string') return v;
-  if (v == null) return fallback;
-  return String(v);
-}
-
 export function createBncrTargetRuntime(runtime: {
   api: Parameters<typeof getOpenClawRuntimeConfigOrDefault>[0];
   channelId: string;

package/src/runtime/outbound-ack-timeout.ts

@@ -1,4 +1,5 @@
 import type { BncrAckObservability, BncrAckStrategy } from '../core/types.ts';
+import { asString, finiteNumberOr } from '../core/value-sanitize.ts';
 
 type ComputeBncrRecommendedAckTimeoutArgs = {
   lateAckOkCount: number;
@@ -76,14 +77,6 @@ export function resolveBncrRuntimeAckTimeoutDecision(args: ComputeBncrRecommende
   return { timeoutMs, reason };
 }
 
-function finiteNumberOr(value: unknown, fallback: number) {
-  return typeof value === 'number' && Number.isFinite(value) ? value : fallback;
-}
-
-function asString(value: unknown, fallback = '') {
-  return typeof value === 'string' ? value : fallback;
-}
-
 export function buildBncrRuntimeAckStrategy(args: {
   ackObservability: BncrAckObservability;
   defaultAckTimeoutMs: number;

package/src/runtime/outbox-transitions.ts

@@ -1,4 +1,5 @@
 import type { OutboxEntry } from '../core/types.ts';
+import { finiteNumberOr } from '../core/value-sanitize.ts';
 import type {
   PushFailureDecision,
   RetryRerouteDecision,
@@ -83,10 +84,6 @@ export function buildBncrOutboxPushSuccessEntryPatch(args: {
   };
 }
 
-function finiteNumberOr(value: unknown, fallback: number): number {
-  return typeof value === 'number' && Number.isFinite(value) ? value : fallback;
-}
-
 export function buildBncrAckOkTelemetryPatch(args: {
   entry: OutboxEntry;
   ackAt: number;

package/src/runtime/status-worker.ts

@@ -1,5 +1,6 @@
 import { normalizeAccountId } from '../core/accounts.ts';
 import type { BncrDiagnosticsSummary } from '../core/types.ts';
+import { finiteNumberOr, nonNegativeFiniteNumberOr } from '../core/value-sanitize.ts';
 
 type StatusRuntimeMeta = {
   pending?: number;
@@ -60,15 +61,6 @@ export function createHealthStatusLogState(): HealthStatusLogState {
   return { emittedSig: null, pendingSig: null, pendingSince: 0 };
 }
 
-function finiteNumberOr(value: unknown, fallback: number): number {
-  const n = Number(value);
-  return Number.isFinite(n) ? n : fallback;
-}
-
-function nonNegativeFiniteNumberOr(value: unknown, fallback: number): number {
-  return Math.max(0, finiteNumberOr(value, fallback));
-}
-
 export function updateHealthStatusLogState(args: {
   state: HealthStatusLogState;
   sig: string;

package/src/messaging/outbound/build-send-action.ts

@@ -1,130 +0,0 @@
-import { randomUUID } from 'node:crypto';
-
-type MinimalBncrSendInput = {
-  channel?: string;
-  action?: string;
-  idempotencyKey?: string;
-  accountId?: string;
-  to?: string;
-  message?: string;
-  caption?: string;
-  path?: string;
-  media?: string;
-  filePath?: string;
-  mediaUrl?: string;
-  mediaUrls?: string[];
-  asVoice?: boolean;
-  audioAsVoice?: boolean;
-  params?: Record<string, unknown>;
-};
-
-type BuiltBncrMessageAction = {
-  channel: string;
-  action: string;
-  idempotencyKey: string;
-  accountId?: string;
-  params: Record<string, unknown>;
-};
-
-function asString(v: unknown, fallback = ''): string {
-  if (typeof v === 'string') return v;
-  if (v == null) return fallback;
-  return String(v);
-}
-
-function isPlainObject(value: unknown): value is Record<string, unknown> {
-  return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
-
-function pickFirstString(...values: unknown[]): string | undefined {
-  for (const value of values) {
-    if (typeof value !== 'string') continue;
-    if (value.length === 0) return '';
-    return value;
-  }
-  return undefined;
-}
-
-function pickFirstBoolean(...values: unknown[]): boolean | undefined {
-  for (const value of values) {
-    if (typeof value === 'boolean') return value;
-  }
-  return undefined;
-}
-
-export function buildBncrMessageAction(input: MinimalBncrSendInput): BuiltBncrMessageAction {
-  const paramsObj = isPlainObject(input.params) ? input.params : {};
-
-  const channel = asString(input.channel || 'bncr').trim() || 'bncr';
-  const action = asString(input.action || 'send').trim() || 'send';
-  const idempotencyKey = asString(input.idempotencyKey || '').trim() || `bncr-${randomUUID()}`;
-  const accountId =
-    asString(pickFirstString(paramsObj.accountId, input.accountId) || '').trim() || undefined;
-
-  const to = asString(pickFirstString(paramsObj.to, input.to) || '').trim();
-  if (!to) throw new Error('bncr send requires to');
-
-  const mediaPath = pickFirstString(
-    paramsObj.media,
-    paramsObj.path,
-    paramsObj.filePath,
-    paramsObj.mediaUrl,
-    input.media,
-    input.path,
-    input.filePath,
-    input.mediaUrl,
-  );
-  const rawMediaUrls = Array.isArray(paramsObj.mediaUrls)
-    ? paramsObj.mediaUrls
-    : Array.isArray(input.mediaUrls)
-      ? input.mediaUrls
-      : undefined;
-  const mediaUrls = rawMediaUrls?.map((value) => asString(value || '').trim()).filter(Boolean);
-
-  const message = pickFirstString(paramsObj.message, input.message) ?? '';
-  const explicitCaption = pickFirstString(paramsObj.caption, input.caption) ?? '';
-  const asVoice = pickFirstBoolean(paramsObj.asVoice, input.asVoice);
-  const audioAsVoice = pickFirstBoolean(paramsObj.audioAsVoice, input.audioAsVoice);
-
-  if ((asVoice === true || audioAsVoice === true) && !mediaPath && !mediaUrls?.length) {
-    throw new Error('bncr voice send requires media path');
-  }
-
-  const normalizedParams: Record<string, unknown> = {
-    ...paramsObj,
-    to,
-  };
-
-  if (mediaPath) {
-    normalizedParams.path = mediaPath;
-    const finalCaption = explicitCaption || message;
-    if (finalCaption) normalizedParams.caption = finalCaption;
-    delete normalizedParams.message;
-    if (mediaUrls?.length) normalizedParams.mediaUrls = mediaUrls;
-  } else {
-    if (mediaUrls?.length) {
-      normalizedParams.mediaUrls = mediaUrls;
-      const finalCaption = explicitCaption || message;
-      if (finalCaption) normalizedParams.caption = finalCaption;
-      delete normalizedParams.message;
-      delete normalizedParams.path;
-    } else {
-      const finalMessage = message || explicitCaption;
-      if (!finalMessage.trim()) throw new Error('bncr send requires message or media');
-      normalizedParams.message = finalMessage;
-      delete normalizedParams.caption;
-    }
-  }
-
-  if (asVoice === true) normalizedParams.asVoice = true;
-  if (audioAsVoice === true) normalizedParams.audioAsVoice = true;
-  if (accountId) normalizedParams.accountId = accountId;
-
-  return {
-    channel,
-    action,
-    idempotencyKey,
-    ...(accountId ? { accountId } : {}),
-    params: normalizedParams,
-  };
-}

package/src/messaging/outbound/durable-message-adapter.ts

@@ -1,121 +0,0 @@
-import type {
-  ChannelMessageAdapterShape,
-  ChannelMessageSendMediaContext,
-  ChannelMessageSendPayloadContext,
-  ChannelMessageSendResult,
-  ChannelMessageSendTextContext,
-} from 'openclaw/plugin-sdk/channel-message';
-import { defineChannelMessageAdapter } from 'openclaw/plugin-sdk/channel-message';
-import type { OpenClawConfig } from 'openclaw/plugin-sdk/core';
-
-import {
-  buildFileTransferOutboxEntry,
-  buildTextOutboxEntry,
-} from '../../core/outbox-entry-builders.ts';
-import type { BncrRoute, OutboxEntry } from '../../core/types.ts';
-import { buildBncrDurableQueuedResult } from './durable-queue-adapter.ts';
-
-export type BncrDurableMessageQueuedAdapterDeps = {
-  enqueueText: (
-    ctx: ChannelMessageSendTextContext<OpenClawConfig>,
-  ) => Promise<OutboxEntry> | OutboxEntry;
-  enqueueMedia?: (
-    ctx: ChannelMessageSendMediaContext<OpenClawConfig>,
-  ) => Promise<OutboxEntry> | OutboxEntry;
-  enqueuePayload?: (
-    ctx: ChannelMessageSendPayloadContext<OpenClawConfig>,
-  ) => Promise<OutboxEntry> | OutboxEntry;
-  now?: () => number;
-};
-
-export type BncrDurableMessageQueuedAdapterBuilderDeps = {
-  createMessageId: () => string;
-  now: () => number;
-  normalizeAccountId: (accountId?: string | null) => string;
-  normalizeReplyToId: (value?: string | null) => string;
-  resolveTarget: (ctx: ChannelMessageSendTextContext<OpenClawConfig>) => {
-    route: BncrRoute;
-    sessionKey: string;
-    accountId?: string | null;
-  };
-  filePushEvent?: string;
-};
-
-// This adapter intentionally models only the OpenClaw -> bncr-plugin handoff.
-// Once a message is accepted into bncr's own outbox, OpenClaw should stop managing it;
-// client/platform ACK, retry, and deadLetter remain owned by the bncr service framework.
-export function createBncrDurableMessageQueuedAdapter(
-  deps: BncrDurableMessageQueuedAdapterDeps,
-): ChannelMessageAdapterShape<OpenClawConfig, ChannelMessageSendResult> {
-  return defineChannelMessageAdapter({
-    id: 'bncr-queued-outbox',
-    receive: {
-      defaultAckPolicy: 'manual',
-      supportedAckPolicies: ['manual'],
-    },
-    send: {
-      text: async (ctx) => toChannelMessageSendResult(await deps.enqueueText(ctx), deps.now),
-      media: deps.enqueueMedia
-        ? async (ctx) => toChannelMessageSendResult(await deps.enqueueMedia?.(ctx), deps.now)
-        : undefined,
-      payload: deps.enqueuePayload
-        ? async (ctx) => toChannelMessageSendResult(await deps.enqueuePayload?.(ctx), deps.now)
-        : undefined,
-    },
-  });
-}
-
-export function createBncrDurableMessageQueuedAdapterFromBuilders(
-  deps: BncrDurableMessageQueuedAdapterBuilderDeps,
-): ChannelMessageAdapterShape<OpenClawConfig, ChannelMessageSendResult> {
-  return createBncrDurableMessageQueuedAdapter({
-    now: deps.now,
-    enqueueText: (ctx) => {
-      const resolved = deps.resolveTarget(ctx);
-      return buildTextOutboxEntry({
-        createMessageId: deps.createMessageId,
-        now: deps.now,
-        normalizeAccountId: deps.normalizeAccountId,
-        normalizeReplyToId: deps.normalizeReplyToId,
-        accountId: resolved.accountId ?? ctx.accountId ?? '',
-        sessionKey: resolved.sessionKey,
-        route: resolved.route,
-        text: ctx.text,
-        kind: 'final',
-        replyToId: ctx.replyToId ?? undefined,
-      });
-    },
-    enqueueMedia: (ctx) => {
-      const resolved = deps.resolveTarget(ctx);
-      return buildFileTransferOutboxEntry({
-        createMessageId: deps.createMessageId,
-        now: deps.now,
-        normalizeAccountId: deps.normalizeAccountId,
-        pushEvent: deps.filePushEvent ?? 'bncr.file.push',
-        accountId: resolved.accountId ?? ctx.accountId ?? '',
-        sessionKey: resolved.sessionKey,
-        route: resolved.route,
-        mediaUrl: ctx.mediaUrl,
-        mediaLocalRoots: ctx.mediaLocalRoots,
-        text: ctx.text,
-        asVoice: ctx.audioAsVoice,
-        audioAsVoice: ctx.audioAsVoice,
-        kind: 'final',
-        replyToId: ctx.replyToId ?? undefined,
-      });
-    },
-  });
-}
-
-function toChannelMessageSendResult(
-  entry: OutboxEntry | undefined,
-  now?: () => number,
-): ChannelMessageSendResult {
-  if (!entry) throw new Error('bncr durable message adapter did not receive an outbox entry');
-  const queued = buildBncrDurableQueuedResult({ entry, sentAt: now?.() });
-  const receipt: ChannelMessageSendResult['receipt'] = queued.receipt;
-  return {
-    receipt,
-    messageId: queued.receipt.primaryPlatformMessageId,
-  };
-}

package/src/messaging/outbound/send-params.ts

@@ -1,114 +0,0 @@
-import { normalizeAccountId } from '../../core/accounts.ts';
-import { readOpenClawBooleanParam, readOpenClawStringParam } from '../../openclaw/sdk-helpers.ts';
-import { extractConsumptionFields, parseBncrMarker } from './marker-parser.ts';
-
-export type NormalizedBncrSendParams = {
-  to: string;
-  accountId: string;
-  message: string;
-  caption: string;
-  mediaUrl?: string;
-  mediaUrls?: string[];
-  asVoice: boolean;
-  audioAsVoice: boolean;
-  type?: string;
-  downloadMedia?: boolean;
-  extra?: Record<string, unknown>;
-};
-
-function isPlainObject(value: unknown): value is Record<string, unknown> {
-  return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
-
-export function normalizeBncrSendParams(input: {
-  params: unknown;
-  accountId: string;
-}): NormalizedBncrSendParams {
-  const paramsObj = isPlainObject(input.params) ? input.params : {};
-  const to = readOpenClawStringParam(paramsObj, 'to', { required: true });
-  const resolvedAccountId = normalizeAccountId(
-    readOpenClawStringParam(paramsObj, 'accountId') ?? input.accountId,
-  );
-
-  const rawMessage = readOpenClawStringParam(paramsObj, 'message', { allowEmpty: true }) ?? '';
-  const rawCaption = readOpenClawStringParam(paramsObj, 'caption', { allowEmpty: true }) ?? '';
-  // Parse [BncrParam:...] markers from message and caption text
-  const { cleanText: parsedMessage, params: msgMarker } = parseBncrMarker(rawMessage);
-  const { cleanText: parsedCaption, params: capMarker } = parseBncrMarker(rawCaption);
-  const message = parsedMessage;
-  const caption = parsedCaption;
-  const mediaUrl =
-    readOpenClawStringParam(paramsObj, 'media', { trim: false }) ??
-    readOpenClawStringParam(paramsObj, 'path', { trim: false }) ??
-    readOpenClawStringParam(paramsObj, 'filePath', { trim: false }) ??
-    readOpenClawStringParam(paramsObj, 'mediaUrl', { trim: false });
-  const rawMediaUrls = paramsObj.mediaUrls;
-  const mediaUrls = Array.isArray(rawMediaUrls)
-    ? Array.from(
-        new Set(rawMediaUrls.map((v) => (typeof v === 'string' ? v.trim() : '')).filter(Boolean)),
-      )
-    : undefined;
-  // 如果 mediaUrl 已经在 mediaUrls 中，去重避免重复发送
-  const dedupedMediaUrls = mediaUrls?.length
-    ? mediaUrl && mediaUrls.includes(mediaUrl)
-      ? mediaUrls
-      : mediaUrl
-        ? [mediaUrl, ...mediaUrls]
-        : mediaUrls
-    : undefined;
-  const asVoice = readOpenClawBooleanParam(paramsObj, 'asVoice') ?? false;
-  const audioAsVoice = readOpenClawBooleanParam(paramsObj, 'audioAsVoice') ?? false;
-  const type = readOpenClawStringParam(paramsObj, 'type') || undefined;
-  const rawExtra = paramsObj.extra;
-  const paramsExtra = isPlainObject(rawExtra) ? { ...rawExtra } : {};
-  // Merge marker params into extra (marker takes priority over direct extra)
-  const mergedExtra = { ...paramsExtra, ...msgMarker, ...capMarker };
-  const extra = Object.keys(mergedExtra).length > 0 ? mergedExtra : undefined;
-
-  // Extract consumption fields from merged extra - marker-specified values override params
-  const { consumed, remaining } = extractConsumptionFields(
-    Object.keys(mergedExtra).length > 0 ? mergedExtra : undefined,
-  );
-  const effectiveAsVoice = consumed.asVoice ?? asVoice;
-  const effectiveAudioAsVoice = consumed.audioAsVoice ?? audioAsVoice;
-  const effectiveDownloadMedia = consumed.downloadMedia === true;
-  const effectiveType = consumed.type ?? type;
-  const cleanedExtra = Object.keys(remaining).length > 0 ? remaining : undefined;
-
-  const hasMedia = Boolean(mediaUrl || dedupedMediaUrls?.length);
-
-  if (effectiveAsVoice && !hasMedia) throw new Error('send voice requires media path');
-
-  const normalizedMessage = hasMedia ? '' : message || caption || '';
-  const normalizedCaption = hasMedia ? caption || message || '' : '';
-
-  if (!normalizedMessage.trim() && !normalizedCaption.trim() && !hasMedia && !extra) {
-    throw new Error('send requires message, media, or extra params');
-  }
-  console.log(
-    '[bncr] normalizeBncrSendParams rawMessage=' +
-      JSON.stringify(rawMessage) +
-      '|msgMarker=' +
-      JSON.stringify(msgMarker) +
-      '|consumed=' +
-      JSON.stringify(consumed) +
-      '|effectiveDownloadMedia=' +
-      effectiveDownloadMedia +
-      '|mediaUrl=' +
-      JSON.stringify(mediaUrl),
-  );
-
-  return {
-    to,
-    accountId: resolvedAccountId,
-    message: normalizedMessage,
-    caption: normalizedCaption,
-    mediaUrl: dedupedMediaUrls?.length ? undefined : mediaUrl || undefined,
-    mediaUrls: dedupedMediaUrls,
-    asVoice: effectiveAsVoice,
-    ...(effectiveDownloadMedia ? { downloadMedia: true } : {}),
-    audioAsVoice: effectiveAudioAsVoice,
-    ...(effectiveType ? { type: effectiveType } : {}),
-    ...(cleanedExtra ? { extra: cleanedExtra } : {}),
-  };
-}