Greptile Summary

This PR fixes a CI gap where a missing screenshot file during a --verify run would silently pass. The fix is small and focused: it adds an early break from the retry loop when opts.verify is set and no expected file exists, which correctly falls through to the existing failure-recording code (exitStatus = 3). A companion guard (opts.diff && expected) prevents convert from being invoked against a non-existent base file.

Key changes:

• opts.verify && !expected now breaks the retry loop immediately and records the test as failed, causing CI to exit with status 3.
• --new mode is preserved: the newly taken screenshot is still saved to newDir for artifact uploading even when the baseline was absent.
• The diff-image generation step is now guarded by expected to avoid passing a missing file to convert.

Confidence Score: 4/5

• This PR is safe to merge; the logic change is correct and well-scoped.
• The change is minimal (two small hunks), the failure path was already in place, and all edge cases (--new, --diff, retry loop) are handled correctly. The only notable concern is a harmless write-then-delete cycle in the --diff-only + missing-screenshot edge case.
• No files require special attention beyond the one changed file.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[takeScreenshot called] --> B{opts.verify?}
    B -->|yes| C{file exists?}
    C -->|yes| D[expected = readFile]
    C -->|no| E[expected = null]
    B -->|no| F[expected = null]
    D & E & F --> G[while retry loop]
    G --> H[take screenshot, assign buf]
    H --> I{opts.verify AND not expected?}
    I -->|yes| J[log error missing screenshot, break]
    I -->|no| K{expected?}
    K -->|yes| L{buf equals expected?}
    L -->|yes| M[log ok, return success]
    L -->|no| N[log error, retry]
    K -->|no| O[writeFile baseline, return success]
    J --> Q
    N --> P{retries exhausted?}
    P -->|no| G
    P -->|yes| Q[console.error FAIL, exitStatus=3]
    Q --> R{opts.diff or opts.new?}
    R -->|yes| S[writeFile buf to outputDir]
    S --> T{opts.diff AND expected?}
    T -->|yes| U[generate diff image via convert]
    T -->|no| V{opts.new?}
    U --> V
    V -->|no| W[unlink bufFile]
    V -->|yes| X[keep bufFile in newDir]
    R -->|no| Y[end]

_{Last reviewed commit: 5a82ab0}

Looks great!

🎉 This PR is included in version 0.16.38 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