@xmoxmo/bncr 0.1.1 → 0.1.3

package/README.md

@@ -187,6 +187,18 @@ npm pack
 - `npm test`：跑回归测试
 - `npm run selfcheck`：检查插件骨架是否完整
 - `npm pack`：确认当前版本可正常打包
+- `npm run check-register-drift -- --duration-sec 300 --interval-sec 15`：静置采样 `bncr.diagnostics`，观察 `registerCount / apiGeneration / postWarmupRegisterCount` 是否在 warmup 后继续增长
+
+示例输出重点：
+
+- `delta.registerCount`
+- `delta.apiGeneration`
+- `delta.postWarmupRegisterCount`
+- `historicalWarmupExternalDrift`
+- `newDriftDuringWindow`
+- `last.postWarmupRegisterCount`
+- `last.unexpectedRegisterAfterWarmup`
+- `driftDetected`
 
 ---
 

package/index.ts

@@ -1,7 +1,7 @@
-import fs from 'node:fs';
-import path from 'node:path';
 import { execFileSync } from 'node:child_process';
+import fs from 'node:fs';
 import { createRequire } from 'node:module';
+import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { BncrConfigSchema } from './src/core/config-schema.ts';
 
@@ -20,7 +20,23 @@ type LoadedRuntime = {
   createBncrChannelPlugin: ChannelModule['createBncrChannelPlugin'];
 };
 
+const BNCR_REGISTER_META = Symbol.for('bncr.register.meta');
+
+type RegisterMeta = {
+  service?: boolean;
+  channel?: boolean;
+  methods?: Set<string>;
+  apiInstanceId?: string;
+  registryFingerprint?: string;
+};
+
+type OpenClawPluginApiWithMeta = OpenClawPluginApi & {
+  [BNCR_REGISTER_META]?: RegisterMeta;
+};
+
 let runtime: LoadedRuntime | null = null;
