openclaw-memory-alibaba-mysql 0.2.2 → 0.2.4

package/config.ts

@@ -59,7 +59,7 @@ export type { MemoryCategory };
 const DEFAULT_MODEL = "text-embedding-v3";
 const DEFAULT_BASE_URL = "https://dashscope.aliyuncs.com/compatible-mode/v1";
 const DEFAULT_TABLE_NAME = "openclaw_memories";
-export const DEFAULT_CAPTURE_MAX_CHARS = 500;
+export const DEFAULT_CAPTURE_MAX_CHARS = 50000;
 
 const EMBEDDING_DIMENSIONS: Record<string, number> = {
   "text-embedding-v3": 1024,
@@ -114,66 +114,43 @@ function assertAllowedKeys(value: Record<string, unknown>, allowed: string[], la
   }
 }
 
-function requireString(obj: Record<string, unknown>, key: string, label: string, allowEmpty = false): string {
+function requireString(obj: Record<string, unknown>, key: string, label: string): string {
   const v = obj[key];
-  if (typeof v !== "string") {
-    throw new Error(`${label}.${key} is required and must be a string`);
-  }
-  if (!allowEmpty && v.length === 0) {
+  if (typeof v !== "string" || v.length === 0) {
     throw new Error(`${label}.${key} is required and must be a non-empty string`);
   }
   return v;
 }
 
 function parseMysqlConfig(raw: unknown): MysqlConnectionConfig {
-  // Allow empty mysql config - will be validated at connection time
   if (!raw || typeof raw !== "object" || Array.isArray(raw)) {
-    return {
-      host: "",
-      port: 3306,
-      user: "",
-      password: "",
-      database: "openclaw_memory",
-      ssl: false,
-    };
+    throw new Error("mysql config is required");
   }
   const m = raw as Record<string, unknown>;
   assertAllowedKeys(m, ["host", "port", "user", "password", "database", "ssl"], "mysql");
 
-  const host = typeof m.host === "string" ? m.host : "";
-  const user = typeof m.user === "string" ? m.user : "";
-  const password = typeof m.password === "string" ? resolveEnvVars(m.password) : "";
-  const database = typeof m.database === "string" ? m.database : "openclaw_memory";
-
   return {
-    host,
+    host: requireString(m, "host", "mysql"),
     port: typeof m.port === "number" ? m.port : 3306,
-    user,
-    password,
-    database,
+    user: requireString(m, "user", "mysql"),
+    password: resolveEnvVars(requireString(m, "password", "mysql")),
+    database: requireString(m, "database", "mysql"),
     ssl: m.ssl === true,
   };
 }
 
 function parseEmbeddingConfig(raw: unknown): EmbeddingConfig {
-  // Allow empty embedding config - will be validated at connection time
   if (!raw || typeof raw !== "object" || Array.isArray(raw)) {
-    return {
-      apiKey: "",
-      model: DEFAULT_MODEL,
-      baseUrl: DEFAULT_BASE_URL,
-      dimensions: undefined,
-    };
+    throw new Error("embedding config is required");
   }
   const e = raw as Record<string, unknown>;
   assertAllowedKeys(e, ["apiKey", "model", "baseUrl", "dimensions"], "embedding");
 
   const model = typeof e.model === "string" ? e.model : DEFAULT_MODEL;
   const explicitDims = typeof e.dimensions === "number" ? e.dimensions : undefined;
-  const apiKey = typeof e.apiKey === "string" ? resolveEnvVars(e.apiKey) : "";
 
   return {
-    apiKey,
+    apiKey: resolveEnvVars(requireString(e, "apiKey", "embedding")),
     model,
     baseUrl: typeof e.baseUrl === "string" ? resolveEnvVars(e.baseUrl) : DEFAULT_BASE_URL,
     dimensions: explicitDims,
@@ -197,8 +174,9 @@ function parseLLMConfig(raw: unknown): LLMConfig {
 
 export const memoryConfigSchema = {
   parse(value: unknown): MemoryConfig {
+    // Allow empty config during plugin registration
     if (!value || typeof value !== "object" || Array.isArray(value)) {
-      throw new Error("memory-alibaba-mysql: plugin config is required");
+      value = {};
     }
     const cfg = value as Record<string, unknown>;
 
@@ -264,8 +242,8 @@ export const memoryConfigSchema = {
 
     const captureMaxChars =
       typeof cfg.captureMaxChars === "number" ? Math.floor(cfg.captureMaxChars) : undefined;
-    if (typeof captureMaxChars === "number" && (captureMaxChars < 100 || captureMaxChars > 10_000)) {
-      throw new Error("captureMaxChars must be between 100 and 10000");
+    if (typeof captureMaxChars === "number" && (captureMaxChars < 100 || captureMaxChars > 100_000)) {
+      throw new Error("captureMaxChars must be between 100 and 100000");
     }
 
     // --- Memory decay: only enableMemoryDecay is configurable; half-life and strategy use defaults ---

package/db.ts

@@ -51,20 +51,6 @@ export class MemoryDB {
   }
 
   private async doInitialize(): Promise<void> {
-    // Validate MySQL config before connecting
-    if (!this.mysqlConfig.host || this.mysqlConfig.host.length === 0) {
-      throw new Error("MySQL host is not configured. Please set mysql.host in the plugin config.");
-    }
-    if (!this.mysqlConfig.user || this.mysqlConfig.user.length === 0) {
-      throw new Error("MySQL user is not configured. Please set mysql.user in the plugin config.");
-    }
-    if (!this.mysqlConfig.password || this.mysqlConfig.password.length === 0) {
-      throw new Error("MySQL password is not configured. Please set mysql.password in the plugin config.");
-    }
-    if (!this.mysqlConfig.database || this.mysqlConfig.database.length === 0) {
-      throw new Error("MySQL database is not configured. Please set mysql.database in the plugin config.");
-    }
-
     this.pool = mysql.createPool({
       host: this.mysqlConfig.host,
       port: this.mysqlConfig.port,
@@ -86,7 +72,7 @@ export class MemoryDB {
           agent_id   VARCHAR(128)  NOT NULL,
           user_id    VARCHAR(128)  NULL DEFAULT NULL,
           session_id VARCHAR(128)  NULL DEFAULT NULL,
-          text       TEXT          NOT NULL,
+          text       LONGTEXT      NOT NULL,
           embedding  VECTOR(${this.vectorDim}) NOT NULL,
           importance FLOAT         DEFAULT 0,
           category   VARCHAR(64)   DEFAULT '${USER_MEMORY_FACT}',

package/index.ts

@@ -81,10 +81,6 @@ class Embeddings {
   }
 
   async embed(text: string): Promise<number[]> {
-    // Validate API key before making request
-    if (!this.client.apiKey || this.client.apiKey.length === 0) {
-      throw new Error("Embedding API key is not configured. Please set embedding.apiKey in the plugin config.");
-    }
     const params: { model: string; input: string; dimensions?: number } = {
       model: this.model,
       input: text,

package/openclaw.plugin.json

@@ -66,7 +66,7 @@
       },
       "autoRecall": { "type": "boolean", "default": true },
       "autoCapture": { "type": "boolean", "default": true },
-      "captureMaxChars": { "type": "number", "default": 500 },
+      "captureMaxChars": { "type": "number", "default": 50000 },
       "enableMemoryDecay": {
         "type": "boolean",
         "default": false,
@@ -176,7 +176,7 @@
     "captureMaxChars": {
       "label": "Capture Max Chars",
       "placeholder": "500",
-      "help": "Max length per captured memory (100–10000)"
+      "help": "Max length per captured memory (100–100000)"
     },
     "enableMemoryDecay": {
       "label": "Enable Memory Decay",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openclaw-memory-alibaba-mysql",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "description": "OpenClaw memory plugin using Alibaba Cloud RDS MySQL vector storage",
   "type": "module",
   "license": "MIT",