[loganj]

🤖

Summary

• Stacked on feat(agents): carry agent types and their handles over the agents port #1733. Two synchronous AgentsClient reads were still missing from the port client: sessionState(key, place), which the composer reads to block a send while a session compacts, and previewRetype(key, type), which settings and the update review call when the owner picks another type.
• With them, the port client from feat(agents): serve the agents service to screens over a MessagePort #1725 implements all of AgentsClient (plus present), so the screens' page can use it in place of the service. Nothing in the app uses the port yet, so behavior does not change.

Details

• sessionState: the first read of an agent and place asks the service for it, and reads as unavailable until the service pushes it. unavailable is the service's own answer for a session it cannot act on, so screens already handle it. After that, each push carries the place's state. The type's session-controls change notice already makes the service push (feat(agents): carry agent types and their handles over the agents port #1733).
• previewRetype: computed on the screens' side with retypePreview (refactor(agents): give screens a type's config functions through present #1727). It uses the shown agent's config and the two types' config functions, which each type's UI entry also presents in that page. It refuses an unknown agent, a missing type and a hosted type with the service's messages, and returns the config unchanged for the same type.
• Difference: the service looks the agent up among all its identities, and the client looks it up among the shown community's agents. Both callers (settings and the update review) ask about an agent the screen shows.
• AgentsPortClient is now AgentsClient & { present }.

Testing

• service.test.ts, over the real service:
  • Changing an agent's type: with the config functions presented, the client's preview equals the service's for a type change that moves, resets and drops settings, and for the same type. A missing type is refused.
  • Session state: a place reads as unavailable, then idle once pushed, then compacting after the harness's change notice. The test fails if the client does not ask for the place (mutation-checked).
• tsc --noEmit passes. The agents and Agents page suites pass, except the Agents page avatar-picker test, which also fails on the base branch in this worktree (its node_modules is a symlink).

🤖 Generated with Claude Code

[loganj]

🤖 Closing: the Manager/Runner split replaces this design (#1748 → #1839, with more to come). The branch stays in place because #1737, #1743 and #1744 are still stacked on it. They will move onto the new stack in the hidden-page step.