+const identityIds = new WeakMap<object, string>();
+let identitySeq = 0;
 
 const tryExec = (command: string, args: string[]) => {
   try {
@@ -105,9 +121,7 @@ const collectOpenClawCandidates = () => {
   }
 
   const packageRoots = unique(
-    directCandidates
-      .map((candidate) => findOpenClawPackageRoot(candidate))
-      .filter(Boolean),
+    directCandidates.map((candidate) => findOpenClawPackageRoot(candidate)).filter(Boolean),
   );
 
   return packageRoots.filter((candidate) => {
@@ -188,11 +202,65 @@ const loadRuntimeSync = (): LoadedRuntime => {
   }
 };
 
+const getIdentityId = (obj: object, prefix: string) => {
+  const existing = identityIds.get(obj);
+  if (existing) return existing;
+  const next = `${prefix}_${++identitySeq}`;
+  identityIds.set(obj, next);
+  return next;
+};
+
+const getRegistryFingerprint = (api: OpenClawPluginApi) => {
+  const serviceId = getIdentityId(api.registerService as object, 'svc');
+  const channelId = getIdentityId(api.registerChannel as object, 'chn');
+  const methodId = getIdentityId(api.registerGatewayMethod as object, 'mth');
+  return `${serviceId}:${channelId}:${methodId}`;
+};
+
+const getRegisterMeta = (api: OpenClawPluginApi): RegisterMeta => {
+  const host = api as OpenClawPluginApiWithMeta;
+  if (!host[BNCR_REGISTER_META]) {
+    host[BNCR_REGISTER_META] = { methods: new Set<string>() };
+  }
+  if (!host[BNCR_REGISTER_META]!.methods) {
+    host[BNCR_REGISTER_META]!.methods = new Set<string>();
+  }
+  if (!host[BNCR_REGISTER_META]!.apiInstanceId) {
+    host[BNCR_REGISTER_META]!.apiInstanceId = getIdentityId(api as object, 'api');
+  }
+  if (!host[BNCR_REGISTER_META]!.registryFingerprint) {
+    host[BNCR_REGISTER_META]!.registryFingerprint = getRegistryFingerprint(api);
+  }
+  return host[BNCR_REGISTER_META]!;
+};
+
+const ensureGatewayMethodRegistered = (
+  api: OpenClawPluginApi,
+  name: string,
+  handler: (opts: any) => any,
+  debugLog: (...args: any[]) => void,
+) => {
+  const meta = getRegisterMeta(api);
+  if (meta.methods?.has(name)) {
+    debugLog(`bncr register method skip ${name} (already registered on this api)`);
+    return;
+  }
+  api.registerGatewayMethod(name, handler);
+  meta.methods?.add(name);
+  debugLog(`bncr register method ok ${name}`);
+};
+
 const getBridgeSingleton = (api: OpenClawPluginApi) => {
   const loaded = loadRuntimeSync();
   const g = globalThis as typeof globalThis & { __bncrBridge?: BridgeSingleton };
-  if (!g.__bncrBridge) g.__bncrBridge = loaded.createBncrBridge(api);
-  return { bridge: g.__bncrBridge, runtime: loaded };
+  let created = false;
+  if (!g.__bncrBridge) {
+    g.__bncrBridge = loaded.createBncrBridge(api);
+    created = true;
+  } else {
+    g.__bncrBridge.bindApi?.(api);
+  }
+  return { bridge: g.__bncrBridge, runtime: loaded, created };
 };
 
 const plugin = {
@@ -201,13 +269,24 @@ const plugin = {
   description: 'Bncr channel plugin',
   configSchema: BncrConfigSchema,
   register(api: OpenClawPluginApi) {
-    const { bridge, runtime } = getBridgeSingleton(api);
+    const meta = getRegisterMeta(api);
+    const { bridge, runtime, created } = getBridgeSingleton(api);
+    bridge.noteRegister?.({
+      source: '~/.openclaw/workspace/plugins/bncr/index.ts',
+      pluginVersion: '0.1.1',
+      apiRebound: !created,
+      apiInstanceId: meta.apiInstanceId,
+      registryFingerprint: meta.registryFingerprint,
+    });
     const debugLog = (...args: any[]) => {
       if (!bridge.isDebugEnabled?.()) return;
       api.logger.info?.(...args);
     };
 
-    debugLog(`bncr plugin register bridge=${(bridge as any)?.bridgeId || 'unknown'}`);
+    debugLog(
+      `bncr plugin register begin bridge=${bridge.getBridgeId?.() || 'unknown'} created=${created}`,
+    );
+    if (!created) debugLog('bncr bridge api rebound');
 
     const resolveDebug = async () => {
       try {
@@ -218,27 +297,85 @@ const plugin = {
       }
     };
 
-    api.registerService({
-      id: 'bncr-bridge-service',
-      start: async (ctx) => {
-        const debug = await resolveDebug();
-        await bridge.startService(ctx, debug);
-      },
-      stop: bridge.stopService,
-    });
+    if (!meta.service) {
+      api.registerService({
+        id: 'bncr-bridge-service',
+        start: async (ctx) => {
+          const debug = await resolveDebug();
+          await bridge.startService(ctx, debug);
+        },
+        stop: bridge.stopService,
+      });
+      meta.service = true;
+      debugLog('bncr register service ok');
+    } else {
+      debugLog('bncr register service skip (already registered on this api)');
+    }
+
+    if (!meta.channel) {
+      api.registerChannel({ plugin: runtime.createBncrChannelPlugin(bridge) });
+      meta.channel = true;
+      debugLog('bncr register channel ok');
+    } else {
+      debugLog('bncr register channel skip (already registered on this api)');
+    }
 
-    api.registerChannel({ plugin: runtime.createBncrChannelPlugin(bridge) });
-
-    api.registerGatewayMethod('bncr.connect', (opts) => bridge.handleConnect(opts));
-    api.registerGatewayMethod('bncr.inbound', (opts) => bridge.handleInbound(opts));
-    api.registerGatewayMethod('bncr.activity', (opts) => bridge.handleActivity(opts));
-    api.registerGatewayMethod('bncr.ack', (opts) => bridge.handleAck(opts));
-    api.registerGatewayMethod('bncr.diagnostics', (opts) => bridge.handleDiagnostics(opts));
-    api.registerGatewayMethod('bncr.file.init', (opts) => bridge.handleFileInit(opts));
-    api.registerGatewayMethod('bncr.file.chunk', (opts) => bridge.handleFileChunk(opts));
-    api.registerGatewayMethod('bncr.file.complete', (opts) => bridge.handleFileComplete(opts));
-    api.registerGatewayMethod('bncr.file.abort', (opts) => bridge.handleFileAbort(opts));
-    api.registerGatewayMethod('bncr.file.ack', (opts) => bridge.handleFileAck(opts));
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.connect',
+      (opts) => bridge.handleConnect(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.inbound',
+      (opts) => bridge.handleInbound(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.activity',
+      (opts) => bridge.handleActivity(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(api, 'bncr.ack', (opts) => bridge.handleAck(opts), debugLog);
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.diagnostics',
+      (opts) => bridge.handleDiagnostics(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.file.init',
+      (opts) => bridge.handleFileInit(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.file.chunk',
+      (opts) => bridge.handleFileChunk(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.file.complete',
+      (opts) => bridge.handleFileComplete(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.file.abort',
+      (opts) => bridge.handleFileAbort(opts),
+      debugLog,
+    );
+    ensureGatewayMethodRegistered(
+      api,
+      'bncr.file.ack',
+      (opts) => bridge.handleFileAck(opts),
+      debugLog,
+    );
+    debugLog('bncr plugin register done');
   },
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xmoxmo/bncr",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "private": false,
   "type": "module",
   "license": "MIT",
@@ -25,12 +25,18 @@
   ],
   "scripts": {
     "selfcheck": "node ./scripts/selfcheck.mjs",
-    "test": "node --import ./tests/register-ts-hooks.mjs --test ./tests/*.test.mjs"
+    "test": "node --import ./tests/register-ts-hooks.mjs --test ./tests/*.test.mjs",
+    "check-register-drift": "node ./scripts/check-register-drift.mjs",
+    "format:check": "biome format --check .",
+    "format": "biome format --write .",
+    "lint": "biome lint .",
+    "check": "biome check ."
   },
   "peerDependencies": {
     "openclaw": ">=2026.3.22"
   },
   "devDependencies": {
+    "@biomejs/biome": "^1.9.4",
     "openclaw": ">=2026.3.22"
   },
   "openclaw": {

package/scripts/check-register-drift.mjs

@@ -0,0 +1,117 @@
+import { execFileSync } from 'node:child_process';
+
+const readNumber = (value, fallback) => {
+  const n = Number(value);
+  return Number.isFinite(n) ? n : fallback;
+};
+
+const args = process.argv.slice(2);
+const options = {
+  durationSec: 300,
+  intervalSec: 15,
+  accountId: 'Primary',
+  gatewayBin: 'openclaw',
+};
+
+for (let i = 0; i < args.length; i += 1) {
+  const arg = args[i];
+  if (arg === '--duration-sec') options.durationSec = readNumber(args[++i], options.durationSec);
+  else if (arg === '--interval-sec')
+    options.intervalSec = readNumber(args[++i], options.intervalSec);
+  else if (arg === '--account-id') options.accountId = args[++i] || options.accountId;
+  else if (arg === '--gateway-bin') options.gatewayBin = args[++i] || options.gatewayBin;
+  else if (arg === '--help' || arg === '-h') {
+    console.log(
+      'Usage: node ./scripts/check-register-drift.mjs [--duration-sec 300] [--interval-sec 15] [--account-id Primary] [--gateway-bin openclaw]\n\nSamples bncr.diagnostics over time and reports whether register counters drift after warmup.',
+    );
+    process.exit(0);
+  }
+}
+
+if (options.durationSec <= 0) throw new Error('durationSec must be > 0');
+if (options.intervalSec <= 0) throw new Error('intervalSec must be > 0');
+
+const sleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+
+const fetchDiagnostics = () => {
+  const raw = execFileSync(
+    options.gatewayBin,
+    [
+      'gateway',
+      'call',
+      'bncr.diagnostics',
+      '--json',
+      '--params',
+      JSON.stringify({ accountId: options.accountId }),
+    ],
+    { encoding: 'utf8' },
+  );
+  const parsed = JSON.parse(raw);
+  const reg = parsed?.diagnostics?.register || {};
+  const summary = reg?.traceSummary || {};
+  return {
+    now: parsed?.now ?? Date.now(),
+    registerCount: reg?.registerCount ?? null,
+    apiGeneration: reg?.apiGeneration ?? null,
+    apiInstanceId: reg?.apiInstanceId ?? null,
+    registryFingerprint: reg?.registryFingerprint ?? null,
+    warmupRegisterCount: summary?.warmupRegisterCount ?? null,
+    postWarmupRegisterCount: summary?.postWarmupRegisterCount ?? null,
+    unexpectedRegisterAfterWarmup: summary?.unexpectedRegisterAfterWarmup ?? null,
+    lastUnexpectedRegisterAt: summary?.lastUnexpectedRegisterAt ?? null,
+    sourceBuckets: summary?.sourceBuckets ?? null,
+  };
+};
+
+const startedAt = Date.now();
+const samples = [];
+const deadline = startedAt + options.durationSec * 1000;
+
+while (true) {
+  const sample = fetchDiagnostics();
+  samples.push(sample);
+  const nextAt = Date.now() + options.intervalSec * 1000;
+  if (nextAt > deadline) break;
+  await sleep(Math.max(0, nextAt - Date.now()));
+}
+
+const first = samples[0] || {};
+const last = samples[samples.length - 1] || {};
+const deltaRegisterCount = (last.registerCount ?? 0) - (first.registerCount ?? 0);
+const deltaApiGeneration = (last.apiGeneration ?? 0) - (first.apiGeneration ?? 0);
+const deltaPostWarmupRegisterCount =
+  (last.postWarmupRegisterCount ?? 0) - (first.postWarmupRegisterCount ?? 0);
+const historicalWarmupExternalDrift = Boolean(first.unexpectedRegisterAfterWarmup);
+const newWarmupExternalDriftDuringWindow = deltaPostWarmupRegisterCount > 0;
+const newDriftDuringWindow =
+  deltaRegisterCount > 0 || deltaApiGeneration > 0 || newWarmupExternalDriftDuringWindow;
+const driftDetected = historicalWarmupExternalDrift || newDriftDuringWindow;
+
+const result = {
+  ok: true,
+  accountId: options.accountId,
+  durationSec: options.durationSec,
+  intervalSec: options.intervalSec,
+  startedAt,
+  endedAt: Date.now(),
+  sampleCount: samples.length,
+  first,
+  last,
+  delta: {
+    registerCount: deltaRegisterCount,
+    apiGeneration: deltaApiGeneration,
+    postWarmupRegisterCount: deltaPostWarmupRegisterCount,
+  },
+  historicalWarmupExternalDrift,
+  newWarmupExternalDriftDuringWindow,
+  newDriftDuringWindow,
+  driftDetected,
+  conclusion: newDriftDuringWindow
+    ? 'new register drift was observed during this sampling window'
+    : historicalWarmupExternalDrift
+      ? 'no new drift during this window, but warmup-external drift had already happened before sampling began'
+      : 'register counters stayed stable during this window and no warmup-external drift was flagged',
+  samples,
+};
+
+console.log(JSON.stringify(result, null, 2));