Retire hot ops at their authored stamps

[TrueDoctor]

No description provided.

[cubic-dev-ai]

1 issue found across 8 files

Confidence score: 4/5

• history.rs can substantially increase peak memory when loading large histories by keeping the input vector alongside a second deltas vector. Keep the supplied vector as History.deltas and derive the indexes from it.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="document/graph-storage/src/history.rs">

<violation number="1" location="document/graph-storage/src/history.rs:35">
P2: Loading a large history now retains the input vector while building a second `deltas` vector, substantially increasing peak memory. Keep the supplied vector as `History.deltas` and derive the indexes from it.</violation>
</file>

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

P2: Loading a large history now retains the input vector while building a second deltas vector, substantially increasing peak memory. Keep the supplied vector as History.deltas and derive the indexes from it.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. At document/graph-storage/src/history.rs, line 35:

<comment>Loading a large history now retains the input vector while building a second `deltas` vector, substantially increasing peak memory. Keep the supplied vector as `History.deltas` and derive the indexes from it.</comment>

<file context>
@@ -27,8 +32,9 @@ impl History {
 	pub fn from_ordered(deltas: Vec<Delta>) -> Self {
-		let index = deltas.iter().enumerate().map(|(position, delta)| (delta.id, position)).collect();
-		Self { deltas, index }
+		let mut history = Self::default();
+		deltas.into_iter().for_each(|delta| history.push(delta));
+		history
</file context>