Fix rapid cache cleanup logic bugs

- Fix Redis error skipping bucket cleanup: Fall back to GCS timestamp when Redis LastAccess fails
- Fix index deletes skipping access recheck: Recheck LastAccess before deletion to prevent removing recently touched objects
- Fix index cleanup error blocking bucket scan: Log error and continue to cleanFromBucket even if cleanFromIndex fails

Author: cursoragent

packages/orchestrator/cmd/clean-rapid-cache/main.go

@@ -73,7 +73,7 @@ func clean(ctx context.Context, bucket string, prefix string, cutoff time.Time,
 
 	deleted, err := cleanFromIndex(ctx, client, bucket, cutoff, maxDeletions, dryRun, index)
 	if err != nil {
-		return err
+		log.Printf("cleanFromIndex failed (continuing with bucket scan): %v", err)
 	}
 
 	return cleanFromBucket(ctx, client, bucket, prefix, cutoff, maxDeletions-deleted, dryRun, index)
@@ -99,6 +99,10 @@ func cleanFromIndex(ctx context.Context, client *gcs.Client, bucket string, cuto
 		if err != nil {
 			return deleted, fmt.Errorf("read cache object metadata: %w", err)
 		}
+		lastAccess, ok, err := index.LastAccess(ctx, path)
+		if err == nil && ok && lastAccess >= cutoff.Unix() {
+			continue
+		}
 		if dryRun {
 			deleted++
 
@@ -135,10 +139,7 @@ func cleanFromBucket(ctx context.Context, client *gcs.Client, bucket string, pre
 			continue
 		}
 		lastAccess, ok, err := index.LastAccess(ctx, attrs.Name)
-		if err != nil {
-			continue
-		}
-		if ok && lastAccess >= cutoff.Unix() {
+		if err == nil && ok && lastAccess >= cutoff.Unix() {
 			continue
 		}
 		matched++