Keep auto-present reminders synchronous to avoid run races #172

Merged
weichen merged 1 commits from feature/genui-auto-trigger-optin into main 2026-10-12 03:57:11 +00:00
Owner

Sync-only reminder append; no async notice-run.

Sync-only reminder append; no async notice-run.
weichen added 1 commit 2026-10-12 03:57:00 +00:00
Keep auto-present reminders synchronous to avoid run races
ci / vet-and-test (pull_request) Successful in 56s
9bd0aae9d8
Why: Async notice-run delivery raced with test teardown and in-flight
conversation runs (database closed / material lifecycle flakes).
Evidence: go test ./internal/runmgr/ -run Material
Impact: Reminders append as system messages only; material path still
inlines into startNoticeRun.
Refs: Dev PR #425 CI test-pg failure

Co-authored-by: Cursor <cursoragent@cursor.com>
weichen added 1 commit 2026-10-12 03:57:00 +00:00
Keep auto-present reminders synchronous to avoid run races
ci / vet-and-test (pull_request) Successful in 56s
9bd0aae9d8
Why: Async notice-run delivery raced with test teardown and in-flight
conversation runs (database closed / material lifecycle flakes).
Evidence: go test ./internal/runmgr/ -run Material
Impact: Reminders append as system messages only; material path still
inlines into startNoticeRun.
Refs: Dev PR #425 CI test-pg failure

Co-authored-by: Cursor <cursoragent@cursor.com>
weichen merged commit cb386023d0 into main 2026-10-12 03:57:11 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: weichen/Agent-System-Backend#172