feat: Introduce ScrollableRow - #337

packages/ui/src/components/scrollable-row.tsx (1)

50-64: Avoid reattaching listeners on every children change.

Including children in the dependency array causes the scroll listener and ResizeObserver to be torn down and reattached each time children changes. React best practices recommend keeping the listener setup stable by depending only on the ref, and triggering updateScrollState() in a separate effect that depends on children. This pattern prevents unnecessary DOM event listener reattachments while still allowing the handler to respond to content changes.

packages/kepler/src/KeplerSlice.ts (1)

357-362: Consider extracting a helper function for updating lastOpenedAt.

The pattern of finding a map and updating its lastOpenedAt timestamp is repeated in setCurrentMapId, deleteMap (twice). A small helper could reduce duplication:

function updateMapLastOpenedAt(
  maps: KeplerSliceConfig['maps'],
  mapId: string
) {
  const map = maps.find((m) => m.id === mapId);
  if (map) map.lastOpenedAt = Date.now();
}

This is a minor suggestion—the current code is clear and works correctly.

Also applies to: 439-444

packages/ai-core/src/AiSlice.ts (1)

628-628: Consider capturing Date.now() before the produce callback for consistency.

This works correctly, but other methods (createSession, switchSession) capture now before the produce call. For consistency, you could follow the same pattern here.

♻️ Optional consistency improvement

         deleteSession: (sessionId: string) => {
           // Clean up per-session state
           const abortController = sessionAbortControllers.get(sessionId);
           if (abortController) {
             abortController.abort(SESSION_DELETED);
           }
           sessionAbortControllers.delete(sessionId);
           sessionChatStops.delete(sessionId);
           sessionChatSendMessages.delete(sessionId);
           sessionAddToolResults.delete(sessionId);

+          const now = Date.now();
           set((state) =>
             produce(state, (draft) => {
               // ... existing code ...
                     if (firstSession) {
                       draft.ai.config.currentSessionId = firstSession.id;
-                      firstSession.lastOpenedAt = Date.now();
+                      firstSession.lastOpenedAt = now;
                     }