Revert "Properly handle execution time<->energy conversion in v8 host (#4884)" (#4927)

# Description of Changes

<!-- Please describe your change, mention any related tickets, and so on
here. -->

This reverts commit 458eac8.

This rolls back a pricing change that we made to V8 (TypeScript) based
modules. This will be applied as a hotfix to 2.2.0.

# API and ABI breaking changes

<!-- If this is an API or ABI breaking change, please apply the
corresponding GitHub label. -->

None

# Expected complexity level and risk

1 - Just a pricing change.

<!--
How complicated do you think these changes are? Grade on a scale from 1
to 5,
where 1 is a trivial change, and 5 is a deep-reaching and complex
change.

This complexity rating applies not only to the complexity apparent in
the diff,
but also to its interactions with existing and future code.

If you answered more than a 2, explain what is complex about the PR,
and what other components it interacts with in potentially concerning
ways. -->

# Testing

<!-- Describe any testing you've done, and any testing you'd like your
reviewers to do,
so that you're confident that all the changes work as expected! -->

- [x] Tested on staging - I didn't verify that the energy used went down
because I don't think I can do that on staging but at least I know this
doesn't appear to break anything.

Co-authored-by: joshua-spacetime <josh@clockworklabs.io>

Author: jdetter

crates/client-api-messages/src/energy.rs

@@ -16,17 +16,13 @@ pub struct EnergyQuanta {
 impl EnergyQuanta {
     pub const ZERO: Self = EnergyQuanta { quanta: 0 };
 
-    // per the comment on [`FunctionBudget::DEFAULT_BUDGET`]: 1 second of wasm runtime is roughtly 2 TeV
-    pub const PER_EXECUTION_SEC: Self = Self::new(2_000_000_000_000);
-    pub const PER_EXECUTION_NANOSEC: Self = Self::new(Self::PER_EXECUTION_SEC.get() / 1_000_000_000);
-
     #[inline]
-    pub const fn new(quanta: u128) -> Self {
+    pub fn new(quanta: u128) -> Self {
         Self { quanta }
     }
 
     #[inline]
-    pub const fn get(&self) -> u128 {
+    pub fn get(&self) -> u128 {
         self.quanta
     }
 

crates/core/src/host/v8/budget.rs

@@ -13,7 +13,7 @@ use core::ptr;
 use core::sync::atomic::Ordering;
 use core::time::Duration;
 use core::{ffi::c_void, sync::atomic::AtomicBool};
-use spacetimedb_client_api_messages::energy::{EnergyQuanta, FunctionBudget};
+use spacetimedb_client_api_messages::energy::FunctionBudget;
 use std::sync::Arc;
 use v8::{Isolate, IsolateHandle};
 
@@ -118,12 +118,7 @@ fn budget_to_duration(_budget: FunctionBudget) -> Duration {
 /// Returns [`EnergyStats`] for a reducer given its `budget`
 /// and the `duration` it took to execute.
 pub(super) fn energy_from_elapsed(budget: FunctionBudget, duration: Duration) -> EnergyStats {
-    let used = duration.as_nanos() * EnergyQuanta::PER_EXECUTION_NANOSEC.get();
-    // in order for duration_nanos * ev_per_ns >= u64::MAX:
-    //              duration_nanos >= u64::MAX / ev_per_ns
-    //              duration_nanos >= (9223372036854775 ns = 106.75 days)
-    // so it's unlikely we'll have to worry about it
-    let used = FunctionBudget::new(u64::try_from(used).unwrap_or(u64::MAX));
+    let used = duration_to_budget(duration);
     let remaining = budget - used;
     EnergyStats { budget, remaining }
 }