@growth-operations/ui-shared 1.6.9 → 1.6.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@growth-operations/ui-shared",
-  "version": "1.6.9",
+  "version": "1.6.11",
   "description": "Shared HubSpot UI-extension modules (Home, Billing) for Growth Operations apps.",
   "license": "MIT",
   "private": false,

package/src/alerts/AlertsTab.jsx

@@ -166,10 +166,25 @@ export function AlertsTab({
 
   // Optimistically remove a dismissed alert from local state instead of
   // re-fetching the whole list. Keeps the dismiss interaction to a single API
-  // call (the dismiss itself).
+  // call (the dismiss itself) in the common case. But if removing the row empties
+  // the current page while other pages still have alerts, fall back to a fetch so
+  // the user isn't stranded on a falsely-empty "No alerts" view: backfill this
+  // page, or step back a page if we just cleared the last one.
   const removeAlertLocally = (id) => {
-    setAlerts((prev) => prev.filter((a) => a.id !== id));
-    setTotal((t) => Math.max(0, t - 1));
+    const remainingHere = alerts.filter((a) => a.id !== id);
+    const newTotal = Math.max(0, total - 1);
+
+    if (remainingHere.length === 0 && newTotal > 0) {
+      // Page emptied but more remain — reload. If this was the last page and it's
+      // now gone, step back one; otherwise reload the same page to pull the next
+      // page's rows up into it.
+      const lastPage = Math.max(1, Math.ceil(newTotal / PAGE_SIZE));
+      load(Math.min(page, lastPage), level);
+      return;
+    }
+
+    setAlerts(remainingHere);
+    setTotal(newTotal);
   };
 
   useEffect(() => {

package/src/home/InstallProgress.jsx

@@ -1,4 +1,4 @@
-import React from "react";
+import React, { useRef } from "react";
 import {
   Flex,
   Tile,
@@ -30,13 +30,22 @@ import {
 //              activation Pub/Sub message via an app endpoint). If omitted, the
 //              FAILED state still renders guidance, just no button.
 export function InstallProgress({ state, onRetry }) {
+  // Clamp the displayed step to be MONOTONIC. A reinstall (re-consent for new
+  // scopes) re-runs activation, which resets activation_step to 0 server-side;
+  // the poll can catch that reset and the bar would visibly jump backwards
+  // (e.g. 37% -> 0% -> 53%). Track the high-water mark per mount so progress
+  // only ever moves forward.
+  const maxStepSeen = useRef(0);
+
   const activation = state?.activation;
   // No activation block => nothing to show (Home's health hero takes over).
   if (!activation) return null;
 
   const status = activation.status;
-  const step = activation.step ?? 0;
+  const rawStep = activation.step ?? 0;
   const total = activation.total ?? 5;
+  maxStepSeen.current = Math.max(maxStepSeen.current, rawStep);
+  const step = maxStepSeen.current;
   const warnings = activation.warnings ?? [];
   const complete = activation.complete === true || status === "completed";