Skip to content

document player properties, ed25519 auth, and utf-8 chat extensions#54

Open
Fran6nd wants to merge 3 commits into
piqueserver:masterfrom
Fran6nd:update-extension-doc
Open

document player properties, ed25519 auth, and utf-8 chat extensions#54
Fran6nd wants to merge 3 commits into
piqueserver:masterfrom
Fran6nd:update-extension-doc

Conversation

@Fran6nd

@Fran6nd Fran6nd commented May 1, 2026

Copy link
Copy Markdown

Bring extensions.md in line with libspades' extension docs — adds
Player Properties (id 0), Ed25519 Authentication (id 1), and a note
about UTF-8 Chat. Also fills in the empty has-packets table in
index.md so it actually lists something.

These are not really proposals as these have already been widely adopted.

Sources:

Comment thread extensions.md Outdated
@Fran6nd Fran6nd force-pushed the update-extension-doc branch from ac25576 to 5993fe0 Compare May 3, 2026 15:16
@Fran6nd

Fran6nd commented May 20, 2026

Copy link
Copy Markdown
Author

Up :) @utf-4096

@utf-4096

Copy link
Copy Markdown
Member

Don’t have review permissions, but looks fine.

@Fran6nd

Fran6nd commented May 20, 2026

Copy link
Copy Markdown
Author

Don’t have review permissions, but looks fine.

Ach sorry I thought you had it.

add per-field and per-offset tables for sub-packet 0 so implementers
have a clear wire-level description rather than just the 12-byte total.
@NotAFile

Copy link
Copy Markdown
Member

Oops I'll have to fix the repo permissions

@Fran6nd

Fran6nd commented May 25, 2026

Copy link
Copy Markdown
Author

BTW, in my last commit (after PR approval) I did split the documentation of player properties into two tables to avoid having an ultra wide table. Let me know if you prefer me to merge them.

@DavidCo113

Copy link
Copy Markdown
Contributor

Health, 0..100. 0 indicates the player is dead.

0--255, 0 HP in betterspades does not kill the player (that's what the kill packet is for. . .) -- does in openspades which doesn't support this packet though

Score | LE Uint

May or may not actually be a (signed) int32_t.

@Fran6nd

Fran6nd commented Jun 24, 2026

Copy link
Copy Markdown
Author

Updated according to your comment @DavidCo113 I did check how score is handled but only on 3 clients though. BetterSpades, KyroSpades and ZeroSpades.

@DavidCo113

Copy link
Copy Markdown
Contributor

Reference clients treat it as unsigned (BetterSpades, . . .)

Did you test it or just go off the source code? TigerSpades certainly treats it as signed, despite the struct being unsigned.

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.

4 participants