Cleaned up cached buff in documentation

[xackery]

https://docs.macroquest.org/reference/data-types/datatype-cachedbuff/?h=stal

the note on top is a bit confusing on what it conveys..

"Cached Buffs" used to be a new way to access [buff](https://docs.macroquest.org/reference/data-types/datatype-buff/) information on players without needing to re-target them. Now, this functionality is fully integrated into MacroQuest and available through the buff datatype. As a result, using cached buffs is now discouraged.

You should only used this type if you need access to its "unique" style of buff lookup.

Consider cleaning this up to more clearly state, "Cached Buffs are now inherited by spawn's Buff, and anything leveraging the term "Cached" in TLO's are deprecated

maybe consider cleaning up this link too: https://docs.macroquest.org/main/features/cached-buffs/?h=cached

to note the same, as I started trying to use cached buffs first time it was a bit confusing what this was conveying