Skip to content

fix: run an Automation only once when its Session is opened afterwards - #2089

Merged
Douglasymlai merged 4 commits into
mainfrom
fix/automation-duplicate-run
Oct 26, 2026
Merged

Douglasymlai merged 4 commits into
mainfrom
fix/automation-duplicate-run

Conversation

@Douglasymlai

@Douglasymlai Douglasymlai commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Pull Request

Related Issue

N/A — maintainer-directed repair; no linked issue required for this PR.

Description

Problem

Background Automations used a random task ID instead of the durable queue request ID, leaving the original request available for another admission when its Session opened.

Change

Start the background task with the existing queue request ID. The backend's admission transaction then marks that exact request admitted and deduplicates retries; terminal-state handling can admit the next execution. Updated against current main.

Testing Evidence (REQUIRED)

79 focused frontend tests and 6 durable queue tests passed, covering deterministic identity, re-entrant dispatch, subsequent execution admission, and request deduplication. Type checking, focused ESLint/Prettier and git diff --check passed.

No new UI primitives, tokens or design exceptions. A full scheduled Automation in the packaged app was not run; remote CI follows the updated head.

Automated checks only; manual app testing is pending.

  • I have included human-verified testing evidence in this PR.
  • This PR includes frontend/UI changes, and I attached screenshot(s) or screen recording(s).
  • No frontend/UI changes in this PR.

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Contribution Guidelines Acknowledgement

@Douglasymlai Douglasymlai self-assigned this Oct 22, 2026
@Douglasymlai
Douglasymlai marked this pull request as ready for review October 22, 2026 21:48
@Douglasymlai
Douglasymlai merged commit 026ee20 into main Oct 26, 2026
10 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.

1 participant