[Bug] Configuration Values Ignored in Task Integrity Checks
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:
- Accept IntegrityConfiguration as a parameter
- Pass config values to TaskCheckInfo and IntegrityCheckStrategy
- Use config.activeThresholdDays, config.recentThresholdDays, config.staleThresholdDays
- Use config.batchSize for processing
- 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
[]This issue has 3 comments on GitHub. Read the full discussion on GitHub ↗