refactor: remove unused UUID parsing function and simplify claimed status check in memory queue

Author: jsbroks

apps/workspace-engine/pkg/events/handler/jobs/jobs.go

@@ -9,14 +9,8 @@ import (
 	"workspace-engine/pkg/workspace"
 
 	"github.com/charmbracelet/log"
-	"github.com/google/uuid"
 )
 
-func isStringUUID(s string) bool {
-	_, err := uuid.Parse(s)
-	return err == nil
-}
-
 func HandleJobUpdated(
 	ctx context.Context,
 	ws *workspace.Workspace,

apps/workspace-engine/pkg/reconcile/memory/memory.go

@@ -153,7 +153,7 @@ func (q *Queue) Enqueue(ctx context.Context, params reconcile.EnqueueParams) err
 		s.ClaimedBy = ""
 		s.ClaimedUntil = nil
 	}
-	if !(s.ClaimedUntil != nil && s.ClaimedUntil.After(now)) {
+	if s.ClaimedUntil == nil || !s.ClaimedUntil.After(now) {
 		s.UpdatedAt = now
 	}
 

apps/workspace-engine/pkg/workspace/releasemanager/policy/evaluator/approval/approval.go

@@ -47,7 +47,6 @@ type Getters interface {
 
 type AnyApprovalEvaluator struct {
 	getters       Getters
-	store         *store.Store
 	ruleId        string
 	ruleCreatedAt string
 	rule          *oapi.AnyApprovalRule

apps/workspace-engine/svc/controllers/deploymentresourceselectoreval/controller.go

@@ -90,6 +90,8 @@ func (c *Controller) Process(ctx context.Context, item reconcile.Item) error {
 		}
 	}
 
+	print("resourceMatchedIds", resourceMatchedIds)
+
 	return nil
 }