fix consistency issue with reading collectibles (vercel/turborepo#3878)

connect original task to collectibles reading to fix strongly consistency

move read collectibles tasks to task and scope instead of a separate cache

reading collectibles is no longer strongly consistent by default, it's opt-in now

crates/turbo-tasks-memory/src/cell.rs

@@ -11,6 +11,8 @@ use turbo_tasks::{
     TaskId, TurboTasksBackendApi,
 };
 
+use crate::MemoryBackend;
+
 #[derive(Default, Debug)]
 pub(crate) enum Cell {
     /// No content has been set yet, or it was removed for memory pressure
@@ -204,7 +206,11 @@ impl Cell {
         }
     }
 
-    pub fn assign(&mut self, content: CellContent, turbo_tasks: &dyn TurboTasksBackendApi) {
+    pub fn assign(
+        &mut self,
+        content: CellContent,
+        turbo_tasks: &dyn TurboTasksBackendApi<MemoryBackend>,
+    ) {
         match self {
             Cell::Empty => {
                 *self = Cell::Value {
@@ -287,7 +293,7 @@ impl Cell {
     }
 
     /// Drops the cell after GC. Will notify all dependent tasks and events.
-    pub fn gc_drop(self, turbo_tasks: &dyn TurboTasksBackendApi) {
+    pub fn gc_drop(self, turbo_tasks: &dyn TurboTasksBackendApi<MemoryBackend>) {
         match self {
             Cell::Empty => {}
             Cell::Recomputing {

crates/turbo-tasks-memory/src/gc.rs

@@ -152,7 +152,7 @@ impl GcQueue {
         &self,
         factor: u8,
         backend: &MemoryBackend,
-        turbo_tasks: &dyn TurboTasksBackendApi,
+        turbo_tasks: &dyn TurboTasksBackendApi<MemoryBackend>,
     ) -> Option<(GcPriority, usize, GcStats)> {
         // Process through the inactive propagation queue.
         while let Ok(task) = self.inactive_propagate_queue.pop() {