Polish Babysit reviewer field spacing

Author: Bill Leoutsakos

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/editor.tsx

@@ -650,7 +650,16 @@ export function Editor() {
                                 : undefined
                             }
                           />
-                          {showDivider && <FieldDivider subblockMarker />}
+                          {showDivider && (
+                            <FieldDivider
+                              subblockMarker
+                              className={
+                                regularSubBlocks[index + 1]?.hideDividerBefore
+                                  ? '[&>div]:invisible'
+                                  : undefined
+                              }
+                            />
+                          )}
                         </div>
                       )
                     })}
@@ -707,7 +716,14 @@ export function Editor() {
                             }
                           />
                           {index < advancedOnlySubBlocks.length - 1 && (
-                            <FieldDivider subblockMarker />
+                            <FieldDivider
+                              subblockMarker
+                              className={
+                                advancedOnlySubBlocks[index + 1]?.hideDividerBefore
+                                  ? '[&>div]:invisible'
+                                  : undefined
+                              }
+                            />
                           )}
                         </div>
                       )

apps/sim/app/workspace/[workspaceId]/w/components/preview/components/preview-editor/preview-editor.tsx

@@ -1482,7 +1482,16 @@ function PreviewEditorContent({
                         subBlockValues={subBlockValues}
                         disabled={true}
                       />
-                      {index < visibleSubBlocks.length - 1 && <FieldDivider subblockMarker />}
+                      {index < visibleSubBlocks.length - 1 && (
+                        <FieldDivider
+                          subblockMarker
+                          className={
+                            visibleSubBlocks[index + 1]?.hideDividerBefore
+                              ? '[&>div]:invisible'
+                              : undefined
+                          }
+                        />
+                      )}
                     </div>
                   ))}
                 </div>

apps/sim/blocks/blocks/pi.test.ts

@@ -108,6 +108,7 @@ describe('Pi Create PR Babysit surface', () => {
     expect(mentions).toMatchObject({
       type: 'short-input',
       defaultValue: '',
+      hideDividerBefore: true,
       required: {
         field: 'mode',
         value: 'cloud',

apps/sim/blocks/blocks/pi.ts

@@ -254,6 +254,7 @@ export const PiBlock: BlockConfig<PiResponse> = {
       placeholder: '@greptile, @cursor review',
       tooltip:
         'Required comma-separated issue comments. Each is posted after PR creation and again after every pushed Babysit fix.',
+      hideDividerBefore: true,
       required: CLOUD_WITH_BABYSIT,
       condition: CLOUD_WITH_BABYSIT,
     },

apps/sim/blocks/types.ts

@@ -314,6 +314,7 @@ export interface SubBlockConfig {
   connectionDroppable?: boolean
   hidden?: boolean
   hideFromPreview?: boolean // Hide this subblock from the workflow block preview
+  hideDividerBefore?: boolean // Visually group this field with the preceding visible subblock
   showWhenEnvSet?: string // Show this subblock only when the named NEXT_PUBLIC_ env var is truthy
   hideWhenHosted?: boolean // Hide this subblock when running on hosted sim
   hideWhenEnvSet?: string // Hide this subblock when the named NEXT_PUBLIC_ env var is truthy