fix(docs): retain nullable overrides on output spreads

Author: Bill Leoutsakos

apps/docs/content/docs/integrations/github.mdx

@@ -66,7 +66,7 @@ Fetch PR details including diff and files changed
 | `body` | string (nullable) | PR description |
 | `merged` | boolean | Whether PR is merged |
 | `mergeable` | boolean (nullable) | Whether PR is mergeable |
-| `merged_by` | object | GitHub user object |
+| `merged_by` | object (nullable) | GitHub user object |
 |   ↳ `login` | string | GitHub username |
 |   ↳ `id` | number | User ID |
 |   ↳ `avatar_url` | string | Avatar image URL |
@@ -673,7 +673,7 @@ Submit a review for a pull request. Use APPROVE, REQUEST_CHANGES, or COMMENT. A
 | Parameter | Type | Description |
 | --------- | ---- | ----------- |
 | `id` | number | Review ID |
-| `user` | object | GitHub user object |
+| `user` | object (nullable) | GitHub user object |
 |   ↳ `login` | string | GitHub username |
 |   ↳ `id` | number | User ID |
 |   ↳ `avatar_url` | string | Avatar image URL |

scripts/generate-docs.test.ts

@@ -83,6 +83,20 @@ describe('documentation tool metadata', () => {
     expect(markdown).toContain('| `count` | number |')
   })
 
+  it('renders nullable overrides on spread-based output definitions', async () => {
+    const [block] = extractAllBlockConfigs(
+      fs.readFileSync(path.resolve('apps/sim/blocks/blocks/github.ts'), 'utf-8')
+    )
+    const markdown = await generateMarkdownForBlock({
+      ...block,
+      tools: { access: ['github_create_pr_review_v2'] },
+    })
+    expect(markdown).toContain('| `user` | object (nullable) |')
+
+    const outputs = parsePropertiesContent('user: { ...USER_OUTPUT, nullable: false },', 'github')
+    expect(outputs.user).toMatchObject({ type: 'object', nullable: false })
+  })
+
   it('uses evaluated outputs for factory-defined tools', async () => {
     const approve = await getToolInfo('sailpoint_approve_access_request')
     const identity = await getToolInfo('sailpoint_get_identity')

scripts/generate-docs.ts

@@ -3628,6 +3628,11 @@ function parseFieldContent(fieldContent: string, toolPrefix?: string, propertyNa
       if (description) {
         result.description = description
       }
+      const nullableOverride = findTopLevelMatch(
+        blankStringsAndComments(fieldContent) ?? '',
+        /\bnullable\s*:\s*(true|false)\b/
+      )
+      if (nullableOverride) result.nullable = nullableOverride[1] === 'true'
       return result
     }
   }