Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

specs: update Lexicon reference "fragment" syntax to allow trailing digits #391

Open
bnewbold opened this issue Jan 13, 2025 · 0 comments
Open
Labels
bug Something isn't working specs

Comments

@bnewbold
Copy link
Contributor

bnewbold commented Jan 13, 2025

Current specs are quite restrictive: https://atproto.com/specs/nsid#:~:text=The%20fragment%20identifier%20string%20(after%20the%20%23)%20has%20the%20same%20syntax%20restrictions%20as%20the%20final%20segment%20of%20an%20NSID%3A%20ASCII%20alphabetic

We definitely have at least "v2" use-cases in the wild.

I think what needs to change here is the spec, not the records. We want these to be valid programming symbols (variable/method names), but trailing digits should be fine (as long as there are no dashes, underscores, or leading digits).

Reported here: https://bsky.app/profile/nekorug.moe/post/3lfnkukmxlc2o

@bnewbold bnewbold added bug Something isn't working specs labels Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working specs
Projects
None yet
Development

No branches or pull requests

1 participant