feat(derived): Reconcile GroupDerivedData rows with inconsistent status - #123853
feat(derived): Reconcile GroupDerivedData rows with inconsistent status#123853kcons wants to merge 3 commits into
Conversation
github-actions
Bot
added
the
Scope: Backend
|
bugbot review |
| category=OutboxCategory.GROUP_ACTION_LOG_EVENT, | ||
| ) | ||
| return ( | ||
| CellOutbox.objects.filter(**filter_kwargs).exists() |
There was a problem hiding this comment.
I guess I can drop this part.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5ae00c1. Configure here.
| group_id=group.id, | ||
| project=group.project, | ||
| actor=SYSTEM_ACTOR, | ||
| ) |
There was a problem hiding this comment.
Stale reconcile overwrites newer status
Medium Severity
The last pending-outbox check and publish_action are not atomic with a group-status re-read. A concurrent resolve or unresolve can commit first; this path then appends a ReconcileStatusAction with the earlier target, so derived processing applies that stale reconcile after the newer action and leaves log-derived status wrong until another view.
Reviewed by Cursor Bugbot for commit 5ae00c1. Configure here.
1 participant
We need the log-derived status to match the actual one.