@workclaw/openclaw-workclaw 1.0.17 → 1.0.18

package/skills/openclaw-workclaw-cron/SKILL.md

@@ -15,13 +15,21 @@ always: true
 
 你没有内存或后台线程，口头承诺"到时候提醒"是无效的——只有调用工具才能真正注册定时任务。
 
-**🔒 参数强制规则**：
-- ❌ **严禁**将 `kind: "cron"` 转换为 `--at` 参数
-- ✅ **必须**保持 `kind: "cron"` 不变
-- ❌ **严禁**将 cron 表达式转换为绝对时间（--at）
-- ✅ **必须**使用 cron 表达式格式（--cron）
-- ✅ **必须**使用 `--session isolated + --message`（agentTurn），**禁止**使用 `--system-event`
-- ✅ **必须**使用 `--channel "openclaw-workclaw"`
+**🔒 时间格式强制规则（核心！）**：
+
+1. **所有时间都必须转换为 cron 表达式**
+   - 相对时间（如 `5m`、`1h`、`1h30m`、`2d`）会自动转换为 cron 表达式
+   - 你只需要传入用户说的原始时间，工具会自动转换
+   - 示例：`"5分钟后提醒"` → 传入 `time: "5m"` → 工具转换为 cron 表达式
+
+2. **严禁使用 `--at` 参数**
+   - ❌ 禁止将 cron 表达式转换为 `--at` 绝对时间参数
+   - ✅ 必须使用 `--cron` 参数传递 cron 表达式
+   - ❌ 禁止使用任何绝对时间格式（如 ISO 时间戳）
+
+3. **禁止使用其他定时方式**
+   - ❌ 禁止使用 `--at`、`--every`、`--deliver` 等参数
+   - ✅ 只能使用 `--cron` 参数
 
 ***
 
@@ -68,24 +76,26 @@ always: true
 
 | 参数 | 说明 | 示例 |
 |------|------|------|
-| `time` | 时间描述，支持相对时间（5m、1h）和 cron 表达式（0 8 * * *） | `"5m"` / `"0 8 * * *"` |
+| `time` | ⏰ **时间（核心！）**，支持相对时间和 cron 表达式，工具会自动统一转换为 cron 表达式 | `"5m"` / `"0 8 * * *"` |
 | `name` | 任务名称 | `"喝水提醒"` |
 | `message` | 提醒内容 | `"该喝水了"` |
 
-### ⏰ 时间格式
+### ⏰ 时间格式说明
 
-**相对时间**：
-- `5m` - 5分钟后
-- `1h` - 1小时后
-- `1h30m` - 1.5小时后
-- `2d` - 2天后
+**⚠️ 重要：所有时间都会被自动转换为 cron 表达式**
 
-**注意：提醒时间不能少于 2 分钟**
+**支持的输入格式**：
+| 输入格式 | 示例 | 说明 |
+|---------|------|------|
+| 相对时间 | `5m` | 5分钟后 |
+| 相对时间 | `1h` | 1小时后 |
+| 相对时间 | `1h30m` | 1.5小时后 |
+| 相对时间 | `2d` | 2天后 |
+| cron 表达式 | `0 8 * * *` | 每天早上8点 |
+| cron 表达式 | `0 22 * * *` | 每天晚上10点 |
+| cron 表达式 | `0 9 * * 1-5` | 工作日早上9点 |
 
-**cron 表达式**：
-- `0 8 * * *` - 每天早上8点
-- `0 22 * * *` - 每天晚上10点
-- `0 9 * * 1-5` - 工作日早上9点
+**⚠️ 提醒时间不能少于 2 分钟**
 
 ### 🔍 OpenClaw cron 工具返回值格式（重要！）
 
@@ -220,7 +230,7 @@ always: true
 | 参数 | 说明 | 示例 |
 |------|------|------|
 | `jobId` | OpenClaw 任务 ID | `"123456"` |
-| `time` | 时间描述，支持相对时间（5m、1h）和 cron 表达式 | `"10m"` / `"0 9 * * *"` |
+| `time` | ⏰ **时间（核心！）**，支持相对时间和 cron 表达式，工具会自动统一转换为 cron 表达式 | `"10m"` / `"0 9 * * *"` |
 | `name` | 任务名称 | `"新喝水提醒"` |
 | `message` | 提醒内容 | `"该喝水了"` |
 
