Remove logging statements for cleaner code

[radioactive11]

Eliminate unnecessary logging in workflow execution functions to enhance code clarity and maintainability.

[greptile-apps]

Greptile Overview

Greptile Summary

This PR removes all LoggerProxy logging statements from three workflow execution functions, reverting the logging infrastructure that was added in PR #15.

Key Changes:

• Removed LoggerProxy import from all three operation files
• Eliminated all Logger.info() calls that tracked operation progress and parameter values
• Removed all Logger.error() calls in error handling blocks
• Error throwing behavior remains unchanged - errors are still properly propagated

Context:
The logging was originally added in commit 97ce507 to replace console.log statements with proper n8n logging. This PR now removes that logging entirely, suggesting a preference for cleaner code without verbose operational logging in these workflow functions.

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The changes only remove logging statements without altering any business logic, error handling, or control flow. All error throwing mechanisms remain intact, ensuring proper error propagation to n8n's error handling system. The removal is consistent across all three files and only affects observability, not functionality.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
nodes/Feather/operations/cancelWorkflowExecution.ts	5/5	Removed all Logger statements (info and error) and LoggerProxy import, simplifying error handling flow
nodes/Feather/operations/createAgentWorkflow.ts	5/5	Eliminated extensive logging throughout workflow creation process, removing LoggerProxy import and all info/error statements
nodes/Feather/operations/createWorkflowExecution.ts	5/5	Removed all logging statements and LoggerProxy import, maintaining validation and error throwing logic

Sequence Diagram

sequenceDiagram
    participant Client as n8n Client
    participant Node as Feather Node
    participant API as Feather API
    
    Note over Client,API: Workflow Execution Operations
    
    alt Create Workflow Execution
        Client->>Node: executeCreateWorkflowExecution()
        Node->>Node: Validate parameters (zipcode/state)
        Node->>Node: Build request body with metadata
        Node->>API: POST /workflow/{id}/execution
        API-->>Node: Execution response
        Node-->>Client: Return execution data
    end
    
    alt Create Agent Workflow
        Client->>Node: executeCreateAgentWorkflow()
        Node->>Node: Parse workflow & TCPA schedules
        Node->>Node: Build workflow definition
        Node->>API: POST /workflow
        API-->>Node: Workflow response
        Node-->>Client: Return workflow data
    end
    
    alt Cancel Workflow Execution
        Client->>Node: executeCancelWorkflowExecution()
        Node->>Node: Get workflowId & executionId
        Node->>API: POST /workflow/{id}/executions/{executionId}/cancel
        API-->>Node: Cancellation response
        Node-->>Client: Return cancellation data
    end

Loading

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}