@useorgx/openclaw-plugin 0.4.8 → 0.7.0

package/dist/activity-store.js

@@ -1,41 +1,43 @@
-import { chmodSync, existsSync, mkdirSync, readFileSync, } from "node:fs";
+import { existsSync, readFileSync, } from "node:fs";
 import { Buffer } from "node:buffer";
 import { enrichActivityActorFieldsList } from "./activity-actor-fields.js";
+import { shouldHideActivityItem } from "./event-sanitization.js";
 import { getOrgxPluginConfigDir, getOrgxPluginConfigPath } from "./paths.js";
-import { backupCorruptFileSync, writeJsonFileAtomicSync } from "./fs-utils.js";
+import { backupCorruptFileSync, writeFileAtomicSync } from "./fs-utils.js";
+import { ensureStoreDirSync, parseJsonSafe } from "./stores/json-store.js";
 const STORE_VERSION = 1;
 const STORE_FILENAME = "activity-store.json";
-const MAX_ITEMS = 50_000;
-const RETENTION_DAYS = 45;
-const FLUSH_DEBOUNCE_MS = 1_250;
+const MAX_ITEMS = 10_000;
+const RETENTION_DAYS = 14;
+const FLUSH_DEBOUNCE_MS = 3_000;
 let cached = null;
 function ensureDir() {
-    const dir = getOrgxPluginConfigDir();
-    mkdirSync(dir, { recursive: true, mode: 0o700 });
-    try {
-        chmodSync(dir, 0o700);
-    }
-    catch {
-        // best effort
-    }
+    ensureStoreDirSync(getOrgxPluginConfigDir());
 }
 function storePath() {
     return getOrgxPluginConfigPath(STORE_FILENAME);
 }
-function parseJson(value) {
-    try {
-        return JSON.parse(value);
-    }
-    catch {
-        return null;
-    }
-}
 function toEpoch(value) {
     if (!value)
         return 0;
     const parsed = Date.parse(value);
     return Number.isFinite(parsed) ? parsed : 0;
 }
+function shallowMetadataEqual(a, b) {
+    if (a == null && b == null)
+        return true;
+    if (a == null || b == null)
+        return false;
+    const keysA = Object.keys(a);
+    const keysB = Object.keys(b);
+    if (keysA.length !== keysB.length)
+        return false;
+    for (const key of keysA) {
+        if (a[key] !== b[key])
+            return false;
+    }
+    return true;
+}
 function compareActivity(a, b) {
     const delta = toEpoch(b.timestamp) - toEpoch(a.timestamp);
     if (delta !== 0)
@@ -61,6 +63,8 @@ function normalizeItems(source) {
             continue;
         if (epoch < cutoffEpoch)
             continue;
+        if (shouldHideActivityItem(item))
+            continue;
         byId.set(id, item);
     }
     return enrichActivityActorFieldsList(Array.from(byId.values()))
@@ -75,7 +79,7 @@ function readPersistedStore() {
     }
     try {
         const raw = readFileSync(file, "utf8");
-        const parsed = parseJson(raw);
+        const parsed = parseJsonSafe(raw);
         if (!parsed || parsed.version !== STORE_VERSION || !Array.isArray(parsed.items)) {
             backupCorruptFileSync(file);
             return { version: STORE_VERSION, updatedAt: new Date().toISOString(), items: [] };
@@ -151,7 +155,7 @@ function scheduleFlush(state) {
             updatedAt: state.storeUpdatedAt,
             items: state.items,
         };
-        writeJsonFileAtomicSync(storePath(), payload, 0o600);
+        writeFileAtomicSync(storePath(), JSON.stringify(payload), { mode: 0o600 });
     }, FLUSH_DEBOUNCE_MS);
     state.flushTimer.unref?.();
 }
