fix concurrency settings w/ a single paused producer

Also fix Queue stories that depend on feature flags.

Author: bgentry

.storybook/preview.tsx

@@ -26,6 +26,7 @@ export const withFeatures: Decorator = (StoryFn, context) => {
   // Default features with story-specific overrides
   const features = {
     hasProducerTable: true,
+    producerQueries: true,
     ...context.parameters?.features,
   };
 

src/components/QueueDetail.stories.tsx

@@ -1,3 +1,4 @@
+import { useFeatures } from "@contexts/Features.hook";
 import { type Producer } from "@services/producers";
 import { type ConcurrencyConfig } from "@services/queues";
 import { Meta, StoryObj } from "@storybook/react-vite";
@@ -88,25 +89,25 @@ export const QueueNotFound: Story = {
   },
 };
 
-// Active queue with no producers (features disabled)
-export const ActiveQueueWithoutPro: Story = {
-  args: {
-    producers: [],
-    queue: queueFactory.active().build(),
-  },
-  parameters: {
-    features: {
-      hasProducerTable: false,
-    },
-  },
-};
-
 // Active queue with no producers (features enabled)
 export const ActiveQueueNoProducers: Story = {
   args: {
     producers: [],
     queue: queueFactory.active().build(),
   },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
 };
 
 // Paused queue with no producers
@@ -115,6 +116,19 @@ export const PausedQueueNoProducers: Story = {
     producers: [],
     queue: queueFactory.paused().build(),
   },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
 };
 
 // Active queue with producers
@@ -123,6 +137,19 @@ export const ActiveQueueWithProducers: Story = {
     producers: createProducers(5, "test-queue"),
     queue: queueFactory.active().build(),
   },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
 };
 
 // Paused queue with some paused producers
@@ -131,31 +158,110 @@ export const PausedQueueWithMixedProducers: Story = {
     producers: createProducers(5, "test-queue", { paused: true }),
     queue: queueFactory.paused().build(),
   },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
 };
 
 // Queue with concurrency settings
-export const QueueWithConcurrencySettings: Story = {
+export const WithConcurrencySettings: Story = {
   args: {
     producers: createProducers(3, "test-queue", { withConcurrency: true }),
     queue: queueFactory.withConcurrency().build(),
   },
+  parameters: {
+    features: {
+      hasProducerTable: true,
+      producerQueries: true,
+    },
+  },
 };
 
 // Queue with inconsistent producer concurrency settings
-export const QueueWithInconsistentConcurrency: Story = {
+export const InconsistentConcurrency: Story = {
   args: {
     producers: createProducers(3, "test-queue", {
       inconsistentConcurrency: true,
       withConcurrency: true,
     }),
     queue: queueFactory.withConcurrency().build(),
   },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
 };
 
 // Queue with many producers
-export const QueueWithManyProducers: Story = {
+export const ManyProducers: Story = {
   args: {
     producers: createProducers(20, "test-queue", { paused: true }),
     queue: queueFactory.active().build(),
   },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
+};
+
+// Queue with single paused producer (should not show concurrency warning)
+export const SinglePausedProducer: Story = {
+  args: {
+    producers: createProducers(1, "test-queue", { paused: true }),
+    queue: queueFactory.active().build(),
+  },
+  parameters: {
+    mockData: [
+      {
+        hook: useFeatures,
+        mockValue: {
+          features: {
+            hasProducerTable: true,
+            producerQueries: true,
+          },
+        },
+      },
+    ],
+  },
+};
+
+// Pro features disabled
+export const WithoutPro: Story = {
+  args: {
+    producers: [],
+    queue: queueFactory.active().build(),
+  },
+  parameters: {
+    features: {
+      hasProducerTable: false,
+    },
+  },
 };

src/components/QueueDetail.tsx

@@ -251,14 +251,13 @@ const ConcurrencySettings = ({
 
   const producerConcurrencyStatus = useMemo(() => {
     if (!producers || producers.length === 0)
-      return { config: null, consistent: false };
+      return { config: null, consistent: true };
 
     // Filter out paused producers
     const activeProducers = producers.filter((p) => !p.pausedAt);
 
-    // If there are no active producers, return null config
-    if (activeProducers.length === 0)
-      return { config: null, consistent: false };
+    // If there are no active producers, return null config but consider it consistent
+    if (activeProducers.length === 0) return { config: null, consistent: true };
 
     const firstProducer = activeProducers[0];
     const allSame = activeProducers.every((p) => {

src/services/features.test.ts

@@ -7,11 +7,11 @@ describe("apiFeaturesToFeatures", () => {
     const apiFeatures = {
       extensions: {
         durable_periodic_jobs: true,
-        producer_queries: true,
-        workflow_queries: true,
         has_client_table: true,
         has_producer_table: true,
         has_workflows: true,
+        producer_queries: true,
+        workflow_queries: true,
       },
       job_list_hide_args_by_default: true,
     } as const;
@@ -34,11 +34,11 @@ describe("apiFeaturesToFeatures", () => {
     const apiFeatures = {
       extensions: {
         durable_periodic_jobs: false,
-        producer_queries: false,
-        workflow_queries: false,
         has_client_table: false,
         has_producer_table: false,
         has_workflows: false,
+        producer_queries: false,
+        workflow_queries: false,
       },
       job_list_hide_args_by_default: false,
     } as const;