fix: include user message in auto-capture when prePromptMessageCount excludes it (#1248)

[r266-tech]

Problem

The hook's auto-capture only captures the assistant reply, not the user's original message. This causes the VLM extractor to receive text with only an [assistant] role message (a recap/acknowledgment), which yields 0 extractable memories.

Root Cause

OpenClaw's (used as in ) points to an index that may exclude the current turn's user message. When is too high, the user message falls outside the extraction window.

Fix

1. text-utils.ts: Add optional parameter to . When true, after normal extraction, check if any user message exists in the extracted results. If not, find the last user message in the full transcript and prepend it.

2. index.ts: Pass when calling from the handler.

This is a defensive fix — it ensures the user message is always captured even if the OpenClaw host passes an incorrect value.

Testing

Verified the logic:

• Normal case (user msg already captured): no duplicate, behavior unchanged
• Bug case (user msg before startIndex): last user msg prepended to results
• Edge case (no user msg in transcript): no crash, returns existing results

Fixes #1248

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 1248 - Fully compliant Compliant requirements: Added defensive fix to include last user message when missing from extraction window Modified afterTurn handler to use the new parameter

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🏅 Score: 78

🧪 No relevant tests

🔒 No security concerns identified

✅ No TODO sections

🔀 No multiple PR themes

⚡ Recommended focus areas for review Incomplete Array Content Handling When prepending the last user message with array-style content, only the first text block is added instead of all text blocks. This is inconsistent with the original loop's behavior which adds all text blocks from array content. } else if (Array.isArray(content)) { for (const block of content) { const b = block as Record<string, unknown>; if (b?.type === "text" && typeof b.text === "string" && b.text.trim()) { lastUserText = `[user]: ${b.text.trim()}`; break; } } Incorrect Count for Multiple Text Blocks When the last user message has multiple text blocks, count is only incremented by 1 instead of the number of text blocks added. This leads to an incorrect newCount return value. if (lastUserText && !texts.some((t) => t.startsWith("[user]:"))) { texts.unshift(lastUserText); count++; }

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Extract shared message parsing helper Extract message content parsing into a shared helper to avoid duplicating logic across extractNewTurnTexts and the new last-user-message block. This reduces maintenance overhead and ensures consistent behavior. examples/openclaw-memory-plugin/text-utils.ts [355-382] -+ // Defensive fix: ensure the last user message is included even if -+ // startIndex skipped it (OpenClaw prePromptMessageCount off-by-one) -+ if (ensureLastUserMessage && messages.length > 0) { -+ let lastUserText: string | null = null; -+ for (let i = messages.length - 1; i >= 0; i--) { -+ const msg = messages[i] as Record<string, unknown>; -+ if (!msg || typeof msg !== "object") continue; -+ if (msg.role === "user") { -+ const content = msg.content; -+ if (typeof content === "string" && content.trim()) { -+ lastUserText = `[user]: ${content.trim()}`; -+ } else if (Array.isArray(content)) { -+ for (const block of content) { -+ const b = block as Record<string, unknown>; -+ if (b?.type === "text" && typeof b.text === "string" && b.text.trim()) { -+ lastUserText = `[user]: ${b.text.trim()}`; -+ break; -+ } -+ } -+ } -+ break; -+ } -+ } -+ if (lastUserText && !texts.some((t) => t.startsWith("[user]:"))) { -+ texts.unshift(lastUserText); -+ count++; -+ } -+ } + // Add this helper function above extractNewTurnTexts + function getFormattedMessageText(msg: Record<string, unknown>): string | null { + const role = msg.role as string; + const content = msg.content; + if (typeof content === "string" && content.trim()) { + return `[${role}]: ${content.trim()}`; + } else if (Array.isArray(content)) { + for (const block of content) { + const b = block as Record<string, unknown>; + if (b?.type === "text" && typeof b.text === "string" && b.text.trim()) { + return `[${role}]: ${b.text.trim()}`; + } + } + } + return null; + } + // Then use it in extractNewTurnTexts main loop and new block: + for (let i = startIndex; i < messages.length; i++) { + const msg = messages[i] as Record<string, unknown>; + if (!msg || typeof msg !== "object") continue; + const formatted = getFormattedMessageText(msg); + if (formatted) { + texts.push(formatted); + count++; + } + } + + // Defensive fix: ensure the last user message is included even if + // startIndex skipped it (OpenClaw prePromptMessageCount off-by-one) + if (ensureLastUserMessage && messages.length > 0) { + let lastUserText: string | null = null; + for (let i = messages.length - 1; i >= 0; i--) { + const msg = messages[i] as Record<string, unknown>; + if (!msg || typeof msg !== "object") continue; + if (msg.role === "user") { + lastUserText = getFormattedMessageText(msg); + break; + } + } + if (lastUserText && !texts.some((t) => t.startsWith("[user]:"))) { + texts.unshift(lastUserText); + count++; + } + } + Suggestion importance[1-10]: 6 __ Why: This suggestion reduces code duplication by extracting shared message parsing logic into a helper, improving maintainability and consistency. It's a valid moderate-improvement suggestion without critical functionality changes.	Low

[CLAassistant]

All committers have signed the CLA.