Retries are an essential part of e2e in order to deal with flakey tests.
When a screenshot is missing, the framework creates a new one and saves it. This causes a retry to PASS. I would argue that the framework should never automatically create a reference screenshot. Or at least there should be a flag to prevent if from doing so. The reference screenshots should be copied manually from screens directory by developers.
Retries are an essential part of e2e in order to deal with flakey tests.
When a screenshot is missing, the framework creates a new one and saves it. This causes a retry to PASS. I would argue that the framework should never automatically create a reference screenshot. Or at least there should be a flag to prevent if from doing so. The reference screenshots should be copied manually from screens directory by developers.