Skip to content

relay: a Mac answers a command it cannot run instead of dropping it (D-195) - #154

Merged
blove merged 1 commit into
mainfrom
blove/unknown-relay-command
Nov 9, 2026
Merged

blove merged 1 commit into
mainfrom
blove/unknown-relay-command

Conversation

@blove

@blove blove commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Found while reassessing D-180 (phone project edits): production's hub (hub-v0.5.1) forwards project.create|update|delete, but the owner's installed prod and Beta Mac apps are Nov 4 builds that don't know those methods. RelayLink skipped any frame it couldn't decode, a command included, so the phone waited out the hub's 15 s reply wait and showed "Not confirmed".

Change (D-195).

  • RelayFrame.decode tells a command with a string id apart from other unreadable frames (HubLinkError.unknownCommand(id:method:)).
  • On a host link, RelayLink replies at once with unsupported_capability: "Your Mac's Agent Workspace can't do “” yet. Update it on the Mac, then try again.", details: { reason: "unknown_method", method }. A client link ignores it; every other unreadable frame is still skipped and the link stays up.
  • No hub or phone change: the hub passes a host's reply through and the phone shows a refusal's message. Hub/API.md documents it.

This helps Mac builds from now on only. The installed prod and Beta Mac apps still need repackaging from main for D-180 (and D-184) to work end to end.

Tests. RelayLinkTests.aMacAnswersACommandItCannotRunAtOnce, aPhoneIgnoresACommandItCannotRead; RelayWireTests.aCommandWithAnUnknownMethodIsNamedNotJustSkipped. swift test: 1471 tests in 183 suites pass; swift format lint --strict and docs-index --check clean.

🤖 Generated with Claude Code

…D-195)

A host link now replies unsupported_capability (reason unknown_method)
to a command whose method its build does not know, so a newer phone
gets a reason at once instead of waiting out the hub's reply timeout.
Other unreadable frames are still skipped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@blove
blove enabled auto-merge November 9, 2026 22:26
@blove
blove merged commit 706bfe7 into main Nov 9, 2026
4 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