[PR #1] Bolt: optimize file walking and string allocations in auth scanning #1

Closed
opened 2026-03-07 21:35:43 +03:00 by kerem · 0 comments
Owner

Original Pull Request: https://github.com/Finesssee/ProxyPilot/pull/1

State: closed
Merged: No


💡 What: Optimized file system traversal and string comparisons in authentication file scanning.
🎯 Why: The original implementation used filepath.Walk (which incurs an extra lstat call per file) and strings.ToLower() (which allocates a new string) for every file visited.
📊 Impact: Reduces memory allocations and system calls during authentication file scanning. Benchmarks on CountAuthFiles showed an improvement from ~955µs to ~878µs per op (~8% faster) on a synthetic workload.
🔬 Measurement: Verified with go test -bench=. ./internal/util/.... Correctness verified with existing unit tests.


PR created automatically by Jules for task 13818555233719697699 started by @Finesssee

**Original Pull Request:** https://github.com/Finesssee/ProxyPilot/pull/1 **State:** closed **Merged:** No --- 💡 What: Optimized file system traversal and string comparisons in authentication file scanning. 🎯 Why: The original implementation used `filepath.Walk` (which incurs an extra `lstat` call per file) and `strings.ToLower()` (which allocates a new string) for every file visited. 📊 Impact: Reduces memory allocations and system calls during authentication file scanning. Benchmarks on `CountAuthFiles` showed an improvement from ~955µs to ~878µs per op (~8% faster) on a synthetic workload. 🔬 Measurement: Verified with `go test -bench=. ./internal/util/...`. Correctness verified with existing unit tests. --- *PR created automatically by Jules for task [13818555233719697699](https://jules.google.com/task/13818555233719697699) started by @Finesssee*
kerem 2026-03-07 21:35:43 +03:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ProxyPilot#1
No description provided.