@@ -443,16 +453,23 @@ AI回复2（任务被删除）："🗑️ 该提醒已被删除，无法重新
 
 ## 🚫 严禁事项
 
+### ⏰ 时间格式相关（最严禁！）
+
+- ❌ **严禁**使用 `--at` 参数或其他绝对时间格式
+- ❌ **严禁**将 cron 表达式转换为 `--at` 绝对时间
+- ❌ **严禁**使用 `--every`、`--deliver` 等其他定时参数
+- ✅ **必须**使用 `--cron` 参数传递 cron 表达式
+
+### 🔧 工具调用相关
+
 - ❌ **严禁**只说"好的，我会提醒你"而不调用工具
-- ❌ **严禁**在定时任务触发时不调用 `openclaw-workclaw-cron-notify-sync`
 - ❌ **严禁**传递虚构的 `jobId`，必须从 cron 工具返回值中提取
 - ❌ **严禁**跳过任何一步骤，必须按顺序执行：参数生成 → cron 工具 → 后端同步
-- ❌ **严禁**将 `kind: "cron"` 转换为 `--at` 参数，必须保持 `kind: "cron"` 不变
-- ❌ **严禁**将 cron 表达式转换为绝对时间（--at），必须使用 cron 表达式格式（--cron）
-- ❌ **严禁**轻易说"任务已被移除"，应先尝试启用（enable）操作
+- ❌ **严禁**在定时任务触发时不调用 `openclaw-workclaw-cron-notify-sync`
 - ❌ **严禁**直接使用 `cron` 命令创建/修改/删除定时任务，必须通过工具参数生成后执行
 - ❌ **严禁**绕过本 SKILL 中定义的工具流程自行实现定时功能
-- ❌ **严禁**使用其他任何未在本 SKILL 中列出的定时任务相关工具
-- ❌ **严禁**使用 `--system-event`，必须使用 `--message`（agentTurn）
-- ❌ **严禁**使用 `--session current`，必须使用 `--session isolated`
-- ❌ **严禁**使用 `--channel "last"`，必须使用 `--channel "openclaw-workclaw"`
+- ❌ **严禁**轻易说"任务已被移除"，应先尝试启用（enable）操作
+
+### 📝 CLI 参数相关
+
+- ❌ **严禁**使用 `--system-event`

package/src/accounts.ts

@@ -1,12 +1,11 @@
 import type { OpenClawConfig } from 'openclaw/plugin-sdk'
 import type {
-  OpenclawWorkclawAccountConfig,
-  OpenclawWorkclawConfig,
-  ResolvedOpenclawWorkclawAccount,
+  ResolvedWorkclawAccount,
+  WorkclawAccountConfig,
+  WorkclawConfig,
 } from './types.js'
-import { getOpenclawWorkclawLogger } from './runtime.js'
-
-const DEFAULT_ACCOUNT_ID = 'default'
+import { DEFAULT_ACCOUNT_ID } from 'openclaw/plugin-sdk/account-id'
+import { getWorkclawLogger, getWorkclawConnectionConfig } from './runtime.js'
 
 // =============================================================================
 // In-memory lookup cache for (userId, agentId) → workspace-N accountId
