fix(cron): compute correctly time boundaries avoiding re-triggering

Signed-off-by: Ettore Di Giacinto <mudler@localai.io>

Author: mudler

core/services/jobs/dispatcher.go

@@ -459,6 +459,10 @@ func (d *Dispatcher) isCronDue(task TaskRecord) bool {
 		return true
 	}
 
+	// Compute the cron interval from two consecutive ticks after the last job.
+	// Use elapsed time since the last job rather than clock-aligned ticks to
+	// avoid re-triggering when the job was created shortly before a tick boundary.
 	nextRun := schedule.Next(lastJob.CreatedAt)
-	return time.Now().After(nextRun)
+	minInterval := schedule.Next(nextRun).Sub(nextRun)
+	return time.Since(lastJob.CreatedAt) >= minInterval
 }

core/services/nodes/file_transfer_server.go

@@ -309,7 +309,15 @@ func validatePathInDir(targetPath, baseDir string) error {
 	}
 	realTarget, err := filepath.EvalSymlinks(absTarget)
 	if err != nil {
-		realTarget = filepath.Clean(absTarget)
+		// File may not exist yet (e.g. upload). Resolve the parent directory
+		// and re-join the filename so symlinks like /tmp -> /private/tmp on
+		// macOS are still resolved correctly.
+		parentReal, perr := filepath.EvalSymlinks(filepath.Dir(absTarget))
+		if perr == nil {
+			realTarget = filepath.Join(parentReal, filepath.Base(absTarget))
+		} else {
+			realTarget = filepath.Clean(absTarget)
+		}
 	}
 
 	if !strings.HasPrefix(realTarget, realBase+string(filepath.Separator)) && realTarget != realBase {