I’m a bit confused with how there are a few different hardware methods to make this work. With the little MOSFET board I used, I only connected two of the digital pins (18, and 19) but I see others using a third pin for theirs.
The mosfet version is better as it has better signal quality and isolation as well as being the simplest. Your wiring looks fine but your logs don’t show the correct data. You only need 2 wires for the mosfet version as it is bidirectional (does read/write using the same pin). Double check that your pinouts match your wiring, that’s the only other thing I can think of that would be an issue.
Edit, it looks like you have the 18/19 connections reversed on the esp. Ie you have the yellow going to pin 18, and green to pin 19 on the esp. Just change the pin definitions in the yaml config. Clock=18, read/write=19