@xmoxmo/bncr 0.5.6 → 0.5.8

package/src/core/targets.ts

@@ -1,4 +1,5 @@
 import type { BncrRoute } from './types.ts';
+import { asString } from './value-sanitize.ts';
 
 type RouteInputLike = {
   platform?: unknown;
@@ -28,14 +29,8 @@ export type BncrExplicitTarget = {
   groupId?: string;
 };
 
-export function asTargetString(value: unknown, fallback = ''): string {
-  if (typeof value === 'string') return value;
-  if (value == null) return fallback;
-  return String(value);
-}
-
-export function parseRouteFromStandardDisplayScope(scope: string): BncrRoute | null {
-  const parts = asTargetString(scope).trim().split(':');
+function parseRouteFromStandardDisplayScope(scope: string): BncrRoute | null {
+  const parts = asString(scope).trim().split(':');
   if (parts.length === 3) {
     const [platform, kind, id] = parts;
     if (!platform || !kind || !id) return null;
@@ -65,7 +60,7 @@ export function parseRouteFromStandardDisplayScope(scope: string): BncrRoute | n
 }
 
 function parseGroupScope(scope: string): { platform: string; groupId: string } | null {
-  const parts = asTargetString(scope).trim().split(':');
+  const parts = asString(scope).trim().split(':');
   if (parts.length !== 2) return null;
   const [platform, groupId] = parts;
   if (!platform || !groupId) return null;
@@ -73,15 +68,15 @@ function parseGroupScope(scope: string): { platform: string; groupId: string } |
 }
 
 function parseDirectScope(scope: string): { platform: string; userId: string } | null {
-  const parts = asTargetString(scope).trim().split(':');
+  const parts = asString(scope).trim().split(':');
   if (parts.length !== 2) return null;
   const [platform, userId] = parts;
   if (!platform || !userId) return null;
   return { platform, userId };
 }
 
-export function normalizeDisplayScopePrefix(scope: string): string {
-  const raw = asTargetString(scope).trim();
+function normalizeDisplayScopePrefix(scope: string): string {
+  const raw = asString(scope).trim();
   if (!raw) return '';
   if (raw.startsWith('Bncr:')) return raw;
   if (/^bncr[:-]/i.test(raw)) return raw;
@@ -89,11 +84,7 @@ export function normalizeDisplayScopePrefix(scope: string): string {
   return `Bncr:${raw}`;
 }
 
-export function resolveCanonicalSessionKind(): BncrSessionKind {
-  return 'direct';
-}
-
-export function buildExplicitTargetResult(args: {
+function buildExplicitTargetResult(args: {
   raw: string;
   source:
     | 'display-scope'
@@ -121,8 +112,8 @@ export function buildExplicitTargetResult(args: {
   };
 }
 
-export function isLowerHex(input: string): boolean {
-  const raw = asTargetString(input).trim();
+function isLowerHex(input: string): boolean {
+  const raw = asString(input).trim();
   return !!raw && /^[0-9a-fA-F]+$/.test(raw) && raw.length % 2 === 0;
 }
 
@@ -131,18 +122,18 @@ export function routeScopeToHex(route: BncrRoute): string {
   return Buffer.from(raw, 'utf8').toString('hex').toLowerCase();
 }
 
-export function groupScopeToHex(route: BncrRoute): string {
+function groupScopeToHex(route: BncrRoute): string {
   const raw = `${route.platform}:${route.groupId}`;
   return Buffer.from(raw, 'utf8').toString('hex').toLowerCase();
 }
 
-export function directScopeToHex(route: BncrRoute): string {
+function directScopeToHex(route: BncrRoute): string {
   const raw = `${route.platform}:${route.userId}`;
   return Buffer.from(raw, 'utf8').toString('hex').toLowerCase();
 }
 
-export function parseRouteFromScope(scope: string): BncrRoute | null {
-  const parts = asTargetString(scope).trim().split(':');
+function parseRouteFromScope(scope: string): BncrRoute | null {
+  const parts = asString(scope).trim().split(':');
   if (parts.length < 3) return null;
   const [platform, groupId, userId] = parts;
   if (!platform || !groupId || !userId) return null;
@@ -150,8 +141,8 @@ export function parseRouteFromScope(scope: string): BncrRoute | null {
   return { platform, groupId, userId };
 }
 
-export function parseRouteFromHexScope(scopeHex: string): BncrRoute | null {
-  const rawHex = asTargetString(scopeHex).trim();
+function parseRouteFromHexScope(scopeHex: string): BncrRoute | null {
+  const rawHex = asString(scopeHex).trim();
   if (!isLowerHex(rawHex)) return null;
 
   try {
@@ -162,8 +153,8 @@ export function parseRouteFromHexScope(scopeHex: string): BncrRoute | null {
   }
 }
 
-export function parseGroupRouteFromHexScope(scopeHex: string): BncrRoute | null {
-  const rawHex = asTargetString(scopeHex).trim();
+function parseGroupRouteFromHexScope(scopeHex: string): BncrRoute | null {
+  const rawHex = asString(scopeHex).trim();
   if (!isLowerHex(rawHex)) return null;
 
   try {
@@ -176,8 +167,8 @@ export function parseGroupRouteFromHexScope(scopeHex: string): BncrRoute | null
   }
 }
 
-export function parseDirectRouteFromHexScope(scopeHex: string): BncrRoute | null {
-  const rawHex = asTargetString(scopeHex).trim();
+function parseDirectRouteFromHexScope(scopeHex: string): BncrRoute | null {
+  const rawHex = asString(scopeHex).trim();
   if (!isLowerHex(rawHex)) return null;
 
   try {
@@ -197,15 +188,15 @@ export function parseStrictBncrSessionKey(input: string): {
   scopeHex: string;
   route: BncrRoute;
 } | null {
-  const raw = asTargetString(input).trim();
+  const raw = asString(input).trim();
   if (!raw) return null;
 
   const m = raw.match(/^agent:([^:]+):bncr:(direct|group):(.+)$/);
   if (!m?.[1] || !m?.[2] || !m?.[3]) return null;
 
-  const inputAgentId = asTargetString(m[1]).trim();
+  const inputAgentId = asString(m[1]).trim();
   const inputKind = m[2] as BncrSessionKind;
-  const payload = asTargetString(m[3]).trim();
+  const payload = asString(m[3]).trim();
   let route: BncrRoute | null = null;
   let scopeHex = '';
 
@@ -243,13 +234,13 @@ export function parseStrictBncrSessionKey(input: string): {
   };
 }
 
-export function parseLegacySessionKey(input: string): {
+function parseLegacySessionKey(input: string): {
   route: BncrRoute;
   inputKind: BncrSessionKind;
   inputAgentId?: string;
   source: 'legacy-direct' | 'legacy-bncr' | 'legacy-agent' | 'hex';
 } | null {
-  const raw = asTargetString(input).trim();
+  const raw = asString(input).trim();
   if (!raw) return null;
 
   const directLegacy = raw.match(/^agent:([^:]+):bncr:direct:([0-9a-fA-F]+):0$/);
@@ -304,10 +295,10 @@ export function parseLegacySessionKey(input: string): {
   return null;
 }
 
-export function isLegacyNoiseRoute(route: BncrRoute): boolean {
-  const platform = asTargetString(route.platform).trim().toLowerCase();
-  const groupId = asTargetString(route.groupId).trim().toLowerCase();
-  const userId = asTargetString(route.userId).trim().toLowerCase();
+function isLegacyNoiseRoute(route: BncrRoute): boolean {
+  const platform = asString(route.platform).trim().toLowerCase();
+  const groupId = asString(route.groupId).trim().toLowerCase();
+  const userId = asString(route.userId).trim().toLowerCase();
 
   if (platform === 'agent' && groupId === 'main' && userId === 'bncr') return true;
   if (platform === 'bncr' && userId === '0' && isLowerHex(groupId)) return true;
@@ -326,7 +317,7 @@ export function parseRouteFromDisplayScope(scope: string): BncrRoute | null {
 }
 
 export function buildCanonicalBncrSessionKey(route: BncrRoute, canonicalAgentId: string): string {
-  const agentId = asTargetString(canonicalAgentId).trim() || 'main';
+  const agentId = asString(canonicalAgentId).trim() || 'main';
   if (route.groupId !== '0') {
     return `agent:${agentId}:bncr:group:${groupScopeToHex(route)}`;
   }
@@ -338,7 +329,7 @@ export function normalizeStoredSessionKey(
   canonicalAgentId?: string | null,
   helpers?: { normalizeTaskKey?: (input: unknown) => string | null },
 ): { sessionKey: string; route: BncrRoute } | null {
-  const raw = asTargetString(input).trim();
+  const raw = asString(input).trim();
   if (!raw) return null;
 
   let taskKey: string | null = null;
@@ -346,7 +337,7 @@ export function normalizeStoredSessionKey(
 
   const taskTagged = raw.match(/^(.*):task:([a-z0-9_-]{1,32})$/i);
   if (taskTagged) {
-    base = asTargetString(taskTagged[1]).trim();
+    base = asString(taskTagged[1]).trim();
     taskKey = helpers?.normalizeTaskKey?.(taskTagged[2]) || null;
   }
 
@@ -370,7 +361,7 @@ export function normalizeStoredSessionKey(
   if (!route) return null;
   if (isLegacyNoiseRoute(route)) return null;
 
-  const finalAgentId = asTargetString(canonicalAgentId).trim() || passthroughAgentId;
+  const finalAgentId = asString(canonicalAgentId).trim() || passthroughAgentId;
   if (!finalAgentId) return null;
 
   const finalSessionKey = buildCanonicalBncrSessionKey(route, finalAgentId);
@@ -385,7 +376,7 @@ export function normalizeInboundSessionKey(
   route: BncrRoute,
   canonicalAgentId: string,
 ): string | null {
-  const raw = asTargetString(scope).trim();
+  const raw = asString(scope).trim();
   let finalRoute: BncrRoute | null = null;
 
   if (!raw) finalRoute = route;
@@ -399,15 +390,11 @@ export function normalizeInboundSessionKey(
   return buildCanonicalBncrSessionKey(finalRoute, canonicalAgentId);
 }
 
-export function buildFallbackSessionKey(route: BncrRoute, canonicalAgentId: string): string {
-  return buildCanonicalBncrSessionKey(route, canonicalAgentId);
-}
-
 export function parseRouteLike(input: unknown): BncrRoute | null {
   const routeInput = input && typeof input === 'object' ? (input as RouteInputLike) : null;
-  const platform = asTargetString(routeInput?.platform || '').trim();
-  const groupId = asTargetString(routeInput?.groupId || '').trim();
-  const userId = asTargetString(routeInput?.userId || '').trim();
+  const platform = asString(routeInput?.platform || '').trim();
+  const groupId = asString(routeInput?.groupId || '').trim();
+  const userId = asString(routeInput?.userId || '').trim();
   if (!platform || !groupId || !userId) return null;
   if (groupId !== '0' && userId !== '0') return null;
   return { platform, groupId, userId };
@@ -421,28 +408,23 @@ export type BncrExplicitTargetSource =
   | 'route-scope';
 
 export function formatDisplayScope(route: BncrRoute): string {
-  const platform = asTargetString(route?.platform).trim();
-  const groupId = asTargetString(route?.groupId || '0').trim() || '0';
-  const userId = asTargetString(route?.userId || '0').trim() || '0';
+  const platform = asString(route?.platform).trim();
+  const groupId = asString(route?.groupId || '0').trim() || '0';
+  const userId = asString(route?.userId || '0').trim() || '0';
   if (groupId !== '0') return `Bncr:${platform}:${groupId}:0`;
   return `Bncr:${platform}:0:${userId}`;
 }
 
 export function formatHumanDisplayScope(route: BncrRoute): string {
-  const platform = asTargetString(route?.platform).trim();
-  const groupId = asTargetString(route?.groupId || '0').trim() || '0';
-  const userId = asTargetString(route?.userId || '0').trim() || '0';
+  const platform = asString(route?.platform).trim();
+  const groupId = asString(route?.groupId || '0').trim() || '0';
+  const userId = asString(route?.userId || '0').trim() || '0';
   if (groupId !== '0') return `Bncr:${platform}:Group:${groupId}`;
   if (userId !== '0') return `Bncr:${platform}:User:${userId}`;
   return formatDisplayScope(route);
 }
 
-export function buildDisplayScopeCandidates(route: BncrRoute): string[] {
-  const candidates = [formatDisplayScope(route)].filter(Boolean);
-  return Array.from(new Set(candidates.map((x) => asTargetString(x).trim()).filter(Boolean)));
-}
-
-export function resolveExplicitTargetRoute(raw: string): {
+function resolveExplicitTargetRoute(raw: string): {
   route: BncrRoute | null;
   source: BncrExplicitTargetSource | null;
 } {
@@ -491,10 +473,10 @@ export function resolveExplicitTargetRoute(raw: string): {
 }
 
 export function parseExplicitTarget(input: string, options?: { canonicalAgentId?: string | null }) {
-  const raw = asTargetString(input).trim();
+  const raw = asString(input).trim();
   if (!raw) return null;
 
-  const canonicalAgentId = asTargetString(options?.canonicalAgentId).trim() || undefined;
+  const canonicalAgentId = asString(options?.canonicalAgentId).trim() || undefined;
   const { route, source } = resolveExplicitTargetRoute(raw);
   if (!route || !source) return null;
 
@@ -510,8 +492,8 @@ export function parseExplicitTarget(input: string, options?: { canonicalAgentId?
   });
 }
 
-export function normalizeTaskKey(input: unknown): string | null {
-  const raw = asTargetString(input).trim().toLowerCase();
+function normalizeTaskKey(input: unknown): string | null {
+  const raw = asString(input).trim().toLowerCase();
   if (!raw) return null;
   const normalized = raw
     .replace(/[^a-z0-9_-]+/g, '-')
@@ -521,7 +503,7 @@ export function normalizeTaskKey(input: unknown): string | null {
 }
 
 export function extractInlineTaskKey(text: string): { taskKey: string | null; text: string } {
-  const raw = asTargetString(text);
+  const raw = asString(text);
   if (!raw) return { taskKey: null, text: '' };
 
   const tagged = raw.match(
@@ -530,7 +512,7 @@ export function extractInlineTaskKey(text: string): { taskKey: string | null; te
   if (tagged) {
     return {
       taskKey: normalizeTaskKey(tagged[1]),
-      text: asTargetString(tagged[2]),
+      text: asString(tagged[2]),
     };
   }
 
@@ -538,7 +520,7 @@ export function extractInlineTaskKey(text: string): { taskKey: string | null; te
   if (spaced) {
     return {
       taskKey: normalizeTaskKey(spaced[1]),
-      text: asTargetString(spaced[2]),
+      text: asString(spaced[2]),
     };
   }
 
@@ -546,7 +528,7 @@ export function extractInlineTaskKey(text: string): { taskKey: string | null; te
 }
 
 export function withTaskSessionKey(sessionKey: string, taskKey?: string | null): string {
-  const base = asTargetString(sessionKey).trim();
+  const base = asString(sessionKey).trim();
   const tk = normalizeTaskKey(taskKey);
   if (!base || !tk) return base;
   if (/:task:[a-z0-9_-]+(?:$|:)/i.test(base)) return base;

package/src/core/types.ts

@@ -21,8 +21,6 @@ export type BncrGatewayCapabilityFlags = {
   inboundOnly: boolean;
 };
 
-export type FileRecvTransferStatus = 'init' | 'transferring' | 'completed' | 'aborted';
-
 type FileRecvTransferBase = {
   transferId: string;
   accountId: string;
@@ -105,21 +103,7 @@ export type OutboxEntry = {
   accountId: string;
   sessionKey: string;
   route: BncrRoute;
-  payload: Record<string, unknown> & {
-    _meta?: {
-      kind?: 'message' | 'file-transfer';
-      retryCount?: number;
-      nextAttemptAt?: number;
-      mediaUrl?: string;
-      mediaLocalRoots?: string[];
-      text?: string;
-      asVoice?: boolean;
-      audioAsVoice?: boolean;
-      replyToId?: string;
-      finalEvent?: string;
-      [key: string]: unknown;
-    };
-  };
+  payload: Record<string, unknown>;
   createdAt: number;
   retryCount: number;
   nextAttemptAt: number;

package/src/core/value-sanitize.ts

@@ -1,15 +1,39 @@
-export function asSanitizedString(value: unknown, fallback = ''): string {
+export function asString(value: unknown, fallback = ''): string {
   if (typeof value === 'string') return value;
   if (value == null) return fallback;
   return String(value);
 }
 
+/** Alias: same as asString, named for call-sites that want a sanitization signal. */
+export const asSanitizedString = asString;
+
 export function finiteNumberOr(value: unknown, fallback: number): number {
   const n = Number(value);
   return Number.isFinite(n) ? n : fallback;
 }
 
+export function now(): number {
+  return Date.now();
+}
+
+export function asBoolean(value: unknown, fallback = false): boolean {
+  if (typeof value === 'boolean') return value;
+  if (typeof value === 'number') return value !== 0;
+  if (typeof value === 'string') {
+    const raw = value.trim().toLowerCase();
+    if (!raw) return fallback;
+    if (['true', '1', 'yes', 'y', 'on'].includes(raw)) return true;
+    if (['false', '0', 'no', 'n', 'off'].includes(raw)) return false;
+  }
+  return fallback;
+}
+
+export function isPlainObject(value: unknown): value is Record<string, unknown> {
+  return typeof value === 'object' && value !== null && !Array.isArray(value);
+}
+
 export function finiteNumberOrNull(value: unknown): number | null {
+  if (value === null || value === undefined) return null;
   const n = Number(value);
   return Number.isFinite(n) ? n : null;
 }
@@ -27,3 +51,16 @@ export function clampFiniteNumber(
   const finite = finiteNumberOr(value, fallback);
   return Math.max(min, Math.min(finite, max));
 }
+
+export function asStringArray(v: unknown): string[] {
+  if (Array.isArray(v)) {
+    return v.map((item) => asString(item).trim()).filter(Boolean);
+  }
+  if (typeof v === 'string') {
+    return v
+      .split(',')
+      .map((item) => item.trim())
+      .filter(Boolean);
+  }
+  return [];
+}

package/src/messaging/inbound/commands.ts

@@ -7,10 +7,7 @@ import {
   resolveBncrPinnedMainDmOwnerFromAllowlist,
 } from '../../openclaw/inbound-session-runtime.ts';
 import { dispatchOpenClawReplyWithBufferedBlockDispatcher } from '../../openclaw/reply-runtime.ts';
-import {
-  type OpenClawResolvedAgentRoute,
-  resolveOpenClawAgentRoute,
-} from '../../openclaw/routing-runtime.ts';
+import { resolveOpenClawAgentRoute } from '../../openclaw/routing-runtime.ts';
 import type {
   BncrEnqueueFromReply,
   BncrInboundApi,
@@ -18,7 +15,7 @@ import type {
   BncrInboundLogger,
   BncrRememberSessionRoute,
 } from './contracts.ts';
-import type { ParsedInbound } from './dispatch-prep.ts';
+import { assertResolvedAgentRoute, type ParsedInbound } from './dispatch-prep.ts';
 import { buildBncrInboundRecordUpdateLastRoute } from './last-route.ts';
 import {
   parseBncrNativeCommand,
@@ -50,25 +47,6 @@ import {
 } from './session-label.ts';
 import { createBncrSessionMetaTaskBarrier } from './session-meta-task.ts';
 
-function assertResolvedAgentRoute(resolvedRoute: OpenClawResolvedAgentRoute): {
-  sessionKey: string;
-  agentId: string;
-  mainSessionKey?: string;
-} {
-  const sessionKey =
-    typeof resolvedRoute.sessionKey === 'string' ? resolvedRoute.sessionKey.trim() : '';
-  const agentId = typeof resolvedRoute.agentId === 'string' ? resolvedRoute.agentId.trim() : '';
-  if (!sessionKey) throw new Error('OpenClaw resolveAgentRoute returned empty sessionKey');
-  if (!agentId) throw new Error('OpenClaw resolveAgentRoute returned empty agentId');
-  return {
-    sessionKey,
-    agentId,
-    ...(typeof resolvedRoute.mainSessionKey === 'string' && resolvedRoute.mainSessionKey.trim()
-      ? { mainSessionKey: resolvedRoute.mainSessionKey }
-      : {}),
-  };
-}
-
 function buildBncrNativeCommandResolvedRoute(args: {
   api: BncrInboundApi;
   cfg: BncrInboundConfig;

package/src/messaging/inbound/dispatch-prep.ts

@@ -95,7 +95,7 @@ const BNCR_GENERIC_MEDIA_TEXTS = new Set([
   '[文件]',
 ]);
 
-function assertResolvedAgentRoute(resolvedRoute: OpenClawResolvedAgentRoute): {
+export function assertResolvedAgentRoute(resolvedRoute: OpenClawResolvedAgentRoute): {
   sessionKey: string;
   agentId: string;
   mainSessionKey?: string;

package/src/messaging/inbound/gate.ts

@@ -6,36 +6,7 @@ export type BncrGateResult = { allowed: true } | { allowed: false; reason: strin
 const REQUIRED_PROTOCOL_VERSION = 'scene-routing-v1';
 const REQUIRED_CAPABILITY = 'scene-routing-v1';
 
-function asString(v: unknown, fallback = ''): string {
-  if (typeof v === 'string') return v;
-  if (v == null) return fallback;
-  return String(v);
-}
-
-function asBoolean(v: unknown, fallback = false): boolean {
-  if (typeof v === 'boolean') return v;
-  if (typeof v === 'number') return v !== 0;
-  if (typeof v === 'string') {
-    const raw = v.trim().toLowerCase();
-    if (!raw) return fallback;
-    if (['true', '1', 'yes', 'y', 'on'].includes(raw)) return true;
-    if (['false', '0', 'no', 'n', 'off'].includes(raw)) return false;
-  }
-  return fallback;
-}
-
-function asCapabilities(v: unknown): string[] {
-  if (Array.isArray(v)) {
-    return v.map((item) => asString(item).trim()).filter(Boolean);
-  }
-  if (typeof v === 'string') {
-    return v
-      .split(',')
-      .map((item) => item.trim())
-      .filter(Boolean);
-  }
-  return [];
-}
+import { asBoolean, asString, asStringArray } from '../../core/value-sanitize.ts';
 
 export async function checkBncrMessageGate(params: {
   parsed: BncrInboundParamsInput & {
@@ -57,7 +28,7 @@ export async function checkBncrMessageGate(params: {
   }
 
   const protocolVersion = asString(parsed?.protocolVersion || '').trim();
-  const capabilities = asCapabilities(parsed?.capabilities);
+  const capabilities = asStringArray(parsed?.capabilities);
   if (
     protocolVersion !== REQUIRED_PROTOCOL_VERSION ||
     !capabilities.includes(REQUIRED_CAPABILITY)

package/src/messaging/inbound/group-history.ts

@@ -1,4 +1,3 @@
-import path from 'node:path';
 import {
   createChannelHistoryWindow,
   DEFAULT_GROUP_HISTORY_LIMIT,
@@ -245,80 +244,6 @@ export function collectBncrHistoryMediaFromEntries(args: {
 }): HistoryMediaEntry[] {
   return args.entries.flatMap((entry) => (Array.isArray(entry.media) ? entry.media : []));
 }
-
-function toBncrPromptMediaPath(value: string | undefined): string | undefined {
-  const normalized = String(value || '').trim();
-  if (!normalized) return undefined;
-  const slashNormalized = normalized.replace(/\\/g, '/');
-  if (!slashNormalized.includes('/media/inbound/')) return undefined;
-  const base = path.posix.basename(slashNormalized);
-  if (!base || base === '.' || base === '..') return undefined;
-  return `media://inbound/${encodeURIComponent(base)}`;
-}
-
-function buildHistoryWindowMediaSummary(
-  media: HistoryMediaEntry[] | undefined,
-): string | undefined {
-  if (!Array.isArray(media) || media.length === 0) return undefined;
-  const kinds = media
-    .map((item) =>
-      String(item?.kind || '')
-        .trim()
-        .toLowerCase(),
-    )
-    .filter(Boolean);
-  const uniformKind = kinds[0] && kinds.every((item) => item === kinds[0]) ? kinds[0] : 'document';
-  const count = media.length;
-  if (uniformKind === 'image')
-    return count === 1 ? '<media:image>' : `<media:image> (${count} images)`;
-  if (uniformKind === 'video')
-    return count === 1 ? '<media:video>' : `<media:video> (${count} videos)`;
-  if (uniformKind === 'audio') {
-    return count === 1 ? '<media:audio>' : `<media:audio> (${count} audio attachments)`;
-  }
-  return count === 1 ? '<media:document>' : `<media:document> (${count} attachments)`;
-}
-
-function buildHistoryWindowMedias(media: HistoryMediaEntry[] | undefined) {
-  if (!Array.isArray(media) || media.length === 0) return [];
-  return media.flatMap((item) => {
-    const promptPath = toBncrPromptMediaPath(item?.path);
-    const payload = {
-      ...(promptPath ? { path: promptPath } : {}),
-      ...(item?.contentType ? { contentType: item.contentType } : {}),
-      ...(item?.kind ? { kind: item.kind } : {}),
-      ...(item?.messageId ? { messageId: item.messageId } : {}),
-    };
-    return Object.keys(payload).length > 0 ? [payload] : [];
-  });
-}
-
-export function buildBncrHistoryWindowContextFromEntries(args: {
-  entries: readonly BncrHistoryEntry[];
-  channelId: string;
-}) {
-  if (args.entries.length === 0) return undefined;
-  const messages = args.entries.map((entry) => ({
-    ...(entry.messageId ? { messageId: entry.messageId } : {}),
-    sender: entry.sender,
-    senderId: entry.senderId,
-    ...(typeof entry.timestamp === 'number' ? { timestampMs: entry.timestamp } : {}),
-    body: entry.body,
-    mediaSummary: buildHistoryWindowMediaSummary(entry.media),
-    medias: buildHistoryWindowMedias(entry.media),
-  }));
-  return {
-    label: 'Bncr history window',
-    source: args.channelId,
-    type: 'bncr.history_window',
-    payload: {
-      relation: 'before_current_message',
-      order: 'chronological',
-      messages,
-    },
-  };
-}
-
 export function clearBncrPendingGroupHistory(args: {
   historyMap: BncrGroupHistoryMap;
   parsed: ParsedInbound;

package/src/messaging/inbound/parse.ts

@@ -2,39 +2,9 @@ import { createHash } from 'node:crypto';
 import { normalizeAccountId } from '../../core/accounts.ts';
 import { extractInlineTaskKey } from '../../core/targets.ts';
 import type { BncrRoute } from '../../core/types.ts';
+import { asBoolean, asString, asStringArray } from '../../core/value-sanitize.ts';
 import type { BncrInboundMediaItem, BncrInboundParamsInput } from './contracts.ts';
 
-function asString(v: unknown, fallback = ''): string {
-  if (typeof v === 'string') return v;
-  if (v == null) return fallback;
-  return String(v);
-}
-
-function asBoolean(v: unknown, fallback = false): boolean {
-  if (typeof v === 'boolean') return v;
-  if (typeof v === 'number') return v !== 0;
-  if (typeof v === 'string') {
-    const raw = v.trim().toLowerCase();
-    if (!raw) return fallback;
-    if (['true', '1', 'yes', 'y', 'on'].includes(raw)) return true;
-    if (['false', '0', 'no', 'n', 'off'].includes(raw)) return false;
-  }
-  return fallback;
-}
-
-function asStringArray(v: unknown): string[] {
-  if (Array.isArray(v)) {
-    return v.map((item) => asString(item).trim()).filter(Boolean);
-  }
-  if (typeof v === 'string') {
-    return v
-      .split(',')
-      .map((item) => item.trim())
-      .filter(Boolean);
-  }
-  return [];
-}
-
 function asInboundMediaItems(params: BncrInboundParamsInput): BncrInboundMediaItem[] {
   const normalized: BncrInboundMediaItem[] = [];
   const rawList = Array.isArray(params?.mediaList) ? params.mediaList : [];
@@ -88,7 +58,7 @@ function asInboundMediaItems(params: BncrInboundParamsInput): BncrInboundMediaIt
   }));
 }
 
-export function inboundDedupKey(params: {
+function inboundDedupKey(params: {
   accountId: string;
   platform: string;
   groupId: string;