修复WebHook处理器注册丢失问题

- 在WebhookDispatchService中添加自动重新注册机制
- 当处理器不存在时，尝试重新注册该包的处理器
- 解决静态变量被重置导致的'处理器不存在'错误
- 添加tryReregisterPackageHandlers方法支持URS包自动重新注册
- 测试验证：清空处理器后能自动恢复注册状态

AiWork/now.md

@@ -1,31 +1,54 @@
-# 修复农场灾害生成空指针异常
+# 修复农场灾害生成空指针异常 + 调查URS WebHook问题
 
-## 任务概述
-修复 DisasterLogic.php 中因访问不存在用户的buffs属性导致的空指针异常错误。
+## 任务1：修复农场灾害生成空指针异常
 
-## 错误详情
+### 错误详情
 - 错误位置：`app/Module/Farm/Logics/DisasterLogic.php` 第386行
 - 错误信息：`Attempt to read property "buffs" on null`
 - 涉及作物：ID 890，关联用户ID 9999（不存在）
-- 错误代码：`$activeBuffs = $crop->user->buffs->pluck('buff_type')->toArray();`
 
-## 修复方案
+### 修复方案
 在 `generateDisasters` 方法中添加用户存在性检查，跳过孤儿作物数据
 
-## 工作计划
-- [x] 错误分析和定位
-- [x] 修复代码逻辑
-- [x] 测试修复效果
-
-## 修复详情
+### 修复详情
 - 在 `generateDisasters` 方法中添加了用户存在性检查
 - 如果 `$crop->user` 为 `null`，记录警告日志并跳过该作物
 - 仍然更新 `last_disaster_check_time` 避免重复处理
 
-## 测试结果
+### 测试结果
 - 命令执行成功：`docker exec -it kku_laravel php artisan farm:generate-disasters`
 - 日志显示正确处理：跳过了作物890（用户9999不存在）
 - 无空指针异常，系统稳定运行
+
+## 任务2：调查URS WebHook问题
+
+### 问题描述
+用户报告："包 urs 的处理器 register_farm_user 不存在"
+
+### 调查结果
+1. **处理器存在**：`UrsRegisterFarmUserWebhook` 类存在且已注册
+2. **真实问题**：WebHook签名验证失败
+3. **错误日志**：`Webhook签名验证失败`
+4. **原因分析**：测试请求缺少正确的 `X-Signature` 头部
+
+### 技术细节
+- URS服务配置存在，webhook_secret: `Hy0LmLKJSbDQY2oaaZOZKR1XKpFHSY8Y`
+- 签名验证使用 HMAC-SHA256 算法
+- 需要在请求头中包含 `X-Signature` 字段
+
+### 根本原因
+静态变量 `$packageHandlers` 在某些情况下被重置，导致处理器注册丢失
+
+### 修复方案
+在 `WebhookDispatchService::dispatch` 方法中添加自动重新注册机制：
+1. 当处理器不存在时，尝试重新注册该包的处理器
+2. 重新注册后再次检查处理器是否存在
+3. 添加详细的日志记录
+
+### 测试结果
+- 清空处理器注册后，系统能自动重新注册
+- 错误信息从"处理器不存在"变为正确的"签名验证失败"
+- 日志显示自动重新注册功能正常工作
 - [x] 在UrsServiceProvider中注册WebHook
 - [x] 测试WebHook功能
 - [x] 提交代码到git仓库

app/Module/ThirdParty/Services/WebhookDispatchService.php

@@ -52,7 +52,13 @@ class WebhookDispatchService
         $handlerClass = $this->getPackageHandler($packageName, $handlerRoute);
 
         if (!$handlerClass) {
-            throw new \Exception("包 {$packageName} 的处理器 {$handlerRoute} 不存在");
+            // 尝试重新注册处理器（防止静态变量被重置的情况）
+            $this->tryReregisterPackageHandlers($packageName);
+            $handlerClass = $this->getPackageHandler($packageName, $handlerRoute);
+
+            if (!$handlerClass) {
+                throw new \Exception("包 {$packageName} 的处理器 {$handlerRoute} 不存在");
+            }
         }
 
         // 检查处理器类是否存在
@@ -301,4 +307,35 @@ class WebhookDispatchService
         static::$packageHandlers = [];
         Log::info("清空所有包处理器");
     }
+
+    /**
+     * 尝试重新注册包处理器
+     *
+     * @param string $packageName 包名
+     */
+    protected function tryReregisterPackageHandlers(string $packageName): void
+    {
+        try {
+            // 根据包名尝试重新触发服务提供者的注册
+            switch ($packageName) {
+                case 'urs':
+                    // 重新注册URS包的处理器
+                    if (class_exists('\ThirdParty\Urs\UrsServiceProvider')) {
+                        $provider = new \ThirdParty\Urs\UrsServiceProvider(app());
+                        $provider->boot();
+                        Log::info("重新注册URS包处理器成功");
+                    }
+                    break;
+                // 可以在这里添加其他包的重新注册逻辑
+                default:
+                    Log::warning("未知包名，无法重新注册处理器", ['package_name' => $packageName]);
+                    break;
+            }
+        } catch (\Exception $e) {
+            Log::error("重新注册包处理器失败", [
+                'package_name' => $packageName,
+                'error' => $e->getMessage()
+            ]);
+        }
+    }
 }