@xmoxmo/bncr 0.4.2 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xmoxmo/bncr",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "private": false,
   "type": "module",
   "license": "MIT",

package/scripts/selfcheck.mjs

@@ -1,3 +1,4 @@
+import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import { createRequire } from 'node:module';
 import path from 'node:path';
@@ -182,6 +183,19 @@ const readPackageVersion = () => {
   return typeof pkg?.version === 'string' ? pkg.version.trim() : '';
 };
 
+const readNpmLatestVersion = (packageName) => {
+  try {
+    const raw = execFileSync('npm', ['view', packageName, 'version'], {
+      encoding: 'utf8',
+      stdio: ['ignore', 'pipe', 'pipe'],
+      timeout: 10000,
+    }).trim();
+    return raw || null;
+  } catch {
+    return null;
+  }
+};
+
 const requiredOpenClawSdkSubpaths = [
   'openclaw/plugin-sdk',
   'openclaw/plugin-sdk/boolean-param',
@@ -213,7 +227,7 @@ const resolveOpenClawSdkSubpaths = () => {
   });
 };
 
-const validateVersionPolicy = (version) => {
+const validateVersionPolicy = (version, latestVersion) => {
   const match = version.match(/^(\d+)\.(\d+)\.(\d+)$/);
   if (!match) {
     return {
@@ -232,12 +246,101 @@ const validateVersionPolicy = (version) => {
     };
   }
 
+  // Prevent jumping minor when current minor still has unused patch slots
+  if (latestVersion) {
+    const lm = latestVersion.match(/^(\d+)\.(\d+)\.(\d+)$/);
+    if (lm) {
+      const vMajor = Number.parseInt(match[1], 10);
+      const vMinor = Number.parseInt(match[2], 10);
+      const lMajor = Number.parseInt(lm[1], 10);
+      const lMinor = Number.parseInt(lm[2], 10);
+      const lPatch = Number.parseInt(lm[3], 10);
+
+      // Version unchanged
+      if (vMajor === lMajor && vMinor === lMinor && patch === lPatch) {
+        return { ok: false, reason: `version unchanged: ${version}`, version };
+      }
+
+      // Downgrade: any component decreased
+      if (
+        vMajor < lMajor ||
+        (vMajor === lMajor && vMinor < lMinor) ||
+        (vMajor === lMajor && vMinor === lMinor && patch < lPatch)
+      ) {
+        return { ok: false, reason: `downgrade from ${latestVersion} to ${version}`, version };
+      }
+
+      // Patch bump: same major and minor, patch must increment by exactly +1
+      if (vMajor === lMajor && vMinor === lMinor) {
+        if (patch === lPatch + 1) return { ok: true, version };
+        return {
+          ok: false,
+          reason: `patch jump from ${latestVersion} to ${version} (delta=${patch - lPatch}); expected ${lMajor}.${lMinor}.${lPatch + 1}`,
+          version,
+        };
+      }
+
+      // Minor bump: same major, minor + 1
+      if (vMajor === lMajor && vMinor === lMinor + 1) {
+        if (lPatch === 9 && patch === 0) return { ok: true, version };
+        if (lPatch !== 9)
+          return {
+            ok: false,
+            reason: `minor bumped from ${latestVersion} to ${version} but ${9 - lPatch} patch slots remain in ${lMajor}.${lMinor}; prefer ${lMajor}.${lMinor}.${lPatch + 1}`,
+            version,
+          };
+        return {
+          ok: false,
+          reason: `minor bump must reset patch to 0; got ${vMajor}.${vMinor}.${patch}`,
+          version,
+        };
+      }
+
+      // Minor jumped by more than 1
+      if (vMajor === lMajor && vMinor > lMinor + 1) {
+        return {
+          ok: false,
+          reason: `minor jumped from ${latestVersion} to ${version}; expected ${lMajor}.${lMinor + 1}.0`,
+          version,
+        };
+      }
+
+      // Major bump: major + 1, requires previous minor fully exhausted
+      if (vMajor === lMajor + 1) {
+        if (lMinor === 9 && lPatch === 9 && vMinor === 0 && patch === 0)
+          return { ok: true, version };
+        if (lMinor !== 9 || lPatch !== 9)
+          return {
+            ok: false,
+            reason: `major bumped from ${latestVersion} to ${version} but previous major series not exhausted; expected ${lMajor + 1}.0.0 after ${lMajor}.9.9`,
+            version,
+          };
+        return {
+          ok: false,
+          reason: `major bump must reset to .0.0; got ${vMajor}.${vMinor}.${patch}`,
+          version,
+        };
+      }
+
+      // Major jumped by more than 1
+      if (vMajor > lMajor + 1) {
+        return {
+          ok: false,
+          reason: `major jumped from ${latestVersion} to ${version}; expected ${lMajor + 1}.0.0`,
+          version,
+        };
+      }
+    }
+  }
+
   return { ok: true, version };
 };
 
 const missing = requiredFiles.filter((rel) => !fs.existsSync(path.join(root, rel)));
 const version = readPackageVersion();
-const versionPolicy = validateVersionPolicy(version);
+const packageName = '@xmoxmo/bncr';
+const latestVersion = readNpmLatestVersion(packageName);
+const versionPolicy = validateVersionPolicy(version, latestVersion);
 const sdkSubpaths = resolveOpenClawSdkSubpaths();
 const missingSdkSubpaths = sdkSubpaths.filter((entry) => !entry.ok);
 const result = {

package/src/channel.ts

@@ -1164,6 +1164,7 @@ class BncrBridgeRuntime {
     asVoice?: boolean;
     audioAsVoice?: boolean;
     type?: string;
+    extra?: Record<string, unknown>;
     kind?: 'tool' | 'block' | 'final';
     replyToId?: string;
     replyTargetPolicy?: OutboundReplyTargetPolicy;
@@ -1182,6 +1183,7 @@ class BncrBridgeRuntime {
       asVoice: params.asVoice,
       audioAsVoice: params.audioAsVoice,
       type: params.type,
+      extra: params.extra,
       kind: params.kind,
       replyToId: asString(params.replyToId || '').trim() || undefined,
       replyTargetPolicy: params.replyTargetPolicy,
@@ -1245,6 +1247,7 @@ class BncrBridgeRuntime {
         mimeType: params.media.mimeType,
       }),
       hintedType: wantsVoice ? 'voice' : asString(params.meta.type || '') || undefined,
+      extra: params.meta.extra as Record<string, unknown> | undefined,
       kind: messageKind,
       replyToId: normalizeReplyToId(params.meta.replyToId) || undefined,
       now: now(),
@@ -1724,6 +1727,10 @@ class BncrBridgeRuntime {
     handleFileTransferPushFailure: (args) => this.handleFileTransferPushFailure(args),
     handleTextPushFailure: (args) => this.handleTextPushFailure(args),
     isPrePushGuardDeferral: (entry) => this.isPrePushGuardDeferral(entry),
+    resolveAccountIdForSession: (sessionKey) => {
+      const hit = this.sessionRoutes.get(sessionKey);
+      return hit ? hit.accountId : BNCR_DEFAULT_ACCOUNT_ID;
+    },
     scheduleSave: () => this.scheduleSave(),
     logInfo: (scope, message, options) => this.logInfo(scope, message, options),
     logWarn: (scope, message, options) => this.logWarn(scope, message, options),

package/src/core/outbox-entry-builders.ts

@@ -16,6 +16,7 @@ export function buildFileTransferOutboxEntry(args: {
   asVoice?: boolean;
   audioAsVoice?: boolean;
   type?: string;
+  extra?: Record<string, unknown>;
   kind?: 'tool' | 'block' | 'final';
   replyToId?: string;
   replyTargetPolicy?: OutboundReplyTargetPolicy;
@@ -38,6 +39,7 @@ export function buildFileTransferOutboxEntry(args: {
         asVoice: args.asVoice === true,
         audioAsVoice: args.audioAsVoice === true,
         type: args.type,
+        ...(args.extra ? { extra: { ...args.extra } } : {}),
         finalEvent: args.pushEvent,
         replyToId:
           normalizeOutboundReplyToId({

package/src/messaging/outbound/media.ts

@@ -68,6 +68,7 @@ export function buildBncrMediaOutboundFrame(params: {
   mediaMsg: string;
   fileName: string;
   hintedType?: string;
+  extra?: Record<string, unknown>;
   kind?: 'tool' | 'block' | 'final';
   replyToId?: string;
   now: number;
@@ -96,6 +97,7 @@ export function buildBncrMediaOutboundFrame(params: {
       base64: params.media.mediaBase64 || '',
       fileName: params.fileName,
       transferMode: params.media.mode,
+      ...(params.extra ? { extra: { ...params.extra } } : {}),
     },
     ts: params.now,
   };

package/src/messaging/outbound/reply-enqueue-media.ts

@@ -62,6 +62,7 @@ export function enqueueReplyMediaFileTransferEntry(
       asVoice: boolean;
       audioAsVoice: boolean;
       type?: string;
+      extra?: Record<string, unknown>;
       kind?: 'tool' | 'block' | 'final';
       replyToId?: string;
       replyTargetPolicy?: OutboundReplyTargetPolicy;
@@ -86,6 +87,7 @@ export function enqueueReplyMediaFileTransferEntry(
       asVoice: params.asVoice,
       audioAsVoice: params.audioAsVoice,
       type: params.type,
+      extra: params.extra,
       kind: params.kind,
       replyToId: params.replyToId || undefined,
       replyTargetPolicy: params.replyTargetPolicy,
@@ -132,6 +134,7 @@ export function enqueueSingleReplyMediaEntry(
     asVoice: params.params.payload.asVoice,
     audioAsVoice: params.params.payload.audioAsVoice,
     type: params.params.payload.type,
+    extra: params.params.payload.extra,
     kind: params.params.payload.kind,
     replyToId: params.params.payload.replyToId,
     replyTargetPolicy: params.params.payload.replyTargetPolicy,

package/src/messaging/outbound/reply-enqueue.ts

@@ -19,6 +19,7 @@ export type ReplyPayloadInput = {
   asVoice?: boolean;
   audioAsVoice?: boolean;
   type?: string;
+  extra?: Record<string, unknown>;
   kind?: 'tool' | 'block' | 'final';
   replyToId?: string;
 };
@@ -31,6 +32,7 @@ export type NormalizedReplyPayload = {
   asVoice: boolean;
   audioAsVoice: boolean;
   type?: string;
+  extra?: Record<string, unknown>;
   kind?: 'tool' | 'block' | 'final';
   replyToId: string;
   replyTargetPolicy: OutboundReplyTargetPolicy;
@@ -64,6 +66,7 @@ export type ReplyMediaFileTransferParams = {
   asVoice: boolean;
   audioAsVoice: boolean;
   type?: string;
+  extra?: Record<string, unknown>;
   kind?: 'tool' | 'block' | 'final';
   replyToId: string;
   replyTargetPolicy: OutboundReplyTargetPolicy;
@@ -268,6 +271,7 @@ export function normalizeReplyPayload(
     asVoice: payload?.asVoice === true,
     audioAsVoice: payload?.audioAsVoice === true,
     ...(type ? { type } : {}),
+    ...(payload?.extra ? { extra: { ...payload.extra } } : {}),
     kind: payload?.kind,
     replyTargetPolicy: options?.replyTargetPolicy ?? 'agent-default',
     replyToId: normalizeOutboundReplyToId({

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

@@ -11,6 +11,7 @@ export type NormalizedBncrSendParams = {
   asVoice: boolean;
   audioAsVoice: boolean;
   type?: string;
+  extra?: Record<string, unknown>;
 };
 
 function isPlainObject(value: unknown): value is Record<string, unknown> {
@@ -51,6 +52,8 @@ export function normalizeBncrSendParams(input: {
   const asVoice = readOpenClawBooleanParam(paramsObj, 'asVoice') ?? false;
   const audioAsVoice = readOpenClawBooleanParam(paramsObj, 'audioAsVoice') ?? false;
   const type = readOpenClawStringParam(paramsObj, 'type') || undefined;
+  const rawExtra = paramsObj.extra;
+  const extra = isPlainObject(rawExtra) ? { ...rawExtra } : undefined;
 
   const hasMedia = Boolean(mediaUrl || dedupedMediaUrls?.length);
 
@@ -73,5 +76,6 @@ export function normalizeBncrSendParams(input: {
     asVoice,
     audioAsVoice,
     ...(type ? { type } : {}),
+    ...(extra ? { extra } : {}),
   };
 }

package/src/messaging/outbound/send.ts

@@ -64,6 +64,7 @@ export async function sendBncrMedia(params: {
   asVoice?: boolean;
   audioAsVoice?: boolean;
   type?: string;
+  extra?: Record<string, unknown>;
   kind?: string;
   replyToId?: string;
   mediaLocalRoots?: readonly string[];
@@ -83,6 +84,7 @@ export async function sendBncrMedia(params: {
       asVoice?: boolean;
       audioAsVoice?: boolean;
       type?: string;
+      extra?: Record<string, unknown>;
       kind?: 'tool' | 'block' | 'final';
       replyToId?: string;
     };
@@ -104,6 +106,7 @@ export async function sendBncrMedia(params: {
       asVoice: params.asVoice === true ? true : undefined,
       audioAsVoice: params.audioAsVoice === true ? true : undefined,
       type: params.type,
+      extra: params.extra,
       kind: normalizeReplyKind(params.kind),
       replyToId: params.replyToId,
     },

package/src/plugin/ack-outbox-runtime-group.ts

@@ -19,6 +19,7 @@ export function createBncrAckOutboxRuntimeGroup(runtime: {
   isPlainObject: (value: unknown) => value is Record<string, unknown>;
   clampFiniteNumber: (value: unknown, fallback: number, min?: number, max?: number) => number;
   normalizeAccountId: (accountId: string) => string;
+  resolveAccountIdForSession: (sessionKey: string) => string | null;
   formatDisplayScope: (route: OutboxEntry['route']) => string;
   isFileTransferEntry: (entry: OutboxEntry) => boolean;
   recommendedAckTimeoutMaxMs: number;
@@ -235,6 +236,7 @@ export function createBncrAckOutboxRuntimeGroup(runtime: {
     handleFileTransferPushFailure: runtime.handleFileTransferPushFailure,
     handleTextPushFailure: runtime.handleTextPushFailure,
     isPrePushGuardDeferral: runtime.isPrePushGuardDeferral,
+    resolveAccountIdForSession: runtime.resolveAccountIdForSession,
     moveToDeadLetter: runtime.moveToDeadLetter,
     scheduleSave: runtime.scheduleSave,
     logInfo: runtime.logInfo,

package/src/plugin/channel-plugin-surface-group.ts

@@ -152,6 +152,7 @@ export function createBncrChannelPluginSurfaceGroup(runtime: {
               asVoice: normalized.asVoice,
               audioAsVoice: normalized.audioAsVoice,
               type: normalized.type,
+              extra: normalized.extra,
               mediaLocalRoots,
               resolveVerifiedTarget: (to, accountId) =>
                 toolActionBridge.resolveVerifiedTarget(to, accountId),

package/src/plugin/channel-runtime-builders-delivery.ts

@@ -337,6 +337,7 @@ export function buildBncrMediaOrchestratorsRuntime(deps: {
     asVoice: boolean;
     audioAsVoice: boolean;
     type?: string;
+    extra?: Record<string, unknown>;
     kind?: 'tool' | 'block' | 'final';
     replyToId?: string;
     replyTargetPolicy?: OutboundReplyTargetPolicy;

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

@@ -49,6 +49,7 @@ function buildReplyMediaEntryHelpers(runtime: {
     asVoice: boolean;
     audioAsVoice: boolean;
     type?: string;
+    extra?: Record<string, unknown>;
     kind?: 'tool' | 'block' | 'final';
     replyToId?: string;
     replyTargetPolicy?: OutboundReplyTargetPolicy;
@@ -203,6 +204,7 @@ export function createBncrMediaOrchestratorsRuntimeGroup(runtime: {
     asVoice: boolean;
     audioAsVoice: boolean;
     type?: string;
+    extra?: Record<string, unknown>;
     kind?: 'tool' | 'block' | 'final';
     replyToId?: string;
     replyTargetPolicy?: OutboundReplyTargetPolicy;

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

@@ -20,6 +20,9 @@ type BncrOutboxDrainScheduleRuntime = {
 type BncrOutboxDrainFailureRuntime = {
   backoffMs: (retryCount: number) => number;
   outbox: Map<string, OutboxEntry>;
+  resolveAccountIdForSession: (sessionKey: string) => string | null;
+  logInfo: (scope: string, message: string, options?: { debugOnly?: boolean }) => void;
+  logWarn: (scope: string, message: string, options?: { debugOnly?: boolean }) => void;
   isPrePushGuardDeferral: (entry: OutboxEntry) => boolean;
   moveToDeadLetter: (entry: OutboxEntry, reason: string) => void;
   scheduleSave: () => void;
@@ -39,6 +42,39 @@ export function createBncrOutboxDrainFailure(runtime: BncrOutboxDrainFailureRunt
     const { accountId, entry, attemptedAt, updateMinOutboxDelay } = args;
     let { localNextDelay } = args;
 
+    // If the entry keeps hitting pre-push guard (no active connection), the
+    // accountId on the entry might be wrong (e.g. constructed from route data).
+    // Try to correct it from recent inbound session context before giving up.
+    if (runtime.isPrePushGuardDeferral(entry) && entry.retryCount > 0) {
+      const corrected = runtime.resolveAccountIdForSession(entry.sessionKey);
+      if (corrected && corrected !== entry.accountId) {
+        const oldAccountId = entry.accountId;
+        runtime.logWarn(
+          'outbound',
+          `account corrected sessionKey=${entry.sessionKey} ${oldAccountId}→${corrected}`,
+        );
+        runtime.logInfo(
+          'outbound',
+          JSON.stringify({
+            event: 'account-corrected',
+            messageId: entry.messageId,
+            sessionKey: entry.sessionKey,
+            oldAccountId,
+            corrected,
+            retryCount: entry.retryCount,
+            lastError: entry.lastError,
+          }),
+          { debugOnly: true },
+        );
+        entry.accountId = corrected;
+        entry.retryCount = 0;
+        entry.lastError = undefined;
+        runtime.outbox.set(entry.messageId, entry);
+        runtime.scheduleSave();
+        return { action: 'continue', localNextDelay };
+      }
+    }
+
     if (runtime.isPrePushGuardDeferral(entry)) {
       const wait = runtime.prePushGuardRetryDelayMs;
       localNextDelay = runtime.outboxDrainSchedule.scheduleAccountWait({

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

@@ -58,6 +58,7 @@ type BncrOutboxDrainRuntime = {
   backoffMs: (retryCount: number) => number;
   isPlainObject: (value: unknown) => value is Record<string, unknown>;
   normalizeAccountId: (accountId: string) => string;
+  resolveAccountIdForSession: (sessionKey: string) => string | null;
   stopped: () => boolean;
   outbox: Map<string, OutboxEntry>;
   deadLetter: () => OutboxEntry[];
@@ -132,7 +133,19 @@ type BncrOutboxDrainRuntime = {
 
 export function createBncrOutboxDrainRuntime(runtime: BncrOutboxDrainRuntime) {
   const handlePushedDrainEntry = createBncrOutboxDrainPostPush(runtime);
-  const handleFailedDrainEntry = createBncrOutboxDrainFailure(runtime);
+  const handleFailedDrainEntry = createBncrOutboxDrainFailure({
+    backoffMs: runtime.backoffMs,
+    outbox: runtime.outbox,
+    resolveAccountIdForSession: runtime.resolveAccountIdForSession,
+    logInfo: runtime.logInfo,
+    logWarn: runtime.logWarn,
+    isPrePushGuardDeferral: runtime.isPrePushGuardDeferral,
+    moveToDeadLetter: runtime.moveToDeadLetter,
+    scheduleSave: runtime.scheduleSave,
+    outboxDrainSchedule: runtime.outboxDrainSchedule,
+    maxRetry: runtime.maxRetry,
+    prePushGuardRetryDelayMs: runtime.prePushGuardRetryDelayMs,
+  });
 
   return createBncrOutboxDrainLoop(runtime, {
     handlePushedDrainEntry,

package/src/plugin/target-runtime.ts

@@ -133,9 +133,17 @@ export function createBncrTargetRuntime(runtime: {
     return verified;
   }
 
+  function resolveSessionAccountId(sessionKey: string): string | null {
+    const key = asString(sessionKey).trim();
+    if (!key) return null;
+    const hit = runtime.sessionRoutes.get(key);
+    return hit ? hit.accountId : null;
+  }
+
   return {
     rememberSessionRoute,
     resolveRouteBySession,
     resolveVerifiedTarget,
+    resolveSessionAccountId,
   };
 }