My reasoning for containing the Spoiler text in React.useMemo() is I believe there's a linear task to track editing / deleting messages, so ultimately when that makes progress it's possible that the text prop will change and we can trigger the dependency change to not get the cached value, but for all other use cases we don't need to keep re-rendering the same spoiler text when the text content is not changing.

Instead of memoizing the return value you should just wrap the whole component in React.memo

Let's avoid the increased indentation level here

Thanks for the feedback - do you mind clarifying what you mean exactly about the indentation?

Something like this example in the codebase:

Ah makes sense, thanks for the reference!