refactor: denylist macros

[zeylahellyer]

Add a Clippy denylist for macros. Denies the use of the following macros:

• serde_json::json: less flexible and performant, use structs and serde_derive
• std::vec: the Vec::new and Vec::from methods are preferred
• std::write: slow to compile and run, use Formatter, Display, or Debug calls instead

Closes #1628.

Todo:

• Fix this one test I have spent too much time on

[vilgotf]

Not that vec![] places the items directly on the heap whereas Vec::from([]) needs to copy the array from the stack to the heap which can cause performance degradations. vec![] also supports creating variable length vectors, i.e. let identified = vec![false; shards.len()]; compiles whereas let identified = Vec::from([false; shards.len()]); does not (example taken from next)