Please address the comments from this code review:

## Overall Comments
- In `Actuator::screencap`, consider ensuring the target `log_dir/screencap` directory exists (e.g., via `create_directories`) before calling `imwrite`, otherwise the action can silently fail on a missing directory.
- The parser and action handler accept any `format` string and any `quality` value; since the docs specify `png|jpg|jpeg` and `0–100`, it would be safer to validate and/or clamp these fields and log when invalid values are provided.
- When `imwrite` fails in `Actuator::screencap`, the log message still reads `screencap saved to`; consider logging a clearer error path (with reason if available) to make diagnosing screencap failures easier.

## Individual Comments

### Comment 1
<location path="source/MaaFramework/Task/Component/Actuator.cpp" line_range="532-537" />
<code_context>
+    std::string filename = param.filename.empty() ? std::format("{}_{}{}", format_now_for_filename(), name, ext) : param.filename + ext;
+    auto filepath = option.log_dir() / "screencap" / path(filename);
+
+    std::vector<int> encode_params;
+    if (param.format == "jpg" || param.format == "jpeg") {
+        encode_params = { cv::IMWRITE_JPEG_QUALITY, param.quality };
+    }
+
+    bool ret = imwrite(filepath, image, encode_params);
+    LogInfo << "screencap saved to" << filepath << VAR(ret);
+
</code_context>
<issue_to_address>
**suggestion:** Add validation/clamping for `format` and `quality` to avoid undefined or backend-dependent behavior.

`param.format` is only matched against lowercase "jpg"/"jpeg" and anything else (or different casing) falls back to OpenCV defaults with no quality parameter. `param.quality` is also passed through unchecked, so values outside the expected `[0, 100]` range may behave inconsistently.

Consider normalizing `param.format` (e.g., lowercase), explicitly handling only supported formats, and clamping `param.quality` to a defined range before constructing `encode_params`. For unsupported formats, either fail fast (`ret = false`) or log a clear warning so behavior is predictable.

Suggested implementation:

```cpp
    const auto& option = MAA_GLOBAL_NS::OptionMgr::get_instance();

    // normalize format to lowercase to avoid casing issues
    std::string normalized_format = param.format;
    std::transform(normalized_format.begin(), normalized_format.end(), normalized_format.begin(),
                   [](unsigned char c) { return static_cast<char>(std::tolower(c)); });

    std::string ext = "." + normalized_format;
    std::string filename = param.filename.empty()
                               ? std::format("{}_{}{}", format_now_for_filename(), name, ext)
                               : param.filename + ext;
    auto filepath = option.log_dir() / "screencap" / path(filename);

    // clamp quality into [0, 100] to keep behavior well-defined across backends
    const int clamped_quality = std::clamp(param.quality, 0, 100);

    std::vector<int> encode_params;
    if (normalized_format == "jpg" || normalized_format == "jpeg") {
        encode_params = { cv::IMWRITE_JPEG_QUALITY, clamped_quality };
    } else if (normalized_format != "png" && normalized_format != "bmp") {
        // Log explicitly for unsupported formats so behavior is predictable
        LogWarn << "Unsupported screencap format: " << param.format
                << ", falling back to OpenCV defaults without explicit quality parameter";
    }

    bool ret = imwrite(filepath, image, encode_params);
    LogInfo << "screencap saved to" << filepath << VAR(ret);


MAA_TASK_NS_BEGIN

```

To compile successfully, ensure:

1. `#include <algorithm>` is present at the top of `source/MaaFramework/Task/Component/Actuator.cpp` (for `std::transform` and `std::clamp`):
   ```cpp
   #include <algorithm>
   ```
   If it is not there yet, add it alongside the other standard library includes.

2. If your codebase uses a custom logging macro for warnings, confirm that `LogWarn` is the correct one; if not, replace `LogWarn` with the appropriate warning-level logging macro used elsewhere in this file.
</issue_to_address>

### Comment 2
<location path="source/MaaFramework/Resource/PipelineTypesV2.h" line_range="307" />
<code_context>
+{
+    std::string filename;
+    std::string format;
+    int quality = 0;
+
+    MEO_TOJSON(filename, format, quality);
</code_context>
<issue_to_address>
**issue (bug_risk):** Align JScreencap `quality` default with the C++ and Python defaults.

`Action::ScreencapParam` and the Python `JScreencap` dataclass both default `quality` to `100`, but this struct defaults it to `0`. As a result, V2 JSON produced by the dumper will include `quality = 100`, while code constructing V2 JSON without a `quality` field will assume `0`. Please change this default to `100` for consistency.
</issue_to_address>