@checkstack/incident-common 0.2.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,73 @@
 # @checkstack/incident-common
 
+## 0.3.1
+
+### Patch Changes
+
+- Updated dependencies [4eed42d]
+  - @checkstack/frontend-api@0.3.0
+
+## 0.3.0
+
+### Minor Changes
+
+- 7a23261: ## TanStack Query Integration
+
+  Migrated all frontend components to use `usePluginClient` hook with TanStack Query integration, replacing the legacy `forPlugin()` pattern.
+
+  ### New Features
+
+  - **`usePluginClient` hook**: Provides type-safe access to plugin APIs with `.useQuery()` and `.useMutation()` methods
+  - **Automatic request deduplication**: Multiple components requesting the same data share a single network request
+  - **Built-in caching**: Configurable stale time and cache duration per query
+  - **Loading/error states**: TanStack Query provides `isLoading`, `error`, `isRefetching` states automatically
+  - **Background refetching**: Stale data is automatically refreshed when components mount
+
+  ### Contract Changes
+
+  All RPC contracts now require `operationType: "query"` or `operationType: "mutation"` metadata:
+
+  ```typescript
+  const getItems = proc()
+    .meta({ operationType: "query", access: [access.read] })
+    .output(z.array(itemSchema))
+    .query();
+
+  const createItem = proc()
+    .meta({ operationType: "mutation", access: [access.manage] })
+    .input(createItemSchema)
+    .output(itemSchema)
+    .mutation();
+  ```
+
+  ### Migration
+
+  ```typescript
+  // Before (forPlugin pattern)
+  const api = useApi(myPluginApiRef);
+  const [items, setItems] = useState<Item[]>([]);
+  useEffect(() => {
+    api.getItems().then(setItems);
+  }, [api]);
+
+  // After (usePluginClient pattern)
+  const client = usePluginClient(MyPluginApi);
+  const { data: items, isLoading } = client.getItems.useQuery({});
+  ```
+
+  ### Bug Fixes
+
+  - Fixed `rpc.test.ts` test setup for middleware type inference
+  - Fixed `SearchDialog` to use `setQuery` instead of deprecated `search` method
+  - Fixed null→undefined warnings in notification and queue frontends
+
+### Patch Changes
+
+- Updated dependencies [7a23261]
+  - @checkstack/frontend-api@0.2.0
+  - @checkstack/common@0.3.0
+  - @checkstack/signal-common@0.1.1
+
 ## 0.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/incident-common",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "type": "module",
   "exports": {
     ".": {

package/src/access.ts

@@ -1,4 +1,4 @@
-import { accessPair } from "@checkstack/common";
+import { access, accessPair } from "@checkstack/common";
 
 /**
  * Access rules for the Incident plugin.
@@ -7,6 +7,7 @@ export const incidentAccess = {
   /**
    * Incident access with both read and manage levels.
    * Read is public by default.
+   * Uses system-level instance access for team-based filtering.
    */
   incident: accessPair(
     "incident",
@@ -15,10 +16,21 @@ export const incidentAccess = {
       manage: "Manage incidents - create, edit, resolve, and delete",
     },
     {
+      idParam: "systemId",
       readIsDefault: true,
       readIsPublic: true,
     }
   ),
+
+  /**
+   * Bulk incident access for viewing incidents for multiple systems.
+   * Uses recordKey for filtering the output record by accessible system IDs.
+   */
+  bulkIncident: access("incident.incident", "read", "View incidents", {
+    recordKey: "incidents",
+    isDefault: true,
+    isPublic: true,
+  }),
 };
 
 /**

package/src/rpc-contract.ts

@@ -1,9 +1,5 @@
-import { oc } from "@orpc/contract";
 import { z } from "zod";
-import {
-  createClientDefinition,
-  type ProcedureMetadata,
-} from "@checkstack/common";
+import { createClientDefinition, proc } from "@checkstack/common";
 import { incidentAccess } from "./access";
 import { pluginMetadata } from "./plugin-metadata";
 import {
@@ -16,15 +12,13 @@ import {
   IncidentStatusEnum,
 } from "./schemas";
 
-const _base = oc.$meta<ProcedureMetadata>({});
-
 export const incidentContract = {
   /** List all incidents with optional filters */
-  listIncidents: _base
-    .meta({
-      userType: "public",
-      access: [incidentAccess.incident.read],
-    })
+  listIncidents: proc({
+    operationType: "query",
+    userType: "public",
+    access: [incidentAccess.incident.read],
+  })
     .input(
       z
         .object({
@@ -37,65 +31,80 @@ export const incidentContract = {
     .output(z.object({ incidents: z.array(IncidentWithSystemsSchema) })),
 
   /** Get a single incident with all details */
-  getIncident: _base
-    .meta({
-      userType: "public",
-      access: [incidentAccess.incident.read],
-    })
+  getIncident: proc({
+    operationType: "query",
+    userType: "public",
+    access: [incidentAccess.incident.read],
+  })
     .input(z.object({ id: z.string() }))
     .output(IncidentDetailSchema.nullable()),
 
   /** Get active incidents for a specific system */
-  getIncidentsForSystem: _base
-    .meta({
-      userType: "public",
-      access: [incidentAccess.incident.read],
-    })
+  getIncidentsForSystem: proc({
+    operationType: "query",
+    userType: "public",
+    access: [incidentAccess.incident.read],
+  })
     .input(z.object({ systemId: z.string() }))
     .output(z.array(IncidentWithSystemsSchema)),
 
+  /** Get active incidents for multiple systems in a single request.
+   * Used for efficient dashboard rendering to avoid N+1 queries.
+   */
+  getBulkIncidentsForSystems: proc({
+    operationType: "query",
+    userType: "public",
+    access: [incidentAccess.bulkIncident],
+  })
+    .input(z.object({ systemIds: z.array(z.string()) }))
+    .output(
+      z.object({
+        incidents: z.record(z.string(), z.array(IncidentWithSystemsSchema)),
+      })
+    ),
+
   /** Create a new incident */
-  createIncident: _base
-    .meta({
-      userType: "authenticated",
-      access: [incidentAccess.incident.manage],
-    })
+  createIncident: proc({
+    operationType: "mutation",
+    userType: "authenticated",
+    access: [incidentAccess.incident.manage],
+  })
     .input(CreateIncidentInputSchema)
     .output(IncidentWithSystemsSchema),
 
   /** Update an existing incident */
-  updateIncident: _base
-    .meta({
-      userType: "authenticated",
-      access: [incidentAccess.incident.manage],
-    })
+  updateIncident: proc({
+    operationType: "mutation",
+    userType: "authenticated",
+    access: [incidentAccess.incident.manage],
+  })
     .input(UpdateIncidentInputSchema)
     .output(IncidentWithSystemsSchema),
 
   /** Add a status update to an incident */
-  addUpdate: _base
-    .meta({
-      userType: "authenticated",
-      access: [incidentAccess.incident.manage],
-    })
+  addUpdate: proc({
+    operationType: "mutation",
+    userType: "authenticated",
+    access: [incidentAccess.incident.manage],
+  })
     .input(AddIncidentUpdateInputSchema)
     .output(IncidentUpdateSchema),
 
   /** Resolve an incident (sets status to resolved) */
-  resolveIncident: _base
-    .meta({
-      userType: "authenticated",
-      access: [incidentAccess.incident.manage],
-    })
+  resolveIncident: proc({
+    operationType: "mutation",
+    userType: "authenticated",
+    access: [incidentAccess.incident.manage],
+  })
     .input(z.object({ id: z.string(), message: z.string().optional() }))
     .output(IncidentWithSystemsSchema),
 
   /** Delete an incident */
-  deleteIncident: _base
-    .meta({
-      userType: "authenticated",
-      access: [incidentAccess.incident.manage],
-    })
+  deleteIncident: proc({
+    operationType: "mutation",
+    userType: "authenticated",
+    access: [incidentAccess.incident.manage],
+  })
     .input(z.object({ id: z.string() }))
     .output(z.object({ success: z.boolean() })),
 };