@@ -52,16 +51,16 @@ export function buildAccountMap(cfg: OpenClawConfig): void {
   accountIndex.clear()
   nextWorkspaceNum = 1
 
-  const _OpenclawWorkclawConfig = (cfg.channels?.['openclaw-workclaw'] || {}) as OpenclawWorkclawConfig
+  const _WorkclawConfig = (cfg.channels?.['openclaw-workclaw'] || {}) as WorkclawConfig
 
-  const accounts = _OpenclawWorkclawConfig?.accounts
+  const accounts = _WorkclawConfig?.accounts
   if (!accounts || typeof accounts !== 'object')
     return
 
   for (const [accountId, accountCfg] of Object.entries(accounts)) {
     if (!accountCfg)
       continue
-    const userId = accountCfg.userId || _OpenclawWorkclawConfig.userId
+    const userId = accountCfg.userId || _WorkclawConfig.userId
     const agentId = accountCfg.agentId
     if (!agentId)
       continue
@@ -82,7 +81,7 @@ export function buildAccountMap(cfg: OpenClawConfig): void {
     }
   }
 
-  getOpenclawWorkclawLogger().info(`[AccountManager] Built account map with ${lookupCache.size} entries, nextWorkspaceNum=${nextWorkspaceNum}`)
+  getWorkclawLogger().info(`[AccountManager] Built account map with ${lookupCache.size} entries, nextWorkspaceNum=${nextWorkspaceNum}`)
 }
 
 /**
@@ -100,13 +99,13 @@ export function resolveAccountByUserIdAndAgentId(
     return lookupCache.get(key)!
   }
 
-  const _OpenclawWorkclawConfig = (cfg.channels?.['openclaw-workclaw'] || {}) as OpenclawWorkclawConfig
-  const accounts = _OpenclawWorkclawConfig?.accounts
+  const _WorkclawConfig = (cfg.channels?.['openclaw-workclaw'] || {}) as WorkclawConfig
+  const accounts = _WorkclawConfig?.accounts
   if (accounts && typeof accounts === 'object') {
     for (const [accountId, accountCfg] of Object.entries(accounts)) {
       if (!accountCfg)
         continue
-      const accountUserId = accountCfg.userId || _OpenclawWorkclawConfig.userId
+      const accountUserId = accountCfg.userId || _WorkclawConfig.userId
       if (String(accountUserId) === userId && String(accountCfg.agentId) === agentId) {
         lookupCache.set(key, accountId)
         accountIndex.set(accountId, { userId, agentId })
@@ -133,7 +132,7 @@ export function allocateWorkerAccountId(
   lookupCache.set(key, accountId)
   accountIndex.set(accountId, { userId, agentId })
 
-  getOpenclawWorkclawLogger().info(`[AccountManager] Allocated accountId=${accountId} for userId=${userId} agentId=${agentId}`)
+  getWorkclawLogger().info(`[AccountManager] Allocated accountId=${accountId} for userId=${userId} agentId=${agentId}`)
   return accountId
 }
 
@@ -152,7 +151,7 @@ export function refreshAccountCache(cfg: OpenClawConfig): void {
  * List all configured account IDs from the accounts field.
  */
 function listConfiguredAccountIds(cfg: OpenClawConfig): string[] {
-  const accounts = (cfg.channels?.['openclaw-workclaw'] as OpenclawWorkclawConfig)?.accounts
+  const accounts = (cfg.channels?.['openclaw-workclaw'] as WorkclawConfig)?.accounts
   if (!accounts || typeof accounts !== 'object') {
     return []
   }
@@ -163,9 +162,9 @@ function listConfiguredAccountIds(cfg: OpenClawConfig): string[] {
  * Check if plugin-level workclaw config exists (appKey + appSecret)
  */
 function hasPluginLevelWorkClawConfig(cfg: OpenClawConfig): boolean {
-  const openclawWorkclawCfg = cfg.channels?.['openclaw-workclaw'] as OpenclawWorkclawConfig | undefined
-  const key = typeof openclawWorkclawCfg?.appKey === 'string' ? openclawWorkclawCfg.appKey.trim() : ''
-  const secret = typeof openclawWorkclawCfg?.appSecret === 'string' ? openclawWorkclawCfg.appSecret.trim() : ''
+  const workclawCfg = cfg.channels?.['openclaw-workclaw'] as WorkclawConfig | undefined
+  const key = typeof workclawCfg?.appKey === 'string' ? workclawCfg.appKey.trim() : ''
+  const secret = typeof workclawCfg?.appSecret === 'string' ? workclawCfg.appSecret.trim() : ''
   return Boolean(key && secret)
 }
 
@@ -173,7 +172,7 @@ function hasPluginLevelWorkClawConfig(cfg: OpenClawConfig): boolean {
  * List all account IDs.
  * If no accounts are configured but plugin has workclaw config, returns [DEFAULT_ACCOUNT_ID].
  */
-export function listOpenclawWorkclawAccountIds(cfg: OpenClawConfig): string[] {
+export function listWorkclawAccountIds(cfg: OpenClawConfig): string[] {
   const ids = listConfiguredAccountIds(cfg)
   if (ids.length === 0) {
     // 如果没有配置账户，但插件级别有 workclaw 配置，返回默认账户
@@ -189,8 +188,8 @@ export function listOpenclawWorkclawAccountIds(cfg: OpenClawConfig): string[] {
 /**
  * Resolve the default account ID.
  */
-export function resolveDefaultOpenclawWorkclawAccountId(cfg: OpenClawConfig): string {
-  const ids = listOpenclawWorkclawAccountIds(cfg)
+export function resolveDefaultWorkclawAccountId(cfg: OpenClawConfig): string {
+  const ids = listWorkclawAccountIds(cfg)
   if (ids.includes(DEFAULT_ACCOUNT_ID)) {
     return DEFAULT_ACCOUNT_ID
   }
@@ -203,8 +202,8 @@ export function resolveDefaultOpenclawWorkclawAccountId(cfg: OpenClawConfig): st
 function resolveAccountConfig(
   cfg: OpenClawConfig,
   accountId: string,
-): OpenclawWorkclawAccountConfig | undefined {
-  const accounts = (cfg.channels?.['openclaw-workclaw'] as OpenclawWorkclawConfig)?.accounts
+): WorkclawAccountConfig | undefined {
+  const accounts = (cfg.channels?.['openclaw-workclaw'] as WorkclawConfig)?.accounts
   if (!accounts || typeof accounts !== 'object') {
     return undefined
   }
@@ -223,7 +222,7 @@ function hasWorkClawConfig(config: {
   return Boolean(key && secret)
 }
 
-export function isOpenclawWorkclawAccountConfigured(config: {
+export function isWorkclawAccountConfigured(config: {
   appKey?: string
   appSecret?: string
   agentId?: string | number
@@ -235,34 +234,34 @@ export function isOpenclawWorkclawAccountConfigured(config: {
  * Merge top-level config with account-specific config.
  * Account-specific fields override top-level fields.
  */
-function mergeOpenclawWorkclawAccountConfig(cfg: OpenClawConfig, accountId: string): OpenclawWorkclawConfig {
-  const openclawWorkclawCfg = cfg.channels?.['openclaw-workclaw'] as OpenclawWorkclawConfig | undefined
+function mergeWorkclawAccountConfig(cfg: OpenClawConfig, accountId: string): WorkclawConfig {
+  const workclawCfg = cfg.channels?.['openclaw-workclaw'] as WorkclawConfig | undefined
 
   // Extract base config (exclude accounts field to avoid recursion)
-  const { accounts: _ignored, ...base } = openclawWorkclawCfg ?? {}
+  const { accounts: _ignored, ...base } = workclawCfg ?? {}
 
   // Get account-specific overrides
   const account = resolveAccountConfig(cfg, accountId) ?? {}
 
   // Merge: account config overrides base config
-  return { ...base, ...account } as OpenclawWorkclawConfig
+  return { ...base, ...account } as WorkclawConfig
 }
 
 /**
  * Resolve a complete account with merged config.
  */
-export function resolveOpenclawWorkclawAccount(params: {
+export function resolveWorkclawAccount(params: {
   cfg: OpenClawConfig
   accountId?: string | null
-}): ResolvedOpenclawWorkclawAccount {
+}): ResolvedWorkclawAccount {
   const accountId = normalizeAccountId(params.accountId)
-  const openclawWorkclawCfg = params.cfg.channels?.['openclaw-workclaw'] as OpenclawWorkclawConfig | undefined
+  const workclawCfg = params.cfg.channels?.['openclaw-workclaw'] as WorkclawConfig | undefined
 
   // Base enabled state (top-level)
-  const baseEnabled = openclawWorkclawCfg?.enabled !== false
+  const baseEnabled = workclawCfg?.enabled !== false
 
   // Merge configs
-  const merged = mergeOpenclawWorkclawAccountConfig(params.cfg, accountId)
+  const merged = mergeWorkclawAccountConfig(params.cfg, accountId)
 
   // Account-level enabled state
   const accountEnabled = merged.enabled !== false
@@ -271,8 +270,8 @@ export function resolveOpenclawWorkclawAccount(params: {
   return {
     accountId,
     enabled,
-    configured: isOpenclawWorkclawAccountConfigured(merged),
-    name: (merged as OpenclawWorkclawAccountConfig).name?.trim() || undefined,
+    configured: isWorkclawAccountConfigured(merged),
+    name: (merged as WorkclawAccountConfig).name?.trim() || undefined,
     config: merged,
   }
 }
@@ -280,8 +279,34 @@ export function resolveOpenclawWorkclawAccount(params: {
 /**
  * List all enabled and configured accounts.
  */
-export function listEnabledOpenclawWorkclawAccounts(cfg: OpenClawConfig): ResolvedOpenclawWorkclawAccount[] {
+export function listEnabledWorkclawAccounts(cfg: OpenClawConfig): ResolvedWorkclawAccount[] {
   return listConfiguredAccountIds(cfg)
-    .map(accountId => resolveOpenclawWorkclawAccount({ cfg, accountId }))
+    .map(accountId => resolveWorkclawAccount({ cfg, accountId }))
     .filter(account => account.enabled && account.configured)
 }
+
+/**
+ * Resolve account and apply cached connection config if available.
+ * This ensures that appKey/appSecret etc. use the stable values captured at connection time,
+ * not potentially mutated values from cfg at resolution time.
+ */
+export function resolveWorkclawAccountWithCache(params: {
+  cfg: OpenClawConfig
+  accountId?: string | null
+}): ResolvedWorkclawAccount {
+  const account = resolveWorkclawAccount(params);
+  const cachedConfig = getWorkclawConnectionConfig(account.accountId);
+
+  if (cachedConfig) {
+    // Apply cached values to ensure stability
+    account.config.appKey = cachedConfig.appKey || account.config.appKey;
+    account.config.appSecret = cachedConfig.appSecret || account.config.appSecret;
+    if (cachedConfig.baseUrl) account.config.baseUrl = cachedConfig.baseUrl;
+    if (cachedConfig.websocketUrl !== undefined) account.config.websocketUrl = cachedConfig.websocketUrl;
+    if (cachedConfig.localIp !== undefined) account.config.localIp = cachedConfig.localIp;
+    if (cachedConfig.allowInsecureTls !== undefined) account.config.allowInsecureTls = cachedConfig.allowInsecureTls;
+    if (cachedConfig.requestTimeout !== undefined) account.config.requestTimeout = cachedConfig.requestTimeout;
+  }
+
+  return account;
+}

package/src/api/accounts-api.ts

@@ -1,55 +1,55 @@
-import type { OpenClawPluginApi } from 'openclaw/plugin-sdk'
-import { Buffer } from 'node:buffer'
-import { normalizeAccountId } from '../accounts.js'
-
-function sendJson(res: any, statusCode: number, payload: unknown): void {
-  res.statusCode = statusCode
-  res.setHeader('Content-Type', 'application/json')
-  res.end(JSON.stringify(payload))
+import type { OpenClawPluginApi } from "openclaw/plugin-sdk";
+import { normalizeAccountId } from "../accounts.js";
+
+
+function sendJson(res: any, statusCode: number, payload: unknown) {
+  res.statusCode = statusCode;
+  res.setHeader("Content-Type", "application/json");
+  res.end(JSON.stringify(payload));
 }
 
 async function readRequestBody(req: any): Promise<string> {
-  const chunks: Buffer[] = []
+  const chunks: Buffer[] = [];
   await new Promise<void>((resolve, reject) => {
-    req.on('data', (chunk: any) => {
-      chunks.push(Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk))
-    })
-    req.on('end', () => resolve())
-    req.on('error', (err: unknown) => reject(err))
-  })
-  return Buffer.concat(chunks).toString('utf-8')
+    req.on("data", (chunk: any) => {
+      chunks.push(Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk));
+    });
+    req.on("end", () => resolve());
+    req.on("error", (err: unknown) => reject(err));
+  });
+  return Buffer.concat(chunks).toString("utf-8");
 }
 
 async function loadRuntimeConfig(api: OpenClawPluginApi): Promise<any> {
-  const runtimeConfig = (api.runtime as any)?.config
+  const runtimeConfig = (api.runtime as any)?.config;
   if (runtimeConfig?.loadConfig) {
-    return runtimeConfig.loadConfig()
+    return runtimeConfig.loadConfig();
   }
-  return api.config
+  return api.config;
 }
 
 async function writeRuntimeConfig(api: OpenClawPluginApi, config: any): Promise<void> {
-  const runtimeConfig = (api.runtime as any)?.config
+  const runtimeConfig = (api.runtime as any)?.config;
   if (runtimeConfig?.writeConfigFile) {
-    await runtimeConfig.writeConfigFile(config)
-    return
+    await runtimeConfig.writeConfigFile(config);
+    return;
   }
-  throw new Error('Config write not supported')
+  throw new Error("Config write not supported");
 }
 
 function getAccountMap(config: any): Record<string, any> {
-  const channels = config?.channels && typeof config.channels === 'object' ? config.channels : {}
-  const openclawWorkclaw
-    = channels['openclaw-workclaw'] && typeof channels['openclaw-workclaw'] === 'object' ? channels['openclaw-workclaw'] : {}
-  const accounts
-    = openclawWorkclaw.accounts && typeof openclawWorkclaw.accounts === 'object' ? openclawWorkclaw.accounts : {}
-  return { ...accounts }
+  const channels = config?.channels && typeof config.channels === "object" ? config.channels : {};
+  const openclawWorkclaw =
+    channels['openclaw-workclaw'] && typeof channels['openclaw-workclaw'] === "object" ? channels['openclaw-workclaw'] : {};
+  const accounts =
+    openclawWorkclaw.accounts && typeof openclawWorkclaw.accounts === "object" ? openclawWorkclaw.accounts : {};
+  return { ...accounts };
 }
 
 function withAccounts(config: any, accounts: Record<string, any>): any {
-  const channels = config?.channels && typeof config.channels === 'object' ? config.channels : {}
-  const openclawWorkclaw
-    = channels['openclaw-workclaw'] && typeof channels['openclaw-workclaw'] === 'object' ? channels['openclaw-workclaw'] : {}
+  const channels = config?.channels && typeof config.channels === "object" ? config.channels : {};
+  const openclawWorkclaw =
+    channels['openclaw-workclaw'] && typeof channels['openclaw-workclaw'] === "object" ? channels['openclaw-workclaw'] : {};
   return {
     ...config,
     channels: {
@@ -59,99 +59,98 @@ function withAccounts(config: any, accounts: Record<string, any>): any {
         accounts,
       },
     },
-  }
+  };
 }
 
 export function createAccountsApiHandler(api: OpenClawPluginApi) {
   return async (req: any, res: any) => {
-    const method = String(req.method ?? 'GET').toUpperCase()
-    const url = new URL(req.url ?? '', 'http://localhost')
-    const idRaw = url.searchParams.get('id') ?? url.searchParams.get('accountId')
-    const accountId = idRaw ? normalizeAccountId(idRaw) : null
+    const method = String(req.method ?? "GET").toUpperCase();
+    const url = new URL(req.url ?? "", "http://localhost");
+    const idRaw = url.searchParams.get("id") ?? url.searchParams.get("accountId");
+    const accountId = idRaw ? normalizeAccountId(idRaw) : null;
 
-    if (method === 'GET') {
-      const cfg = await loadRuntimeConfig(api)
-      const accounts = getAccountMap(cfg)
+    if (method === "GET") {
+      const cfg = await loadRuntimeConfig(api);
+      const accounts = getAccountMap(cfg);
 
       if (accountId) {
-        const account = accounts[accountId]
+        const account = accounts[accountId];
         if (!account) {
-          sendJson(res, 404, { ok: false, error: 'Not Found' })
-          return
+          sendJson(res, 404, { ok: false, error: "Not Found" });
+          return;
         }
-        sendJson(res, 200, { ok: true, accountId, config: account })
-        return
+        sendJson(res, 200, { ok: true, accountId, config: account });
+        return;
       }
 
       const entries = Object.entries(accounts).map(([id, config]) => ({
         accountId: id,
         config,
-      }))
-      sendJson(res, 200, { ok: true, accounts: entries })
-      return
+      }));
+      sendJson(res, 200, { ok: true, accounts: entries });
+      return;
     }
 
-    if (method === 'POST' || method === 'PUT') {
-      const raw = await readRequestBody(req)
-      let input: any = {}
+    if (method === "POST" || method === "PUT") {
+      const raw = await readRequestBody(req);
+      let input: any = {};
       try {
-        input = raw ? JSON.parse(raw) : {}
-      }
-      catch {
-        sendJson(res, 400, { ok: false, error: 'Invalid JSON' })
-        return
+        input = raw ? JSON.parse(raw) : {};
+      } catch {
+        sendJson(res, 400, { ok: false, error: "Invalid JSON" });
+        return;
       }
 
-      const inputId = input?.accountId ?? input?.id
-      const normalizedId = inputId ? normalizeAccountId(String(inputId)) : ''
-      const patch = input?.config
+      const inputId = input?.accountId ?? input?.id;
+      const normalizedId = inputId ? normalizeAccountId(String(inputId)) : "";
+      const patch = input?.config;
 
-      if (!normalizedId || !patch || typeof patch !== 'object') {
-        sendJson(res, 400, { ok: false, error: 'Missing accountId or config' })
-        return
+      if (!normalizedId || !patch || typeof patch !== "object") {
+        sendJson(res, 400, { ok: false, error: "Missing accountId or config" });
+        return;
       }
 
-      const cfg = await loadRuntimeConfig(api)
-      const accounts = getAccountMap(cfg)
+      const cfg = await loadRuntimeConfig(api);
+      const accounts = getAccountMap(cfg);
 
-      if (method === 'POST' && accounts[normalizedId]) {
-        sendJson(res, 409, { ok: false, error: 'Account already exists' })
-        return
+      if (method === "POST" && accounts[normalizedId]) {
+        sendJson(res, 409, { ok: false, error: "Account already exists" });
+        return;
       }
 
-      if (method === 'PUT' && !accounts[normalizedId]) {
-        sendJson(res, 404, { ok: false, error: 'Not Found' })
-        return
+      if (method === "PUT" && !accounts[normalizedId]) {
+        sendJson(res, 404, { ok: false, error: "Not Found" });
+        return;
       }
 
-      const next = method === 'PUT' ? { ...accounts[normalizedId], ...patch } : patch
-      const updated = withAccounts(cfg, { ...accounts, [normalizedId]: next })
-      await writeRuntimeConfig(api, updated)
-      sendJson(res, 200, { ok: true, accountId: normalizedId })
-      return
+      const next = method === "PUT" ? { ...accounts[normalizedId], ...patch } : patch;
+      const updated = withAccounts(cfg, { ...accounts, [normalizedId]: next });
+      await writeRuntimeConfig(api, updated);
+      sendJson(res, 200, { ok: true, accountId: normalizedId });
+      return;
     }
 
-    if (method === 'DELETE') {
+    if (method === "DELETE") {
       if (!accountId) {
-        sendJson(res, 400, { ok: false, error: 'Missing accountId' })
-        return
+        sendJson(res, 400, { ok: false, error: "Missing accountId" });
+        return;
       }
 
-      const cfg = await loadRuntimeConfig(api)
-      const accounts = getAccountMap(cfg)
+      const cfg = await loadRuntimeConfig(api);
+      const accounts = getAccountMap(cfg);
 
       if (!accounts[accountId]) {
-        sendJson(res, 404, { ok: false, error: 'Not Found' })
-        return
+        sendJson(res, 404, { ok: false, error: "Not Found" });
+        return;
       }
 
-      const { [accountId]: _ignored, ...rest } = accounts
-      const updated = withAccounts(cfg, rest)
-      await writeRuntimeConfig(api, updated)
-      sendJson(res, 200, { ok: true, accountId, deleted: true })
-      return
+      const { [accountId]: _ignored, ...rest } = accounts;
+      const updated = withAccounts(cfg, rest);
+      await writeRuntimeConfig(api, updated);
+      sendJson(res, 200, { ok: true, accountId, deleted: true });
+      return;
     }
 
-    sendJson(res, 405, { ok: false, error: 'Method Not Allowed' })
-  }
+    sendJson(res, 405, { ok: false, error: "Method Not Allowed" });
+  };
 }