Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .jules/bolt.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,9 @@
## 2026-07-12 - Optimize renderTaskRow DOM allocations
**Learning:** Caching unattached template nodes and instantiating them via `.cloneNode(false)` reduces DOM instantiation overhead in O(N) render loops significantly.
**Action:** Apply this optimization to other hot-path rendering elements such as rows, cells, and stack containers.
## 2026-07-12 - Replacing Array callbacks and Map with for loops and Int32Array
**Learning:** For high-performance O(N) loops in JavaScript, replacing `Array.prototype.reduce`/`forEach` and `Map` caching with standard `for` loops and typed arrays (e.g., `Int32Array`) eliminates JS engine callback allocation, garbage collection, and hash-lookup overhead.
**Action:** Apply standard `for` loops and typed arrays for numerical caches instead of higher-order array methods and Maps on hot paths or large datasets to ensure maximum performance.
## 2026-08-28 - Float64Array over Int32Array for Numeric Safety
**Learning:** Using `Int32Array` in JavaScript for caching standard numeric values can introduce silent truncation errors if the values ever become fractional or `NaN`, which breaks math operations down the line.
**Action:** Default to `Float64Array` or a standard `Array(length)` when caching calculated numbers unless the type is strictly known and guaranteed to be an integer.
23 changes: 14 additions & 9 deletions app.js
Original file line number Diff line number Diff line change
Expand Up @@ -1370,21 +1370,26 @@ function validateDateRange(startLabel, startValue, endLabel, endValue, errors) {
}

function computeTaskMetrics() {
// ⚡ Bolt: Cache durationDays during total calculation to avoid recalculating for every task
const durationCache = new Map();
const totalDays = state.tasks.reduce((sum, task) => {
// ⚡ Bolt: Optimize with standard for loops and Float64Array to avoid JS callback overhead, GC overhead and hash lookups for duration tracking
const tasksLen = state.tasks.length;
const durationCache = new Float64Array(tasksLen);
let totalDays = 0;

for (let i = 0; i < tasksLen; i++) {
const task = state.tasks[i];
const duration = calculateDurationDays(task.plannedStartDate, task.plannedEndDate);
durationCache.set(task.id, duration);
return sum + duration;
}, 0);
durationCache[i] = duration;
totalDays += duration;
}

const baseDate = state.baseDate;
const byTask = new Map();
let totalWeightedPlannedRatio = 0;
let totalWeightedActualRatio = 0;

state.tasks.forEach((task) => {
const durationDays = durationCache.get(task.id);
for (let i = 0; i < tasksLen; i++) {
const task = state.tasks[i];
const durationDays = durationCache[i];
Comment on lines +1375 to +1392

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 Info: Index-based cache changes duplicate-id behavior

The previous durationCache Map keyed on task.id; the new one keys on array index. For duplicate ids the old code shared one cached duration, the new code gives each task its own. Both loops iterate state.tasks in the same order, so unique-id behavior is unchanged.

Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

const weightRatio = totalDays > 0 ? durationDays / totalDays : 0;
const plannedProgressRatio = calculatePlannedProgressRatio(baseDate, task.plannedStartDate, task.plannedEndDate, durationDays);
const actualProgressRatio = (ACTUAL_PROGRESS_MAP[task.actualProgressStatus] || 0) / 100;
Expand All @@ -1408,7 +1413,7 @@ function computeTaskMetrics() {
plannedDateWarning,
actualDateWarning
});
});
}
Comment thread
devin-ai-integration[bot] marked this conversation as resolved.

return {
totalDays,
Expand Down
Loading