@lark-apaas/openclaw-scripts-diagnose-cli 0.1.5-alpha.8 → 0.1.5

package/dist/index.cjs

@@ -53,7 +53,7 @@ let json_diff = require("json-diff");
 * it terse and parseable.
 */
 function getVersion() {
-	return "0.1.5-alpha.8";
+	return "0.1.5";
 }
 //#endregion
 //#region src/rule-engine/base.ts
@@ -781,10 +781,12 @@ FeishuChannelRule = __decorate([Rule({
 //#endregion
 //#region src/rules/feishu-default-account.ts
 /**
-* Owner of the multi-agent feishu channel shape: migrates legacy v1/v2
+* Owns the multi-agent feishu-channel migration: turns legacy v1/v2
 * (top-level appId + defaultAccount/default) into v3 (`bot-<appId>` account),
-* detects + fixes drift on the main bot's appId/appSecret. Single-agent
-* configs (no `accounts`) are out of scope — handled by `feishu_channel`.
+* preserving the user's top-level `appId` / `appSecret` verbatim. Once on
+* v3 this rule does not touch account values — the user owns them.
+* Single-agent configs (no `accounts`) are out of scope — handled by
+* `feishu_channel`.
 */
 /** Top-level `channels.feishu.*` account-policy fields migrated into the main bot. */
 const TOP_FIELDS_TO_MIGRATE = [
@@ -797,25 +799,11 @@ let FeishuDefaultAccountRule = class FeishuDefaultAccountRule extends DiagnoseRu
 	validate(ctx) {
 		const feishu = getNestedMap(ctx.config, "channels", "feishu");
 		if (!feishu) return { pass: true };
-		const accounts = asRecord(feishu.accounts);
-		if (!accounts) return { pass: true };
-		const topAppId = feishu.appId;
-		if (typeof topAppId === "string" && topAppId !== "") return {
+		if (!asRecord(feishu.accounts)) return { pass: true };
+		if (hasLegacyTopLevel(feishu)) return {
 			pass: false,
 			message: "channels.feishu has legacy shape; needs migration to accounts.bot-<appId>"
 		};
-		const mainBot = findMainBotAccount(ctx.config, accounts);
-		if (mainBot) {
-			const expectedAppId = ctx.vars?.feishuAppID;
-			if (typeof expectedAppId === "string" && expectedAppId !== "" && mainBot.acc.appId !== expectedAppId) return {
-				pass: false,
-				message: `accounts.${mainBot.accountId}.appId mismatch: got ${mainBot.acc.appId}, expected ${expectedAppId}`
-			};
-			if (!secretMatchesCanonical(mainBot.acc.appSecret)) return {
-				pass: false,
-				message: `accounts.${mainBot.accountId}.appSecret drift`
-			};
-		}
 		return { pass: true };
 	}
 	repair(ctx) {
@@ -823,16 +811,10 @@ let FeishuDefaultAccountRule = class FeishuDefaultAccountRule extends DiagnoseRu
 		if (!feishu) return;
 		const accounts = asRecord(feishu.accounts);
 		if (!accounts) return;
-		const topAppId = feishu.appId;
-		if (typeof topAppId === "string" && topAppId !== "") {
-			this.migrate(ctx, feishu, accounts, topAppId);
-			return;
-		}
-		this.enforceMainBotValues(ctx, accounts);
+		if (hasLegacyTopLevel(feishu)) this.migrate(ctx, feishu, accounts, feishu.appId, feishu.appSecret);
 	}
-	migrate(ctx, feishu, accounts, topAppId) {
-		const effectiveAppId = nonEmpty(ctx.vars?.feishuAppID) ?? topAppId;
-		const expectedKey = `bot-${effectiveAppId}`;
+	migrate(ctx, feishu, accounts, topAppId, topAppSecret) {
+		const expectedKey = `bot-${topAppId}`;
 		const existingBot = asRecord(accounts[expectedKey]) ?? {};
 		const defaultAccount = asRecord(accounts.defaultAccount) ?? {};
 		const defaultAcc = asRecord(accounts.default) ?? {};
@@ -849,8 +831,8 @@ let FeishuDefaultAccountRule = class FeishuDefaultAccountRule extends DiagnoseRu
 			...topGroups
 		};
 		if (Object.keys(fusedGroups).length > 0) merged.groups = fusedGroups;
-		merged.appId = effectiveAppId;
-		merged.appSecret = DEFAULT_FEISHU_APP_SECRET;
+		merged.appId = topAppId;
+		merged.appSecret = topAppSecret;
 		const chatID = ctx.vars?.teamChatID;
 		if (typeof chatID === "string" && chatID !== "") {
 			const existingGroups = asRecord(merged.groups) ?? {};
@@ -867,14 +849,6 @@ let FeishuDefaultAccountRule = class FeishuDefaultAccountRule extends DiagnoseRu
 		for (const k of TOP_FIELDS_TO_MIGRATE) delete feishu[k];
 		this.rewireBindings(ctx.config, expectedKey);
 	}
-	enforceMainBotValues(ctx, accounts) {
-		const mainBot = findMainBotAccount(ctx.config, accounts);
-		if (!mainBot) return;
-		const acc = accounts[mainBot.accountId];
-		const expectedAppId = nonEmpty(ctx.vars?.feishuAppID);
-		if (expectedAppId !== void 0 && acc.appId !== expectedAppId) acc.appId = expectedAppId;
-		if (!secretMatchesCanonical(acc.appSecret)) acc.appSecret = DEFAULT_FEISHU_APP_SECRET;
-	}
 	rewireBindings(config, expectedKey) {
 		if (!Array.isArray(config.bindings)) return;
 		const bindings = config.bindings;
@@ -920,32 +894,18 @@ FeishuDefaultAccountRule = __decorate([Rule({
 	repairMode: "standard",
 	usesVars: ["feishuAppID", "teamChatID"]
 })], FeishuDefaultAccountRule);
-function nonEmpty(v) {
-	return typeof v === "string" && v !== "" ? v : void 0;
-}
-function findMainBotAccount(config, accounts) {
-	const mainId = findMainAgent(config)?.id;
-	if (typeof mainId !== "string" || mainId === "") return void 0;
-	const bindings = Array.isArray(config.bindings) ? config.bindings : [];
-	for (const b of bindings) {
-		const rec = asRecord(b);
-		const match = asRecord(rec?.match);
-		if (rec && match && rec.agentId === mainId && match.channel === "feishu") {
-			const accountId = match.accountId;
-			if (typeof accountId === "string") {
-				const acc = asRecord(accounts[accountId]);
-				if (acc) return {
-					accountId,
-					acc
-				};
-			}
-		}
-	}
-}
-/** Bot accounts must carry the canonical provider-ref `appSecret`. */
-function secretMatchesCanonical(secret) {
-	if (typeof secret !== "object" || secret === null || Array.isArray(secret)) return false;
-	return matchMap(secret, DEFAULT_FEISHU_APP_SECRET);
+/**
+* Legacy top-level shape only counts as "needs migration" when BOTH appId and
+* appSecret are present. Either one alone is treated as a partial config the
+* user owns; doctor leaves it untouched and won't synthesize the missing half
+* from defaults. `appId` must be a non-empty string (used as bot key);
+* `appSecret` may be a string OR a provider-ref object, so we only require
+* `!== undefined`.
+*/
+function hasLegacyTopLevel(feishu) {
+	const appId = feishu.appId;
+	if (typeof appId !== "string" || appId === "") return false;
+	return feishu.appSecret !== void 0;
 }
 //#endregion
 //#region src/rules/gateway.ts
@@ -3012,6 +2972,7 @@ async function installExtension(tag, ossFileMap, opts = {}) {
 	else console.error(`[install-extension] skipConfigUpdate=true — not touching openclaw.json`);
 	console.error(`[install-extension] done ${targets.length}/${targets.length} in ${Date.now() - t0}ms`);
 }
+const MEM0_PLUGIN_NAME = "openclaw-mem0-plugin";
 /**
 * Merge each installed extension's installMetadata into openclaw.json's
 * plugins.installs[<pkg.name>]. Atomic write via tmp + rename.
@@ -3019,6 +2980,11 @@ async function installExtension(tag, ossFileMap, opts = {}) {
 * - No openclaw.json → log + return (not an error; some install contexts don't have it yet)
 * - Extension without installMetadata in manifest → skip that entry (log)
 * - Existing plugins.installs entries for other extensions left untouched
+*
+* Special-case for mem0: when openclaw-mem0-plugin is among the installed
+* targets, also append it to plugins.allow (idempotent) and seed
+* plugins.entries.openclaw-mem0-plugin = { enabled: false } when the key
+* is absent. Existing values are preserved (user toggles survive).
 */
 function updatePluginInstalls(configPath, installedPkgs) {
 	if (!node_fs.default.existsSync(configPath)) {
@@ -3038,6 +3004,15 @@ function updatePluginInstalls(configPath, installedPkgs) {
 		installs[pkg.name] = pkg.installMetadata;
 		updated++;
 	} else skipped++;
+	if (installedPkgs.some((p) => p.name === MEM0_PLUGIN_NAME)) {
+		const allowRaw = plugins.allow;
+		const allow = Array.isArray(allowRaw) ? allowRaw : [];
+		if (!allow.filter((e) => typeof e === "string").includes(MEM0_PLUGIN_NAME)) allow.push(MEM0_PLUGIN_NAME);
+		plugins.allow = allow;
+		if (!plugins.entries || typeof plugins.entries !== "object" || Array.isArray(plugins.entries)) plugins.entries = {};
+		const entries = plugins.entries;
+		if (!(MEM0_PLUGIN_NAME in entries)) entries[MEM0_PLUGIN_NAME] = { enabled: false };
+	}
 	const tmpPath = configPath + ".installs-tmp";
 	node_fs.default.writeFileSync(tmpPath, JSON.stringify(config, null, 2), "utf-8");
 	moveSafe(tmpPath, configPath);
@@ -4314,7 +4289,7 @@ async function reportCliRun(opts) {
 //#region src/help.ts
 const BIN = "mclaw-diagnose";
 function versionBanner() {
-	return `v0.1.5-alpha.8`;
+	return `v0.1.5`;
 }
 const COMMANDS = [
 	{

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lark-apaas/openclaw-scripts-diagnose-cli",
-  "version": "0.1.5-alpha.8",
+  "version": "0.1.5",
   "description": "CLI for OpenClaw config diagnose and repair with JSON5 support",
   "main": "dist/index.cjs",
   "bin": {