fix: functions run command with API key header and variable support

[giordano-lucas]

Fixed two critical issues with the 'functions run' command:

1. Missing x-notte-api-key header - The FunctionRunStartParams required field was not being populated, causing 400 errors
2. Missing request body - API requires function_id in JSON body but generated client doesn't support it

Added variable support for workflow execution:

• --var flag for key=value pairs (can be used multiple times)
• --vars flag for JSON object string
• Variables are passed to the function run API

Changes:

• Added APIKey() method to NotteClient
• Rewrote runFunctionRun to manually construct HTTP request with body
• Added integration tests for function run and variables
• Added unit test for APIKey() method

Resolves issue with function run failing with API key validation error

Greptile Overview

Greptile Summary

Fixed critical issues with the functions run command where the generated API client lacked support for required authentication header and request body.

Key Changes:

• Added method to NotteClient for exposing authentication credentials
• Rewrote runFunctionRun to manually construct HTTP request with body containing function ID and variables
• Added support for passing variables through command-line flags (individual pairs or JSON object)
• Added comprehensive integration and unit tests

The implementation reuses the existing HTTP client for retry logic and circuit breaker functionality.

Confidence Score: 4/5

• This PR is safe to merge with minor considerations around dual authentication headers
• The implementation correctly fixes the critical bug and adds comprehensive tests. Score of 4 (not 5) because the manual HTTP request results in dual authentication headers (both Authorization Bearer and x-notte-api-key) being sent, though this should be harmless as the API will use the appropriate header
• No files require special attention - all changes are well-tested and straightforward

Important Files Changed

Filename	Overview
internal/api/client.go	Added APIKey() getter method to expose the API key, enabling manual HTTP requests with proper authentication
internal/api/client_test.go	Added unit test for APIKey() method, verifying it returns the correct API key
internal/cmd/functions.go	Rewrote runFunctionRun to manually construct HTTP request with body containing function_id and variables, added --var and --vars flags for variable support
tests/integration/functions_test.go	Added comprehensive integration tests for function run command with and without variables, covering all flag combinations

[greptile-apps]

_{4 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}