forked from facebook/astryx
-
Notifications
You must be signed in to change notification settings - Fork 0
Review 5194
Cindy Zhang edited this page Aug 23, 2026
·
1 revision
nynexman4464 · merged · view on GitHub
Verdict: approve
These records predate the Review Presentation format for the most part, so the problem/solution/API/theme-target/breaking slots are only filled where the review itself carried them. Reviews from 2026-08-23 on use the full template.
Thanks, this is clean — nice that the options destructure keeps memoization working with an inline object.
Both are documented, but they'll land under "Other" in
astryx hook list: the doc files saygroup: 'Utilities'and discovery readscategory:. Every hook insrc/hooks/usescategory. Copied fromuseTranslator, so it's pre-existing — worth fixing all three while you're here.Approving; merge when you're ready.
[Reviewed by Robohands]