The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
docs	Skipped		Jan 28, 2026 1:57am

Greptile Overview

Greptile Summary

This PR improves type safety and log display by correctly handling JSON inputs across 24 block integrations and the block executor.

Key Changes:

• Changed input type declarations from 'string' to 'json' for parameters that accept JSON objects/arrays across all block integrations (Apollo, Elasticsearch, GitHub, MongoDB, PostHog, Mailchimp, etc.)
• Added new parseJsonInputs() method in block-executor.ts that parses JSON strings for log display
• Applied JSON parsing to block logs at three key points: initial input logging, success callback, and error callback
• The parsing is defensive: only attempts to parse strings that look like JSON (start with { or [), wrapped in try-catch to gracefully handle invalid JSON

Impact:

• Block logs will now display parsed JSON objects/arrays instead of raw JSON strings, improving readability
• Type declarations now accurately reflect that these inputs expect JSON structures
• No changes to actual block execution logic - parsing is purely for display purposes

Confidence Score: 4/5

• This PR is safe to merge with minimal risk - changes are primarily type annotations and defensive log formatting
• Score of 4 reflects that the changes are well-contained and defensive (try-catch protection), but the isJSONString heuristic could be improved to avoid unnecessary parse attempts on non-JSON strings
• No files require special attention - all changes follow the same safe pattern

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant BlockExecutor
    participant VariableResolver
    participant parseJsonInputs
    participant BlockLog
    participant onBlockComplete

    User->>BlockExecutor: Execute block with inputs
    BlockExecutor->>VariableResolver: resolveInputs(ctx, params)
    VariableResolver-->>BlockExecutor: resolvedInputs (strings)
    
    alt Block logging enabled
        BlockExecutor->>parseJsonInputs: parseJsonInputs(resolvedInputs)
        parseJsonInputs->>parseJsonInputs: Check each value with isJSONString()
        parseJsonInputs->>parseJsonInputs: JSON.parse() on JSON-like strings
        parseJsonInputs-->>BlockExecutor: parsed inputs (objects/arrays)
        BlockExecutor->>BlockLog: Set blockLog.input with parsed data
    end
    
    BlockExecutor->>BlockExecutor: Execute block operation
    
    alt Success path
        BlockExecutor->>parseJsonInputs: parseJsonInputs(resolvedInputs)
        parseJsonInputs-->>BlockExecutor: parsed inputs
        BlockExecutor->>onBlockComplete: callback(ctx, node, block, parsed inputs, output)
    else Error path
        BlockExecutor->>parseJsonInputs: parseJsonInputs(input)
        parseJsonInputs-->>BlockExecutor: parsed inputs
        BlockExecutor->>BlockLog: Set blockLog.input with parsed data
        BlockExecutor->>onBlockComplete: callback(ctx, node, block, parsed inputs, errorOutput)
    end

@cursor review

@cursor review

@cursor review