Some shallow observations without really getting into the code:
misc
- Was hosting
labeler/ResNet50_nsfw_model.pthin the repository really necessary?
(I like my--filter=tree:0clones to be maximally fast and small.) - Why not declare all dependencies in the workspace?
- How old is the code (for real)?
rsky-crypto
anyhowin library code.- Not liking that
multibasedependency much either. I know thatbase64at least got re-written since that crate's last update (Nov 6, 2020).
rsky-feedgen
serde_cboris long dead (I was a user myself).- I'm not even sure why
serde_cborandserde_ipld_dagcborare dependencies anyway. - We moved from
lazy_statictoonce_cella long time ago. And your use is available in std on stable Rust today.
rsky-firehose
- Here, all three CBOR dependencies are actually used.
- CBOR (the format) used in
$CURRENT_YEARis meh anyway. But I guess that comes from IPLD.
rsky-identity
anyhowin library code.
rsky-pds
- That's quite the dependency list! Too long for me to take a closer look.
- How many
*base*dependencies does one need? All of them of course!
rsky-syntax
lazy_static and anyhow again.
That's all from a code organization and ecosystem PoV. Otherwise, things look normal and not fancy (which is good).
Unfortunately, I don't have the time to look beyond that at this moment.