fix(menus): keep empty-row actions together

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/[documentId]/components/chunk-context-menu/chunk-context-menu.tsx

@@ -71,7 +71,6 @@ export function ChunkContextMenu({
   const hasEditSection = !isMultiSelect && (!!onEdit || !!onCopyContent)
   const hasStateSection = !!onToggleEnabled
   const hasDestructiveSection = !!onDelete
-  /** @see `.claude/rules/sim-list-ordering.md` — one rule, before the destructive group. */
   const hasActionsAboveDestructive = hasNavigationSection || hasEditSection || hasStateSection
 
   return (

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/document-context-menu/document-context-menu.tsx

@@ -71,7 +71,6 @@ export function DocumentContextMenu({
   const hasEditSection = !isMultiSelect && (!!onRename || !!onViewTags)
   const hasStateSection = !!onToggleEnabled
   const hasDestructiveSection = !!onDelete
-  /** @see `.claude/rules/sim-list-ordering.md` — one rule, before the destructive group. */
   const hasActionsAboveDestructive = hasNavigationSection || hasEditSection || hasStateSection
 
   return (

apps/sim/app/workspace/[workspaceId]/knowledge/components/knowledge-base-context-menu/knowledge-base-context-menu.tsx

@@ -75,7 +75,6 @@ export const KnowledgeBaseContextMenu = memo(function KnowledgeBaseContextMenu({
   const hasMoveSection = !disableEdit && !!onMove && !!moveOptions && moveOptions.length > 0
   const hasEditSection = (showEdit && !!onEdit) || hasMoveSection
   const hasDestructiveSection = showDelete && !!onDelete
-  /** @see `.claude/rules/sim-list-ordering.md` — one rule, before the destructive group. */
   const hasActionsAboveDestructive = hasNavigationSection || hasInfoSection || hasEditSection
 
   return (

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/headers/workflow-group-meta-cell.tsx

@@ -180,15 +180,15 @@ export function ColumnOptionsMenu({
               <DropdownMenuItem onSelect={() => onRunColumnIncomplete?.()}>
                 {runLabels.incomplete}
               </DropdownMenuItem>
-              <DropdownMenuItem onSelect={() => onRunColumnAll?.()}>
-                {runLabels.all}
-              </DropdownMenuItem>
               {onRunColumnLimited &&
                 LIMITED_RUN_PRESETS.map((max) => (
                   <DropdownMenuItem key={max} onSelect={() => onRunColumnLimited(max)}>
                     {runLabels.limited(max)}
                   </DropdownMenuItem>
                 ))}
+              <DropdownMenuItem onSelect={() => onRunColumnAll?.()}>
+                {runLabels.all}
+              </DropdownMenuItem>
             </DropdownMenuSubContent>
           </DropdownMenuSub>
         )}
@@ -519,12 +519,12 @@ export function WorkflowGroupMetaCell({
               <DropdownMenuItem onSelect={handleRunIncomplete}>
                 {runLabels.incomplete}
               </DropdownMenuItem>
-              <DropdownMenuItem onSelect={handleRunAll}>{runLabels.all}</DropdownMenuItem>
               {LIMITED_RUN_PRESETS.map((max) => (
                 <DropdownMenuItem key={max} onSelect={() => handleRunLimited(max)}>
                   {runLabels.limited(max)}
                 </DropdownMenuItem>
               ))}
+              <DropdownMenuItem onSelect={handleRunAll}>{runLabels.all}</DropdownMenuItem>
             </DropdownMenuContent>
           </DropdownMenu>
         )}

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/workflow-list/components/context-menu/context-menu.test.tsx

@@ -138,15 +138,6 @@ describe('sidebar context menu dismissal', () => {
   })
 })
 
-/**
- * Separator invariants. The menu carries exactly one rule, immediately before the
- * destructive group, and it may never render with an empty group on either side —
- * see the grouping section of `.claude/rules/sim-list-ordering.md`.
- *
- * These pin the shape the flag matrix used to get wrong: `showLeave` in the rule's
- * guard without the `&& onLeave` its item requires produced a trailing rule under
- * the last item, and nothing covered it.
- */
 describe('separators', () => {
   function renderWith(props: Partial<React.ComponentProps<typeof ContextMenu>>) {
     ;(globalThis as { IS_REACT_ACT_ENVIRONMENT?: boolean }).IS_REACT_ACT_ENVIRONMENT = true
@@ -169,7 +160,6 @@ describe('separators', () => {
     )
   }
 
-  /** Menu children in render order, each as 'sep' or its label. */
   function menuShape(): string[] {
     const content = document.querySelector('[role="menu"]')
     if (!content) return []

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/workflow-list/components/context-menu/context-menu.tsx

@@ -165,29 +165,7 @@ export function ContextMenu({
   showUploadLogo = false,
   disableUploadLogo = false,
 }: ContextMenuProps) {
-  /**
-   * One rule, immediately before the destructive group — see the menu-grouping
-   * section of `.claude/rules/sim-list-ordering.md`.
-   *
-   * This menu previously carried four semantic bands (navigation / status / edit /
-   * copy / destructive) behind up to five separators. No toolbar in the app renders
-   * a divider — every header is a flat `gap-1` chip row — so those bands taught a
-   * taxonomy the user met nowhere else, and each caller's flag combination banded
-   * the same action differently (Pin alone here, Pin beside Duplicate there). Order
-   * still mirrors the surface's toolbar, which is what the ordering rule actually
-   * requires; only the rules between groups are gone.
-   *
-   * Every term below is the exact render condition of the item it stands for, so a
-   * separator can never outlive the group on either side of it. `showLeave` was the
-   * one asymmetric term — it omitted `&& onLeave`, so a caller passing `showLeave`
-   * from a permission check with a conditional `onLeave` (the `x ? fn : undefined`
-   * shape used for `onDuplicate`/`onTogglePin`/`onCloseTab` elsewhere) would have
-   * rendered a trailing rule under the last item.
-   */
   const hasActionsAboveDestructive =
-    /* No `openInNewTabPosition` term: the item renders in the 'first' slot or the
-       'last' one, and the prop is a closed two-value union, so `showOpenInNewTab &&
-       onOpenInNewTab` already means exactly "the nav item renders somewhere above". */
     (showOpenInNewTab && onOpenInNewTab) ||
     (showMarkAsRead && onMarkAsRead) ||
     (showMarkAsUnread && onMarkAsUnread) ||