fix(tailscale): normalize controls and output schemas

Author: waleedlatif1

apps/docs/content/docs/integrations/tailscale.mdx

@@ -5,9 +5,9 @@ interface SwitchProps {
   blockId: string
   subBlockId: string
   title: string
-  value?: boolean
+  value?: boolean | string
   isPreview?: boolean
-  previewValue?: boolean | null
+  previewValue?: boolean | string | null
   disabled?: boolean
 }
 
@@ -20,7 +20,7 @@ export function Switch({
   previewValue,
   disabled = false,
 }: SwitchProps) {
-  const [storeValue, setStoreValue] = useSubBlockValue<boolean>(blockId, subBlockId)
+  const [storeValue, setStoreValue] = useSubBlockValue<boolean | string>(blockId, subBlockId)
 
   const value = isPreview ? previewValue : (storeValue ?? propValue)
 
@@ -34,7 +34,7 @@ export function Switch({
     <div className='flex items-center gap-x-3'>
       <UISwitch
         id={`${blockId}-${subBlockId}`}
-        checked={Boolean(value)}
+        checked={value === true || value === 'true'}
         onCheckedChange={handleChange}
         disabled={isPreview || disabled}
       />

apps/docs/content/docs/integrations/vanta.mdx

@@ -252,15 +252,15 @@ export const TailscaleBlock: BlockConfig = {
       required: true,
       title: 'Authorized',
       type: 'switch',
-      value: () => 'true',
+      defaultValue: true,
       condition: { field: 'operation', value: 'authorize_device' },
     },
     {
       id: 'keyExpiryDisabled',
       required: true,
       title: 'Key Expiry Disabled',
       type: 'switch',
-      value: () => 'true',
+      defaultValue: true,
       condition: { field: 'operation', value: 'update_device_key' },
     },
     {
@@ -310,7 +310,7 @@ export const TailscaleBlock: BlockConfig = {
       required: true,
       title: 'MagicDNS',
       type: 'switch',
-      value: () => 'true',
+      defaultValue: true,
       condition: { field: 'operation', value: 'set_dns_preferences' },
     },
     {
@@ -339,23 +339,23 @@ export const TailscaleBlock: BlockConfig = {
       id: 'reusable',
       title: 'Reusable',
       type: 'switch',
-      value: () => 'false',
+      defaultValue: false,
       condition: { field: 'operation', value: 'create_auth_key' },
       mode: 'advanced',
     },
     {
       id: 'ephemeral',
       title: 'Ephemeral',
       type: 'switch',
-      value: () => 'false',
+      defaultValue: false,
       condition: { field: 'operation', value: 'create_auth_key' },
       mode: 'advanced',
     },
     {
       id: 'preauthorized',
       title: 'Preauthorized',
       type: 'switch',
-      value: () => 'true',
+      defaultValue: true,
       condition: { field: 'operation', value: 'create_auth_key' },
       mode: 'advanced',
     },
@@ -424,7 +424,7 @@ export const TailscaleBlock: BlockConfig = {
       id: 'details',
       title: 'Policy Diagnostics',
       type: 'switch',
-      value: () => 'false',
+      defaultValue: false,
       condition: { field: 'operation', value: 'get_acl' },
       mode: 'advanced',
     },
@@ -519,7 +519,7 @@ export const TailscaleBlock: BlockConfig = {
         value: 'list_auth_keys',
       },
       mode: 'advanced',
-      value: () => 'true',
+      defaultValue: true,
     },
     {
       id: 'userType',
@@ -760,7 +760,7 @@ export const TailscaleBlock: BlockConfig = {
           if (value !== undefined && value !== '')
             mapped[field] = value === true || value === 'true'
         }
-        if (params.expirySeconds !== undefined && params.expirySeconds !== '') {
+        if (params.expirySeconds != null && params.expirySeconds !== '') {
           const expirySeconds = Number(params.expirySeconds)
           if (!Number.isFinite(expirySeconds)) throw new Error('Expiry must be a finite number')
           mapped.expirySeconds = expirySeconds
@@ -924,7 +924,7 @@ export const TailscaleBlock: BlockConfig = {
       type: 'json',
       description: 'Device enrollment capabilities (reusable, ephemeral, preauthorized, tags)',
     },
-    acl: { type: 'string', description: 'ACL policy as JSON string' },
+    acl: { type: 'string', description: 'Policy as JSON or original HuJSON for detailed Get ACL' },
     etag: { type: 'string', description: 'ACL ETag for conditional updates' },
     userId: { type: 'string', description: 'User ID' },
   },

apps/docs/content/docs/integrations/zoho_desk.mdx

@@ -86,12 +86,12 @@ export const tailscaleCreateAuthKeyTool: ToolConfig<
             reusable: params.reusable ?? false,
             ephemeral: params.ephemeral ?? false,
             preauthorized: params.preauthorized ?? true,
-            ...(params.tags ? { tags: parseTailscaleList(params.tags) } : {}),
+            ...(params.tags?.trim() ? { tags: parseTailscaleList(params.tags) } : {}),
           },
         },
       },
       ...(params.description ? { description: params.description } : {}),
-      ...(params.expirySeconds !== undefined ? { expirySeconds: params.expirySeconds } : {}),
+      ...(params.expirySeconds != null ? { expirySeconds: params.expirySeconds } : {}),
     }),
   },
   transformResponse: async (response) => {
@@ -101,6 +101,10 @@ export const tailscaleCreateAuthKeyTool: ToolConfig<
   },
   outputs: {
     ...KEY_OUTPUT_PROPERTIES,
-    key: { type: 'string', description: 'Auth key secret, returned only at creation' },
+    key: {
+      type: 'string',
+      description: 'Auth key secret, returned only at creation',
+      nullable: true,
+    },
   },
 }

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/switch/switch.tsx

