server to use defined subprotocol?

[steffkes]

I was reading across the docs and wondering if the server is supposed to use the subprotocol that was defined in the previous step, rather than a static string? As we do with the client

defined as part of the project

docs/guide/starting/project.md

Lines 18 to 22 in 3e83837

	Create `api/index.ts` with subprotocol version (client-server API).
	```ts
	export const SUBPROTOCOL = '1.0.0'
	```

imported for server but afterwards not used for

docs/guide/starting/server.md

Lines 22 to 34 in 3e83837

	Create `index.ts` with:
	```js
	import { Server } from '@logux/server'
	import { SUBPROTOCOL } from '../api/index.js'
	const server = new Server(
	Server.loadOptions(process, {
	subprotocol: '1.0.0',
	supports: '1.x',
	fileUrl: import.meta.url
	})
	)

because the client does

docs/guide/starting/client.md

Lines 13 to 28 in 3e83837

	Open `src/index.ts` and add:
	```tsx
	import { CrossTabClient, badge, badgeEn, log } from '@logux/client'
	import { badgeStyles } from '@logux/client/badge/styles'
	import { SUBPROTOCOL } from '../api'
	const client = new CrossTabClient({
	prefix: 'appName',
	server: process.env.NODE_ENV === 'development'
	? 'ws://localhost:31337'
	: 'wss://logux.example.com',
	subprotocol: SUBPROTOCOL,
	userId: 'anonymous', // TODO: We will fill it in Authentication recipe
	token: '' // TODO: We will fill it in Authentication recipe
	})

[ai]

Server needs subprotocol to mark actions created on this server by server.log.add(action) or server.process(action).

Later you can work with old actions from log knowing what server on server created this action. It is useful in big production projects with many changes.

Does it answer your question? Feel free to ask extra details if needed.

[steffkes]

sorry, maybe me asking this way made the conversation go this way.

I'm mainly wondering, if the SUBPROTOCOL const that is imported shouldn't be used instead of the static '1.0.0' that is currently displayed as part of the docs.

as in: should this be changed in the docs to show a coherent usage of the configuration.

[ai]

Got it! Yes, we should use const. Fixed 3a53dde