Skip to content

Resolve server-side spec gaps#116

Open
maximmaxim345 wants to merge 2 commits into
mainfrom
fix/server-breaking-changes
Open

Resolve server-side spec gaps#116
maximmaxim345 wants to merge 2 commits into
mainfrom
fix/server-breaking-changes

Conversation

@maximmaxim345

@maximmaxim345 maximmaxim345 commented Jul 2, 2026

Copy link
Copy Markdown
Member

The PR with server-implementation-breaking changes I mentioned in #113. Clients do not have to adjust anything for these; they tighten what a conforming server must do.

Specifically:

  • Role activation: a server MUST NOT activate a role or version the client did not list in supported_roles.
  • Forward compatibility: servers (not just clients) MUST ignore unrecognized payload fields, and MUST NOT send fields the spec does not define.
  • stream/request-format: a request the server cannot honor is not an error; the stream continues in a configuration the client supports. For players, stream/start MUST use a format from the client's supported_formats, so the server can never start a stream the client cannot decode.
  • Reconnect backoff: after repeated consecutive Noise-handshake failures, the server SHOULD NOT keep auto-reconnecting until there is reason to expect success (operator re-initiates pairing, network conditions change).
  • Artwork late join: after an artwork stream/start (initial or post-reconnect), the server SHOULD immediately send the current image for each channel whose source is not 'none', so a client joining mid-track is not left blank until the next track change.

@maximmaxim345 maximmaxim345 force-pushed the fix/available-state branch from 5b517e1 to feec536 Compare July 7, 2026 06:58
@maximmaxim345 maximmaxim345 force-pushed the fix/server-breaking-changes branch from f4c53e3 to 8ec0b89 Compare July 7, 2026 07:06
Base automatically changed from fix/available-state to main July 7, 2026 13:00
@maximmaxim345 maximmaxim345 force-pushed the fix/server-breaking-changes branch from 8ec0b89 to 46d514d Compare July 7, 2026 13:04
@maximmaxim345 maximmaxim345 marked this pull request as ready for review July 7, 2026 13:04
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