Please address the comments from this code review:

## Overall Comments
- In `ActionHelper::wait_freezes`, `comparator.draws()` calls an `&&`-qualified method on an lvalue and will not compile as written; you likely want `VisionBase::save_draws(draw_name, std::move(comparator).draws());` instead.
- `VisionBase::save_draws` generates the same filename for every element in `draws` within the loop, so later images may overwrite earlier ones; consider appending the loop index or a per-image ID to the formatted filename.

## Individual Comments

### Comment 1
<location path="source/MaaFramework/Vision/VisionBase.cpp" line_range="90-96" />
<code_context>
+    auto dir = option.log_dir() / "vision";
+    std::filesystem::create_directories(dir);
+
+    for (const auto& draw : draws) {
+        std::string filename = std::format("{}_{}.jpg", format_now_for_filename(), name);
+        auto filepath = dir / std::filesystem::path(filename);
+
+        std::ofstream of(filepath, std::ios::out | std::ios::binary);
+        of.write(reinterpret_cast<const char*>(draw.data()), draw.size());
+        LogDebug << "save draw to" << filepath;
+    }
+}
</code_context>
<issue_to_address>
**issue (bug_risk):** Multiple draws in a single call will all use the same filename and overwrite each other.

Because `filename` is based only on `format_now_for_filename()` and `name`, every iteration in the loop writes to the same `filepath`, so only the final buffer is kept. To prevent silently overwriting earlier draws, either add a per-draw index/unique suffix to the filename or generate a single multi-image artifact instead of one file per draw.
</issue_to_address>