Best possible solution:

Close this duplicate branch and continue review on #94401 as the canonical session-memory transcript-only assistant filter for the linked user bug.

Do we have a high-confidence way to reproduce the issue?

Yes at source level: current main extracts text from every accepted assistant JSONL row, so raw thinking output and cleaned transcript-only assistant rows can both enter generated session-memory content. I did not run tests or live provider flows because this review is read-only.

Is this the best way to solve the issue?

No, this branch is no longer the best landing path. The newer canonical PR uses the existing shared transcript-only assistant predicate, has focused coverage and proof-sufficient labeling, and avoids keeping another parentId/text dedupe implementation open.

Security review:

Security review cleared: The diff only changes local transcript filtering and tests; it does not touch dependencies, workflows, secrets, permissions, network calls, or code-execution surfaces.

AGENTS.md: found and applied where relevant.

What I checked:

• Repository policy read: Root AGENTS.md was read fully; no scoped AGENTS.md owns src/hooks, so the root PR-review, duplicate, proof, and session-state guidance applies. (AGENTS.md:1, 23b8f5d03782)
• Current main still has the original reader behavior: Current main accepts user/assistant messages, extracts text, and appends it without filtering duplicate or transcript-only assistant rows. (src/hooks/bundled/session-memory/transcript.ts:44, 23b8f5d03782)
• Session-memory output path is user-visible: The hook reads recent transcript content through getRecentSessionContentWithResetFallback before writing generated memory files for the /new or /reset path. (src/hooks/bundled/session-memory/handler.ts:217, 23b8f5d03782)
• Shared semantic predicate exists on main: Current main already centralizes transcript-only OpenClaw assistant rows as assistant messages from provider openclaw with delivery-mirror or gateway-injected models. (src/shared/transcript-only-openclaw-assistant.ts:17, 23b8f5d03782)
• Canonical replacement is viable: Live GitHub data shows fix(session-memory): skip transcript-only assistant messages in getRecentSessionContent #94401 is open, mergeable, clean, maintainer-modifiable, proof-sufficient labeled, and currently changes only the session-memory transcript reader and focused handler test coverage. (4abfeb47edb3)
• Canonical replacement uses the shared helper: The replacement PR imports isTranscriptOnlyOpenClawAssistantMessage and skips transcript-only assistant rows before extracting session-memory text, with focused coverage for delivery-mirror and gateway-injected rows. (src/hooks/bundled/session-memory/transcript.ts:46, 4abfeb47edb3)

Likely related people:

• Vincent Koc: Current blame on the session-memory transcript helper and shared transcript-only assistant helper points to recent current-main work. (role: recent area contributor; confidence: high; commits: 78b561807134; files: src/hooks/bundled/session-memory/transcript.ts, src/shared/transcript-only-openclaw-assistant.ts)
• Peter Steinberger: History shows the internal hooks system introduced the session-memory hook surface and later runtime seam work in this area. (role: introduced behavior and adjacent owner; confidence: high; commits: faba508fe0ae, 8727338372b4; files: src/hooks/bundled/session-memory/handler.ts, src/hooks/bundled/session-memory/HOOK.md, src/hooks/bundled/session-memory/transcript.ts)
• Tomas Hajek: Added the rotated transcript fallback used by the same session-memory reader path after /new. (role: adjacent transcript fallback contributor; confidence: medium; commits: 19ae7a4e17d8; files: src/hooks/bundled/session-memory/handler.ts, src/hooks/bundled/session-memory/handler.test.ts, src/hooks/bundled/session-memory/transcript.ts)
• NianJiuZst: Reviewed the newer replacement PR and steered it toward the shared transcript-only helper plus focused regression coverage. (role: reviewer of canonical replacement; confidence: medium; files: src/hooks/bundled/session-memory/transcript.ts, src/hooks/bundled/session-memory/handler.test.ts)

Codex review notes: model internal, reasoning high; reviewed against 23b8f5d03782.