Chat turns are not jobs; link bulk loads to their job; notification audit #63

Merged
KofTwentyTwo merged 5 commits from feature/turns-not-jobs into develop 2026-10-19 13:44:44 +00:00
KofTwentyTwo commented 2026-10-16 20:30:56 +00:00 (Migrated from github.com)

Ordinary assistant answers currently create redundant tracking jobs and send a completion notification after every turn. Remove that tracker and retain the assistant's existing durable turn leases, event replay, cancellation and orphan recovery. Actual bulk loads still launch durable jobs; their tool and confirmation results now link to the job record.

Scheduled task notifications use core QAlerts with only the task owner's audience, a status-only body and a conversation link. Budget and provider-health alerts remain in place. Removing the public run-tracker accessors is intentional and documented in UPGRADE.md; hosts using them must remove those calls. There is no schema migration for this change.

Integrated the merged attachment implementation without changing its behavior. Review checked the complete diff and retained timeout, wedged-provider stop, cross-pod cancellation, orphan recovery and shutdown coverage.

Validation: 45 focused tests passed across durable turns, scheduled tasks, bulk loads and attachment turns. Full eight-module reactor verify passed, including coverage and guardrail gates, in 4m32s. This PR is a dependency for Website-Backend #304/#306, not proof of host adoption or production deployment.

AI-assisted implementation by Claude Code; integration and review by Codex.

Ordinary assistant answers currently create redundant tracking jobs and send a completion notification after every turn. Remove that tracker and retain the assistant's existing durable turn leases, event replay, cancellation and orphan recovery. Actual bulk loads still launch durable jobs; their tool and confirmation results now link to the job record. Scheduled task notifications use core QAlerts with only the task owner's audience, a status-only body and a conversation link. Budget and provider-health alerts remain in place. Removing the public run-tracker accessors is intentional and documented in UPGRADE.md; hosts using them must remove those calls. There is no schema migration for this change. Integrated the merged attachment implementation without changing its behavior. Review checked the complete diff and retained timeout, wedged-provider stop, cross-pod cancellation, orphan recovery and shutdown coverage. Validation: 45 focused tests passed across durable turns, scheduled tasks, bulk loads and attachment turns. Full eight-module reactor verify passed, including coverage and guardrail gates, in 4m32s. This PR is a dependency for Website-Backend #304/#306, not proof of host adoption or production deployment. AI-assisted implementation by Claude Code; integration and review by Codex.
Sign in to join this conversation.