Address embik's comments

Signed-off-by: Dr. Stefan Schimanski <[email protected]>

Author: sttts

pkg/multicluster/multicluster.go

@@ -30,6 +30,9 @@ type Aware interface {
 	// The given context is tied to the Cluster's lifecycle and will be cancelled when the
 	// Cluster is removed or an error occurs.
 	//
+	// Engage is a no-op if the passed cluster is equal to the existing, already
+	// engaged cluster (equal means interface equality, i.e. the same instance).
+	//
 	// Implementers should return an error if they cannot start operations for the given Cluster,
 	// and should ensure this operation is re-entrant and non-blocking.
 	//

pkg/source/kind.go

@@ -56,7 +56,7 @@ func TypedKind[object client.Object, request mcreconcile.ClusterAware[request]](
 		obj:        obj,
 		handler:    handler,
 		predicates: predicates,
-		project:    func(_ cluster.Cluster, o object) (object, error) { return o, nil },
+		project:    func(_ cluster.Cluster, obj object) (object, error) { return obj, nil },
 		resync:     0, // no periodic resync by default
 	}
 }