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
3 changes: 3 additions & 0 deletions .jules/bolt.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
## 2026-08-22 - [Time Formatting Array Lookups]
**Learning:** For high-throughput render paths in React `ink` terminal UIs, prefer pre-computed array lookups for bounded data (like time formatting 0-59) over repeated string allocations (like `String().padStart()`) to reduce performance overhead.
**Action:** When implementing rendering logic that formats small ranges of numbers, utilize pre-computed lookup tables to minimize string allocations.
10 changes: 7 additions & 3 deletions src/cli/ui/components/messageList/utils.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
const PAD_2 = Array.from({ length: 60 }, (_, i) => (i < 10 ? `0${i}` : `${i}`));

export function formatTime(timestamp: Date): string {
const hours = String(timestamp.getHours()).padStart(2, '0');
const minutes = String(timestamp.getMinutes()).padStart(2, '0');
const seconds = String(timestamp.getSeconds()).padStart(2, '0');
// Performance: Pre-computed array lookups for 0-59 are faster than String().padStart()
// avoiding repeated string allocations in high-throughput render paths.
const hours = PAD_2[timestamp.getHours()];
const minutes = PAD_2[timestamp.getMinutes()];
const seconds = PAD_2[timestamp.getSeconds()];
return `${hours}:${minutes}:${seconds}`;
}
Loading