Codex review: found issues before merge. Reviewed July 4, 2026, 12:32 AM ET / 04:32 UTC.

Summary
The PR adds eager Gateway/WebChat user-turn transcript persistence, Pi suppression metadata, and tests/docs for avoiding duplicate persistence across rollover, media, and fallback paths.

PR surface: Source +519, Tests +916, Docs +1. Total +1436 across 17 files.

Reproducibility: no. high-confidence live current-main reproduction was run in this read-only review. Source inspection plus the canonical issue show the timing class around chat.send ACK before durable user-turn visibility, and the PR branch still skips the busy same-session case.

Review metrics: 2 noteworthy metrics.

• Internal persistence handoff fields: 3 added. The PR adds suppression, session-id, and entry-id fields through internal reply/runner contracts, so maintainers need the handoff aligned with current recorder ownership before merge.
• Live mergeability: dirty, rebaseable=false. The current branch cannot land as-is and needs a current-main port before functional review can become a land-ready verdict.

Stored data model
Persistent data-model change detected: persistent cache schema: src/gateway/server.chat.gateway-server-chat-b.test.ts, serialized state: src/agents/pi-embedded-runner/run/attempt.prompt-helpers.test.ts, serialized state: src/agents/session-tool-result-guard-wrapper.ts, serialized state: src/gateway/server.chat.gateway-server-chat-b.test.ts, serialized state: src/gateway/session-utils.ts. Confirm migration or upgrade compatibility proof before merge.

Merge readiness
Overall: 🧂 unranked krab
Proof: 🦞 diamond lobster
Patch quality: 🧂 unranked krab
Result: blocked by patch quality or review findings.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

• Port the implementation onto current main's shared user-turn recorder/session-accessor boundary.
• [P2] Add or preserve a focused test for the busy same-session path before chat.send dispatch.
• Refresh mergeability and rerun the targeted Gateway/embedded-runner checks.

Mantis proof suggestion
A short WebChat proof would help confirm visible send latency and duplicate suppression after a current-main port. A maintainer can ask Mantis to capture proof by posting this exact PR comment:

@openclaw-mantis visual task: verify a WebChat user message appears immediately after chat.send ACK while an existing same-session run is busy, and that the transcript has no duplicate user turn.

Risk before merge

• [P1] The branch is dirty and rebaseable=false, so maintainers cannot land it without a current-main port.
• [P1] The PR's eager persistence gate skips active same-session runs, leaving the highest-risk stuck/queued lane timing class unresolved.
• [P1] Merging an older parallel transcript-persistence design could duplicate, stale, or misorder user turns against the current shared recorder path.
• [P1] Changing the chat.send ACK/durable boundary affects latency, hooks, rollover behavior, duplicate suppression, and message delivery semantics.

Maintainer options:

1. Port Through Shared Recorder (recommended)
   Rebase onto current main and route durable WebChat user-turn persistence through the shared recorder/session-accessor boundary instead of the branch's parallel eager append path.
2. Pause For Contract Decision
   If maintainers have not settled whether chat.send ACK should require durable user-turn persistence, pause this branch and review the canonical issue first.
3. Accept Runtime-Owned Durability
   Maintainers could intentionally keep delayed runtime-owned persistence, but then this PR should not claim to fix the remaining WebChat send durability issue.

Next step before merge

• [P2] The blocker is maintainer/author rework around the durable chat.send contract and current-main port, not a safe narrow automation repair.

Security
Cleared: No concrete security or supply-chain concern is visible from the diff; remaining blockers are session/message correctness and upgrade safety.

Review findings

• [P1] Cover the busy same-session path — src/gateway/server-methods/chat.ts:2350-2353

Maintainer merge request for head $sha.

Summary:

• Kept the WebChat eager user transcript fix, so chat.send persists the user turn before entering the agent/session lane.
• Fixed the attachment regression from the eager path: attachment media persistence no longer blocks dispatch, and the live session.message is emitted with media metadata only after the media rewrite completes.

After-fix verification from local OpenClaw source checkout:

ode scripts/test-projects.mjs src/gateway/server-methods/chat.directive-tags.test.ts -> 73 passed.

ode scripts/test-projects.mjs src/agents/pi-embedded-runner/run/attempt.prompt-helpers.test.ts src/gateway/server.chat.gateway-server-chat.test.ts src/gateway/server.chat.gateway-server-chat-b.test.ts src/auto-reply/reply/agent-runner-utils.test.ts src/auto-reply/reply/agent-runner-execution.test.ts src/auto-reply/reply/followup-runner.test.ts src/agents/pi-embedded-runner/run.overflow-compaction.loop.test.ts -> 7 files / 195 tests passed.

ode_modules.bin\oxfmt.CMD --check --threads=1 src\gateway\server-methods\chat.ts -> passed.

• corepack pnpm tsgo:core -> passed.
• corepack pnpm tsgo:core:test -> passed.
• corepack pnpm check:changed -> passed.
• git diff --check and git diff --check origin/main..HEAD -> passed.

GitHub checks observed on the latest SHA:

• Real behavior proof -> success.
• label, label-issues, and dispatch -> success.
• One �uto-response check was cancelled for the superseded push; the rerun on the latest SHA completed successfully.

