@checkstack/maintenance-backend 0.5.2 → 0.5.4

package/CHANGELOG.md

@@ -1,5 +1,53 @@
 # @checkstack/maintenance-backend
 
+## 0.5.4
+
+### Patch Changes
+
+- Updated dependencies [f676e11]
+- Updated dependencies [48c2080]
+  - @checkstack/common@0.6.2
+  - @checkstack/backend-api@0.6.0
+  - @checkstack/auth-common@0.5.5
+  - @checkstack/catalog-common@1.2.7
+  - @checkstack/command-backend@0.1.9
+  - @checkstack/integration-backend@0.1.9
+  - @checkstack/integration-common@0.2.5
+  - @checkstack/maintenance-common@0.4.5
+  - @checkstack/notification-common@0.2.5
+  - @checkstack/signal-common@0.1.6
+
+## 0.5.3
+
+### Patch Changes
+
+- 9551fd7: Fix creator display in incident and maintenance status updates
+
+  - Show the creator's profile name instead of UUID in status updates
+  - For maintenances, now properly displays the creator name (was missing)
+  - For incidents, replaces UUID with human-readable profile name
+  - System-generated updates (automatic maintenance transitions) show no creator
+
+- c208a5b: ### @checkstack/incident-backend
+
+  Added notifications for incident status changes via the "Add Update" functionality:
+
+  - Notifications are now sent when an incident is reopened (status changed from resolved)
+  - Notifications are now sent when an incident status changes to any new value
+  - Notifications are now sent when an incident is resolved via addUpdate
+  - Extracted `notifyAffectedSystems` into a reusable module with proper importance logic:
+    - Resolved incidents always use "info" importance (good news)
+    - Reopened/created/updated incidents derive importance from severity
+
+  ### @checkstack/maintenance-backend
+
+  Fixed missing notification in `closeMaintenance` handler - the "Close" button now sends a "completed" notification to subscribers.
+
+- Updated dependencies [e5079e1]
+- Updated dependencies [9551fd7]
+  - @checkstack/catalog-common@1.2.6
+  - @checkstack/maintenance-common@0.4.4
+
 ## 0.5.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/maintenance-backend",
