[One Workflow] Hide Add step group when search has no matching actions

Per designer feedback, the "Add step" group label should only appear when
there are matching actions to show under it. The Commands and Jump-to-a-step
groups already follow this rule — apply the same gate to Add step in both
normal search and Steps: prefix mode.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: Kiryous

src/platform/plugins/shared/workflows_management/public/features/actions_menu_popover/lib/use_display_options.test.ts

@@ -43,9 +43,24 @@ describe('buildDisplayOptions', () => {
   };
 
   it('shows Add step + Commands sections when no search is active', () => {
-    const result = buildDisplayOptions(base);
+    const result = buildDisplayOptions({ ...base, options: [makeAction('a', 'A')] });
     expect(groupLabels(result)).toEqual(['Add step', 'Commands']);
-    expect(dataKinds(result)).toEqual(['command', 'command']);
+    expect(dataKinds(result)).toEqual(['action', 'command', 'command']);
+  });
+
+  it('hides Add step section when no actions match the search', () => {
+    const result = buildDisplayOptions({ ...base, options: [], searchTerm: 'zzz' });
+    expect(groupLabels(result)).not.toContain('Add step');
+  });
+
+  it('hides Add step section in Steps: mode when no actions match', () => {
+    const result = buildDisplayOptions({
+      ...base,
+      options: [],
+      searchTerm: `${STEPS_PREFIX}zzz`,
+    });
+    expect(groupLabels(result)).not.toContain('Add step');
+    expect(result).toHaveLength(0);
   });
 
   it('returns action items directly when inside a sub-group', () => {

src/platform/plugins/shared/workflows_management/public/features/actions_menu_popover/lib/use_display_options.ts

@@ -91,37 +91,41 @@ export function buildDisplayOptions({
   }
 
   if (isStepsMode) {
+    if (options.length > 0) {
+      result.push({
+        label: i18n.translate('workflows.actionsMenu.addStepGroupLabel', {
+          defaultMessage: 'Add step',
+        }),
+        isGroupLabel: true,
+      });
+      for (const opt of options) {
+        result.push({ label: opt.label, data: { menuItem: { kind: 'action', action: opt } } });
+      }
+    }
+    return result;
+  }
+
+  const visibleOptions = hasSearch ? options.slice(0, MAX_VISIBLE_STEPS) : options;
+  if (visibleOptions.length > 0) {
     result.push({
       label: i18n.translate('workflows.actionsMenu.addStepGroupLabel', {
         defaultMessage: 'Add step',
       }),
       isGroupLabel: true,
     });
-    for (const opt of options) {
+    for (const opt of visibleOptions) {
       result.push({ label: opt.label, data: { menuItem: { kind: 'action', action: opt } } });
     }
-    return result;
-  }
-
-  result.push({
-    label: i18n.translate('workflows.actionsMenu.addStepGroupLabel', {
-      defaultMessage: 'Add step',
-    }),
-    isGroupLabel: true,
-  });
-  const visibleOptions = hasSearch ? options.slice(0, MAX_VISIBLE_STEPS) : options;
-  for (const opt of visibleOptions) {
-    result.push({ label: opt.label, data: { menuItem: { kind: 'action', action: opt } } });
-  }
 
-  if (hasSearch && options.length > MAX_VISIBLE_STEPS) {
-    result.push({
-      label: i18n.translate('workflows.actionsMenu.viewAllSteps', {
-        defaultMessage: 'View all steps to add',
-      }),
-      className: 'compactOption',
-      data: { menuItem: { kind: 'nav', target: 'viewAll' } },
-    });
+    if (hasSearch && options.length > MAX_VISIBLE_STEPS) {
+      result.push({
+        label: i18n.translate('workflows.actionsMenu.viewAllSteps', {
+          defaultMessage: 'View all steps to add',
+        }),
+        className: 'compactOption',
+        data: { menuItem: { kind: 'nav', target: 'viewAll' } },
+      });
+    }
   }
 
   const filteredCmds = (commands ?? []).filter(