openclaw-memory-alibaba-mysql 0.2.2 → 0.2.3

package/config.ts

@@ -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>;
 

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,

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/package.json

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