Fix clang-tidy findings in scheduler headers.

Remove redundant member init on WaitEntry::slot and explicitly delete Scheduler copy/move operations to satisfy cppcoreguidelines-special-member-functions.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: TrentHouliston

src/threading/scheduler/Group.hpp

@@ -68,7 +68,7 @@ namespace threading {
                 /// token decrement, and the loser skips its own adjustment. This makes the
                 /// keep/hand-back decision exact regardless of how many other waiters are parked,
                 /// instead of inferring it from the (unreliable) emptiness of the wait buckets.
-                std::shared_ptr<std::atomic<bool>> slot{};
+                std::shared_ptr<std::atomic<bool>> slot;
             };
 
             struct DrainResult {

src/threading/scheduler/Scheduler.hpp

@@ -59,6 +59,11 @@ namespace threading {
              */
             ~Scheduler();
 
+            Scheduler(const Scheduler&)            = delete;
+            Scheduler(Scheduler&&)                 = delete;
+            Scheduler& operator=(const Scheduler&) = delete;
+            Scheduler& operator=(Scheduler&&)      = delete;
+
             /**
              * Starts the scheduler, and begins executing tasks.
              *