Skip to content

CRE-5715: Defer cron trigger org resolution and event emission - #784

Open
bolekk wants to merge 2 commits into
mainfrom
cre-5715-just-order-swap
Open

bolekk wants to merge 2 commits into
mainfrom
cre-5715-just-order-swap

Conversation

@bolekk

@bolekk bolekk commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Move org ID resolution and TriggerExecutionStarted event emission to run after the trigger response is written back to the store and sent, so slow network calls (org resolver, event bus) never delay trigger generation or bookkeeping.

🤖 Generated with Claude Code

Deployment Validation: Confirm that CRON trigger events are emitted as expected for canaries (either via DB check or UI).

@bolekk
bolekk force-pushed the cre-5715-just-order-swap branch from 5f9b343 to 84ffd83 Compare September 20, 2026 17:48
@bolekk bolekk changed the title CRE-5715: Defer cron trigger bookkeeping, dedupe near-simultaneous fires CRE-5715: Defer cron trigger org resolution and event emission Sep 20, 2026
…-5715)

Org ID resolution and TriggerExecutionStarted emission can be slow
(org resolver lookup, event bus). Run them after the trigger response
is written back and sent, so slow bookkeeping never delays trigger
generation or the store write-back.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@bolekk
bolekk force-pushed the cre-5715-just-order-swap branch from 84ffd83 to d2d084a Compare September 20, 2026 19:32
…(CRE-5715)

The task callback's debug log was renamed to "sending trigger event"
but the test still checked for the old "task callback sending
trigger response" text.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@cl-sonarqube-production

Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
51.7% Coverage on New Code (required ≥ 75%)

See analysis details on SonarQube

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