Skip to content

feat(seer): Wire legacy autofix write site through SeerRun outbox#115228

Draft
trevor-e wants to merge 1 commit intotelkins/seer-run-outboxfrom
telkins/seer-run-outbox-legacy-autofix
Draft

feat(seer): Wire legacy autofix write site through SeerRun outbox#115228
trevor-e wants to merge 1 commit intotelkins/seer-run-outboxfrom
telkins/seer-run-outbox-legacy-autofix

Conversation

@trevor-e
Copy link
Copy Markdown
Member

@trevor-e trevor-e commented May 8, 2026

Summary

  • Behind organizations:seer-run-mirror, _call_autofix() now creates a SeerRun row + CellOutbox entry instead of calling Seer directly
  • The existing outbox receiver (SEER_RUN_CREATE / SeerRunType.AUTOFIX) handles the HTTP call and writes back seer_run_state_id on success
  • Non-flagged path is unchanged — no behavior change without the flag

Stacked on #115111. This is the second write site conversion (after assisted-query).

Test plan

  • New test test_outbox_path_creates_run_and_returns_run_id verifies SeerRun creation, outbox dispatch, and run_id return
  • Existing TestCallAutofix::test_call_autofix still passes (non-flagged path unchanged)
  • All 11 HandleSeerRunCreateTest receiver tests still pass
  • mypy + ruff clean

Behind the existing `organizations:seer-run-mirror` flag, `_call_autofix`
now creates a SeerRun row and CellOutbox entry instead of calling Seer
directly. The outbox receiver handles the actual HTTP call and writes
back the `seer_run_state_id` on success.

This is the second write site conversion (after assisted-query). The
non-flagged path is unchanged.

Co-Authored-By: Claude <noreply@anthropic.com>
@trevor-e trevor-e requested a review from a team as a code owner May 8, 2026 20:44
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 8, 2026
@trevor-e trevor-e marked this pull request as draft May 8, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant