feat: 添加搜索页面缓存清理任务到可用任务列表，并调整首次执行基准时间

[ModerRAS]

这个 pull 请求为 Telegram 机器人中的计划任务管理功能引入了改进，具体增强了任务可见性和执行逻辑。主要变化包括更新帮助信息以列出所有可用任务，并优化确定计划任务何时运行的逻辑，确保新任务在首次检查时不会被遗漏。

计划器帮助信息改进：

• 更新了ScheduledTaskController.cs中的/scheduler history帮助信息，将SearchPageCacheCleanup作为可用任务列出，使用户更清楚哪些任务可以管理。

任务执行逻辑增强：

• 改进了 SchedulerService.cs 中计划任务的基线计算，以便如果任务从未被执行过，其第一个符合条件的运行时间会设置在当前时间稍早（使用检查间隔或默认一分钟）。这可以防止新任务在初次执行检查时被跳过。

[Copilot]

Pull Request Overview

This PR enhances the scheduled task management functionality in the Telegram bot by improving task visibility and execution logic. The changes ensure that newly scheduled tasks are properly executed on their first check and that users have better visibility into available tasks.

• Added SearchPageCacheCleanup to the available tasks list in help information
• Modified task execution baseline calculation to prevent new tasks from being skipped on first execution
• Improved scheduler help message to include all available tasks

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
TelegramSearchBot/Service/Scheduler/SchedulerService.cs	Enhanced baseline time calculation for first-time task execution to prevent skipping
TelegramSearchBot/Controller/Manage/ScheduledTaskController.cs	Updated help message to include SearchPageCacheCleanup in available tasks list

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[claude]

Claude finished @ModerRAS's task —— View job

---

Code Review Completed ✅

Review Checklist:

• Read and analyze changed files
• Review code quality and best practices
• Check for potential bugs or issues
• Evaluate performance considerations
• Assess security concerns
• Review test coverage implications
• Provide constructive feedback

Overall Assessment

This is a well-implemented enhancement that improves the scheduler functionality with minimal risk. The changes are focused, logical, and follow good practices.

Code Quality Analysis ✅

SchedulerService.cs:147-150 - The baseline time calculation improvement is excellent:

• ✅ Good logic: Using _checkInterval as baseline offset prevents first-execution skipping
• ✅ Fallback safe: Default TimeSpan.FromMinutes(1) provides sensible default
• ✅ Clear comments: Chinese comments explain the rationale clearly
• ✅ Maintains compatibility: No breaking changes to existing behavior

ScheduledTaskController.cs:87 - Help text update:

• ✅ User-friendly: Adding the new task to help text improves discoverability
• ✅ Consistent formatting: Maintains existing markdown format
• ✅ Accurate information: Task name matches the actual implementation

Security & Performance ✅

• ✅ No security concerns: Changes don't introduce new attack vectors
• ✅ Performance neutral: Baseline calculation is O(1) and runs once per task check
• ✅ Memory efficient: No additional memory allocations or leaks

Potential Improvements 🔄

1. Test Coverage: Consider adding unit tests for the baseline calculation logic in SchedulerService.ShouldExecuteTaskAsync() when lastExecution == null

2. Magic Numbers: The TimeSpan.FromMinutes(1) fallback could be a constant for better maintainability:

   private readonly TimeSpan _defaultBaselineOffset = TimeSpan.FromMinutes(1);

3. Documentation: The enhancement could be documented in code comments for future maintainers

Minor Suggestions 💡

• Consider adding a null check for _checkInterval in the baseline calculation, though the current logic handles it correctly
• The improvement could benefit from a integration test that verifies new tasks execute on their first scheduled run

Recommendation: APPROVE ✅

This is a low-risk, high-value improvement that fixes a real issue where newly scheduled tasks could be skipped on their first execution. The code is well-written, follows existing patterns, and maintains backward compatibility.

Impact: Users will now see newly added tasks execute immediately when their schedule conditions are met, improving the reliability of the scheduler system.

---

[github-actions]

🔍 PR检查报告

📋 检查概览

• PR: feat: 添加搜索页面缓存清理任务到可用任务列表，并调整首次执行基准时间 #161
• 分支: feature/search-enhanced → master
• 触发事件: pull_request
• 提交: 5dd039b

🧪 测试结果

平台	状态	详情
Ubuntu	🔴 失败	测试结果不可用
Windows	🔴 失败	测试结果不可用

📊 代码质量

• ✅ 代码格式化检查
• ✅ 安全漏洞扫描
• ✅ 依赖包分析
• ✅ 代码覆盖率收集

📁 测试产物

• 测试结果文件已上传为artifacts
• 代码覆盖率已上传到Codecov

🔗 相关链接

• PR页面
• Actions日志

---

此报告由GitHub Actions自动生成