Move logger setup to after MODE selection - #126
Merged
Conversation
Because we call `setup_logging` before deciding on the mode the log file is always `.claude/hook-approvals.log` even for cursor This split the logger setup in two, first we do basic logging setup with the right format etc so that we don't lose messages related to parsing input and setup and then once we know the mode we are in we call the file logger.
basicConfig(level=INFO) attached an INFO stderr handler, so every decision printed to stderr on each invocation. Set root+file to INFO (audit log unchanged) but the stderr handler to WARNING, so only early failures (bad JSON, unknown tool) surface there.
orgoj
pushed a commit
to orgoj/Dippy
that referenced
this pull request
Aug 10, 2026
…yton#126) setup_logging() ran before mode detection, so the log file always went to ~/.claude/ regardless of mode. Split it: detect the mode first, then attach the file handler so the log lands in the right directory. Also add an early stderr handler at WARNING level so failures that happen before the file handler exists (bad JSON, unknown tool) are still visible, while routine INFO decisions stay quiet on stderr and continue to the audit file as before. Co-authored-by: Lily Dayton <43729618+ldayton@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Because we call
setup_loggingbefore deciding on the mode the log file is always.claude/hook-approvals.logeven for cursorThis split the logger setup in two, first we do basic logging setup with the right format etc so that we don't lose messages related to parsing input and setup and then once we know the mode we are in we call the file logger.