union definition was already good, but word it more strongly, and explicitly say token strings are not valid (I had been fuzzy on this)
unknown needs to be an object. It can't be things like a blob, link, or bytes, even though those are "objects" in JSON encoding.
Also recommend against using unknown in records, to give us some wiggle room about strict data validation. AKA, we haven't really defined or enforced some of the corner cases around "generic data validation".
tl;dr:
uniondefinition was already good, but word it more strongly, and explicitly saytokenstrings are not valid (I had been fuzzy on this)unknownneeds to be an object. It can't be things like a blob, link, or bytes, even though those are "objects" in JSON encoding.Also recommend against using
unknownin records, to give us some wiggle room about strict data validation. AKA, we haven't really defined or enforced some of the corner cases around "generic data validation".Closes: https://github.com/bluesky-social/atproto-website/issues/317 Closes: https://github.com/bluesky-social/atproto-website/issues/319