Skip to content

eve#627: offer Adopt on SSH-host rows relay marks adoptable - #641

Merged
barelyworkingcode merged 6 commits into
epic/626-host-adoptfrom
feat/627-host-adopt
Oct 23, 2026
Merged

barelyworkingcode merged 6 commits into
epic/626-host-adoptfrom
feat/627-host-adopt

Conversation

@barelyworkingcode

Copy link
Copy Markdown
Owner

Closes #627

PR 2 of 2 for #627 (PR 1: #639, the pin). The Agents board offers Adopt, or Interrupt and adopt, on a Claude row on an SSH host that relay marks adoptable. showsAdopt drops its host term; host_unsupported reads "Relay can't adopt sessions on this host."

Non-obvious decisions:

  • The Agents rail has no host chip, and the contract forbids a new one. G6.110 and the tests read "host chip" as the row naming its host project.
  • The journey list lives in devboxverify/journeys.js; the planner amended ownership by one line (Amendment 1 on Offer Adopt on SSH-host rows that relay marks adoptable #627 and Verify and close the host-adopt epic (eve) #628).
  • Approved behaviour change: the existing G6.107.r1 and G6.108.r1 expectations take the new host_unsupported text.
  • outside-adopt-host is a screen journey; the nightly runs it.

Fix attempts: 0. Review rounds: 1 (Sonnet).

Minor review findings, not fixed:

  • The host_link_lost spec asserts the "Can't adopt: " prefix, not relay's message text; the passthrough is pinned by other specs.
  • The journey deletes the adopted session only when the 201 arrived.

Tests: three G6.110 cases in g6-outside-adopt.spec.js. Full Playwright suite at 4db42d1: 846 passed, 33 skipped, 0 failed.

Review: no blocking findings (Sonnet)
Journey: outside-adopt-host (screen; the nightly runs it), added in this PR
Break-and-restore: put && !r.item.hostId back into showsAdopt: two G6.110 cases failed (no Adopt button); restored byte for byte
Harness weakening: none (one journey added; nothing removed, skipped or loosened)
Main harness vs head: n/a: the screen journey runs in the nightly; eve's --stack run at the joint close covers the api journeys

🤖 Generated with Claude Code

barelyworkingcode and others added 6 commits October 22, 2026 15:39
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contract amendment 1 on eve#627.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@barelyworkingcode
barelyworkingcode merged commit ae73cfe into epic/626-host-adopt Oct 23, 2026
@barelyworkingcode

Copy link
Copy Markdown
Owner Author

merge-check: merged. approved, target, contract, evidence, owner, weakening

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