npm - forge-openclaw-plugin - Versions diffs - 0.2.43 → 0.2.44 - Mend

forge-openclaw-plugin 0.2.43 → 0.2.44

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (14) hide show

package/dist/assets/index-DtEvFzXp.css +1 -0
package/dist/assets/index-s24CefIb.js +91 -0
package/dist/assets/index-s24CefIb.js.map +1 -0
package/dist/index.html +2 -2
package/dist/openclaw/tools.js +42 -10
package/dist/server/server/src/app.js +93 -8
package/dist/server/server/src/movement.js +2 -3
package/dist/server/server/src/openapi.js +19 -1
package/openclaw.plugin.json +1 -1
package/package.json +1 -1
package/skills/forge-openclaw/entity_conversation_playbooks.md +14 -4
package/dist/assets/index-Bqqi_RSB.js +0 -91
package/dist/assets/index-Bqqi_RSB.js.map +0 -1
package/dist/assets/index-IrxlatFN.css +0 -1

package/dist/index.html CHANGED Viewed

@@ -13,14 +13,14 @@
     />
     <link rel="icon" type="image/png" href="/forge/assets/favicon-BCHm9dUV.ico" />
     <link rel="alternate icon" href="/forge/assets/favicon-BCHm9dUV.ico" />
-    <script type="module" crossorigin src="/forge/assets/index-Bqqi_RSB.js"></script>
+    <script type="module" crossorigin src="/forge/assets/index-s24CefIb.js"></script>
     <link rel="modulepreload" crossorigin href="/forge/assets/vendor-D_NZFJze.js">
     <link rel="modulepreload" crossorigin href="/forge/assets/board-CAszQU7Y.js">
     <link rel="modulepreload" crossorigin href="/forge/assets/ui-B5MjRjKe.js">
     <link rel="modulepreload" crossorigin href="/forge/assets/motion-CU5aNClV.js">
     <link rel="modulepreload" crossorigin href="/forge/assets/table-CK0KcPYW.js">
     <link rel="stylesheet" crossorigin href="/forge/assets/vendor-DT3pnAKJ.css">
-    <link rel="stylesheet" crossorigin href="/forge/assets/index-IrxlatFN.css">
+    <link rel="stylesheet" crossorigin href="/forge/assets/index-DtEvFzXp.css">
   </head>
   <body class="bg-canvas text-ink antialiased">
     <div id="root"></div>

package/dist/openclaw/tools.js CHANGED Viewed

@@ -11,6 +11,46 @@ function jsonResult(payload) {
         details: payload
     };
 }
+function normalizeText(value) {
+    return typeof value === "string" ? value.trim() : "";
+}
+function normalizeOptionalText(value) {
+    const text = normalizeText(value);
+    return text.length > 0 ? text : undefined;
+}
+function normalizeTaskRunStartRequest(params) {
+    const taskId = normalizeText(params.taskId);
+    if (!taskId) {
+        throw new Error("forge_start_task_run requires a non-empty taskId.");
+    }
+    const actor = normalizeText(params.actor);
+    if (!actor) {
+        throw new Error("forge_start_task_run requires a non-empty actor.");
+    }
+    const timerMode = params.timerMode === "planned" ? "planned" : "unlimited";
+    const plannedDurationSeconds = typeof params.plannedDurationSeconds === "number" &&
+        Number.isInteger(params.plannedDurationSeconds)
+        ? params.plannedDurationSeconds
+        : null;
+    if (timerMode === "planned" && plannedDurationSeconds === null) {
+        throw new Error("forge_start_task_run requires plannedDurationSeconds when timerMode is planned.");
+    }
+    return {
+        taskId,
+        body: {
+            actor,
+            timerMode,
+            plannedDurationSeconds: timerMode === "planned" ? plannedDurationSeconds : null,
+            overrideReason: normalizeOptionalText(params.overrideReason),
+            isCurrent: typeof params.isCurrent === "boolean" ? params.isCurrent : undefined,
+            leaseTtlSeconds: typeof params.leaseTtlSeconds === "number" &&
+                Number.isInteger(params.leaseTtlSeconds)
+                ? params.leaseTtlSeconds
+                : undefined,
+            note: normalizeOptionalText(params.note)
+        }
+    };
+}
 async function runRead(config, path) {
     const result = await callConfiguredForgeApi(config, {
         method: "GET",
@@ -954,19 +994,11 @@ export function registerForgePluginTools(api, config) {
             note: Type.Optional(Type.String())
         }),
         async execute(_toolCallId, params) {
-            const typed = params;
+            const typed = normalizeTaskRunStartRequest(params);
             return jsonResult(await runWrite(config, {
                 method: "POST",
                 path: `/api/v1/tasks/${typed.taskId}/runs`,
-                body: {
-                    actor: typed.actor,
-                    timerMode: typed.timerMode,
-                    plannedDurationSeconds: typed.plannedDurationSeconds,
-                    overrideReason: typed.overrideReason,
-                    isCurrent: typed.isCurrent,
-                    leaseTtlSeconds: typed.leaseTtlSeconds,
-                    note: typed.note
-                }
+                body: typed.body
             }));
         }
     });

