fix: max hit (#1193)

max_hit should only be restricted to PipelineTask

---

https://github.com/MaaEnd/MaaEnd/actions/runs/22839301359?pr=1112

## Summary by Sourcery

将 `max_hit` 的跟踪和限制仅应用于 `PipelineTask` 的识别流程，而不是共享的 `TaskBase` 实现。

Bug Fixes:
- 确保只有在 `PipelineTask` 中达到 `max_hit` 时才跳过识别尝试，从而避免对其他任务类型施加意外的限制。

Enhancements:
- 当在 `PipelineTask` 中达到 `max_hit` 以及发生识别命中时，记录 debug 和 info
日志消息，以提高可观测性。

<details>
<summary>Original summary in English</summary>

## Summary by Sourcery

Limit max_hit tracking and enforcement to PipelineTask recognition flow
instead of the shared TaskBase implementation.

Bug Fixes:
- Ensure recognition attempts are skipped when max_hit is reached only
within PipelineTask, preventing unintended limits on other task types.

Enhancements:
- Log debug and info messages when max_hit is reached and when
recognition hits occur within PipelineTask for better observability.

</details>

Author: neko-para

source/MaaFramework/Task/Context.cpp

@@ -393,6 +393,16 @@ bool& Context::need_to_stop()
     return *need_to_stop_;
 }
 
+bool Context::check_hit_count(const PipelineData& data)
+{
+    size_t current_hit = get_hit_count(data.name);
+    if (current_hit >= static_cast<size_t>(data.max_hit)) {
+        LogDebug << "max_hit reached" << VAR(data.name) << VAR(current_hit) << VAR(data.max_hit);
+        return false;
+    }
+    return true;
+}
+
 void Context::increment_hit_count(const std::string& node_name)
 {
     task_state_->hit_count[node_name]++;

source/MaaFramework/Task/Context.h

@@ -69,6 +69,7 @@ class Context
     std::vector<cv::Mat> get_images(const std::vector<std::string>& names);
 
     bool& need_to_stop();
+    bool check_hit_count(const PipelineData& data);
     void increment_hit_count(const std::string& node_name);
 
 private:

source/MaaFramework/Task/PipelineTask.cpp

@@ -282,8 +282,22 @@ RecoResult PipelineTask::recognize_list(const cv::Mat& image, const std::vector<
             recognizer.prefetch_batch_ocr(batch_plan->entries);
         }
 
+        if (!pipeline_data.enabled) {
+            LogDebug << "node disabled" << pipeline_data.name << VAR(pipeline_data.enabled);
+            return { };
+        }
+
+        if (!context_->check_hit_count(pipeline_data)) {
+            continue;
+        }
+
         RecoResult result = run_recognition(image, pipeline_data, ocr_cache);
 
+        if (result.box) {
+            LogInfo << "reco hit" << VAR(result.name) << VAR(result.box);
+            context_->increment_hit_count(pipeline_data.name);
+        }
+
         if (context_->need_to_stop()) {
             LogWarn << "need_to_stop";
             break;
@@ -323,8 +337,7 @@ std::optional<PipelineTask::BatchOCRPlan> PipelineTask::prepare_batch_ocr(const
             continue;
         }
 
-        size_t current_hit = context_->get_hit_count(data.name);
-        if (current_hit >= static_cast<size_t>(data.max_hit)) {
+        if (!context_->check_hit_count(data)) {
             continue;
         }
 

source/MaaFramework/Task/TaskBase.cpp

@@ -61,17 +61,6 @@ RecoResult TaskBase::run_recognition(const cv::Mat& image, const PipelineData& d
         return { };
     }
 
-    if (!data.enabled) {
-        LogDebug << "node disabled" << data.name << VAR(data.enabled);
-        return { };
-    }
-
-    size_t current_hit = context_->get_hit_count(data.name);
-    if (current_hit >= static_cast<size_t>(data.max_hit)) {
-        LogDebug << "max_hit reached" << VAR(data.name) << VAR(current_hit) << VAR(data.max_hit);
-        return { };
-    }
-
     Recognizer recognizer(tasker_, *context_, image, std::move(ocr_cache));
 
     json::value cb_detail {
@@ -93,11 +82,6 @@ RecoResult TaskBase::run_recognition(const cv::Mat& image, const PipelineData& d
     cb_detail["reco_details"] = result;
     notify(result.box ? MaaMsg_Node_Recognition_Succeeded : MaaMsg_Node_Recognition_Failed, cb_detail);
 
-    if (result.box) {
-        LogInfo << "reco hit" << VAR(result.name) << VAR(result.box);
-        context_->increment_hit_count(data.name);
-    }
-
     return result;
 }