@checkstack/anomaly-frontend 0.5.17 → 0.6.0

package/CHANGELOG.md

@@ -1,5 +1,103 @@
 # @checkstack/anomaly-frontend
 
+## 0.6.0
+
+### Minor Changes
+
+- 5236e41: Scope anomaly rows by (check, environment), completing the deferred follow-up
+  from the per-environment work in #375 (which env-scoped only baselines).
+
+  Previously the `anomalies` table was cross-environment: the inline spike
+  detector and the drift evaluator located and created the open row by
+  `(systemId, configurationId, fieldPath, kind)` with no environment predicate.
+  When a `(system, configuration)` assignment fanned out to multiple environments,
+  a healthy value in environment A shared one row with an anomaly in environment B,
+  so one env could mask (or merge with) another.
+
+  - **Schema.** New nullable `anomalies.environment_id` column (migration
+    `0007_uneven_trauma.sql`, a single `ADD COLUMN`). No unique constraint is
+    added: the table intentionally allows multiple rows per identity tuple (a
+    `recovered` historical row plus a fresh active row), so uniqueness would break
+    the state machine.
+  - **Detection.** The spike detector (from the `checkCompleted` hook) and the
+    drift evaluator (from the analyzer's per-environment loop) now locate/create
+    the open row by `(systemId, configurationId, environmentId, fieldPath, kind)`,
+    matching `environment_id = <id>` when present or `IS NULL` for the env-less
+    slice - mirroring the per-environment baseline lookup.
+  - **Reads.** `getAnomalies` gains an optional `environmentId` tristate filter
+    (`undefined` = all envs, `null` = env-less slice, string = that env), and both
+    `AnomalyDto` and `getActiveSignalAnomalies` surface `environmentId`. The
+    system-detail widget renders an environment pill on env-scoped anomaly rows.
+  - **Notifications.** An env-scoped anomaly appends its environment id to the
+    collapse key, so two failing environments render as two independent cards
+    instead of collapsing into one. The env-less slice keeps the pre-feature
+    two-segment key. Mutes stay env-agnostic (per system / per field).
+
+  BREAKING (semantics, not types; BETA so minor only):
+
+  - **Anomaly row identity now includes `environmentId`.** For a fanned-out check,
+    an anomaly in one environment is a distinct row from another environment. Any
+    code that assumed a single anomaly row per `(system, config, field, kind)`
+    must account for the environment dimension.
+  - **`AnomalyDto` and `getActiveSignalAnomalies` rows carry a new
+    `environmentId: string | null` field**, and `getAnomalies` accepts a new
+    optional `environmentId` filter. Additive on the wire; consumers that reject
+    unknown fields should be updated.
+  - **Upgrade behaviour.** Existing rows backfill to `null` (the env-less slice)
+    and stay until they recover; the next detection tick opens fresh
+    per-environment rows for fanned-out checks. This mirrors how #375 handled
+    baselines.
+
+  State and scale: the anomaly state lives entirely in the shared `anomalies`
+  Postgres table. `environmentId` is just another column on the row, so every pod
+  reads the same per-`(system, config, env, field, kind)` state - no pod-local
+  state, and reads return the same answer on every pod. The baseline cache key
+  already carries the env segment (#375), so there is no cross-env cache shadowing.
+
+### Patch Changes
+
+- Updated dependencies [52c55bf]
+- Updated dependencies [5236e41]
+- Updated dependencies [d1b71b6]
+- Updated dependencies [0d912a3]
+- Updated dependencies [a07b375]
+- Updated dependencies [d9f4654]
+- Updated dependencies [b45be8e]
+- Updated dependencies [d9f4654]
+- Updated dependencies [3420d24]
+- Updated dependencies [dea02f0]
+- Updated dependencies [21e0d88]
+- Updated dependencies [52c55bf]
+- Updated dependencies [935d34e]
+- Updated dependencies [e430fbe]
+- Updated dependencies [eab80e3]
+- Updated dependencies [259b93c]
+- Updated dependencies [53666a7]
+- Updated dependencies [b3b547c]
+- Updated dependencies [d2d49cf]
+- Updated dependencies [0d912a3]
+- Updated dependencies [0d912a3]
+- Updated dependencies [692fa18]
+  - @checkstack/anomaly-common@1.6.0
+  - @checkstack/healthcheck-common@1.10.0
+  - @checkstack/healthcheck-frontend@0.27.0
+  - @checkstack/notification-common@1.5.0
+  - @checkstack/notification-frontend@0.7.0
+  - @checkstack/ui@1.22.0
+  - @checkstack/frontend-api@0.13.0
+  - @checkstack/common@0.19.0
+  - @checkstack/catalog-common@2.6.0
+  - @checkstack/signal-frontend@0.3.2
+
+## 0.5.18
+
+### Patch Changes
+
+- Updated dependencies [baf9b6e]
+  - @checkstack/ui@1.21.0
+  - @checkstack/healthcheck-frontend@0.26.1
+  - @checkstack/notification-frontend@0.6.4
+
 ## 0.5.17
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/anomaly-frontend",
-  "version": "0.5.17",
+  "version": "0.6.0",
   "license": "Elastic-2.0",
   "type": "module",
   "sideEffects": [
@@ -16,16 +16,16 @@
     "lint:code": "eslint . --max-warnings 0"
   },
   "dependencies": {
-    "@checkstack/anomaly-common": "1.5.4",
-    "@checkstack/catalog-common": "2.5.0",
-    "@checkstack/common": "0.18.0",
-    "@checkstack/frontend-api": "0.12.1",
-    "@checkstack/healthcheck-common": "1.9.0",
-    "@checkstack/healthcheck-frontend": "0.26.0",
-    "@checkstack/notification-common": "1.4.2",
-    "@checkstack/notification-frontend": "0.6.3",
-    "@checkstack/signal-frontend": "0.3.1",
-    "@checkstack/ui": "1.20.0",
+    "@checkstack/anomaly-common": "1.6.0",
+    "@checkstack/catalog-common": "2.6.0",
+    "@checkstack/common": "0.19.0",
+    "@checkstack/frontend-api": "0.13.0",
+    "@checkstack/healthcheck-common": "1.10.0",
+    "@checkstack/healthcheck-frontend": "0.27.0",
+    "@checkstack/notification-common": "1.5.0",
+    "@checkstack/notification-frontend": "0.7.0",
+    "@checkstack/signal-frontend": "0.3.2",
+    "@checkstack/ui": "1.22.0",
     "date-fns": "^4.4.0",
     "lucide-react": "^1.17.0",
     "react": "19.2.7",
@@ -33,7 +33,7 @@
     "zod": "^4.2.1"
   },
   "devDependencies": {
-    "@checkstack/scripts": "0.6.5",
+    "@checkstack/scripts": "0.7.0",
     "@checkstack/tsconfig": "0.0.7",
     "@types/react": "^19.0.0",
     "typescript": "^5.0.0"

package/src/components/AnomalySignalsFiller.tsx

@@ -47,6 +47,7 @@ export const AnomalySignalsFiller: React.FC<Props> = ({
       .map((a) => ({
         systemId: a.systemId,
         configurationId: a.configurationId,
+        environmentId: a.environmentId,
         fieldPath: a.fieldPath,
         startedAt: a.startedAt,
         state: a.state,

package/src/components/SystemAnomalyWidget.tsx

@@ -5,7 +5,7 @@ import {
   accessApiRef,
   type SlotContext,
 } from "@checkstack/frontend-api";
-import { SystemDetailsSlot } from "@checkstack/catalog-common";
+import { SystemDetailsSlot, CatalogApi } from "@checkstack/catalog-common";
 import {
   AnomalyApi,
   anomalyAccess,
@@ -33,6 +33,7 @@ import {
   Bell,
   BellOff,
   EyeOff,
+  Layers,
 } from "lucide-react";
 import { formatDistanceToNow } from "date-fns";
 import { Link } from "react-router-dom";
@@ -123,6 +124,7 @@ function humanizeCollectorSource(strategyId: string, collectorId: string): strin
 function AnomalyRow({
   anomaly,
   systemId,
+  environmentLabel,
   isMuted,
   onToggleMute,
   isToggling,
@@ -133,6 +135,12 @@ function AnomalyRow({
 }: {
   anomaly: AnomalyDto;
   systemId: string;
+  /**
+   * Resolved environment label for an env-scoped anomaly (rendered as a pill
+   * next to the field name). `undefined` for the env-less slice / cross-env
+   * feed - no pill, matching the pre-per-env single-row look.
+   */
+  environmentLabel?: string;
   isMuted: boolean;
   onToggleMute: (fieldPath: string, isMuted: boolean) => void;
   isToggling: boolean;
@@ -198,6 +206,12 @@ function AnomalyRow({
               {parsed.source}
             </span>
           ) : undefined}
+          {environmentLabel ? (
+            <span className="inline-flex shrink-0 items-center gap-1 rounded-full border border-border/60 bg-surface-inset px-1.5 py-0.5 text-[10px] text-muted-foreground">
+              <Layers className="h-2.5 w-2.5" />
+              {environmentLabel}
+            </span>
+          ) : undefined}
           {isMuted && (
             <span className="text-[10px] text-muted-foreground bg-muted/60 px-1.5 py-0.5 rounded font-medium shrink-0 inline-flex items-center gap-0.5">
               <BellOff className="h-2.5 w-2.5" />
@@ -299,6 +313,7 @@ function AnomalyRow({
 
 export const SystemAnomalyWidget: React.FC<Props> = ({ system }) => {
   const anomalyClient = usePluginClient(AnomalyApi);
+  const catalogClient = usePluginClient(CatalogApi);
   const toast = useToast();
   const accessApi = useApi(accessApiRef);
   // Muting notifications is a per-user preference any LOGGED-IN user may set
@@ -329,6 +344,19 @@ export const SystemAnomalyWidget: React.FC<Props> = ({ system }) => {
       { staleTime: 30_000 },
     );
 
+  // Resolve env names for env-scoped anomaly rows (two envs of the same field
+  // now surface as two rows). Same query the healthcheck overview/drawer issue,
+  // so the cache entry is shared when those surfaces mount alongside this one.
+  const { data: systemEnvironments = [] } =
+    catalogClient.getSystemEnvironments.useQuery(
+      { systemId: system.id },
+      { staleTime: 30_000 },
+    );
+  const envNameById = React.useMemo(
+    () => new Map(systemEnvironments.map((e) => [e.id, e.name])),
+    [systemEnvironments],
+  );
+
   const mutedFields = React.useMemo(
     () => new Set(mutes.map((m) => m.fieldPath)),
     [mutes],
@@ -502,6 +530,12 @@ export const SystemAnomalyWidget: React.FC<Props> = ({ system }) => {
               key={anomaly.id}
               anomaly={anomaly}
               systemId={system.id}
+              environmentLabel={
+                anomaly.environmentId
+                  ? (envNameById.get(anomaly.environmentId) ??
+                    anomaly.environmentId)
+                  : undefined
+              }
               isMuted={isSystemMuted || mutedFields.has(anomaly.fieldPath)}
               onToggleMute={handleToggleMute}
               isToggling={isToggling}