@@ -42,8 +42,8 @@ export const tailscaleGetAclTool: ToolConfig<TailscaleGetAclParams, TailscaleGet
     await assertTailscaleResponse(response)
     const data: Record<string, unknown> & {
       acl?: string
-      warnings?: string[]
-      errors?: string[]
+      warnings?: string[] | null
+      errors?: string[] | null
     } = await response.json()
     return {
       success: true,
@@ -62,7 +62,7 @@ export const tailscaleGetAclTool: ToolConfig<TailscaleGetAclParams, TailscaleGet
     errors: { type: 'array', description: 'Policy parsing errors', items: { type: 'string' } },
     acl: {
       type: 'string',
-      description: 'Tailnet policy file as a JSON string',
+      description: 'Tailnet policy as JSON, or the original HuJSON when diagnostics are enabled',
     },
     etag: {
       type: 'string',

apps/sim/blocks/blocks/tailscale.ts

@@ -55,12 +55,10 @@ export const tailscaleGetDnsConfigurationTool: ToolConfig<
     dnsConfiguration: {
       type: 'json',
       description: 'Dns configuration',
-      optional: true,
       properties: {
         nameservers: {
           type: 'array',
           description: 'Global DNS resolvers to use',
-          optional: true,
           items: {
             type: 'json',
             properties: {
@@ -82,20 +80,18 @@ export const tailscaleGetDnsConfigurationTool: ToolConfig<
           type: 'json',
           description:
             'Map of DNS name suffixes (domains) to lists of resolvers for Split DNS and advanced routing overlays',
-          optional: true,
+          nullable: true,
         },
         searchPaths: {
           type: 'array',
           description: 'Search domain paths to apply',
-          optional: true,
           items: {
             type: 'string',
           },
         },
         preferences: {
           type: 'json',
           description: 'Preferences',
-          optional: true,
           properties: {
             overrideLocalDNS: {
               type: 'boolean',
@@ -109,6 +105,7 @@ export const tailscaleGetDnsConfigurationTool: ToolConfig<
               optional: true,
             },
           },
+          nullable: true,
         },
       },
     },

apps/sim/tools/generated/tool-outputs.ts

@@ -65,92 +65,83 @@ export const tailscaleGetTailnetSettingsTool: ToolConfig<
     settings: {
       type: 'json',
       description: 'Settings for a tailnet',
-      optional: true,
       properties: {
         aclsExternallyManagedOn: {
           type: 'boolean',
           description:
             'Prevents users from editing policies in the admin console to avoid conflicts with external management workflows like GitOps or Terraform',
-          optional: true,
           nullable: true,
         },
         aclsExternalLink: {
           type: 'string',
           description:
             'Link to the external tailnet policy definition or management solution for this tailnet',
-          optional: true,
+          nullable: true,
         },
         adminConsoleSessionTimeoutMinutes: {
           type: 'number',
           description:
             'The [admin console session timeout](/docs/account/admin-console-session-timeout) for members of this tailnet',
-          optional: true,
+          nullable: true,
         },
         devicesApprovalOn: {
           type: 'boolean',
           description:
             'Whether [device approval](/docs/features/access-control/device-management/device-approval) is enabled for the tailnet',
-          optional: true,
           nullable: true,
         },
         devicesAutoUpdatesOn: {
           type: 'boolean',
           description:
             'Whether [auto updates](/docs/features/client/update#auto-updates) are enabled for devices that belong to this tailnet',
-          optional: true,
           nullable: true,
         },
         devicesKeyDurationDays: {
           type: 'number',
           description:
             'The [key expiry](/docs/features/access-control/key-expiry) duration for devices on this tailnet',
-          optional: true,
+          nullable: true,
         },
         usersApprovalOn: {
           type: 'boolean',
           description:
             'Whether [user approval](/docs/features/access-control/user-approval) is enabled for this tailnet',
-          optional: true,
           nullable: true,
         },
         usersRoleAllowedToJoinExternalTailnets: {
           type: 'string',
           description:
             'Which user roles are allowed to [join external tailnets](/docs/features/sharing/how-to/invite-any-user)',
-          optional: true,
+          nullable: true,
         },
         networkFlowLoggingOn: {
           type: 'boolean',
           description:
             'Whether [network flog logs](/docs/features/logging/network-flow-logs) are enabled for the tailnet',
-          optional: true,
           nullable: true,
         },
         regionalRoutingOn: {
           type: 'boolean',
           description:
             'Whether [regional routing](/docs/features/route-selection#regional-routing) is enabled for the tailnet',
-          optional: true,
           nullable: true,
         },
         routeSelection: {
           type: 'string',
           description:
             'The [route selection](/docs/features/route-selection) algorithm used by the tailnet: * `active-passive-failover` - Active-passive failover (formerly known as "Failover") * `regional-routing` - Regional routing * `regional-routing-failover` - Regional routing with in-region failover * `magicroute` - MagicRoute A PATCH request must not specify both the `regionalRoutingOn` and `routeSelection` fields',
-          optional: true,
+          nullable: true,
         },
         postureIdentityCollectionOn: {
           type: 'boolean',
           description:
             'Whether [identity collection](/docs/features/access-control/device-management/how-to/manage-identity) is enabled for [device posture](/docs/features/device-posture) integrations for the tailnet',
-          optional: true,
           nullable: true,
         },
         httpsEnabled: {
           type: 'boolean',
           description:
             'Whether provisioning of [HTTPS certificates](/docs/how-to/set-up-https-certificates) is enabled for this tailnet',
-          optional: true,
           nullable: true,
         },
       },