Skip to content

fix(ui): deduplicate scheduling confirmations - #2868

Merged
esokullu merged 3 commits into
webbrain-one:mainfrom
alectimison-maker:fix/deduplicate-scheduling-message
Aug 21, 2026
Merged

fix(ui): deduplicate scheduling confirmations#2868
esokullu merged 3 commits into
webbrain-one:mainfrom
alectimison-maker:fix/deduplicate-scheduling-message

Conversation

@alectimison-maker

Copy link
Copy Markdown
Contributor

Summary

  • reconcile schedule-created broadcasts and composer confirmations by scheduled job ID
  • preserve one confirmation across either event/response ordering and restored off-tab chat
  • mirror the behavior in Chrome and Firefox with regression coverage

Motivation

Issue #2867 creates a single scheduled job but can display two identical confirmation messages. The scheduler already emits one creation event; the duplicate came from two UI producers: the runtime event handler and the schedule composer response.

Design

Scheduled-job confirmations now carry the canonical job ID in the DOM. A shared renderer reuses the existing confirmation regardless of whether the runtime event or composer response arrives first. Messages without a job ID keep the prior behavior.

Testing

  • npm test — passed: 33 toolbar guard tests, 1,953 unit tests, offline benchmark, and 60/60 security tests
  • unpacked Chrome 33.0.8 — /schedule Issue 2867 final browser check produced one job card and one confirmation, with no console or page errors

Compatibility and risks

  • no scheduling, storage, alarm, or API changes
  • Firefox manual UI was not run; the mirrored Firefox path is covered by the same behavioral tests

Scope

This intentionally leaves runtime message transport unchanged and makes only the presentation layer idempotent by canonical job ID.

Closes #2867

@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

@alectimison-maker is attempting to deploy a commit to the esokullu's projects Team on Vercel.

A member of the Team first needs to authorize it.

@alectimison-maker
alectimison-maker marked this pull request as ready for review August 21, 2026 02:25
@esokullu
esokullu merged commit b1833b2 into webbrain-one:main Aug 21, 2026
1 of 2 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.

bug: duplicate scheduling message

2 participants