[Bug] Configuration Values Ignored in Task Integrity Checks

Resolved 💬 3 comments Opened Sep 19, 2025 by DocChrisKing Closed Jan 6, 2026

Bug Description
⏺ THE REAL PROBLEM - Configuration Values NOT Used!

You're absolutely right. The code has HARDCODED thresholds instead of using the configuration:

In TaskCheckInfo.determineCheckFrequency():

// HARDCODED VALUES - NOT USING CONFIG!
if (daysSinceLastEvent < 7) Always // Should use activeThresholdDays
else if (daysSinceLastEvent < 30) Daily // Should use recentThresholdDays
else if (daysSinceLastEvent < 90) Weekly // Should use staleThresholdDays
else Monthly

What Needs to be Fixed:

The performOptimizedTaskIntegrityChecks method needs to:

  1. Accept IntegrityConfiguration as a parameter
  2. Pass config values to TaskCheckInfo and IntegrityCheckStrategy
  3. Use config.activeThresholdDays, config.recentThresholdDays, config.staleThresholdDays
  4. Use config.batchSize for processing
  5. Use config.maxConcurrentChecks for parallelism

Let me create a proper fix that actually USES the configuration values:

⏺ This is a MASSIVE oversight. The optimization code exists but doesn't use ANY of the configuration values. It's all hardcoded. This needs to be completely
refactored to pass the configuration through the entire chain.

This is why you're seeing no effect - the code ignores all the carefully designed configuration parameters and uses hardcoded values instead

THe above is what claude did AFTER we did a big plan and wrote it all into a detailed.md file. 2 days wasted

Environment Info

  • Platform: darwin
  • Terminal: WarpTerminal
  • Version: 1.0.111
  • Feedback ID: dd366b57-1841-4455-998e-a0c2e5d9fd7e

Errors

[]

View original on GitHub ↗

This issue has 3 comments on GitHub. Read the full discussion on GitHub ↗