Make `Uuid` Nameable by kistz · Pull Request #4011 · clockworklabs/SpacetimeDB

kistz · 2026-01-12T21:20:58Z

Description of Changes

Implements the Nameable interface for the generated uuid client bindings.

API and ABI breaking changes

None.

Expected complexity level and risk

Only additive changes pretty much copy pasted from the other types where it was already implemented.

Testing

Before it didnt work in my project now it works

cloutiertyler

This looks good to me!

egormanga

Just a friendly lint for you

crates/bindings-typescript/src/lib/type_builders.ts

kistz · 2026-01-12T22:13:05Z

Yeah sorry guys forgot to do that 😓should be good now :> orrrr my pnpm format is broken idk xD

egormanga · 2026-01-12T22:15:04Z

orrrr my pnpm format is broken idk

Yeah, it just added a bunch of unrelated changes for some reason (those are all valid, though).

kistz · 2026-01-12T22:30:09Z

As discussed with @cloutiertyler on discord we decieded to check in all the files that currently fail formatting

You can still review commit per commit to make it easier 👍

kistz · 2026-01-13T17:01:13Z

This is ready to review and good to merge 👍
my statement yesterday in the stream that something is afoot with the react hooks can be discarded :>
(well technically its still not working to do where(eq("uuidField", "im-a-uuid")) but this is a separate limitation in the react hooks which i will try to do in a followup.
This pr fixes the way more critical bug that the ts sdk is not even compiling because of the missing nameable 😇

# Description of Changes Implements the `Nameable` interface for the generated uuid client bindings. # API and ABI breaking changes None. # Expected complexity level and risk 1. Only additive changes pretty much copy pasted from the other types where it was already implemented. # Testing - [x] Before it didnt work in my project now it works --------- Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>

# Description of Changes Dpends on and includes: #4011 Will remove commits whenever that merges 👍 This impacts the `useTable()` functions `where`clause. We add a new possibility for the `Value` type which is the new `Uuid` type. Also my formatter is bugging again :( whenever i `pnpm format` it does things differently than the guidelines? Would be cool for that reason if someone can run format for me maybe bfops like the last time :> # API and ABI breaking changes None # Expected complexity level and risk 1. Only lifts the filter for the where clause a bit to further support `Uuid` alongside bool, string, and number. # Testing - [x] Getting syntax highlithing for the new availbale row - [x] Getting row back from subscription when supplying a uuid - [x] Fix edge case with string --------- Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>

make uuid nameable

c42505b

cloutiertyler approved these changes Jan 12, 2026

View reviewed changes

egormanga reviewed Jan 12, 2026

View reviewed changes

crates/bindings-typescript/src/lib/type_builders.ts Outdated Show resolved Hide resolved

crates/bindings-typescript/src/lib/type_builders.ts Outdated Show resolved Hide resolved

pnpm format

cc84dc5

pnpm format on all the things

9047070

[nameable-uuid]: lint

75fbcf5

bfops enabled auto-merge January 13, 2026 18:30

bfops disabled auto-merge January 13, 2026 18:38