[turbopack] Replace uses of `Value<ExecutionEnvironment>` with just `ExecutionEnvironment` (#80135)
Remove Value<ExecutionEnvironment> instead just use ExecutionEnvironment
Why?
The Value<> type is confusing and error prone, lets destroy it!
As discussed over slack, turbo_tasks::Value type always implements is_resolved as true and is_transient as false which is incorrect and can enable smuggling transient vcs into turbo tasks.
• https://vercel.slack.com/archives/C03EWR7LGEN/p1743456121241529
• https://vercel.slack.com/archives/C04NGV98TPS/p1743455453764879