Skip to content

eve#624: pin relay's epic head for the projectless adopt - #630

Merged
barelyworkingcode merged 1 commit into
epic/623-projectless-adoptfrom
chore/624-relay-pin
Oct 22, 2026
Merged

barelyworkingcode merged 1 commit into
epic/623-projectless-adoptfrom
chore/624-relay-pin

Conversation

@barelyworkingcode

Copy link
Copy Markdown
Owner

Closes #624

PR 1 of 2 for #624: pin test/e2e/relay-pin.json to relay's epic head (fa33abf, relay epic/585-projectless-adopt with barelyworkingcode/relay#592 merged). Targets epic/623-projectless-adopt.

Approved behaviour change (not weakening)

The new fakerelay makes a Claude outside row in no project adoptable and refuses no_project on it (barelyworkingcode/relay#585 open question 1; #623 planner risk 2). g6-outside-adopt.spec.js changes only where that rule forces it:

  • "Orphan notes" (desktop and phone) stays a Claude row in no project that is not adoptable, now with not_registered. It still asserts the reason line and no button.
  • The "Unfiled run" row went: it duplicated not_registered, which "Orphan notes" now carries, so a strict-mode text match would hit two rows.
  • The NO_PROJECT constant and its two assertions went: no fakerelay world can produce that line now.

Known gap

The "Not in a project, so it can't be adopted." line is now only for Pi and Codex rows, and fakerelay worlds hold only Claude rows (barelyworkingcode/relay#591). Until then no hermetic spec proves it. The devbox pass on barelyworkingcode/relay#587 (step 9, a Pi in no project) covers it.

Proof

npx playwright test on 952d1f0: 836 passed, 33 skipped, 0 failed (4.5m).

Fix attempts: 0. Tokens: about 100k.

Review: no blocking findings (Sonnet)
Journey: n/a: pin bump and fixture edits; no screen behaviour changes in this PR
Weakening reviewed: removed the Unfiled run row (duplicate not_registered coverage) and the NO_PROJECT assertions (unbuildable under the approved fakerelay rule); coverage moves to the relay#587 devbox pass until relay#591

🤖 Generated with Claude Code

…kerelay rule

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@barelyworkingcode
barelyworkingcode merged commit a35ca70 into epic/623-projectless-adopt Oct 22, 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