Skip to content

Fix upload - #2

Merged
gregmeyer merged 2 commits into
mainfrom
fix-upload
Apr 7, 2025
Merged

Fix upload#2
gregmeyer merged 2 commits into
mainfrom
fix-upload

Conversation

@gregmeyer

Copy link
Copy Markdown
Owner

This PR implements verbose mode functionality and enhances logging throughout the ProcessBuilder:

Key Changes

  • Added set_input_handler classmethod to ProcessBuilder for CLI input handling
  • Implemented proper warning capture for missing OpenAI API key
  • Added verbose mode toggle with set_verbose_mode classmethod
  • Enhanced logging system with proper debug/warning levels
  • Updated README with verbose mode documentation

Implementation Details

  1. ProcessBuilder Changes:

    • Added class-level verbose mode control
    • Fixed warning log handling for API key validation
    • Streamlined OpenAI client initialization
    • Added proper debug logging throughout
  2. Documentation Updates:

    • Added verbose mode to Features section
    • Created Configuration section for verbose mode
    • Updated Usage with verbose mode examples
    • Added debug output examples

Testing

  • Added test cases for verbose mode:
    • test_verbose_mode_off
    • test_verbose_mode_on
    • test_verbose_mode_toggle
  • All tests passing with proper log capture

Usage

Users can now enable verbose mode in two ways:

# Class-level verbose mode
ProcessBuilder.set_verbose_mode(True)

# Instance-level verbose mode
builder = ProcessBuilder('My Process', verbose=True)

Follow-up Tasks

  • Consider adding log file output option
  • Add more granular log level control
  • Consider adding structured logging format option

@gregmeyer
gregmeyer merged commit 7a336c4 into main Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant