fix(pi): align babysit toggle visibility

Author: Bill Leoutsakos

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

@@ -112,7 +112,7 @@ describe('Pi Create PR Babysit surface', () => {
       condition: {
         field: 'mode',
         value: 'cloud',
-        and: { field: 'babysitMode', value: true },
+        and: { field: 'babysitMode', value: [true, 'true'] },
       },
     })
     expect(mentions).toMatchObject({
@@ -122,12 +122,12 @@ describe('Pi Create PR Babysit surface', () => {
       required: {
         field: 'mode',
         value: 'cloud',
-        and: { field: 'babysitMode', value: true },
+        and: { field: 'babysitMode', value: [true, 'true'] },
       },
       condition: {
         field: 'mode',
         value: 'cloud',
-        and: { field: 'babysitMode', value: true },
+        and: { field: 'babysitMode', value: [true, 'true'] },
       },
     })
     for (const output of [
@@ -142,7 +142,7 @@ describe('Pi Create PR Babysit surface', () => {
         condition: {
           field: 'mode',
           value: 'cloud',
-          and: { field: 'babysitMode', value: true },
+          and: { field: 'babysitMode', value: [true, 'true'] },
         },
       })
     }
@@ -161,6 +161,15 @@ describe('Pi Create PR Babysit surface', () => {
     expect(evaluateSubBlockCondition(draft?.condition, { mode: 'cloud', babysitMode: true })).toBe(
       false
     )
+    expect(
+      evaluateSubBlockCondition(draft?.condition, { mode: 'cloud', babysitMode: 'true' })
+    ).toBe(false)
+    expect(
+      evaluateSubBlockCondition(
+        PiBlock.subBlocks.find((subBlock) => subBlock.id === 'reviewMentions')?.condition,
+        { mode: 'cloud', babysitMode: 'true' }
+      )
+    ).toBe(true)
     expect(evaluateSubBlockCondition(skills?.condition, { mode: 'cloud' })).toBe(true)
     expect(evaluateSubBlockCondition(tools?.condition, { mode: 'cloud' })).toBe(false)
     expect(evaluateSubBlockCondition(memory?.condition, { mode: 'cloud' })).toBe(true)

apps/sim/blocks/blocks/pi.ts

@@ -52,23 +52,30 @@ const CLOUD_ANY: { field: 'mode'; value: Array<'cloud' | 'cloud_review'> } = {
   field: 'mode',
   value: ['cloud', 'cloud_review'],
 }
+const BABYSIT_ENABLED_VALUES: Array<true | 'true'> = [true, 'true']
 const CLOUD_WITH_BABYSIT: {
   field: 'mode'
   value: 'cloud'
-  and: { field: 'babysitMode'; value: true }
+  and: { field: 'babysitMode'; value: Array<true | 'true'> }
 } = {
   field: 'mode',
   value: 'cloud',
-  and: { field: 'babysitMode', value: true },
+  and: { field: 'babysitMode', value: BABYSIT_ENABLED_VALUES },
 }
-const CLOUD_WITHOUT_BABYSIT: {
+function getCloudWithoutBabysitCondition(values?: Record<string, unknown>): {
   field: 'mode'
   value: 'cloud'
-  and: { field: 'babysitMode'; value: true; not: true }
-} = {
-  field: 'mode',
-  value: 'cloud',
-  and: { field: 'babysitMode', value: true, not: true },
+  and: { field: 'babysitMode'; value: true | 'true'; not: true }
+} {
+  return {
+    field: 'mode',
+    value: 'cloud',
+    and: {
+      field: 'babysitMode',
+      value: values?.babysitMode === 'true' ? 'true' : true,
+      not: true,
+    },
+  }
 }
 const LOCAL: { field: 'mode'; value: 'local' } = { field: 'mode', value: 'local' }
 const AUTHORING_MODES: { field: 'mode'; value: Array<'cloud' | 'local'> } = {
@@ -276,7 +283,7 @@ export const PiBlock: BlockConfig<PiResponse> = {
       type: 'switch',
       defaultValue: true,
       mode: 'advanced',
-      condition: CLOUD_WITHOUT_BABYSIT,
+      condition: getCloudWithoutBabysitCondition,
     },
     {
       id: 'prTitle',