package/dist/server/server/src/app.js CHANGED Viewed

@@ -1980,6 +1980,29 @@ function buildPreferredMutationPath(entityType) {
             return "Read-only surface.";
     }
 }
+function buildPreferredMutationTool(entityType) {
+    if (entityType in AGENT_ONBOARDING_BATCH_ROUTE_BASES) {
+        return "forge_create_entities | forge_update_entities | forge_delete_entities | forge_search_entities";
+    }
+    switch (entityType) {
+        case "wiki_page":
+            return "forge_upsert_wiki_page";
+        case "calendar_connection":
+            return "forge_connect_calendar_provider | forge_sync_calendar_connection";
+        case "task_run":
+            return "forge_start_task_run | forge_heartbeat_task_run | forge_focus_task_run | forge_complete_task_run | forge_release_task_run";
+        case "questionnaire_run":
+            return "forge_start_questionnaire_run | forge_update_questionnaire_run | forge_complete_questionnaire_run";
+        case "preference_judgment":
+            return "forge_submit_preferences_judgment";
+        case "preference_signal":
+            return "forge_submit_preferences_signal";
+        case "work_adjustment":
+            return "forge_adjust_work_minutes";
+        default:
+            return null;
+    }
+}
 function buildPreferredReadPath(entityType) {
     if (entityType in AGENT_ONBOARDING_BATCH_ROUTE_BASES) {
         return AGENT_ONBOARDING_BATCH_ROUTE_BASES[entityType];
@@ -2024,11 +2047,10 @@ function enrichOnboardingEntityGuide(entry) {
             : null,
         preferredMutationPath: buildPreferredMutationPath(entry.entityType),
         preferredReadPath: buildPreferredReadPath(entry.entityType),
-        preferredMutationTool: classification === "batch_crud_entity"
-            ? "forge_create_entities | forge_update_entities | forge_delete_entities | forge_search_entities"
-            : classification === "specialized_domain_surface"
+        preferredMutationTool: buildPreferredMutationTool(entry.entityType) ??
+            (classification === "specialized_domain_surface"
                 ? "Follow forge_get_agent_onboarding.entityRouteModel.specializedDomainSurfaces for the dedicated route family."
-                : null
+                : null)
     };
 }
 const AGENT_ONBOARDING_ENTITY_CATALOG = [
@@ -2758,7 +2780,8 @@ const AGENT_ONBOARDING_CONVERSATION_RULES = [
     "For specialized surfaces, ask what would make the answer or change useful before you ask route-shaped details such as provider, weekday, flow id, run id, or trip id.",
     "When the user already named a precise correction or review target, do not widen back out into a meta lane question. Confirm only the missing route-selecting detail and then act.",
     "Once the route family is clear, say it plainly enough that another agent could follow the same path without guessing.",
-    "For Movement specifically, treat missing-data corrections as user-defined overlay boxes unless the user is editing an already-recorded stay or trip. When the user already gave a clear instruction like 'that missing block was home', act after only the last ambiguity is resolved."
+    "For Movement specifically, treat missing-data corrections as user-defined overlay boxes unless the user is editing an already-recorded stay or trip. When the user already gave a clear instruction like 'that missing block was home', act after only the last ambiguity is resolved.",
+    "For meaning-bearing updates, especially in Psyche, briefly say what feels newly true before you ask for the one structural detail that still changes the save."
 ];
 const AGENT_ONBOARDING_ENTITY_CONVERSATION_PLAYBOOKS = [
     {
@@ -2917,7 +2940,8 @@ const AGENT_ONBOARDING_ENTITY_CONVERSATION_PLAYBOOKS = [
             "Confirm the task.",
             "Confirm the actor only if it is not already obvious.",
             "Ask whether the run should be planned or unlimited only if that changes the action.",
-            "Start the run instead of turning it into a longer intake."
+            "Start the run instead of turning it into a longer intake.",
+            "Use the dedicated task-run tool for start, heartbeat, focus, complete, and release work instead of bouncing to the UI or a generic web route."
         ]
     },
     {
@@ -3071,6 +3095,7 @@ const AGENT_ONBOARDING_ENTITY_CONVERSATION_PLAYBOOKS = [
             "Ask whether the focus is a stay, a trip, a place, a timeline window, or a selected span.",
             "Ask for the time window, place, or movement item that makes the question concrete.",
             "Ask what they are trying to notice, preserve, or answer through that movement context.",
+            "Choose the dedicated day, month, all-time, timeline, places, trip-detail, or selection route once the question shape is clear.",
             "Skip the meta lane question when the user already named the exact correction or review target and only one ambiguity remains.",
             "If the request is filling a missing-data gap, use a user-defined movement box rather than a raw stay or trip patch.",
             "If the request is repairing already-saved movement data, use the repair route that matches the saved object instead of treating it like a missing span.",
@@ -3104,6 +3129,7 @@ const AGENT_ONBOARDING_ENTITY_CONVERSATION_PLAYBOOKS = [
             "Ask whether they need the flow contract, a run result, a published output, or a node result.",
             "Ask what the user is trying to learn, repair, or publish through that flow.",
             "If the user already named the flow and action clearly, skip the meta lane question and ask only for the missing run, node, or output scope.",
+            "If the user wants a stable public input contract or published output, prefer those dedicated reads instead of detouring through run history first.",
             "If the user is still shaping a payload or edit, prefer flow detail or box catalog reads before asking for structured inputs.",
             "Prefer flow detail or published-output reads for stable contracts, and use run or node-result routes only when the user is asking about execution history or debugging.",
             "Route to the dedicated workbench route family once the execution lane is clear."
@@ -3853,6 +3879,61 @@ const AGENT_ONBOARDING_TOOL_INPUT_CATALOG = [
         example: '{"taskRunId":"run_123","actor":"aurel","note":"Stopping for now; blocked on feedback","closeoutNote":{"contentMarkdown":"Blocked on feedback from design before I can continue."}}'
     }
 ];
+const VALIDATION_ROUTE_TOOL_MAP = {
+    "POST /api/v1/entities/search": "forge_search_entities",
+    "POST /api/v1/entities/create": "forge_create_entities",
+    "POST /api/v1/entities/update": "forge_update_entities",
+    "POST /api/v1/entities/delete": "forge_delete_entities",
+    "POST /api/v1/entities/restore": "forge_restore_entities",
+    "POST /api/v1/tasks/:id/runs": "forge_start_task_run",
+    "POST /api/v1/task-runs/:id/heartbeat": "forge_heartbeat_task_run",
+    "POST /api/v1/task-runs/:id/focus": "forge_focus_task_run",
+    "POST /api/v1/task-runs/:id/complete": "forge_complete_task_run",
+    "POST /api/v1/task-runs/:id/release": "forge_release_task_run",
+    "POST /api/tasks/:id/runs": "forge_start_task_run",
+    "POST /api/task-runs/:id/heartbeat": "forge_heartbeat_task_run",
+    "POST /api/task-runs/:id/focus": "forge_focus_task_run",
+    "POST /api/task-runs/:id/complete": "forge_complete_task_run",
+    "POST /api/task-runs/:id/release": "forge_release_task_run"
+};
+function formatValidationSummary(issues) {
+    return issues
+        .slice(0, 3)
+        .map((issue) => `${issue.path}: ${issue.message}`)
+        .join("; ");
+}
+function buildValidationHelp(method, routeUrl, issues) {
+    const route = routeUrl || "unknown_route";
+    const toolName = VALIDATION_ROUTE_TOOL_MAP[`${method.toUpperCase()} ${route}`];
+    const toolInput = toolName
+        ? AGENT_ONBOARDING_TOOL_INPUT_CATALOG.find((entry) => entry.toolName === toolName)
+        : undefined;
+    return {
+        route,
+        validationSummary: formatValidationSummary(issues),
+        ...(toolInput
+            ? {
+                expectedShape: {
+                    toolName: toolInput.toolName,
+                    inputShape: toolInput.inputShape,
+                    requiredFields: [...toolInput.requiredFields],
+                    example: toolInput.example,
+                    notes: [...toolInput.notes]
+                }
+            }
+            : {
+                expectedShape: {
+                    inputShape: "See details[] for the rejected fields on this route.",
+                    requiredFields: [],
+                    example: null,
+                    notes: [
+                        "Retry with only the documented top-level keys for this route.",
+                        "Omit optional fields instead of sending null unless the route contract explicitly allows null."
+                    ]
+                }
+            })
+    };
+}
 function buildAgentOnboardingPayload(request) {
     const origin = getRequestOrigin(request);
     return {
@@ -5075,6 +5156,10 @@ export async function buildServer(options = {}) {
     });
     app.setErrorHandler((error, request, reply) => {
         const validationIssues = error instanceof ZodError ? formatValidationIssues(error) : undefined;
+        const routeUrl = request.routeOptions.url || request.url;
+        const validationHelp = validationIssues
+            ? buildValidationHelp(request.method, routeUrl, validationIssues)
+            : undefined;
         const statusCode = isHttpError(error)
             ? error.statusCode
             : isManagerError(error)
@@ -5082,7 +5167,6 @@ export async function buildServer(options = {}) {
                 : error instanceof ZodError
                     ? 400
                     : 500;
-        const routeUrl = request.routeOptions.url || request.url;
         if (!shouldSkipAutomaticDiagnosticRoute(routeUrl)) {
             try {
                 recordDiagnosticLog({
@@ -5123,10 +5207,11 @@ export async function buildServer(options = {}) {
                         ? "invalid_request"
                         : "internal_error",
             error: validationIssues
-                ? "Request validation failed"
+                ? `Request validation failed for ${request.method.toUpperCase()} ${routeUrl}. ${validationHelp?.validationSummary ?? ""}`.trim()
                 : getErrorMessage(error),
             statusCode,
             ...(validationIssues ? { details: validationIssues } : {}),
+            ...(validationHelp ?? {}),
             ...(isHttpError(error) && error.details ? error.details : {}),
             ...(isManagerError(error) && error.details ? error.details : {})
         });

package/dist/server/server/src/movement.js CHANGED Viewed

@@ -2780,10 +2780,9 @@ function updateMovementBoxOverrideState(id, input) {
            overridden_user_box_ids_json = ?,
            override_ranges_json = ?,
            is_overridden = ?,
-           is_fully_hidden = ?,
-           updated_at = ?
+           is_fully_hidden = ?
        WHERE id = ?`)
-        .run(input.overrideCount, JSON.stringify(input.overriddenAutomaticBoxIds), input.trueStartedAt, input.trueEndedAt, input.overriddenStartedAt, input.overriddenEndedAt, input.overriddenByBoxId, JSON.stringify(input.overriddenUserBoxIds), JSON.stringify(input.overrideRanges), input.isOverridden ? 1 : 0, input.isFullyHidden ? 1 : 0, nowIso(), id);
+        .run(input.overrideCount, JSON.stringify(input.overriddenAutomaticBoxIds), input.trueStartedAt, input.trueEndedAt, input.overriddenStartedAt, input.overriddenEndedAt, input.overriddenByBoxId, JSON.stringify(input.overriddenUserBoxIds), JSON.stringify(input.overrideRanges), input.isOverridden ? 1 : 0, input.isFullyHidden ? 1 : 0, id);
 }
 function recomputeMovementBoxOverrideState(userId) {
     const rows = listMovementBoxRows({ userIds: [userId] });

package/dist/server/server/src/openapi.js CHANGED Viewed

@@ -333,6 +333,18 @@ export function buildOpenApiDocument() {
             message: { type: "string" }
         }
     };
+    const validationExpectedShape = {
+        type: "object",
+        additionalProperties: true,
+        required: ["inputShape", "requiredFields", "notes"],
+        properties: {
+            toolName: { type: "string" },
+            inputShape: { type: "string" },
+            requiredFields: arrayOf({ type: "string" }),
+            example: { type: ["string", "null"] },
+            notes: arrayOf({ type: "string" })
+        }
+    };
     const errorResponse = {
         type: "object",
         additionalProperties: true,
@@ -341,7 +353,12 @@ export function buildOpenApiDocument() {
             code: { type: "string" },
             error: { type: "string" },
             statusCode: { type: "integer" },
-            details: arrayOf({ $ref: "#/components/schemas/ValidationIssue" })
+            route: { type: "string" },
+            validationSummary: { type: "string" },
+            details: arrayOf({ $ref: "#/components/schemas/ValidationIssue" }),
+            expectedShape: {
+                $ref: "#/components/schemas/ValidationExpectedShape"
+            }
         }
     };
     const tag = {
@@ -4098,6 +4115,7 @@ export function buildOpenApiDocument() {
         components: {
             schemas: {
                 ValidationIssue: validationIssue,
+                ValidationExpectedShape: validationExpectedShape,
                 ErrorResponse: errorResponse,
                 Tag: tag,
                 Goal: goal,

package/openclaw.plugin.json CHANGED Viewed

@@ -2,7 +2,7 @@
   "id": "forge-openclaw-plugin",
   "name": "Forge",
   "description": "Curated OpenClaw adapter for the Forge collaboration API, UI entrypoint, and localhost auto-start runtime.",
-  "version": "0.2.43",
+  "version": "0.2.44",
   "skills": [
     "./skills"
   ],

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "forge-openclaw-plugin",
-  "version": "0.2.43",
+  "version": "0.2.44",
   "description": "Curated OpenClaw adapter for the Forge collaboration API, UI entrypoint, and localhost auto-start runtime.",
   "type": "module",
   "license": "MIT",

package/skills/forge-openclaw/entity_conversation_playbooks.md CHANGED Viewed

@@ -61,6 +61,9 @@ Forge correctly, and gather only the structure that still matters.
   and then act.
 - Once the route family is clear, say it plainly enough that another agent could follow
   the same path without guessing.
+- For meaning-bearing updates, especially in Psyche-adjacent work, briefly say what
+  feels newly true before you ask for the one structural detail that still changes the
+  save.
 - Do not read schema fields out loud unless the user explicitly wants a checklist.
 - One focused question is the default. Ask two only when both questions serve the same
   job and the user is steady enough for it.
@@ -780,6 +783,7 @@ Arc:
 2. Confirm the actor only if it is not already obvious.
 3. Ask whether the run should be planned or unlimited only if that changes the action.
 4. Start the run instead of turning it into intake.
+5. Use the dedicated task-run tool for start, heartbeat, focus, complete, and release work. Do not bounce to the Forge UI, a browser session, or a generic web route for those actions unless the user explicitly wants the visual surface.
 Ready to start when:
@@ -1011,9 +1015,11 @@ Arc:
 3. Ask whether the focus is a stay, a trip, a place, a timeline window, or a selected span.
 4. Ask for the time window, place, or movement item that makes the question concrete.
 5. Ask what they are trying to notice, preserve, or answer through that movement context.
-6. Skip the meta lane question when the user already named the exact correction or
+6. Choose the dedicated day, month, all-time, timeline, places, trip-detail, or
+   selection route once the question shape is clear.
+7. Skip the meta lane question when the user already named the exact correction or
    review target and only one ambiguity remains.
-7. Route to the dedicated movement read or write path once the surface is clear.
+8. Route to the dedicated movement read or write path once the surface is clear.
 Direct action rules:
@@ -1101,7 +1107,9 @@ Arc:
    when the lane is still blurred.
 6. If the user already named the life-force lane clearly, skip the meta lane question
    and ask only for the specific weekday, profile field, or signal that still matters.
-7. Route to the dedicated life-force path once the lane is clear.
+7. If the user wants to see what changed after a write, read the overview back instead
+   of leaving the result implicit.
+8. Route to the dedicated life-force path once the lane is clear.
 Helpful follow-up lanes:
@@ -1156,7 +1164,9 @@ Arc:
 4. Ask whether they need the flow contract, a run result, a published output, or a node result.
 5. If the user already named the flow and action clearly, skip the meta lane
    question and ask only for the missing run, node, or output scope.
-6. Route to the dedicated workbench route family once the execution lane is clear.
+6. If the user wants a stable public input contract or published output, prefer those
+   dedicated reads instead of detouring through run history first.
+7. Route to the dedicated workbench route family once the execution lane is clear.
 Helpful follow-up lanes: