agentlife 2.6.11 → 2.6.13

package/dist/index.js

@@ -908,12 +908,9 @@ All inter-system signals are fixed strings. Nothing else is a signal.
 **Work-order markers — you RECEIVE (always process, never treat as end-of-conversation):**
 - \`[action:<name>] surfaceId=<id> [key=value ...]\` — user tapped a button on a widget you own. \`<name>\` is whatever you put in \`action=\` when you pushed the widget. Process it, then update or delete the source widget in the same turn.
 - \`[system:dismiss-requested] surfaceId=<id>\` — user tapped dismiss on a widget. You have ~30s to push an \`input\` surface with contextual alternatives before the platform finalizes the dismiss.
-- \`[event:dismissed] surfaceId=<id>\` — dismiss completed. Widget is already gone — do NOT push \`delete\`. Clean up any custom infra tied to the surface, then \`NO_REPLY\`.
+- \`[event:dismissed] surfaceId=<id>\` — the platform deleted a widget without your involvement (non-guided dismiss path). The widget is already gone — do NOT push \`delete\`. Clean up any custom infra tied to the surface, then \`NO_REPLY\`. Does NOT fire for widgets you deleted yourself with the \`delete\` DSL — no notification is needed when you initiated the delete.
 - \`[system:cancelled]\` — user cancelled the turn mid-flight. Reply \`NO_REPLY\` and stop.
-
-**Input-surface control signals — platform EMITS when the user closes/skips an input surface:**
-- \`input_closed\` — user tapped the [×] on an input surface. Push it again later if still relevant.
-- \`input_skipped\` — user tapped Skip. Do not re-ask this session.
+- \`[system] <free-form text>\` — platform-generated instruction (scheduled followup firing, Supervisor notification, quality/regression signal). The \`<text>\` after \`[system] \` is the instruction; read it and act. No fixed playbook — the text is the playbook.
 
 **Input-surface \`action=\` values — canonical set (anything else is a QUALITY ERROR on input surfaces):**
 - \`action=choice\` or \`action=select\` → numbered single-select (one tap fires). Dispatched as \`[action:choice] label=<label>\`.
@@ -954,7 +951,7 @@ A widget is a **goal** — a persistent objective that lives on the dashboard un
 
 **Goal validation:** could you have written this goal text BEFORE doing the work? If yes, it's a template — the goal must be derived from your findings. For one-shot operations, the goal is the unresolved question, decision, or opportunity your work surfaces.
 
-**Face vs detail.** Face = metrics, badges, gauges, one-line headers (2-second glance). Prose, analysis, lists → \`detail:\`.
+**Face vs detail.** Face = metrics, badges, gauges, one-line headers (2-second glance). \`detail:\` = markdown: analysis, trends, patterns, next-step context, personality. Every final (non-loading, non-input) widget MUST include \`detail:\` — that is where your expertise and voice live. Without it the widget is just a receipt and the platform enforces with a QUALITY ERROR. If you genuinely have nothing meaningful to add beyond the face, the goal is too shallow — rethink the goal before pushing an empty-detail widget.
 
 ### surfaceId
 
@@ -993,7 +990,7 @@ Button shapes and dispatch format for input surfaces are in **Signal Protocol**
 
 **One widget, one goal.** Set at creation, MUST NOT change. If an action shifts the objective, delete and create with a new surfaceId. Platform rejects goal mutations with a QUALITY ERROR.
 
-**User-initiated dismiss** (\`[system:dismiss-requested] surfaceId=<id>\`): you have ~30s to push an \`input\` surface with 2-3 contextual options + "Remove it" last. If the user picks an alternative, act on it and the dismiss is cancelled. If the user picks "Remove it" or skips, \`[event:dismissed]\` arrives — the widget is already gone. See \`DISMISS_ALTERNATIVES_GUIDANCE\` (injected on the dismiss event) for crafting rules.
+**User-initiated dismiss** (\`[system:dismiss-requested] surfaceId=<id>\`): you have ~30s to push an \`input\` surface with 2-3 contextual options + "Remove it" last. If the user picks an alternative, act on it and the dismiss is cancelled. If the user picks "Remove it" or the followup fires on timeout, push \`delete <parentSurfaceId>\` to finalize removal (and delete the dismiss-alt input surface in the same block). See \`DISMISS_ALTERNATIVES_GUIDANCE\` (injected on the dismiss event) for crafting rules.
 
 ### Autonomy & approval
 
@@ -1223,7 +1220,11 @@ The user is dismissing this widget. You have ~30s to push an \`input\` surface w
 - "Remove it" is always last. Use \`action=choice\` so it renders as a numbered list.
 - surfaceId: \`dismiss-alt-{parent-surfaceId}\`. Goal: reference the parent. Followup: ~1m so the flow doesn't stall if the user walks away.
 
-If the user picks an alternative → act on it; the dismiss is cancelled. If they pick "Remove it" or skip → \`[event:dismissed]\` arrives, the widget is already gone, clean up any custom infra.`;
+If the user picks an alternative → act on it; the dismiss is cancelled.
+
+If the user picks "Remove it" → push \`delete <parentSurfaceId>\` in the same turn (and delete the dismiss-alt input surface alongside it; both go in one \`agentlife_push\` with \`---\` between them). No \`[event:dismissed]\` fires in the guided flow — you initiated the delete, so no notification is needed. Clean up any custom infra inline before the delete.
+
+If the 1-minute followup fires with no user response → same move: push \`delete <parentSurfaceId>\` + \`delete dismiss-alt-<parentSurfaceId>\`.`;
 var ORCHESTRATOR_AGENTS_MD = `# AgentLife Orchestrator
 
 You are a message router. You never answer questions, perform tasks, push widgets, or produce content.
@@ -1918,6 +1919,27 @@ function matchMetadata(block, key) {
   }
   return v || null;
 }
+function hasDetailContent(rawDsl) {
+  const lines = rawDsl.split(`
+`);
+  for (let i = 0;i < lines.length; i++) {
+    const m = lines[i].match(/^\s*detail:\s*(.*)$/);
+    if (!m)
+      continue;
+    if (m[1].trim())
+      return true;
+    for (let j = i + 1;j < lines.length; j++) {
+      const next = lines[j];
+      if (next.trim() === "")
+        continue;
+      if (/^\s/.test(next))
+        return true;
+      return false;
+    }
+    return false;
+  }
+  return false;
+}
 
 // dsl/validate.ts
 var KNOWN_BODY_KEYWORDS = new Set([
@@ -4724,7 +4746,7 @@ ${parts.join(`
   });
 }
 var QUALITY_CHECK_MAX_WAIT_MS = 5000;
-var TERMINAL_MARKERS = new Set(["NO_REPLY", "ANNOUNCE_SKIP", "REPLY_SKIP", "HEARTBEAT_OK", "done"]);
+var TERMINAL_MARKERS = new Set(["NO_REPLY", "ANNOUNCE_SKIP", "REPLY_SKIP", "HEARTBEAT_OK"]);
 function processPendingQualityChecks(state) {
   if (state.disabled || !state.historyDb)
     return 0;
@@ -6316,6 +6338,7 @@ function registerWidgetPushTool(api, state2) {
         const errors = [];
         const goalChanges = [];
         const missingFollowup = [];
+        const missingDetail = [];
         for (const r of results) {
           if (r.kind !== "push")
             continue;
@@ -6334,11 +6357,19 @@ function registerWidgetPushTool(api, state2) {
               data: JSON.stringify({ issue: "missing_followup", surfaceId: r.surfaceId })
             });
           }
+          if (view && view.state !== "loading" && !view.isInput && !hasDetailContent(view.rawDsl) && !goalChanges.includes(r.surfaceId) && !missingFollowup.includes(r.surfaceId)) {
+            missingDetail.push(r.surfaceId);
+            recordActivity(state2, "quality_warning", sessionKey, agentId, {
+              data: JSON.stringify({ issue: "missing_detail", surfaceId: r.surfaceId })
+            });
+          }
         }
         if (goalChanges.length > 0)
           errors.push(buildGoalChangedError(goalChanges));
         if (missingFollowup.length > 0)
           errors.push(buildMissingFollowupError(missingFollowup));
+        if (missingDetail.length > 0)
+          errors.push(buildMissingDetailError(missingDetail));
         const validationErrors = collectValidationErrors(state2, results, sessionKey, agentId);
         errors.push(...validationErrors);
         if (ownershipViolations.length > 0) {
@@ -6370,6 +6401,9 @@ function buildGoalChangedError(ids) {
 function buildMissingFollowupError(ids) {
   return `[QUALITY ERROR] ${ids.join(", ")}: missing followup. Every widget MUST have a followup — it is the widget's heartbeat and ownership signal. Push again with a followup: line, or delete the widget if work is done.`;
 }
+function buildMissingDetailError(ids) {
+  return `[QUALITY ERROR] ${ids.join(", ")}: missing detail. Every final widget MUST include a \`detail:\` block — it's where your analysis, trends, and voice live. The face alone is just a receipt. Push again with a detail: block containing analysis/personality/next-step context appropriate to the domain.`;
+}
 function collectValidationErrors(state2, results, sessionKey, agentId) {
   const errors = [];
   const unknownKeywordHits = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentlife",
-  "version": "2.6.11",
+  "version": "2.6.13",
   "type": "module",
   "scripts": {
     "build": "bun build index.ts --outfile dist/index.js --target node --external openclaw/plugin-sdk",