@@ -172,6 +176,8 @@ export function appendActivityItems(items) {
             continue;
         if (!Number.isFinite(Date.parse(item.timestamp)))
             continue;
+        if (shouldHideActivityItem(item))
+            continue;
         const id = item.id.trim();
         const existing = state.byId.get(id);
         if (!existing) {
@@ -189,7 +195,7 @@ export function appendActivityItems(items) {
             (existing.executorAgentId ?? null) !== (item.executorAgentId ?? null) ||
             (existing.executorAgentName ?? null) !== (item.executorAgentName ?? null) ||
             existing.summary !== item.summary ||
-            JSON.stringify(existing.metadata ?? null) !== JSON.stringify(item.metadata ?? null)) {
+            !shallowMetadataEqual(existing.metadata, item.metadata)) {
             state.byId.set(id, item);
             updated += 1;
         }
@@ -197,8 +203,14 @@ export function appendActivityItems(items) {
     if (appended === 0 && updated === 0) {
         return { appended: 0, updated: 0, total: state.items.length };
     }
-    // Rebuild sorted list from map. This is O(n) but bounded by MAX_ITEMS.
-    state.items = normalizeItems(Array.from(state.byId.values()));
+    // Rebuild sorted list from map, bounded by MAX_ITEMS.
+    // Skip full normalizeItems (enrichment + dedup) for small batches — items are already validated above.
+    if (appended + updated > 50) {
+        state.items = normalizeItems(Array.from(state.byId.values()));
+    }
+    else {
+        state.items = Array.from(state.byId.values()).sort(compareActivity).slice(0, MAX_ITEMS);
+    }
     state.byId = new Map(state.items.map((item) => [item.id, item]));
     state.dirty = true;
     scheduleFlush(state);

package/dist/agent-context-store.js

@@ -1,8 +1,73 @@
-import { chmodSync, existsSync, mkdirSync, readFileSync, rmSync, } from "node:fs";
+import { existsSync, readFileSync, } from "node:fs";
 import { getOrgxPluginConfigDir, getOrgxPluginConfigPath } from "./paths.js";
 import { backupCorruptFileSync, writeJsonFileAtomicSync } from "./fs-utils.js";
+import { clearStoreFileSync, ensureStoreDirSync, parseJsonSafe, } from "./stores/json-store.js";
 const MAX_AGENTS = 120;
 const MAX_RUNS = 480;
+function timestampSortValue(value) {
+    const parsed = Date.parse(value);
+    return Number.isFinite(parsed) ? parsed : 0;
+}
+function isRecord(value) {
+    return Boolean(value) && typeof value === "object" && !Array.isArray(value);
+}
+function asOptionalString(value) {
+    if (typeof value !== "string")
+        return null;
+    const trimmed = value.trim();
+    return trimmed.length > 0 ? trimmed : null;
+}
+function asTimestamp(value) {
+    if (typeof value === "string" && value.trim().length > 0)
+        return value;
+    return new Date().toISOString();
+}
+function normalizeAgentContextsMap(input) {
+    if (!isRecord(input))
+        return {};
+    const normalized = {};
+    for (const [key, value] of Object.entries(input)) {
+        if (!isRecord(value))
+            continue;
+        const hasExplicitAgentId = Object.prototype.hasOwnProperty.call(value, "agentId");
+        const id = hasExplicitAgentId ? asOptionalString(value.agentId) : asOptionalString(key);
+        if (!id)
+            continue;
+        normalized[id] = {
+            agentId: id,
+            initiativeId: asOptionalString(value.initiativeId),
+            initiativeTitle: asOptionalString(value.initiativeTitle),
+            workstreamId: asOptionalString(value.workstreamId),
+            taskId: asOptionalString(value.taskId),
+            updatedAt: asTimestamp(value.updatedAt),
+        };
+    }
+    return normalized;
+}
+function normalizeRunContextsMap(input) {
+    if (!isRecord(input))
+        return {};
+    const normalized = {};
+    for (const [key, value] of Object.entries(input)) {
+        if (!isRecord(value))
+            continue;
+        const hasExplicitRunId = Object.prototype.hasOwnProperty.call(value, "runId");
+        const runId = hasExplicitRunId ? asOptionalString(value.runId) : asOptionalString(key);
+        const agentId = asOptionalString(value.agentId);
+        if (!runId || !agentId)
+            continue;
+        normalized[runId] = {
+            runId,
+            agentId,
+            initiativeId: asOptionalString(value.initiativeId),
+            initiativeTitle: asOptionalString(value.initiativeTitle),
+            workstreamId: asOptionalString(value.workstreamId),
+            taskId: asOptionalString(value.taskId),
+            updatedAt: asTimestamp(value.updatedAt),
+        };
+    }
+    return normalized;
+}
 function contextDir() {
     return getOrgxPluginConfigDir();
 }
@@ -10,42 +75,27 @@ function contextFile() {
     return getOrgxPluginConfigPath("agent-contexts.json");
 }
 function ensureContextDir() {
-    const dir = contextDir();
-    mkdirSync(dir, { recursive: true, mode: 0o700 });
-    try {
-        chmodSync(dir, 0o700);
-    }
-    catch {
-        // best effort
-    }
-}
-function parseJson(value) {
-    try {
-        return JSON.parse(value);
-    }
-    catch {
-        return null;
-    }
+    ensureStoreDirSync(contextDir());
 }
 function normalizeContext(input) {
     return {
         agentId: input.agentId.trim(),
-        initiativeId: input.initiativeId ?? null,
-        initiativeTitle: input.initiativeTitle ?? null,
-        workstreamId: input.workstreamId ?? null,
-        taskId: input.taskId ?? null,
-        updatedAt: input.updatedAt,
+        initiativeId: asOptionalString(input.initiativeId),
+        initiativeTitle: asOptionalString(input.initiativeTitle),
+        workstreamId: asOptionalString(input.workstreamId),
+        taskId: asOptionalString(input.taskId),
+        updatedAt: asTimestamp(input.updatedAt),
     };
 }
 function normalizeRunContext(input) {
     return {
         runId: input.runId.trim(),
         agentId: input.agentId.trim(),
-        initiativeId: input.initiativeId ?? null,
-        initiativeTitle: input.initiativeTitle ?? null,
-        workstreamId: input.workstreamId ?? null,
-        taskId: input.taskId ?? null,
-        updatedAt: input.updatedAt,
+        initiativeId: asOptionalString(input.initiativeId),
+        initiativeTitle: asOptionalString(input.initiativeTitle),
+        workstreamId: asOptionalString(input.workstreamId),
+        taskId: asOptionalString(input.taskId),
+        updatedAt: asTimestamp(input.updatedAt),
     };
 }
 export function readAgentContexts() {
@@ -55,17 +105,15 @@ export function readAgentContexts() {
             return { updatedAt: new Date().toISOString(), agents: {}, runs: {} };
         }
         const raw = readFileSync(file, "utf8");
-        const parsed = parseJson(raw);
+        const parsed = parseJsonSafe(raw);
         if (!parsed || typeof parsed !== "object") {
             backupCorruptFileSync(file);
             return { updatedAt: new Date().toISOString(), agents: {}, runs: {} };
         }
-        const agents = parsed.agents && typeof parsed.agents === "object" ? parsed.agents : {};
-        const runs = parsed.runs && typeof parsed.runs === "object" ? parsed.runs : {};
+        const agents = normalizeAgentContextsMap(parsed.agents);
+        const runs = normalizeRunContextsMap(parsed.runs);
         return {
-            updatedAt: typeof parsed.updatedAt === "string"
-                ? parsed.updatedAt
-                : new Date().toISOString(),
+            updatedAt: asTimestamp(parsed.updatedAt),
             agents: agents,
             runs: runs,
         };
@@ -109,7 +157,7 @@ export function upsertAgentContext(input) {
     // Prune if the file grows unbounded (rare).
     const values = Object.values(next.agents);
     if (values.length > MAX_AGENTS) {
-        values.sort((a, b) => Date.parse(b.updatedAt) - Date.parse(a.updatedAt));
+        values.sort((a, b) => timestampSortValue(b.updatedAt) - timestampSortValue(a.updatedAt));
         const keep = new Set(values.slice(0, MAX_AGENTS).map((c) => c.agentId));
         for (const key of Object.keys(next.agents)) {
             if (!keep.has(key)) {
@@ -144,7 +192,7 @@ export function upsertRunContext(input) {
     next.updatedAt = new Date().toISOString();
     const values = Object.values(next.runs);
     if (values.length > MAX_RUNS) {
-        values.sort((a, b) => Date.parse(b.updatedAt) - Date.parse(a.updatedAt));
+        values.sort((a, b) => timestampSortValue(b.updatedAt) - timestampSortValue(a.updatedAt));
         const keep = new Set(values.slice(0, MAX_RUNS).map((c) => c.runId));
         for (const key of Object.keys(next.runs)) {
             if (!keep.has(key)) {
@@ -157,11 +205,5 @@ export function upsertRunContext(input) {
     return next;
 }
 export function clearAgentContexts() {
-    const file = contextFile();
-    try {
-        rmSync(file, { force: true });
-    }
-    catch {
-        // best effort
-    }
+    clearStoreFileSync(contextFile());
 }

package/dist/agent-run-store.js

@@ -1,6 +1,7 @@
-import { chmodSync, existsSync, mkdirSync, readFileSync, rmSync, } from "node:fs";
+import { existsSync, readFileSync, } from "node:fs";
 import { getOrgxPluginConfigDir, getOrgxPluginConfigPath } from "./paths.js";
 import { backupCorruptFileSync, writeJsonFileAtomicSync } from "./fs-utils.js";
+import { clearStoreFileSync, ensureStoreDirSync, parseJsonSafe, } from "./stores/json-store.js";
 const MAX_RUNS = 240;
 function runDir() {
     return getOrgxPluginConfigDir();
@@ -9,22 +10,7 @@ function runFile() {
     return getOrgxPluginConfigPath("agent-runs.json");
 }
 function ensureRunDir() {
-    const dir = runDir();
-    mkdirSync(dir, { recursive: true, mode: 0o700 });
-    try {
-        chmodSync(dir, 0o700);
-    }
-    catch {
-        // best effort
-    }
-}
-function parseJson(value) {
-    try {
-        return JSON.parse(value);
-    }
-    catch {
-        return null;
-    }
+    ensureStoreDirSync(runDir());
 }
 function normalizeNullableString(value) {
     if (typeof value !== "string")
@@ -32,7 +18,24 @@ function normalizeNullableString(value) {
     const trimmed = value.trim();
     return trimmed.length > 0 ? trimmed : null;
 }
+function normalizeTimestamp(value, fallback) {
+    if (typeof value !== "string")
+        return fallback;
+    const trimmed = value.trim();
+    if (!trimmed)
+        return fallback;
+    return Number.isFinite(Date.parse(trimmed)) ? trimmed : fallback;
+}
+function normalizeNullableTimestamp(value) {
+    if (typeof value !== "string")
+        return null;
+    const trimmed = value.trim();
+    if (!trimmed)
+        return null;
+    return Number.isFinite(Date.parse(trimmed)) ? trimmed : null;
+}
 function normalizeRecord(input) {
+    const now = new Date().toISOString();
     return {
         runId: input.runId.trim(),
         agentId: input.agentId.trim(),
@@ -44,8 +47,8 @@ function normalizeRecord(input) {
         initiativeTitle: normalizeNullableString(input.initiativeTitle),
         workstreamId: normalizeNullableString(input.workstreamId),
         taskId: normalizeNullableString(input.taskId),
-        startedAt: input.startedAt,
-        stoppedAt: input.stoppedAt ?? null,
+        startedAt: normalizeTimestamp(input.startedAt, now),
+        stoppedAt: normalizeNullableTimestamp(input.stoppedAt),
         status: input.status === "stopped" ? "stopped" : "running",
     };
 }
@@ -56,15 +59,39 @@ export function readAgentRuns() {
             return { updatedAt: new Date().toISOString(), runs: {} };
         }
         const raw = readFileSync(file, "utf8");
-        const parsed = parseJson(raw);
+        const parsed = parseJsonSafe(raw);
         if (!parsed || typeof parsed !== "object") {
             backupCorruptFileSync(file);
             return { updatedAt: new Date().toISOString(), runs: {} };
         }
         const runs = parsed.runs && typeof parsed.runs === "object" ? parsed.runs : {};
+        const normalizedRuns = {};
+        for (const [runKey, runValue] of Object.entries(runs)) {
+            if (!runValue || typeof runValue !== "object")
+                continue;
+            const input = runValue;
+            const normalized = normalizeRecord({
+                runId: typeof input.runId === "string" ? input.runId : runKey,
+                agentId: typeof input.agentId === "string" ? input.agentId : "",
+                pid: typeof input.pid === "number" ? input.pid : null,
+                message: typeof input.message === "string" ? input.message : null,
+                provider: typeof input.provider === "string" ? input.provider : null,
+                model: typeof input.model === "string" ? input.model : null,
+                initiativeId: typeof input.initiativeId === "string" ? input.initiativeId : null,
+                initiativeTitle: typeof input.initiativeTitle === "string" ? input.initiativeTitle : null,
+                workstreamId: typeof input.workstreamId === "string" ? input.workstreamId : null,
+                taskId: typeof input.taskId === "string" ? input.taskId : null,
+                startedAt: typeof input.startedAt === "string" ? input.startedAt : "",
+                stoppedAt: typeof input.stoppedAt === "string" ? input.stoppedAt : null,
+                status: input.status === "stopped" ? "stopped" : "running",
+            });
+            if (!normalized.runId || !normalized.agentId)
+                continue;
+            normalizedRuns[normalized.runId] = normalized;
+        }
         return {
             updatedAt: typeof parsed.updatedAt === "string" ? parsed.updatedAt : new Date().toISOString(),
-            runs: runs,
+            runs: normalizedRuns,
         };
     }
     catch {
@@ -148,11 +175,5 @@ export function markAgentRunStopped(runId) {
     return updated ? normalizeRecord(updated) : null;
 }
 export function clearAgentRuns() {
-    const file = runFile();
-    try {
-        rmSync(file, { force: true });
-    }
-    catch {
-        // best effort
-    }
+    clearStoreFileSync(runFile());
 }

package/dist/agent-suite.d.ts

@@ -4,6 +4,7 @@ export type OrgxSuiteAgentSpec = {
     name: string;
     domain: OrgxSuiteDomain;
 };
+export type OrgxAgentConfigHealthStatus = "healthy" | "needs_apply" | "conflict";
 export declare const ORGX_AGENT_SUITE_PACK_ID = "orgx-agent-suite";
 export declare const ORGX_AGENT_SUITE_AGENTS: OrgxSuiteAgentSpec[];
 declare const SUITE_FILES: readonly ["AGENTS.md", "TOOLS.md", "IDENTITY.md", "SKILL.md", "SOUL.md", "USER.md", "HEARTBEAT.md"];
@@ -47,6 +48,14 @@ export type OrgxAgentSuiteStatus = {
         workspace: string;
         configuredInOpenclaw: boolean;
         workspaceExists: boolean;
+        configHealth: {
+            status: OrgxAgentConfigHealthStatus;
+            lastChangedAt: string | null;
+            evalPassRate: number;
+            totalChecks: number;
+            passedChecks: number;
+            failedChecks: number;
+        };
     }>;
 };
 export type OrgxAgentSuitePlan = OrgxAgentSuiteStatus & {