Please merge when maintainer-required CI/review is satisfied.

Update after rebasing onto latest origin/main (839e9e9): head is now $sha.

Mergeability:

• Rebased cleanly after resolving the only conflict in CHANGELOG.md by dropping the contributor changelog entry, per repository guidance that maintainer/AI handles changelog during landing.
• GitHub now reports the PR as mergeable (mergeable=true); current unstable state is pending CI on the new SHA.

Re-run local verification after rebase:

ode scripts/test-projects.mjs src/gateway/server-methods/chat.directive-tags.test.ts -> 73 passed.

ode scripts/test-projects.mjs src/agents/pi-embedded-runner/run/attempt.prompt-helpers.test.ts src/gateway/server.chat.gateway-server-chat.test.ts src/gateway/server.chat.gateway-server-chat-b.test.ts src/auto-reply/reply/agent-runner-utils.test.ts src/auto-reply/reply/agent-runner-execution.test.ts src/auto-reply/reply/followup-runner.test.ts src/agents/pi-embedded-runner/run.overflow-compaction.loop.test.ts -> 7 files / 195 tests passed.

ode_modules.bin\oxfmt.CMD --check --threads=1 src\gateway\server-methods\chat.ts src\gateway\server-methods\chat-user-transcript-persistence.ts src\gateway\server.chat.gateway-server-chat-b.test.ts src\agents\pi-embedded-runner\run\attempt.prompt-helpers.ts src\agents\pi-embedded-runner\run\attempt.prompt-helpers.test.ts -> passed.

• corepack pnpm tsgo:core -> passed.
• corepack pnpm check:changed -> passed.
• git diff --check origin/main..HEAD -> passed.

Known local check gap:

• corepack pnpm tsgo:core:test currently fails on latest main in unrelated ACP tests: src/acp/translator.session-rate-limit.test.ts and src/acp/translator.stop-reason.test.ts. This PR has no src/acp/** changes, so I did not expand this latency fix into ACP test typing cleanup.

Please merge once required CI/review is satisfied.

Re-review progress:

• State: Complete
• Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
• Run: https://github.com/openclaw/clawsweeper/actions/runs/25686750806
• Updated: 2026-05-11T17:45:05.037Z

CI follow-up for the latest head (c3f576d54342c54cfffcddcdb3d1051cc8860971):

• Fixed the checks-node-agentic-gateway-methods failure in chat.directive-tags.test.ts by waiting for the intentionally backgrounded eager media transcript path to drain before the test exits. The previous assertion only waited for media saves, so a late transcript update from "serial please" could leak into the following "quick command" test on CI.
• This is test-only; the gateway behavior remains the same.

Local verification after the fix:

node scripts/test-projects.mjs src/gateway/server-methods/chat.directive-tags.test.ts
# 1 file / 73 tests passed

node scripts/test-projects.mjs src/gateway/server-methods/chat.directive-tags.test.ts src/gateway/server.chat.gateway-server-chat.test.ts src/gateway/server.chat.gateway-server-chat-b.test.ts
# 3 files / 122 tests passed

node_modules\.bin\oxfmt.CMD --check --threads=1 src/gateway/server-methods/chat.directive-tags.test.ts
# passed

corepack pnpm check:changed
# passed

git diff --check origin/main..HEAD
git diff --check
# passed

I also attempted the full CI shard locally with test/vitest/vitest.gateway-methods.config.ts, but the Windows local run exceeded a 5 minute timeout before producing a result, so I am relying on the targeted gateway-method tests above plus the GitHub shard rerun.

Maintainer merge request update:

• The previous Real behavior proof blocker is fixed and passing on the latest head.
• The follow-up checks-node-agentic-gateway-methods failure is fixed by c3f576d54342c54cfffcddcdb3d1051cc8860971; the GitHub shard is now passing.
• Current state has no failed checks; remaining checks are still queued in GitHub Actions.

Please merge once the required queued checks complete, unless another maintainer concern comes up.

This pull request has been automatically marked as stale due to inactivity.
Please add updates or it will be closed.

ClawSweeper PR egg

✨ Hatched: 🥚 common Pearl Diff Drake

Hatch command

Comment @clawsweeper hatch when this PR is hatchable.

Hatchability rules:

• Merged PRs are hatchable.
• Open PRs are hatchable when they are status: 👀 ready for maintainer look, status: 🚀 automerge armed, or labeled clawsweeper:automerge.
• Closed unmerged PRs are hatchable only when one of those hatchable labels is still present in the durable record.

Rarity: 🥚 common.
Trait: polishes edge cases.
Image traits: location green-check meadow; accessory tiny test log scroll; palette violet, aqua, and starlight; mood watchful; pose leaning over a miniature review desk; shell starlit enamel shell; lighting moonlit rim light; background small green status lights.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Pearl Diff Drake in ClawSweeper.

This pull request has been automatically marked as stale due to inactivity.
Please add updates or it will be closed.

ClawSweeper status: review started.

I am starting a fresh review of this pull request: fix(gateway): reduce WebChat ingress latency This is item 1/1 in the current shard. Shard 8/20.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.