Skip to content

fix(telemetry): close recovered task lifecycles - #277

Closed
IRONICBo wants to merge 1 commit into
socai-io:mainfrom
IRONICBo:fix/task-lifecycle-recovery-telemetry
Closed

fix(telemetry): close recovered task lifecycles#277
IRONICBo wants to merge 1 commit into
socai-io:mainfrom
IRONICBo:fix/task-lifecycle-recovery-telemetry

Conversation

@IRONICBo

Copy link
Copy Markdown
Collaborator

Summary

  • remember tasks that were running when startup recovery marks them interrupted
  • emit one recovered socai_agent_task_end event on the next launch
  • exclude queued tasks because they never emitted a start event
  • document the recovered_on_startup marker

Evidence

The sanitized seven-day Axiom export for v0.5.5 contains two macOS task IDs with one start and no end, plus one Windows task ID with two starts and one end. The persisted task registry already converts leftover running tasks to interrupted at startup, but that recovery path did not emit the matching terminal telemetry event.

This change is prospective. It closes future recovered lifecycles; it cannot backfill historical gaps that were already persisted as interrupted.

Verification

  • cargo check -p socai_app
  • cargo test -p socai_app tasks::tests::three_tasks_run_while_the_fourth_waits_for_a_released_permit
  • git diff --check

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

@IRONICBo is attempting to deploy a commit to the socai Team on Vercel.

A member of the Team first needs to authorize it.

@IRONICBo

IRONICBo commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator Author

Closing per maintainer request to keep the active delivery queue focused on four PRs only: audio/video integration, Douyin, TikTok, and the consolidated browser self-healing PR #292. This PR remains available in history and can be restored separately if needed.

@IRONICBo IRONICBo closed this Sep 6, 2026
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