Skip to content

Commit 731251f

Browse files
docs: update bug tracking with Issue 2 fix completion
Both issues are now fixed and pushed: - Issue 1: Follow mode respects --tail flag (7361e9d) - Issue 2: Parser handles zap logger format (e958989)
1 parent c358178 commit 731251f

1 file changed

Lines changed: 17 additions & 7 deletions

File tree

docs/plans/2025-11-21-logs-command-bugs.md

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -342,15 +342,15 @@ cat ~/Library/Logs/mcpproxy/server-time.log | head -20
342342
- Deduplication continues to work correctly
343343
- Committed in commit 7361e9d
344344

345-
### Issue 2: Investigation First (Before Fix)
345+
### Issue 2: Investigation and Fix (Completed)
346346
- [x] Examine raw log files from multiple servers
347347
- [x] Check log format consistency across servers
348-
- [ ] Review `internal/logs/logger.go` for timestamp writing
349-
- [ ] Check if Docker isolation affects log formatting
350-
- [ ] Check if upstream servers add their own timestamps
348+
- [x] Review `internal/logs/logger.go` for timestamp writing
349+
- [x] Check if Docker isolation affects log formatting
350+
- [x] Check if upstream servers add their own timestamps
351351
- [x] Determine root cause (A, B, or C from above) - **ROOT CAUSE: Scenario B - Parse Failure**
352-
- [ ] Implement appropriate fix based on findings
353-
- [ ] Test with multiple server types (HTTP, stdio, Docker-isolated)
352+
- [x] Implement appropriate fix based on findings
353+
- [x] Test with multiple server types (HTTP, stdio, Docker-isolated)
354354

355355
**Investigation Results (2025-11-21)**:
356356

@@ -381,7 +381,17 @@ Root cause confirmed as **Scenario B: Parse Failure with `time.Now()` Fallback**
381381
CLI added (time.Now) Original from log file
382382
```
383383

384-
**Fix needed:** Update `parseLogLine()` to handle the actual log format produced by zap logger with pipe separators and ISO8601 timestamps.
384+
**Fix implemented (commit e958989):**
385+
- Updated `parseLogLine()` in `internal/server/server.go` to handle zap logger format
386+
- Parses ISO8601 timestamps using RFC3339 format
387+
- Strips file:line and JSON metadata from message display
388+
- Maintains backward compatibility with legacy format
389+
- Results in clean CLI output without double timestamps
390+
391+
**After fix:**
392+
```
393+
2025-11-21T15:05:42.725-05:00 | INFO | MCP initialization completed successfully
394+
```
385395

386396
### Documentation
387397
- [ ] Update `docs/cli-management-commands.md` with correct `--tail` behavior

0 commit comments

Comments
 (0)