Skip to content

Deliver mailbox mail as parent inbound so Skywalker stays idle - #874

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-7518-deliver-mailbox-mail-as-parent-inbound-so-skywalker-stays
Sep 10, 2026
Merged

Deliver mailbox mail as parent inbound so Skywalker stays idle#874
TheGreatAxios merged 1 commit into
mainfrom
cl-7518-deliver-mailbox-mail-as-parent-inbound-so-skywalker-stays

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Occupancy delivers worker done/fail reports as system inbound so an idle Skywalker starts a new turn, including while siblings still run. TUI-primary wait_agents yields as a timeout (workers untouched) when mailbox mail, a parked ask, or a queued Enter steer is ready. Nested orchestrators still collect with wait_agents.

The mailbox already holds terminal reports and director questions. What was wrong was wait_agents as a parent-busy tool: Skywalker sat inside a blocking read, so Enter queued until the wait returned. Idle-with-fleet already lets the parent go idle after spawn; collection is still a poll unless mail wakes the parent.

  • Already-collected waits return status without a second report body (no double-deliver).
  • Skywalker prompt and docs say spawn then idle; do not poll wait_agents. list_agents stays.
  • Fleet-dry occupancy still owns the last-lane 1→0 + open-tasks shot.
  • Yield is timeout-shaped; workers are not interrupted.

Verification

  • bun run typecheck, bun run build, and bun run test pass
  • bun run check passes (lint, typecheck, build, guarded suite, seed 424242)
  • Critic: no blocking findings on the TUI happy path

Fixes CL-7518

@linear-code

linear-code Bot commented Sep 10, 2026

Copy link
Copy Markdown

CL-7518

@TheGreatAxios
TheGreatAxios marked this pull request as draft September 10, 2026 13:39
@TheGreatAxios
TheGreatAxios force-pushed the cl-7518-deliver-mailbox-mail-as-parent-inbound-so-skywalker-stays branch from 2dbe485 to 966c5ad Compare September 10, 2026 14:42
@TheGreatAxios
TheGreatAxios marked this pull request as ready for review September 10, 2026 14:44
@TheGreatAxios
TheGreatAxios merged commit 4914046 into main Sep 10, 2026
9 checks passed
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