openclaw-memory-alibaba-mysql 0.2.1 → 0.2.2

package/config.ts

@@ -114,43 +114,66 @@ function assertAllowedKeys(value: Record<string, unknown>, allowed: string[], la
   }
 }
 
-function requireString(obj: Record<string, unknown>, key: string, label: string): string {
+function requireString(obj: Record<string, unknown>, key: string, label: string, allowEmpty = false): string {
   const v = obj[key];
-  if (typeof v !== "string" || v.length === 0) {
+  if (typeof v !== "string") {
+    throw new Error(`${label}.${key} is required and must be a string`);
+  }
+  if (!allowEmpty && 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)) {
-    throw new Error("mysql config is required");
+    return {
+      host: "",
+      port: 3306,
+      user: "",
+      password: "",
+      database: "openclaw_memory",
+      ssl: false,
+    };
   }
   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: requireString(m, "host", "mysql"),
+    host,
     port: typeof m.port === "number" ? m.port : 3306,
-    user: requireString(m, "user", "mysql"),
-    password: resolveEnvVars(requireString(m, "password", "mysql")),
-    database: requireString(m, "database", "mysql"),
+    user,
+    password,
+    database,
     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)) {
-    throw new Error("embedding config is required");
+    return {
+      apiKey: "",
+      model: DEFAULT_MODEL,
+      baseUrl: DEFAULT_BASE_URL,
+      dimensions: undefined,
+    };
   }
   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: resolveEnvVars(requireString(e, "apiKey", "embedding")),
+    apiKey,
     model,
     baseUrl: typeof e.baseUrl === "string" ? resolveEnvVars(e.baseUrl) : DEFAULT_BASE_URL,
     dimensions: explicitDims,

package/db.ts

@@ -51,6 +51,20 @@ 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,

package/index.ts

@@ -81,6 +81,10 @@ 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/package.json

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