-  "version": "0.5.2",
+  "version": "0.5.4",
   "type": "module",
   "main": "src/index.ts",
   "scripts": {
@@ -10,23 +10,24 @@
     "lint:code": "eslint . --max-warnings 0"
   },
   "dependencies": {
-    "@checkstack/backend-api": "0.5.1",
-    "@checkstack/maintenance-common": "0.4.2",
-    "@checkstack/notification-common": "0.2.3",
-    "@checkstack/catalog-common": "1.2.4",
-    "@checkstack/command-backend": "0.1.7",
-    "@checkstack/signal-common": "0.1.4",
-    "@checkstack/integration-backend": "0.1.7",
-    "@checkstack/integration-common": "0.2.3",
+    "@checkstack/backend-api": "0.5.2",
+    "@checkstack/maintenance-common": "0.4.4",
+    "@checkstack/notification-common": "0.2.4",
+    "@checkstack/catalog-common": "1.2.6",
+    "@checkstack/auth-common": "0.5.4",
+    "@checkstack/command-backend": "0.1.8",
+    "@checkstack/signal-common": "0.1.5",
+    "@checkstack/integration-backend": "0.1.8",
+    "@checkstack/integration-common": "0.2.4",
     "drizzle-orm": "^0.45.1",
     "zod": "^4.2.1",
-    "@checkstack/common": "0.6.0"
+    "@checkstack/common": "0.6.1"
   },
   "devDependencies": {
-    "@checkstack/drizzle-helper": "0.0.2",
-    "@checkstack/scripts": "0.1.0",
-    "@checkstack/test-utils-backend": "0.1.7",
-    "@checkstack/tsconfig": "0.0.2",
+    "@checkstack/drizzle-helper": "0.0.3",
+    "@checkstack/scripts": "0.1.1",
+    "@checkstack/test-utils-backend": "0.1.8",
+    "@checkstack/tsconfig": "0.0.3",
     "@orpc/server": "^1.13.2",
     "@types/bun": "^1.0.0",
     "drizzle-kit": "^0.31.8",

package/src/index.ts

@@ -15,6 +15,7 @@ import { integrationEventExtensionPoint } from "@checkstack/integration-backend"
 import { MaintenanceService } from "./service";
 import { createRouter } from "./router";
 import { CatalogApi } from "@checkstack/catalog-common";
+import { AuthApi } from "@checkstack/auth-common";
 import { registerSearchProvider } from "@checkstack/command-backend";
 import { resolveRoute, type InferClient } from "@checkstack/common";
 import { maintenanceHooks } from "./hooks";
@@ -105,6 +106,7 @@ export default createBackendPlugin({
 
         catalogClient = rpcClient.forPlugin(CatalogApi);
         maintenanceClient = rpcClient.forPlugin(MaintenanceApi);
+        const authClient = rpcClient.forPlugin(AuthApi);
 
         maintenanceService = new MaintenanceService(
           database as SafeDatabase<typeof schema>,
@@ -113,6 +115,7 @@ export default createBackendPlugin({
           maintenanceService,
           signalService,
           catalogClient,
+          authClient,
           logger,
         );
         rpc.registerRouter(router, maintenanceContract);

package/src/router.ts

@@ -11,16 +11,53 @@ import {
 import type { SignalService } from "@checkstack/signal-common";
 import type { MaintenanceService } from "./service";
 import { CatalogApi } from "@checkstack/catalog-common";
+import { AuthApi } from "@checkstack/auth-common";
 import type { InferClient } from "@checkstack/common";
 import { maintenanceHooks } from "./hooks";
 import { notifyAffectedSystems } from "./notifications";
+import type { MaintenanceUpdate } from "@checkstack/maintenance-common";
 
 export function createRouter(
   service: MaintenanceService,
   signalService: SignalService,
   catalogClient: InferClient<typeof CatalogApi>,
+  authClient: InferClient<typeof AuthApi>,
   logger: Logger,
 ) {
+  /**
+   * Resolve user IDs to profile names for a list of updates.
+   * Falls back to undefined if the user cannot be found.
+   */
+  async function resolveUserNames(
+    updates: MaintenanceUpdate[],
+  ): Promise<MaintenanceUpdate[]> {
+    const userIds = [
+      ...new Set(updates.map((u) => u.createdBy).filter(Boolean)),
+    ];
+    if (userIds.length === 0) return updates;
+
+    const userMap = new Map<string, string>();
+    await Promise.all(
+      userIds.map(async (userId) => {
+        try {
+          const user = await authClient.getUserById({ userId: userId! });
+          if (user?.name) {
+            userMap.set(userId!, user.name);
+          }
+        } catch {
+          // User not found, skip
+        }
+      }),
+    );
+
+    return updates.map((update) => ({
+      ...update,
+      createdByName: update.createdBy
+        ? (userMap.get(update.createdBy) ?? undefined)
+        : undefined,
+    }));
+  }
+
   const os = implement(maintenanceContract)
     .$context<RpcContext>()
     .use(autoAuthMiddleware);
@@ -32,8 +69,13 @@ export function createRouter(
 
     getMaintenance: os.getMaintenance.handler(async ({ input }) => {
       const result = await service.getMaintenance(input.id);
-      // eslint-disable-next-line unicorn/no-null -- oRPC contract requires null for missing values
-      return result ?? null;
+      if (!result) {
+        // eslint-disable-next-line unicorn/no-null -- oRPC contract requires null for missing values
+        return null;
+      }
+      // Resolve user names for updates
+      const updatesWithNames = await resolveUserNames(result.updates);
+      return { ...result, updates: updatesWithNames };
     }),
 
     getMaintenancesForSystem: os.getMaintenancesForSystem.handler(
@@ -226,6 +268,16 @@ export function createRouter(
           action: "closed",
         });
 
+        // Send notifications to system subscribers
+        await notifyAffectedSystems({
+          catalogClient,
+          logger,
+          maintenanceId: result.id,
+          maintenanceTitle: result.title,
+          systemIds: result.systemIds,
+          action: "completed",
+        });
+
         return result;
       },
     ),