update refactor plan file

Author: charleslien

plans/logger-refactor-plan.md

@@ -8,7 +8,18 @@ This is a live document. You should update it with any unintutive cases you find
 - Set `cwd: "backend"` to limit search scope
 - Exclude websocket-action.ts files
 
-### Step 2: For each file (except websocket-action.ts)
+### Step 2: For each file (except websocket-action.ts and library-integrated functions)
+
+**Exception**: Do NOT refactor functions that are directly integrated with external libraries and must maintain a specific signature required by that library. Examples include:
+
+- Express route handlers passed directly as middleware (e.g., `usageHandler`, `isRepoCoveredHandler`) - must maintain `(req, res, next?)` signature
+- WebSocket handlers that conform to a specific library interface
+- Event handlers or callbacks that match a library's expected signature
+- Any function where changing the signature would break the integration with an external library
+
+These functions should continue to import and use the logger directly, as they cannot accept custom parameter objects without breaking their integration.
+
+For all other files:
 
 - Remove the `import { logger }` line
 - Refactor function signature to use single `params` object containing all arguments including `logger: Logger`
@@ -28,7 +39,7 @@ This is a live document. You should update it with any unintutive cases you find
 ### Step 4: Commit changes
 
 - **Do NOT use git-committer agent** - it's too slow
-- Instead, manually run: `git add <files>` then `git commit -m "<message>"`
+- Instead, manually run: `git add <files> && git commit -m "<message>" && git push`
+  - This will push to a branch, where I can manually review the changes.
 - Keep commit message concise but descriptive
 - Include Codebuff footer in commit message
-- Afterward, run `git push`. This will push to a branch, where I can manually review the changes.