@useorgx/openclaw-plugin 0.4.9 → 0.7.0

package/dist/http/helpers/autopilot-operations.d.ts

@@ -3,6 +3,7 @@ import type { upsertAgentRun as upsertAgentRunType } from "../../agent-run-store
 type AutopilotSliceArtifact = {
     name: string;
     artifact_type?: string | null;
+    confidence_score?: number | null;
     description?: string | null;
     url?: string | null;
     verification_steps?: string[] | null;
@@ -109,7 +110,10 @@ export declare function createAutopilotOperations(deps: CreateAutopilotOperation
         agentId: string;
         agentName?: string | null;
         workstreamId: string;
+        fallbackMilestoneId?: string | null;
+        fallbackTaskIds?: string[] | null;
         artifact: AutopilotSliceArtifact;
+        isMockWorker?: boolean;
     }) => Promise<{
         ok: boolean;
         id: string | null;
@@ -118,6 +122,10 @@ export declare function createAutopilotOperations(deps: CreateAutopilotOperation
         initiativeId: string;
         runId: string;
         correlationId: string;
+        agentId?: string | null;
+        agentName?: string | null;
+        outputPath?: string | null;
+        logPath?: string | null;
         taskUpdates: Array<{
             task_id: string;
             status: string;
@@ -128,9 +136,20 @@ export declare function createAutopilotOperations(deps: CreateAutopilotOperation
             status: string;
             reason?: string | null;
         }>;
+        isMockWorker?: boolean;
     }) => Promise<{
         applied: number;
         buffered: boolean;
+        taskUpdates: Array<{
+            taskId: string;
+            status: string;
+            reason: string | null;
+        }>;
+        milestoneUpdates: Array<{
+            milestoneId: string;
+            status: string;
+            reason: string | null;
+        }>;
     }>;
     resolveAgentDisplayName: (agentId: string, fallbackName?: string | null) => Promise<string | null>;
     dispatchFallbackWorkstreamTurn: (input: {

package/dist/http/helpers/autopilot-operations.js

@@ -3,11 +3,37 @@ import { registerArtifact } from "../../artifacts/register-artifact.js";
 export function createAutopilotOperations(deps) {
     async function registerArtifactSafe(input) {
         const now = new Date().toISOString();
-        const name = (input.artifact.name ?? "").trim();
+        const name = typeof input.artifact.name === "string" && input.artifact.name.trim().length > 0
+            ? input.artifact.name.trim()
+            : "";
         if (!name)
             return { ok: false, id: null };
-        const artifactType = (input.artifact.artifact_type ?? "other").trim() || "other";
+        const artifactType = typeof input.artifact.artifact_type === "string" && input.artifact.artifact_type.trim().length > 0
+            ? input.artifact.artifact_type.trim()
+            : "other";
+        const confidenceScore = typeof input.artifact.confidence_score === "number" &&
+            Number.isFinite(input.artifact.confidence_score) &&
+            input.artifact.confidence_score >= 0 &&
+            input.artifact.confidence_score <= 1
+            ? input.artifact.confidence_score
+            : null;
         const artifactId = deps.randomUUID();
+        const milestoneId = (typeof input.artifact.milestone_id === "string" &&
+            input.artifact.milestone_id.trim().length > 0
+            ? input.artifact.milestone_id.trim()
+            : null) ??
+            (typeof input.fallbackMilestoneId === "string" &&
+                input.fallbackMilestoneId.trim().length > 0
+                ? input.fallbackMilestoneId.trim()
+                : null);
+        const taskIds = (Array.isArray(input.artifact.task_ids) && input.artifact.task_ids.length > 0
+            ? input.artifact.task_ids
+            : Array.isArray(input.fallbackTaskIds)
+                ? input.fallbackTaskIds
+                : [])
+            .filter((taskId) => typeof taskId === "string")
+            .map((taskId) => taskId.trim())
+            .filter(Boolean);
         const verificationSteps = Array.isArray(input.artifact.verification_steps)
             ? input.artifact.verification_steps
                 .filter((step) => typeof step === "string")
@@ -27,14 +53,15 @@ export function createAutopilotOperations(deps) {
         const createdByType = hasUuidAgent ? "agent" : "human";
         const createdById = hasUuidAgent ? input.agentId : null;
         try {
-            const entityType = input.artifact.milestone_id ? "milestone" : "initiative";
-            const entityId = input.artifact.milestone_id ? input.artifact.milestone_id : input.initiativeId;
+            const entityType = milestoneId ? "milestone" : "initiative";
+            const entityId = milestoneId ?? input.initiativeId;
             const result = await registerArtifact(deps.client, deps.client.getBaseUrl(), {
                 artifact_id: artifactId,
                 entity_type: entityType,
                 entity_id: entityId,
                 name,
                 artifact_type: artifactType,
+                confidence_score: confidenceScore,
                 created_by_type: createdByType,
                 created_by_id: createdById,
                 description,
@@ -47,12 +74,45 @@ export function createAutopilotOperations(deps) {
                     run_id: input.runId,
                     initiative_id: input.initiativeId,
                     workstream_id: input.workstreamId,
-                    milestone_id: input.artifact.milestone_id ?? null,
-                    task_ids: input.artifact.task_ids ?? null,
+                    milestone_id: milestoneId,
+                    task_ids: taskIds.length > 0 ? taskIds : null,
+                    confidence_score: confidenceScore,
                 },
                 // Make persistence validation opt-in to avoid adding latency to every slice by default.
                 validate_persistence: process.env.ORGX_VALIDATE_ARTIFACT_PERSISTENCE === "1",
             });
+            if (result.ok) {
+                try {
+                    await deps.emitActivitySafe({
+                        initiativeId: input.initiativeId,
+                        runId: input.runId,
+                        correlationId: input.runId,
+                        phase: "handoff",
+                        level: "info",
+                        message: `Artifact registered: ${name}`,
+                        progressPct: 100,
+                        metadata: {
+                            event: "artifact_registered",
+                            artifact_id: result.artifact_id ?? artifactId,
+                            artifact_type: artifactType,
+                            confidence_score: confidenceScore,
+                            initiative_id: input.initiativeId,
+                            workstream_id: input.workstreamId,
+                            milestone_id: milestoneId,
+                            task_ids: taskIds,
+                            agent_id: input.agentId,
+                            agent_name: input.agentName ?? null,
+                            executor_agent_id: input.agentId,
+                            executor_agent_name: input.agentName ?? null,
+                            source_run_id: input.runId,
+                            ...(input.isMockWorker ? { mock: true } : {}),
+                        },
+                    });
+                }
+                catch {
+                    // best effort
+                }
+            }
             return { ok: result.ok, id: result.artifact_id };
         }
         catch (err) {
@@ -63,10 +123,11 @@ export function createAutopilotOperations(deps) {
                     timestamp: now,
                     payload: {
                         artifact_id: artifactId,
-                        entity_type: input.artifact.milestone_id ? "milestone" : "initiative",
-                        entity_id: input.artifact.milestone_id ?? input.initiativeId,
+                        entity_type: milestoneId ? "milestone" : "initiative",
+                        entity_id: milestoneId ?? input.initiativeId,
                         name,
                         artifact_type: artifactType,
+                        confidence_score: confidenceScore,
                         created_by_type: createdByType,
                         created_by_id: createdById,
                         description,
@@ -93,9 +154,11 @@ export function createAutopilotOperations(deps) {
                             source: "openclaw_local_fallback",
                             event: "autopilot_slice_artifact_buffered",
                             artifact_type: artifactType,
+                            confidence_score: confidenceScore,
                             artifact_id: artifactId,
                             url: input.artifact.url ?? null,
                             error: deps.safeErrorMessage(err),
+                            ...(input.isMockWorker ? { mock: true } : {}),
                         },
                     },
                 });
@@ -161,11 +224,18 @@ export function createAutopilotOperations(deps) {
             return null;
         };
         const operations = [];
+        const normalizedTaskUpdates = [];
+        const normalizedMilestoneUpdates = [];
         for (const update of input.taskUpdates) {
             const taskId = (update?.task_id ?? "").trim();
             const status = normalizeTaskStatus(update?.status ?? "");
             if (!taskId || !status)
                 continue;
+            normalizedTaskUpdates.push({
+                taskId,
+                status,
+                reason: typeof update?.reason === "string" && update.reason.trim().length > 0 ? update.reason.trim() : null,
+            });
             operations.push({ op: "task.update", task_id: taskId, status });
         }
         for (const update of input.milestoneUpdates) {
@@ -173,28 +243,81 @@ export function createAutopilotOperations(deps) {
             const status = normalizeMilestoneStatus(update?.status ?? "");
             if (!milestoneId || !status)
                 continue;
+            normalizedMilestoneUpdates.push({
+                milestoneId,
+                status,
+                reason: typeof update?.reason === "string" && update.reason.trim().length > 0
+                    ? update.reason.trim()
+                    : null,
+            });
             operations.push({ op: "milestone.update", milestone_id: milestoneId, status });
         }
-        if (operations.length === 0)
-            return { applied: 0, buffered: false };
+        if (operations.length === 0) {
+            return {
+                applied: 0,
+                buffered: false,
+                taskUpdates: [],
+                milestoneUpdates: [],
+            };
+        }
+        const normalizedAgentId = typeof input.agentId === "string" && input.agentId.trim().length > 0
+            ? input.agentId.trim()
+            : null;
+        const normalizedAgentName = typeof input.agentName === "string" && input.agentName.trim().length > 0
+            ? input.agentName.trim()
+            : null;
+        const runIdForApply = input.runId?.trim() ?? "";
+        const correlationIdForApply = input.correlationId?.trim() ?? "";
+        const applyPayloadBase = {
+            initiative_id: input.initiativeId,
+            source_client: "openclaw",
+            idempotency_key: deps.idempotencyKey([
+                "openclaw",
+                "autopilot",
+                "slice_status",
+                input.initiativeId,
+                input.correlationId,
+            ]),
+            operations: operations,
+        };
         try {
             await deps.client.applyChangeset({
-                initiative_id: input.initiativeId,
-                run_id: input.runId,
-                correlation_id: input.correlationId,
-                source_client: "openclaw",
-                idempotency_key: deps.idempotencyKey([
-                    "openclaw",
-                    "autopilot",
-                    "slice_status",
-                    input.initiativeId,
-                    input.correlationId,
-                ]),
-                operations: operations,
+                ...applyPayloadBase,
+                run_id: runIdForApply || undefined,
+                correlation_id: correlationIdForApply || undefined,
             });
-            return { applied: operations.length, buffered: false };
+            return {
+                applied: operations.length,
+                buffered: false,
+                taskUpdates: normalizedTaskUpdates,
+                milestoneUpdates: normalizedMilestoneUpdates,
+            };
         }
         catch (err) {
+            const errMsg = deps.safeErrorMessage(err);
+            // Common local-path case: run_id is only known locally. Retry with
+            // correlation_id-only semantics so OrgX can attach deterministically.
+            if (runIdForApply &&
+                /^404\b/.test(errMsg) &&
+                /\brun\b/i.test(errMsg) &&
+                /not found/i.test(errMsg)) {
+                try {
+                    await deps.client.applyChangeset({
+                        ...applyPayloadBase,
+                        run_id: undefined,
+                        correlation_id: correlationIdForApply || `openclaw_run_${runIdForApply.replace(/-/g, "").slice(0, 24)}`,
+                    });
+                    return {
+                        applied: operations.length,
+                        buffered: false,
+                        taskUpdates: normalizedTaskUpdates,
+                        milestoneUpdates: normalizedMilestoneUpdates,
+                    };
+                }
+                catch {
+                    // Fall through to local outbox buffering.
+                }
+            }
             const timestamp = new Date().toISOString();
             try {
                 await appendToOutbox(input.initiativeId, {
@@ -220,12 +343,12 @@ export function createAutopilotOperations(deps) {
                         type: "run_started",
                         title: `Buffered status updates for slice ${input.runId}`,
                         description: null,
-                        agentId: null,
-                        agentName: null,
-                        requesterAgentId: null,
-                        requesterAgentName: null,
-                        executorAgentId: null,
-                        executorAgentName: null,
+                        agentId: normalizedAgentId,
+                        agentName: normalizedAgentName,
+                        requesterAgentId: normalizedAgentId,
+                        requesterAgentName: normalizedAgentName,
+                        executorAgentId: normalizedAgentId,
+                        executorAgentName: normalizedAgentName,
                         runId: input.runId,
                         initiativeId: input.initiativeId,
                         timestamp,
@@ -234,7 +357,30 @@ export function createAutopilotOperations(deps) {
                         metadata: {
                             source: "openclaw_local_fallback",
                             event: "autopilot_slice_status_updates_buffered",
-                            error: deps.safeErrorMessage(err),
+                            error: errMsg,
+                            run_id: input.runId,
+                            correlation_id: input.correlationId,
+                            output_path: typeof input.outputPath === "string" && input.outputPath.trim().length > 0
+                                ? input.outputPath.trim()
+                                : null,
+                            log_path: typeof input.logPath === "string" && input.logPath.trim().length > 0
+                                ? input.logPath.trim()
+                                : null,
+                            task_update_count: normalizedTaskUpdates.length,
+                            milestone_update_count: normalizedMilestoneUpdates.length,
+                            task_updates: normalizedTaskUpdates.map((entry) => ({
+                                task_id: entry.taskId,
+                                status: entry.status,
+                                reason: entry.reason,
+                            })),
+                            milestone_updates: normalizedMilestoneUpdates.map((entry) => ({
+                                milestone_id: entry.milestoneId,
+                                status: entry.status,
+                                reason: entry.reason,
+                            })),
+                            ...(normalizedAgentId ? { agent_id: normalizedAgentId } : {}),
+                            ...(normalizedAgentName ? { agent_name: normalizedAgentName } : {}),
+                            ...(input.isMockWorker ? { mock: true } : {}),
                         },
                     },
                 });
@@ -242,7 +388,12 @@ export function createAutopilotOperations(deps) {
             catch {
                 // best effort
             }
-            return { applied: operations.length, buffered: true };
+            return {
+                applied: operations.length,
+                buffered: true,
+                taskUpdates: normalizedTaskUpdates,
+                milestoneUpdates: normalizedMilestoneUpdates,
+            };
         }
     }
     async function resolveAgentDisplayName(agentId, fallbackName) {

package/dist/http/helpers/autopilot-runtime.d.ts

@@ -19,6 +19,7 @@ export declare function createAutopilotRuntime(deps: CreateAutopilotRuntimeDeps)
         cwd: string;
         logPath: string;
         outputPath: string;
+        outputSchemaPath?: string;
         env: Record<string, string | undefined>;
     }) => {
         pid: number | null;