I prefer `tag_stale/1` had a clause that matched `<<"stale:", _/binary>>` and you could lose this case statement, and keep the knowledge about tagging encapsulated in the `tag_stale/1` function, or is the state mutation too expensive in that case since you'd be setting `State#state{value=V}` even if it is unchanged.