Skip to content

fix(web): group streamed run message deltas - #252

Merged
SaladDay merged 1 commit into
mainfrom
fix/group-run-message-deltas
Sep 4, 2026
Merged

fix(web): group streamed run message deltas#252
SaladDay merged 1 commit into
mainfrom
fix/group-run-message-deltas

Conversation

@kapelame

@kapelame kapelame commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Group message.delta events into one raw event block per run step.
  • Concatenate streamed text in event order while retaining the latest payload metadata.
  • Keep non-delta raw event blocks, styling, and interactions unchanged across harnesses.
  • Keep the formatter outside the oversized RunsPage.tsx.

Scope

In scope: admin run raw-event formatting and rendering only.

Out of scope: backend or API changes, state management, harness-specific branches, and UI restyling.

Testing

  • Targeted formatter assertions for grouped deltas and unchanged non-delta events
  • pnpm --filter @parsar/web typecheck
  • pnpm --filter @parsar/web build
  • pnpm --filter @parsar/web exec eslint src/lib/agent-run-event-format.ts src/pages/admin/RunsPage.tsx --rule 'react-hooks/set-state-in-effect: off'
  • make check (passed; Docker was unavailable, so the repository script skipped its Postgres migration smoke test)

Review

Two independent blind reviews of the complete diff found no issues.

@SaladDay
SaladDay force-pushed the fix/group-run-message-deltas branch from 4da04b2 to 0efbfa5 Compare September 4, 2026 09:29
@SaladDay
SaladDay merged commit 4ff82de into main Sep 4, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants