diff --git a/.jules/bolt.md b/.jules/bolt.md index 29665ec..b326912 100644 --- a/.jules/bolt.md +++ b/.jules/bolt.md @@ -21,3 +21,7 @@ ## 2024-10-25 - Avoid mime.ParseMediaType for exact header matching **Learning:** Using `mime.ParseMediaType` for exact or prefix header matching in hot paths introduces unnecessary memory allocations. **Action:** When substituting with `strings.HasPrefix`, explicitly handle case-insensitivity (e.g., via `strings.ToLower`) to maintain HTTP standard compliance while preventing heavy parsing allocations. + +## 2024-10-25 - Avoid strings.ToLower for case-insensitive prefix matching +**Learning:** Using `strings.ToLower` for case-insensitive prefix matching forces a memory allocation. In hot paths, this creates unnecessary garbage collection overhead. +**Action:** Use `len(str) >= len(prefix) && strings.EqualFold(str[:len(prefix)], prefix)` to achieve zero-allocation case-insensitive matching. diff --git a/cmd/sergeant-mcp-client/proxy.go b/cmd/sergeant-mcp-client/proxy.go index 5d69ba6..1c2a805 100644 --- a/cmd/sergeant-mcp-client/proxy.go +++ b/cmd/sergeant-mcp-client/proxy.go @@ -174,8 +174,9 @@ func (p *proxy) writeResponse(resp *response) { return } - // Performance optimization: Avoid mime.ParseMediaType allocation for exact header matching - isEventStream := strings.HasPrefix(strings.ToLower(resp.contentType), "text/event-stream") + // Performance optimization: Avoid mime.ParseMediaType and strings.ToLower allocations for prefix header matching + prefix := "text/event-stream" + isEventStream := len(resp.contentType) >= len(prefix) && strings.EqualFold(resp.contentType[:len(prefix)], prefix) p.outMu.Lock() defer p.outMu.Unlock() if isEventStream {